ma27/hydra
0
0
Fork 0
forked from lix-project/hydra
Code Pull requests Activity

hydra-eval-jobs: Mirror eval errors in STDERR

Browse source 
Otherwise, errors will not be shown to end-users, which makes debugging
long evals pretty much impossible.
This commit is contained in:
Samuel Dionne-Riel 2020-03-30 14:36:16 -04:00
parent 12cc46cdb3
commit 76299b9174
1 changed files with 8 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/hydra-eval-jobs/hydra-eval-jobs.cc
View file

@ -200,7 +200,12 @@ static void worker(
else throw TypeError("attribute '%s' is %s, which is not supported", attrPath, showType(*v)); else throw TypeError("attribute '%s' is %s, which is not supported", attrPath, showType(*v));
} catch (EvalError & e) { } catch (EvalError & e) {
// Transmits the error we got from the previous evaluation
// in the JSON output.
reply["error"] = filterANSIEscapes(e.msg(), true); reply["error"] = filterANSIEscapes(e.msg(), true);
// Don't forget to print it into the STDERR log, this is
// what's shown in the Hydra UI.
printError("error: %s", reply["error"]);
} }
writeLine(to.get(), reply.dump()); writeLine(to.get(), reply.dump());
@ -286,6 +291,9 @@ int main(int argc, char * * argv)
nlohmann::json err; nlohmann::json err;
err["error"] = e.what(); err["error"] = e.what();
writeLine(to->get(), err.dump()); writeLine(to->get(), err.dump());
// Don't forget to print it into the STDERR log, this is
// what's shown in the Hydra UI.
printError("error: %s", err["error"]);
} }
}, },
ProcessOptions { .allowVfork = false }); ProcessOptions { .allowVfork = false });