Github user ramaddepally commented on a diff in the pull request:

    https://github.com/apache/spark/pull/23172#discussion_r237272661
  
    --- Diff: project/SparkBuild.scala ---
    @@ -494,7 +494,12 @@ object KubernetesIntegrationTests {
         dockerBuild := {
           if (shouldBuildImage) {
             val dockerTool = s"$sparkHome/bin/docker-image-tool.sh"
    -        val cmd = Seq(dockerTool, "-m", "-t", imageTag.value, "build")
    +        val bindingsDir = 
s"$sparkHome/resource-managers/kubernetes/docker/src/main/dockerfiles/spark/bindings"
    +        val cmd = Seq(dockerTool, "-m",
    +          "-t", imageTag.value,
    +          "-p", s"$bindingsDir/python/Dockerfile",
    --- End diff --
    
    Don't we need to build SparkR image here? 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to