Skip to content

Fix triggerer KEDA database connection rendering#67538

Merged
jscheffl merged 1 commit into
apache:mainfrom
henry3260:triggerer-keda-db-conn
May 26, 2026
Merged

Fix triggerer KEDA database connection rendering#67538
jscheffl merged 1 commit into
apache:mainfrom
henry3260:triggerer-keda-db-conn

Conversation

@henry3260

Copy link
Copy Markdown
Contributor

Why

Triggerer KEDA can require KEDA_DB_CONN when using a MySQL metadata database, or when PgBouncer is
enabled but triggerer.keda.usePgbouncer=false.

Before this change, the chart only rendered KEDA_DB_CONN and the metadata Secret's kedaConnection
based on worker KEDA conditions. As a result, deployments with triggerer KEDA enabled but worker KEDA
disabled could render a ScaledObject that references KEDA_DB_CONN, while the triggerer container and
metadata Secret did not provide it.

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

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg Bot added the area:helm-chart Airflow Helm Chart label May 26, 2026
@henry3260 henry3260 force-pushed the triggerer-keda-db-conn branch from d96705b to a7e8c5b Compare May 26, 2026 12:00
@henry3260 henry3260 force-pushed the triggerer-keda-db-conn branch from a7e8c5b to f19eaf3 Compare May 26, 2026 15:14
@jscheffl jscheffl added the backport-to-chart/v1-2x-test Automatic backport to chart 1.2x maintenance branch label May 26, 2026
@jscheffl jscheffl added this to the Airflow Helm Chart 1.22.0 milestone May 26, 2026
@jscheffl jscheffl merged commit 0cf0d63 into apache:main May 26, 2026
102 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: chart/v1-2x-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
chart/v1-2x-test PR Link

github-actions Bot pushed a commit that referenced this pull request May 26, 2026
…67538)

(cherry picked from commit 0cf0d63)

Co-authored-by: Henry Chen <henryhenry0512@gmail.com>
jscheffl pushed a commit that referenced this pull request May 26, 2026
…67538) (#67573)

(cherry picked from commit 0cf0d63)

Co-authored-by: Henry Chen <henryhenry0512@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: chart/v1-2x-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
chart/v1-2x-test PR Link

@henry3260 henry3260 deleted the triggerer-keda-db-conn branch May 26, 2026 23:55
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 backport-to-chart/v1-2x-test Automatic backport to chart 1.2x maintenance branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants