Uh oh!
There was an error while loading. Please reload this page.
Unhide auth logout and add traceability comments - #4719
Merged
Conversation
The auth logout feature (PRs #4613, #4616, #4647) was squashed into a single commit cb3c326 titled after #4647 only, losing traceability for the new command itself. This followup: - Adds a comment block to logout.go linking each original PR with its commit range and purpose. - Adds a NEXT_CHANGELOG entry for auth logout under CLI. - Removes Hidden: true so the command appears in help and completion. - Aligns the Long description with the public documentation.
mihaimitrea-dbtemporarily deployed
to
test-trigger-is
March 12, 2026 20:02 — with 
GitHub Actions
Inactive
Collaborator
Commit: be532db
16 interesting tests: 7 KNOWN, 7 SKIP, 2 flaky
Top 27 slowest tests (at least 2 minutes):
|
mihaimitrea-db
marked this pull request as ready for review
March 13, 2026 09:18
mihaimitrea-db
requested review from
andrewnester, anton-107, denik, pietern, shreyas-goenka and simonfaltum
as code ownersMarch 13, 2026 09:18
mihaimitrea-dbtemporarily deployed
to
test-trigger-is
March 13, 2026 10:14 — with GitHub Actions
Inactive
simonfaltum
left a comment
Member
There was a problem hiding this comment.
Review from OpenCode + Isaac (2-round swarm review).
Verdict: Approved with a few suggestions on the help text.
0 Critical | 0 Major | 0 Gap | 4 Nit | 1 Suggestion
Uh oh!
There was an error while loading. Please reload this page.
Comment on lines
+60
to
+75
| Command behavior: | ||
| 1. If you specify --profile, the command logs out of that profile. In an | ||
| interactive terminal you'll be asked to confirm unless --force is set. | ||
| 2. If you omit --profile in an interactive terminal, you'll be shown | ||
| an interactive picker listing all profiles from your configuration file. | ||
| You can search by profile name, host, or account ID. After selecting a | ||
| profile, you'll be asked to confirm unless --force is specified. | ||
| interactive terminal, it asks for confirmation unless you also specify | ||
| --force. | ||
| 3. If you omit --profile in a non-interactive environment (e.g. CI/CD pipeline), | ||
| the command will fail with an error asking you to specify --profile. | ||
| 2. If you omit --profile in an interactive terminal, the command shows a | ||
| searchable profile picker. You can search by profile name, host, or | ||
| account ID. After you select a profile, the command asks for confirmation | ||
| unless you also specify --force. | ||
| 4. Use --force to skip the confirmation prompt. This is required when | ||
| running in non-interactive environments. | ||
| 3. If you omit --profile in a non-interactive environment, the command fails | ||
| and asks you to specify --profile. | ||
| 5. Use --delete to also remove the selected profile from ~/.databrickscfg.`, | ||
| 4. In a non-interactive environment, use --profile together with --force to | ||
| skip confirmation.`, |
Member
There was a problem hiding this comment.
[Nit] The old description had 5 numbered items with a dedicated point for --delete. The new list covers --force (item 4) but drops --delete, which is only mentioned in the opening paragraph. This creates an asymmetry where --force gets its own item but --delete does not. Consider adding an item like: "Use --delete to also remove the profile from the configuration file."
Also: I believe we are introducing a global --yes flag in a separate PR. Once that lands, --force here should probably be replaced with --yes for consistency. Worth noting as a follow-up.
ContributorAuthor
There was a problem hiding this comment.
Yes, I am waiting for the --yes PR to be merged. Then I will create a PR for that change to ensure we are not releasing this command while also making changes to it. After that is fine we can merge this one.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
simonfaltumtemporarily deployed
to
test-trigger-is
March 14, 2026 21:46 — with GitHub Actions
Inactive
Clarify shared token cleanup and align the help text with the actual non-interactive and --delete behavior so the public description is less misleading.
mihaimitrea-dbtemporarily deployed
to
test-trigger-is
March 16, 2026 08:56 — with GitHub Actions
Inactive
…ogout-followup # Conflicts: # cmd/auth/logout.go
…ogout-followup # Conflicts: # cmd/auth/logout.go
simonfaltum
left a comment
Member
There was a problem hiding this comment.
where is the actual "unhiding" happening? this is all text updates?
| // The auth logout command was implemented across three stacked PRs that were | ||
| // inadvertently squashed into a single commit cb3c326 (titled after #4647 only): | ||
| // | ||
| // - #4613: core logout command with --profile, --force, |
- Remove Hidden: true so the command appears in help and tab completion. - Add paragraph about token cleanup only applying to databricks-auth-login profiles and shared token handling. - Update traceability comment to reflect --force → --auto-approve rename. - Fix changelog bullet style (- to *) for consistency.
simonfaltum
approved these changes
Apr 10, 2026
Uh oh!
There was an error while loading. Please reload this page.
denik pushed a commit
that referenced
this pull request
May 20, 2026
## Changes - Add a traceability comment block at the top of `cmd/auth/logout.go` linking to the three original PRs (#4613, #4616, #4647). These PRs were inadvertently squashed into a single commit `48eb749` titled after #4647 only. - The comment reflects the `--force` → `--auto-approve` rename from #4755. - Add a `NEXT_CHANGELOG.md` entry for `auth logout` under the CLI section. - Remove `Hidden: true` from the logout command so it appears in `databricks auth -h` and tab completion. - Improve the command's long description to: - Document that token cleanup only applies to profiles created by `databricks auth login` - Note that logging out of one profile does not affect others sharing the same cached token - Condense the numbered items (drop the redundant `--auto-approve` item since it's already mentioned in items 1 and 2; fold `--delete` into a single item) ## Why The `auth logout` feature was implemented across three stacked PRs (#4613, #4616, #4647) that were squashed into one commit on merge. The commit title only references the last PR ("Extract shared SelectProfile helper"), so `auth logout` itself has no changelog entry and no git-blame traceability. This followup restores that context, unhides the command for users, and adds the missing changelog entry. ## Tests No functional changes; existing unit and acceptance tests for `auth logout` continue to pass. --------- Co-authored-by: simon <simon.faltum@databricks.com>
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 freeto 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.
Changes
cmd/auth/logout.golinking to the three original PRs (Add auth logout command with --profile and --force flags #4613, Add interactive profile picker to auth logout #4616, Extract shared SelectProfile helper to eliminate duplicate profile pickers #4647). These PRs were inadvertently squashed into a single commitcb3c326titled after Extract shared SelectProfile helper to eliminate duplicate profile pickers #4647 only.--force→--auto-approverename from Switchauth logoutfrom--forceto--auto-approve#4755.NEXT_CHANGELOG.mdentry forauth logoutunder the CLI section.Hidden: truefrom the logout command so it appears indatabricks auth -hand tab completion.databricks auth login--auto-approveitem since it's already mentioned in items 1 and 2; fold--deleteinto a single item)Why
The
auth logoutfeature was implemented across three stacked PRs (#4613, #4616, #4647) that were squashed into one commit on merge. The commit title only references the last PR ("Extract shared SelectProfile helper"), soauth logoutitself has no changelog entry and no git-blame traceability.This followup restores that context, unhides the command for users, and adds the missing changelog entry.
Tests
No functional changes; existing unit and acceptance tests for
auth logoutcontinue to pass.