Skip to content

Commit

Permalink
fix(chart): add role on builder pod (#1027)
Browse files Browse the repository at this point in the history
The builder pod always need access to the kubernetes API since it will
spawn kaniko builder pods.

Signed-off-by: Matthieu Blottière Seroux <[email protected]>
  • Loading branch information
mblottiere authored Jan 6, 2025
1 parent 86619f3 commit 198bce4
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
6 changes: 5 additions & 1 deletion charts/substra-backend/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
# Changelog

<!-- towncrier release notes start -->
## [26.15.1] - 2025-01-06

Fix missing role on builder pod

## [26.15.0] - 2024-12-16

Bump app version to 1.0.0
Expand All @@ -25,7 +29,7 @@ Pods relying on redis uses a new init container `wait-redis` which wait for redi

# Added

`.Values.builder.saveImageTimeoutSeconds` configuring the timeout for the Celery task `SaveImageTask`.
`.Values.builder.saveImageTimeoutSeconds` configuring the timeout for the Celery task `SaveImageTask`.

## [26.12.0] - 2024-09-30

Expand Down
2 changes: 1 addition & 1 deletion charts/substra-backend/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v2
name: substra-backend
home: https://github.com/Substra
version: 26.15.0
version: 26.15.1
appVersion: 1.0.0
kubeVersion: '>= 1.19.0-0'
description: Main package for Substra
Expand Down
3 changes: 0 additions & 3 deletions charts/substra-backend/templates/statefulset-builder.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,10 +33,7 @@ spec:
app.kubernetes.io/name: {{ template "substra.name" . }}-builder
{{ include "substra.labels" . | nindent 8 }}
app.kubernetes.io/component: substra-builder
{{- if .Values.privateCa.enabled }}
# Needed to create the secret with the certificate
role-api-k8s-client: 'true'
{{- end }}
role-database-client: 'true'
role-internet-egress: 'true'
role-orchestrator-client: 'true'
Expand Down

0 comments on commit 198bce4

Please sign in to comment.