Make cluster.proxy settings empty string by default (#1026)

giantswarm · Sep 26, 2023 · 979672d · 979672d
1 parent b5aa48c
commit 979672d
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 12 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project's packages adheres to [Semantic Versioning](http://semver.org/s
 
 ## [Unreleased]
 
+### Fixed
+
+- Fixed default values for `.proxy` and `.cluster.proxy` values and updated Helm values schema accordingly.
+
 ## [2.35.1] - 2023-09-21
 
 ### Changed

diff --git a/helm/chart-operator/values.schema.json b/helm/chart-operator/values.schema.json
@@ -41,13 +41,13 @@
                     "type": "object",
                     "properties": {
                         "http": {
-                            "type": "null"
+                            "type": "string"
                         },
                         "https": {
-                            "type": "null"
+                            "type": "string"
                         },
                         "noProxy": {
-                            "type": "null"
+                            "type": "string"
                         }
                     }
                 }
@@ -210,13 +210,13 @@
             "type": "object",
             "properties": {
                 "http": {
-                    "type": "null"
+                    "type": "string"
                 },
                 "https": {
-                    "type": "null"
+                    "type": "string"
                 },
                 "noProxy": {
-                    "type": "null"
+                    "type": "string"
                 }
             }
         },

diff --git a/helm/chart-operator/values.yaml b/helm/chart-operator/values.yaml
@@ -9,9 +9,9 @@ cluster:
   # is getting overwritten by the top level proxy if set
   # These values are generated via cluster-apps-operator
   proxy:
-    noProxy:
-    http:
-    https:
+    noProxy: ""
+    http: ""
+    https: ""
 
 clusterDNSIP: 172.31.0.10
 
@@ -60,9 +60,9 @@ project:
 
 # set the HTTP_PROXY, HTTPS_PROXY and NO_PROXY variable
 proxy:
-  noProxy:
-  http:
-  https:
+  noProxy: ""
+  http: ""
+  https: ""
 
 # Resource names are truncated to 47 characters. Kubernetes allows 63 characters
 # limit for resource names. When pods for deployments are created they have