Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

Commit 6b1aae9

Browse files
authored
Merge pull request #173 from 11xor6/deployment-update-strategy
Fix missing permission and allow setting Deployment update strategy.
2 parents ed9eaac + b0a1a16 commit 6b1aae9

File tree

3 files changed

+11
-0
lines changed

3 files changed

+11
-0
lines changed

deploy/charts/kube-oidc-proxy/templates/clusterrole.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ rules:
1717
- "authentication.k8s.io"
1818
resources:
1919
- "userextras/scopes"
20+
- "userextras/remote-client-ip"
2021
- "tokenreviews"
2122
verbs:
2223
- "create"

deploy/charts/kube-oidc-proxy/templates/deployment.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ spec:
1313
matchLabels:
1414
app.kubernetes.io/name: {{ include "kube-oidc-proxy.name" . }}
1515
app.kubernetes.io/instance: {{ .Release.Name }}
16+
{{- if .Values.rollingUpdateStrategy }}
17+
strategy: {{ .Values.rollingUpdateStrategy | nindent 4}}
18+
{{ end }}
1619
template:
1720
metadata:
1821
labels:

deploy/charts/kube-oidc-proxy/values.yaml

+7
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,13 @@ ingress:
9191
# hosts:
9292
# - chart-example.local
9393

94+
# Allows setting the Deployment update strategy
95+
#rollingUpdateStrategy:
96+
# type: RollingUpdate
97+
# rollingUpdate:
98+
# maxSurge: 34%
99+
# maxUnavailable: 33%
100+
94101
# Enable Pod Disruption Budget
95102
podDisruptionBudget:
96103
enabled: false

0 commit comments

Comments
 (0)