You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by ro...@apache.org on 2025/09/16 13:15:06 UTC

(airflow) branch main updated: helm: statsd deployment volume mount without subpath for live reloading (#54986)

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

rom 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 32871179895 helm: statsd deployment volume mount without subpath for live reloading (#54986)
32871179895 is described below

commit 3287117989559f17e3b71bb2d26985cfba12e6c6
Author: Maico Timmerman <Ma...@users.noreply.github.com>
AuthorDate: Tue Sep 16 15:14:58 2025 +0200

    helm: statsd deployment volume mount without subpath for live reloading (#54986)
    
    Co-authored-by: maico.timmerman@gmail.com <ma...@adyen.com>
---
 chart/templates/statsd/statsd-deployment.yaml    |  4 ++--
 helm-tests/tests/helm_tests/other/test_statsd.py | 12 ++++++------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/chart/templates/statsd/statsd-deployment.yaml b/chart/templates/statsd/statsd-deployment.yaml
index b97c98d2afd..1eab1fda8b8 100644
--- a/chart/templates/statsd/statsd-deployment.yaml
+++ b/chart/templates/statsd/statsd-deployment.yaml
@@ -130,8 +130,8 @@ spec:
             timeoutSeconds: 5
           volumeMounts:
             - name: config
-              mountPath: /etc/statsd-exporter/mappings.yml
-              subPath: mappings.yml
+              mountPath: /etc/statsd-exporter
+              readOnly: true
       volumes:
         - name: config
           configMap:
diff --git a/helm-tests/tests/helm_tests/other/test_statsd.py b/helm-tests/tests/helm_tests/other/test_statsd.py
index 5d80b75a5bd..dad5bb26304 100644
--- a/helm-tests/tests/helm_tests/other/test_statsd.py
+++ b/helm-tests/tests/helm_tests/other/test_statsd.py
@@ -38,8 +38,8 @@ class TestStatsd:
 
         assert {
             "name": "config",
-            "mountPath": "/etc/statsd-exporter/mappings.yml",
-            "subPath": "mappings.yml",
+            "mountPath": "/etc/statsd-exporter",
+            "readOnly": True,
         } in jmespath.search("spec.template.spec.containers[0].volumeMounts", docs[0])
 
         default_args = ["--statsd.mapping-config=/etc/statsd-exporter/mappings.yml"]
@@ -62,8 +62,8 @@ class TestStatsd:
 
         assert {
             "name": "config",
-            "mountPath": "/etc/statsd-exporter/mappings.yml",
-            "subPath": "mappings.yml",
+            "mountPath": "/etc/statsd-exporter",
+            "readOnly": True,
         } in jmespath.search("spec.template.spec.containers[0].volumeMounts", docs[0])
 
     def test_should_add_volume_and_volume_mount_when_exist_override_mappings(self):
@@ -83,8 +83,8 @@ class TestStatsd:
 
         assert {
             "name": "config",
-            "mountPath": "/etc/statsd-exporter/mappings.yml",
-            "subPath": "mappings.yml",
+            "mountPath": "/etc/statsd-exporter",
+            "readOnly": True,
         } in jmespath.search("spec.template.spec.containers[0].volumeMounts", docs[0])
 
     @pytest.mark.parametrize(