drop log messages regarding attestation storage to debug

[bobcallaway]

Summary

Release Note

Documentation

[codecov]

Codecov Report

Merging #1408 (b980226) into main (14e3c4b) will increase coverage by 21.34%.
The diff coverage is 50.00%.

@@             Coverage Diff             @@
##             main    #1408       +/-   ##
===========================================
+ Coverage   43.06%   64.41%   +21.34%     
===========================================
  Files          74       82        +8     
  Lines        7458     7908      +450     
===========================================
+ Hits         3212     5094     +1882     
+ Misses       3910     2164     -1746     
- Partials      336      650      +314     

Flag	Coverage Δ
e2etests	47.85% <50.00%> (?)
unittests	43.02% <0.00%> (-0.05%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pkg/api/entries.go	65.94% <50.00%> (+65.94%)	⬆️

... and 54 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[vaikas]

I think this is a fine change, but I don't think it's going to address the bug linked from here, since those were related to getting the attestations?

error fetching attestation by uuid

[haydentherapper]

that looks to be debug already -

rekor/pkg/api/entries.go

Line 142 in 01a2321

log.ContextLogger(ctx).Debugf("error fetching attestation by uuid: %s %v", entryIDstruct.UUID, fetchErr)

[vaikas]

Yeah, but still showing up as error in the console, so I think that's what @k4leung4 was alluding to about that it's where it's being logged that's making it show up as an error?