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 Oct 18, 2024
1 parent 16a9afd commit e33734d
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 @@ -264,7 +264,7 @@ def setup_dependencies(manifests, config_map, distro, arch):
finally:
if container_ids:
print("📦 Stopping containers")
out, _ = testlib.runcmd(["podman", "stop", " ".join(container_ids)])
out, _ = testlib.runcmd(["podman", "stop", *container_ids])
print(out.decode())


Expand Down

0 comments on commit e33734d

Please sign in to comment.