-
Notifications
You must be signed in to change notification settings - Fork 3
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
DATAGO-81660: event-management-agent Dockerfile must account for correct user and user home directory setup #196
DATAGO-81660: event-management-agent Dockerfile must account for correct user and user home directory setup #196
Conversation
@@ -1,8 +1,20 @@ | |||
ARG BASE_IMAGE | |||
FROM ${BASE_IMAGE} | |||
|
|||
# default user name is "emauser" | |||
ARG USER=emauser |
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.
Note: you can override the username by --build-arg USER=someuser
|
||
COPY .terraformrc /root/.terraformrc | ||
# switch to the created user from root | ||
USER $USER |
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.
This is the earliest we can switch to user as line 24 requires to use apk
SonarQube Quality Gate |
What is the purpose of this change?
Implement asks of https://sol-jira.atlassian.net/browse/DATAGO-81660
How was this change implemented?
By changing the docker file
emauser
(Note: the username can be overiddeen by passing the arfument to the Dockerfile).terraformrc
file is copied to the/home/emauser
directoryemauser
How was this change tested?
Tested configpush and scan for :
Is there anything the reviewers should focus on/be aware of?