Skip to content
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.

Sign up for GitHub

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

Jump to bottom

podman run --rm: ERRO[0000] Removing container SHA: error removing storage for container "SHA": identifier is not a container #11775

Closed
edsantiago opened this issue Sep 28, 2021 · 1 comment · Fixed by #11787
Closed

podman run --rm: ERRO[0000] Removing container SHA: error removing storage for container "SHA": identifier is not a container #11775

edsantiago opened this issue Sep 28, 2021 · 1 comment · Fixed by #11787
Assignees
@rhatdan
Labels
flakes Flakes from Continuous Integration locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. remote Problem is in podman-remote

Comments

@edsantiago
Copy link
Collaborator

New podman-remote flake. Reproducer:

$ while :;do bin/podman-remote run --tty -i --rm quay.io/libpod/testimage:20210610 echo hello;done
hello
hello
...
ERRO[0000] Removing container e76ac83d715b96b12b85e82356ba6a639496c963fbd740f32ebed9946b90c0cf: error removing storage for container "e76ac83d715b96b12b85e82356ba6a639496c963fbd740f32ebed9946b90c0cf": identifier is not a container
...

main @ cc23e05 but I see it also in two CI runs on 09-22, both in the int remote fedora-34 root host test on #11604, in two different subtests.

This is kind of important because this is not a flake that will trigger a CI failure -- YET. I'm working on fixes to the system tests that are much stricter, and will fail on this flake. Obviously I can't even submit these fixes until this flake is corrected.
@edsantiago edsantiago added flakes Flakes from Continuous Integration remote Problem is in podman-remote labels Sep 28, 2021
@edsantiago edsantiago mentioned this issue Sep 28, 2021
Merged
rhatdan added a commit to rhatdan/podman that referenced this issue Sep 29, 2021
@rhatdan
Storage can remove ErrNotAContainer as well
55843f9 
Fixes: containers#11775

[NO TESTS NEEDED] No easy way to cause this problem in CI.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
@rhatdan rhatdan mentioned this issue Sep 29, 2021
Merged
rhatdan added a commit to rhatdan/podman that referenced this issue Sep 29, 2021
@rhatdan
Storage can remove ErrNotAContainer as well
c23f516 
Fixes: containers#11775

[NO TESTS NEEDED] No easy way to cause this problem in CI.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan added a commit to rhatdan/podman that referenced this issue Sep 29, 2021
@rhatdan
Storage can remove ErrNotAContainer as well
c9ea2ca 
Fixes: containers#11775

[NO TESTS NEEDED] No easy way to cause this problem in CI.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
@edsantiago
Copy link
Collaborator Author

Seen just now in bodhi, f36 remote rootless. Build is 3.4.2-1, and git tag --contains c9ea2cae1 shows no hits, so even though this was fixed way way back in September I'm going to just assume that 3.4 does not contain the fix.

@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 21, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rhatdan rhatdan

Labels
flakes Flakes from Continuous Integration locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. remote Problem is in podman-remote
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Storage can remove ErrNotAContainer as well rhatdan/podman
2 participants
@edsantiago @rhatdan