-
Notifications
You must be signed in to change notification settings - Fork 59
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
feat: support Airflow v2 #92
Conversation
1cff20b
to
1c5eb91
Compare
9765c8c
to
78eb54b
Compare
78eb54b
to
3f7c83a
Compare
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
CHANGELOG.md
Outdated
@@ -8,11 +8,17 @@ Update `entrypoint.sh` to support the new version of Airflow. | |||
|
|||
BREAKING CHANGES: | |||
|
|||
`ENABLE_AIRFLOW_RBAC_SETUP_AUTH` takes on a different meaning for Airflow V2, | |||
as only the RBAC UI is available in V2 while RBAC and non-RBAC UI is available | |||
in V1. It just means to creating an user for V2 with the given env var values. |
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.
Nit: Weird grammar.
Do you mean: "For V2, set to "true", to automatically create the user?
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.
Fixed, thanks for the catch.
CHANGELOG.md
Outdated
@@ -12,7 +12,7 @@ Update `entrypoint.sh` to support the new version of Airflow. | |||
|
|||
`ENABLE_AIRFLOW_RBAC_SETUP_AUTH` takes on a different meaning for Airflow V2, | |||
as only the RBAC UI is available in V2 while RBAC and non-RBAC UI is available | |||
in V1. It just means to creating an user for V2 with the given env var values. | |||
in V1. It just means to create an user for V2 with the given env vars. |
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.
Nit: a user
Add builds for Airflow v2.