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.

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

Reactivate tests removed in #3259 #3263

Merged
merged 4 commits into from Nov 23, 2021
Merged

Conversation

EnricoMi
Copy link
Collaborator

@EnricoMi EnricoMi commented Nov 8, 2021

Reactivates Spark tests disabled in #3259. Fixes #3262.

@EnricoMi EnricoMi force-pushed the branch-ci-reactivate-spark-tests branch from 467c354 to 1161e09 Compare November 8, 2021 08:32
@github-actions
Copy link

github-actions bot commented Nov 8, 2021

Unit Test Results

     798 files  +  32       798 suites  +32   8h 15m 56s ⏱️ + 2h 8m 45s
     716 tests +114       672 ✔️ +106       44 💤 +    8  0 ±0 
17 172 runs  +912  12 122 ✔️ +735  5 050 💤 +177  0 ±0 

Results for commit 00c82b1. ± Comparison against base commit d395a88.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Nov 8, 2021

Unit Test Results (with flaky tests)

     946 files  +     68       946 suites  +68   9h 29m 38s ⏱️ + 2h 59m 22s
     716 tests +   114       670 ✔️ +   105       44 💤 +    8  2 +1 
20 636 runs  +1 728  14 370 ✔️ +1 254  6 262 💤 +471  4 +3 

For more details on these failures, see this check.

Results for commit 00c82b1. ± Comparison against base commit d395a88.

♻️ This comment has been updated with latest results.

@EnricoMi EnricoMi force-pushed the branch-ci-reactivate-spark-tests branch from d474890 to e887ba1 Compare November 8, 2021 18:51
@EnricoMi EnricoMi marked this pull request as ready for review November 8, 2021 18:53
@EnricoMi EnricoMi force-pushed the branch-ci-reactivate-spark-tests branch 7 times, most recently from c9c1a85 to 9e402de Compare November 10, 2021 17:25
@EnricoMi EnricoMi force-pushed the branch-ci-reactivate-spark-tests branch 4 times, most recently from e38f404 to 6ea6af5 Compare November 20, 2021 22:27
Signed-off-by: Enrico Minack <github@enrico.minack.dev>
@EnricoMi EnricoMi force-pushed the branch-ci-reactivate-spark-tests branch from 6ea6af5 to 611598a Compare November 20, 2021 22:32
Signed-off-by: Enrico Minack <github@enrico.minack.dev>
@EnricoMi EnricoMi force-pushed the branch-ci-reactivate-spark-tests branch from baca5ac to c721ab3 Compare November 21, 2021 12:56
Signed-off-by: Enrico Minack <github@enrico.minack.dev>
@EnricoMi EnricoMi force-pushed the branch-ci-reactivate-spark-tests branch from c721ab3 to 7624a27 Compare November 21, 2021 20:40
Signed-off-by: Enrico Minack <github@enrico.minack.dev>
@EnricoMi EnricoMi merged commit 57db30b into master Nov 23, 2021
@EnricoMi EnricoMi deleted the branch-ci-reactivate-spark-tests branch November 23, 2021 10:43
@EnricoMi
Copy link
Collaborator Author

In the meanwhile, commit 16933b5 sneaked into master which introduced a bug that is only caught by the formerly disabled Spark tests. Now that they are reactivated, master fails on them.

Details: #3279 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Fix flaky spark integration tests
2 participants