You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by co...@apache.org on 2017/07/10 14:01:50 UTC

[1/2] ranger git commit: Revert "Switch to use 127.0.0.1 explicitly to see if it works"

Repository: ranger
Updated Branches:
  refs/heads/master e7c13864f -> d592d3fa2


Revert "Switch to use 127.0.0.1 explicitly to see if it works"

This reverts commit e7c13864fc37f98b0e1fc4f9add2a681c43197f0.


Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/d2e1d099
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/d2e1d099
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/d2e1d099

Branch: refs/heads/master
Commit: d2e1d099ebbd16af54144391fbe3803f1f204b79
Parents: e7c1386
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Mon Jul 10 15:01:32 2017 +0100
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Mon Jul 10 15:01:32 2017 +0100

----------------------------------------------------------------------
 .../kafka/authorizer/KafkaRangerAuthorizerGSSTest.java          | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ranger/blob/d2e1d099/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
----------------------------------------------------------------------
diff --git a/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java b/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
index eedc697..096c000 100644
--- a/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
+++ b/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
@@ -90,7 +90,7 @@ public class KafkaRangerAuthorizerGSSTest {
 
         configureKerby(basedir);
 
-        String address = "127.0.0.1"; //InetAddress.getLocalHost().getHostAddress();
+        String address = InetAddress.getLocalHost().getHostAddress();
 
         // JAAS Config file - We need to point to the correct keytab files
         Path path = FileSystems.getDefault().getPath(basedir, "/src/test/resources/kafka_kerberos.jaas");
@@ -168,7 +168,8 @@ public class KafkaRangerAuthorizerGSSTest {
         kerbyServer.init();
 
         // Create principals
-        String zookeeper = "zookeeper/127.0.0.1@kafka.apache.org";
+        String address = InetAddress.getLocalHost().getHostAddress();
+        String zookeeper = "zookeeper/" + address + "@kafka.apache.org";
         String kafka = "kafka/localhost@kafka.apache.org";
         String client = "client@kafka.apache.org";
 


[2/2] ranger git commit: Revert "Temporarily adding some logs to try to debug the jenkins failures"

Posted by co...@apache.org.
Revert "Temporarily adding some logs to try to debug the jenkins failures"

This reverts commit eb77374d14120f20c50b8e862fd0e7ad2dd53c01.


Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/d592d3fa
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/d592d3fa
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/d592d3fa

Branch: refs/heads/master
Commit: d592d3fa2c2ea9f48f87b94f9ea5287f928df30d
Parents: d2e1d09
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Mon Jul 10 15:01:44 2017 +0100
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Mon Jul 10 15:01:44 2017 +0100

----------------------------------------------------------------------
 .../kafka/authorizer/KafkaRangerAuthorizerGSSTest.java   | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ranger/blob/d592d3fa/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
----------------------------------------------------------------------
diff --git a/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java b/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
index 096c000..d072a6f 100644
--- a/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
+++ b/plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
@@ -54,9 +54,6 @@ import kafka.server.KafkaServerStartable;
 import kafka.utils.ZKStringSerializer$;
 import kafka.utils.ZkUtils;
 
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
-
 /**
  * A simple test that starts a Kafka broker, creates "test" and "dev" topics,
  * sends a message to them and consumes it.
@@ -73,8 +70,6 @@ import org.apache.commons.logging.LogFactory;
  */
 public class KafkaRangerAuthorizerGSSTest {
 
-    private static final Log LOG = LogFactory.getLog(KafkaRangerAuthorizerGSSTest.class);
-
     private static KafkaServerStartable kafkaServer;
     private static TestingServer zkServer;
     private static int port;
@@ -156,7 +151,7 @@ public class KafkaRangerAuthorizerGSSTest {
 
     private static void configureKerby(String baseDir) throws Exception {
 
-        System.setProperty("sun.security.krb5.debug", "true");
+        //System.setProperty("sun.security.krb5.debug", "true");
         System.setProperty("java.security.krb5.conf", baseDir + "/target/krb5.conf");
 
         kerbyServer = new SimpleKdcServer();
@@ -190,19 +185,15 @@ public class KafkaRangerAuthorizerGSSTest {
 
     @org.junit.AfterClass
     public static void cleanup() throws Exception {
-        LOG.info("Shutting down Kafka");
         if (kafkaServer != null) {
             kafkaServer.shutdown();
         }
-        LOG.info("Shutting down ZK");
         if (zkServer != null) {
             zkServer.stop();
         }
-        LOG.info("Shutting down Kerby");
         if (kerbyServer != null) {
             kerbyServer.stop();
         }
-        LOG.info("Cleanup finished");
     }
 
     // The "public" group can write to and read from "test"