This is an automated email from the ASF dual-hosted git repository.

vanzin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 6b3c832  [SPARK-26882] Check the Kubernetes integration tests scalatyle
6b3c832 is described below

commit 6b3c832dacbc724e6e4e60b3009d165ad281fbaf
Author: Holden Karau <hol...@pigscanfly.ca>
AuthorDate: Tue Feb 19 13:49:47 2019 -0800

    [SPARK-26882] Check the Kubernetes integration tests scalatyle
    
    ## What changes were proposed in this pull request?
    
    Add the kubernetes integration tests to the scalastyle profiles.
    
    ## How was this patch tested?
    
    Run ./dev/scalastyle with a bad change manually
    
    ## Follow on work
    
    See SPARK-26898 to add scalastyle for k8s integration to the CI
    
    Closes #23792 from 
holdenk/SPARK-26882-check-k8s-integration-tests-when-linting.
    
    Authored-by: Holden Karau <hol...@pigscanfly.ca>
    Signed-off-by: Marcelo Vanzin <van...@cloudera.com>
---
 dev/scalastyle                                                   | 1 +
 .../deploy/k8s/integrationtest/backend/minikube/Minikube.scala   | 9 ++++++---
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/dev/scalastyle b/dev/scalastyle
index ff6dba5..d5cebb3 100755
--- a/dev/scalastyle
+++ b/dev/scalastyle
@@ -29,6 +29,7 @@ ERRORS=$(echo -e "q\n" \
         -Phive-thriftserver \
         -Pspark-ganglia-lgpl \
         -Pdocker-integration-tests \
+       -Pkubernetes-integration-tests \
         scalastyle test:scalastyle \
     | awk '{if($1~/error/)print}' \
 )
diff --git 
a/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/backend/minikube/Minikube.scala
 
b/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/backend/minikube/Minikube.scala
index 58aa177..8f406d8 100644
--- 
a/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/backend/minikube/Minikube.scala
+++ 
b/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/backend/minikube/Minikube.scala
@@ -66,9 +66,12 @@ private[spark] object Minikube extends Logging {
     val kubernetesConf = new ConfigBuilder()
       .withApiVersion("v1")
       .withMasterUrl(kubernetesMaster)
-      .withCaCertFile(Paths.get(userHome, MINIKUBE_PATH, 
"ca.crt").toFile.getAbsolutePath)
-      .withClientCertFile(Paths.get(userHome, MINIKUBE_PATH, 
"apiserver.crt").toFile.getAbsolutePath)
-      .withClientKeyFile(Paths.get(userHome, MINIKUBE_PATH, 
"apiserver.key").toFile.getAbsolutePath)
+      .withCaCertFile(
+        Paths.get(userHome, MINIKUBE_PATH, "ca.crt").toFile.getAbsolutePath)
+      .withClientCertFile(
+        Paths.get(userHome, MINIKUBE_PATH, 
"apiserver.crt").toFile.getAbsolutePath)
+      .withClientKeyFile(
+        Paths.get(userHome, MINIKUBE_PATH, 
"apiserver.key").toFile.getAbsolutePath)
       .build()
     new DefaultKubernetesClient(kubernetesConf)
   }


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

Reply via email to