Skip to content

Commit

Permalink
feat(charts): disable hostNetwork on node plugin
Browse files Browse the repository at this point in the history
Signed-off-by: Abhinandan Purkait <[email protected]>
Signed-off-by: Niladri Halder <[email protected]>
  • Loading branch information
Abhinandan-Purkait authored and niladrih committed Mar 18, 2024
1 parent 4ffa157 commit 3a386e7
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion deploy/helm/charts/templates/lvm-node.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ spec:
priorityClassName: {{ template "lvmlocalpv.lvmNode.priorityClassName" . }}
{{- end }}
serviceAccountName: {{ .Values.serviceAccount.lvmNode.name }}
hostNetwork: true
hostNetwork: {{ .Values.lvmNode.hostNetwork }}
containers:
- name: {{ .Values.lvmNode.driverRegistrar.name }}
image: "{{ .Values.lvmNode.driverRegistrar.image.registry }}{{ .Values.lvmNode.driverRegistrar.image.repository }}:{{ .Values.lvmNode.driverRegistrar.image.tag }}"
Expand Down
2 changes: 1 addition & 1 deletion deploy/helm/charts/templates/psp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
allowPrivilegeEscalation: true
allowedCapabilities: ['*']
volumes: ['*']
hostNetwork: true
hostNetwork: {{ .Values.lvmNode.hostNetwork}}
hostIPC: true
hostPID: true
runAsUser:
Expand Down
2 changes: 2 additions & 0 deletions deploy/helm/charts/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,8 @@ lvmNode:
# Configure the maximum number of queries allowed after
# accounting for rolled over qps from previous seconds.
burst: 0
# Disable or enable the use of hostNetwork for the lvm node daemonset.
hostNetwork: false


# lvmController contains the configurables for
Expand Down
1 change: 0 additions & 1 deletion deploy/lvm-operator.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1633,7 +1633,6 @@ spec:
spec:
priorityClassName: openebs-lvm-localpv-csi-node-critical
serviceAccountName: openebs-lvm-node-sa
hostNetwork: true
containers:
- name: csi-node-driver-registrar
image: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0
Expand Down
1 change: 0 additions & 1 deletion deploy/yamls/lvm-driver.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1197,7 +1197,6 @@ spec:
spec:
priorityClassName: openebs-lvm-localpv-csi-node-critical
serviceAccountName: openebs-lvm-node-sa
hostNetwork: true
containers:
- name: csi-node-driver-registrar
image: registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.8.0
Expand Down

0 comments on commit 3a386e7

Please sign in to comment.