Skip to content

Commit 8912bf6

Browse files
fix(migration): add internet access role for OIDC enabled jobs (#1032)
* fix(migration): add internet access role for OIDC enabled jobs Signed-off-by: Guilhem Barthés <guilhem.barthes@owkin.com> * fix: replace ingress by egress Signed-off-by: Guilhem Barthés <guilhem.barthes@owkin.com> * feat: add doc & bump chart Signed-off-by: Guilhem Barthés <guilhem.barthes@owkin.com> --------- Signed-off-by: Guilhem Barthés <guilhem.barthes@owkin.com>
1 parent b691f7a commit 8912bf6

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

charts/substra-backend/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
# Changelog
22

33
<!-- towncrier release notes start -->
4+
## [26.15.3] - 2025-30-05
5+
6+
Allow OIDC connection to job-migration
7+
48
## [26.15.2] - 2025-02-17
59

610
Inject OIDC config in migration job & bump app version to 1.0.1

charts/substra-backend/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
apiVersion: v2
22
name: substra-backend
33
home: https://github.com/Substra
4-
version: 26.15.2
4+
version: 26.15.3
55
appVersion: 1.0.1
66
kubeVersion: '>= 1.19.0-0'
77
description: Main package for Substra

charts/substra-backend/templates/job-migrations.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@ spec:
1515
labels:
1616
{{ include "substra.labels" . | nindent 10 }}
1717
role-database-client: 'true'
18+
# If OIDC is enabled, we need to allow the job to access the internet to fetch the user info, as it is based on the user info.
19+
{{- if .Values.oidc.enabled }}
20+
role-internet-egress: 'true'
21+
{{- end }}
1822
spec:
1923
restartPolicy: OnFailure
2024
automountServiceAccountToken: false

0 commit comments

Comments
 (0)