Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5775: (Addendum) Make SSL cluster actually come up in test_client_ssl.py ......................................................................
IMPALA-5775: (Addendum) Make SSL cluster actually come up in test_client_ssl.py The non-wildcard certs in test_client_ssl.py require that the hostname of the process is 'localhost' for clients to validate them. This wasn't the case for one test, and so the cluster wouldn't actually start. Although the test would still pass (because the shell wasn't actually checking the certificate), it's better hygiene to have the cluster correctly configured to make sure we're testing what we think we are. Testing: test continues to pass Change-Id: Idad8bbf3b8be853d3406bcbaed24909501500ea9 Reviewed-on: http://gerrit.cloudera.org:8080/7732 Reviewed-by: Henry Robinson <[email protected]> Tested-by: Impala Public Jenkins --- M tests/custom_cluster/test_client_ssl.py 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Impala Public Jenkins: Verified Henry Robinson: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/7732 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idad8bbf3b8be853d3406bcbaed24909501500ea9 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Henry Robinson <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Sailesh Mukil <[email protected]>
