-
Notifications
You must be signed in to change notification settings - Fork 4
part of docker containers should run with non root user #136
Conversation
@@ -44,7 +57,6 @@ terraform.debug: | |||
terraform.init: gomplate terraform | |||
@ \ | |||
cd $(ENV_DIR) && \ | |||
mkdir -p $(ENV_DIR)/.terraform && \ |
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.
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.
we should remove this row since running terraform container with non-root user create .terraform
folder with correct permission
@kobrikx ?
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.
Seems like this should be safe to remove
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.
Looks great! In the future it would be great to produce junit.xml compatible file and publish tests to github.
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.
LGTM! Works fine for me
Description
part of containers should run with non-root user
Tests
covered within unit-tests