You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
BinderHub spawns Build pods that communicates with the docker-api setup on the k8s node's, and they need to pass credentials to push to an external container registry. So, for this to work, BinderHub starts these pods up to mount a k8s Secret with a docker client config file including credentials.
A person installing the chart should be able to provide server / username / password to a container registry and have BinderHub spawned build pods able to push to it.
I've not got the option to work out, such as kubectl create secret docker-registry <name> --docker-server=... and then reference that k8s Secret.
We should support re-using previously provided credentials
This makes someone do initial setup with helm install --set ... and then not need to retain the sensitive credentials in a git repo etc. We do this for a lot of z2jh credentials already for example.
We should support having these credentials cleaned up when the helm release is deleted
This comes out of the box by managing the k8s Secret we create via the Helm chart
The text was updated successfully, but these errors were encountered:
BinderHub spawns Build pods that communicates with the docker-api setup on the k8s node's, and they need to pass credentials to push to an external container registry. So, for this to work, BinderHub starts these pods up to mount a k8s Secret with a docker client config file including credentials.
I've not got the option to work out, such as
kubectl create secret docker-registry <name> --docker-server=...
and then reference that k8s Secret.This makes someone do initial setup with
helm install --set ...
and then not need to retain the sensitive credentials in a git repo etc. We do this for a lot of z2jh credentials already for example.This comes out of the box by managing the k8s Secret we create via the Helm chart
The text was updated successfully, but these errors were encountered: