Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20667 )
Change subject: [ranger] disable Tomcat's shutdown port ...................................................................... [ranger] disable Tomcat's shutdown port This patch explicitly disables Tomcat's shutdown port when running MiniRanger and MiniRangerKMS wrappers. The port hasn't been used in Kudu tests anyway, so now it's less hustle about dealing with those. In addition, the HTTPS service port is now explicitly disabled as well to add a bit more clarity for humans who might be looking into the XML configuration files. This patch also adds a sanity check to make sure WaitForTcpBind() run against Ranger and RangerKMS detects the port exactly as configured. Change-Id: I8b8d330ecd747ac8535d4138d7d49902991e31b0 Reviewed-on: http://gerrit.cloudera.org:8080/20583 Tested-by: Kudu Jenkins Reviewed-by: Zoltan Martonka <[email protected]> Reviewed-by: Yingchun Lai <[email protected]> (cherry picked from commit 05d631e4a1035a04fe0c4e11c7c1b1e6b9530d99) Conflicts: src/kudu/ranger-kms/mini_ranger_kms.cc src/kudu/ranger/mini_ranger.cc Reviewed-on: http://gerrit.cloudera.org:8080/20667 --- M src/kudu/ranger-kms/mini_ranger_kms.cc M src/kudu/ranger-kms/mini_ranger_kms_configs.h M src/kudu/ranger/mini_ranger.cc M src/kudu/ranger/mini_ranger_configs.h 4 files changed, 45 insertions(+), 30 deletions(-) Approvals: Kudu Jenkins: Verified Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20667 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I8b8d330ecd747ac8535d4138d7d49902991e31b0 Gerrit-Change-Number: 20667 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Zoltan Martonka <[email protected]>
