Fix nil map in full snapshot configmap reconcile

If a full reconcile wins the race against sync of an individual snapshot resource, or someone intentionally deletes the configmap, the data map could be nil and cause a crash. Signed-off-by: Brad Davidson <brad.davidson@rancher.com>

Fix nil map in full snapshot configmap reconcile
319dca3e · Brad Davidson · Brad Davidson · db7091b3 · 319dca3e
Commit 319dca3e authored Dec 13, 2023 by Brad Davidson Committed by Brad Davidson Jan 04, 2024
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

snapshot_controller.go pkg/etcd/snapshot_controller.go +4 -0

No files found.
--- a/pkg/etcd/snapshot_controller.go
+++ b/pkg/etcd/snapshot_controller.go
@@ -252,6 +252,10 @@ func (e *etcdSnapshotHandler) reconcile() error {
 		}
 	}

+	if len(snapshots) > 0 && snapshotConfigMap.Data == nil {
+		snapshotConfigMap.Data = map[string]string{}
+	}
+
 	// Ensure keys for existing snapshots
 	for sfKey, esf := range snapshots {
 		sf := snapshotFile{}