You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "pranayyelugam (via GitHub)" <gi...@apache.org> on 2026/08/22 18:22:06 UTC

[PR] [chart/v1-2x-test] Helm: Scope KEDA_DB_CONN to worker and triggerer pods (#70213) [airflow]

pranayyelugam opened a new pull request, #71982:
URL: https://github.com/apache/airflow/pull/71982

   Manual backport of #70213 to `chart/v1-2x-test` (the automated backport failed on conflicts).
   
   Adapted to this branch: the KEDA logic now lives in a dedicated `keda_airflow_environment` helper that stays fallback-aware for the deprecated `workers.keda` section (removed on `main`), and call sites use this branch's current `standard_airflow_environment` signature.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4.8)
   
   Generated-by: Claude Code (Opus 4.8) 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] Helm: Scope KEDA_DB_CONN to worker and triggerer pods (#70213) [airflow]

Posted by "Miretpl (via GitHub)" <gi...@apache.org>.
Miretpl merged PR #71982:
URL: https://github.com/apache/airflow/pull/71982


-- 
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] Helm: Scope KEDA_DB_CONN to worker and triggerer pods (#70213) [airflow]

Posted by "Miretpl (via GitHub)" <gi...@apache.org>.
Miretpl commented on code in PR #71982:
URL: https://github.com/apache/airflow/pull/71982#discussion_r3837005407


##########
chart/templates/_helpers.yaml:
##########
@@ -175,6 +159,27 @@ If release name contains chart name it will be used as a full name.
   {{- end }}
 {{- end }}
 
+{{/* KEDA scaler database connection for the worker and triggerer autoscalers */}}

Review Comment:
   Could you do the same logic simplification as was done in the main PR?



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