> On May 16, 2017, 3:01 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py > > Line 258 (original), 259 (patched) > > <https://reviews.apache.org/r/59305/diff/1/?file=1720721#file1720721line259> > > > > Does livy have an HTTPS port?
Currently Livy https use the same port as http, and Livy cannot start both https and http, user could only configure one. - Saisai ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59305/#review175106 ----------------------------------------------------------- On May 16, 2017, 5:58 a.m., Saisai Shao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59305/ > ----------------------------------------------------------- > > (Updated May 16, 2017, 5:58 a.m.) > > > Review request for Ambari and Sumit Mohanty. > > > Bugs: AMBARI-21012 > https://issues.apache.org/jira/browse/AMBARI-21012 > > > Repository: ambari > > > Description > ------- > > Current Ambari Livy script doesn't handle HTTPS/SSL enabled scenario, it will > always use "http" scheme to check Livy. We should change the scheme to > support HTTPS if SSL is enabled. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/alerts/alert_spark_livy_port.py > 746a98e > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/params.py > 42396bd > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/service_check.py > 9d74779 > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/alerts/alert_spark2_livy_port.py > 44c284f > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/params.py > 1df3f2f > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/service_check.py > 8e7a766 > > > Diff: https://reviews.apache.org/r/59305/diff/1/ > > > Testing > ------- > > Local verification is done. > > > Thanks, > > Saisai Shao > >