You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sentry.apache.org by Colm O hEigeartaigh <co...@apache.org> on 2017/07/10 15:00:46 UTC

Re: Review Request 60275: SENTRY-1810 - Add kerberos tests for Kafka

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60275/
-----------------------------------------------------------

(Updated July 10, 2017, 3 p.m.)


Review request for sentry.


Bugs: SENTRY-1810
    https://issues.apache.org/jira/browse/SENTRY-1810


Repository: sentry


Description
-------

Currently, the Kafka tests rely on SSL client authentication. We should add tests as well for Kerberos.


Diffs (updated)
-----

  pom.xml 578e60f5 
  sentry-tests/sentry-tests-kafka/pom.xml 48dd1454 
  sentry-tests/sentry-tests-kafka/src/main/java/org/apache/sentry/tests/e2e/kafka/KafkaSSLTestServer.java PRE-CREATION 
  sentry-tests/sentry-tests-kafka/src/main/java/org/apache/sentry/tests/e2e/kafka/KafkaTestServer.java e7273ee2 
  sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/AbstractAuthorizationTestBase.java PRE-CREATION 
  sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/AbstractKafkaSentryTestBase.java 5807f681 
  sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/KafkaGSSTestServer.java PRE-CREATION 
  sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/StaticUserGroupRole.java 96b7cf43 
  sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAclsCrud.java a52c8d67 
  sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAuthorize.java 0b1ef68a 
  sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAuthorizeGSS.java PRE-CREATION 
  sentry-tests/sentry-tests-kafka/src/test/resources/kafka_kerberos.jaas PRE-CREATION 


Diff: https://reviews.apache.org/r/60275/diff/3/

Changes: https://reviews.apache.org/r/60275/diff/2-3/


Testing
-------


Thanks,

Colm O hEigeartaigh