Skip to content

Conversation

@nhuantho
Copy link
Contributor

@nhuantho nhuantho commented Dec 23, 2025

closes: #59742

What:

  • defaultUser can be a part of apiServer; It should belong to createUserJob, so we don't need to add it to apiServer config.
  • In value.yaml, defaultUser has never been used in apiServer config, but it is presented at api-server parameters.
image

How

  • Remove defaultUser of apiServer in values.schema.json

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@nhuantho
Copy link
Contributor Author

#59742

@romsharon98 romsharon98 merged commit 846a334 into apache:main Dec 24, 2025
93 checks passed
@nhuantho nhuantho deleted the helm/update-values.shema branch December 26, 2025 04:31
amoghrajesh pushed a commit to astronomer/airflow that referenced this pull request Dec 29, 2025
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Jan 2, 2026
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
@jscheffl jscheffl added this to the Airflow Helm Chart 1.19.0 milestone Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[doc][helm] Redundant apiServer parameters

3 participants