Skip to content

Commit

Permalink
Remove remaining staticcheck violations
Browse files Browse the repository at this point in the history
This commit removes remaining static check violations in the CMO that
showed up after fixing all the other issues.

Signed-off-by: Daniel Mellado <dmellado@redhat.com>
  • Loading branch information
danielmellado committed Jun 6, 2023
1 parent 5ff7b96 commit 9beb4ce
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion cmd/operator/main.go
Expand Up @@ -207,7 +207,7 @@ func Main() int {
}
wg.Go(func() error { return srv.Run(ctx) })

term := make(chan os.Signal)
term := make(chan os.Signal, 1)
signal.Notify(term, os.Interrupt, syscall.SIGTERM)

select {
Expand Down
3 changes: 1 addition & 2 deletions pkg/tasks/prometheus.go
Expand Up @@ -274,8 +274,7 @@ func (t *PrometheusTask) create(ctx context.Context) error {
return errors.Wrap(err, "initializing Metrics Client Certs secret failed")
}

//nolint:ineffassign
metricsCerts, err = t.client.WaitForSecret(ctx, metricsCerts)
_, err = t.client.WaitForSecret(ctx, metricsCerts)
if err != nil {
return errors.Wrap(err, "waiting for Metrics Client Certs secret failed")
}
Expand Down

0 comments on commit 9beb4ce

Please sign in to comment.