test to not keep

Author: Tof1973

pkg/frontend/openshiftcluster_putorpatch.go

@@ -252,12 +252,6 @@ func (f *frontend) _putOrPatchOpenShiftCluster(ctx context.Context, log *logrus.
 	if isCreate {
 		putOrPatchClusterParameters.converter.ToInternal(ext, doc.OpenShiftCluster)
 
-		err = f.setDefaultAndValidateInstallVersion(ctx, doc.OpenShiftCluster)
-		if err != nil {
-			return nil, err
-		}
-		ext = putOrPatchClusterParameters.converter.ToExternal(doc.OpenShiftCluster)
-
 		err = f.ValidateNewCluster(ctx, subscription, doc.OpenShiftCluster, putOrPatchClusterParameters.staticValidator, ext, putOrPatchClusterParameters.path)
 		if err != nil {
 			return nil, err
@@ -284,6 +278,11 @@ func (f *frontend) _putOrPatchOpenShiftCluster(ctx context.Context, log *logrus.
 	}
 
 	if isCreate {
+		err = f.setDefaultAndValidateInstallVersion(ctx, doc.OpenShiftCluster)
+		if err != nil {
+			return nil, err
+		}
+
 		// on create, make the cluster resourcegroup ID lower case to work
 		// around LB/PLS bug
 		doc.OpenShiftCluster.Properties.ClusterProfile.ResourceGroupID = strings.ToLower(doc.OpenShiftCluster.Properties.ClusterProfile.ResourceGroupID)