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

Support authentication with 3rd party docker registries #8

Open
zootalures opened this issue Dec 14, 2017 · 4 comments
Open

Support authentication with 3rd party docker registries #8

zootalures opened this issue Dec 14, 2017 · 4 comments

Comments

@zootalures
Copy link
Member

zootalures commented Dec 14, 2017

Fn (dind) doesn't get docker regstry credentials.

I think fn needs to understand these eventually in some form but a stop gap

Ideally we should be able to share one or more k8s docker image pull secrets with the fn container to allow secured registries to be used.

e.g. (elswhere in k8s ):

kubectl create secret docker-registry wcrsecret--docker-server=wcr --docker-username=testserver --docker-password=$(cat ~/.wercker/token)  [email protected]

then in values.yaml

fnserver
   imageSecrets:
       - wcrsecret 
@carimura
Copy link
Member

cc @derekschultz

@rdallman
Copy link
Contributor

we support multiple registries configured in ~/.docker/config or through DOCKER_AUTH env var in fn. i am less sure if it works properly, but there was an attempt. it should be possible to thread in either way to k8s

@venkat50
Copy link

Please also consider support for private registry (with and without authentication).

@lenalebt
Copy link

lenalebt commented Feb 13, 2018

One important aspect to consider when using a private registry is the nesting level of docker image names. You can only have 3 at max, see https://github.com/fnproject/fn/blob/f27d47f2dd9520647f8799043bfcb3d121709958/api/agent/drivers/driver.go#L283

If you use more than 3, it falls back to assuming the image comes from docker hub and does not provide correct credentials. This cost me about 3 days of debugging, hoping that others do not run in the same thing...

Shorter nestings are okay.

I opened a bug report here: fnproject/fn#764

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

No branches or pull requests

5 participants