Vishalup29 opened a new pull request, #58864:
URL: https://github.com/apache/airflow/pull/58864
Summary
The Helm chart templates already allow arbitrary PodDisruptionBudget
configuration via:
{{ toYaml .Values.<component>.podDisruptionBudget.config | nindent 2 }}
This means users should be able to set any valid PDB fields, including newer
ones such as unhealthyPodEvictionPolicy.
However, the values.schema.json file currently restricts these config blocks
with:
"additionalProperties": false
This causes Helm validation to fail when users provide valid PDB fields not
explicitly listed in the schema.
What this PR changes
This PR updates the schema for all components’ PodDisruptionBudget sections:
podDisruptionBudget.additionalProperties
podDisruptionBudget.config.additionalProperties
These are now set to true, allowing arbitrary valid fields to be passed
through exactly as the template already supports.
Why this is needed
Without this change, using any non-standard (but fully valid) PDB field
results in:
values don't meet the specifications of the schema(s):
<component>.podDisruptionBudget.config: Additional property <x> is not
allowed
This prevents users from adopting newer Kubernetes PDB fields and
contradicts the behavior of the chart templates.
Scope of the change
✔ Only values.schema.json is updated.
✔ No changes to templates or chart behavior.
✔ No functional changes — the schema now reflects the existing intended
behavior.
Issue
Fixes: #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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
