You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "geocomm-shenningsgard (via GitHub)" <gi...@apache.org> on 2025/11/24 21:29:35 UTC

[I] podDisruptionBudget.config does not allow additional properties [airflow]

geocomm-shenningsgard opened a new issue, #58650:
URL: https://github.com/apache/airflow/issues/58650

   ### Official Helm Chart version
   
   1.18.0 (latest released)
   
   ### Apache Airflow version
   
   2.10.x
   
   ### Kubernetes Version
   
   1.32
   
   ### Helm Chart configuration
   
   ```yaml
   webserver:
       podDisruptionBudget:
         enabled: true
         config:
           unhealthyPodEvictionPolicy: AlwaysAllow
           maxUnavailable: 1
   ```
   (same for scheduler, and for api-server in Airflow 3)
   
   ### Docker Image customizations
   
   _No response_
   
   ### What happened
   
   It appears the schema for the values file does not match the intention of the template. The template at `templates/webserver/webserver-poddisruptionbudget.yaml` has the following, suggesting that the intention was to enable a user to supply arbitrary configuration for the pod disruption budget:
   
   ```yaml
   spec:
     selector:
       matchLabels:
         tier: airflow
         component: webserver
         release: {{ .Release.Name }}
     {{- toYaml .Values.webserver.podDisruptionBudget.config | nindent 2 }}
   ```
   
   Yet, the schema prevents this. I need to be able to set `unhealthyPodEvictionPolicy` so that the deployment responds appropriately in the case that Pods are unhealthy.
   
   ### What you think should happen instead
   
   The chart should either pass through any configuration supplied in (component).podDisruptionBudget.config, or if the desire is to restrict to valid configuration, allow all valid configuration values (such as `unhealthyPodEvictionPolicy`).
   
   ### How to reproduce
   
   Add something other than `maxUnavailable` or `minAvailable` (such as `unhealthyPodEvictionPolicy`) to `webserver.podDisruptionBudget.config` and deploy. You should get the following error:
   
   ```
   Error: UPGRADE FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
   airflow:
   - webserver.podDisruptionBudget.config: Additional property <your-property> is not allowed
   ```
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [x] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] podDisruptionBudget.config does not allow additional properties [airflow]

Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on issue #58650:
URL: https://github.com/apache/airflow/issues/58650#issuecomment-3572787930

   Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] podDisruptionBudget.config does not allow additional properties [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk closed issue #58650: podDisruptionBudget.config does not allow additional properties
URL: https://github.com/apache/airflow/issues/58650


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] podDisruptionBudget.config does not allow additional properties [airflow]

Posted by "geocomm-shenningsgard (via GitHub)" <gi...@apache.org>.
geocomm-shenningsgard commented on issue #58650:
URL: https://github.com/apache/airflow/issues/58650#issuecomment-3572861038

   > Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
   
   I would submit a PR, but I'd trust someone else more that's worked in the Helm chart before to know which path to take.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] podDisruptionBudget.config does not allow additional properties [airflow]

Posted by "Vishalup29 (via GitHub)" <gi...@apache.org>.
Vishalup29 commented on issue #58650:
URL: https://github.com/apache/airflow/issues/58650#issuecomment-3592296031

   @geocomm-shenningsgard I am a first time contributor and have opened a PR fixing this one: https://github.com/apache/airflow/pull/58864
   Pls review. i am open to suggestion and guidance in case the course of changes need to be in a different direction


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] podDisruptionBudget.config does not allow additional properties [airflow]

Posted by "vbottu (via GitHub)" <gi...@apache.org>.
vbottu commented on issue #58650:
URL: https://github.com/apache/airflow/issues/58650#issuecomment-3579342454

   I can give it a try  @geocomm-shenningsgard 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org