Ádám Bakai has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20708
Change subject: [messenger] Initialize SASL proto name to principal ...................................................................... [messenger] Initialize SASL proto name to principal Adding master in case of custom kerberos principal name didn't work because the non-default principal name was not propagated to the MessengerBuilder every time and it used "kudu" instead of the one provided by the kerberos service. This patch makes sure that if there is a custom principal name, it is set up every time in the MessengerBuilder constructor. If it is not set, it still uses the default "kudu" value. It can be changed with the set_sasl_proto_name command after intialization. However this change is not enough because ServerBase::ShutdownImpl called DestroyKerberosForServer which destroyed the global state which MessengerBuilder uses. The function call was removed, and since the function is not used anywhere else, the function was removed as well. The new test is using non default principals, adds two new masters, runs a smoke test. After that one master is shut down to simulate recoverable master error, and the smoke test is executed again. Change-Id: Ie8decbd0b3e54df42bb0b9b14fc5ec291cd70b8b --- M src/kudu/integration-tests/security-itest.cc M src/kudu/rpc/messenger.cc M src/kudu/security/init.cc M src/kudu/security/init.h M src/kudu/server/server_base.cc 5 files changed, 32 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/08/20708/1 -- To view, visit http://gerrit.cloudera.org:8080/20708 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie8decbd0b3e54df42bb0b9b14fc5ec291cd70b8b Gerrit-Change-Number: 20708 Gerrit-PatchSet: 1 Gerrit-Owner: Ádám Bakai <[email protected]>
