ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexandr Antonenko" <hiv...@gmail.com>
Subject Re: Review Request 42585: Service deploy wizard is not always restored to the correct state If browser session is lost while deploy is running
Date Thu, 21 Jan 2016 17:44:17 GMT

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

Ship it!


Ship It!

- Alexandr Antonenko


On Jan. 21, 2016, 5:30 a.m., Sangeeta Ravindran wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42585/
> -----------------------------------------------------------
> 
> (Updated Jan. 21, 2016, 5:30 a.m.)
> 
> 
> Review request for Ambari, Andriy Babiichuk and Alexandr Antonenko.
> 
> 
> Bugs: Ambari-13514
>     https://issues.apache.org/jira/browse/Ambari-13514
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> While deploying a service, if the browser session is lost, then on restarting the browser
and opening Ambari web console, the deploy wizard does not always return to the correct state.
> 
> Specifically, in the following cases, an inconsistent state is reached.
> 
> 1) When cluster deploy starts and user is on Install, Start and Test page of the wizard
(Step 7) and cluster state is either ADD_SERVICES_INSTALLING_3 or SERVICE_STARTING_3 before
session is lost.
> 
> In this case, services are either installing or being started. Hence, if browser session
is lost, the controller should return to the Step 7 and processing should continue. Currently
however, the state is being set to step 6 (Review). 
> 
> case 'ADD_SERVICES_INSTALLING_3' :
> case 'SERVICE_STARTING_3' :
>     addServiceController.setCurrentStep('6');
>     
>  As a result, user might end up clicking next on the Review page under these conditions
and get an error like org.apache.ambari.server.controller.spi.ResourceAlreadyExistsException:
Attempted to create a host_component which already exists: [clusterName=hdpcluster, hostName=myhost.mydomain.com
componentName=HDFS_CLIENT]
> 
> 2) When cluster has finished installing and user is on Summary page (Step 8) of the wizard
before session is lost. 
> 
> In this case, the cluster status gets reset to CLUSTER_NOT_CREATED_1. In add_service_routes,
currently this case is not considered. As a result, the addServiceController step is not set
correctly and defaults to step 1 (Choose Services). Since this state can happen either when
you first open the Add Service Wizard or at the end of service install and before user clicks
on Complete in the wizard, to fix this, the addServiceController's current step should be
set to the currentStep persisted in the App.db.data.AddService.currentStep variable which
contains the step that the add service process is currently in.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/routes/add_service_routes.js 30bdcbc 
> 
> Diff: https://reviews.apache.org/r/42585/diff/
> 
> 
> Testing
> -------
> 
> Did manual testing.
> 
> 23108 tests complete (24 seconds)
> 165 tests pending
> 
> 
> Thanks,
> 
> Sangeeta Ravindran
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message