search

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
631 stars 473 forks source link

fix: update PodDisruptionBudget to use `policy/v1` #646

Closed ianlofs closed 1 year ago

ianlofs commented 1 year ago

What issues does your PR fix?

helm lint returns a warning when run against this chart. Specifically the following error is returned:

[WARNING] templates/scheduler/scheduler-pdb.yaml: policy/v1beta1 PodDisruptionBudget is deprecated in v1.21+, unavailable in v1.25+; use policy/v1 PodDisruptionBudget

What does your PR do?

This fixes the issue so that if the policy/v1 api exists then it is used in place of policy/v1beta1.

For all Pull Requests

ianlofs commented 1 year ago

Hi @thesuperzapper! Sorry this took me awhile but I finally got around to it. Thanks for the help! This is ready for your review now.

thesuperzapper commented 1 year ago

@ianlofs closing in favor of https://github.com/airflow-helm/charts/pull/685 as that one is ready to merge.