Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[iam] fix: reduce flakiness #3293

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
Merged
tmatsuo merged 3 commits into from
Apr 8, 2020
Merged

[iam] fix: reduce flakiness #3293

tmatsuo merged 3 commits into from
Apr 8, 2020

Conversation

@tmatsuo
Copy link
Contributor

@tmatsuo tmatsuo commented Apr 7, 2020

fixes #3292

  • use uuid.uuid4() instead of random
  • retry write operations in the tests

[iam] fix: reduce flakiness
14baca8 
fixes GoogleCloudPlatform#3292

* use `uuid.uuid4()` instead of random
* retry write operations in the tests
@tmatsuo tmatsuo requested review from kurtisvg and leahecole April 7, 2020 20:43
@tmatsuo tmatsuo requested a review from a team as a code owner April 7, 2020 20:43
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 7, 2020
@tmatsuo tmatsuo merged commit 442efa4 into GoogleCloudPlatform:master Apr 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fhinkel fhinkel fhinkel approved these changes

@leahecole leahecole Awaiting requested review from leahecole

+1 more reviewer

@kurtisvg kurtisvg kurtisvg approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@busunkim96 busunkim96

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

iam:access_test.test_set_policy failed

5 participants

@tmatsuo @fhinkel @kurtisvg @googlebot @busunkim96