Skip to content

Commit e542fd4

Browse files
kvapsclaude
andcommitted
test(stand): point csi-sanity at test-worker-1, drop fake-node init
Using --node=test-worker-1 means linstor-csi's volume reconciliation lookups land on the real node that autoplace selects. The fake csi-sanity-node init container was a workaround for the topology lookup that this approach makes unnecessary. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
1 parent c1647dc commit e542fd4

1 file changed

Lines changed: 8 additions & 18 deletions

File tree

stand/csi-sanity-job.yaml

Lines changed: 8 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -15,29 +15,19 @@ spec:
1515
template:
1616
spec:
1717
restartPolicy: Never
18-
initContainers:
19-
# csi-sanity is launched with --node=csi-sanity-node, so
20-
# linstor-csi expects that name to exist as a Node before any
21-
# CreateVolume call. Pre-register it as a SATELLITE — we don't
22-
# care about its actual presence; the topology lookup just
23-
# needs an entry.
24-
- name: register-csi-sanity-node
25-
image: curlimages/curl:8.10.1
26-
command:
27-
- sh
28-
- -c
29-
- |
30-
set -e
31-
curl -fsS -XPOST -H 'content-type: application/json' \
32-
http://blockstor-controller.blockstor-system.svc:3370/v1/nodes \
33-
-d '{"name":"csi-sanity-node","type":"SATELLITE","net_interfaces":[{"name":"default","address":"127.0.0.1","satellite_port":3366,"satellite_encryption_type":"PLAIN"}]}' \
34-
|| true
3518
containers:
3619
- name: linstor-csi
3720
image: quay.io/piraeusdatastore/piraeus-csi:v1.10.1
21+
# Use a real Node (test-worker-1) so linstor-csi's
22+
# ControllerPublishVolume / NodeStageVolume lookups land on
23+
# the node where autoplace actually placed the Resource.
24+
# csi-sanity drives CSI gRPC against this single pod, so
25+
# there's no stricture that --node match the test framework
26+
# node — we only need the name to exist in the Node CRD store
27+
# and have a storage pool that autoplace will pick.
3828
args:
3929
- --csi-endpoint=$(CSI_ENDPOINT)
40-
- --node=csi-sanity-node
30+
- --node=test-worker-1
4131
- --linstor-endpoint=$(LS_CONTROLLERS)
4232
env:
4333
- {name: CSI_ENDPOINT, value: "unix:///csi/csi.sock"}

0 commit comments

Comments
 (0)