fix bug (#208)

openUmbrella · lisong · commit 8ff23fad5086 · 2019-07-12T23:41:34.000+08:00
diff --git a/core/services/client-manager.js b/core/services/client-manager.js
@@ -278,7 +278,7 @@ proto.reportStatusDeploy = function (deploymentKey, label, clientUniqueId, other
           if (_.isEmpty(metrics)) {
             return;
           }
-          if (constConfig.DEPLOYMENT_SUCCEEDED) {
+          if (_.eq(status, constConfig.DEPLOYMENT_SUCCEEDED)) {
             return metrics.increment(['installed', 'active'],{by: 1});
           } else {
             return metrics.increment(['installed', 'failed'],{by: 1});

Original file line number	Diff line number	Diff line change
`@@ -278,7 +278,7 @@ proto.reportStatusDeploy = function (deploymentKey, label, clientUniqueId, other`
`278`	`278`	`if (_.isEmpty(metrics)) {`
`279`	`279`	`return;`
`280`	`280`	`}`
`281`		`- if (constConfig.DEPLOYMENT_SUCCEEDED) {`
	`281`	`+ if (_.eq(status, constConfig.DEPLOYMENT_SUCCEEDED)) {`
`282`	`282`	`return metrics.increment(['installed', 'active'],{by: 1});`
`283`	`283`	`} else {`
`284`	`284`	`return metrics.increment(['installed', 'failed'],{by: 1});`