search

cockpit-project / cockpit-podman

Cockpit UI for podman containers
GNU Lesser General Public License v2.1
435 stars 90 forks source link

Settle containers lifecycle #1772

Closed jelly closed 3 months ago

jelly commented 3 months ago

Played around in this PR https://github.com/cockpit-project/cockpit-podman/pull/1771

martinpitt commented 3 months ago

This failure is related. Not sure if it's a missed spot or the patch here needs to try/wait harder?

jelly commented 3 months ago

And it goes red of course. The screenshot clearly shows that not all memory was allocated yet.

I could change this to wait on memory to be 300, that would be better, because CPU usage is not a real indicator anyway. or alternatively wait for a smaller cpu usage, locally it is 0%. Lowered it to < 10, lets see now it might need a longer timeout...

jelly commented 3 months ago

There is one failure left, where it flakes in a different way. I'd say this is progress :)

https://cockpit-logs.us-east-1.linodeobjects.com/pull-1772-338bd188-20240628-103015-rhel-9-4/log.html#38

martinpitt commented 3 months ago

This failure is also something else.