[kudu-CR](branch-1.7.x) KUDU-2364 Add extra check in ksck for tserver ID
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9855 ) Change subject: KUDU-2364 Add extra check in ksck for tserver ID .. KUDU-2364 Add extra check in ksck for tserver ID ksck did not validate tablet server ID when checking connectivity. Whenever the TabletServer was nuked and readded, ksck would report the connection was successful to the old tablet servers. Now it will report an error like below: ./bin/kudu cluster ksck localhost Connected to the Master WARNING: Unable to connect to Tablet Server d92197fa2f034b33aa0bf998c41f637b (va1022.halxg.cloudera.com:7073): Remote error: ID reported by tablet server (50effcf1fe284ab693e7d1d43c5f18ad) doesn't match the expected ID: d92197fa2f034b33aa0bf998c41f637b Tablet Server Summary UUID | RPC Address | Status --++--- 22ec2c07d8aa4e1ba33a4eb42d4c3a21 | va1022.halxg.cloudera.com:7072 | HEALTHY 50effcf1fe284ab693e7d1d43c5f18ad | va1022.halxg.cloudera.com:7073 | HEALTHY a05c93549cca4ceebc275651d8117065 | va1022.halxg.cloudera.com:7074 | HEALTHY d92197fa2f034b33aa0bf998c41f637b | va1022.halxg.cloudera.com:7073 | WRONG_SERVER_UUID WARNING: Fetched info from 3 Tablet Servers, 1 weren't reachable The cluster doesn't have any matching tables == Errors: == error fetching info from tablet servers: Could not gather complete information from all tablet servers FAILED Runtime error: ksck discovered errors Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 Reviewed-on: http://gerrit.cloudera.org:8080/9855 Reviewed-by: Grant HenkeTested-by: Grant Henke Reviewed-by: Will Berkeley --- M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck.cc M src/kudu/tools/ksck.h M src/kudu/tools/ksck_remote-test.cc M src/kudu/tools/ksck_remote.cc 5 files changed, 138 insertions(+), 4 deletions(-) Approvals: Grant Henke: Looks good to me, but someone else must approve; Verified Will Berkeley: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9855 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: merged Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 Gerrit-Change-Number: 9855 Gerrit-PatchSet: 3 Gerrit-Owner: Attila Bukor Gerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley
[kudu-CR](branch-1.7.x) KUDU-2364 Add extra check in ksck for tserver ID
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9855 ) Change subject: KUDU-2364 Add extra check in ksck for tserver ID .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9855 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: comment Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 Gerrit-Change-Number: 9855 Gerrit-PatchSet: 2 Gerrit-Owner: Attila BukorGerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley Gerrit-Comment-Date: Thu, 05 Apr 2018 15:31:04 + Gerrit-HasComments: No
[kudu-CR](branch-1.7.x) KUDU-2364 Add extra check in ksck for tserver ID
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9855 ) Change subject: KUDU-2364 Add extra check in ksck for tserver ID .. Patch Set 2: Verified+1 Code-Review+1 The failures are unrelated Jenkins state issues. -- To view, visit http://gerrit.cloudera.org:8080/9855 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: comment Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 Gerrit-Change-Number: 9855 Gerrit-PatchSet: 2 Gerrit-Owner: Attila BukorGerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley Gerrit-Comment-Date: Thu, 05 Apr 2018 14:12:27 + Gerrit-HasComments: No
[kudu-CR](branch-1.7.x) KUDU-2364 Add extra check in ksck for tserver ID
Grant Henke has removed a vote on this change. Change subject: KUDU-2364 Add extra check in ksck for tserver ID .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/9855 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: deleteVote Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 Gerrit-Change-Number: 9855 Gerrit-PatchSet: 2 Gerrit-Owner: Attila BukorGerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley
[kudu-CR](branch-1.7.x) KUDU-2364 Add extra check in ksck for tserver ID
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9855 to look at the new patch set (#2). Change subject: KUDU-2364 Add extra check in ksck for tserver ID .. KUDU-2364 Add extra check in ksck for tserver ID ksck did not validate tablet server ID when checking connectivity. Whenever the TabletServer was nuked and readded, ksck would report the connection was successful to the old tablet servers. Now it will report an error like below: ./bin/kudu cluster ksck localhost Connected to the Master WARNING: Unable to connect to Tablet Server d92197fa2f034b33aa0bf998c41f637b (va1022.halxg.cloudera.com:7073): Remote error: ID reported by tablet server (50effcf1fe284ab693e7d1d43c5f18ad) doesn't match the expected ID: d92197fa2f034b33aa0bf998c41f637b Tablet Server Summary UUID | RPC Address | Status --++--- 22ec2c07d8aa4e1ba33a4eb42d4c3a21 | va1022.halxg.cloudera.com:7072 | HEALTHY 50effcf1fe284ab693e7d1d43c5f18ad | va1022.halxg.cloudera.com:7073 | HEALTHY a05c93549cca4ceebc275651d8117065 | va1022.halxg.cloudera.com:7074 | HEALTHY d92197fa2f034b33aa0bf998c41f637b | va1022.halxg.cloudera.com:7073 | WRONG_SERVER_UUID WARNING: Fetched info from 3 Tablet Servers, 1 weren't reachable The cluster doesn't have any matching tables == Errors: == error fetching info from tablet servers: Could not gather complete information from all tablet servers FAILED Runtime error: ksck discovered errors Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 --- M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck.cc M src/kudu/tools/ksck.h M src/kudu/tools/ksck_remote-test.cc M src/kudu/tools/ksck_remote.cc 5 files changed, 138 insertions(+), 4 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/55/9855/2 -- To view, visit http://gerrit.cloudera.org:8080/9855 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 Gerrit-Change-Number: 9855 Gerrit-PatchSet: 2 Gerrit-Owner: Attila BukorGerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley
[kudu-CR](branch-1.7.x) KUDU-2364 Add extra check in ksck for tserver ID
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/9855 ) Change subject: KUDU-2364 Add extra check in ksck for tserver ID .. Patch Set 1: > Patch Set 1: Verified-1 > > Build Failed > > http://jenkins.kudu.apache.org/job/kudu-gerrit/12754/ : FAILURE LINT failed for some reason but I don't see why. -- To view, visit http://gerrit.cloudera.org:8080/9855 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.7.x Gerrit-MessageType: comment Gerrit-Change-Id: Ia2c18ba7af8eaa6f5e4d7842f18754d2c1e32526 Gerrit-Change-Number: 9855 Gerrit-PatchSet: 1 Gerrit-Owner: Attila BukorGerrit-Reviewer: Attila Bukor Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley Gerrit-Comment-Date: Thu, 29 Mar 2018 17:21:53 + Gerrit-HasComments: No