Skip to content

Use Podman 5.5.2 for hosted integration tests - #1280

Open
Christof Marti (chrmarti) wants to merge 1 commit into
mainfrom
copilot/fix-podman-runner-storage
Open

Use Podman 5.5.2 for hosted integration tests#1280
Christof Marti (chrmarti) wants to merge 1 commit into
mainfrom
copilot/fix-podman-runner-storage

Conversation

@chrmarti

@chrmarti Christof Marti (chrmarti) commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Use Podman 5.5.2 for the Podman Feature integration matrix until a supported newer package includes the upstream layered-build fix.

Session Context

Key decisions and findings from the development session:

  • Preserve rootless coverage: The Podman tests and Feature workloads remain byte-for-byte identical to main; only the Podman executable used by that matrix entry changes.
  • Regression boundary: Podman 5.5.2 / Buildah 1.40.1 passes. Podman 5.6.0-rc2 / Buildah 1.41.1 and Podman 5.8.6 / Buildah 1.43.2 reset /tmp permissions after layered RUN --mount and fail APT in the next Feature.
  • Newest release checked: Podman 6.1.0 / Buildah 1.45.0 passes, but there is no official full-engine Linux binary, Ubuntu package, or mgoltzsche/podman-static 6.x bundle. Building it also requires Podman 6-specific recipe and storage configuration changes.
  • Temporary choice: Use the matching published mgoltzsche/podman-static 5.5.2 recipe for now. This avoids disabling layers, changing production Dockerfile generation, or weakening the test to rootful Podman.
  • Supply-chain pinning: The external podman-static build context is pinned to peeled commit 9ccf992536ecf0a4a3b82ac27fc708f00ab141ab, not the mutable v5.5.2 tag.

Changes

  • Build and install the Podman 5.5.2 static binary only for src/test/cli.podman.test.ts.
  • Pin the external static-build recipe to an immutable commit.
  • Make no changes to the Podman tests or Feature workloads.

Validation

@chrmarti
Christof Marti (chrmarti) force-pushed the copilot/fix-podman-runner-storage branch from 98e2814 to 07d0952 Compare August 17, 2026 07:20
@chrmarti
Christof Marti (chrmarti) changed the base branch from copilot/secure-oci-auth to main August 17, 2026 07:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR restores rootless Podman feature-test reliability on GitHub-hosted runners by overriding a runner-image Podman storage option (overlay.ignore_chown_errors) that breaks APT’s _apt user during feature builds.

Changes:

  • Reset the rootless Podman system state on GitHub Actions to start from a clean store.
  • Write a minimal user-level storage.conf to set the overlay driver and disable ignore_chown_errors.
  • Assert the effective Podman storage option via podman info before running the packaged CLI tests.
Show a summary per file
File Description
src/test/cli.podman.test.ts Adds GitHub Actions–specific rootless Podman storage reset + minimal storage.conf override, with a verification assertion before running the Podman feature tests.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

@chrmarti
Christof Marti (chrmarti) force-pushed the copilot/fix-podman-runner-storage branch from 24af44e to 40f5914 Compare August 17, 2026 07:32
@chrmarti Christof Marti (chrmarti) changed the title Fix rootless Podman tests on hosted runners Use lightweight Feature in Podman tests Aug 17, 2026
@chrmarti
Christof Marti (chrmarti) force-pushed the copilot/fix-podman-runner-storage branch from 24be598 to e24d7ea Compare August 17, 2026 08:27
@chrmarti Christof Marti (chrmarti) changed the title Use lightweight Feature in Podman tests Run hosted Podman Feature tests rootfully Aug 17, 2026
@chrmarti Christof Marti (chrmarti) changed the title Run hosted Podman Feature tests rootfully Keep hosted Podman Feature tests rootless Aug 18, 2026
@chrmarti Christof Marti (chrmarti) changed the title Keep hosted Podman Feature tests rootless Test Podman 5.6.0-rc2 in CI Aug 19, 2026
@chrmarti Christof Marti (chrmarti) changed the title Test Podman 5.6.0-rc2 in CI Test Podman 5.5.2 in CI Aug 19, 2026
@chrmarti Christof Marti (chrmarti) changed the title Test Podman 5.5.2 in CI Test Podman 6.1.0 in CI Aug 19, 2026
@chrmarti Christof Marti (chrmarti) changed the title Test Podman 6.1.0 in CI Use Podman 5.5.2 for hosted integration tests Aug 20, 2026
Pin the external static-build recipe to an immutable commit while keeping the existing rootless Podman Feature workloads unchanged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@chrmarti
Christof Marti (chrmarti) force-pushed the copilot/fix-podman-runner-storage branch from 55f140b to 3617a02 Compare August 20, 2026 07:15
@chrmarti
Christof Marti (chrmarti) marked this pull request as ready for review August 20, 2026 07:16
@chrmarti
Christof Marti (chrmarti) requested a review from a team as a code owner August 20, 2026 07:16
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