-
-
Notifications
You must be signed in to change notification settings - Fork 3
feat: Airflow 3.0.1 (experimental) #630
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First batch, will do some more testing.
tests/templates/kuttl/external-access/install-airflow-cluster.yaml.j2
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry if I missed it, but it looks like the supported-version.adoc is missing the 3.0.1 (experimental) addition
No, you were correct - thanks! 9dfe57e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if tests (OS) pass :)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description
Related to #618
Most changes in this PR are due to the fact that the API has changed and both old and new versions need to be available for the tests. Code changes can be summarised as:
N.B. this PR does not cover updates to OPA-Auth (covered separately).
Definition of Done Checklist
Author
CRD changes approvedCRD documentation for all fields, following the [style guide]Reviewer
Acceptance