PRODENG-2719 MSR3 with no MSR2 panic (#499)

- fix MSR2 upgrade check panic when there is no MSR2 config, but there is MSR3 @note that there is no MR3 upgrade check, which can be done in a future effort Signed-off-by: James Nesbitt <[email protected]>
Mirantis · Aug 14, 2024 · fbaac31 · fbaac31
1 parent 5558daf
commit fbaac31
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/pkg/product/mke/phase/upgrade_check.go b/pkg/product/mke/phase/upgrade_check.go
@@ -55,7 +55,7 @@ func (p *UpgradeCheck) Run() (err error) {
 		}
 	}
 
-	if p.Config.Spec.ContainsMSR() {
+	if p.Config.Spec.ContainsMSR2() {
 		msrv, err := hub.LatestTag("mirantis", "dtr", strings.Contains(p.Config.Spec.MSR2.Version, "-"))
 		if err != nil {
 			log.Errorf("failed to check for MSR upgrade: %s", err.Error())
@@ -79,5 +79,7 @@ func (p *UpgradeCheck) Run() (err error) {
 		}
 	}
 
+	// @TODO MSR3 version upgrade check
+
 	return nil
 }