operator: fix upgradeImages

hj-johannes-lee · hj-johannes-lee · commit acc52d9c61ac · 2024-10-04T16:51:33.000+03:00
For sha256 image tags, it has not been working properly.
Fix it to make it work both semantic and sha256.

Signed-off-by: Hyeongju Johannes Lee &lt;hyeongju.lee@intel.com&gt;
diff --git a/pkg/controllers/reconciler.go b/pkg/controllers/reconciler.go
@@ -160,7 +160,7 @@ func UpgradeImages(ctx context.Context, image *string, initimage *string) (upgra
 		}
 
 		if parts := strings.SplitN(*s, ":", 2); len(parts) == 2 && len(parts[0]) > 0 {
-			name, version := parts[0], parts[1]
+			name, version := strings.Split(parts[0], "@")[0], parts[1]
 
 			envVarValue := os.Getenv(strings.ReplaceAll(strings.ToUpper(filepath.Base(name)), "-", "_") + "_SHA")
 

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ func UpgradeImages(ctx context.Context, image string, initimage string) (upgra`
`160`	`160`	`}`
`161`	`161`
`162`	`162`	`if parts := strings.SplitN(*s, ":", 2); len(parts) == 2 && len(parts[0]) > 0 {`
`163`		`- name, version := parts[0], parts[1]`
	`163`	`+ name, version := strings.Split(parts[0], "@")[0], parts[1]`
`164`	`164`
`165`	`165`	`envVarValue := os.Getenv(strings.ReplaceAll(strings.ToUpper(filepath.Base(name)), "-", "_") + "_SHA")`
`166`	`166`