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

Update Docker image #710

Open
wants to merge 2 commits into
base: add-docker-image
Choose a base branch
from

Conversation

zuzak
Copy link
Contributor

@zuzak zuzak commented Mar 12, 2019

If you merge this pull request, it'll update #631, which you can then merge into master.
This pull request is not against master.

This PR removes the usage data config from the code in #631.
It also slightly selfishly adds an explicit user group, as it would be needed to run the container on our Kubernetes infrastructure.

This commit removes configuration steps rendered superfluous by changes
introduced in 919e98e.
Due to kubernetes@445393fdcefa6d0354b7ce32a2304a7765fbd305 one must use
numeric user groups if you want to run a container as non-root on
Kubernetes.

This commit ensures the container runs as a non-root user with an
explicit numeric ID, so that it can run on a Kubernetes cluster that
forces MustRunAsNonRoot.
@kellylee-gds kellylee-gds added new and removed new labels Mar 13, 2019
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.

2 participants