chang(render): overriden vars split to not break values with = signal

jusbrasil · Oct 16, 2020 · 7c1381b · 7c1381b
1 parent a888a6c
commit 7c1381b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/kuberender/render.py b/kuberender/render.py
@@ -35,7 +35,9 @@ def render(filename):
 
 def parse_overriden_vars(overriden_vars):
     def parse_statement(override_statement):
-        key, value = override_statement.split('=')
+        equal_idx = override_statement.index('=')
+        key = override_statement[:equal_idx]
+        value = override_statement[equal_idx + 1:]
         obj = {}
         dpath.util.new(obj, key, value, separator='.')
         return obj

diff --git a/tests/resources/test-basic-manifest/deployment.yaml b/tests/resources/test-basic-manifest/deployment.yaml
@@ -21,6 +21,7 @@ spec:
       containers:
       - name: {{NAME}}
         image: "{{image.repository}}:{{image.tag}}"
+        test: "{{ other.test }}"
         imagePullPolicy: {{ image.pullPolicy }}
         resources:
           limits:

diff --git a/tests/test_kuberender.py b/tests/test_kuberender.py
@@ -123,11 +123,12 @@ def test_overriding_vars(self):
 
         manifest = self._load_template_manifest(self._partial_render()(
             context_files=['base.yaml'],
-            overriden_vars=['image.tag=3.0.7', 'image.repository=bitnami/redis']
+            overriden_vars=['image.tag=3.0.7', 'image.repository=bitnami/redis', 'other.key=test=']
         ))
 
         # Image and tag were overriden but pullPolicy (which is a sibling) wasn't modified
         assert 'bitnami/redis:3.0.7' == get_value(manifest, 'spec/template/spec/containers/0/image')
+        assert 'test=' == get_value(manifest, 'spec/template/spec/containers/0/test')
         assert 'Always' == get_value(manifest, 'spec/template/spec/containers/0/imagePullPolicy')