Skip to content

Commit 8896f4a

Browse files
committed
Revert "[SPARK-32253][INFRA] Show errors only for the sbt tests of github actions"
### What changes were proposed in this pull request? This reverts commit 026b0b9. ### Why are the changes needed? As HyukjinKwon pointed out in #29133 (comment), there is no JUnit test report after #29133. Let's revert #29133 for now and find a better solution to improve the log output later. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? GitHub Actions build Closes #29219 from gengliangwang/revertErrorOnly. Authored-by: Gengliang Wang <[email protected]> Signed-off-by: Gengliang Wang <[email protected]>
1 parent 84efa04 commit 8896f4a

File tree

2 files changed

+0
-10
lines changed

2 files changed

+0
-10
lines changed

dev/run-tests.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -653,9 +653,6 @@ def main():
653653
# If we're running the tests in Github Actions, attempt to detect and test
654654
# only the affected modules.
655655
if test_env == "github_actions":
656-
# Set the log level of sbt as ERROR to make the output more readable.
657-
if build_tool == "sbt":
658-
extra_profiles.append("--error")
659656
if os.environ["GITHUB_BASE_REF"] != "":
660657
# Pull requests
661658
changed_files = identify_changed_files_from_git_commits(

project/SparkBuild.scala

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1031,13 +1031,6 @@ object TestSettings {
10311031
}.getOrElse(Nil): _*),
10321032
// Show full stack trace and duration in test cases.
10331033
testOptions in Test += Tests.Argument("-oDF"),
1034-
// Show only the failed test cases with full stack traces in github action to make the log more
1035-
// readable.
1036-
// Check https://www.scalatest.org/user_guide/using_the_runner for the details of options .
1037-
testOptions in Test += Tests.Argument(TestFrameworks.ScalaTest,
1038-
sys.env.get("GITHUB_ACTIONS").map { _ =>
1039-
Seq("-eNCXEHLOPQMDF")
1040-
}.getOrElse(Nil): _*),
10411034
testOptions in Test += Tests.Argument(TestFrameworks.JUnit, "-v", "-a"),
10421035
// Required to detect Junit tests for each project, see also https://github.com/sbt/junit-interface/issues/35
10431036
crossPaths := false,

0 commit comments

Comments
 (0)