-
Notifications
You must be signed in to change notification settings - Fork 524
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
Adding Cloudsmith Registry #1887
Conversation
Thank you for the submission @apoclyps! Your change actually gives me an idea that would allow us to improve the generic |
@bwateratmsft It would be fantastic to see this merged as generic support is worked towards, especially as we've got users asking for it and the effort was already completed. 🙏 Some of the functionality of Cloudsmith differs a little (esp. around authorisation), which is why we had to make some changes on our side to adapt; we're very willing to help collaborate on it for the next refactor! |
@apoclyps - My name is Mike Morton and I am the program manager on the Docker extension here at Microsoft. This PR has brought to our attention that the current way of adding registries to the extension does not scale well. We are in the middle or designing how we plan to support this going forward. As soon as we finalize the design we will share it with you so you can change to use the new design. |
For now I'll close this. We'd love to hear your thoughts on our registry extensibility proposal: #2033 |
@bwateratmsft Not a problem! I believe Kyle has been interacting elsewhere. We're happy to try and fit it into the new system, and we'll certainly be offering suggestions if required. Thank you for your help. :-) |
@lskillen Glad to help! We have started work on the new model but want to make sure we hear from more than just ourselves before we start putting it into the code. Interfaces are hard to change once they are in use by outside parties, so we want to be sure we get it right (or as close as possible) the first time. 😄 |
@bwateratmsft Maybe a good time to reconsider? :) |
We still wanna do the registry extensibility model, but other things have been prioritized, like the ACI or Docker contexts work, or me being on parental leave last winter (and soon, again! 👶 x 2). We definitely haven't forgotten and it's still important to me. |
Ha! 🎉 There's nothing better. :-) Equal 2x Proud Dad over here. So a huge congratulations there (2x)! 😁 No problem at all. This was just a friendly poke; I know you'll do right by us when you get the chance. Enjoy the parental leave! |
What's Changed
Adds support for the Cloudsmith Docker Registry.
Steps to Test
testing-private
USERNAME
is the name of the user created.Docker Registries: Connect Registry....
Cloudsmith
as the registry providerhttps://docker.cloudsmith.io/v2/USERNAME/REPOSITORY/
Setup Video