-
Notifications
You must be signed in to change notification settings - Fork 614
Conversation
Signed-off-by: lanquarden <lanquarden@gmail.com>
Hi @lanquarden |
Signed-off-by: lanquarden <lanquarden@gmail.com>
I hope latest commit fixes the although I can't trigger the CI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few minor linting changes
charts/incubator/mopidy/Chart.yaml
Outdated
version: 0.0.1 | ||
kubeVersion: ">=1.16.0-0" | ||
keywords: | ||
- mopidy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- mopidy | |
- mopidy |
charts/incubator/mopidy/Chart.yaml
Outdated
home: https://github.com/k8s-at-home/charts/tree/master/charts/stable/mopidy | ||
icon: https://mopidy.org/icon | ||
sources: | ||
- https://github.com/mopidy/mopidy-docker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- https://github.com/mopidy/mopidy-docker | |
- https://github.com/mopidy/mopidy-docker |
charts/incubator/mopidy/Chart.yaml
Outdated
- name: lanquarden | ||
email: lanquarden@gmail.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- name: lanquarden | |
email: lanquarden@gmail.com | |
- name: lanquarden | |
email: lanquarden@gmail.com |
charts/incubator/mopidy/Chart.yaml
Outdated
- name: common | ||
repository: https://library-charts.k8s-at-home.com | ||
version: 4.4.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- name: common | |
repository: https://library-charts.k8s-at-home.com | |
version: 4.4.2 | |
- name: common | |
repository: https://library-charts.k8s-at-home.com | |
version: 4.4.2 |
Signed-off-by: lanquarden <lanquarden@gmail.com>
Updated indent for yaml lists, did same update in the template |
@lanquarden Awesome, ta I forgot in the review - you can (like im just writing in the snapcast chart) use |
Signed-off-by: lanquarden <lanquarden@gmail.com>
I think I got it all, |
Thanks for the PR! @all-contributors can you please add @lanquarden for code |
I've put up a pull request to add @lanquarden! 🎉 |
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
[home-assistant]
)Chart.yaml
according to semver.artifacthub.io/changes
changelog annotation has been updated inChart.yaml
. See Artifact Hub documentation for more info.values.yaml
file.