Skip to content

Fix sandbox SSH during stop transition - #6893

Merged
pietern merged 1 commit into
databricks:mainfrom
akshaysingla-db:stack/sandbox-ssh-stopping
Oct 1, 2026
Merged

pietern merged 1 commit into
databricks:mainfrom
akshaysingla-db:stack/sandbox-ssh-stopping

Conversation

@akshaysingla-db

Copy link
Copy Markdown
Contributor

Summary

databricks sandbox ssh now waits for an in-flight stop to reach Stopped before issuing the start request. Post-start polling remains strict, so returning to Stopped after /start still surfaces as a failure.

Adds lifecycle regression coverage for initial Stopping, Stopped, and Running states, plus a failed start that returns to Stopped.

Testing

go test ./cmd/sandbox -count=1

@akshaysingla-db
akshaysingla-db marked this pull request as ready for review October 1, 2026 03:54
@akshaysingla-db
akshaysingla-db requested review from a team as code owners October 1, 2026 03:54
@akshaysingla-db
akshaysingla-db force-pushed the stack/sandbox-ssh-stopping branch from 18f749b to 18db582 Compare October 1, 2026 03:57
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

An authorized user can trigger integration tests manually by following the instructions below:

Trigger:
go/deco-tests-run/cli

Inputs:

  • PR number: 6893
  • Commit SHA: 18db58203b34a8716f937f1f1e156634490b2802

Checks will be approved automatically on success.

@akshaysingla-db
akshaysingla-db requested review from anwell-db and samhuan-db and removed request for rclarey October 1, 2026 04:06

@pietern pietern left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The SDK ships with the right enums.

Can be done as part of swapping out the DIY structs and API calls for the SDK structs and methods.

@pietern
pietern merged commit e1ddf6d into databricks:main Oct 1, 2026
18 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants