-
Notifications
You must be signed in to change notification settings - Fork 81
chore: add docker group command #949
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
Conversation
✨ Coder.com for PR #949 deployed! It will be updated on every commit.
|
This pull request has been linked to Shortcut Story #22848: chore: Update CVM Docker image docs to include docker group and docker-compose Dockerfile steps. |
RUN curl -L "https://github.com/docker/compose/releases/download/1.29.2/docker-compose-$(uname -s)-$(uname -m)" -o /usr/local/bin/docker-compose | ||
RUN chmod +x /usr/local/bin/docker-compose |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these can be joined to reduce the amount of layers using \ &&
RUN useradd coder \ | ||
--create-home \ | ||
--shell=/bin/bash \ | ||
# Add user `coder` to the `docker` group |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this work? I thought you couldn't put comments in the middle of a command in bash
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, not sure. i added that just to point it out. i can remove
updates our CVM image docs to include adding the
coder
user to thedocker
group.this is necessary for any non-root user to use Docker, as the Docker Engine is run as
root
.[sc-22848]