-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
DO NOT MERGE: Tests for https://github.com/containers/storage/pull/1607 #18634
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mtrmac The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
c6a5172
to
dc4625c
Compare
This does show failures; this PR now includes #18636, so that one of the failures is correctly reported. |
e33d5d0
to
3cd4e59
Compare
Seeing consistent failures:
Uh, ?? … so adding more logging to see how we got there. |
7670e86
to
205306c
Compare
Nerf the bloat check, this PR is testing-only Signed-off-by: Miloslav Trmač <[email protected]>
Signed-off-by: Miloslav Trmač <[email protected]>
205306c
to
70430b4
Compare
The c/storage PR was merged (regardless of test failures here…) |
Does this PR introduce a user-facing change?