feat(helm): use named ports for memcached services targetports (#18990)

Signed-off-by: drfaust92 <[email protected]>
Signed-off-by: Ilia Lazebnik <[email protected]>
Signed-off-by: Jay Clifford <[email protected]>
Co-authored-by: Jay Clifford <[email protected]>

Author: DrFaust92

production/helm/loki/CHANGELOG.md

@@ -13,6 +13,7 @@ Entries should include a reference to the pull request that introduced the chang
 
 ## Unreleased
 
+- [FEATURE] Use named ports for memcached services targetports. [#18990](https://github.com/grafana/loki/pull/18990)
 - [BREAKING]  The Loki UI has been moved to a Grafana Plugin. Enabling the UI in the helm chart will now only enable the APIs needed by the plugin, and will host them on the querier. The gateway will now forward all UI requests to the queriers.
 
 ## 6.42.0

production/helm/loki/templates/memcached/_memcached-svc.tpl

@@ -29,11 +29,11 @@ spec:
   ports:
     - name: memcached-client
       port: {{ .port }}
-      targetPort: {{ .port }}
+      targetPort: client
     {{ if $.ctx.Values.memcachedExporter.enabled -}}
     - name: http-metrics
       port: 9150
-      targetPort: 9150
+      targetPort: http-metrics
     {{ end }}
   selector:
     {{- include "loki.selectorLabels" $.ctx | nindent 4 }}