Code coverage hit after updating to go1.21

[justaugustus]

We took a code coverage hit after updating to go1.21 in #3387.
Not sure if this is a blip, but worth investigating.

Codecov Report

Merging #3387 (9b7c9ae) into main (198aa96) will decrease coverage by 6.22%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3387      +/-   ##
==========================================
- Coverage   74.30%   68.08%   -6.22%     
==========================================
  Files         178      175       -3     
  Lines       12876    12856      -20     
==========================================
- Hits         9567     8753     -814     
- Misses       2766     3593     +827     
+ Partials      543      510      -33     

Originally posted by @codecov[bot] in #3387 (comment)

[spencerschrock]

This is due to e2e tests which run only on push to main, due to their use of a PAT. So all PRs currently show this 6% decrease, but have the higher coverage when merged.
cc @naveensrinivasan

[naveensrinivasan]

This is due to e2e tests which run only on push to main, due to their use of a PAT. So all PRs currently show this 6% decrease, but have the higher coverage when merged. cc @naveensrinivasan

Yes, It is the case. I am closing it. Please reopen if you have concerns.