-
Notifications
You must be signed in to change notification settings - Fork 42
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
Helm Chart for installing on k8s cluster needed #23
Comments
Actually, I just wrote a very simple Helm chart which has Ingress, Service, and Deployment definitions. Could share it to be used as a basis |
That would be very useful @timurkhafizov. |
I'll try to prepare and share it this weekend. |
I'm not sure that the current implementation of ChartMuseumUI is feature complete enough to embed in the ChartMuseum chart. It will only be able to show the charts in Chartmuseum for some very specific cases:
So i think a seperate chart would be best for now. But an embedded definition would be awesome at a later point. |
Hi - @timurkhafizov if you're still willing, the right thing is probably to submit it here: https://github.com/chartmuseum/charts We are working towards hosting our own helm repo (probably using chartmuseum) for chartmuseum-related charts to submit to the Helm Hub: https://hub.helm.sh/ We can setup CI to push new chart versions there upon merge to master. |
I was about to open a PR for this issue, but I decided to lookup first :-D
IMHO this seems better than having a second helm chart just for this, since the UI part is:
What do you guys think of this? Edit: I just read @timurkhafizov already proposed it 💯 and there's a PR open 🤦♂️ |
There should be a helm chart for installing chartmuseumui with helm.
I propose a helm chart that allows users to specify
What do you think @jdolitsky & @idobry ?
We could go for a chart to https://github.com/helm/charts/tree/master/incubator ?
The text was updated successfully, but these errors were encountered: