This makes it easier, for me, to follow the output. It's more like
what I see in the terminal if I run the commands directly. If this
turns out to be bad for others, I'll revert.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
They're in the build log too, and come from the trigger message, but
it's helpful for debugging to log them clearly.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
The CI broker has dropped the RunResult::Error variant, so drop any
use of that. The Request::version field has been added, so deal with
that.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
Previously, every run log would contain:
Note: switching to 'HEAD^'.
You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.
If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:
git switch -c <new-branch-name>
Or undo this operation with:
git switch -
Turn off this advice by setting config variable advice.detachedHead to false
HEAD is now at 2d0d98e feat: add branch or patch information to run log
This should now be gone.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
If any command run by CI for a repository fails, set the run to failed
in the run info builder.
Also, we were hiding that failure with later errors, if, say, updating
a log file or removing temporary files failed, so don't do that.
In the old engine it's hard to follow how various errors are handled.
This new engine should make it clearer.
This isn't entirely a pure refactoring. It subtly changed how the
external behavior is, to fix problems.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
It turns out that writing the to a file is sometimes a useful thing to
do. Who could've foreseen this. Well, they're using a paper bag on
their head as they're writing this commit message.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
We have a run log, aimed at the developer for whose project CI is run,
and the node admin log, aimed at the node admin. We don't need a third
log aimed at the developer of the native CI engine itself.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
We're about to make HTML logs, to .html is the right suffix, not .txt.
Also drop a commented-out log message.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
Lars Wirzenius