You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ta...@apache.org on 2019/05/06 22:41:14 UTC

[impala] 02/02: IMPALA-8488: Fix hardcoded path in Ranger E2E test on S3

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

tarmstrong pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git

commit ab47981098a6492bdc5c36c90fd9f742100298b5
Author: Austin Nobis <an...@cloudera.com>
AuthorDate: Fri May 3 11:59:24 2019 -0500

    IMPALA-8488: Fix hardcoded path in Ranger E2E test on S3
    
    A hardcoded path in test_ranger.py for URI testing was updated to
    support S3, local, and HDFS as opposed to just HDFS.
    
    Testing:
    - Ran authorization E2E tests
    - Ran all FE tests
    - Ran test_ranger.py with S3
    
    Change-Id: Ie2c021ce212f483a644fdab4e77ab95031066b14
    Reviewed-on: http://gerrit.cloudera.org:8080/13234
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 tests/authorization/test_ranger.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/authorization/test_ranger.py b/tests/authorization/test_ranger.py
index b66cc58..cd9ff57 100644
--- a/tests/authorization/test_ranger.py
+++ b/tests/authorization/test_ranger.py
@@ -24,6 +24,7 @@ import requests
 
 from getpass import getuser
 from tests.common.custom_cluster_test_suite import CustomClusterTestSuite
+from tests.util.hdfs_util import NAMENODE
 
 ADMIN = "admin"
 RANGER_AUTH = ("admin", "admin")
@@ -261,7 +262,7 @@ class TestRanger(CustomClusterTestSuite):
       result = self.client.execute("show grant {0} {1} on uri '{2}'"
                                    .format(kw, id, uri))
       TestRanger._check_privileges(result, [
-        [kw, id, "", "", "", "hdfs://localhost:20500" + uri, "", "all", "false"]])
+        [kw, id, "", "", "", "{0}{1}".format(NAMENODE, uri), "", "all", "false"]])
 
       # Revoke uri privileges and verify
       admin_client.execute("revoke all on uri '{0}' from {1} {2}"