Skip to content

Conversation

@adwk67
Copy link
Member

@adwk67 adwk67 commented May 26, 2025

N.B. testing this requires stackabletech/airflow-operator#630 i.e. until this has been merged you will need to install the operator manually e.g. stackable op in airflow=0.0.0-pr630, as well as changing your local stacks-v2.yaml and demos-v2.yaml to point to the changed files.

sbernauer
sbernauer previously approved these changes May 27, 2025
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM otherwise

@sbernauer sbernauer moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering May 27, 2025
@adwk67 adwk67 added this pull request to the merge queue May 27, 2025
@sbernauer sbernauer removed this pull request from the merge queue due to a manual request May 27, 2025
@sbernauer
Copy link
Member

@adwk67 I removed it from the merge queue because if I understood correctly this requires stackabletech/airflow-operator#630, should we merge that first?

@adwk67
Copy link
Member Author

adwk67 commented May 27, 2025

@adwk67 I removed it from the merge queue because if I understood correctly this requires stackabletech/airflow-operator#630, should we merge that first?

Yes, you're right. I'm running some tests on Openshift and then I'll put that PR up for review.

@adwk67 adwk67 added this pull request to the merge queue Jun 3, 2025
Merged via the queue into main with commit 48ed7fd Jun 3, 2025
2 checks passed
@adwk67 adwk67 deleted the chore/bump-to-airflow-3 branch June 3, 2025 15:19
@adwk67 adwk67 moved this from Development: In Review to Development: Done in Stackable Engineering Jun 3, 2025
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants