fix(gcp-resources): Change gcp_appengine_firewall_ingress_rules PK to _cq_id (#6531)

erezrokah · pull[bot] · commit 9927ede42058 · 2023-10-17T14:10:54.000Z
#### Summary

The PK for `gcp_appengine_firewall_ingress_rules` was `project_id` which I think means we only save 1 firewall rule per project (in `overwrite*` modes). Doesn't seems like there's another field to use as a PK

Non breaking as the data exists

&lt;!--
diff --git a/plugins/source/gcp/docs/tables/gcp_appengine_firewall_ingress_rules.md b/plugins/source/gcp/docs/tables/gcp_appengine_firewall_ingress_rules.md
@@ -2,17 +2,17 @@
 
 https://cloud.google.com/appengine/docs/admin-api/reference/rest/v1/apps.firewall.ingressRules#FirewallRule
 
-The primary key for this table is **project_id**.
+The primary key for this table is **_cq_id**.
 
 ## Columns
 
 | Name          | Type          |
 | ------------- | ------------- |
 |_cq_source_name|String|
 |_cq_sync_time|Timestamp|
-|_cq_id|UUID|
+|_cq_id (PK)|UUID|
 |_cq_parent_id|UUID|
-|project_id (PK)|String|
+|project_id|String|
 |priority|Int|
 |action|String|
 |source_range|String|
diff --git a/plugins/source/gcp/resources/services/appengine/firewall_ingress_rules.go b/plugins/source/gcp/resources/services/appengine/firewall_ingress_rules.go
@@ -25,9 +25,6 @@ func FirewallIngressRules() *schema.Table {
 				Name:     "project_id",
 				Type:     schema.TypeString,
 				Resolver: client.ResolveProject,
-				CreationOptions: schema.ColumnCreationOptions{
-					PrimaryKey: true,
-				},
 			},
 		},
 	}
