fix(region): skip set lost name when purge deleted vm (#19752)

yunionio · Mar 20, 2024 · c73ac42 · c73ac42
1 parent 6cac127
commit c73ac42
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/pkg/compute/models/guests.go b/pkg/compute/models/guests.go
@@ -2974,13 +2974,6 @@ func (self *SGuest) SyncRemoveCloudVM(ctx context.Context, userCred mcclient.Tok
 		}
 	}
 
-	if !lostNamePattern.MatchString(self.Name) {
-		db.Update(self, func() error {
-			self.Name = fmt.Sprintf("%s-lost@%s", self.Name, timeutils.ShortDate(time.Now()))
-			return nil
-		})
-	}
-
 	if self.Status != api.VM_UNKNOWN {
 		self.SetStatus(ctx, userCred, api.VM_UNKNOWN, "Sync lost")
 	}
@@ -2995,6 +2988,15 @@ func (self *SGuest) SyncRemoveCloudVM(ctx context.Context, userCred mcclient.Tok
 			Obj:    self,
 			Action: notifyclient.ActionSyncDelete,
 		})
+
+		return nil
+	}
+
+	if !lostNamePattern.MatchString(self.Name) {
+		db.Update(self, func() error {
+			self.Name = fmt.Sprintf("%s-lost@%s", self.Name, timeutils.ShortDate(time.Now()))
+			return nil
+		})
 	}
 
 	return nil