Skip to content

Chart: warn on deprecated per-component securityContext values#62729

Merged
jedcunningham merged 1 commit intoapache:mainfrom
astronomer:securitycontext_deprecation_warnings
Mar 2, 2026
Merged

Chart: warn on deprecated per-component securityContext values#62729
jedcunningham merged 1 commit intoapache:mainfrom
astronomer:securitycontext_deprecation_warnings

Conversation

@jedcunningham
Copy link
Member

Add DEPRECATION WARNING notices in NOTES.txt for
scheduler.securityContext, webserver.securityContext, and statsd.securityContext. These fields were already marked deprecated in values.yaml (use securityContexts instead) but no warning was shown to users at install/upgrade time.

Scenario Result
scheduler.securityContext set ✅ warning fires
webserver.securityContext set ✅ warning fires
statsd.securityContext set ✅ warning fires
Default values (all empty) ✅ no warnings
All three set simultaneously ✅ all three warnings fire independently
Deprecated value still applied to pod spec ✅ backward compat intact (runAsUser: 12345 renders correctly)
New securityContexts path used ✅ no spurious warning

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: Cursor

Add DEPRECATION WARNING notices in NOTES.txt for
scheduler.securityContext, webserver.securityContext, and
statsd.securityContext. These fields were already marked deprecated in
values.yaml (use securityContexts instead) but no warning was shown to
users at install/upgrade time.
@jedcunningham jedcunningham merged commit fbf687c into apache:main Mar 2, 2026
96 checks passed
@jedcunningham jedcunningham deleted the securitycontext_deprecation_warnings branch March 2, 2026 22:24
dominikhei pushed a commit to dominikhei/airflow that referenced this pull request Mar 11, 2026
…e#62729)

Add DEPRECATION WARNING notices in NOTES.txt for
scheduler.securityContext, webserver.securityContext, and
statsd.securityContext. These fields were already marked deprecated in
values.yaml (use securityContexts instead) but no warning was shown to
users at install/upgrade time.
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.

2 participants