Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10300
to look at the new patch set (#4).
Change subject: [tools] ksck imp. [8/n]: Easy-to-see errors when running w/o
admin
......................................................................
[tools] ksck imp. [8/n]: Easy-to-see errors when running w/o admin
If ksck is not run as an admin user, then it cannot gather all of the
information it uses to do cluster checks. "Not authorized" errors are
easy to mistake for other errors. This patch handles these errors
specially to make it obvious to administrators when they have forgotten
to run as an administrative user.
The new errors at the end of the ksck output look like this:
==================
Errors:
==================
Not authorized: error fetching info from masters: failed to gather info from 1
of 1 masters due to lack of admin privileges
Not authorized: error fetching info from tablet servers: failed to gather info
from 11 of 12 tablet servers due to lack of admin privileges
FAILED
Not authorized: please run ksck with administrator privileges
Change-Id: I644957103efefceeba4b82f4557376a603507423
---
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck.cc
M src/kudu/tools/ksck_results.cc
M src/kudu/tools/ksck_results.h
M src/kudu/tools/tool.proto
5 files changed, 77 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/10300/4
--
To view, visit http://gerrit.cloudera.org:8080/10300
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I644957103efefceeba4b82f4557376a603507423
Gerrit-Change-Number: 10300
Gerrit-PatchSet: 4
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley <[email protected]>
