AaryanCdry opened a new pull request, #70913: URL: https://github.com/apache/airflow/pull/70913
Closes #70843 ## Summary Extract the `AIRFLOW__API_AUTH__JWT_SECRET` environment variable handling out of `standard_airflow_environment` into a new dedicated `jwt_secret_environment` helper, removing the `IncludeJwtSecret` flag that was threaded through merge dicts at every call site. ## Change - **chart/templates/_helpers.yaml**: Remove the JWT secret block (and its `IncludeJwtSecret` conditional) from `standard_airflow_environment`; add a new `jwt_secret_environment` helper. - **chart/templates/api-server/api-server-deployment.yaml** and **chart/templates/scheduler/scheduler-deployment.yaml**: Include `jwt_secret_environment` explicitly (these components need the JWT secret). - **chart/templates/dag-processor**, **triggerer**, **workers**: Call sites simplified back to plain `standard_airflow_environment` (no merge dict). ## Verification Behavior unchanged, verified via `helm template` renders: - JWT secret env present in: api-server, scheduler ✅ - JWT secret env absent in: dag-processor, workers, triggerer ✅ - Scheduler env vars match `test_have_all_variables` expectations exactly Pure readability refactor - no functional change. -- 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]
