-
Notifications
You must be signed in to change notification settings - Fork 35
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
[feature request] Docker registry auth via driver config #92
Comments
Yes, this will be added to the |
@Oloremo I did some more research on this. Currently, in Nomad docker driver, Task Config
This currently is supported in Plugin config
We can support the same thing in
You can specify the location of your
When you launch a job using
When you launch a job using the I ll open a PR for (1) which should atleast get you going. |
Thank you for looking into this! Option 1) is indeed the most simple one and way better than defining it per task but it's not very secure. While I see how option 2) could be considered docker-specific, I'm not sure if it should be a good reason to not go with it. I could be wrong, but my understanding is that even if Docker engine will go away, the Dockerfile format and everything related to the registry(like the protocol) - will not. If we look at this in that light, isn't it makes sense to support that approach? I mean, we can rename config to |
@Oloremo I agree with you, and you are right, most of the orchestration systems are moving away from Docker. k8s has already moved to Having said that, I think I didn't articulate my response properly. You can call it Definitely need this, but not super high priority right now. |
Thank you for your detailed response! Makes sense. Now I wonder what is the plan for the whole container ecosystem moving on in terms of the registry... |
@Oloremo Option (1) is available to use. I ll keep the issue open, and see if we can make some progress on (2) at somepoint. |
I have a PR ready to go for options (2) and (3), are you able to give me access to create a branch and push my code? |
@devops-at-home You have to This might be helpful: https://sqldbawithabeard.com/2019/11/29/how-to-fork-a-github-repository-and-contribute-to-an-open-source-project/ |
Apologies for the delay with this. I ran in to a few issues and have since been flat out with work. I see @shoenig has since updated the dependencies and this was one of the problems I experienced. I skipped A problem with the config spec at the top of this thread is that Also for anyone who wants to use Amazon's ECR Helper on
One final gotcha is that if you are using AWS SSM as
in your |
Any reasons to skip option 2? |
@devops-at-home mind to PR to main repo? so many user can use your third method? |
Registry auth in task config was recently added in #89
This is great! But adding auth info into every Nomad job hcl seems like a bit inconvenient.
Nomad docker drive has an option to set registry auth in drive config: https://www.nomadproject.io/docs/drivers/docker#authentication
And it's very useful from both operations and security perspectives.
Any plans to add it?
The text was updated successfully, but these errors were encountered: