Provide automatic migration 23.1 -> 23.4

Author: sbernauer

CHANGELOG.md

@@ -7,15 +7,23 @@ All notable changes to this project will be documented in this file.
 ### Added
 
 - Generate OLM bundle for Release 23.4.0 ([#74]).
+- Provide automatic migration 23.1 -> 23.4 ([#77]).
 
 [#74]: https://github.com/stackabletech/listener-operator/pull/74
+[#77]: https://github.com/stackabletech/listener-operator/pull/77
 
 ## [23.4.0] - 2023-04-17
 
 ### Added
 
 - Allow configuring CSI docker images ([#61]).
 
+### Changed
+
+- Shortened the registration socket path for Microk8s compatibility ([#45]).
+  - The old CSI registration path will be automatically migrated during upgrade to `23.4.1` ([#77]).
+  - You might need to manually remove `/var/lib/kubelet/plugins_registry/listeners.stackable.tech-reg.sock` when downgrading.
+
 [#61]: https://github.com/stackabletech/listener-operator/pull/61
 
 ## [23.1.0] - 2023-01-23
@@ -24,14 +32,10 @@ All notable changes to this project will be documented in this file.
 
 - Helm installation on OpenShift ([#29]).
 - `operator-rs` `0.25.2` -> `0.27.1` ([#34]).
-- Shortened the registration socket path for Microk8s compatibility ([#45]).
-  - After upgrading you will need to
-    `rmdir /var/lib/kubelet/plugins_registry/listeners.stackable.tech-reg.sock` manually.
-    This applies to *all* users, not just Microk8s.
 - Made kubeletDir configurable ([#45]).
   - Microk8s users will need to `--set kubeletDir=/var/snap/microk8s/common/var/lib/kubelet`.
 
-
 [#29]: https://github.com/stackabletech/listener-operator/pull/29
 [#34]: https://github.com/stackabletech/listener-operator/pull/34
 [#45]: https://github.com/stackabletech/listener-operator/pull/45
+[#77]: https://github.com/stackabletech/listener-operator/pull/77

deploy/helm/listener-operator/templates/node-daemonset.yaml

@@ -63,6 +63,31 @@ spec:
               mountPath: /registration
             - name: csi
               mountPath: /csi
+      initContainers:
+        # https://github.com/stackabletech/listener-operator/issues/76
+        # In https://github.com/stackabletech/listener-operator/pull/45 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/listeners.stackable.tech-reg.sock but it's not a socket.: &{name:listeners.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 }}
+          command:
+            - /bin/bash
+            - -euo
+            - pipefail
+            - -x
+            - -c
+            - |
+              ls -la /registration
+              echo "Removing old (long) CSI registration path"
+              if [ -d "/registration/listeners.stackable.tech-reg.sock" ]; then rmdir /registration/listeners.stackable.tech-reg.sock; fi
+              ls -la /registration
+          volumeMounts:
+            - name: registration-sock
+              mountPath: /registration
+          securityContext:
+            runAsUser: 0
       volumes:
         - name: registration-sock
           hostPath: