Remove send_empty_value for suspended and disableRollbackIfRolloutPending fields (#14982)

Author: zli82016

mmv1/products/clouddeploy/Automation.yaml

@@ -104,7 +104,6 @@ properties:
   - name: 'suspended'
     type: Boolean
     description: "Optional. When Suspended, automation is deactivated from execution."
-    send_empty_value: true
   - name: 'serviceAccount'
     type: String
     description: "Required. Email address of the user-managed IAM service account that creates Cloud Deploy release and rollout resources."
@@ -242,7 +241,6 @@ properties:
                         description: "Optional. The starting phase ID for the Rollout. If unspecified, the Rollout will start in the stable phase."
                       - name: 'disableRollbackIfRolloutPending'
                         type: Boolean
-                        send_empty_value: true
                         description: "Optional. If pending rollout exists on the target, the rollback operation will be aborted."
         - name: 'timedPromoteReleaseRule'
           type: NestedObject

mmv1/products/clouddeploy/DeployPolicy.yaml

@@ -94,7 +94,6 @@ properties:
   - name: "suspended"
     type: Boolean
     description: "When suspended, the policy will not prevent actions from occurring, even if the action violates the policy."
-    send_empty_value: true
   - name: "selectors"
     type: Array
     description: "Selected resources to which the policy will be applied. At least one selector is required. If one selector matches the resource the policy applies. For example, if there are two selectors and the action being attempted matches one of them, the policy will apply to that action."

mmv1/third_party/terraform/services/clouddeploy/resource_clouddeploy_automation_test.go

@@ -40,6 +40,15 @@ func TestAccClouddeployAutomation_update(t *testing.T) {
 				ImportStateVerify:       true,
 				ImportStateVerifyIgnore: []string{"location", "delivery_pipeline", "annotations", "labels", "terraform_labels"},
 			},
+			{
+				Config: testAccClouddeployAutomation_basic(context),
+			},
+			{
+				ResourceName:            "google_clouddeploy_automation.automation",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"location", "delivery_pipeline", "annotations", "labels", "terraform_labels"},
+			},
 		},
 	})
 }

mmv1/third_party/terraform/services/clouddeploy/resource_clouddeploy_deploy_policy_test.go

@@ -38,6 +38,15 @@ func TestAccClouddeployDeployPolicy_update(t *testing.T) {
 				ImportStateVerify:       true,
 				ImportStateVerifyIgnore: []string{"location", "annotations", "labels", "terraform_labels"},
 			},
+			{
+				Config: testAccClouddeployDeployPolicy_basic(context),
+			},
+			{
+				ResourceName:            "google_clouddeploy_deploy_policy.deploy_policy",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"location", "annotations", "labels", "terraform_labels"},
+			},
 		},
 	})
 }
@@ -85,6 +94,7 @@ resource "google_clouddeploy_deploy_policy" "deploy_policy" {
       id = "tf-test-cd-pipeline%{random_suffix}"
     }
   }
+  suspended = true
   rules {
     rollout_restriction {
       id = "rule"