stoty opened a new pull request #267: KNOX-2238 CM discovery - Add TLS support to Phoenix auto discovery URL: https://github.com/apache/knox/pull/267 ## What changes were proposed in this pull request? Add TLS support for Phoenix Query Server for the CM service discovery module The TLS support was omitted from the original discovery module, as PQS didn't have TLS support yet. Now I have re-added the TLS support. The code is a little different from most similar modules, because the ssl_enabled property is not present in older CM releases, and we have to handle that as well. ## How was this patch tested? * Wrote and run new unit test * Tested against live Cloudera Manager instance which had the the TLS for PQS support * Tested against live older Cloudera Manager instance which did not have the TLS for PQS support In both cases I was able to connect to the auto discovered PQS service.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
