Merge pull request #19549 from rhatdan/quadlet

podman stop --cidfile missing --ignore

cmd/podman/containers/stop.go

@@ -2,6 +2,7 @@ package containers
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"os"
 	"strings"
@@ -110,6 +111,9 @@ func stop(cmd *cobra.Command, args []string) error {
 	for _, cidFile := range stopCidFiles {
 		content, err := os.ReadFile(cidFile)
 		if err != nil {
+			if stopOptions.Ignore && errors.Is(err, os.ErrNotExist) {
+				continue
+			}
 			return fmt.Errorf("reading CIDFile: %w", err)
 		}
 		id := strings.Split(string(content), "\n")[0]

docs/source/markdown/podman-rm.1.md.in

@@ -20,6 +20,8 @@ Remove all containers.  Can be used in conjunction with **-f** as well.
 
 @@option cidfile.read
 
+Command does not fail when *file* is missing and user specified --ignore.
+
 #### **--depend**
 
 Remove selected container and recursively remove all containers that depend on it.

docs/source/markdown/podman-stop.1.md.in

@@ -23,6 +23,8 @@ Stop all running containers.  This does not include paused containers.
 
 @@option cidfile.read
 
+Command does not fail when *file* is missing and user specified --ignore.
+
 #### **--filter**, **-f**=*filter*
 
 Filter what containers are going to be stopped.

test/system/050-stop.bats

@@ -85,6 +85,13 @@ load helpers
 
     run_podman stop --ignore $name
     is "$output" "" "podman stop nonexistent container, with --ignore"
+
+    nosuchfile=$PODMAN_TMPDIR/no-such-file
+    run_podman 125 stop --cidfile=$nosuchfile
+    is "$output" "Error: reading CIDFile: open $nosuchfile: no such file or directory" "podman stop with missing cidfile, without --ignore"
+
+    run_podman stop --cidfile=$nosuchfile --ignore
+    is "$output" "" "podman stop with missing cidfile, with --ignore"
 }