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

(Updated March 2, 2015, 11:48 p.m.)


Review request for Ambari, Andrew Onischuk, Emil Anca, John Speidel, and Robert 
Nettleton.


Bugs: AMBARI-9852
    https://issues.apache.org/jira/browse/AMBARI-9852


Repository: ambari


Description
-------

The Kerberos _service check_ needs to generate it's own unique identity to use 
for testing and then destroy it when complete.  This will ensure that any 
_known_ identities (such as the smokeuser, usually ambari-qa) does not 
accidentally get removed if shared between clusters or if the service check is 
run after Kerberos is enabled. 

The service check must perform the following steps:

1. Create a unique principal in the relevant KDC (server)
2. Test that the principal can be used to authenticate via kinit (agent)
3. Destroy the principal (server)


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 2bf0cbf 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java
 8dd6c4d 
  
ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/package/scripts/params.py
 3705cfe 
  
ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/package/scripts/service_check.py
 ee4a4c3 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java
 e16f22f 

Diff: https://reviews.apache.org/r/31656/diff/


Testing (updated)
-------

Manual testing in several scenarios

#Jenkins test results:

Running org.apache.ambari.server.controller.KerberosHelperTest
Tests run: 24, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.797 sec

Running tests for stack:2.2 service:KERBEROS
test_configure_cross_realm_trust (test_kerberos_server.TestKerberosServer) ... 
ok
test_configure_managed_kdc (test_kerberos_server.TestKerberosServer) ... ok
test_configure_unmanaged_ad (test_kerberos_server.TestKerberosServer) ... ok
test_configure_unmanaged_kdc (test_kerberos_server.TestKerberosServer) ... ok
test_configure_cross_realm_trust (test_kerberos_client.TestKerberosClient) ... 
ok
test_configure_managed_kdc (test_kerberos_client.TestKerberosClient) ... ok
test_configure_unmanaged_ad (test_kerberos_client.TestKerberosClient) ... ok
test_configure_unmanaged_kdc (test_kerberos_client.TestKerberosClient) ... ok
test_configure_unmanaged_kdc_and_krb5conf 
(test_kerberos_client.TestKerberosClient) ... ok
test_delete_keytab (test_kerberos_client.TestKerberosClient) ... ok
test_get_property (test_kerberos_client.TestKerberosClient) ... ok
test_set_keytab (test_kerberos_client.TestKerberosClient) ... ok

----------------------------------------------------------------------
Ran 12 tests in 0.177s

**Note: Overall Jenkins test run failed due to errors unrelated to this patch.**


Thanks,

Robert Levas

Reply via email to