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 non-root user in Dockerfile #48

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

yash2189
Copy link

@yash2189 yash2189 commented Apr 24, 2024

Updated worker slice gateway sidecar Dockerfile with non-root user

Tested the changes by building the Dockerfile locally and creating a running container of gateway-sidecar

Closes #42

@yash2189 yash2189 changed the title Use non-root user in Dockerfile Update non-root user in Dockerfile Apr 24, 2024
@yash2189
Copy link
Author

Local Testing logs:
side_car_logs.txt

@narmidm
Copy link
Member

narmidm commented Apr 25, 2024

@yash2189 yash2189 force-pushed the dockerfile_changes branch 2 times, most recently from fb7f407 to a5f8d06 Compare April 25, 2024 15:53
Signed-off-by: Yash Ajgaonkar <[email protected]>
Signed-off-by: Yash Ajgaonkar <[email protected]>
@yash2189 yash2189 force-pushed the dockerfile_changes branch from a5f8d06 to 11e9c76 Compare April 25, 2024 15:56
@yash2189
Copy link
Author

@narmidm I have signed off my commit

@narmidm narmidm enabled auto-merge April 25, 2024 16:06
@narmidm narmidm disabled auto-merge April 25, 2024 16:06
@narmidm
Copy link
Member

narmidm commented May 20, 2024

@yash2189 - Could you sync your branch with the master branch once? - we have new checks on PR.

@yash2189
Copy link
Author

@narmidm I have synced the branch

@narmidm
Copy link
Member

narmidm commented May 20, 2024

LGTM! We will postpone the merging of this PR for a few days due to an ongoing release.

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.

Update worker slice gateway sidecar dockerfile with non-root user
2 participants