-
Notifications
You must be signed in to change notification settings - Fork 476
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
airflow 2.3.X breaks sync-users script #607
Comments
@asosnovsky-sumologic this issue is caused by airflow 2.3.X moving an import (which broke our We have fixed this issue in PR #592, but that PR is not part of a release yet. I plan to include the fix in chart version Follow issue #572 (comment) to get notified about other 2.3.X fixes. |
@thesuperzapper thank you I have downgraded for now, should have checked your compatibility table better before using 2.3.x |
@asosnovsky-sumologic I really should have pushed 8.6.1 out already, I am sorry for taking so long (it's been ready for ages, I have just been hesitant to cut, as I really don't want to "encourage" upgrading yet). However, I will make it my mission to cut 8.6.1 in the next 12 hours. Airflow 2.3.X really should have worked without needing chart changes (if airflow had not made unnecessarily breaking changes), and as of 2.3.2 I have actually worked with airflow to fix most of the issues upstream (but the But in general, for production usage of airflow, I would recommend waiting a bit for any new minor release to stabilize before using it in anything critical (Airflow has a tendency to make breaking changes with every release, even patch ones). |
@asosnovsky-sumologic the long-belated |
Hello all, |
Checks
User-Community Airflow Helm Chart
.Chart Version
8.6.0
Kubernetes Version
Helm Version
Description
We are deploying a new helm chart via argoCD, and the
sync-users
containers appear to failing due to an issue in the python code.Relevant Logs
Custom Helm Values
The text was updated successfully, but these errors were encountered: