fix(run): do not show aggregator status failure too early. SPLAT-624

redhat-openshift-ecosystem · Jun 30, 2022 · b069750 · b069750
1 parent 41636de
commit b069750
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/pkg/run/run.go b/pkg/run/run.go
@@ -3,6 +3,7 @@ package run
 import (
 	"context"
 	"fmt"
+	"time"
 
 	configv1 "github.com/openshift/api/config/v1"
 	operatorv1 "github.com/openshift/api/operator/v1"
@@ -88,6 +89,9 @@ func NewCmdRun() *cobra.Command {
 				return
 			}
 
+			// Sleep to give status time to appear
+			time.Sleep(status.StatusInterval)
+
 			s := status.NewStatusOptions(o.watch)
 			err = s.WaitForStatusReport(cmd.Context(), sclient)
 			if err != nil {

diff --git a/pkg/status/status.go b/pkg/status/status.go
@@ -146,7 +146,6 @@ func (s *StatusOptions) WaitForStatusReport(ctx context.Context, sclient sonobuo
 
 		err = s.Update(sclient)
 		if err != nil {
-			// TODO Should the warning be shown to user by default? It can be misleading during startup
 			log.WithError(err).Warn("error retrieving current aggregator status")
 		} else if s.Latest.Status != "" {
 			return true, nil