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

special_testing_bindings_task, podman remove a paused container by id with force test fails #5325

Closed
cevich opened this issue Feb 25, 2020 · 5 comments
Closed

special_testing_bindings_task, podman remove a paused container by id with force test fails #5325

cevich opened this issue Feb 25, 2020 · 5 comments
Assignees
@baude
Labels
kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. stale-issue

Comments

@cevich
Copy link
Member

cevich commented Feb 25, 2020

/kind bug

Description

On F31:

...cut...
[+0504s] time="2020-02-25T09:50:42-05:00" level=debug msg="Paused container fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2"
[+0504s] time="2020-02-25T09:50:44-05:00" level=debug msg="APIHandler -- Method: DELETE URL: /v1.24/libpod/containers/fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2?force=true&vols=false (conn 1/4)"
[+0504s] time="2020-02-25T09:50:44-05:00" level=debug msg="Sending signal 9 to container fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2"
[+0504s] the container `fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2` is not running: No such process
[+0504s] time="2020-02-25T09:50:44-05:00" level=info msg="Request Failed(Internal Server Error): `/usr/bin/crun resume fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2` failed: exit status 1"
[+0504s] fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2
[+0504s] 
• Failure [8.980 seconds]
[+0504s] Podman containers 
[+0504s] 
/var/tmp/go/src/github.com/containers/libpod/pkg/bindings/test/containers_test.go:15
[+0504s]   
podman remove a paused container by id with force [It]
[+0504s]   
/var/tmp/go/src/github.com/containers/libpod/pkg/bindings/test/containers_test.go:177
[+0504s] 
[+0504s]   
Expected
[+0504s]       <utils.ErrorModel>: {
[+0504s]           Because: "`/usr/bin/crun resume fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2` failed: exit status 1",
[+0504s]           Message: "`/usr/bin/crun resume fad95b817abd7f418bc075d9738828d348050fd62db276676d144230e09133d2` failed: exit status 1",
[+0504s]           ResponseCode: 500,
[+0504s]       }
[+0504s]   to be nil
[+0504s] 
...

Steps to reproduce the issue:

  1. make
  2. cd pkg/bindings/test && ginkgo -r

Describe the results you received:

Test fails

Describe the results you expected:

Test passes

Additional information you deem important (e.g. issue happens only occasionally):

Discovered in/before commit 6babc9f on master
@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Feb 25, 2020
@cevich
Copy link
Member Author

cevich commented Feb 25, 2020

Marking test to be skipped via commit Temp. skip "remove pause by id" bindings test on #3901

@github-actions
Copy link

A friendly reminder that this issue had no activity for 30 days.

@rhatdan
Copy link
Member

rhatdan commented Mar 27, 2020

@cevich What is going on with this?

@cevich
Copy link
Member Author

cevich commented Apr 2, 2020

@lsm5 do you know about this, I'm still showing it as being skipped in master:

https://github.com/containers/libpod/blob/master/pkg/bindings/test/containers_test.go#L184-L201

@rhatdan
Copy link
Member

rhatdan commented Jun 9, 2020

Seems to be working now

@rhatdan rhatdan closed this as completed Jun 9, 2020
edsantiago added a commit to edsantiago/libpod that referenced this issue Aug 31, 2020
@edsantiago
bindings: reenable flaky(?) pause/unpause test
d36c4e0 
Reference: containers#6518, a very-frequently-flaking CI test, disabled
a month ago (containers#7143) because it was triggering so often in CI.
Unfortunately, that seems to have simply swept the problem
under the rug. AFAICT nobody has bothered to look at the
root bug, so let's just reenable. If the problem persists,
I'll let annoyed developers squeaky-wheel 6158 so there's
some incentive to fix it. If the problem has miraculously
gone away in the last month, that's a win too.

(This test failure does not reproduce on my laptop, nor
does it lend itself to devising a simple reproducer on
a test VM.)

Also: since containers#5325 appears to have been closed as fixed,
remove a 'Skip' that references it. Unfortunately this
also requires removing a lot of other cruft. This was
an incidental oh-by-the-way addition that I thought
would be trivial but ended up causing a much larger diff.

Signed-off-by: Ed Santiago <[email protected]>
@edsantiago edsantiago mentioned this issue Aug 31, 2020
Merged
@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 23, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@baude baude

Labels
kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. stale-issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@cevich @rhatdan @baude @openshift-ci-robot