Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor operator reconcile function towards fetch/build/apply split #242

Open
Tracked by #355
vsupalov opened this issue Feb 22, 2023 · 0 comments
Open
Tracked by #355

Refactor operator reconcile function towards fetch/build/apply split #242

vsupalov opened this issue Feb 22, 2023 · 0 comments

Comments

@vsupalov
Copy link
Contributor

vsupalov commented Feb 22, 2023 •
edited
Loading

Tracked by in stackabletech/issues#355

Similar to stackabletech/druid-operator#391

This ticket is mostly a refactoring task. It differs from the druid operator, as airflow contains two reconcile loops.

  • Restructure each reconcile function into fetch/build/apply steps.
  • Write two smoke test for the resulting build functions.
  • Improve on the POC folder structure and naming scheme.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor/improve testability stackabletech/airflow-operator
1 participant
@vsupalov