k8s-at-home / charts

⚠️ Deprecated : Helm charts for applications you run at home
https://docs.k8s-at-home.com
Apache License 2.0
1.45k stars 623 forks source link

[mopidy] Add new chart #1616

Closed lanquarden closed 2 years ago

lanquarden commented 2 years ago

Signed-off-by: lanquarden lanquarden@gmail.com

Description of the change

Adding mopidy chart

Benefits

Easily run a mopidy instance in your cluster

Possible drawbacks

It's a official container from the Iris project (web frontend), but it is not following semver, they just update the latest tag with every release of the web frontend. I think that I'll probably end up creating a headless container and a one with the frontend where the different plugins to install are configurable.

Applicable issues

Additional information

The same as for the snapcast chart, bare bones helm that was tested manually with a helm install. s I learn how to successfully deploy this together with a snapcast chart I'll try to find a way to poor that back into the documentation/values.yaml

Checklist

truxnell commented 2 years ago

Hi @lanquarden Thanks for the PR! Looks like it is failing install with default config, can you have a look? Ideally our charts will pass CI for a basic install without erroring.

lanquarden commented 2 years ago

I hope latest commit fixes the although I can't trigger the CI

lanquarden commented 2 years ago

Updated indent for yaml lists, did same update in the template

truxnell commented 2 years ago

@lanquarden Awesome, ta

I forgot in the review - you can (like im just writing in the snapcast chart) use appVersion: latest in the Chart.yaml and then leave tag: as blank in the values.yaml The vaules.yaml inherits from the Chart.yaml appversion (thats what # @default -- chart.appVersion does). This saves you having to update both tags if you chose to keep images bumped in the helm - you can just bump in chart only.

lanquarden commented 2 years ago

I think I got it all, latest in Chart, no tag in values.yaml in both PRs

truxnell commented 2 years ago

Thanks for the PR!

@all-contributors can you please add @lanquarden for code

allcontributors[bot] commented 2 years ago

@Truxnell

I've put up a pull request to add @lanquarden! :tada: