Drop in-built support for bitnami's memcached

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## master / unreleased
 
+* [BREAKING] Drop in-built support for bitnami's memcached #567
+  * memcached dependency and subchart has been removed
+  * Chart users have to take care of the deployment of memcached now (if they require it)
 * [CHANGE] Memcacheds are now enabled by default #551
 * [CHANGE] Change default config: distributors, rulers, queriers, query-frontend, nginx and compactors can be disrupted until 30%
 * [CHANGE] Remove startup probes from compactors from default values. It's not recommended in general #542

Chart.yaml

@@ -15,24 +15,3 @@ maintainers:
 name: cortex
 sources:
   - https://github.com/cortexproject/cortex-helm-chart
-dependencies:
-  - name: memcached
-    alias: memcached-frontend
-    version: 6.14.0
-    repository: https://charts.bitnami.com/bitnami
-    condition: memcached-frontend.enabled
-  - name: memcached
-    alias: memcached-blocks-index
-    version: 6.14.0
-    repository: https://charts.bitnami.com/bitnami
-    condition: memcached-blocks-index.enabled
-  - name: memcached
-    alias: memcached-blocks
-    version: 6.14.0
-    repository: https://charts.bitnami.com/bitnami
-    condition: memcached-blocks.enabled
-  - name: memcached
-    alias: memcached-blocks-metadata
-    version: 6.14.0
-    repository: https://charts.bitnami.com/bitnami
-    condition: memcached-blocks-metadata.enabled

README.md

@@ -84,13 +84,6 @@ Source code can be found [here](https://cortexmetrics.io/)
 
 Kubernetes: `^1.19.0-0`
 
-| Repository | Name | Version |
-|------------|------|---------|
-| https://charts.bitnami.com/bitnami | memcached-frontend(memcached) | 6.14.0 |
-| https://charts.bitnami.com/bitnami | memcached-blocks-index(memcached) | 6.14.0 |
-| https://charts.bitnami.com/bitnami | memcached-blocks(memcached) | 6.14.0 |
-| https://charts.bitnami.com/bitnami | memcached-blocks-metadata(memcached) | 6.14.0 |
-
 ## Values
 
 | Key | Type | Default | Description |
@@ -382,46 +375,6 @@ Kubernetes: `^1.19.0-0`
 | ingress.​tls | list | `[]` |  |
 | memberlist.​service.​annotations | object | `{}` |  |
 | memberlist.​service.​labels | object | `{}` |  |
-| memcached-blocks-index.​architecture | string | `"high-availability"` |  |
-| memcached-blocks-index.​enabled | bool | `true` | Enables support for block index caching |
-| memcached-blocks-index.​extraEnvVars[0] | object | `{"name":"MEMCACHED_CACHE_SIZE","value":"1024"}` | MEMCACHED_CACHE_SIZE is the amount of memory allocated to memcached for object storage |
-| memcached-blocks-index.​extraEnvVars[1] | object | `{"name":"MEMCACHED_MAX_CONNECTIONS","value":"1024"}` | MEMCACHED_MAX_CONNECTIONS is the maximum number of simultaneous connections to the memcached service |
-| memcached-blocks-index.​extraEnvVars[2] | object | `{"name":"MEMCACHED_THREADS","value":"4"}` | MEMCACHED_THREADS is the number of threads to use when processing incoming requests. By default, memcached is configured to use 4 concurrent threads. The threading improves the performance of storing and retrieving data in the cache, using a locking system to prevent different threads overwriting or updating the same values. |
-| memcached-blocks-index.​metrics.​enabled | bool | `true` |  |
-| memcached-blocks-index.​metrics.​serviceMonitor.​enabled | bool | `false` |  |
-| memcached-blocks-index.​replicaCount | int | `2` |  |
-| memcached-blocks-index.​resources | object | `{}` |  |
-| memcached-blocks-index.​service.​clusterIP | string | `"None"` |  |
-| memcached-blocks-metadata.​architecture | string | `"high-availability"` |  |
-| memcached-blocks-metadata.​enabled | bool | `true` | Enables support for block metadata caching |
-| memcached-blocks-metadata.​extraEnvVars[0] | object | `{"name":"MEMCACHED_CACHE_SIZE","value":"1024"}` | MEMCACHED_CACHE_SIZE is the amount of memory allocated to memcached for object storage |
-| memcached-blocks-metadata.​extraEnvVars[1] | object | `{"name":"MEMCACHED_MAX_CONNECTIONS","value":"1024"}` | MEMCACHED_MAX_CONNECTIONS is the maximum number of simultaneous connections to the memcached service |
-| memcached-blocks-metadata.​extraEnvVars[2] | object | `{"name":"MEMCACHED_THREADS","value":"4"}` | MEMCACHED_THREADS is the number of threads to use when processing incoming requests. By default, memcached is configured to use 4 concurrent threads. The threading improves the performance of storing and retrieving data in the cache, using a locking system to prevent different threads overwriting or updating the same values. |
-| memcached-blocks-metadata.​metrics.​enabled | bool | `true` |  |
-| memcached-blocks-metadata.​metrics.​serviceMonitor.​enabled | bool | `false` |  |
-| memcached-blocks-metadata.​replicaCount | int | `2` |  |
-| memcached-blocks-metadata.​resources | object | `{}` |  |
-| memcached-blocks-metadata.​service.​clusterIP | string | `"None"` |  |
-| memcached-blocks.​architecture | string | `"high-availability"` |  |
-| memcached-blocks.​enabled | bool | `true` | Enables support for block caching |
-| memcached-blocks.​extraEnvVars[0] | object | `{"name":"MEMCACHED_CACHE_SIZE","value":"1024"}` | MEMCACHED_CACHE_SIZE is the amount of memory allocated to memcached for object storage |
-| memcached-blocks.​extraEnvVars[1] | object | `{"name":"MEMCACHED_MAX_CONNECTIONS","value":"1024"}` | MEMCACHED_MAX_CONNECTIONS is the maximum number of simultaneous connections to the memcached service |
-| memcached-blocks.​extraEnvVars[2] | object | `{"name":"MEMCACHED_THREADS","value":"4"}` | MEMCACHED_THREADS is the number of threads to use when processing incoming requests. By default, memcached is configured to use 4 concurrent threads. The threading improves the performance of storing and retrieving data in the cache, using a locking system to prevent different threads overwriting or updating the same values. |
-| memcached-blocks.​metrics.​enabled | bool | `true` |  |
-| memcached-blocks.​metrics.​serviceMonitor.​enabled | bool | `false` |  |
-| memcached-blocks.​replicaCount | int | `2` |  |
-| memcached-blocks.​resources | object | `{}` |  |
-| memcached-blocks.​service.​clusterIP | string | `"None"` |  |
-| memcached-frontend.​architecture | string | `"high-availability"` |  |
-| memcached-frontend.​enabled | bool | `true` | Enables support for caching queries in the frontend |
-| memcached-frontend.​extraEnvVars[0] | object | `{"name":"MEMCACHED_CACHE_SIZE","value":"1024"}` | MEMCACHED_CACHE_SIZE is the amount of memory allocated to memcached for object storage |
-| memcached-frontend.​extraEnvVars[1] | object | `{"name":"MEMCACHED_MAX_CONNECTIONS","value":"1024"}` | MEMCACHED_MAX_CONNECTIONS is the maximum number of simultaneous connections to the memcached service |
-| memcached-frontend.​extraEnvVars[2] | object | `{"name":"MEMCACHED_THREADS","value":"4"}` | MEMCACHED_THREADS is the number of threads to use when processing incoming requests. By default, memcached is configured to use 4 concurrent threads. The threading improves the performance of storing and retrieving data in the cache, using a locking system to prevent different threads overwriting or updating the same values. |
-| memcached-frontend.​metrics.​enabled | bool | `true` |  |
-| memcached-frontend.​metrics.​serviceMonitor.​enabled | bool | `false` |  |
-| memcached-frontend.​replicaCount | int | `2` |  |
-| memcached-frontend.​resources | object | `{}` |  |
-| memcached-frontend.​service.​clusterIP | string | `"None"` |  |
 | nginx.​affinity | object | `{}` |  |
 | nginx.​annotations | object | `{}` |  |
 | nginx.​autoscaling.​behavior | object | `{}` | Ref: https://kubernetes.io/docs/tasks/run-application/horizontal-pod-autoscale/#support-for-configurable-scaling-behavior |

ci/test-configmap-values.yaml

@@ -145,36 +145,3 @@ store_gateway:
   extraVolumeMounts:
     - name: tmp-test
       mountPath: /tmp-test
-
-memcached-frontend:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks-index:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks-metadata:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0

ci/test-deployment-values.yaml

@@ -152,36 +152,3 @@ store_gateway:
   extraVolumeMounts:
     - name: tmp-test
       mountPath: /tmp-test
-
-memcached-frontend:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks-index:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks-metadata:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0

ci/test-sts-values.yaml

@@ -143,36 +143,3 @@ store_gateway:
   extraVolumeMounts:
     - name: tmp-test
       mountPath: /tmp-test
-
-memcached-frontend:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks-index:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0
-memcached-blocks-metadata:
-  enabled: true
-  architecture: standalone
-  replicaCount: 1
-  resources:
-    requests:
-      cpu: 0
-      memory: 0

ct.yaml

@@ -5,5 +5,4 @@ charts:
   - "./"
 chart-repos:
   - base-charts=https://charts.helm.sh/stable
-  - bitnami=https://charts.bitnami.com/bitnami
 helm-extra-args: --timeout 600s

renovate.json

@@ -47,16 +47,6 @@
             ],
             "depNameTemplate": "kiwigrid/k8s-sidecar",
             "datasourceTemplate": "docker"
-        },
-        {
-            "description": "Update memcached image in README.md",
-            "fileMatch": ["^README\\.md$"],
-            "matchStrings": [
-                "\\|[^|]*\\|[^|]*\\(memcached\\)[^|]*\\|\\s+(?<currentValue>.*?)\\s+\\|\\s+"
-            ],
-            "depNameTemplate": "memcached",
-            "registryUrlTemplate": "https://charts.bitnami.com/bitnami",
-            "datasourceTemplate": "helm"
         }
     ]
 }

templates/_helpers.tpl

@@ -81,33 +81,6 @@ app.kubernetes.io/name: {{ include "cortex.name" . }}
 app.kubernetes.io/instance: {{ .Release.Name }}
 {{- end }}
 
-{{/*
-Create configuration parameters for memcached configuration
-*/}}
-{{- define "cortex.memcached" -}}
-{{- if index .Values "memcached-blocks-index" "enabled" }}
-- "-blocks-storage.bucket-store.index-cache.backend=memcached"
-- "-blocks-storage.bucket-store.index-cache.memcached.addresses=dns+{{ .Release.Name }}-memcached-blocks-index.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:11211"
-{{- end -}}
-{{- if index .Values "memcached-blocks" "enabled" }}
-- "-blocks-storage.bucket-store.chunks-cache.backend=memcached"
-- "-blocks-storage.bucket-store.chunks-cache.memcached.addresses=dns+{{ .Release.Name }}-memcached-blocks.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:11211"
-{{- end -}}
-{{- if index .Values "memcached-blocks-metadata" "enabled" }}
-- "-blocks-storage.bucket-store.metadata-cache.backend=memcached"
-- "-blocks-storage.bucket-store.metadata-cache.memcached.addresses=dns+{{ .Release.Name }}-memcached-blocks-metadata.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:11211"
-{{- end -}}
-{{- end -}}
-
-{{/*
-Create configuration for frontend memcached configuration
-*/}}
-{{- define "cortex.frontend-memcached" -}}
-{{- if index .Values "memcached-frontend" "enabled" }}
-- "-frontend.memcached.addresses=dns+{{ .Release.Name }}-memcached-frontend.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:11211"
-{{- end -}}
-{{- end -}}
-
 {{/*
 Determine the policy api version
 */}}

templates/compactor/compactor-statefulset.yaml

@@ -107,7 +107,6 @@ spec:
           args:
             - "-target=compactor"
             - "-config.file=/etc/cortex/cortex.yaml"
-            {{- include "cortex.memcached" . | nindent 12}}
             {{- range $key, $value := .Values.compactor.extraArgs }}
             - "-{{ $key }}={{ $value }}"
             {{- end }}

templates/ingester/ingester-dep.yaml

@@ -54,7 +54,6 @@ spec:
           args:
             - "-target=ingester"
             - "-config.file=/etc/cortex/cortex.yaml"
-            {{- include "cortex.memcached" . | nindent 12}}
             {{- range $key, $value := .Values.ingester.extraArgs }}
             - "-{{ $key }}={{ $value }}"
             {{- end }}

templates/ingester/ingester-statefulset.yaml

@@ -110,7 +110,6 @@ spec:
           args:
             - "-target=ingester"
             - "-config.file=/etc/cortex/cortex.yaml"
-            {{- include "cortex.memcached" . | nindent 12}}
             {{- range $key, $value := .Values.ingester.extraArgs }}
             - "-{{ $key }}={{ $value }}"
             {{- end }}

templates/querier/querier-dep.yaml

@@ -58,7 +58,6 @@ spec:
           {{- if and .Values.query_frontend.enabled (not .Values.query_scheduler.enabled) }}
             - "-querier.frontend-address={{ template "cortex.queryFrontendFullname" . }}-headless.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.config.server.grpc_listen_port }}"
           {{- end }}
-          {{- include "cortex.memcached" . | nindent 12}}
           {{- range $key, $value := .Values.querier.extraArgs }}
             - "-{{ $key }}={{ $value }}"
           {{- end }}

templates/query-frontend/query-frontend-dep.yaml

@@ -50,7 +50,6 @@ spec:
           args:
             - "-target=query-frontend"
             - "-config.file=/etc/cortex/cortex.yaml"
-            {{- include "cortex.frontend-memcached" . | nindent 12 }}
           {{- if .Values.query_scheduler.enabled }}
             - "-frontend.scheduler-address={{ template "cortex.querySchedulerFullname" . }}-headless.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.config.server.grpc_listen_port }}"
           {{- end }}

templates/ruler/ruler-dep.yaml

@@ -112,7 +112,6 @@ spec:
             - "-ruler.alertmanager-url=http://{{ template "cortex.alertmanagerFullname" . }}.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}:{{ .Values.config.server.http_listen_port }}/api/prom/alertmanager/"
               {{- end }}
             {{- end }}
-            {{- include "cortex.memcached" . | nindent 12}}
           {{- range $key, $value := .Values.ruler.extraArgs }}
             - "-{{ $key }}={{ $value }}"
           {{- end }}

templates/store-gateway/store-gateway-statefulset.yaml

@@ -110,7 +110,6 @@ spec:
           args:
             - "-target=store-gateway"
             - "-config.file=/etc/cortex/cortex.yaml"
-            {{- include "cortex.memcached" . | nindent 12}}
             {{- range $key, $value := .Values.store_gateway.extraArgs }}
             - "-{{ $key }}={{ $value }}"
             {{- end }}