-
-
Notifications
You must be signed in to change notification settings - Fork 17k
CI podman detection is not fully reliable #113129
Copy link
Copy link
Closed
Labels
A-contributor-roadblockArea: Makes things more difficult for new or seasoned contributors to RustArea: Makes things more difficult for new or seasoned contributors to RustT-infraRelevant to the infrastructure team, which will review and decide on the PR/issue.Relevant to the infrastructure team, which will review and decide on the PR/issue.
Description
Activity
Metadata
Metadata
Assignees
Labels
A-contributor-roadblockArea: Makes things more difficult for new or seasoned contributors to RustArea: Makes things more difficult for new or seasoned contributors to RustT-infraRelevant to the infrastructure team, which will review and decide on the PR/issue.Relevant to the infrastructure team, which will review and decide on the PR/issue.
podman is (by design) not fully compatible with docker when it comes to user permissions. Therefore,
./src/ci/docker/run.shneeds to look behind thedockercurtain to figure out whether it's actually podman and opt out of the user isolation.rust/src/ci/docker/run.sh
Lines 219 to 226 in eb76764
With ubuntus docker podman compat package this works fine, because there
docker --versionprints something containing podman. But if you symlinkdockertopodman(something NixOS does if you set up docker-podman compat) thendocker --versionactually printsdocker version 4.5.0. Podman outsmarts CI's detection.I'm not sure about the ideal way to fix this, but
docker system --helpcontains "manage podman" for me, so do whatever you want with this information.