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


Ship it!




Ship It!

- Alexandr Antonenko


On Nov. 18, 2016, 11:04 p.m., Aleksandr Kovalenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53907/
> -----------------------------------------------------------
> 
> (Updated Nov. 18, 2016, 11:04 p.m.)
> 
> 
> Review request for Ambari, Alexandr Antonenko, Jaimin Jetly, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-18939
>     https://issues.apache.org/jira/browse/AMBARI-18939
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> STR:
> Add LIVY_SERVER as serviceComponent dependency to "ZEPPELIN_MASTER" in 
> Zeppelin's service definiion
> Add Spark without Livy (slave component with cardinality of 0+)
> Use "Add Service Wizard" to add Zeppelin service.
> Expected Result with new work to be done: User should be able to edit 
> (add/change) slave dependencies (LIVY_SERVER) as dependency rule is broken 
> while adding Zeppelin service i.e. Zeppelin mater depends on livy server and 
> there is no livy server installed on deployed cluster.
> Actual Result as of now: "Assign Slave and Client component" page is skipped 
> while adding zeppelin
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/main/service/add_controller.js 04427a2 
>   ambari-web/app/controllers/wizard.js 89d439a 
>   ambari-web/app/controllers/wizard/step6_controller.js 87b0ee5 
>   ambari-web/app/mappers/stack_service_mapper.js 888dcdf 
>   ambari-web/app/utils/ajax/ajax.js 6ca014a 
>   ambari-web/test/controllers/main/service/add_controller_test.js 51c8dbf 
>   ambari-web/test/controllers/wizard/step6_test.js 4cf155c 
> 
> Diff: https://reviews.apache.org/r/53907/diff/
> 
> 
> Testing
> -------
> 
> 30624 tests complete (33 seconds)
>   153 tests pending
> 
> 
> Thanks,
> 
> Aleksandr Kovalenko
> 
>

Reply via email to