From 52b97ea561c684db4555fd6970223e4ddc5f8c78 Mon Sep 17 00:00:00 2001 From: dima Date: Thu, 2 Apr 2026 18:31:03 +0100 Subject: [PATCH 1/2] Fix logic bug - it should be if the backup section is blank OR it's present but the length of the storages is 0 --- pkg/pxc/backup/restore.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/pxc/backup/restore.go b/pkg/pxc/backup/restore.go index 82d5fde2c6..affaaf4596 100644 --- a/pkg/pxc/backup/restore.go +++ b/pkg/pxc/backup/restore.go @@ -290,7 +290,7 @@ func RestoreJob( } if pitr { - if cluster.Spec.Backup == nil && len(cluster.Spec.Backup.Storages) == 0 { + if cluster.Spec.Backup == nil || len(cluster.Spec.Backup.Storages) == 0 { return nil, errors.New("no storage section") } volumeMounts = []corev1.VolumeMount{} From a46d46bba4e2f56f86b3a0868e6ba9a3624c684f Mon Sep 17 00:00:00 2001 From: Andrii Dema Date: Tue, 14 Apr 2026 16:00:12 +0300 Subject: [PATCH 2/2] remove the check --- pkg/pxc/backup/restore.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkg/pxc/backup/restore.go b/pkg/pxc/backup/restore.go index affaaf4596..20df85502f 100644 --- a/pkg/pxc/backup/restore.go +++ b/pkg/pxc/backup/restore.go @@ -290,9 +290,6 @@ func RestoreJob( } if pitr { - if cluster.Spec.Backup == nil || len(cluster.Spec.Backup.Storages) == 0 { - return nil, errors.New("no storage section") - } volumeMounts = []corev1.VolumeMount{} volumes = []corev1.Volume{} command = []string{"/opt/percona/pitr", "recover"}