vromero / activemq-artemis-helm

Helm chart for a cluster of ActiveMQ Artemis (Work in progress)
42 stars 68 forks source link

Fix missing apiVersion in Chart.yaml #46

Open bendahl opened 4 years ago

bendahl commented 4 years ago

This pull request adds the missing field "apiVersion" to the Chart.yaml file. According to the helm docs this is a required field (see helm v2 doc and helm v3 doc. I came across this issue by accident, as I was having some initial trouble getting this to install. Performing helm lint activemq-artemis showed 1 error - the missing apiVersion. It appears that this was not directly related to my initial troubles, but double checking the documentation of helm, it seems that adding this information would be advised. Note that I've tested this using helm 3, but this shouldn't be an issue, as the field is required in helm 2 as well.