fix

dataroaring · Sep 17, 2024 · 8e9f94c · 8e9f94c
1 parent 6103b0c
commit 8e9f94c
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/regression-test/framework/src/main/groovy/org/apache/doris/regression/util/DebugPoint.groovy b/regression-test/framework/src/main/groovy/org/apache/doris/regression/util/DebugPoint.groovy
@@ -103,6 +103,10 @@ class DebugPoint {
  def enableDebugPointForAllBEs(String name, Map<String, String> params = null) {
  operateDebugPointForAllBEs({ host, port ->
  logger.info("enable debug point ${name} with params ${params} for BE $host:$port")
+ if (port == -1) {
+ logger.info("skip for BE $host:$port")
+ return
+ }
  enableDebugPoint(host, port, NodeType.BE, name, params)
  })
  }
@@ -111,6 +115,10 @@ class DebugPoint {
  def disableDebugPointForAllBEs(String name) {
  operateDebugPointForAllBEs { host, port ->
  logger.info("disable debug point ${name} for BE $host:$port")
+ if (port == -1) {
+ logger.info("skip for BE $host:$port")
+ return
+ }
  disableDebugPoint(host, port, NodeType.BE, name)
  }
  }
@@ -119,6 +127,10 @@ class DebugPoint {
  def clearDebugPointsForAllBEs() {
  operateDebugPointForAllBEs { host, port ->
  logger.info("clear debug point for BE $host:$port")
+ if (port == -1) {
+ logger.info("skip for BE $host:$port")
+ return
+ }
  clearDebugPoints(host, port, NodeType.BE)
  }
  }
@@ -137,6 +149,10 @@ class DebugPoint {
  def enableDebugPointForAllFEs(String name, Map<String, String> params = null) {
  operateDebugPointForAllFEs({ host, port ->
  logger.info("enable debug point ${name} with params ${params} for FE $host:$port")
+ if (port == -1) {
+ logger.info("skip for FE $host:$port")
+ return
+ }
  enableDebugPoint(host, port, NodeType.FE, name, params)
  })
  }
@@ -145,7 +161,7 @@ class DebugPoint {
  operateDebugPointForAllFEs { host, port ->
  logger.info("disable debug point ${name} for FE $host:$port")
  if (port == -1) {
- logger.info("skip for BE $host:$port")
+ logger.info("skip for FE $host:$port")
  return
  }
  disableDebugPoint(host, port, NodeType.FE, name)
@@ -156,7 +172,7 @@ class DebugPoint {
  operateDebugPointForAllFEs { host, port ->
  logger.info("clear debug point for FE $host:$port")
  if (port == -1) {
- logger.info("skip for BE $host:$port")
+ logger.info("skip for FE $host:$port")
  return
  }
  clearDebugPoints(host, port, NodeType.FE)

diff --git a/regression-test/suites/cloud_p0/auth/test_grant_revoke_compute_group_to_user.groovy b/regression-test/suites/cloud_p0/auth/test_grant_revoke_compute_group_to_user.groovy
@@ -66,6 +66,7 @@ suite("test_grant_revoke_compute_group_to_user", "cloud_auth") {
  }
 
  def groups = sql " SHOW COMPUTE GROUPS; "
+ logger.info("compute groups {}", groups);
  assertTrue(!groups.isEmpty())
  def validCluster = groups[0][0]