allow matches_storage_class empty array (#4809) (#3282)

modular-magician · web-flow · commit 28181b29c038 · 2021-05-24T10:37:56.000-07:00
Signed-off-by: Modular Magician &lt;magic-modules@google.com&gt;
diff --git a/.changelog/4809.txt b/.changelog/4809.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+storage: fixed error when `matches_storage_class` is set empty on `google_storage_bucket`
+```
diff --git a/google-beta/resource_storage_bucket.go b/google-beta/resource_storage_bucket.go
@@ -194,7 +194,6 @@ func resourceStorageBucket() *schema.Resource {
 									"matches_storage_class": {
 										Type:        schema.TypeList,
 										Optional:    true,
-										MinItems:    1,
 										Elem:        &schema.Schema{Type: schema.TypeString},
 										Description: `Storage Class of objects to satisfy this condition. Supported values include: MULTI_REGIONAL, REGIONAL, NEARLINE, COLDLINE, ARCHIVE, STANDARD, DURABLE_REDUCED_AVAILABILITY.`,
 									},
diff --git a/google-beta/resource_storage_bucket_test.go b/google-beta/resource_storage_bucket_test.go
@@ -1332,6 +1332,7 @@ resource "google_storage_bucket" "bucket" {
       type = "Delete"
     }
     condition {
+	  matches_storage_class = []
       age = 10
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	+```release-note:bug
	`2`	+storage: fixed error when `matches_storage_class` is set empty on `google_storage_bucket`
	`3`	+```
Original file line number	Diff line number	Diff line change
`@@ -1332,6 +1332,7 @@ resource "google_storage_bucket" "bucket" {`
`1332`	`1332`	`type = "Delete"`
`1333`	`1333`	`}`
`1334`	`1334`	`condition {`
	`1335`	`+ matches_storage_class = []`
`1335`	`1336`	`age = 10`
`1336`	`1337`	`}`
`1337`	`1338`	`}`