Skip to content

Show "DEFAULT" as the default profile for databricks auth login - #3252

Merged
lennartkats-db merged 3 commits into
mainfrom
auth-login-ux
Aug 1, 2025
Merged

Show "DEFAULT" as the default profile for databricks auth login#3252
lennartkats-db merged 3 commits into
mainfrom
auth-login-ux

Conversation

@lennartkats-db

@lennartkats-dblennartkats-db commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

Changes

Show "DEFAULT" as the default profile for the databricks auth login command. This change uses the same UX as seen in the databricks bundle init command, i.e. it shows the default value between square backets rather than in the input field.

Why

This change aims to make databricks auth login more self-documenting. I want to reference it from templates in the future.

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Jul 14, 2025

Copy link
Copy Markdown
Collaborator

Run: 16276472619

Env✅‌pass🔄‌flaky🙈‌skip
✅‌aws linux302417
🔄‌aws windows3003416
🔄‌aws-ucws linux4008317
🔄‌aws-ucws windows4009316
✅‌azure linux302416
✅‌azure windows303415
🔄‌azure-ucws linux4028314
🔄‌azure-ucws windows4056313
✅‌gcp linux301418
✅‌gcp windows302417
12 failing tests:
Test Nameaws windowsaws-ucws linuxaws-ucws windowsazure-ucws linuxazure-ucws windows
TestAccept🔄‌flaky✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/deploy/dashboard/detect-change✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky✅‌pass
TestAccept/bundle/deploy/dashboard/nested-folders✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky✅‌pass
TestAccept/bundle/deploy/dashboard/simple✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky🔄‌flaky
TestAccept/bundle/deploy/dashboard/simple_outside_bundle_root✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky🔄‌flaky
TestAccept/bundle/deploy/dashboard/simple_syncroot✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky🔄‌flaky
TestAccept/bundle/deployment/bind/dashboard✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky🔄‌flaky
TestAccept/bundle/deployment/bind/dashboard/recreation✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky🔄‌flaky
TestAccept/bundle/templates/default-python/integration_classic🔄‌flaky✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/templates/default-python/integration_classic/DATABRICKS_CLI_DEPLOYMENT=terraform/UV_PYTHON=3.13🔄‌flaky✅‌pass✅‌pass✅‌pass✅‌pass
TestDashboardAssumptions_WorkspaceImport✅‌pass🔄‌flaky🔄‌flaky🔄‌flaky🔄‌flaky
TestFetchRepositoryInfoAPI_FromRepo✅‌pass✅‌pass🔄‌flaky✅‌pass✅‌pass

@lennartkats-db

Copy link
Copy Markdown
ContributorAuthor

Note that this also makes the command more similar to databricks configure which always defaults to DEFAULT

@lennartkats-db
lennartkats-db added this pull request to the merge queueAug 1, 2025
Merged via the queue into main with commit be331f8Aug 1, 2025
13 checks passed
@lennartkats-db
lennartkats-db deleted the auth-login-ux branch August 1, 2025 09:33
deco-sdk-taggingBot added a commit that referenced this pull request Aug 6, 2025
## Release v0.263.0
### CLI
* Show "DEFAULT" as the default profile for `databricks auth login` [#3252](#3252)
### Bundles
* Always set default value for source field for tasks if not explicitly set already ([#3359](#3359))
* Fix panic in auto completion handler for bundler run/open. Regression introduced in 0.259.0. [#3358](#3358)
* Fix error propagation in the `bundle generate dashboard` command. Regression introduced in 0.259.0. [#3354](#3354)
alyssa-db pushed a commit that referenced this pull request Aug 7, 2025
## Release v0.263.0
### CLI
* Show "DEFAULT" as the default profile for `databricks auth login` [#3252](#3252)
### Bundles
* Always set default value for source field for tasks if not explicitly set already ([#3359](#3359))
* Fix panic in auto completion handler for bundler run/open. Regression introduced in 0.259.0. [#3358](#3358)
* Fix error propagation in the `bundle generate dashboard` command. Regression introduced in 0.259.0. [#3354](#3354)
pietern added a commit that referenced this pull request May 11, 2026
## Summary
- Remove the `Default` field from `cmdio.PromptOptions`. No production
caller sets it; `databricks auth login` renders the default in the label
and substitutes it manually on empty input (see PR #3252). Follow-up to
#5177, which removed the now-orphaned `AllowEdit` companion field.
- Drop the corresponding `--default` flag from `databricks selftest tui
prompt`, which only existed to exercise the field.
This pull request and its description were written by Isaac.
denik pushed a commit that referenced this pull request May 20, 2026
)
## Changes
Show `"DEFAULT"` as the default profile for the `databricks auth login`
command. This change uses the same UX as seen in the `databricks bundle
init` command, i.e. it shows the default value between square backets
rather than in the input field.
## Why
This change aims to make `databricks auth login` more self-documenting.
I want to reference it from templates in the future.
denik pushed a commit that referenced this pull request May 20, 2026
## Release v0.263.0
### CLI
* Show "DEFAULT" as the default profile for `databricks auth login` [#3252](#3252)
### Bundles
* Always set default value for source field for tasks if not explicitly set already ([#3359](#3359))
* Fix panic in auto completion handler for bundler run/open. Regression introduced in 0.259.0. [#3358](#3358)
* Fix error propagation in the `bundle generate dashboard` command. Regression introduced in 0.259.0. [#3354](#3354)
denik pushed a commit that referenced this pull request May 20, 2026
## Summary
- Remove the `Default` field from `cmdio.PromptOptions`. No production
caller sets it; `databricks auth login` renders the default in the label
and substitutes it manually on empty input (see PR #3252). Follow-up to
#5177, which removed the now-orphaned `AllowEdit` companion field.
- Drop the corresponding `--default` flag from `databricks selftest tui
prompt`, which only existed to exercise the field.
This pull request and its description were written by Isaac.
TanishqDatabricks pushed a commit to TanishqDatabricks/cli that referenced this pull request May 22, 2026
## Summary
- Remove the `Default` field from `cmdio.PromptOptions`. No production
caller sets it; `databricks auth login` renders the default in the label
and substitutes it manually on empty input (see PR databricks#3252). Follow-up to
databricks#5177, which removed the now-orphaned `AllowEdit` companion field.
- Drop the corresponding `--default` flag from `databricks selftest tui
prompt`, which only existed to exercise the field.
This pull request and its description were written by Isaac.
Sign up for freeto 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

@lennartkats-db@eng-dev-ecosystem-bot@shreyas-goenka