-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Custom registry support #61
Comments
This will be my next priority after #17 Thanks for reporting it ;) |
Some initial thoughts. We should provide a new section in the target configuration to configure custom registries with:
When configuring a target, we can use those credentials to populate the docker cli, _ := command.NewDockerCli()
cli.Initialize(flags.NewClientOptions())
store := cli.ConfigFile().GetCredentialsStore("https://index.docker.io/v1/") // Look like the domain here is only used to get an appropriate helper
err = store.Store(types.AuthConfig{
Username: "<username>",
Password: "<personal access token>",
ServerAddress: "https://index.docker.io/v1/", // Here for the docker hub private registries
}) |
When you're talking about your private registry, are you talking about Docker Hub private ones or self-hosted with the |
I mean my personal self-hosted one, yes |
I'd never use custom self-hosted docker registries. Could you provide the command you use to start the registry and how you log into it so I can reproduce it exactly? |
This is my setup for it on
I run the This is the
Let me know if you need more info |
I got something working but would like your opinion. At first, I've decided to add a custom registries option to the Docker provider (so they are configured per target) but I'm wondering if it should be a resource on its own, like Apps & Targets :/ What do you think? And thanks for providing your configuration, I got it working locally to test my implementation :) |
The custom registries aren't a property of the running Docker daemon, so to me it makes sense to have them separately. OR just use the ones that the running Docker has logins for (in I do prefer the first option 😃 |
So from your perspective registries should be shared across targets right? And yes, the goal is to have an UI so you don't have to mess with the CLI :) |
I think they should be shared, yes. I can run the same custom registry image on multiple Docker daemons, and configuring it just once would make the most sense. |
I'm almost done, will work on the appropriate documentation this afternoon. |
🎉 This issue has been resolved in version 2.3.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Excited to try this out later, kudos!
On 27 May 2024 2:31:02 pm GMT+01:00, Julien LEICHER ***@***.***> wrote:
:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:
- `v2.3.0`
- [GitHub release](https://github.com/YuukanOO/seelf/releases/tag/v2.3.0)
Your **[semantic-release](https://github.com/semantic-release/semantic-release)** bot 📦🚀
--
Reply to this email directly or view it on GitHub:
#61 (comment)
You are receiving this because you authored the thread.
Message ID: ***@***.***>
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
Fingers crossed! I hope everything will be good :) |
Ok, just tried it and works flawlessly! Thanks - I did find an issue but will open it separately |
I logged in to my private registry on the local Docker daemon and tried creating an app with an image from that registry, but it says it's missing basic auth.
The text was updated successfully, but these errors were encountered: