Merge pull request #48 from DaoCloud/fix-relocate-container-images

fix update image registry and repository
DaoCloud · May 6, 2024 · c3c61dc · c3c61dc
2 parents 2b4af0c + 167e039
commit c3c61dc
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/internal/chart/chart.go b/internal/chart/chart.go
@@ -18,11 +18,16 @@ var (
 
 // updateValuesFile performs some substitutions to a given values.yaml file.
 func updateValuesFile(valuesFile string, targetRepo *api.Target) error {
-	if err := updateContainerImageRegistry(valuesFile, targetRepo); err != nil {
-		return errors.Annotatef(err, "error updating %s file", valuesFile)
+	if targetRepo.ContainerRegistry != "" {
+		if err := updateContainerImageRegistry(valuesFile, targetRepo); err != nil {
+			return errors.Annotatef(err, "error updating %s file", valuesFile)
+		}
 	}
-	if err := updateContainerImageRepository(valuesFile, targetRepo); err != nil {
-		return errors.Annotatef(err, "error updating %s file", valuesFile)
+
+	if targetRepo.ContainerRepository != "" {
+		if err := updateContainerImageRepository(valuesFile, targetRepo); err != nil {
+			return errors.Annotatef(err, "error updating %s file", valuesFile)
+		}
 	}
 	return nil
 }