Github user rvesse commented on a diff in the pull request:
https://github.com/apache/spark/pull/23053#discussion_r234967639
--- Diff: bin/docker-image-tool.sh ---
@@ -102,33 +114,37 @@ function build {
error "Failed to build Spark JVM Docker image, please refer to Docker
build output for details."
fi
- docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
- -t $(image_ref spark-py) \
- -f "$PYDOCKERFILE" .
- if [ $? -ne 0 ]; then
- error "Failed to build PySpark Docker image, please refer to Docker
build output for details."
+ if [ "${PYDOCKERFILE}" != "skip" ]; then
+ docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
+ -t $(image_ref spark-py) \
+ -f "$PYDOCKERFILE" .
+ if [ $? -ne 0 ]; then
+ error "Failed to build PySpark Docker image, please refer to
Docker build output for details."
--- End diff --
Script should bail out here, i.e. `exit` if the `docker build` fails
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
