> On May 24, 2017, 8:24 p.m., Di Li wrote:
> > ambari-web/app/messages.js
> > Lines 974 (patched)
> > <https://reviews.apache.org/r/59532/diff/1/?file=1731664#file1731664line974>
> >
> >     Can we update the msg to the following (I removed details on how bp 
> > register rest call), where XXXX is dynamically replaced by the cluster name 
> > the user typed in on the UI ?
> >     
> >     ... ...
> >     You will be downloading blueprint.json and clustertemplate.json to your 
> > local machine.
> >     
> >     You can use them to deploy a cluster via AMBARI REST API. 
> >     Update clustertemplate.json with a custom cluster name if you do not 
> > want to use xxxx.
> >     
> >     If you want to use custom repo URLs, make sure to update them via 
> > Ambari REST API before deploying the cluster.

Hello Di, 
Please review the updated patch.


- Amruta


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


On May 24, 2017, 10:17 p.m., Amruta Borkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59532/
> -----------------------------------------------------------
> 
> (Updated May 24, 2017, 10:17 p.m.)
> 
> 
> Review request for Ambari, Alexandr Antonenko, Di Li, and Sangeeta Ravindran.
> 
> 
> Bugs: AMBARI-21104
>     https://issues.apache.org/jira/browse/AMBARI-21104
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Add Confirmation pop up for 'Generate Blueprint' button.
> Notify user that two files will be downloaded and provide information on how 
> to use those files.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/wizard/step8_controller.js f2b4ca77c9 
>   ambari-web/app/messages.js 0c15a1935e 
>   ambari-web/app/templates/wizard/step8.hbs 907b3d4bbf 
> 
> 
> Diff: https://reviews.apache.org/r/59532/diff/2/
> 
> 
> Testing
> -------
> 
> Tested manually. Attached image of the Confirmation box.
> 
> 
> File Attachments
> ----------------
> 
> Generate_blueprint_confirmation
>   
> https://reviews.apache.org/media/uploaded/files/2017/05/24/0c9086f1-b97b-4c2e-ac6b-d0666df7dfa0__Generate_blueprint_confirmation.png
> 
> 
> Thanks,
> 
> Amruta Borkar
> 
>

Reply via email to