1
82%
master: 82%

Ran 13 May 2026 09:44AM UTC

Files 102

Run time 14s

Badge

Embed ▾

Committed 13 May 2026 09:42AM UTC coverage: 82.295% (-0.02%) from 82.313%

Job # 25791217132.1

Build Type

push

github

Committed by

web-flow

Commit Message

🐛 (grafana/v1alpha): Fix file handle leak on config read error (#5682)

* fix(grafana/v1alpha): close file handle on config read error

Replace the manual close pattern with idiomatic defer f.Close()
immediately after the successful open. The original code only closed
the file on the success path; if configReader returned an error, the
file descriptor leaked.

Since this is a read-only file (os.Open), close errors are not
actionable.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

* Address review: check close error on both paths instead of nolint

Remove defer f.Close() with nolint:errcheck directive. Instead, check
the close error explicitly on both the error and success paths, which
satisfies errcheck without suppressing the lint rule.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

---------

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

Coverage Stats

7716 of 9376 relevant lines covered (82.3%)

73.57 hits per line

kubernetes-sigs / kubebuilder / 25791217132 / 1
82%
master: 82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 25791217132.1

kubernetes-sigs / kubebuilder / 25791217132 / 1 82% master: 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 25791217132.1

kubernetes-sigs / kubebuilder / 25791217132 / 1
82%
master: 82%

README BADGES
x