Skip to content

ci: add Linux rootless and rootful Podman to e2e test matrix#278

Merged
skevetter merged 3 commits into
mainfrom
2a7d-20db-linux-podman-ci
May 13, 2026
Merged

ci: add Linux rootless and rootful Podman to e2e test matrix#278
skevetter merged 3 commits into
mainfrom
2a7d-20db-linux-podman-ci

Conversation

@skevetter

@skevetter skevetter commented May 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add two Linux Podman entries to the e2e CI matrix: rootless and rootful modes
  • Rootless: installs Podman via apt, runs as current user
  • Rootful: installs Podman, starts root socket service, sets DOCKER_HOST and forwards it through sudo env passthrough
  • Job name template includes Podman mode for distinct CI display names (e.g., "Test up-provider-podman (rootless) on ubuntu-latest")
  • Uses socket poll instead of sleep for rootful service readiness

Summary by CodeRabbit

  • Chores
    • Updated continuous integration workflow configuration to enhance testing capabilities.

Review Change Stack

skevetter added 2 commits May 13, 2026 15:18
The CI matrix only tested Podman on Windows. Add two new Linux matrix
entries (rootless and rootful) that install Podman on ubuntu-latest and
run the up-provider-podman e2e tests against real Podman runtimes.
Pass DOCKER_HOST through sudo env passthrough so rootful Podman socket
reaches the test process. Add install-podman mode to job name template
for distinct CI display names. Replace sleep with socket poll for
rootful service readiness.
@skevetter
skevetter enabled auto-merge (squash) May 13, 2026 20:25
@netlify

netlify Bot commented May 13, 2026

Copy link
Copy Markdown

Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit f31778c
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a04fad6dcee2b0008cfb170

@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 68aceb89-ce68-4f51-ba16-8c92f8af6df2

📥 Commits

Reviewing files that changed from the base of the PR and between 16a4d5e and f31778c.

📒 Files selected for processing (1)
  • .github/workflows/pr-ci.yml

📝 Walkthrough

Walkthrough

The PR updates the CI workflow to support Podman installation in both rootless and rootful modes. The integration-tests job matrix is expanded with install-podman entries, job display names conditionally show the mode, and new Linux setup steps install Podman with mode-specific configuration before running tests.

Changes

Podman installation modes in integration tests

Layer / File(s) Summary
Matrix configuration and job display name
.github/workflows/pr-ci.yml
The integration-tests job matrix entries for up-provider-podman are split to include install-podman: rootless and install-podman: rootful options with adjusted install-kind values. Job display name now conditionally appends the podman mode suffix when matrix.install-podman is set.
Podman installation and environment setup
.github/workflows/pr-ci.yml
New conditional steps for Linux install Podman in rootless mode (with podman info check) or rootful mode (with podman.socket enablement, readiness wait, and DOCKER_HOST environment variable). Test execution conditionally injects DOCKER_HOST into the sudo environment when present.

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding Linux rootless and rootful Podman configurations to the CI test matrix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

The background `podman system service` process didn't create the socket
within the 10-second timeout on GitHub Actions runners (exit code 124).
Switch to `systemctl enable --now podman.socket` which is the standard
way to activate the Podman API socket on systemd-based runners.
@skevetter
skevetter merged commit f5c7361 into main May 13, 2026
100 of 104 checks passed
@skevetter
skevetter deleted the 2a7d-20db-linux-podman-ci branch May 13, 2026 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant