-
Notifications
You must be signed in to change notification settings - Fork 90
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
Add manifest conditions #1305
Add manifest conditions #1305
Conversation
29aee35
to
f2852b4
Compare
this failure looks relevant. The rest look like unrelated flakes. |
Well yeah, |
f2852b4
to
e9512b1
Compare
Rawhide fails! I wonder why hmmmmmm
|
@jelly Because TF's rawhide image is still a bit older, the new 294 version isn't in that yet? |
So hmm this should use the Python bridge. |
e9512b1
to
9c7b6a6
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.
Hmm, it still failed on most OSes, although the logic looks okay to me 😢
9c7b6a6
to
d231d66
Compare
Check for the existence of the podman.socket for the Cockpit client scenario.
d231d66
to
d67bf2c
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.
Cheers!
This is really stubborn, I already retried like 5 times. @jelly this smells a lot like cockpit-project/bots#4829 , let's just copy the naughty? |
I filed cockpit-project/bots#4884 to fix this. Landing this PR as it's unrelated. |
Check for the existence of the podman.socket for the Cockpit client scenario.
See cockpit-project/cockpit#18935