Skip to content

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

Closed
skevetter wants to merge 2 commits into
mainfrom
2a7d-20db-ci-podman-linux
Closed

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

Conversation

@skevetter

@skevetter skevetter commented May 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds Linux rootless and rootful Podman entries to the e2e test matrix in .github/workflows/pr-ci.yml
  • Installs Podman via apt, starts appropriate socket (user for rootless, system for rootful)
  • Forwards DOCKER_HOST through sudo to test process for rootful mode
  • Uses socket readiness polling instead of sleep for reliability
  • Distinct job display names: "Test up-provider-podman (rootless)" and "Test up-provider-podman (rootful)"
  • All existing CI entries unchanged (purely additive)

skevetter added 2 commits May 13, 2026 15:22
@skevetter
ci: add Linux Podman (rootless and rootful) to e2e test matrix
6d8fabf 
The CI matrix only tested Podman on Windows. Add two new Linux entries
for rootless and rootful Podman so the up-provider-podman tests run
against both privilege modes on ubuntu-latest. DOCKER_HOST is set via
GITHUB_ENV and forwarded through the sudo test invocation.
@skevetter
fix(ci): address review findings for Linux Podman matrix entries
39d30b6 
Always forward DOCKER_HOST through sudo (empty string is harmless for
Docker entries). Add variant field to matrix entries so rootless and
rootful jobs get distinct display names. Replace podman info with
socket-exists polls for deterministic readiness checks.
@netlify

netlify Bot commented May 13, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit 39d30b6
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a04df2e51bff700083bc42b

@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ef36cd3a-a295-4257-8dc1-7a6586005e60

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@skevetter

skevetter commented May 14, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by PR #278 which was merged. This was a duplicate from a dead Manager agent.

@skevetter skevetter closed this May 14, 2026
@skevetter
skevetter deleted the 2a7d-20db-ci-podman-linux branch May 17, 2026 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size/s

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@skevetter