Skip to content

ci: test podman in CI - #374

Merged
muchzill4 merged 3 commits into
arm:mainfrom
muchzill4:podman-ci
Jul 27, 2026
Merged

ci: test podman in CI#374
muchzill4 merged 3 commits into
arm:mainfrom
muchzill4:podman-ci

Conversation

@muchzill4

@muchzill4 muchzill4 commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Install podman and docker-compose in CI, so that podman tests are executed
  • Turns out podman <5.3 doesn’t support PODMAN_COMPOSE_WARNING_LOGS=false, so I had to fix nd json parsing
    • This will affect ps in the future where we explicitly parse the output, something to think about there and then

Checklist

  • 🤖 This change is covered by tests as required.
  • 🤹 All required manual testing has been performed.
  • 📖 All documentation updates are complete.

Signed-off-by: Bartek Mucha <bartosz.mucha@arm.com>
@muchzill4 muchzill4 changed the title Test podman in CI ci: Test podman in CI Jul 27, 2026
Signed-off-by: Bartek Mucha <bartosz.mucha@arm.com>
Signed-off-by: Bartek Mucha <bartosz.mucha@arm.com>
@muchzill4 muchzill4 changed the title ci: Test podman in CI ci: test podman in CI Jul 27, 2026
Comment thread .github/workflows/ci.yaml
with:
distribution: Ubuntu-24.04
# Docker and Podman require the Linux kernel and container primitives provided by WSL2.
wsl-version: 2

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is default, but good to call out explicitly.

@muchzill4
muchzill4 marked this pull request as ready for review July 27, 2026 08:33
@muchzill4
muchzill4 requested a review from a team as a code owner July 27, 2026 08:33
Comment thread .github/workflows/ci.yaml
Comment on lines +92 to +96
case "$(uname -m)" in
x86_64) compose_arch=x86_64 ;;
aarch64|arm64) compose_arch=aarch64 ;;
*) echo "Unsupported architecture: $(uname -m)" >&2; exit 1 ;;
esac

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: Is this needed? We don't have any runners on x86_32 or any other arch

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We don’t, but it’d be good if it exploded otherwise. 🤷

Comment thread .github/workflows/ci.yaml

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeesh we need to break this up into something like

ci.yaml
setup_windows_containers.yaml
setup_linux_containers.yaml

Doesn't block this though, separate refactor for a GH yaml lover to take on later!

@muchzill4
muchzill4 merged commit dfaefb9 into arm:main Jul 27, 2026
8 checks passed
@muchzill4
muchzill4 deleted the podman-ci branch July 27, 2026 14:19
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