Skip to content

issue triage 20143#25037

Open
aevesdocker wants to merge 1 commit into
docker:mainfrom
aevesdocker:issue-20143
Open

issue triage 20143#25037
aevesdocker wants to merge 1 commit into
docker:mainfrom
aevesdocker:issue-20143

Conversation

@aevesdocker
Copy link
Copy Markdown
Contributor

@aevesdocker aevesdocker commented May 12, 2026

Description

With help from my pal Claude, Closes #20143

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit aba9cf5
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a030ce5396fb900080e1d12
😎 Deploy Preview https://deploy-preview-25037--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the area/desktop Issue affects a desktop edition of Docker. E.g docker for mac label May 12, 2026
Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

No significant issues found. The new FAQ section about bind-mounting the Docker socket is technically accurate, well-structured, and matches the existing document's scope and tone.

@aevesdocker aevesdocker requested a review from a team May 12, 2026 11:29
@aevesdocker aevesdocker added the status/review Pull requests that are ready for review label May 12, 2026
@dvdksn
Copy link
Copy Markdown
Contributor

dvdksn commented May 12, 2026

I think this PR might be solving a problem that just got fixed upstream. There's a commit in Docker Desktop from two weeks ago (https://github.com/docker/pinata/commit/86f61d143baee802ed2d0e8be1293fe719ea0c66) that adds ~/.docker/desktop/docker.sock to the rewriter — so mounting that path directly now works the same as mounting /var/run/docker.sock. The original issue reporter back in 2024 was correct that it didn't work at the time, but it should now (?)

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

Labels

area/desktop Issue affects a desktop edition of Docker. E.g docker for mac status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document special behavior when bind-mounting docker socket?

3 participants