-
Notifications
You must be signed in to change notification settings - Fork 84
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
test: Work around leaking podman processes and broken API after idle timeout #473
Conversation
Stopping podman.service often leaves processes behind, due to the broken `KillMode=process`. This was fixed upstream, but is not in a release yet. Also fix the unit type, to avoid a forever-hanging "activating" state. This is also already fixed in upstream master. Closes cockpit-project#473
Much much better! Still breaks testNotRunning But also, I lied in that commit -- it's not really a race condition in podman.service, the socket is already handed over correctly. With curl it works reliably all the time. I also don't get that error interactively, it works. So there must be something more subtle going on. |
Stopping podman.service often leaves processes behind, due to the broken `KillMode=process`. This was fixed upstream, but is not in a release yet. Also fix the unit type, to avoid a forever-hanging "activating" state. This is also already fixed in upstream master. Closes cockpit-project#473
Stopping podman.service often leaves processes behind, due to the broken `KillMode=process`. This was fixed upstream, but is not in a release yet. Also fix the unit type, to avoid a forever-hanging "activating" state. This is also already fixed in upstream master.
I finally have this working on my local system now, let's see what CI says to that. Now I will try to find CLI reproducers for these problems, and report them upstream. |
Hooray, I finally understand the madness that's going on here: containers/podman#7180 (comment) |
Working with user podman API leaves behind a `podman pause` process which is not attached to/stopped by podman.service. This completely breaks the API: containers/podman#7180 Closes cockpit-project#473
Alas, that was a hard one. In hindsight it's pretty obvious, but I got led astray in so many different wrong directions here.. @marusak, this is ready for review. 🎉 |
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.
Cool! Thank you @martinpitt for figuring this one out!
No description provided.