pulsar-helm-chart icon indicating copy to clipboard operation
pulsar-helm-chart copied to clipboard

Helm chart dependencies do not follow best practices

Open imunhatep opened this issue 5 years ago • 1 comments

Is your feature request related to a problem? Please describe. Charts do not follow common guideline: chart templates bring dependencies into the main chart instead of using Helm dependency

Describe the solution you'd like Move services like prometheus, alert-manager, zookeeper to a chart dependencies, That would simplified helm chart support and usage.

Describe alternatives you've considered Do not see other options, cause the issue is that Chart structure is inconsistent and does not follow best practices

Additional context As a chart reference with a lot of dependencies GitLab helm chart could be taken..

imunhatep avatar Sep 25 '20 13:09 imunhatep

+1 This would also fix the improper use of Prometheus as a deployment. Prometheus needs to be a stateful set due to it's pvc.

chrisjohnson00 avatar Aug 07 '21 06:08 chrisjohnson00