Skip to content

Infer Docker registry host from workspace profile - #6781

Closed
yolocs wants to merge 1 commit into
databricks:mainfrom
yolocs:chen-shou_data/stack/auth-docker-infer-region-hosts
Closed

yolocs wants to merge 1 commit into
databricks:mainfrom
yolocs:chen-shou_data/stack/auth-docker-infer-region-hosts

Conversation

@yolocs

@yolocs yolocs commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

🥞 Stacked PR (generated by git ns)

Use this link to review incremental changes.


Changes

Infer the Artifact Registry region for auth docker configure from the workspace metastore, while retaining the deprecated --region override for one release. Add auth docker host --profile <name> to report the registry hostname and whether its credential helper is configured.

Why

Users should not need to discover and manually supply a workspace's home region or reconstruct its registry hostname.

Tests

  • ./task checks
  • go test ./libs/dockercredentials ./cmd/auth ./cmd/root -count=1
  • go test ./acceptance -run '^TestAccept/cmd/auth/docker/help$' -count=1 -timeout=10m

This PR was written by Codex.

@yolocs
yolocs force-pushed the chen-shou_data/stack/auth-docker-infer-region-hosts branch from 7bc25dc to d4fef20 Compare September 21, 2026 17:21
@yolocs
yolocs marked this pull request as ready for review September 21, 2026 17:21
@yolocs
yolocs marked this pull request as draft September 21, 2026 17:21
@github-actions

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: 6781
  • Commit SHA: d4fef205e276b931713ae3c07a131d34f8940685

Checks will be approved automatically on success.

@github-actions

Copy link
Copy Markdown
Contributor

Approval status: pending

/cmd/auth/ - needs approval

6 files changed
Suggested: @simonfaltum
Also eligible: @renaudhartert-db, @hectorcast-db, @parthban-db, @tanmay-db, @Divyansh-db, @tejaskochar-db, @mihaimitrea-db, @chrisst, @rauchy

General files (require maintainer)

6 files changed
Based on git history:

  • @pietern -- recent work in cmd/auth/docker/, libs/dockercredentials/, .nextchanges/cli/

Any maintainer (@andrewnester, @anton-107, @denik, @pietern, @shreyas-goenka, @simonfaltum, @renaudhartert-db, @janniklasrose, @lennartkats-db, @rugpanov, @rclarey) can approve all areas.
See OWNERS for ownership rules.

@yolocs yolocs closed this Sep 21, 2026
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.

1 participant