Skip to content

Conversation

@Miretpl
Copy link
Contributor

@Miretpl Miretpl commented Jun 4, 2025

In default values.yaml file there are missing values for workers.kerberosInitContainer. This PR adds missing fields.


^ 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.

@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Jun 4, 2025
@Miretpl Miretpl changed the title Add missing kerberosInitContainer in workers configuration Add missing workers.kerberosInitContainer configuration in values.yaml Jun 4, 2025
@eladkal eladkal force-pushed the add-missing-kerberos-settings branch from d0ca1b5 to 23a7230 Compare June 13, 2025 19:56
@eladkal eladkal requested a review from romsharon98 June 17, 2025 06:49
Miretpl and others added 2 commits June 17, 2025 17:37
Co-authored-by: rom sharon <33751805+romsharon98@users.noreply.github.com>
@eladkal eladkal force-pushed the add-missing-kerberos-settings branch from 331e4ad to ac7dfff Compare June 17, 2025 14:37
@eladkal eladkal requested a review from romsharon98 June 17, 2025 14:37
@eladkal eladkal merged commit 0336e3d into apache:main Jun 17, 2025
66 checks passed
RoyLee1224 pushed a commit to RoyLee1224/airflow that referenced this pull request Jun 21, 2025
apache#51405)

* Add missing kerberosInitContainer in workers

* Update chart/values.yaml

Co-authored-by: rom sharon <33751805+romsharon98@users.noreply.github.com>

---------

Co-authored-by: Elad Kalif <45845474+eladkal@users.noreply.github.com>
Co-authored-by: rom sharon <33751805+romsharon98@users.noreply.github.com>
@Miretpl Miretpl deleted the add-missing-kerberos-settings branch June 26, 2025 20:42
@Miretpl
Copy link
Contributor Author

Miretpl commented Jun 26, 2025

This change was part of the work on #51460.

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.

4 participants