This is an automated email from the ASF dual-hosted git repository.

yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.1 by this push:
     new d0a9e5656dc [fix](auth)Add some log in auth case (#38289)
d0a9e5656dc is described below

commit d0a9e5656dcfa62c44a671a6fcb208bb96823f46
Author: zfr95 <87513668+zfr9...@users.noreply.github.com>
AuthorDate: Thu Jul 25 14:46:01 2024 +0800

    [fix](auth)Add some log in auth case (#38289)
    
    pick from master #37362
    
    ## Proposed changes
    
    [fix](auth)Add some log in auth case
---
 .../test_master_slave_consistency_auth.groovy      | 49 ++++++++++++++--------
 1 file changed, 31 insertions(+), 18 deletions(-)

diff --git 
a/regression-test/suites/auth_p0/test_master_slave_consistency_auth.groovy 
b/regression-test/suites/auth_p0/test_master_slave_consistency_auth.groovy
index 33fbbdeccb7..acb59e74326 100644
--- a/regression-test/suites/auth_p0/test_master_slave_consistency_auth.groovy
+++ b/regression-test/suites/auth_p0/test_master_slave_consistency_auth.groovy
@@ -19,8 +19,9 @@ suite ("test_follower_consistent_auth","p0,auth") {
 
     def get_follower_ip = {
         def result = sql """show frontends;"""
+        logger.info("result:" + result)
         for (int i = 0; i < result.size(); i++) {
-            if (result[i][7] == "FOLLOWER" && result[i][8] == "false") {
+            if (result[i][7] == "FOLLOWER" && result[i][8] == "false" && 
result[i][11] == "true") {
                 return result[i][1]
             }
         }
@@ -29,7 +30,10 @@ suite ("test_follower_consistent_auth","p0,auth") {
     def switch_ip = get_follower_ip()
     if (switch_ip != "null") {
         logger.info("switch_ip: " + switch_ip)
-        def new_jdbc_url = context.config.jdbcUrl.replaceAll(/\/\/[0-9.]+:/, 
"//${switch_ip}:")
+
+        def tokens = context.config.jdbcUrl.split('/')
+        def url_tmp1 = tokens[0] + "//" + tokens[2] + "/" + 
"information_schema" + "?"
+        def new_jdbc_url = url_tmp1.replaceAll(/\/\/[0-9.]+:/, 
"//${switch_ip}:")
         logger.info("new_jdbc_url: " + new_jdbc_url)
 
         String user = 'test_follower_consistent_user'
@@ -101,8 +105,17 @@ suite ("test_follower_consistent_auth","p0,auth") {
         sql """refresh MATERIALIZED VIEW ${dbName}.${mtmv_name} auto"""
         sql """grant select_priv on regression_test to ${user}"""
 
+        //cloud-mode
+        if (isCloudMode()) {
+            def clusters = sql " SHOW CLUSTERS; "
+            assertTrue(!clusters.isEmpty())
+            def validCluster = clusters[0][0]
+            sql """GRANT USAGE_PRIV ON CLUSTER ${validCluster} TO ${user}""";
+        }
 
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        logger.info("url_tmp1:" + url_tmp1)
+        logger.info("new_jdbc_url:" + new_jdbc_url)
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             try {
                 sql "SHOW CATALOG RECYCLE BIN WHERE NAME = '${catalog_name}'"
             } catch (Exception e) {
@@ -118,7 +131,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
                 assertTrue(e.getMessage().contains("denied"))
             }
         }
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             try {
                 sql "SHOW DATA"
             } catch (Exception e) {
@@ -135,7 +148,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
             }
         }
 
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             try {
                 sql "select username from ${dbName}.${tableName}"
             } catch (Exception e) {
@@ -152,7 +165,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
             }
         }
         sql """grant select_priv(username) on ${dbName}.${tableName} to 
${user}"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             sql "select username from ${dbName}.${tableName}"
         }
         connect(user=user, password="${pwd}", url=new_jdbc_url) {
@@ -160,7 +173,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
         }
 
 
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             try {
                 sql "select username from ${dbName}.${view_name}"
             } catch (Exception e) {
@@ -177,7 +190,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
             }
         }
         sql """grant select_priv(username) on ${dbName}.${view_name} to 
${user}"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             sql "select username from ${dbName}.${view_name}"
         }
         connect(user=user, password="${pwd}", url=new_jdbc_url) {
@@ -185,7 +198,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
         }
 
 
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             try {
                 sql "select username from ${dbName}.${mtmv_name}"
             } catch (Exception e) {
@@ -202,7 +215,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
             }
         }
         sql """grant select_priv(username) on ${dbName}.${mtmv_name} to 
${user}"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             sql "select username from ${dbName}.${mtmv_name}"
         }
         connect(user=user, password="${pwd}", url=new_jdbc_url) {
@@ -215,7 +228,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
 
         // user
         sql """grant select_priv on ${dbName}.${tableName} to ${user}"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             sql "select username from ${dbName}.${tableName}"
         }
         connect(user=user, password="${pwd}", url=new_jdbc_url) {
@@ -223,7 +236,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
         }
 
         sql """revoke select_priv on ${dbName}.${tableName} from ${user}"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             try {
                 sql "select username from ${dbName}.${tableName}"
             } catch (Exception e) {
@@ -244,7 +257,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
         sql """grant select_priv on ${dbName}.${tableName} to ROLE '${role}'"""
         sql """grant Load_priv on ${dbName}.${tableName} to ROLE '${role}'"""
         sql """grant '${role}' to '${user}'"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             sql "select username from ${dbName}.${tableName}"
             sql """insert into ${dbName}.`${tableName}` values (4, "444")"""
         }
@@ -254,7 +267,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
         }
 
         sql """revoke '${role}' from '${user}'"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             try {
                 sql "select username from ${dbName}.${tableName}"
             } catch (Exception e) {
@@ -273,7 +286,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
 
 
         // workload group
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             sql """set workload_group = '${wg}';"""
             try {
                 sql "select username from ${dbName}.${tableName}"
@@ -292,7 +305,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
             }
         }
         sql """GRANT USAGE_PRIV ON WORKLOAD GROUP '${wg}' TO '${user}';"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             sql """set workload_group = '${wg}';"""
             sql """select username from ${dbName}.${tableName}"""
         }
@@ -302,7 +315,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
         }
 
         // resource group
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             def res = sql """SHOW RESOURCES;"""
             assertTrue(res == [])
         }
@@ -311,7 +324,7 @@ suite ("test_follower_consistent_auth","p0,auth") {
             assertTrue(res == [])
         }
         sql """GRANT USAGE_PRIV ON RESOURCE ${rg} TO ${user};"""
-        connect(user=user, password="${pwd}", url=context.config.jdbcUrl) {
+        connect(user=user, password="${pwd}", url=url_tmp1) {
             def res = sql """SHOW RESOURCES;"""
             assertTrue(res.size == 10)
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to