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}"