hjacobs / kube-resource-report

Report Kubernetes cluster and pod resource requests vs usage and generate static HTML
GNU General Public License v3.0
714 stars 102 forks source link

Question : is the helm chart available on an helm repository ? #115

Open eddycharly opened 4 years ago

eddycharly commented 4 years ago

Is the helm chart distributed on a helm repository somewhere ? Or do we have to clone this repository to install it ?

Thanks

chetankapoor commented 4 years ago

@eddycharly I installed it by cloning the repo. It's not available on the helm repository.

eddycharly commented 4 years ago

@chetankapoor thanks, i did the same. just asked if @hjacobs hosted it somewhere on his own repo.

hjacobs commented 4 years ago

No, I'm not using Helm myself, so my support for Helm Charts is pretty basic.

ghost commented 4 years ago

@hjacobs would it be possible to implement an helm index in this repo? Just like kube-janitor does here -> https://github.com/hjacobs/kube-janitor/blob/master/unsupported/helm/index.yaml I could try to fork it and give a try if that's ok with you

eddycharly commented 4 years ago

FWIW setting up a helm repo with github actions is free and pretty easy.

ghost commented 4 years ago

@eddycharly do you have an example somewhere?

eddycharly commented 4 years ago

you can look here https://github.com/eddycharly/tekton-helm it uses github pages to host the index.yaml file and github releases to host packaged charts

ghost commented 4 years ago

@eddycharly i found an easier way to deal with this:

Here is my helmfile for reference:

repositories:
- name: hjacobs-kube-resource-report
  url: git+https://github.com/hjacobs/kube-resource-report@unsupported/chart/kube-resource-report

releases:
  - name: kube-resource-report
    chart: hjacobs-kube-resource-report/kube-resource-report
    namespace: kube-system
    values:
    <snip>
eddycharly commented 4 years ago

Nice ! It’s still working with source though, it would have been nice to have the chart tested/packaged/released properly.

But at least it works 🥳

gitkent commented 4 years ago

@eddycharly agreed that chart need to be packaged and released properly.

@hjacobs, remembered we discussed before about hosting this helm chart and I did some research on this. Below is the summary:

The original centralised helm charts repo located here: https://github.com/helm/charts will become deprecated in a few months. The comment was Helm Chart owner will have to find somewhere to host their chart.

Since helm chart (at least the docker image version) need to be aware of the version bump of the app, it make more sense to sit with the source in a separate folder. Hence I have some commits ready to raise a PR if you guys are interested: https://github.com/hjacobs/kube-resource-report/compare/master...gitkent:master. This is basically published the helm chart into Github Pages.

In term of support, it looks like there are some interested helm users including me should be able to provide ad-hoc support in helm chart and it looks tidier without the unsupported folder. :)

For next steps we can look into publishing the helm chart to Helm Hub and add linting and testing of chart in CI pipeline.

Let me know your thoughts.