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




ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py (line 
143)
<https://reviews.apache.org/r/46661/#comment197182>

    I dont think you need to do this.  Can you remove this line an retest. See 
that `recommendHIVEConfigurations` does not call `self.putPropery` yet 
`putHiveInteractiveEnvProperty` is available for use.



ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
(lines 145 - 148)
<https://reviews.apache.org/r/46661/#comment197183>

    what needs to happen when security is disbled and 
`hbase.master.ui.readonly` is `true`?  Does this property need to be removed or 
set to `false`?


- Robert Levas


On May 11, 2016, 7:31 p.m., Ajit Kumar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46661/
> -----------------------------------------------------------
> 
> (Updated May 11, 2016, 7:31 p.m.)
> 
> 
> Review request for Ambari, Robert Levas, Sumit Mohanty, and Sid Wagle.
> 
> 
> Bugs: AMBARI-16164
>     https://issues.apache.org/jira/browse/AMBARI-16164
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Currently in secure deployment, user can request compaction / splitting 
> through hbase master UI.
> This potentially exposes vulnerability to various attacks.
> There is config parameter, hbase.master.ui.readonly, with default value of 
> false.
> In secure deployment, Master UI should be put to readonly mode (setting the 
> above parameter to true).
> Admin can always request compaction / splitting through hbase shell.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
> 3d689d7dd4f9c5bd98d5d4e0152775fd7ea012a2 
> 
> Diff: https://reviews.apache.org/r/46661/diff/
> 
> 
> Testing
> -------
> 
> Manual testing, sceenshot attached (after kerberizing cluster).
> 
> 
> File Attachments
> ----------------
> 
> hbase-site.png
>   
> https://reviews.apache.org/media/uploaded/files/2016/05/11/75e8d2ae-99a4-4e8d-8a69-52cd974734fb__hbase-site.png
> 
> 
> Thanks,
> 
> Ajit Kumar
> 
>

Reply via email to