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

Set file permissions so another container can read them #398

Merged
merged 1 commit into from Feb 13, 2016

Conversation

Projects
None yet
3 participants
@paulbakker
Contributor

paulbakker commented Jan 24, 2016

When used on Kubernetes as a side-car, the other container should have read permissions to the cloned files. This is not necessarily the case without explicitly setting it.

I ran into this problem in combination with an Nginx container and a shared volume.

@googlebot googlebot added the cla: yes label Jan 24, 2016

@bgrant0607

This comment has been minimized.

Member

bgrant0607 commented Feb 13, 2016

LGTM, thanks.

bgrant0607 added a commit that referenced this pull request Feb 13, 2016

Merge pull request #398 from paulbakker/paulbakker/permissions
Set file permissions so another container can read them

@bgrant0607 bgrant0607 merged commit 25ecb8b into kubernetes:master Feb 13, 2016

2 checks passed

cla/google All necessary CLAs are signed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment