fix: do not reset the clone and do not change the clone status if it is not started yet (#199)

agneum · agneum · commit d56038862536 · 2020-11-12T10:21:07.000+07:00
diff --git a/pkg/services/cloning/mode_base.go b/pkg/services/cloning/mode_base.go
@@ -277,17 +277,17 @@ func (c *baseCloning) ResetClone(cloneID string) error {
 		return models.New(models.ErrCodeNotFound, "clone not found")
 	}
 
+	if w.session == nil {
+		return models.New(models.ErrCodeNotFound, "clone is not started yet")
+	}
+
 	if err := c.updateCloneStatus(cloneID, models.Status{
 		Code:    models.StatusResetting,
 		Message: models.CloneMessageResetting,
 	}); err != nil {
 		return errors.Wrap(err, "failed to update clone status")
 	}
 
-	if w.session == nil {
-		return models.New(models.ErrCodeNotFound, "clone is not started yet")
-	}
-
 	go func() {
 		err := c.provision.ResetSession(w.session, w.snapshot.ID)
 		if err != nil {
