Riley Karson
Riley Karson
@melinath to fix metadata (this references the wrong test)
Hmm, looks like an issue with the provisioner / connection behaviour? @paddycarver do you mind looking in to this when you have cycles?
I confirmed this worked with subnet-level permissions explicitly with the below repro (infra/main.tf with a highly permissioned account, and application/main.tf with the SA created in infra/main.tf), and project-level permissions were...
Sized around adding a sweeper
Both reviewers that have been on this change are unavailable for different periods. @melinath can you pick up review or hand off when you're back?
I think we would want to accept nil/null (take default) as distinct from empty, empty is explicit empty, and other nonempty arrays are the same as before.
@c2thorn @NickElliot @BBBmau fyi. I'm gonna close the PR as stale, but could be worth referencing for y'all.
No timeline- I'm not certain that's entirely a bug either, just incorrectly specified. You could define the label in both places, right?
That issue is from https://github.com/hashicorp/terraform-provider-google/issues/14679
This makes sense, we support `google_iam_policy` (the whole `policy` entry), and `google_iam_binding` (a single entry in bindings), but could support `google_iam_bindings` to manage the `bindings` item at https://cloud.google.com/resource-manager/reference/rest/Shared.Types/Policy