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.

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

Add ansible to PACKAGE_LIST #11

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Conversation

jakecorrenti
Copy link
Member

No description provided.

@l0rd
Copy link
Member

l0rd commented Jan 21, 2025

I have manually tested the PR:

export PACKAGE_LIST="procps-ng openssh-server net-tools iproute dhcp-client crun-wasm wasmedge-rt qemu-user-static subscription-manager gvisor-tap-vsock-gvforwarder cifs-utils nfs-utils-coreos ansible"
podman create --name fedora-update docker.io/library/fedora:40 sleep 7200
podman start fedora-update
podman exec -it fedora-update sh -c "dnf update -y && dnf -y install podman podman-docker $PACKAGE_LIST && dnf clean all && rm -rf /var/cache/yum"
podman export --output rootfs-with-ansible.tar fedora-update
zstd -T0 --auto-threads=logical --ultra -22 --long --rm --verbose rootfs-with-ansible.tar

I was able to run the command ansible-playbook using the simple playbook from the podman PR (with a warning):

[root@b471ee7a90b6 /]# ansible-playbook ./playbook.yml
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'

PLAY [Simple podman machine example] **********************************************************************************************************

TASK [Gathering Facts] ************************************************************************************************************************
ok: [localhost]

TASK [create a file] **************************************************************************************************************************
ok: [localhost]

PLAY RECAP ************************************************************************************************************************************
localhost                  : ok=2    changed=0    unreachable=0    failed=0    skipped=0    rescued=0    ignored=0

The image size difference:

# +600MB for the tar
❯ ls -lh *.tar
-rw-r--r--. 1 mariolet mariolet 787M Jan 21 14:21 rootfs.tar
-rw-r--r--. 1 mariolet mariolet 1.3G Jan 21 14:16 rootfs-with-ansible.tar

# +48MB when compressed
❯ ls -lh *.zst
-rw-r--r--. 1 mariolet mariolet 178M Jan 21 14:21 rootfs.tar.zst
-rw-r--r--. 1 mariolet mariolet 223M Jan 21 14:16 rootfs-with-ansible.tar.zst

Signed-off-by: Jake Correnti <[email protected]>
@jakecorrenti
Copy link
Member Author

jakecorrenti commented Jan 24, 2025

@l0rd PTAL, I added the ansible-core pkg instead. Thanks!

@l0rd l0rd merged commit 5ae32f0 into containers:main Jan 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants