Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12518 )
Change subject: [docker] Use only the kudu binary in the kudu image ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/12518/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/12518/2//COMMIT_MSG@13 PS2, Line 13: donwloaded downloaded http://gerrit.cloudera.org:8080/#/c/12518/2/docker/Dockerfile File docker/Dockerfile: http://gerrit.cloudera.org:8080/#/c/12518/2/docker/Dockerfile@233 PS2, Line 233: COPY --from=build /kudu/build/latest/bin/kudu ./ Consider adding symlinks for kudu-master and kudu-tserver for folks used to the old binaries, and to solve the log_filename problem. Actually then you wouldn't need the entrypoint change at all. -- To view, visit http://gerrit.cloudera.org:8080/12518 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I406d18790b17307e9207a5fa67bd733d4689470d Gerrit-Change-Number: 12518 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Tue, 19 Feb 2019 22:57:46 +0000 Gerrit-HasComments: Yes
