Split integration-short so commits to main run the full cloud suite#5705
Open
pietern wants to merge 2 commits into
Open
Split integration-short so commits to main run the full cloud suite#5705pietern wants to merge 2 commits into
pietern wants to merge 2 commits into
Conversation
integration-short did both -short (skip CloudSlow) and DATABRICKS_TEST_SKIPLOCAL=1 (skip everything with testserver coverage). Since #5644 pointed push-to-main at the PR workflow, main inherited both and stopped catching testserver-vs-cloud divergence. Split into three targets: - integration: full suite incl. CloudSlow (nightly cron, unchanged). - integration-short: -short only, for commits to main. - integration-short-skiplocal: -short + skiplocal, for PRs. The two -short targets revert to -timeout=2h; the 12h bump (#5658) is only needed for the full suite. Co-authored-by: Isaac
denik
approved these changes
Jun 24, 2026
Collaborator
Integration test reportCommit: 257ff9f
434 interesting tests: 384 MISS, 33 FAIL, 5 PANIC, 4 flaky, 4 RECOVERED, 3 SKIP, 1 KNOWN
Top 50 slowest tests (at least 2 minutes):
|
The full -short cloud suite that runs on commits to main hits the 2h timeout (#5705 integration run timed out with 384 MISS). It includes testserver-covered tests, so it's much larger than the PR suite. Keep 2h only for integration-short-skiplocal (PRs). Co-authored-by: Isaac
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
integration-shortdid both-short(skipCloudSlow) andDATABRICKS_TEST_SKIPLOCAL=1(skip everything with testserver coverage). Since #5644 pointed push-to-main at the PR workflow, main inherited both and stopped catching testserver-vs-cloud divergence.Split into three targets:
integration— full suite incl.CloudSlow(nightly cron, unchanged).integration-short—-shortonly, for commits to main.integration-short-skiplocal—-short+ skiplocal, for PRs (whatintegration-shortused to do).The two
-shorttargets revert to-timeout=2h; the 12h bump (#5658) is only needed for the full suite. The PR workflow will be repointed tointegration-short-skiplocalseparately.This pull request and its description were written by Isaac.