Laszlo Gaal has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/24645


Change subject: IMPALA-15230: Restrict the check for "tests" to the jar name 
proper
......................................................................

IMPALA-15230: Restrict the check for "tests" to the jar name proper

...during Docker image builds.

The context builder script that runs during Impala Docker image builds
screens out test jars by checking the file name for the "tests" substring
for certain jars. This check is too aggressive: it triggers if any
part of the full pathname, even an enclosing directory name
contains "tests", breaking the container build.

This patch constrains the check to the jar name itself by splitting
off the basename of the jar before checking. It also converts the
assertion into an Exception with a message so that the check is not
skipped if the script happens to run in Python's optimized mode.

Change-Id: Idb783e3c7dfc064d7a6ed6804e429393a0de3cbf
---
M docker/setup_build_context.py
1 file changed, 9 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/45/24645/1
--
To view, visit http://gerrit.cloudera.org:8080/24645
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idb783e3c7dfc064d7a6ed6804e429393a0de3cbf
Gerrit-Change-Number: 24645
Gerrit-PatchSet: 1
Gerrit-Owner: Laszlo Gaal <[email protected]>

Reply via email to