Repository: spark
Updated Branches:
  refs/heads/master e017cb396 -> 1fb3759f2


[SPARK-25930][K8S] Fix scala string detection in k8s tests

## What changes were proposed in this pull request?

- Issue is described in detail in 
[SPARK-25930](https://issues.apache.org/jira/browse/SPARK-25930). Since we rely 
on the std output, pick always the last line which contains the wanted value. 
Although minor, current implementation breaks tests.

## How was this patch tested?
manually. rm -rf ~/.m2 and then run the tests.

Closes #22931 from skonto/fix_scala_detection.

Authored-by: Stavros Kontopoulos <stavros.kontopou...@lightbend.com>
Signed-off-by: Sean Owen <sean.o...@databricks.com>


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/1fb3759f
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/1fb3759f
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/1fb3759f

Branch: refs/heads/master
Commit: 1fb3759f2b60a2e7c5e2a82afe1a580d848e0f8c
Parents: e017cb3
Author: Stavros Kontopoulos <stavros.kontopou...@lightbend.com>
Authored: Mon Nov 5 08:40:25 2018 -0600
Committer: Sean Owen <sean.o...@databricks.com>
Committed: Mon Nov 5 08:40:25 2018 -0600

----------------------------------------------------------------------
 .../integration-tests/dev/dev-run-integration-tests.sh        | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/1fb3759f/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
----------------------------------------------------------------------
diff --git 
a/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
 
b/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
index 3c7cc93..68f284c 100755
--- 
a/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
+++ 
b/resource-managers/kubernetes/integration-tests/dev/dev-run-integration-tests.sh
@@ -29,7 +29,12 @@ SERVICE_ACCOUNT=
 CONTEXT=
 INCLUDE_TAGS="k8s"
 EXCLUDE_TAGS=
-SCALA_VERSION="$($TEST_ROOT_DIR/build/mvn 
org.apache.maven.plugins:maven-help-plugin:2.1.1:evaluate 
-Dexpression=scala.binary.version | grep -v '\[' )"
+MVN="$TEST_ROOT_DIR/build/mvn"
+
+SCALA_VERSION=$("$MVN" help:evaluate -Dexpression=scala.binary.version 
2>/dev/null\
+    | grep -v "INFO"\
+    | grep -v "WARNING"\
+    | tail -n 1)
 
 # Parse arguments
 while (( "$#" )); do


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

Reply via email to