Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Look up host config registry by hostname or by address #478

Closed
wants to merge 1 commit into from

Conversation

guineveresaenger
Copy link
Contributor

For login via host auth config, we have discovered that what is stored
in the credentials location is a lookup map, which sometimes contains
the registry host name, and sometimes the registry hostname with an
https:// prefix. We have not root caused this discrepancy but it seems
to be the former on Linux and the latter on MacOs.

Fixes #472.

Adds an extra lookup with https:// prefix to the authConfig lookup

Adds an extra lookup with `https://` prefix to the authConfig lookup
@github-actions
Copy link

github-actions bot commented Feb 2, 2023

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

1 similar comment
@github-actions
Copy link

github-actions bot commented Feb 2, 2023

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@squaremo
Copy link

squaremo commented Feb 3, 2023

For the bits of Flux that deal with image registries, I ended up using https://github.com/google/go-containerregistry/, which deals with all the details of authorisation from various sources, as well as listing tags, and pushing and pulling images. Much easier than docker/distribution (which the first version of Flux used -- it sucked getting that to work).

@guineveresaenger
Copy link
Contributor Author

Closing as fixed via #480

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.

"https://" prefix on push config is required on some systems and not on others
2 participants