fix: sync-users for airflow 2.7.0

airflow-helm / charts

The User-Community Airflow Helm Chart is the standard way to deploy Apache Airflow on Kubernetes with Helm. Originally created in 2017, it has since helped thousands of companies create production-ready deployments of Airflow on Kubernetes.

https://github.com/airflow-helm/charts/tree/main/charts/airflow

Apache License 2.0

665 stars 476 forks source link

fix: sync-users for airflow 2.7.0 #772

Closed SamWheating closed 1 year ago

SamWheating commented 1 year ago

What issues does your PR fix?

fixes https://github.com/apache/airflow/issues/33635 (which was created in the airflow repo)
fixes https://github.com/airflow-helm/charts/discussions/771

What does your PR do?

Updates this import path to be Airflow 2.7.0-friendly

Checklist

For all Pull Requests

[x] Commits are signed off
[x] Commits have semantic messages
[x] Documentation updated
[ ] Passes ct linting

For releasing ONLY

[ ] Chart.yaml version bumped
[ ] CHANGELOG.md updated

VictorAtPL commented 1 year ago

Thank you @SamWheating,

I deployed this modified Helm chart version and it's working fine for last 15 minutes. Will continue monitoring.

giancarloromeo commented 1 year ago

@SamWheating: I tried your patch and it works fine.

Amar1404 commented 1 year ago

Is there any alternative way untill this is implemented.

khmelevskiy commented 1 year ago

Hello! when is this issue going to be fixed? this prevents our team from updating the airflow :(

thesuperzapper commented 1 year ago

@SamWheating thanks for this!

I will test with Airflow 2.7 and cut a release with this fix shortly.

monircefalo commented 1 year ago

Hi @thesuperzapper , i am using Airflow 2.5. still this is happening. Wanted to use Airflow 2.7. Waiting for you fix. Thanks

khmelevskiy commented 1 year ago

hello, please tell me when approximately the merge is planned?

thesuperzapper commented 1 year ago

@khmelevskiy about 43 minuets ago!

It's in version 8.8.0 of the chart.