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 ImagePullSecrets to pull charts from private registry #241

Open
raphasle opened this issue Oct 15, 2024 · 1 comment
Open

Support ImagePullSecrets to pull charts from private registry #241

raphasle opened this issue Oct 15, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@raphasle
Copy link

What problem are you facing?

Pulling charts from a private repository is possible by using forProvider.chart.pullSecretRef which points to a secret containing username and password to pull the chart. This is simple and perfectly understandable when reading the documentation or the source code, and it makes sense when looking at the underlying pull action. However, this is IMO not intuitive from a user's perspective (the person writing the composition). Furthermore, limiting to only one pullSecretRef makes migrations harder (e.g. when the Helm chart moves from one private registry to another, the username and pw in the secret must be updated at the same time the new compositions as part of a Configuration Package are released).

How could Crossplane help solve your problem?

It would be more intuitive if the helm provider uses ImagePullSecrets as it is e.g. the case how images are pulled before starting a pod. PullSecretRef would ideally also support secrets of type kubernetes.io/dockerconfigjson (which can contain multiple credentials). Furthermore, it would be even nicer when the helm provider would use the ImagePullSecrets of its own service account when no pullSecretRef is specified and the chart cannot be pulled without credentials. This would allow assigning an ImagePullSecret (that most likely anyways already exists in the cluster for other purposes) to the crossplane service account (of which the helm provider service account will inherit the ImagePullSecret) without having to create a secret with registry credentials specifically for the crossplane helm provider.

@raphasle raphasle added the enhancement New feature or request label Oct 15, 2024
@raphasle
Copy link
Author

After seeing and reading https://github.com/crossplane/crossplane/blob/main/design/one-pager-package-image-config.md, it makes probably more sense to support the new pkg.crossplane.io/v1alpha1 ImageConfig API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant