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

feat(k8s): Adds a helm chart #248

Merged
merged 1 commit into from
Feb 2, 2022
Merged

feat(k8s): Adds a helm chart #248

merged 1 commit into from
Feb 2, 2022

Conversation

mtiller
Copy link

@mtiller mtiller commented Feb 2, 2022

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.

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 getmeli#233 but which
also applies to getmeli#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 getmeli#246.
@gempain
Copy link
Contributor

gempain commented Feb 2, 2022

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

@gempain gempain merged commit 8a731eb into getmeli:next Feb 2, 2022
@Thanzex
Copy link

Thanzex commented Mar 18, 2022

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
Copy link
Author

mtiller commented Mar 25, 2022

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

#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
Copy link
Contributor

gempain commented Mar 25, 2022

Just merged it 😉

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

Successfully merging this pull request may close these issues.

4 participants