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

Ship it!


Ship It!

- Srimanth Gunturi


On March 23, 2015, 6:13 p.m., Brandon Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32318/
> -----------------------------------------------------------
> 
> (Updated March 23, 2015, 6:13 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Srimanth Gunturi.
> 
> 
> Bugs: AMBARI-9438
>     https://issues.apache.org/jira/browse/AMBARI-9438
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> AMBARI-9438 Add configuration and monitor section for HDFS-NFS gateway
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/configuration/hadoop-env.xml
>  1d6618d 
>   
> ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/configuration/hdfs-site.xml
>  8cad375 
>   
> ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params.py
>  d20e8e0 
>   ambari-web/app/data/HDP2/site_properties.js b09d0cf 
>   ambari-web/app/messages.js 1730b62 
>   ambari-web/app/models/service/hdfs.js 21678b9 
>   ambari-web/app/models/service_config.js cb8e5a9 
>   ambari-web/app/models/stack_service.js 8df3882 
>   ambari-web/app/templates/main/service/services/hdfs.hbs 57638a3 
>   ambari-web/app/views/main/service/services/hdfs.js 888b0c8 
> 
> Diff: https://reviews.apache.org/r/32318/diff/
> 
> 
> Testing
> -------
> 
> Manually tested the UI on a VM.
> 
> Please hold the commit. 
> The ambari trunk has been updated. I rebased the patch to find 2 test 
> failures now.
> 
> I ran the same test again and it passed this time:
> 
>      [exec] 
>      [exec]   5649 tests complete (8 seconds)
>      [exec]   68 tests pending
>      [exec]
> 
> 
> Thanks,
> 
> Brandon Li
> 
>

Reply via email to