Hello Zoltan Martonka, Yingchun Lai, Kudu Jenkins,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/20667
to review the following change.
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/20667/1
--
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: newchange
Gerrit-Change-Id: I8b8d330ecd747ac8535d4138d7d49902991e31b0
Gerrit-Change-Number: 20667
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
