-
Notifications
You must be signed in to change notification settings - Fork 104
Support narrowly scoped registry authentication #840
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
Labels
Milestone
Comments
Related to #836 |
This requires a redesign of the auth module, which further requires a design revisit. |
Closed
1 task
chmeliik
added a commit
to chmeliik/build-definitions
that referenced
this issue
Apr 24, 2025
This script is already installed in the 'oras' and 'build-trusted-artifacts' container images to work around oras not supporting containers-auth.json properly [1]. Add the script to the appstudio-utils image as well, which will enable us to also use this workaround for cosign. Cosign uses the go-containerregistry module for authentication, which works a bit better than oras, but still not well enough [2]. The script was copied from the build-trusted-artifacts repo [3]. [1]: oras-project/oras-go#840 [2]: google/go-containerregistry#1966 [3]: https://github.com/konflux-ci/build-trusted-artifacts/blob/ddb050d092df562860fe2522d4de234ed49dabd3/select-oci-auth.sh Signed-off-by: Adam Cmiel <acmiel@redhat.com>
github-merge-queue bot
pushed a commit
to konflux-ci/build-definitions
that referenced
this issue
Apr 25, 2025
This script is already installed in the 'oras' and 'build-trusted-artifacts' container images to work around oras not supporting containers-auth.json properly [1]. Add the script to the appstudio-utils image as well, which will enable us to also use this workaround for cosign. Cosign uses the go-containerregistry module for authentication, which works a bit better than oras, but still not well enough [2]. The script was copied from the build-trusted-artifacts repo [3]. [1]: oras-project/oras-go#840 [2]: google/go-containerregistry#1966 [3]: https://github.com/konflux-ci/build-trusted-artifacts/blob/ddb050d092df562860fe2522d4de234ed49dabd3/select-oci-auth.sh Signed-off-by: Adam Cmiel <acmiel@redhat.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Global access might not be granted for an entire registry. Instead, multiple service accounts/robots may be used for narrowly scoped push/pull actions. Registry authentication should be done from most-specific to least-specific.
https://github.com/containers/image/blob/main/docs/containers-auth.json.5.md#format
This is similar to the proposed change in google/go-containerregistry#1966.
The text was updated successfully, but these errors were encountered: