Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion bootc-ubuntu-setup/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ runs:
echo "deb ${mirror} plucky universe main" | sudo tee /etc/apt/sources.list.d/plucky.list
/bin/time -f '%E %C' sudo apt update
# skopeo is currently older in plucky for some reason hence --allow-downgrades
/bin/time -f '%E %C' sudo apt install -y --allow-downgrades crun/plucky podman/plucky skopeo/plucky just
/bin/time -f '%E %C' sudo apt install -y --allow-downgrades crun/plucky buildah/plucky podman/plucky skopeo/plucky just
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

Mixing packages from a future release (plucky) with an LTS base (noble) is risky. As plucky (25.04) evolves, its packages will eventually depend on a newer version of libc6 (glibc) than what is available in Ubuntu 24.04. This can lead to broken dependencies or a failed installation that renders the runner unusable. Consider using official static binaries for a more stable way to get the latest container tools in CI.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

For better readability and easier maintenance of this package list, consider splitting the apt install command into multiple lines.

        /bin/time -f '%E %C' sudo apt install -y --allow-downgrades \
          crun/plucky \
          buildah/plucky \
          podman/plucky \
          skopeo/plucky \
          just

# This is the default on e.g. Fedora derivatives, but not Debian.
# Only needed when libvirt/virtualization is requested.
- name: Enable unprivileged /dev/kvm access
Expand Down