Fall back to imageRegistry if tridentImageRegistry is not set

NetApp · Oct 18, 2022 · 22b0f6b · 22b0f6b
1 parent 2c009f0
commit 22b0f6b
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/cli/cmd/install.go b/cli/cmd/install.go
@@ -194,7 +194,7 @@ func init() {
 	installCmd.Flags().StringVar(&kubeletDir, "kubelet-dir", "/var/lib/kubelet",
 		"The host location of kubelet's internal state.")
 	installCmd.Flags().StringVar(&imageRegistry, "image-registry", "",
-		"The address/port of an internal image registry containing csi sidecar images.")
+		"The address/port of an internal image registry containing csi sidecar images. If trident-image-registry is not set this is also used for trident images")
 	installCmd.Flags().StringVar(&autosupportProxy, "autosupport-proxy", "",
 		"The address/port of a proxy for sending Autosupport Telemetry")
 	installCmd.Flags().StringVar(&autosupportCustomURL, "autosupport-custom-url", "", "Custom Autosupport endpoint")
@@ -299,6 +299,8 @@ func discoverInstallationEnvironment() error {
 		// Override registry only if using the default Trident image name and an alternate registry was supplied
 		if tridentImageRegistry != "" {
 			tridentImage = utils.ReplaceImageRegistry(tridentImage, tridentImageRegistry)
+		} else if imageRegistry != "" {
+			tridentImage = utils.ReplaceImageRegistry(tridentImage, imageRegistry)
 		}
 	}
 	log.Debugf("Trident image: %s", tridentImage)
@@ -308,6 +310,8 @@ func discoverInstallationEnvironment() error {
 
 		if tridentImageRegistry != "" {
 			autosupportImage = utils.ReplaceImageRegistry(autosupportImage, tridentImageRegistry)
+		} else if imageRegistry != "" {
+			autosupportImage = utils.ReplaceImageRegistry(autosupportImage, imageRegistry)
 		}
 	}
 	log.Debugf("Autosupport image: %s", autosupportImage)

diff --git a/helm/trident-operator/templates/_helpers.tpl b/helm/trident-operator/templates/_helpers.tpl
@@ -69,7 +69,7 @@ Trident operator image
 {{- if .Values.operatorImage }}
 {{- .Values.operatorImage }}
 {{- else }}
-{{- .Values.tridentImageRegistry | default "docker.io/netapp" }}/trident-operator:{{ .Values.operatorImageTag | default .Chart.AppVersion }}
+{{- .Values.tridentImageRegistry | default .Values.imageRegistry | default "docker.io/netapp" }}/trident-operator:{{ .Values.operatorImageTag | default .Chart.AppVersion }}
 {{- end }}
 {{- end }}
 
@@ -112,7 +112,7 @@ Trident AutoSupport image
 {{- if .Values.tridentAutosupportImage }}
 {{- .Values.tridentAutosupportImage }}
 {{- else }}
-{{- .Values.tridentImageRegistry | default "docker.io/netapp"}}/trident-autosupport:{{ .Values.tridentAutosupportImageTag | default .Chart.AppVersion | trunc 5}}
+{{- .Values.tridentImageRegistry | default .Values.imageRegistry | default "docker.io/netapp" }}/trident-autosupport:{{ .Values.tridentAutosupportImageTag | default .Chart.AppVersion | trunc 5}}
 {{- end }}
 {{- end }}
 
@@ -145,6 +145,6 @@ Trident image
 {{- if .Values.tridentImage }}
 {{- .Values.tridentImage }}
 {{- else }}
-{{- .Values.tridentImageRegistry | default "docker.io/netapp"}}/trident:{{ .Values.tridentImageTag | default .Chart.AppVersion }}
+{{- .Values.tridentImageRegistry | default .Values.imageRegistry | default "docker.io/netapp" }}/trident:{{ .Values.tridentImageTag | default .Chart.AppVersion }}
 {{- end }}
 {{- end }}
diff --git a/helm/trident-operator/values.yaml b/helm/trident-operator/values.yaml
@@ -33,7 +33,7 @@ affinity: {}
 
 
 
-# imageRegistry identifies the registry for official kubernetes csi sidecar images.  Leave empty to accept the default.
+# imageRegistry identifies the registry for official kubernetes csi sidecar images. Value is also used for tridentImageRegistry if this is not set. Leave empty to accept the default.
 imageRegistry: ""
 
 # imagePullPolicy sets the image pull policy for the trident-operator.
@@ -89,7 +89,7 @@ tridentLogFormat: "text"
 # tridentImage allows the complete override of the image for Trident.
 tridentImage: ""
 
-# tridentImageRegistry identifies the registry for the trident-operator, trident, and other images.  Leave empty to accept the default.
+# tridentImageRegistry identifies the registry for the trident-operator, trident, and other images. Leave empty to accept the default.
 tridentImageRegistry: ""
 
 # tridentImageTag allows overriding the tag of the image for Trident.