Skip to content

Commit

Permalink
test/generate-ostree-build-config: fix podman stop command
Browse files Browse the repository at this point in the history
The podman stop command was passing the list of container IDs as a
single argument with spaces in it, which is wrong.  The arguments need
to be separate, so we should expand the list of IDs instead.

This was never an issue because we never had multiple containers in a
distro's ostree build config until now.
  • Loading branch information
achilleas-k committed Sep 3, 2024
1 parent 0a55fca commit 3ee4b46
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/scripts/generate-ostree-build-config
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ def setup_dependencies(manifests, config_map, distro, arch):
finally:
if container_ids:
print("📦 Stopping containers")
out, _ = testlib.runcmd(["sudo", "podman", "stop", " ".join(container_ids)])
out, _ = testlib.runcmd(["sudo", "podman", "stop", *container_ids])
print(out.decode())


Expand Down

0 comments on commit 3ee4b46

Please sign in to comment.