Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,10 @@ All notable changes to this project will be documented in this file.
- Upgrade csi-provisioner to 5.2.0 ([#594]).
- Use versioned common structs ([e5224ab]).

### Removed

- Remove CSI registration path migration job ([#610]).

### Fixed

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

## [25.3.0] - 2025-03-21

Expand Down
27 changes: 0 additions & 27 deletions deploy/helm/secret-operator/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -83,33 +83,6 @@ spec:
mountPath: /registration
- name: csi
mountPath: /csi
initContainers:
# https://github.com/stackabletech/secret-operator/issues/251
# In https://github.com/stackabletech/secret-operator/pull/231 we introduced a breaking change by shortening the CSI registration path
# 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]}}"
# This init container removes the "old" directory, so that a unix socket with the same path can be created instead
# TODO: Should be removed in a release after 23.4
- name: migrate-longer-csi-registration-path
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
imagePullPolicy: {{ .Values.image.pullPolicy }}
resources:
{{ .Values.node.driver.resources | toYaml | nindent 12 }}
command:
- /bin/bash
- -euo
- pipefail
- -x
- -c
- |
ls -la /registration
echo "Removing old (long) CSI registration path"
if [ -d "/registration/secrets.stackable.tech-reg.sock" ]; then rmdir /registration/secrets.stackable.tech-reg.sock; fi
ls -la /registration
volumeMounts:
- name: registration-sock
mountPath: /registration
securityContext:
runAsUser: 0
volumes:
- name: registration-sock
hostPath:
Expand Down