Merge branch '15954-boot-test-cluster'

[arvados.git] / lib / boot / cmd.go

diff --git a/lib/boot/cmd.go b/lib/boot/cmd.go
index 60955dfa0faccad76b47b513678ff171cde6050b..1abc93722d8b872cce0d524bf55f5277e6487a5c 100644 (file)
--- a/lib/boot/cmd.go
+++ b/lib/boot/cmd.go
@@ -22,7 +22,7 @@ type supervisedTask interface {
        // done enough to satisfy a dependency relationship (e.g., the
        // service is running and ready). If the task starts a
        // goroutine that fails after Run returns (e.g., the service
-       // shuts down), it should call cancel.
+       // shuts down), it should call fail().
        Run(ctx context.Context, fail func(error), super *Supervisor) error
        String() string
 }
@@ -77,12 +77,15 @@ func (bootCommand) RunCommand(prog string, args []string, stdin io.Reader, stdou
 
        super.Start(ctx, cfg)
        defer super.Stop()
-       if url, ok := super.WaitReady(); ok {
-               fmt.Fprintln(stdout, url)
-               // Wait for signal/crash + orderly shutdown
-               <-super.done
-               return 0
-       } else {
+       url, ok := super.WaitReady()
+       if !ok {
                return 1
        }
+       // Write controller URL to stdout. Nothing else goes to
+       // stdout, so this provides an easy way for a calling script
+       // to discover the controller URL when everything is ready.
+       fmt.Fprintln(stdout, url)
+       // Wait for signal/crash + orderly shutdown
+       <-super.done
+       return 0
 }