From ce309505f9157cfc2cba21b7d5480cbcc0b5f2e6 Mon Sep 17 00:00:00 2001 From: Nick Hudson Date: Mon, 11 Dec 2023 22:41:15 -0600 Subject: [PATCH] only check backup status on restores --- tembo-operator/src/controller.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/tembo-operator/src/controller.rs b/tembo-operator/src/controller.rs index 99d924d05..af47c291f 100644 --- a/tembo-operator/src/controller.rs +++ b/tembo-operator/src/controller.rs @@ -313,8 +313,9 @@ impl CoreDB { reconcile_extensions(self, ctx.clone(), &coredbs, &name).await?; // Make sure the initial backup has completed if enabled before finishing - // reconciliation of the CoreDB resource + // reconciliation of the CoreDB resource that is being restored if cfg.enable_backup + && self.spec.restore.is_some() && self .status .as_ref() @@ -322,10 +323,6 @@ impl CoreDB { .is_none() { check_backups_status(self, ctx.clone()).await?; - // if let Err(e) = check_backups_status(self, ctx.clone()).await { - // error!("Error checking backups status: {:?}", e); - // return Err(Action::requeue(Duration::from_secs(300))); - // } } let recovery_time = self.get_recovery_time(ctx.clone()).await?;