----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48205/ -----------------------------------------------------------
Review request for Ambari, Eugene Chekanskiy, Jonathan Hurley, Myroslav Papirkovskyy, and Nate Cole. 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 ----- 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 eeb1a8b 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 ff47ac2 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