This is an automated email from the ASF dual-hosted git repository.

Miretpl pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 4d54940ceef Fix PgBouncer NetworkPolicy using namespace labels of 
disabled triggerer KEDA (#74565)
4d54940ceef is described below

commit 4d54940ceef10367bfa4ab9bb35fd5367efbec9d
Author: Huy Bui <[email protected]>
AuthorDate: Fri Nov 6 14:12:40 2026 -0500

    Fix PgBouncer NetworkPolicy using namespace labels of disabled triggerer 
KEDA (#74565)
    
    Co-authored-by: Tiennguyentony <[email protected]>
    Co-authored-by: Przemysław Mirowski 
<[email protected]>
---
 chart/templates/pgbouncer/pgbouncer-networkpolicy.yaml | 2 +-
 chart/tests/helm_tests/other/test_pgbouncer.py         | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/chart/templates/pgbouncer/pgbouncer-networkpolicy.yaml 
b/chart/templates/pgbouncer/pgbouncer-networkpolicy.yaml
index f1f83b93d4e..c6d0ea2ab15 100644
--- a/chart/templates/pgbouncer/pgbouncer-networkpolicy.yaml
+++ b/chart/templates/pgbouncer/pgbouncer-networkpolicy.yaml
@@ -55,7 +55,7 @@ spec:
     - namespaceSelector:
        matchLabels: {{- toYaml .Values.workers.celery.keda.namespaceLabels | 
nindent 10 }}
       podSelector:
-      {{- else if and .Values.triggerer.enabled 
.Values.triggerer.keda.namespaceLabels }}
+      {{- else if and $triggererKedaEnabled 
.Values.triggerer.keda.namespaceLabels }}
     - namespaceSelector:
         matchLabels: {{- toYaml .Values.triggerer.keda.namespaceLabels | 
nindent 10 }}
       podSelector:
diff --git a/chart/tests/helm_tests/other/test_pgbouncer.py 
b/chart/tests/helm_tests/other/test_pgbouncer.py
index 93b0f70ebc4..29d10f7d895 100644
--- a/chart/tests/helm_tests/other/test_pgbouncer.py
+++ b/chart/tests/helm_tests/other/test_pgbouncer.py
@@ -944,6 +944,11 @@ class TestPgbouncerNetworkPolicy:
                 "workers": {"celery": {"keda": {"enabled": True}}},
                 "triggerer": {"keda": {"enabled": True}},
             },
+            # namespaceLabels of disabled triggerer.keda must not replace the 
workers.celery.keda peer
+            {
+                "workers": {"celery": {"keda": {"enabled": True}}},
+                "triggerer": {"keda": {"enabled": False, "namespaceLabels": 
{"env": "inactive"}}},
+            },
         ],
     )
     def test_pod_selectors_with_keda_without_namespace_labels(self, executor, 
values):

Reply via email to