deviceinsight / activemq-artemis-helm Goto Github PK

Some settings require other settings to be already defined. The address settings in configMap are always sorted by name so there is no way to define the required order of settings.
E.g.
redeliveryCollisionAvoidanceFactor require redeliveryDelay or redeliveryDelayMultiplier to be already defined but the settings from .values

• match: "statesQueue"
  settings:
  redeliveryCollisionAvoidanceFactor: '0.15'
  redeliveryDelayMultiplier: '1.5'
  redeliveryDelay: '5000'
  maxRedeliveryDelay: '30000'
  maxDeliveryAttempts: '-1'

are expanded into

   <address-setting match="statesQueue">
    <max-delivery-attempts>-1</max-delivery-attempts>
    <max-redelivery-delay>30000</max-redelivery-delay>
    <redelivery-collision-avoidance-factor>0.15</redelivery-collision-avoidance-factor>
    <redelivery-delay>5000</redelivery-delay>
    <redelivery-delay-multiplier>1.5</redelivery-delay-multiplier>
   </address-setting>

This problem probably applies to other settings from configMap like clustering.xml or clustering.xml

We have two services defined for the statefulsets:

• a service for the live nodes
• a service for the backup nodes

We should have an additional service including all ready endpoints

Values.yaml currently contains quite some parameters, that are undocumented.

• All configuration should be contained in the values.yml with an explanation, what they do
• Documentation comments should start with two '#'
• Documentation comments should start with the full qualified name of the parameter
• Sample: ## ha.enabled if truea second stateful set running backups will be installed

The artemis chart should be downloadable via http as binary, so it can be installed via the helm client.

A good approach can be to use github pages: https://medium.com/@mattiaperi/create-a-public-helm-chart-repository-with-github-pages-49b180dbb417.

Providing a CI pipeline would be great, but can also be covered in a separate PR

With #2 and #1 done, we should create a new release 0.2.0, that can be publicly used