search

NeonGeckoCom / neon-diana-utils

Scripts and Automation for running Neon DIANA
Other
5 stars 6 forks source link

fix: create ingress-common charts #53

Closed mikejgray closed 7 months ago

mikejgray commented 7 months ago

Description

Creates by default, with a flag to skip creation of nginx ingress.

NeonDaniel commented 7 months ago

Looking at this now, I see the ingress charts are published as ingress-common so maybe a simpler solution would be to update docs to apply the chart from the helm repo directly? helm install diana/ingress-common should do it

mikejgray commented 7 months ago

Looking at this now, I see the ingress charts are published as ingress-common so maybe a simpler solution would be to update docs to apply the chart from the helm repo directly? helm install diana/ingress-common should do it

So rather than dropping everything off in the output folder, just helm install from the repo's copy since there's nothing to configure?

NeonDaniel commented 7 months ago

Looking at this now, I see the ingress charts are published as ingress-common so maybe a simpler solution would be to update docs to apply the chart from the helm repo directly? helm install diana/ingress-common should do it

So rather than dropping everything off in the output folder, just helm install from the repo's copy since there's nothing to configure?

That's my thinking. It also means we don't have to worry about updating charts in the output directory if the ingress chart is updated