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

Helm chart #278

Closed
eddycharly opened this issue Mar 20, 2020 · 8 comments
Closed

Helm chart #278

eddycharly opened this issue Mar 20, 2020 · 8 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@eddycharly
Copy link
Member

I'm using a helm chart i wrote to deploy Tekton for a few months now and it's working great.

Would you be interested in a contribution for this ?

@afrittoli
Copy link
Member

afrittoli commented Mar 20, 2020

@eddycharly thanks, I think it would be a nice idea. We do have an operator already, but there's a whole ecosystem of people that uses prominently helm, so it would be good.
Would you be interested in maintaining it and documenting it - at least until more contributors come along? What does it take to host an helm chart repo where people could download the chart from?
@vdemeester @bobcatfish thoughts?

@afrittoli afrittoli added the kind/feature Categorizes issue or PR as related to a new feature. label Mar 20, 2020
@eddycharly
Copy link
Member Author

eddycharly commented Mar 20, 2020

Sure, i will be happy to maintain it.
Technical documentation is already available as it is generated with helm-docs.

I also have a couple of prow jobs to automate releasing new versions of the chart.

On the hosting side, it can be as easy as putting a tar.gz and index.yaml in a bucket. That's the easiest solution, another option is to use chartmuseum.

I currently use chartmuseum because it is hosted on a private chart repository but for public charts, there's nothing better/more cost efficient than the bucket solution i guess.

@eddycharly
Copy link
Member Author

I'll try to open a PR today or tomorrow, it will be easier to discuss about it.

@afrittoli
Copy link
Member

OK - thank you. Would you like to add the PR to the experimental repo?
If we decide to host this, I think it might go into a dedicated repo.

@eddycharly
Copy link
Member Author

Ok for the experimental repo.
It will start the discussion, if you want to put it somewhere else i will close the PR and open another one on the target repo.

@eddycharly
Copy link
Member Author

I opened a PR with some WIP here tektoncd/experimental#494

I started with a copy of my helm chart and took the opportunity to rewrite/improve a few things.
I hope it's a good starting point.

@eddycharly
Copy link
Member Author

Thinking about a versioning policy for helm charts.

A good approach would be to use the same major and minor version as the target Tekton component, keeping the patch to increment on bug fixes.

In any case, the appVersion in the chart description should be the full target component version.

Ok for you ?

@eddycharly
Copy link
Member Author

Closing. Too broad. More relevant issues exist in the experimental repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

2 participants