Skip to content

Commit 87eb688

Browse files
authored
Merge pull request kubernetes#90234 from jsafrane/remove-prestop-hook
Sync csi-hostpath manifests
2 parents 72fb952 + 4bcb3cd commit 87eb688

File tree

6 files changed

+59
-47
lines changed

6 files changed

+59
-47
lines changed

test/e2e/testing-manifests/storage-csi/hostpath/hostpath/csi-hostpath-attacher.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ spec:
4040
serviceAccountName: csi-attacher
4141
containers:
4242
- name: csi-attacher
43-
image: quay.io/k8scsi/csi-attacher:v2.1.0
43+
image: quay.io/k8scsi/csi-attacher:v2.2.0
4444
args:
4545
- --v=5
4646
- --csi-address=/csi/csi.sock

test/e2e/testing-manifests/storage-csi/hostpath/hostpath/csi-hostpath-driverinfo.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,10 @@ kind: CSIDriver
33
metadata:
44
name: hostpath.csi.k8s.io
55
spec:
6-
# Supports both modes, but needs pod info for that to determine the actual mode.
7-
podInfoOnMount: true
6+
# Supports persistent and ephemeral inline volumes.
87
volumeLifecycleModes:
98
- Persistent
109
- Ephemeral
10+
# To determine at runtime which mode a volume uses, pod info and its
11+
# "csi.storage.k8s.io/ephemeral" entry are needed.
12+
podInfoOnMount: true

test/e2e/testing-manifests/storage-csi/hostpath/hostpath/csi-hostpath-plugin.yaml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,7 @@ spec:
3636
spec:
3737
containers:
3838
- name: node-driver-registrar
39-
image: quay.io/k8scsi/csi-node-driver-registrar:v1.2.0
40-
lifecycle:
41-
preStop:
42-
exec:
43-
command: ["/bin/sh", "-c", "rm -rf /registration/csi-hostpath /registration/csi-hostpath-reg.sock"]
39+
image: quay.io/k8scsi/csi-node-driver-registrar:v1.3.0
4440
args:
4541
- --v=5
4642
- --csi-address=/csi/csi.sock
@@ -71,6 +67,8 @@ spec:
7167
- "--v=5"
7268
- "--endpoint=$(CSI_ENDPOINT)"
7369
- "--nodeid=$(KUBE_NODE_NAME)"
70+
# The only difference to github.com/kubernetes-csi/csi-driver-host-path/deploy
71+
# - we have a tests that checks node limits.
7472
- "--maxvolumespernode=10"
7573
env:
7674
- name: CSI_ENDPOINT
@@ -107,15 +105,13 @@ spec:
107105
name: csi-data-dir
108106
- mountPath: /dev
109107
name: dev-dir
110-
111108
- name: liveness-probe
112109
volumeMounts:
113110
- mountPath: /csi
114111
name: socket-dir
115112
image: quay.io/k8scsi/livenessprobe:v1.1.0
116113
args:
117114
- --csi-address=/csi/csi.sock
118-
- --connection-timeout=3s
119115
- --health-port=9898
120116

121117
volumes:

test/e2e/testing-manifests/storage-csi/hostpath/hostpath/csi-hostpath-provisioner.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,11 @@ spec:
4040
serviceAccountName: csi-provisioner
4141
containers:
4242
- name: csi-provisioner
43-
image: quay.io/k8scsi/csi-provisioner:v1.5.0
43+
image: quay.io/k8scsi/csi-provisioner:v1.6.0
4444
args:
4545
- -v=5
4646
- --csi-address=/csi/csi.sock
47-
- --connection-timeout=15s
47+
- --feature-gates=Topology=true
4848
securityContext:
4949
# This is necessary only for systems with SELinux, where
5050
# non-privileged sidecar containers cannot access unix domain socket

test/e2e/testing-manifests/storage-csi/hostpath/hostpath/csi-hostpath-resizer.yaml

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,16 +27,23 @@ spec:
2727
labels:
2828
app: csi-hostpath-resizer
2929
spec:
30+
affinity:
31+
podAffinity:
32+
requiredDuringSchedulingIgnoredDuringExecution:
33+
- labelSelector:
34+
matchExpressions:
35+
- key: app
36+
operator: In
37+
values:
38+
- csi-hostpathplugin
39+
topologyKey: kubernetes.io/hostname
3040
serviceAccountName: csi-resizer
3141
containers:
3242
- name: csi-resizer
33-
image: quay.io/k8scsi/csi-resizer:v0.4.0
43+
image: quay.io/k8scsi/csi-resizer:v0.5.0
3444
args:
35-
- "--v=5"
36-
- "--csi-address=$(ADDRESS)"
37-
env:
38-
- name: ADDRESS
39-
value: /csi/csi.sock
45+
- -v=5
46+
- -csi-address=/csi/csi.sock
4047
securityContext:
4148
# This is necessary only for systems with SELinux, where
4249
# non-privileged sidecar containers cannot access unix domain socket
Lines changed: 36 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,52 +1,59 @@
11
kind: Service
22
apiVersion: v1
33
metadata:
4-
name: csi-snapshotter
4+
name: csi-hostpath-snapshotter
55
labels:
6-
app: csi-snapshotter
6+
app: csi-hostpath-snapshotter
77
spec:
88
selector:
9-
app: csi-snapshotter
9+
app: csi-hostpath-snapshotter
1010
ports:
11-
- name: dummy
12-
port: 12345
11+
- name: dummy
12+
port: 12345
1313

1414
---
1515
kind: StatefulSet
1616
apiVersion: apps/v1
1717
metadata:
18-
name: csi-snapshotter
18+
name: csi-hostpath-snapshotter
1919
spec:
20-
serviceName: "csi-snapshotter"
20+
serviceName: "csi-hostpath-snapshotter"
2121
replicas: 1
2222
selector:
2323
matchLabels:
24-
app: csi-snapshotter
24+
app: csi-hostpath-snapshotter
2525
template:
2626
metadata:
2727
labels:
28-
app: csi-snapshotter
28+
app: csi-hostpath-snapshotter
2929
spec:
30+
affinity:
31+
podAffinity:
32+
requiredDuringSchedulingIgnoredDuringExecution:
33+
- labelSelector:
34+
matchExpressions:
35+
- key: app
36+
operator: In
37+
values:
38+
- csi-hostpathplugin
39+
topologyKey: kubernetes.io/hostname
3040
serviceAccount: csi-snapshotter
3141
containers:
32-
- name: csi-snapshotter
33-
image: quay.io/k8scsi/csi-snapshotter:v2.0.0
34-
args:
35-
- "--v=5"
36-
- "--csi-address=$(ADDRESS)"
37-
env:
38-
- name: ADDRESS
39-
value: /csi/csi.sock
40-
securityContext:
41-
# This is necessary only for systems with SELinux, where
42-
# non-privileged sidecar containers cannot access unix domain socket
43-
# created by privileged CSI driver container.
44-
privileged: true
45-
volumeMounts:
46-
- name: socket-dir
47-
mountPath: /csi
42+
- name: csi-snapshotter
43+
image: quay.io/k8scsi/csi-snapshotter:v2.1.0
44+
args:
45+
- -v=5
46+
- --csi-address=/csi/csi.sock
47+
securityContext:
48+
# This is necessary only for systems with SELinux, where
49+
# non-privileged sidecar containers cannot access unix domain socket
50+
# created by privileged CSI driver container.
51+
privileged: true
52+
volumeMounts:
53+
- mountPath: /csi
54+
name: socket-dir
4855
volumes:
49-
- hostPath:
50-
path: /var/lib/kubelet/plugins/csi-hostpath
51-
type: DirectoryOrCreate
52-
name: socket-dir
56+
- hostPath:
57+
path: /var/lib/kubelet/plugins/csi-hostpath
58+
type: DirectoryOrCreate
59+
name: socket-dir

0 commit comments

Comments
 (0)