From 1e4d5f3e674eeb8b1af235d56045c4dd4ad304e9 Mon Sep 17 00:00:00 2001 From: Kousik Kumar Date: Tue, 26 Jan 2021 14:30:53 -0500 Subject: [PATCH] Add FAILED: prefix to RBE Fatal error message This error message it currently being logged to stdout which goes to build.log file . We want it to be logged to build_error.log file instead so that downstream systems that need to run based on the error message will continue to work. Test: Tested by running `m toybox` and ensuring that FAILED: prefix is at the beginning of the line. Bug: b/177221705 Change-Id: Ie0c9acee2739acf149384e33804b5fca3273c24f --- ui/build/rbe.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ui/build/rbe.go b/ui/build/rbe.go index d9c33f61e..1fabd9262 100644 --- a/ui/build/rbe.go +++ b/ui/build/rbe.go @@ -106,7 +106,7 @@ func startRBE(ctx Context, config Config) { cmd := Command(ctx, config, "startRBE bootstrap", rbeCommand(ctx, config, bootstrapCmd)) if output, err := cmd.CombinedOutput(); err != nil { - ctx.Fatalf("rbe bootstrap failed with: %v\n%s\n", err, output) + ctx.Fatalf("Unable to start RBE reproxy\nFAILED: RBE bootstrap failed with: %v\n%s\n", err, output) } }