-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Fixed blobdescriptorservice unit test flake #13289
Conversation
[test] |
@legionus could you give this a quick look please? |
I've extended the PR for a fix for a data race found in #13274. |
5d59364
to
fdfda0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Avoid a race when trying to delete just uploaded manifest. The purpose of the test is just to verify that our blobdescriptorservice is in use. Signed-off-by: Michal Minář <[email protected]>
Initialize the log level just once. Disable upload purging. Signed-off-by: Michal Minář <[email protected]>
fdfda0f
to
0ca5955
Compare
Evaluated for origin test up to 0ca5955 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/235/) (Base Commit: 1eb44c7) |
@legionus may I ask for your merge powers? |
[merge] |
Evaluated for origin merge up to 0ca5955 |
It seems something went wrong. Please merge it. |
Nothing is wrong, the bot shows you are in the queue -- you must wait your turn to merge |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/115/) (Base Commit: 5b8b4dc) (Image: devenv-rhel7_6074) |
Avoid a race when trying to delete just uploaded manifest. The purpose of the
test is just to verify that our blobdescriptorservice is in use.
Resolves #13288