David Ribeiro Alves has submitted this change and it was merged. Change subject: Make ExternalDaemon::StartProcess() handle fault injection ......................................................................
Make ExternalDaemon::StartProcess() handle fault injection Occasionally starting a kudu daemon fails becuase of fault injection. We recently fixed this error manifesting as SIGPIPE, but we didn't fix the underlying cause: under load, tests doing fault injection might fail before StartProcess() completes making this method return a non-OK() status. This patch makes sure that if the exit status is an expected one (85) we still return as if starting the process had succeeded. Subsequent calls to WaitForInjectedCrash() will return Status::OK(); To make sure this works, this patch includes a test that enables a "fake" fault early in the startup process and makes sure that both WaitForInjectedCrash and StartProcess() return OK() in spite of it. Change-Id: I6046e34a321de3e324e20e3d63249e4073712447 Reviewed-on: http://gerrit.cloudera.org:8080/6211 Reviewed-by: Todd Lipcon <[email protected]> Reviewed-by: Mike Percy <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/external_mini_cluster-test.cc M src/kudu/integration-tests/external_mini_cluster.cc M src/kudu/integration-tests/external_mini_cluster.h M src/kudu/tserver/tablet_server_main.cc 4 files changed, 37 insertions(+), 3 deletions(-) Approvals: Mike Percy: Looks good to me, approved Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6211 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6046e34a321de3e324e20e3d63249e4073712447 Gerrit-PatchSet: 10 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
