Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

tests: further harden blob teardown against EBH - #548

Merged
tseaver merged 5 commits into
masterfrom
435-harden-ebh-systest-further
Aug 24, 2021
Merged

tests: further harden blob teardown against EBH#548
tseaver merged 5 commits into
masterfrom
435-harden-ebh-systest-further

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

Closes#435.

@tseaver
tseaver requested review from a teamAugust 19, 2021 21:19
@product-auto-labelproduct-auto-labelBot added the api: storage Issues related to the googleapis/python-storage API. label Aug 19, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Aug 19, 2021
Comment threadtests/system/_helpers.py Outdated
@tseaver

Copy link
Copy Markdown
ContributorAuthor

Systest flake reported as #551.

@tseaver
tseaver requested a review from cojencoAugust 23, 2021 19:47

@cojencocojenco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@tseavertseaver added the automerge Merge the pull request once unit tests and other checks pass. label Aug 24, 2021
@gcf-merge-on-green

Copy link
Copy Markdown
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-greengcf-merge-on-greenBot removed the automerge Merge the pull request once unit tests and other checks pass. label Aug 24, 2021
@tseaver
tseaver merged commit e5f2d13 into masterAug 24, 2021
@tseaver
tseaver deleted the 435-harden-ebh-systest-further branch August 24, 2021 16:50
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storageIssues related to the googleapis/python-storage API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

'test_bucket_w_default_event_based_hold' systest is flaky

3 participants

@tseaver@Ark-kun@cojenco