From c2c9edd9c69a6aa476b4f60f5a3c7a69c98f658e Mon Sep 17 00:00:00 2001
From: Qu Xuan <quxuan@yunionyun.com>
Date: Wed, 11 Nov 2020 15:25:37 +0800
Subject: [PATCH] fix: avoid check region log every time when account status is
 unknown

---
 pkg/compute/models/cloudaccounts.go | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/pkg/compute/models/cloudaccounts.go b/pkg/compute/models/cloudaccounts.go
index 97345d7856..6c1d5643ea 100644
--- a/pkg/compute/models/cloudaccounts.go
+++ b/pkg/compute/models/cloudaccounts.go
@@ -2413,8 +2413,10 @@ func (account *SCloudaccount) probeAccountStatus(ctx context.Context, userCred m
 		case cloudprovider.ErrNoBalancePermission:
 			status = api.CLOUD_PROVIDER_HEALTH_NO_PERMISSION
 		default:
-			log.Errorf("manager.GetBalance %s fail %s", account.Name, err)
 			status = api.CLOUD_PROVIDER_HEALTH_UNKNOWN
+			if account.Status != status {
+				logclient.AddSimpleActionLog(account, logclient.ACT_PROBE, errors.Wrapf(err, "GetBalance"), userCred, false)
+			}
 		}
 	}
 	version := manager.GetVersion()