-
Notifications
You must be signed in to change notification settings - Fork 6.7k
docs: adds a section for OCI private repos in OCI dcos #25572
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
docs: adds a section for OCI private repos in OCI dcos #25572
Conversation
Signed-off-by: Patroklos Papapetrou <[email protected]>
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
blakepettersson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That comment applies if you use a "helm classic" repository credential. Private repositories are fully supported with OCI (if you use a credential of type oci)
Thanks @blakepettersson , how should I phrase it then? 🤔 |
|
@ppapapetrou76 hmm I could have sworn that there was a part for OCI repos in the docs but it seems like there isn't 🤔 There is something in the CLI docs, but that's not too helpful... Anyways using the example in the OCI docs, if there is a need to have credentials for it, a repository credential of type *oci needs to be created. This is a bit confusing, since in the case of Helm repositories there is another (incompatible) way to use OCI credentials with Helm (which is the one you documented) The confusing part here is that helm credentials are not compatible with oci credentials and vice versa, and the only way those repo types are distinguished are via their IMO I think the Helm OCI credential type should be deprecated and removed in the next major release, but that's a topic for another day. |
Signed-off-by: Patroklos Papapetrou <[email protected]>
|
thanks @blakepettersson |
blakepettersson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally LGTM, just a minor comment
Signed-off-by: Patroklos Papapetrou <[email protected]>
Signed-off-by: Patroklos Papapetrou <[email protected]> Signed-off-by: Alexandre Gaudreault <[email protected]>
Checklist: