Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11422 )

Change subject: [location_awareness] Add location info in ksck report
......................................................................


Patch Set 6:

The master can't register the new tablet server because it can't assign a 
location to it:

    E0929 00:13:42.477679  5367 ts_descriptor.cc:241] Unable to assign location 
to tablet server e646a933f9ae402a8af4b57b51130a56 (127.0.111.4): Runtime error: 
failed to run location mapping command: Runtime error: 
/tmp/dist-test-taskJPSQi9/build/debug/bin/testdata/first_argument.sh: process 
exited with non-zero status 2: E0929 00:13:42.458889  5896 subprocess.cc:455] 
Couldn't exec 
/tmp/dist-test-taskJPSQi9/build/debug/bin/testdata/first_argument.sh: No such 
file or directory [2]

When you (or I) run the test locally, that script can be found, but it's not 
packaged and sent to Jenkins as part of the tests.

To get that to happen, take a look at 
https://github.com/apache/kudu/blob/master/src/kudu/tools/CMakeLists.txt#L175. 
I believe you'll need to do something similar for ksck_remote-test.


--
To view, visit http://gerrit.cloudera.org:8080/11422
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ideff2dd4975c99a1135002624debbbb2620fb95c
Gerrit-Change-Number: 11422
Gerrit-PatchSet: 6
Gerrit-Owner: Fengling Wang <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Fengling Wang <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Mon, 01 Oct 2018 19:08:54 +0000
Gerrit-HasComments: No

Reply via email to