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

features: add authenticated registries documentation #68

Conversation

wainersm
Copy link
Member

Explain how to configure for using authenticated registries.

Related-to: #52


I tested the instructions locally.

In my explanation to why a docker-registry secret is needed, the culprint is nydus-snapshotter. But in reality I don't remember whether it's him or containerd (or both) that requires the credentials. So please check the information is accurate.

Copy link

netlify bot commented Dec 10, 2024

Deploy Preview for frolicking-manatee-96c0c8 ready!

Name Link
🔨 Latest commit 4cc6332
🔍 Latest deploy log https://app.netlify.com/sites/frolicking-manatee-96c0c8/deploys/6759cc4da0ccc40008142e33
😎 Deploy Preview https://deploy-preview-68--frolicking-manatee-96c0c8.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@wainersm wainersm force-pushed the features_authenticated_registries branch from 18c42d0 to d2f1823 Compare December 11, 2024 17:28
Explain how to configure for using authenticated registries.

Related-to: confidential-containers#52
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>
@wainersm wainersm force-pushed the features_authenticated_registries branch from d2f1823 to 4cc6332 Compare December 11, 2024 17:30
@wainersm
Copy link
Member Author

Updated to delint the article.

Copy link
Member

@fitzthum fitzthum left a comment

Choose a reason for hiding this comment

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

LGTM

@fitzthum fitzthum merged commit d85d3bc into confidential-containers:main Dec 11, 2024
7 checks 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