-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Samples - Change the memcached image used in the samples so that we do not need to use RunAsUser #5934
Comments
Done a PR to see if we can update the image: docker-library/memcached#79 It was tested locally and the Pod worked successfully without RunAsUser as expected:
|
blocked until docker-library/memcached#79 merges, but @camilamacedo86 will keep an eye on it. |
I am unable to get the PR docker-library/memcached#79 merged. Therefore, the other solution is to maintain a short Dockerfile that's auto-built with just FROM memcached \n USER 11211:11211 (if it has to be baked into the image itself) Remove the assignment to allow others to work on this one. |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
Rotten issues close after 30d of inactivity. Reopen the issue by commenting /close |
@openshift-bot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
What is the problem
I cannot run the samples on Openshift
Why
Because we are setting RunAsUser(1001) therefore, it will not work on OCP
Proposed Solution
Change the Memcached image: docker-library/memcached#79 ( To have the USER ID define )
OR
maintain a short Dockerfile that's auto-built with just FROM memcached \n USER 11211:11211 (if it has to be baked into the image itself)
The text was updated successfully, but these errors were encountered: