-
Notifications
You must be signed in to change notification settings - Fork 316
Is the addition of an IAM (particularly authorized view permissions) additive or destructive? #569
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
This replaces the existing ACLs. As far as race conditions are concerned, the client library uses the
This happens automatically if you first fetch a dataset with |
Okay, so this is if one is using multiple Python clients. But in my case, I am expecting a clash between the Python client library and Terraform (via GitHub automation, Atlantis) as well.
Do you mean, the existing ones (such as "foo" and "bar") will be dropped and recreated?
|
I mean that the whole ACLs property is replaced atomically. |
Regarding clash between Python and Terraform, I'm not as familiar with the Terraform provider. It's probably worth confirming with them at https://github.com/hashicorp/terraform-provider-google to see what they do with the ACLs property. |
…amples#569) * Add tests for quickstarts * Update secrets
Is the addition of an IAM (particularly authorized view permissions) additive or destructive?
In a process to add an access_entry to a view_dataset, does BQ has to drop existing ones and then create all (along with new addition) of them, or is it smart enough to keep the existing ones untouched? If it is a destructive one, then similar calls (adding different access entries to the same dataset) from two different clients can encounter race conditions.
The corresponding API (https://cloud.google.com/bigquery/docs/reference/rest/v2/datasets/patch) (and data object https://cloud.google.com/bigquery/docs/reference/rest/v2/datasets) also doesn't seem to provide any details on this.
The text was updated successfully, but these errors were encountered: