Upgrade dcl

mmv1/third_party/terraform/go.mod.erb

@@ -5,7 +5,7 @@ go 1.18
 
 require (
 	cloud.google.com/go/bigtable v1.17.0
-	github.com/GoogleCloudPlatform/declarative-resource-client-library v1.28.0
+	github.com/GoogleCloudPlatform/declarative-resource-client-library v1.30.0
 	github.com/apparentlymart/go-cidr v1.1.0
 	github.com/client9/misspell v0.3.4
 	github.com/davecgh/go-spew v1.1.1

mmv1/third_party/terraform/go.sum

@@ -1289,3 +1289,9 @@ rsc.io/binaryregexp v0.2.0/go.mod h1:qTv7/COck+e2FymRvadv62gMdZztPaShugOCi3I+8D8
 rsc.io/quote/v3 v3.1.0/go.mod h1:yEA65RcK8LyAZtP9Kv3t0HmxON59tX3rD+tICJqUlj0=
 rsc.io/sampler v1.3.0/go.mod h1:T1hPZKmBbMNahiBKFy5HrXp6adAjACjK9JXDnKaTXpA=
 sigs.k8s.io/yaml v1.1.0/go.mod h1:UJmg0vDUVViEyp3mgSv9WPwZCDxu4rQW1olrI1uml+o=
+github.com/GoogleCloudPlatform/declarative-resource-client-library v1.29.0 h1:RmGDbvUWkRLYyWngSad04GR4LAnqflbXvzI1iBO2+rU=
+github.com/GoogleCloudPlatform/declarative-resource-client-library v1.29.0/go.mod h1:pL2Qt5HT+x6xrTd806oMiM3awW6kNIXB/iiuClz6m6k=
+github.com/GoogleCloudPlatform/declarative-resource-client-library v1.29.1-0.20221217025133-b2d522d3684b h1:RgZjFUQKRCJV61R5emEFGhDOX84D0BPZG+p/E0hilSk=
+github.com/GoogleCloudPlatform/declarative-resource-client-library v1.29.1-0.20221217025133-b2d522d3684b/go.mod h1:pL2Qt5HT+x6xrTd806oMiM3awW6kNIXB/iiuClz6m6k=
+github.com/GoogleCloudPlatform/declarative-resource-client-library v1.30.0 h1:1DMSUzQO2osKXy03/xZmr5n8bMlW3RyMhHR2JpkkVzY=
+github.com/GoogleCloudPlatform/declarative-resource-client-library v1.30.0/go.mod h1:pL2Qt5HT+x6xrTd806oMiM3awW6kNIXB/iiuClz6m6k=

mmv1/third_party/terraform/tests/resource_os_config_os_policy_assignment_test.go.erb

@@ -0,0 +1,147 @@
+<% autogen_exception -%>
+
+package google
+
+import (
+	"context"
+	"fmt"
+	dcl "github.com/GoogleCloudPlatform/declarative-resource-client-library/dcl"
+	osconfig "github.com/GoogleCloudPlatform/declarative-resource-client-library/services/google/osconfig<%= dcl_version(version) -%>"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
+	"strings"
+	"testing"
+)
+
+func TestAccOsConfigOsPolicyAssignment_PercentOsPolicyAssignment(t *testing.T) {
+	// Tests fail due to long rollout times
+	skipIfVcr(t)
+	t.Parallel()
+
+	context := map[string]interface{}{
+		"project_name":  getTestProjectFromEnv(),
+		"zone":          getTestZoneFromEnv(),
+		"random_suffix": randString(t, 10),
+	}
+
+	vcrTest(t, resource.TestCase{
+		PreCheck:     func() { testAccPreCheck(t) },
+		Providers:    testAccProviders,
+		CheckDestroy: testAccCheckOsConfigOsPolicyAssignmentDestroyProducer(t),
+		Steps: []resource.TestStep{
+			{
+				Config: testAccOsConfigOsPolicyAssignment_PercentOsPolicyAssignment(context),
+			},
+			{
+				ResourceName:            "google_os_config_os_policy_assignment.primary",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"rollout.0.min_wait_duration"},
+			},
+		},
+	})
+}
+
+func testAccOsConfigOsPolicyAssignment_PercentOsPolicyAssignment(context map[string]interface{}) string {
+	return Nprintf(`
+resource "google_os_config_os_policy_assignment" "primary" {
+  instance_filter {
+    all = false
+    exclusion_labels {
+      labels = {
+        label-two = "value-two"
+      }
+    }
+    inclusion_labels {
+      labels = {
+        label-one = "value-one"
+      }
+    }
+    inventories {
+      os_short_name = "centos"
+      os_version    = "8.*"
+    }
+  }
+
+  location = "europe-west1-a"
+  name     = "tf-test-assignment%{random_suffix}"
+
+  os_policies {
+    id   = "policy"
+    mode = "VALIDATION"
+
+    resource_groups {
+      resources {
+        id = "apt-to-yum"
+
+        repository {
+          apt {
+            archive_type = "DEB"
+            components   = ["doc"]
+            distribution = "debian"
+            uri          = "https://atl.mirrors.clouvider.net/debian"
+            gpg_key      = ".gnupg/pubring.kbx"
+          }
+        }
+      }
+    }
+  }
+
+  rollout {
+    disruption_budget {
+      percent = 100
+    }
+
+    min_wait_duration = "3s"
+  }
+
+  description = "A test os policy assignment"
+  project     = "%{project_name}"
+}
+
+
+`, context)
+}
+
+func testAccCheckOsConfigOsPolicyAssignmentDestroyProducer(t *testing.T) func(s *terraform.State) error {
+	return func(s *terraform.State) error {
+		for name, rs := range s.RootModule().Resources {
+			if rs.Type != "rs.google_os_config_os_policy_assignment" {
+				continue
+			}
+			if strings.HasPrefix(name, "data.") {
+				continue
+			}
+
+			config := googleProviderConfig(t)
+
+			billingProject := ""
+			if config.BillingProject != "" {
+				billingProject = config.BillingProject
+			}
+
+			obj := &osconfig.OSPolicyAssignment{
+				Location:           dcl.String(rs.Primary.Attributes["location"]),
+				Name:               dcl.String(rs.Primary.Attributes["name"]),
+				Description:        dcl.String(rs.Primary.Attributes["description"]),
+				Project:            dcl.StringOrNil(rs.Primary.Attributes["project"]),
+				SkipAwaitRollout:   dcl.Bool(rs.Primary.Attributes["skip_await_rollout"] == "true"),
+				Baseline:           dcl.Bool(rs.Primary.Attributes["baseline"] == "true"),
+				Deleted:            dcl.Bool(rs.Primary.Attributes["deleted"] == "true"),
+				Etag:               dcl.StringOrNil(rs.Primary.Attributes["etag"]),
+				Reconciling:        dcl.Bool(rs.Primary.Attributes["reconciling"] == "true"),
+				RevisionCreateTime: dcl.StringOrNil(rs.Primary.Attributes["revision_create_time"]),
+				RevisionId:         dcl.StringOrNil(rs.Primary.Attributes["revision_id"]),
+				RolloutState:       osconfig.OSPolicyAssignmentRolloutStateEnumRef(rs.Primary.Attributes["rollout_state"]),
+				Uid:                dcl.StringOrNil(rs.Primary.Attributes["uid"]),
+			}
+
+			client := NewDCLOsConfigClient(config, config.userAgent, billingProject, 0)
+			_, err := client.GetOSPolicyAssignment(context.Background(), obj)
+			if err == nil {
+				return fmt.Errorf("google_os_config_os_policy_assignment still exists %v", obj)
+			}
+		}
+		return nil
+	}
+}