[GitHub] [pulsar-helm-chart] Imunhatep opened a new issue #65: Helm chart dependencies do not follow best practices

[GitBox <gi...@apache.org>]

Imunhatep opened a new issue #65:
URL: https://github.com/apache/pulsar-helm-chart/issues/65


   **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](https://helm.sh/docs/helm/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..
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar-helm-chart] chrisjohnson00 commented on issue #65: Helm chart dependencies do not follow best practices

[GitBox <gi...@apache.org>]

chrisjohnson00 commented on issue #65:
URL: https://github.com/apache/pulsar-helm-chart/issues/65#issuecomment-894615515


   +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.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org