fix logic where we remove stalled backups

tembo-io · Sep 27, 2024 · d3e0fbf · d3e0fbf
1 parent 219f0fe
commit d3e0fbf
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/tembo-operator/src/cloudnativepg/hibernate.rs b/tembo-operator/src/cloudnativepg/hibernate.rs
@@ -180,6 +180,10 @@ pub async fn reconcile_cluster_hibernation(cdb: &CoreDB, ctx: &Arc<Context>) ->
                 name, hibernation_value
             );
             if cdb.spec.stop {
+                // Only remove stalled backups if the instance is stopped/paused
+                info!("Remove any stalled backups for paused instance {}", name);
+                removed_stalled_backups(cdb, ctx).await?;
+
                 info!("Fully reconciled stopped instance {}", name);
                 return Err(requeue_normal_with_jitter());
             }
@@ -204,9 +208,6 @@ pub async fn reconcile_cluster_hibernation(cdb: &CoreDB, ctx: &Arc<Context>) ->
     patch_cdb_status_merge(&coredbs, &name, patch_status).await?;
 
     if cdb.spec.stop {
-        // Only remove stalled backups if the instance is stopped/paused
-        info!("Remove any stalled backups for paused instance {}", name);
-        removed_stalled_backups(cdb, ctx).await?;
         info!("Fully reconciled stopped instance {}", name);
         return Err(requeue_normal_with_jitter());
     }