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

(Updated July 25, 2017, 12:53 a.m.)


Review request for Ambari, Balázs Bence Sári, Juanjo  Marron, and Sid Wagle.


Changes
-------

Updated Patch with requested changes.


Bugs: AMBARI-21325
    https://issues.apache.org/jira/browse/AMBARI-21325


Repository: ambari


Description
-------

Add quicklink support through Knox.
Changes:
1. Add new json properties knox_url, knox_path, supports_knox
   a. knox_url: template to be used for urls that are proxied through Knox 
   b. knox_path: Knox gateway path that will be added to the proxy url.
   c. supports_knox: whether link will be redirected through Knox
2. Add above json properties to quicklinks.json
3. Add HDFSUI & DATANODE,YARNUI & NODEUI, JOBHISTORYUI, HBASEUI, OOZIEUI, 
SPARKUI services to Knox topology template.
4. Automate protocol and port added to Knox topology file. Based on whether SSL 
is enabled for the services listed above, the port and protocol in 
params_linux.py will be updated.
5. Update quick_view_link_view.js so that when Knox is installed and 
support_knox is true, quicklink url follows knox url template specified in the 
quicklinks.json for the service/component.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/Link.java 
1d2e712 
  
ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params_linux.py
 4558069 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/quicklinks/quicklinks.json
 5568122 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/HDFS/quicklinks/quicklinks.json
 a4216e3 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/OOZIE/quicklinks/quicklinks.json
 81e7cbe 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/quicklinks-mapred/quicklinks.json
 36f71b5 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/quicklinks/quicklinks.json
 0aca8e3 
  
ambari-server/src/main/resources/stacks/HDP/2.6/services/KNOX/configuration/topology.xml
 df4c1b4 
  ambari-server/src/test/python/stacks/2.2/configs/knox_upgrade.json 1805c3b 
  ambari-web/app/views/common/quick_view_link_view.js e39f230 


Diff: https://reviews.apache.org/r/61043/diff/2/

Changes: https://reviews.apache.org/r/61043/diff/1-2/


Testing
-------

1. ambari-server: mvn test
2. mvn -DskipSurefireTests -Dpython.test.mask=test_knox_gateway.py test
3. ambari-web: mvn test
     21213 passing (54s)
     128 pending


Thanks,

Chandana Mirashi

Reply via email to