Skip to content

Commit

Permalink
don't return an error when execute faied in loop
Browse files Browse the repository at this point in the history
  • Loading branch information
kurochan committed Nov 3, 2021
1 parent 8b83a7e commit 3121014
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions pkg/app/launcher/cmd/launcher/launcher.go
Expand Up @@ -276,11 +276,8 @@ func (l *launcher) run(ctx context.Context, input cli.Input) error {
for {
select {
case <-ticker.C:
if err := execute(); err != nil {
input.Logger.Error("LAUNCHER: failed while checking whether relaunch required or relaunching. Skip the loop this time", zap.Error(err))
// Don't return an error to continue piped execution.
return nil
}
// Don't return an error to continue piped execution.
_ = execute()

case <-ctx.Done():
// Stop old piped process and clean its data.
Expand Down

0 comments on commit 3121014

Please sign in to comment.