Make HaltOnCertificateError optional

... to simplify upgrades. This is the safe option: the value defaults to false, so existing setups keep their current behaviour. Signed-off-by: Stephen Kitt <[email protected]>
submariner-io · Oct 31, 2023 · cc230db · cc230db
1 parent 39fd429
commit cc230db
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/api/v1alpha1/submariner_types.go b/api/v1alpha1/submariner_types.go
@@ -172,7 +172,7 @@ type SubmarinerSpec struct {
 	// Halt on certificate error (so the pod gets restarted).
 	// +operator-sdk:csv:customresourcedefinitions:type=spec,displayName="Halt (and restart) on certificate error"
 	// +operator-sdk:csv:customresourcedefinitions:type=spec,xDescriptors={"urn:alm:descriptor:com.tectonic.ui:booleanSwitch"}
-	HaltOnCertificateError bool `json:"haltOnCertificateError"`
+	HaltOnCertificateError bool `json:"haltOnCertificateError,omitempty"`
 
 	// Name of the custom CoreDNS configmap to configure forwarding to Lighthouse.
 	// It should be in <namespace>/<name> format where <namespace> is optional and defaults to kube-system.

diff --git a/pkg/embeddedyamls/yamls.go b/pkg/embeddedyamls/yamls.go
@@ -300,7 +300,6 @@ spec:
             - clusterCIDR
             - clusterID
             - debug
-            - haltOnCertificateError
             - namespace
             - natEnabled
             - serviceCIDR