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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to