Skip to content

allow overriding DOCKER_SOCKET#69

Closed
parkan wants to merge 4 commits intoFilOzone:mainfrom
parkan:feat/docker-socket-env
Closed

allow overriding DOCKER_SOCKET#69
parkan wants to merge 4 commits intoFilOzone:mainfrom
parkan:feat/docker-socket-env

Conversation

@parkan
Copy link

@parkan parkan commented Feb 27, 2026

portainer currently hardcodes rootful docker socket, this allows overriding the socket location if you run a locked down system with podman (which I highly recommend anyone doing blockchain development do)

@FilOzzy FilOzzy added this to FOC Feb 27, 2026
@github-project-automation github-project-automation bot moved this to 📌 Triage in FOC Feb 27, 2026
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting review in FOC Feb 27, 2026
@rjan90 rjan90 requested a review from redpanda-f February 27, 2026 17:02
@rjan90 rjan90 added this to the M4.1: mainnet ready milestone Feb 27, 2026
@github-project-automation github-project-automation bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Feb 27, 2026
@parkan
Copy link
Author

parkan commented Feb 27, 2026

slight formatting issue, fixing

defaults to /var/run/docker.sock when unset. allows rootless podman
users to point portainer at their user socket, e.g.
DOCKER_SOCKET=/run/user/1000/podman/podman.sock
@parkan parkan force-pushed the feat/docker-socket-env branch from ec53e4a to c726f94 Compare February 27, 2026 17:12
adds bind_mount() helper that appends :z to all host path mounts.
:z works on both docker (no-op without SELinux) and podman (relabeling).
builder containers use --userns=keep-id instead of -u uid:gid when
podman is detected, avoiding UID remapping issues.
@parkan
Copy link
Author

parkan commented Feb 27, 2026

err this got a little out of hand, I'm going to close and find a simpler path to fix

@parkan parkan closed this Feb 27, 2026
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FOC Feb 27, 2026
@redpanda-f
Copy link
Collaborator

@parkan can you please convert this to draft while it is still being added with commits. Please consider earlier approval stale. Will check this on Monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

3 participants