Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9378
to look at the new patch set (#2).
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, 46 insertions(+), 25 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/78/9378/2
--
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: newpatchset
Gerrit-Change-Id: I603ebc22e998bac9bd00edc939577ae339587f26
Gerrit-Change-Number: 9378
Gerrit-PatchSet: 2
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
