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

Add Helm chart. #63

Closed
wants to merge 1 commit into from
Closed

Add Helm chart. #63

wants to merge 1 commit into from

Conversation

robinelfrink
Copy link

Helm chart for flatcar-linux-update-operator

This PR adds a Helm Chart for flatcar-linux-update-operator.

How to use

Populate /etc/flatcar/update.conf as needed, and install the chart:

$ helm install fluo ./charts/fluo

Testing done

Succesfully ran the operator and agents on a Flatcar/Kubespray-cluster, installing updates through a Nebraska server.

/closes #24.

@invidian
Copy link
Member

Hey @robinelfrink, thanks for your PR! I think I need to put some more thought into it before merging this PR. Ideally we would maintain only one set of manifests. We could then render them to example/deploy to retain existing functionality and make sure via CI that those manifests are up to date.

I'm also wondering if this repo is the right place for the Helm chart. I've seen other projects having dedicated repository for Helm charts with Helm repository automation (though I personally like having chart source together with source code).

@robinelfrink
Copy link
Author

Hey @robinelfrink, thanks for your PR! I think I need to put some more thought into it before merging this PR. Ideally we would maintain only one set of manifests. We could then render them to example/deploy to retain existing functionality and make sure via CI that those manifests are up to date.

I'm also wondering if this repo is the right place for the Helm chart. I've seen other projects having dedicated repository for Helm charts with Helm repository automation (though I personally like having chart source together with source code).

No worries. I've chosen to pu the chart in charts/ as most usefull Github-actions expect charts in that location :)

If you have another place I'd be happy to create a new PR for that.

@invidian
Copy link
Member

invidian commented Apr 4, 2021

I'll play around with Helm repo and GitHub actions tomorrow to try to push things forward here.

@mkilchhofer
Copy link

Maybe this is a good point to think about one central repository for your helm charts.
I contributed the nebraska helm chart (flatcar/nebraska#330) and texted with @joaquimrocha. The nebraska guys would be happy to separate the helm releases from the application releases.

I'd propose to establish a central helm repository with all the kinvolk charts using GitHub Pages and GH actions. Also in the lokomotive subproject I saw some charts. Once a central "hub" for all your charts exists, I'd link it to https://artifacthub.io/ so your awesome software gets even more popular.
If you need help to setup such a central repo, let me know, I'd love to help.

@invidian
Copy link
Member

invidian commented Apr 8, 2021

I tested exactly that and it works well, we just need to sync internally how to proceed with it, thanks!

@jonkerj
Copy link

jonkerj commented Sep 7, 2021

Hi guys, any movement in this matter?

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.

Provide helm chart as install option
4 participants