Skip to content

Commit 63598bb

Browse files
authored
add option to set ingesters podManagementPolicy (#236)
* add option to set ingesters podManagementPolicy Signed-off-by: Andre Ziviani <[email protected]> * update changelog Signed-off-by: Andre Ziviani <[email protected]> * update readme Signed-off-by: Andre Ziviani <[email protected]> * add a warning about podmanagementpolicy Signed-off-by: Andre Ziviani <[email protected]>
1 parent bed9d1c commit 63598bb

File tree

4 files changed

+5
-0
lines changed

4 files changed

+5
-0
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
* [ENHANCEMENT] Optionally generate endpoints for `X-Scope-OrgID` injection (multi-tenancy) #180
1818
* [ENHANCEMENT] Optionally configure Basic Auth within Nginx #205
1919
* [ENHANCEMENT] Add extraEndpointSpec to all servicemonitors for specifying things like TLS #216
20+
* [ENHANCEMENT] Add option to specify `podManagementPolicy` when using ingesters as statefulset #236
2021
* [BUGFIX] Fix whitespace in runtime-config annotations, introduced in #209, fixed in #212
2122
* [BUGFIX] Correcting nginx config for auth orgs to right proxy_pass #192
2223
* [DEPENDENCY] Update Helm release memcached to v5.15.4 #228

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -434,6 +434,7 @@ Kubernetes: `^1.19.0-0`
434434
| ingester.&ZeroWidthSpace;startupProbe.&ZeroWidthSpace;initialDelaySeconds | int | `120` | |
435435
| ingester.&ZeroWidthSpace;startupProbe.&ZeroWidthSpace;periodSeconds | int | `30` | |
436436
| ingester.&ZeroWidthSpace;statefulSet.&ZeroWidthSpace;enabled | bool | `false` | |
437+
| ingester.&ZeroWidthSpace;statefulSet.&ZeroWidthSpace;podManagementPolicy | string | `"OrderedReady"` | see https://cortexmetrics.io/docs/guides/ingesters-scaling-up-and-down/#scaling-down and https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#pod-management-policies for scaledown details |
437438
| ingester.&ZeroWidthSpace;statefulStrategy.&ZeroWidthSpace;type | string | `"RollingUpdate"` | |
438439
| ingester.&ZeroWidthSpace;strategy.&ZeroWidthSpace;rollingUpdate.&ZeroWidthSpace;maxSurge | int | `0` | |
439440
| ingester.&ZeroWidthSpace;strategy.&ZeroWidthSpace;rollingUpdate.&ZeroWidthSpace;maxUnavailable | int | `1` | |

templates/ingester/ingester-statefulset.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ spec:
1818
{{- include "cortex.ingesterSelectorLabels" . | nindent 6 }}
1919
updateStrategy:
2020
{{- toYaml .Values.ingester.statefulStrategy | nindent 4 }}
21+
podManagementPolicy: "{{ .Values.ingester.statefulSet.podManagementPolicy }}"
2122
serviceName: {{ template "cortex.fullname" . }}-ingester-headless
2223
{{- if .Values.ingester.persistentVolume.enabled }}
2324
volumeClaimTemplates:

values.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -507,6 +507,8 @@ ingester:
507507
## This is useful when using WAL
508508
##
509509
enabled: false
510+
# -- see https://cortexmetrics.io/docs/guides/ingesters-scaling-up-and-down/#scaling-down and https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#pod-management-policies for scaledown details
511+
podManagementPolicy: OrderedReady
510512

511513
service:
512514
annotations: {}

0 commit comments

Comments
 (0)