Skip to content

Commit

Permalink
don't set SyncedToRemote True in replicate
Browse files Browse the repository at this point in the history
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
  • Loading branch information
ushitora-anqou committed Oct 3, 2024
1 parent 7d2e4a7 commit 3eb81bb
Showing 1 changed file with 1 addition and 10 deletions.
11 changes: 1 addition & 10 deletions internal/controller/mantlebackup_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -385,7 +385,7 @@ func (r *MantleBackupReconciler) SetupWithManager(mgr ctrl.Manager) error {

func (r *MantleBackupReconciler) replicate(
ctx context.Context,
logger *slog.Logger,
_ *slog.Logger,
backup *mantlev1.MantleBackup,
) (ctrl.Result, error) {
// Skip replication if SyncedToRemote condition is true.
Expand Down Expand Up @@ -474,15 +474,6 @@ func (r *MantleBackupReconciler) replicate(
return ctrl.Result{}, err
}

// Update the status of the MantleBackup.
if err := r.updateStatusCondition(ctx, logger, backup, metav1.Condition{
Type: mantlev1.BackupConditionSyncedToRemote,
Status: metav1.ConditionTrue,
Reason: mantlev1.BackupReasonNone,
}); err != nil {
return ctrl.Result{}, err
}

return ctrl.Result{}, nil
}

Expand Down

0 comments on commit 3eb81bb

Please sign in to comment.