rabbitmq-cluster-operator: rename http-webhook port to https-webhook (b…

…itnami#25552) It's actually https, not http. Istio does magic based on the name of the port and expects http traffic on http-<name> ports, not https. This results in a "connection reset by peer" when trying to deploy a RabbitmqCluster resource, when it tries to invoke the message topology operator's webhook. Signed-off-by: Boris Brönner <[email protected]> Co-authored-by: Boris Brönner <[email protected]> Signed-off-by: Matheus Gonçalves <[email protected]>
matheusmgon · May 8, 2024 · ba780b1 · ba780b1
1 parent c5df528
commit ba780b1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/bitnami/rabbitmq-cluster-operator/Chart.yaml b/bitnami/rabbitmq-cluster-operator/Chart.yaml
@@ -37,4 +37,4 @@ maintainers:
 name: rabbitmq-cluster-operator
 sources:
 - https://github.com/bitnami/charts/tree/main/bitnami/rabbitmq-cluster-operator
-version: 4.2.5
+version: 4.2.6
diff --git a/bitnami/rabbitmq-cluster-operator/templates/messaging-topology-operator/deployment.yaml b/bitnami/rabbitmq-cluster-operator/templates/messaging-topology-operator/deployment.yaml
@@ -128,7 +128,7 @@ spec:
           resources: {{- include "common.resources.preset" (dict "type" .Values.msgTopologyOperator.resourcesPreset) | nindent 12 }}
           {{- end }}
           ports:
-            - name: http-webhook
+            - name: https-webhook
               containerPort: 9443
               protocol: TCP
             - name: http-metrics

diff --git a/bitnami/rabbitmq-cluster-operator/templates/messaging-topology-operator/webhook-service.yaml b/bitnami/rabbitmq-cluster-operator/templates/messaging-topology-operator/webhook-service.yaml
@@ -39,9 +39,9 @@ spec:
   sessionAffinityConfig: {{- include "common.tplvalues.render" (dict "value" .Values.msgTopologyOperator.service.sessionAffinityConfig "context" $) | nindent 4 }}
   {{- end }}
   ports:
-    - name: http
+    - name: https
       port: {{ .Values.msgTopologyOperator.service.ports.webhook }}
-      targetPort: http-webhook
+      targetPort: https-webhook
       protocol: TCP
       {{- if (and (or (eq .Values.msgTopologyOperator.service.type "NodePort") (eq .Values.msgTopologyOperator.service.type "LoadBalancer")) (not (empty .Values.msgTopologyOperator.service.nodePorts.http))) }}
       nodePort: {{ .Values.msgTopologyOperator.service.nodePorts.http }}