Repository navigation
Add --resume and --timeout to databricks pipelines test - #7426
Open
tharunbhuk-db wants to merge 1 commit into
Open
tharunbhuk-db wants to merge 1 commit into
tharunbhuk-db wants to merge 1 commit into
Conversation
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-pr2-output
branch
from
November 10, 2026 00:25
23e0bb0 to
b45969f
Compare
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-a4-run-control
branch
from
November 10, 2026 00:25
cd64420 to
17f12d7
Compare
This was referenced Nov 10, 2026
Contributor
|
Commit: Regression test reportSelected 13 changed tests (limit=20, 0 not selected) 13 tests: regression=13 unreleased=0 coverage=0 failing=0
TestAccept/pipelines/databricks-cli-help: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): same output as main (688c228) TestAccept/pipelines/test/arguments: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): TestAccept/pipelines/test/immutable: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): same output as main (688c228) TestAccept/pipelines/test/pipeline-id: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): same output as main (688c228) TestAccept/pipelines/test/results/failed: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): same output as main (688c228) TestAccept/pipelines/test/results/incomplete: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): same output as main (688c228) TestAccept/pipelines/test/results/passed: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): same output as main (688c228) TestAccept/pipelines/test/run-control/canceled: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): same output as main (688c228) TestAccept/pipelines/test/run-control/not-test-update: branch ✅ | main (688c228) ❌ | latest (v1.25.0) ❌main (688c228): latest (v1.25.0): 4 more failure details omitted, see the full report in the run. |
Collaborator
✅ Integration test reportCommit: 17f12d7
Top 3 slowest tests (at least 2 minutes):
|
tharunbhuk-db
added a commit
that referenced
this pull request
Nov 10, 2026
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-pr2-output
branch
from
November 10, 2026 04:14
b45969f to
e8a772a
Compare
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-a4-run-control
branch
from
November 10, 2026 04:14
17f12d7 to
2f8b6d9
Compare
This was referenced Nov 10, 2026
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-pr2-output
branch
from
November 10, 2026 06:26
e8a772a to
e7de143
Compare
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-a4-run-control
branch
from
November 10, 2026 06:57
2f8b6d9 to
e225070
Compare
tharunbhuk-db
marked this pull request as ready for review
November 10, 2026 07:08
tharunbhuk-db
requested review from
pranshupand-db and
renaudhartert-db
and removed request for
a team and
renaudhartert-db
November 10, 2026 07:08
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-a4-run-control
branch
from
November 10, 2026 09:08
e225070 to
1059c51
Compare
--timeout and Ctrl+C end only the local wait and print the command that reads the results later; the update keeps running. That command keeps the KEY or --pipeline-id, --target, --profile and every --var. --resume <update-id> reads the results of an existing test update instead of starting one, and rejects an update that the service reports is not a test update. The signal handler is installed before the update starts, but only the wait listens to it, so a Ctrl+C during Start Update still prints the update ID. The first signal releases the handler, so a second one ends the process. Co-authored-by: Isaac <no-reply@databricks.com>
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-pr2-output
branch
from
November 10, 2026 17:13
e7de143 to
d0d7138
Compare
tharunbhuk-db
force-pushed
the
tharun/pipelines-test-a4-run-control
branch
from
November 10, 2026 17:14
1059c51 to
66654a5
Compare
pranshupand-db
approved these changes
Nov 10, 2026
This branch has not been deployed
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
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.
🥞 Stacked PR
--timeoutand Ctrl+C stop the local wait and print the--resumecommand; the update keeps running.--resume <update-id>reads an existing test update.Tests: unit tests; acceptance in
acceptance/pipelines/test/run-control/*.This pull request and its description were written by Isaac.