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

(Updated June 3, 2016, 9:56 a.m.)


Review request for Ambari, Eugene Chekanskiy, Jonathan Hurley, Myroslav 
Papirkovskyy, and Nate Cole.


Changes
-------

Fixed merge issues.


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


Repository: ambari


Description
-------

Cluster operator and ServiceAdministrator are not allowed to create a config 
group even though it should be.
It is visible in the UI but creating it gives a 403 error in the backend and 
the UI goes for a toss.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterKerberosDescriptorResourceProvider.java
 6fe29db 
  
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationFilter.java
 2b9f304 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterKerberosDescriptorResourceProviderTest.java
 d56ed44 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ConfigGroupResourceProviderTest.java
 2913cf5 
  
ambari-server/src/test/java/org/apache/ambari/server/security/TestAuthenticationFactory.java
 4301bf8 
  
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationFilterTest.java
 96b2cfb 

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


Testing
-------

Manually tested....

#Local test results: 
Unrelated unit test failures:
```
Results :

Tests in error:
  StackManagerTest.testMetricsLoaded:669 » Ambari File 
/Users/rlevas/git/ambari/...
  StackManagerTest.testServicesWithLogsearchRoleCommandOrder:820 » Ambari File 
/...
  StackManagerTest.testServicesWithRangerPluginRoleCommandOrder:713 » Ambari 
Fil...
  ServicePropertiesTest.validatePropertySchemaOfServiceXMLs:50 » Ambari File 
/Us...

Tests run: 4458, Failures: 0, Errors: 4, Skipped: 34
```

#Jenkins test results: PENDING


Thanks,

Robert Levas

Reply via email to