[bitnami/memcached] Fix ClusterIP invalid value and /tmp volumeMount using high-availability (#5047)

* Memcached. Fix error when try to deploy with service type LoadBalancer

* Memcached. Fix error when try to deploy with architecture high-availability and persistence.enabled: false

* Memcached. Fix Chart version

* Update bitnami/memcached/templates/service.yaml

* Bump patch version

Co-authored-by: Francisco de Paz Galán <fdepaz@vmware.com>

bitnami/memcached/Chart.yaml

@@ -22,4 +22,4 @@ name: memcached
 sources:
   - https://github.com/bitnami/bitnami-docker-memcached
   - http://memcached.org/
-version: 5.4.2
+version: 5.4.3

bitnami/memcached/templates/service.yaml

@@ -8,7 +8,7 @@ metadata:
   {{- end }}
 spec:
   type: {{ .Values.service.type }}
-  {{- if eq .Values.architecture "high-availability" }}
+  {{- if and (eq .Values.architecture "high-availability") (eq .Values.service.type "ClusterIP") }}
   clusterIP: None
   {{- end }}
   {{- if and (not (empty .Values.service.loadBalancerIP)) (eq .Values.service.type "LoadBalancer") }}

bitnami/memcached/templates/statefulset.yaml

@@ -98,13 +98,13 @@ spec:
           securityContext:
             readOnlyRootFilesystem: {{ .Values.securityContext.readOnlyRootFilesystem }}
           {{- end }}
-          {{- if .Values.persistence.enabled }}
           volumeMounts:
+          {{- if .Values.persistence.enabled }}
             - name: data
               mountPath: /cache-state
+          {{- end }}
             - name: tmp
               mountPath: /tmp
-          {{- end }}
         {{- if .Values.metrics.enabled }}
         - name: metrics
           image: {{ template "memcached.metrics.image" . }}