Repository navigation
Conversation
This was referenced Nov 13, 2026
HsingTang
force-pushed
the
HsingTang/stack/sandbox-ssh-setup
branch
from
November 13, 2026 08:25
dc841c7 to
b9424a8
Compare
HsingTang
force-pushed
the
HsingTang/stack/sandbox-ssh-port-forward
branch
from
November 13, 2026 08:25
5cd926a to
417c077
Compare
Contributor
|
Commit: Regression test reportSelected 1 changed tests (limit=20, 0 not selected) 1 tests: regression=1 unreleased=0 coverage=0 failing=0
TestAccept/cmd/sandbox/ssh-setup: branch ✅ | main (573c36e) ❌ | latest (v1.26.0) ❌main (573c36e): latest (v1.26.0): same output as main (573c36e) |
Collaborator
✅ Integration test reportCommit: a725f0c
Top 3 slowest tests (at least 2 minutes):
|
HsingTang
force-pushed
the
HsingTang/stack/sandbox-ssh-setup
branch
from
November 13, 2026 21:43
b9424a8 to
c26a360
Compare
HsingTang
force-pushed
the
HsingTang/stack/sandbox-ssh-port-forward
branch
from
November 13, 2026 21:43
417c077 to
5441e69
Compare
HsingTang
force-pushed
the
HsingTang/stack/sandbox-ssh-setup
branch
from
November 13, 2026 22:12
c26a360 to
a725f0c
Compare
HsingTang
force-pushed
the
HsingTang/stack/sandbox-ssh-port-forward
branch
from
November 13, 2026 22:12
5441e69 to
9434fda
Compare
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
Changes
Add
databricks sandbox ssh-setup SANDBOX_ID --name ALIAS --profile PROFILEto write a local SSH Host configuration using the existing sandbox proxy mode. Nativessh,scp, andsftpcan then connect using that alias. Setup requires an explicit Databricks profile, reuses local keys, retains a selected custom Databricks configuration file, and preserves unrelated SSH configuration. It does not start the sandbox.Sandbox SSH configuration helpers live alongside the other new sandbox implementation under
experimental/ssh/sandbox.Why
Support native SSH tools through the same authenticated forwarding path as
databricks sandbox ssh. Interactive CLI SSH continues to work without a persistent Host entry or an explicit setup step.Tests
Unit tests cover profile requirements, alias validation, idempotency, overwrite protection, and preservation of unrelated configuration. Installed OpenSSH parses the generated configuration. Acceptance tests exercise setup, repeated setup, saved connection settings, and invalid input.
./task test-sandbox test-exp-ssh, affected race tests, and./task fmt checks lintpass. The package-layout change was reviewed locally with Isaac, with no blocking findings.This PR was written with Codex.