Skip to content

fix: chain registry auth properly#122

Merged
pspurlock merged 1 commit intomainfrom
fix-data-tool-reg-auth
Apr 25, 2026
Merged

fix: chain registry auth properly#122
pspurlock merged 1 commit intomainfrom
fix-data-tool-reg-auth

Conversation

@ktarplee
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Kyle M. Tarplee <kyle.tarplee@udri.udayton.edu>
@ktarplee ktarplee requested a review from pspurlock April 25, 2026 17:55
@ktarplee ktarplee self-assigned this Apr 25, 2026
Comment thread data-tool/main.go
regConfig := dag.RegistryConfig().WithRegistryAuth(address, username, secret)
m.Container = m.Container.WithMountedSecret("/root/.docker/config.json", regConfig.Secret())
m.RegistryConfig = m.RegistryConfig.WithRegistryAuth(address, username, secret)
m.Container = m.Container.WithMountedSecret("/root/.docker/config.json", m.RegistryConfig.Secret())
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We might consider skipping this step and then adding a function that returns a container that mounts this once. The Container field would need to be private in that case.

@pspurlock pspurlock merged commit 2b92a3b into main Apr 25, 2026
1 check passed
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