weaveworks / sa-demos

This repository is dealing with the Weave GitOps demos and demo environments. Please use it to file and update issues that you are seeing. You can also put your demo guides and demo scripts here.
1 stars 0 forks source link

Can't use regular Helm Repositories as Application Source #50

Closed LutzLange closed 1 year ago

LutzLange commented 2 years ago

I'm preparing a Webinar with Isolavent. I wanted to activate a HelmRepository for use in the UI.

This fails : Screenshot from 2022-09-15 14-53-20

I did check that the Isolavent helm repo does not list too many charts. It has 6 charts. This should work.

LutzLange commented 2 years ago

Here are my HelmRepository definitions for this :

For the management cluster : https://github.com/weavegitops/demo2-repo/blob/main/weave-gitops-platform/capi-profiles/isolavent-charts.yaml

For the leaf clusters : https://github.com/weavegitops/demo2-repo/blob/main/clusters/bases/sources/isolavent-charts.yaml

LutzLange commented 2 years ago

DavidS tells me that this is being worked on by Kevin. The plan is to add annotations to the HelmRepo definition that allows adding missing values.

LutzLange commented 1 year ago

Kevin pointed me to including an annotation in the HelmRepository : Screenshot from 2022-09-20 20-17-25

But this does not change the fact that I can't see any charts / profiles. Screenshot from 2022-09-20 20-11-38

LutzLange commented 1 year ago

I had to add the annotation for the management cluster and for the leaf cluster. I can see the helm charts after that.