From 16cfab47e676846b966d27b8eee498d86dc2146c Mon Sep 17 00:00:00 2001
From: Qu Xuan <quxuan@yunionyun.com>
Date: Thu, 13 May 2021 11:05:53 +0800
Subject: [PATCH] fix(region): optimized sku sched clear

---
 pkg/compute/models/cloudsync.go  | 5 -----
 pkg/compute/models/skus.go       | 7 +++++++
 pkg/compute/models/skus_tools.go | 6 ------
 3 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/pkg/compute/models/cloudsync.go b/pkg/compute/models/cloudsync.go
index 87193309fc..6de027f3be 100644
--- a/pkg/compute/models/cloudsync.go
+++ b/pkg/compute/models/cloudsync.go
@@ -153,11 +153,6 @@ func syncRegionSkus(ctx context.Context, userCred mcclient.TokenCredential, loca
 			// logSyncFailed(provider, task, msg)
 			return
 		}
-
-		_, err = modules.SchedManager.SyncSku(auth.GetAdminSession(ctx, options.Options.Region, ""), false)
-		if err != nil {
-			log.Errorf("SchedManager SyncSku %s", err)
-		}
 	}
 
 	if localRegion.GetDriver().IsSupportedElasticcache() {
diff --git a/pkg/compute/models/skus.go b/pkg/compute/models/skus.go
index 17cf0b8222..3f0cf360c3 100644
--- a/pkg/compute/models/skus.go
+++ b/pkg/compute/models/skus.go
@@ -1257,6 +1257,13 @@ func (manager *SServerSkuManager) SyncServerSkus(ctx context.Context, userCred m
 			syncResult.Add()
 		}
 	}
+
+	// notfiy sched manager
+	_, err = modules.SchedManager.SyncSku(auth.GetAdminSession(ctx, options.Options.Region, ""), false)
+	if err != nil {
+		log.Errorf("SchedManager SyncSku %s", err)
+	}
+
 	return syncResult
 }
 
diff --git a/pkg/compute/models/skus_tools.go b/pkg/compute/models/skus_tools.go
index 36baf260e0..a7164cf4e1 100644
--- a/pkg/compute/models/skus_tools.go
+++ b/pkg/compute/models/skus_tools.go
@@ -479,12 +479,6 @@ func SyncServerSkusByRegion(ctx context.Context, userCred mcclient.TokenCredenti
 	notes := fmt.Sprintf("SyncServerSkusByRegion %s result: %s", region.Name, result.Result())
 	log.Infof(notes)
 
-	// notfiy sched manager
-	_, err = modules.SchedManager.SyncSku(auth.GetAdminSession(ctx, options.Options.Region, ""), false)
-	if err != nil {
-		log.Errorf("SchedManager SyncSku %s", err)
-	}
-
 	return result
 }