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: support docker-in-docker #166

Merged

Conversation

n1ru4l
Copy link
Collaborator

@n1ru4l n1ru4l commented Mar 24, 2020

The solution is based on a simple shell script that will detect whether the docker socket is mounted inside the container or not.

I also added an example for docker-in-docker that uses the official docker dind image.

See https://stackoverflow.com/a/60830857/4202031 for an explanation.

Closes #159

@n1ru4l n1ru4l force-pushed the feat-docker-in-docker-support branch from 49c703e to be471fd Compare March 24, 2020 15:47
@n1ru4l n1ru4l force-pushed the feat-docker-in-docker-support branch 4 times, most recently from f4818ed to 688952e Compare March 30, 2020 06:18
The solution is based on a simple shell script that will detect whether the docker socket is mounted inside the container or not.
@n1ru4l n1ru4l force-pushed the feat-docker-in-docker-support branch 4 times, most recently from 358d142 to 4899ef5 Compare March 30, 2020 07:00
@n1ru4l n1ru4l force-pushed the feat-docker-in-docker-support branch 2 times, most recently from 207c85f to a94c535 Compare March 30, 2020 09:30
@n1ru4l n1ru4l force-pushed the feat-docker-in-docker-support branch from a94c535 to e9cc601 Compare March 30, 2020 09:31
@n1ru4l
Copy link
Collaborator Author

n1ru4l commented Apr 16, 2020

hey @erikengervall do you have plans on reviewing this any time soon? 😇

@erikengervall
Copy link
Owner

hey @erikengervall do you have plans on reviewing this any time soon? 😇

Hey!

Yes, definitely! Thanks for calling me out - I'll take a look as soon as possible 🙂

Copy link
Owner

@erikengervall erikengervall left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@erikengervall erikengervall merged commit 09949c6 into erikengervall:master Apr 18, 2020
@erikengervall
Copy link
Owner

Available in 2.0.2 🙂

@n1ru4l n1ru4l deleted the feat-docker-in-docker-support branch April 18, 2020 10:35
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.

better docker in docker/docker socket injected into docker support
2 participants