Skip to content

Commit c3bffe0

Browse files
Merge branch 'main' into feat/proxyVarsFromSecret
2 parents c078e90 + 2e4cf2f commit c3bffe0

File tree

7 files changed

+63
-15
lines changed

7 files changed

+63
-15
lines changed

.github/renovate.json5

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
automerge: false,
77
assignees: [
88
'pierluigilenoci',
9+
'tuunit',
910
],
1011
schedule: [
1112
'before 5am',

.github/workflows/lint-test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939

4040
- name: Install Prometheus Operator CRDs
4141
id: prom
42-
run: kubectl apply --server-side -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/release-0.78/example/prometheus-operator-crd/monitoring.coreos.com_servicemonitors.yaml
42+
run: kubectl apply --server-side -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.80.0/example/prometheus-operator-crd/monitoring.coreos.com_servicemonitors.yaml
4343
if: steps.list-changed.outputs.changed == 'true'
4444

4545
- name: Run chart-testing (install)

helm/oauth2-proxy/Chart.yaml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: oauth2-proxy
2-
version: 7.10.4
2+
version: 7.10.5
33
apiVersion: v2
44
appVersion: 7.8.1
55
home: https://oauth2-proxy.github.io/oauth2-proxy/
@@ -22,15 +22,12 @@ sources:
2222
- https://github.com/oauth2-proxy/oauth2-proxy
2323
- https://github.com/oauth2-proxy/manifests
2424
maintainers:
25-
- name: desaintmartin
26-
27-
- name: tlawrie
28-
- name: NickMeves
29-
30-
- name: JoelSpeed
31-
3225
- name: pierluigilenoci
3326
27+
- name: tuunit
28+
29+
- name: JoelSpeed
30+
3431
kubeVersion: ">=1.16.0-0"
3532
annotations:
3633
artifacthub.io/changes: |
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
sessionStorage:
2+
type: redis
3+
redis:
4+
clientType: sentinel
5+
sentinel:
6+
password: "foo"
7+
masterName: "mymaster"
8+
connectionUrls:
9+
- "redis://oauth2-proxy-redis:26379"
10+
- "redis://oauth2-proxy-redis:26379"
11+
redis:
12+
# provision an instance of the redis sub-chart
13+
enabled: true
14+
fullnameOverride: oauth2-proxy-redis
15+
architecture: replication
16+
sentinel:
17+
enabled: true
18+
masterSet: mymaster
19+
global:
20+
redis:
21+
password: "foo"
22+
initContainers:
23+
waitForRedis:
24+
enabled: true
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
sessionStorage:
2+
type: redis
3+
redis:
4+
clientType: sentinel
5+
sentinel:
6+
password: "foo"
7+
masterName: "mymaster"
8+
connectionUrls: "redis://oauth2-proxy-redis:26379,redis://oauth2-proxy-redis:26379"
9+
redis:
10+
# provision an instance of the redis sub-chart
11+
enabled: true
12+
fullnameOverride: oauth2-proxy-redis
13+
architecture: replication
14+
sentinel:
15+
enabled: true
16+
masterSet: mymaster
17+
global:
18+
redis:
19+
password: "foo"
20+
initContainers:
21+
waitForRedis:
22+
enabled: true

helm/oauth2-proxy/templates/deployment.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,12 +79,12 @@ spec:
7979
- name: OAUTH2_PROXY_REDIS_USE_CLUSTER
8080
value: "true"
8181
- name: OAUTH2_PROXY_REDIS_CLUSTER_CONNECTION_URLS
82-
value: {{ .Values.sessionStorage.redis.cluster.connectionUrls }}
82+
value: {{ join "," .Values.sessionStorage.redis.cluster.connectionUrls }}
8383
{{- else if eq (default "" .Values.sessionStorage.redis.clientType) "sentinel" }}
8484
- name: OAUTH2_PROXY_REDIS_USE_SENTINEL
8585
value: "true"
8686
- name: OAUTH2_PROXY_REDIS_SENTINEL_CONNECTION_URLS
87-
value: {{ .Values.sessionStorage.redis.sentinel.connectionUrls }}
87+
value: {{ join "," .Values.sessionStorage.redis.sentinel.connectionUrls }}
8888
{{- end }}
8989
{{- if .Values.initContainers.waitForRedis.securityContext.enabled }}
9090
{{- $securityContext := unset .Values.initContainers.waitForRedis.securityContext "enabled" }}
@@ -218,14 +218,14 @@ spec:
218218
- name: OAUTH2_PROXY_REDIS_USE_CLUSTER
219219
value: "true"
220220
- name: OAUTH2_PROXY_REDIS_CLUSTER_CONNECTION_URLS
221-
value: {{ .Values.sessionStorage.redis.cluster.connectionUrls }}
221+
value: {{ join "," .Values.sessionStorage.redis.cluster.connectionUrls }}
222222
{{- else if eq (default "" .Values.sessionStorage.redis.clientType) "sentinel" }}
223223
- name: OAUTH2_PROXY_REDIS_USE_SENTINEL
224224
value: "true"
225225
- name: OAUTH2_PROXY_REDIS_SENTINEL_MASTER_NAME
226226
value: {{ .Values.sessionStorage.redis.sentinel.masterName }}
227227
- name: OAUTH2_PROXY_REDIS_SENTINEL_CONNECTION_URLS
228-
value: {{ .Values.sessionStorage.redis.sentinel.connectionUrls }}
228+
value: {{ join "," .Values.sessionStorage.redis.sentinel.connectionUrls }}
229229
{{- if or .Values.sessionStorage.redis.sentinel.existingSecret .Values.sessionStorage.redis.existingSecret .Values.sessionStorage.redis.sentinel.password }}
230230
- name: OAUTH2_PROXY_REDIS_SENTINEL_PASSWORD
231231
valueFrom:

helm/oauth2-proxy/values.yaml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -374,8 +374,10 @@ sessionStorage:
374374
# URL of redis standalone server for redis session storage (e.g. `redis://HOST[:PORT]`). Automatically generated if not set
375375
connectionUrl: ""
376376
cluster:
377-
# List of Redis cluster connection URLs (e.g. `["redis://127.0.0.1:8000", "redis://127.0.0.1:8000"]`)
377+
# List of Redis cluster connection URLs. Array or single string allowed.
378378
connectionUrls: []
379+
# - "redis://127.0.0.1:8000"
380+
# - "redis://127.0.0.1:8001"
379381
sentinel:
380382
# Name of the Kubernetes secret containing the redis sentinel password value (see also `sessionStorage.redis.sentinel.passwordKey`). Default: `sessionStorage.redis.existingSecret`
381383
existingSecret: ""
@@ -385,8 +387,10 @@ sessionStorage:
385387
passwordKey: "redis-sentinel-password"
386388
# Redis sentinel master name
387389
masterName: ""
388-
# List of Redis sentinel connection URLs (e.g. `["redis://127.0.0.1:8000", "redis://127.0.0.1:8000"]`)
390+
# List of Redis cluster connection URLs. Array or single string allowed.
389391
connectionUrls: []
392+
# - "redis://127.0.0.1:8000"
393+
# - "redis://127.0.0.1:8001"
390394

391395
# Enables and configure the automatic deployment of the redis subchart
392396
redis:

0 commit comments

Comments
 (0)