Skip to content

gcp docker auth will parse the devcontainer dir to look for artifact registry url#315

Merged
yuhuyoyo merged 5 commits intomasterfrom
yuhuyoyo/dockerauthgcp
Feb 4, 2026
Merged

gcp docker auth will parse the devcontainer dir to look for artifact registry url#315
yuhuyoyo merged 5 commits intomasterfrom
yuhuyoyo/dockerauthgcp

Conversation

@yuhuyoyo
Copy link
Copy Markdown
Contributor

@yuhuyoyo yuhuyoyo commented Feb 3, 2026

currently we by default docker auth to the user's workspace location. However, users may bring their own image from another region. Instead of authing to all the regions, we parse the devcontainer to make a best effort attempt to get the image url. For backward compatibility, we will still take in the default region of the workspace. this will replace this inline script.

@yuhuyoyo yuhuyoyo marked this pull request as ready for review February 3, 2026 12:29
@yuhuyoyo yuhuyoyo requested review from a team as code owners February 3, 2026 12:29
Copy link
Copy Markdown
Contributor

@pantherman594 pantherman594 left a comment

Choose a reason for hiding this comment

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

This should be fine for the simple case, but won't work if a devcontainer references some common files outside of the folder (like we do with aou-common and jupyter-common). Why don't we want to auth to all regions?

Long term we could maybe expose artifactRegistryLocations in the UI

@yuhuyoyo
Copy link
Copy Markdown
Contributor Author

yuhuyoyo commented Feb 3, 2026

ong term we could maybe expose artifactRegistry

ya the description says best effort.. it took a long time to list them.. I noticed that it first pull a gcloud image

@yuhuyoyo yuhuyoyo merged commit 98d507b into master Feb 4, 2026
15 checks passed
@yuhuyoyo yuhuyoyo deleted the yuhuyoyo/dockerauthgcp branch February 4, 2026 12:55
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.

2 participants