update status when needed only
This commit is contained in:
parent
69128181ef
commit
a55046f3d7
@ -194,6 +194,10 @@ func (r *RestoreSessionReconciler) StatusUpdate() error {
|
|||||||
case formolv1alpha1.Failure:
|
case formolv1alpha1.Failure:
|
||||||
log.V(0).Info("last restore task failed. Stop here", "target", currentTargetStatus.Name)
|
log.V(0).Info("last restore task failed. Stop here", "target", currentTargetStatus.Name)
|
||||||
r.RestoreSession.Status.SessionState = formolv1alpha1.Failure
|
r.RestoreSession.Status.SessionState = formolv1alpha1.Failure
|
||||||
|
if err := r.Status().Update(ctx, r.RestoreSession); err != nil {
|
||||||
|
log.Error(err, "unable to update restoresession")
|
||||||
|
return err
|
||||||
|
}
|
||||||
case formolv1alpha1.Running:
|
case formolv1alpha1.Running:
|
||||||
log.V(0).Info("task is still running", "target", currentTargetStatus.Name)
|
log.V(0).Info("task is still running", "target", currentTargetStatus.Name)
|
||||||
return nil
|
return nil
|
||||||
@ -206,13 +210,13 @@ func (r *RestoreSessionReconciler) StatusUpdate() error {
|
|||||||
if targetStatus == nil {
|
if targetStatus == nil {
|
||||||
// No more task to start. The restore is over
|
// No more task to start. The restore is over
|
||||||
r.RestoreSession.Status.SessionState = formolv1alpha1.Success
|
r.RestoreSession.Status.SessionState = formolv1alpha1.Success
|
||||||
|
if err := r.Status().Update(ctx, r.RestoreSession); err != nil {
|
||||||
|
log.Error(err, "unable to update restoresession")
|
||||||
|
return err
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if err := r.Status().Update(ctx, r.RestoreSession); err != nil {
|
|
||||||
log.Error(err, "unable to update restoresession")
|
|
||||||
return err
|
|
||||||
}
|
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user