Skip to content

Commit

Permalink
Apply last version of the Chart
Browse files Browse the repository at this point in the history
  • Loading branch information
garronej committed Sep 26, 2023
1 parent a125704 commit e2056aa
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
5 changes: 2 additions & 3 deletions helm-chart/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v2
name: onyxia
description: Onyxia is your datalab's hub.
icon: https://onyxia.sh/chart-icon.svg
icon: https://inseefrlab.github.io/onyxia/icon.svg
# A chart can be either an 'application' or a 'library' chart.
#
# Application charts are a collection of templates that can be packaged into versioned archives
Expand All @@ -14,5 +14,4 @@ type: application

# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
version: 4.0.1

version: 4.1.1
2 changes: 1 addition & 1 deletion helm-chart/templates/cluster-role-binding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ metadata:
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: ClusterRole
name: cluster-admin
name: {{ default "cluster-admin" .Values.serviceAccount.existingClusterRole }}
subjects:
- kind: ServiceAccount
name: {{ include "onyxia.api.serviceAccountName" . }}
Expand Down
6 changes: 5 additions & 1 deletion helm-chart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,11 @@ hostAliases: []
serviceAccount:
# Specifies whether a service account should be created
create: true
clusterAdmin: false # If true, give cluster admin permissions. Otherwise, be admin scoped to the namespace
# If true, the created service account is bound to a ClusterRole (default : cluster-admin) ;
# if false, it is instead bound to the admin Role, and thus scoped to the namespace
clusterAdmin: false
# Existing ClusterRole to use for the CRB if clusterAdmin is set to true
# existingClusterRole: ""
# Annotations to add to the service account
annotations: {}
# The name of the service account to use.
Expand Down

0 comments on commit e2056aa

Please sign in to comment.