[stable/rabbitmq-ha] Only set the nodePort for NodePort service type (#17160)

* Only set the nodePort for NodePort service type

This resolves an issue with older version of kubernetes that fails on
parsing the `null` value as it only expects integer values. (which
means that on older clusters the chart as is now fails to install)

```
...
failed: Service in version "v1" cannot be handled as a Service: v1.Service: Spec: v1.ServiceSpec: Ports: []v1.ServicePort: v1.ServicePort: NodePort: readUint32: unexpected character: �, parsing 304 ...odePort":n...
...
```

Signed-off-by: Miroslav Prasil <miroslav@prasil.info>

* Just bump the bugfix number

Signed-off-by: Miroslav Prasil <miroslav@prasil.info>

* Also use nodePort for LoadBalancer service type

Signed-off-by: Miroslav Prasil <miroslav@prasil.info>
This commit is contained in:
Miro Prasil
2019-09-16 09:08:38 -07:00
committed by Kubernetes Prow Robot
parent 5e7b6640dd
commit ae8ef2c026
2 changed files with 7 additions and 1 deletions
+1 -1
View File
@@ -1,7 +1,7 @@
name: rabbitmq-ha
apiVersion: v1
appVersion: 3.7.15
version: 1.32.0
version: 1.32.1
description: Highly available RabbitMQ cluster, the open source message broker
software that implements the Advanced Message Queuing Protocol (AMQP).
keywords:
@@ -40,17 +40,23 @@ spec:
- name: http
protocol: TCP
port: {{ .Values.rabbitmqManagerPort }}
{{- if eq .Values.service.type "NodePort" "LoadBalancer" }}
nodePort: {{ .Values.service.managerNodePort }}
{{- end }}
targetPort: http
- name: amqp
protocol: TCP
port: {{ .Values.rabbitmqNodePort }}
{{- if eq .Values.service.type "NodePort" "LoadBalancer" }}
nodePort: {{ .Values.service.amqpNodePort }}
{{- end }}
targetPort: amqp
- name: epmd
protocol: TCP
port: {{ .Values.rabbitmqEpmdPort }}
{{- if eq .Values.service.type "NodePort" "LoadBalancer" }}
nodePort: {{ .Values.service.epmdNodePort }}
{{- end }}
targetPort: epmd
{{- if .Values.rabbitmqSTOMPPlugin.enabled }}
- name: stomp-tcp