Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12344 )
Change subject: [docker] Add a kudu user to the kudu image ...................................................................... [docker] Add a kudu user to the kudu image As a best practice application images/containers should not run as root if they don’t need to. This patch creates a kudu user and group and creates/owns the kudu specific directories. I also added a tip to the readme about removing old tags. I found myself needing to do this while working on this patch. Change-Id: Ia0d51cf8c6a585f98641de9b91d51e41dec9dbd0 Reviewed-on: http://gerrit.cloudera.org:8080/12344 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M docker/Dockerfile M docker/README.adoc 2 files changed, 17 insertions(+), 0 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12344 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia0d51cf8c6a585f98641de9b91d51e41dec9dbd0 Gerrit-Change-Number: 12344 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
