Skip to content

Commit 82ed930

Browse files
authored
chore: remove CSI registration path migration job (#610)
* chore: remove CSI path migration job * add changelog entry
1 parent 35268ef commit 82ed930

File tree

2 files changed

+5
-27
lines changed

2 files changed

+5
-27
lines changed

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,10 @@ All notable changes to this project will be documented in this file.
3636
- Upgrade csi-provisioner to 5.2.0 ([#594]).
3737
- Use versioned common structs ([e5224ab]).
3838

39+
### Removed
40+
41+
- Remove CSI registration path migration job ([#610]).
42+
3943
### Fixed
4044

4145
- Use `json` file extension for log files ([#586]).
@@ -48,6 +52,7 @@ All notable changes to this project will be documented in this file.
4852
[#591]: https://github.com/stackabletech/secret-operator/pull/591
4953
[#594]: https://github.com/stackabletech/secret-operator/pull/594
5054
[e5224ab]: https://github.com/stackabletech/secret-operator/commit/e5224ab480e219e434ddc695c9361a16a56a43ed
55+
[#610]: https://github.com/stackabletech/secret-operator/pull/610
5156

5257
## [25.3.0] - 2025-03-21
5358

deploy/helm/secret-operator/templates/daemonset.yaml

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -83,33 +83,6 @@ spec:
8383
mountPath: /registration
8484
- name: csi
8585
mountPath: /csi
86-
initContainers:
87-
# https://github.com/stackabletech/secret-operator/issues/251
88-
# In https://github.com/stackabletech/secret-operator/pull/231 we introduced a breaking change by shortening the CSI registration path
89-
# This resulted in the following error "node_register.go:43] file exists in socketPath /registration/secrets.stackable.tech-reg.sock but it's not a socket.: &{name:secrets.stackable.tech-reg.sock size:4096 mode:2147484141 modTime:{wall:984732078ext:63815759330 loc:0xf864a0} sys:{Dev:64769 Ino:43688551 Nlink:2 Mode:16877 Uid:0 Gid:0 X__pad0:0 Rdev:0 Size:4096 Blksize:4096 Blocks:8 Atim:{Sec:1680162505 Nsec:36073186} Mtim:{Sec:1680162530 Nsec:984732078} Ctim:{Sec:1680162530 Nsec:984732078} X__unused:[0 0 0]}}"
90-
# This init container removes the "old" directory, so that a unix socket with the same path can be created instead
91-
# TODO: Should be removed in a release after 23.4
92-
- name: migrate-longer-csi-registration-path
93-
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
94-
imagePullPolicy: {{ .Values.image.pullPolicy }}
95-
resources:
96-
{{ .Values.node.driver.resources | toYaml | nindent 12 }}
97-
command:
98-
- /bin/bash
99-
- -euo
100-
- pipefail
101-
- -x
102-
- -c
103-
- |
104-
ls -la /registration
105-
echo "Removing old (long) CSI registration path"
106-
if [ -d "/registration/secrets.stackable.tech-reg.sock" ]; then rmdir /registration/secrets.stackable.tech-reg.sock; fi
107-
ls -la /registration
108-
volumeMounts:
109-
- name: registration-sock
110-
mountPath: /registration
111-
securityContext:
112-
runAsUser: 0
11386
volumes:
11487
- name: registration-sock
11588
hostPath:

0 commit comments

Comments
 (0)