Adding the support for skip_k8s_upgrade and unit test addition

Author: manimaran89

spectrocloud/resource_cluster_aws.go

@@ -529,7 +529,7 @@ func flattenMachinePoolConfigsAws(machinePools []*models.V1AwsMachinePoolConfig)
 			oi["max_price"] = machinePool.SpotMarketOptions.MaxPrice
 		}
 		// Flatten skip_k8s_upgrade (worker pools only); default "disabled" for backward compat when API omits field
-		if machinePool.SkipK8sUpgrade != nil {
+		if machinePool.SkipK8sUpgrade != nil && *machinePool.SkipK8sUpgrade != "" {
 			oi["skip_k8s_upgrade"] = *machinePool.SkipK8sUpgrade
 		}
 		oi["disk_size_gb"] = int(machinePool.RootDeviceSize)

spectrocloud/resource_cluster_aws_expand_test.go

@@ -91,8 +91,8 @@ func TestToMachinePoolAws(t *testing.T) {
 					IsControlPlane:        false,
 					Labels:                []string{"worker"},
 					NodeRepaveInterval:    10,
-					SkipK8sUpgrade:        types.Ptr("disabled"),
 					UpdateStrategy:        &models.V1UpdateStrategy{Type: "RollingUpdateScaleOut"},
+					SkipK8sUpgrade:        types.Ptr("disabled"),
 					AdditionalLabels:      map[string]string{},
 					AdditionalAnnotations: map[string]string{},
 				},

spectrocloud/resource_cluster_aws_flatten_test.go

@@ -60,7 +60,6 @@ func TestFlattenMachinePoolConfigsAws(t *testing.T) {
 					"disk_size_gb":               8,
 					"az_subnets":                 map[string]string{"us-west-2d": "subnet-87654321"},
 					"update_strategy":            "RollingUpdateScaleOut",
-					"skip_k8s_upgrade":           "disabled",
 					"additional_security_groups": []string{"sg-1234567890"},
 				},
 			},