fix(gcp-resources): Remove bucket_name as PK of gcp_storage_bucket_policies (#6550)

erezrokah · web-flow · commit fb6a2de387ec · 2023-01-10T10:29:43.000Z
#### Summary

`gcp_storage_bucket_policies` used the parent bucket's name as a PK. This means we could only get 1 policy per bucket in `overwrite*` modes.
Per a better field, this PR fixes it so we use `_cq_id`

&lt;!--
diff --git a/plugins/source/gcp/docs/tables/gcp_storage_bucket_policies.md b/plugins/source/gcp/docs/tables/gcp_storage_bucket_policies.md
@@ -2,7 +2,7 @@
 
 https://cloud.google.com/iam/docs/reference/rest/v1/Policy
 
-The primary key for this table is **bucket_name**.
+The primary key for this table is **_cq_id**.
 
 ## Relations
 
@@ -14,8 +14,8 @@ This table depends on [gcp_storage_buckets](gcp_storage_buckets.md).
 | ------------- | ------------- |
 |_cq_source_name|String|
 |_cq_sync_time|Timestamp|
-|_cq_id|UUID|
+|_cq_id (PK)|UUID|
 |_cq_parent_id|UUID|
 |project_id|String|
-|bucket_name (PK)|String|
+|bucket_name|String|
 |bindings|JSON|
diff --git a/plugins/source/gcp/resources/services/storage/bucket_policies.go b/plugins/source/gcp/resources/services/storage/bucket_policies.go
@@ -24,9 +24,6 @@ func BucketPolicies() *schema.Table {
 				Name:     "bucket_name",
 				Type:     schema.TypeString,
 				Resolver: schema.ParentColumnResolver("name"),
-				CreationOptions: schema.ColumnCreationOptions{
-					PrimaryKey: true,
-				},
 			},
 		},
 	}
