Skip to content

Commit

Permalink
fix two %w not associated to the PR
Browse files Browse the repository at this point in the history
  • Loading branch information
felipemadero committed Aug 11, 2022
1 parent fecd0ca commit bf50ccc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions local/node_process.go
Original file line number Diff line number Diff line change
Expand Up @@ -156,15 +156,15 @@ func (p *nodeProcess) Stop(ctx context.Context) int {
p.lock.Unlock()

if err := proc.Signal(os.Interrupt); err != nil {
p.log.Warn("sending SIGINT errored: %w", err)
p.log.Warn("sending SIGINT errored: %s", err)
}

select {
case <-ctx.Done():
p.log.Warn("context cancelled while waiting for node %q to stop", p.name)
killDescendants(int32(proc.Pid), p.log)
if err := proc.Signal(os.Kill); err != nil {
p.log.Warn("sending SIGKILL errored: %w", err)
p.log.Warn("sending SIGKILL errored: %s", err)
}
case <-p.closedOnStop:
}
Expand Down

0 comments on commit bf50ccc

Please sign in to comment.