Skip to content
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

fix(providers): Upgrade to kubernetes provider 8.3.4. #2108

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

mikaeld
Copy link
Contributor

@mikaeld mikaeld commented Oct 7, 2024

Description

Patch k8s provider to version 8.3.4 to fix the issue of executor open slots reaching negative number
image

Validation

image
new provider installed

image
running clean_gke_pods

Related Tickets & Documents

@mikaeld mikaeld self-assigned this Oct 7, 2024
@@ -1,2 +1,3 @@
bigeye-airflow==0.1.34
bigeye-sdk==0.4.81
apache-airflow-providers-cncf-kubernetes==8.3.4
Copy link
Contributor

Choose a reason for hiding this comment

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

Tested this locally and it doesn't look like it will affect any other packages/dependencies. 👍

Copy link
Contributor

@sean-rose sean-rose Oct 7, 2024

Choose a reason for hiding this comment

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

Also, looking forward to when we upgrade Airflow itself, this override will still be necessary for Airflow 2.9.3 since it defaults to apache-airflow-providers-cncf-kubernetes version 8.3.3, but it can be removed once we upgrade to Airflow 2.10.2 since that defaults to apache-airflow-providers-cncf-kubernetes version 8.4.1.

@four-keys-mozilla four-keys-mozilla bot temporarily deployed to dev October 7, 2024 19:42 Inactive
@mikaeld mikaeld merged commit 7989bc6 into main Oct 7, 2024
9 checks passed
@mikaeld mikaeld deleted the fix/k8s-provider branch October 7, 2024 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants