Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add optional container user namespace for Ryuk #2702

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

scorpionknifes
Copy link

What does this PR do?

Users can configure which container user namespace to use for the Ryuk container by setting the TESTCONTAINERS_RYUK_CONTAINER_USER_NAMESPACE environment variable, or the ryuk.container.user.namespace property.

Why is it important?

required for the following error.

docker: Error response from daemon: privileged mode is incompatible to user namespaces. You must run the container in the host namespace when running privileged mode'

Related issues

Closes #2684

@scorpionknifes scorpionknifes requested a review from a team as a code owner August 8, 2024 04:03
Copy link

netlify bot commented Aug 8, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit c4fc646
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/66b443a51a2b480008516dda
😎 Deploy Preview https://deploy-preview-2702--testcontainers-go.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 site configuration.

Copy link

netlify bot commented Aug 8, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 38ed280
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/66effd4e41968c0008e1ae58
😎 Deploy Preview https://deploy-preview-2702--testcontainers-go.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 site configuration.

@eddumelendez eddumelendez changed the title feat: Add optional container user namespace for Ryku feat: Add optional container user namespace for Ryuk Aug 8, 2024
@mdelapenya
Copy link
Member

Duplicated of #2594

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: Add optional container user namespace for Ryku
2 participants