You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "rohan9446 (via GitHub)" <gi...@apache.org> on 2026/08/04 23:00:08 UTC
[PR] [chart/v1-2x-test] Separate JWT secret env var from the standard Airflow environment helper (#70896) [airflow]
rohan9446 opened a new pull request, #71129:
URL: https://github.com/apache/airflow/pull/71129
Manual backport of #70896 to `chart/v1-2x-test`, as requested by @Miretpl.
The automated backport conflicted because this branch still supports Airflow 2 alongside 3:
- the `AIRFLOW__API_AUTH__JWT_SECRET` block is additionally gated on `semverCompare ">=3.0.0" .Values.airflowVersion` — that gate now lives inside the new `jwt_secret_environment` helper, so Airflow 2 rendering is unchanged
- there is no `keda_airflow_environment` helper on this branch, so the new define sits before `custom_airflow_environment` instead (the cherry-picked commit message references it — that's carried over from main)
- the helm tests live under `helm-tests/tests/` rather than `chart/tests/`
Otherwise identical to main: `IncludeJwtSecret` is gone from `standard_airflow_environment` and all ten call sites, and the secret is included explicitly in the API server and scheduler containers only.
Tested locally with helm v3.21.3: `pytest helm-tests/tests/helm_tests/` → 3454 passed.
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes (please specify the tool below)
Generated-by: Claude (Cowork) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [chart/v1-2x-test] Separate JWT secret env var from the standard Airflow environment helper (#70896) [airflow]
Posted by "Miretpl (via GitHub)" <gi...@apache.org>.
Miretpl merged PR #71129:
URL: https://github.com/apache/airflow/pull/71129
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [chart/v1-2x-test] Separate JWT secret env var from the standard Airflow environment helper (#70896) [airflow]
Posted by "Miretpl (via GitHub)" <gi...@apache.org>.
Miretpl commented on PR #71129:
URL: https://github.com/apache/airflow/pull/71129#issuecomment-5259327690
Failures in the test are not related.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org