Skip to content
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.

Sign up for GitHub

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

Jump to bottom

iam:access_test.test_set_policy failed #3292

Closed
tmatsuo opened this issue Apr 7, 2020 · 1 comment · Fixed by #3293
Closed

iam:access_test.test_set_policy failed #3292

tmatsuo opened this issue Apr 7, 2020 · 1 comment · Fixed by #3293
Assignees
@tmatsuo

Comments

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 7, 2020

Build:
https://source.cloud.google.com/results/invocations/f7538001-0aa4-41b3-a08e-394c23c10558/targets

Error:

googleapiclient.errors.HttpError: <HttpError 409 when requesting https://cloudresourcemanager.googleapis.com/v1/projects/python-docs-samples-tests:setIamPolicy?alt=json returned "There were concurrent policy changes. Please retry the whole read-modify-write with exponential backoff.">
@tmatsuo tmatsuo self-assigned this Apr 7, 2020
@tmatsuo tmatsuo mentioned this issue Apr 7, 2020
Merged
@tmatsuo
Copy link
Contributor Author

tmatsuo commented Apr 7, 2020

The resource name in the test is created by the following code:

rand = str(random.randint(0, 1000))
name = "python-test-" + rand

I think it actually conflicted among the multiple builds. I'm going to use uuid and it'll be fine.

tmatsuo pushed a commit to tmatsuo/python-docs-samples that referenced this issue Apr 7, 2020
[iam] fix: reduce flakiness
14baca8 
fixes GoogleCloudPlatform#3292

* use `uuid.uuid4()` instead of random
* retry write operations in the tests
@tmatsuo tmatsuo mentioned this issue Apr 7, 2020
Merged
tmatsuo pushed a commit that referenced this issue Apr 8, 2020
[iam] fix: reduce flakiness (#3293)
442efa4 
fixes #3292

* use `uuid.uuid4()` instead of random
* retry write operations in the tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmatsuo tmatsuo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[iam] fix: reduce flakiness tmatsuo/python-docs-samples
1 participant
@tmatsuo