Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SPARK-25957][K8S] Make building alternate language binding docker images optional #23053

Closed
wants to merge 6 commits into from
Closed

[SPARK-25957][K8S] Make building alternate language binding docker images optional #23053

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
ae1d743
[SPARK-25957][K8S] Add ability to skip building optional k8s docker i…
ramaddepally Nov 15, 2018
8887b5a
Address review comments
ramaddepally Nov 16, 2018
11080c9
Make building alternate language binding images opt-in rather than de…
ramaddepally Nov 20, 2018
5ec7cb7
Update usage
ramaddepally Nov 20, 2018
7601cbc
Update setup-integration-test-env.sh to build PySpark and R images as…
ramaddepally Nov 20, 2018
86a11a8
Update docs/running-on-kubernetes.md file to reflect the changes
ramaddepally Nov 21, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
56 changes: 35 additions & 21 deletions bin/docker-image-tool.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,18 @@ function image_ref {
echo "$image"
}

function docker_push {
local image_name="$1"
if [ ! -z $(docker images -q "$(image_ref ${image_name})") ]; then
docker push "$(image_ref ${image_name})"
if [ $? -ne 0 ]; then
error "Failed to push $image_name Docker image."
ramaddepally marked this conversation as resolved.
Show resolved Hide resolved
fi
else
echo "$(image_ref ${image_name}) image not found. Skipping push for this image."
ramaddepally marked this conversation as resolved.
Show resolved Hide resolved
fi
}

function build {
local BUILD_ARGS
local IMG_PATH
Expand Down Expand Up @@ -102,33 +114,33 @@ 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 [ "${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."
ramaddepally marked this conversation as resolved.
Show resolved Hide resolved
fi
else
echo "Skipped building PySpark docker image."
fi

if [ "${RDOCKERFILE}" != "skip" ] && [ -d "${SPARK_HOME}/R/lib" ]; then
docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
-t $(image_ref spark-r) \
-f "$RDOCKERFILE" .
if [ $? -ne 0 ]; then
error "Failed to build PySpark Docker image, please refer to Docker build output for details."
error "Failed to build SparkR Docker image, please refer to Docker build output for details."
fi
docker build $NOCACHEARG "${BINDING_BUILD_ARGS[@]}" \
-t $(image_ref spark-r) \
-f "$RDOCKERFILE" .
if [ $? -ne 0 ]; then
error "Failed to build SparkR Docker image, please refer to Docker build output for details."
else
echo "Skipped building SparkR docker image."
ramaddepally marked this conversation as resolved.
Show resolved Hide resolved
fi
}

function push {
docker push "$(image_ref spark)"
if [ $? -ne 0 ]; then
error "Failed to push Spark JVM Docker image."
fi
docker push "$(image_ref spark-py)"
if [ $? -ne 0 ]; then
error "Failed to push PySpark Docker image."
fi
docker push "$(image_ref spark-r)"
if [ $? -ne 0 ]; then
error "Failed to push SparkR Docker image."
fi
docker_push "spark"
docker_push "spark-py"
docker_push "spark-r"
}

function usage {
Expand All @@ -145,6 +157,8 @@ Options:
-f file Dockerfile to build for JVM based Jobs. By default builds the Dockerfile shipped with Spark.
-p file Dockerfile to build for PySpark Jobs. Builds Python dependencies and ships with Spark.
-R file Dockerfile to build for SparkR Jobs. Builds R dependencies and ships with Spark.
-pskip Skip building PySpark docker image.
ramaddepally marked this conversation as resolved.
Show resolved Hide resolved
-Rskip Skip building SparkR docker image.
-r repo Repository address.
-t tag Tag to apply to the built image, or to identify the image to be pushed.
-m Use minikube's Docker daemon.
Expand Down