getmeli / meli

Platform for deploying static sites and frontend applications easily. Automatic SSL, deploy previews, reverse proxy, and more.
Other
2.4k stars 97 forks source link

feat(k8s): Adds a helm chart #248

Closed mtiller closed 2 years ago

mtiller commented 2 years ago

This commit includes a helm chart or Meli. I created it based on the k8s manifest contributed by @Berndinox. Hopefully that values.yaml file I provided sufficiently explains the various knobs to turn. One potential area of improvement would be allowing existing PVCs to be used.

Note that this includes the feature I mentioned in #233 but which also applies to #238 which is the ability to explicitly list all sites so that the ingress configuration can fetch SSL certificates for branch host names.

Ideally, this helm chart would be packaged up and shared...somewhere. I'm actually not that well versed in how to host helm charts for open source software. I know there used to be a public registry hosted by Google but then they withdrew support and the landscape got very fragmented. If there is a place to host such a chart, it would be good to publish this there so that an ordinary helm installation could reference it.

Not that the values.yaml file include the image and tag. This can be customized to run a custom Meli image instead of the official Docker hub image.

This closes #246.

gempain commented 2 years ago

This is awesome, you rock @mtiller, thanks so much for all the time spent improving this project!

Thanzex commented 2 years ago

Hi @mtiller and thanks for your contribution. Unfortunately, since you added the values.yaml file to .gitignore you did not provide any values file, I can't see any even on your fork. Could you please provide an example one?

mtiller commented 2 years ago

@Thanzex sorry for the delayed response. I've created a pull request that fixes this:

https://github.com/getmeli/meli/pull/256

Give that a try. It should give you a reasonable place to start. Hopefully it will get merged in quickly to the next branch as well.

gempain commented 2 years ago

Just merged it 😉