Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11898 )
Change subject: De-flake sentry_client-test and sentry_authz_provider-test ...................................................................... De-flake sentry_client-test and sentry_authz_provider-test Currently, sentry_client-test and sentry_authz_provider-test often fail with 'BindException: Address already in use'. Because Sentry uses a default port number '8038' if 'sentry.service.server.rpc-port' is not set. This commit sets this configuration with port 0 to ensure the mini Sentry service can always find a free port to bound to. When running sentry_client-test and sentry_authz_provider-test in parallel on the same machine: before the change, one of the tests failed with BindException; after the change, both tests passed without any errors. Change-Id: I0e30ce88b7c2ab4afe451c50e13ef739237709e0 Reviewed-on: http://gerrit.cloudera.org:8080/11898 Tested-by: Hao Hao <[email protected]> Reviewed-by: Grant Henke <[email protected]> Reviewed-by: Andrew Wong <[email protected]> Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/sentry/mini_sentry.cc M src/kudu/sentry/mini_sentry.h 2 files changed, 13 insertions(+), 1 deletion(-) Approvals: Hao Hao: Verified Grant Henke: Looks good to me, approved Andrew Wong: Looks good to me, approved Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11898 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0e30ce88b7c2ab4afe451c50e13ef739237709e0 Gerrit-Change-Number: 11898 Gerrit-PatchSet: 2 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
