bazel icon indicating copy to clipboard operation
bazel copied to clipboard

Replace `INFO` with `FINE` logging in `CoverageOutputGenerator`

Open fmeum opened this issue 1 year ago • 3 comments

This reduces unconditional log spam for coverage report actions. If needed, users can configure the loglevel by wrapping the CoverageOutputGenerator jar into a java_binary with appropriate jvmopts.

fmeum avatar May 06 '24 19:05 fmeum

If needed, users can configure the loglevel by wrapping the CoverageOutputGenerator jar into a java_binary with appropriate jvmopts

I don't think this is a good way to configure logging levels, although I don't see much of an alternative without adding to Bazel's large collection of flags...

c-mita avatar May 07 '24 12:05 c-mita

@c-mita Are you going to create a new release after the merge?

fmeum avatar May 07 '24 13:05 fmeum

Yes, although I might need to review (or actually formalize) the process first. There's a long standing issue re. provenance (https://github.com/bazelbuild/bazel/issues/15459).

c-mita avatar May 07 '24 17:05 c-mita

Would it be possible to release a remote_coverage_tools with this fix? The coverage log spam is still making our build logs pretty hard to read.

lalten avatar Aug 30 '24 09:08 lalten

@keertk

fmeum avatar Aug 30 '24 09:08 fmeum

@c-mita can you take a look?

keertk avatar Aug 30 '24 14:08 keertk

friendly bump on this

lalten avatar Sep 18 '24 08:09 lalten