Skip to content

Commit

Permalink
update(yaml): update priority class name for csi driver (#148)
Browse files Browse the repository at this point in the history
* update(yaml): update priority class name for csi driver

use jiva in priority class name to avoid naming clash between
different storage engine.


* give option to change priority class value

now user can set priority value in priority class
use csiController.priorityClass.value and
csiNode.priorityClass.value key to define the values

Signed-off-by: Shovan Maity <[email protected]>
  • Loading branch information
shovanmaity authored Sep 9, 2021
1 parent 35d9d9a commit cb4fd15
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 7 deletions.
2 changes: 1 addition & 1 deletion deploy/helm/charts/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ description: Helm chart for OpenEBS Jiva Operator. Jiva provides highly availabl
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: 2.12.2
version: 2.12.3
# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application.
appVersion: 2.12.2
Expand Down
4 changes: 2 additions & 2 deletions deploy/helm/charts/templates/priority-class.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: {{ template "jiva.csiController.priorityClassName" . }}
value: 900000000
value: {{ .Values.csiController.priorityClass.value }}
globalDefault: false
description: "This priority class should be used for the OpenEBS CSI driver controller deployment only."
{{- end }}
Expand All @@ -13,7 +13,7 @@ apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: {{ template "jiva.csiNode.priorityClassName" . }}
value: 900001000
value: {{ .Values.csiNode.priorityClass.value }}
globalDefault: false
description: "This priority class should be used for the OpenEBS CSI driver node deployment only."
{{- end }}
2 changes: 2 additions & 0 deletions deploy/helm/charts/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ csiController:
priorityClass:
create: true
name: jiva-csi-controller-critical
value: 900000000
componentName: "openebs-jiva-csi-controller"
attacher:
name: "csi-attacher"
Expand Down Expand Up @@ -118,6 +119,7 @@ csiNode:
priorityClass:
create: true
name: jiva-csi-node-critical
value: 900001000
componentName: "openebs-jiva-csi-node"
driverRegistrar:
name: "csi-node-driver-registrar"
Expand Down
8 changes: 4 additions & 4 deletions deploy/jiva-csi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ metadata:
apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: openebs-csi-controller-critical
name: openebs-jiva-csi-controller-critical
value: 900000000
globalDefault: false
description: "This priority class should be used for the CStor CSI driver controller deployment only."
Expand All @@ -45,7 +45,7 @@ description: "This priority class should be used for the CStor CSI driver contro
apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: openebs-csi-node-critical
name: openebs-jiva-csi-node-critical
value: 900001000
globalDefault: false
description: "This priority class should be used for the CStor CSI driver node deployment only."
Expand Down Expand Up @@ -132,7 +132,7 @@ spec:
openebs.io/component-name: openebs-jiva-csi-controller
openebs.io/version: dev
spec:
priorityClassName: openebs-csi-controller-critical
priorityClassName: openebs-jiva-csi-controller-critical
serviceAccount: openebs-jiva-csi-controller-sa
containers:
- name: csi-provisioner
Expand Down Expand Up @@ -358,7 +358,7 @@ spec:
openebs.io/component-name: openebs-jiva-csi-node
openebs.io/version: dev
spec:
priorityClassName: openebs-csi-node-critical
priorityClassName: openebs-jiva-csi-node-critical
serviceAccount: openebs-jiva-csi-node-sa
hostNetwork: true
containers:
Expand Down

0 comments on commit cb4fd15

Please sign in to comment.