fix: avoid panicking if serverRecoveryWindow has still not been set (cloudnative-pg#525)

leonardoce · rossigee · commit 9e3ccac22c81 · 2025-10-07T04:37:48.000+07:00
Closes: cloudnative-pg#523 Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
diff --git a/internal/cnpgi/instance/recovery_window.go b/internal/cnpgi/instance/recovery_window.go
@@ -64,6 +64,7 @@ func setLastFailedBackupTime(
 			objectStore.Status.ServerRecoveryWindow = make(map[string]barmancloudv1.RecoveryWindow)
 		}
 		objectStore.Status.ServerRecoveryWindow[serverName] = recoveryWindow
+
 		return c.Status().Update(ctx, &objectStore)
 	})
 }

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,7 @@ func setLastFailedBackupTime(`
`64`	`64`	`objectStore.Status.ServerRecoveryWindow = make(map[string]barmancloudv1.RecoveryWindow)`
`65`	`65`	`}`
`66`	`66`	`objectStore.Status.ServerRecoveryWindow[serverName] = recoveryWindow`
	`67`	`+`
`67`	`68`	`return c.Status().Update(ctx, &objectStore)`
`68`	`69`	`})`
`69`	`70`	`}`