Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9378
Change subject: KUDU-2039: /masters can show the wrong list of masters ...................................................................... KUDU-2039: /masters can show the wrong list of masters The ListMasters RPC returned masters based on the -master_addresses flag. It should return masters based on the consensus configuration. Since we verify the two sets of addresses are the same at startup, this usually makes no difference. However, if a quorum master is replaced by a server with a new UUID, previously we would report it in ListMasters and therefore on /masters. This patch adds an additional check that the UUIDs match, and returns an error along with the registration information if there is a mismatch. Screenshot for down master: https://github.com/wdberkeley/kudu/blob/kudu2309screenshots/www/mastersonedown.png Screenshot for UUID mismatch: https://github.com/wdberkeley/kudu/blob/kudu2309screenshots/www/mastersmismatch.png Change-Id: I603ebc22e998bac9bd00edc939577ae339587f26 --- M src/kudu/master/master.cc M src/kudu/master/master_path_handlers.cc M src/kudu/master/sys_catalog.h M www/masters.mustache 4 files changed, 44 insertions(+), 22 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/78/9378/1 -- To view, visit http://gerrit.cloudera.org:8080/9378 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I603ebc22e998bac9bd00edc939577ae339587f26 Gerrit-Change-Number: 9378 Gerrit-PatchSet: 1 Gerrit-Owner: Will Berkeley <[email protected]>
