Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-25 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- (Updated June 26, 2016, 12:11 a.m.) Review request for Ambari, Oliver Szabo,

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- (Updated June 24, 2016, 8:39 p.m.) Review request for Ambari, Oliver Szabo,

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- (Updated June 24, 2016, 5:13 p.m.) Review request for Ambari, Oliver Szabo,

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-23 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- (Updated June 23, 2016, 2:37 p.m.) Review request for Ambari, Oliver Szabo,

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-23 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/#review139258 --- Ship it! Ship It! - Oliver Szabo On June 23, 2016, 2:37

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-23 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- (Updated June 23, 2016, 9:21 a.m.) Review request for Ambari, Oliver Szabo,

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-21 Thread Robert Nettleton
> On June 21, 2016, 7:28 p.m., Robert Nettleton wrote: > > This looks fine to me. > > > > One question: Have you tested out the Ambari UI integration once HTTPS is > > enabled? I would expect this to fail, since this configuration option did > > not exist when the integration code was

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-21 Thread Oliver Szabo
> On June 21, 2016, 7:28 p.m., Robert Nettleton wrote: > > This looks fine to me. > > > > One question: Have you tested out the Ambari UI integration once HTTPS is > > enabled? I would expect this to fail, since this configuration option did > > not exist when the integration code was

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-21 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/#review138916 --- Ship it! This looks fine to me. One question: Have you tested

Review Request 49036: Allow https prtotocol for Log Search

2016-06-21 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty.