fix(cloudid): avoid sync clouduser twice

This commit is contained in:
Qu Xuan
2021-08-16 15:11:12 +08:00
parent 916fa8d4b0
commit 1bd783aef8

View File

@@ -552,7 +552,7 @@ func (self *SCloudaccount) getCloudusers() ([]SClouduser, error) {
func (self *SCloudaccount) GetCloudusers() ([]SClouduser, error) {
users := []SClouduser{}
q := ClouduserManager.Query().Equals("status", api.CLOUD_USER_STATUS_AVAILABLE).Equals("cloudaccount_id", self.Id)
q := ClouduserManager.Query().Equals("cloudaccount_id", self.Id)
err := db.FetchModelObjects(ClouduserManager, q, &users)
if err != nil {
return nil, errors.Wrap(err, "db.FetchModelObjects")
@@ -564,7 +564,7 @@ func (self *SCloudaccount) SyncCloudusers(ctx context.Context, userCred mcclient
result := compare.SyncResult{}
dbUsers, err := self.GetCloudusers()
if err != nil {
result.Error(errors.Wrap(err, "GetCloudusersByProviderId"))
result.Error(errors.Wrap(err, "GetCloudusers"))
return nil, nil, result
}