opensource/podman
1
0
Fork 0
mirror of https://github.com/containers/podman.git synced 2025-06-20 00:51:16 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix podman rm to have correct exit codes

Browse Source 
If you attempt to remove a running container is it supposed to exit with
2
If you attempt to remove a non existing container is is supposed to exit with
1

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
This commit is contained in:
Daniel J Walsh
2020-04-22 11:50:52 -04:00
parent 7f1d00108e
commit 1be8ded964
2 changed files with 14 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
cmd/podman/containers/rm.go
View File

@ -95,11 +95,9 @@ func rm(cmd *cobra.Command, args []string) error {
}
responses, err := registry.ContainerEngine().ContainerRm(context.Background(), args, rmOptions)
if err != nil {
// TODO exitcode is a global main variable to track exit codes.
// we need this enabled
//if len(c.InputArgs) < 2 {
// exitCode = setExitCode(err)
//}
if len(args) < 2 {
setExitCode(err)
}
return err
}
for _, r := range responses {
@ -108,6 +106,7 @@ func rm(cmd *cobra.Command, args []string) error {
if errors.Cause(err) == define.ErrWillDeadlock {
logrus.Errorf("Potential deadlock detected - please run 'podman system renumber' to resolve")
}
setExitCode(r.Err)
errs = append(errs, r.Err)
} else {
fmt.Println(r.Id)
@ -115,3 +114,13 @@ func rm(cmd *cobra.Command, args []string) error {
}
return errs.PrintErrors()
}
func setExitCode(err error) {
cause := errors.Cause(err)
switch cause {
case define.ErrNoSuchCtr:
registry.SetExitCode(1)
case define.ErrCtrStateInvalid:
registry.SetExitCode(2)
}
}

1
test/e2e/rm_test.go
View File

@ -17,7 +17,6 @@ var _ = Describe("Podman rm", func() {
)
BeforeEach(func() {
Skip(v2fail)
tempdir, err = CreateTempDirInTempDir()
if err != nil {
os.Exit(1)