Skip to content

Commit ef4614e

Browse files
authored
refactoring (#1)
* refactoring
1 parent b0aff95 commit ef4614e

File tree

14 files changed

+46
-39
lines changed

14 files changed

+46
-39
lines changed

Chart.yaml

Lines changed: 0 additions & 7 deletions
This file was deleted.

charts/vcloud-csi/Chart.yaml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
apiVersion: v2
2+
name: vcloud-csi
3+
description: |
4+
Container Storage Interface (CSI) driver for VMware Cloud Director Named Independent Disks.
5+
Source: https://github.com/vmware/cloud-director-named-disk-csi-driver
6+
type: application
7+
version: v1.6.0
8+
appVersion: "1.6.0"
9+
icon: https://raw.githubusercontent.com/kubernetes/cloud-provider-vsphere/master/docs/vmware_logo.png
10+
home: https://github.com/clastix/vcloud-csi
11+
sources: ["https://github.com/clastix/vcloud-csi"]
12+

templates/csi-controller.yaml renamed to charts/vcloud-csi/templates/csi-controller.yaml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ apiVersion: v1
33
kind: ServiceAccount
44
metadata:
55
name: csi-vcd-controller-sa
6-
namespace: kube-system
6+
namespace: {{ .Release.Namespace }}
77

88
---
99
# external attacher
@@ -36,7 +36,7 @@ metadata:
3636
subjects:
3737
- kind: ServiceAccount
3838
name: csi-vcd-controller-sa
39-
namespace: kube-system
39+
namespace: {{ .Release.Namespace }}
4040
roleRef:
4141
kind: ClusterRole
4242
name: csi-attacher-role
@@ -82,7 +82,7 @@ metadata:
8282
subjects:
8383
- kind: ServiceAccount
8484
name: csi-vcd-controller-sa
85-
namespace: kube-system
85+
namespace: {{ .Release.Namespace }}
8686
roleRef:
8787
kind: ClusterRole
8888
name: csi-provisioner-role
@@ -119,7 +119,7 @@ metadata:
119119
subjects:
120120
- kind: ServiceAccount
121121
name: csi-vcd-controller-sa
122-
namespace: kube-system
122+
namespace: {{ .Release.Namespace }}
123123
roleRef:
124124
kind: ClusterRole
125125
name: csi-resizer-role
@@ -130,7 +130,7 @@ kind: Deployment
130130
apiVersion: apps/v1
131131
metadata:
132132
name: csi-vcd-controllerplugin
133-
namespace: kube-system
133+
namespace: {{ .Release.Namespace }}
134134
spec:
135135
replicas: 1
136136
selector:
@@ -190,7 +190,7 @@ spec:
190190
capabilities:
191191
add: ["SYS_ADMIN"]
192192
allowPrivilegeEscalation: true
193-
image: {{ .Values.csiImage.repository }}:{{ .Values.csiImage.tag }}
193+
image: "{{ .Values.csiImage.repository }}:{{ .Values.csiImage.tag | default .Chart.AppVersion }}"
194194
imagePullPolicy: {{ .Values.csiImage.pullPolicy }}
195195
command:
196196
- /opt/vcloud/bin/cloud-director-named-disk-csi-driver
@@ -250,7 +250,7 @@ spec:
250250
name: vcloud-csi-configmap
251251
- name: vcloud-basic-auth-volume
252252
secret:
253-
secretName: vcloud-basic-auth
253+
secretName: {{ .Values.authentication.credentialsSecret.name }}
254254
- name: vcloud-capvcd-zones
255255
configMap:
256256
name: vcloud-capvcd-zones

templates/csi-driver.yaml renamed to charts/vcloud-csi/templates/csi-driver.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,4 @@ metadata:
66
spec:
77
attachRequired: true
88
podInfoOnMount: false
9-
---
9+
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ apiVersion: v1
33
kind: ServiceAccount
44
metadata:
55
name: csi-vcd-node-sa
6-
namespace: kube-system
6+
namespace: {{ .Release.Namespace }}
77

88
---
99
kind: ClusterRole
@@ -23,7 +23,7 @@ metadata:
2323
subjects:
2424
- kind: ServiceAccount
2525
name: csi-vcd-node-sa
26-
namespace: kube-system
26+
namespace: {{ .Release.Namespace }}
2727
roleRef:
2828
kind: ClusterRole
2929
name: csi-nodeplugin-role
@@ -34,7 +34,7 @@ kind: DaemonSet
3434
apiVersion: apps/v1
3535
metadata:
3636
name: csi-vcd-nodeplugin
37-
namespace: kube-system
37+
namespace: {{ .Release.Namespace }}
3838
spec:
3939
selector:
4040
matchLabels:
@@ -92,7 +92,7 @@ spec:
9292
capabilities:
9393
add: ["SYS_ADMIN"]
9494
allowPrivilegeEscalation: true
95-
image: {{ .Values.csiImage.repository }}:{{ .Values.csiImage.tag }}
95+
image: "{{ .Values.csiImage.repository }}:{{ .Values.csiImage.tag | default .Chart.AppVersion }}"
9696
imagePullPolicy: {{ .Values.csiImage.pullPolicy }}
9797
command :
9898
- /opt/vcloud/bin/cloud-director-named-disk-csi-driver
@@ -157,7 +157,7 @@ spec:
157157
name: vcloud-csi-configmap
158158
- name: vcloud-basic-auth-volume
159159
secret:
160-
secretName: vcloud-basic-auth
160+
secretName: {{ .Values.authentication.credentialsSecret.name }}
161161
- name: vcloud-capvcd-zones
162162
configMap:
163163
name: vcloud-capvcd-zones
File renamed without changes.
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
{{- if .Values.authentication.credentialsSecret.generate -}}
2+
apiVersion: v1
3+
kind: Secret
4+
metadata:
5+
name: {{ .Values.authentication.credentialsSecret.name | default "default-secret-name" | quote }}
6+
labels:
7+
component: {{ .Chart.Name }}
8+
namespace: {{ .Release.Namespace }}
9+
data:
10+
username: {{ .Values.authentication.username | b64enc | quote }}
11+
password: {{ .Values.authentication.password | b64enc | quote }}
12+
refreshToken: ""
13+
{{- end -}}

0 commit comments

Comments
 (0)