search

jboss-container-images / jboss-amq-7-broker-openshift-image

Apache License 2.0
15 stars 30 forks source link

Number of replicas should be integer #64

Open dariodsa opened 4 years ago

dariodsa commented 4 years ago

Branch name: amq-broker-77-dev File name: jboss-amq-7-broker-openshift-image/templates/amq-broker-77-persistence-clustered.yaml

When I was using RedHat AMQ Broker 7.7 clustered template, it couldn't create StatefulSet so it failed. The reason why it failed was because AMQ_REPLICAS was used as a parameter and it was interpreted as a string, but it should be interpreted as a integer (Link to code).

AMQ_REPLICAS should be in a form of string (for example "2") in the following line because of the JSON properties. https://github.com/jboss-container-images/jboss-amq-7-broker-openshift-image/blob/82eb44cb22e7df939db431ffb79dee151c53b7f4/templates/amq-broker-77-persistence-clustered.yaml#L225 But it should be in string format in following line. So error occurs in this line because replicas expect integer form, not string ( format should be replicas: 2, not replicas: "2"). https://github.com/jboss-container-images/jboss-amq-7-broker-openshift-image/blob/82eb44cb22e7df939db431ffb79dee151c53b7f4/templates/amq-broker-77-persistence-clustered.yaml#L274