Tim Armstrong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/13899


Change subject: IMPALA-8622,IMPALA-8696: fix docker dependencies, add image list
......................................................................

IMPALA-8622,IMPALA-8696: fix docker dependencies, add image list

Adds a plain-text space-separated image list in
docker/docker-images.txt. This is generated based on the images built by
CMake, so is kept in sync with images added to or removed from the
CMake file.

Duplicated logic per image is removed - instead there is a helper
function that is called for each daemon image to be built.

Rips out the timestamp mechanism that was intended to avoid unnecessary
container rebuilds, but has turned out to be brittle. Instead the
containers are rebuilt each time the rule is invoked.

This moves some subdirectories so that the image tag matches the
subdirectory, to simplify the build scripts.

Change-Id: I4d8e215e9b07c6491faa4751969a30f0ed373fe3
---
M .gitignore
M docker/CMakeLists.txt
R docker/impalad_coord_exec/Dockerfile
R docker/impalad_coordinator/Dockerfile
R docker/impalad_executor/Dockerfile
M docker/push-images.sh
6 files changed, 35 insertions(+), 101 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/99/13899/2
--
To view, visit http://gerrit.cloudera.org:8080/13899
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4d8e215e9b07c6491faa4751969a30f0ed373fe3
Gerrit-Change-Number: 13899
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>

Reply via email to