-
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
WIP: TESTING: DO NOT MERGE #7342
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cevich The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
cc3b4ad
to
8b7a2ba
Compare
@edsantiago I think the spot you're seeing the "hang" is a red-herring, I think something's happening on another level to cause this. Look at the example from this PR: https://cirrus-ci.com/task/5905973485240320?command=system_test#L156 Execution simply/literally stops w/o notice, until the "BIG" timeout clobbers it. All the other tests passed, so...geeze...the only thing I can think is something must be happening on the cirrus side. Maybe related to the underlying VMs being preemptable and a secondary bug related to output processing when that happens? https://cloud.google.com/compute/docs/instances/preemptible |
....Actually, no...scratch that. Our VMs are NOT preemptable...we use the expensive ones 😕 |
FWIW: acting on the possibility that the |
Excellent, thanks for trying that. One more question-mark item we can check off the list. ONE of my tests also hit the 120min timeout: https://github.com/containers/podman/pull/7342/checks?check_run_id=995072852 but appears almost totally different from yours. I'm taking this as meaning the problem isn't specific to your PR, and we should basically expect to see this problem appear everywhere. I'm going to try and re-run it to see if the behavior changes at all (IIRC, yours always failed for at least 3-5 retries). |
Signed-off-by: Chris Evich <[email protected]>
8b7a2ba
to
8ea8c0e
Compare
Problem fixed by #7354 |
Signed-off-by: Chris Evich [email protected]