cluster-api-provider-linode icon indicating copy to clipboard operation
cluster-api-provider-linode copied to clipboard

[improvement] : simplify rate limits as we now have only one limit for POST calls

Open rahulait opened this issue 1 year ago • 1 comments

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer:

TODOs:

  • [ ] squashed commits
  • [ ] includes documentation
  • [ ] adds unit tests
  • [ ] adds or updates e2e tests

rahulait avatar Oct 11 '24 13:10 rahulait

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 64.91%. Comparing base (1c49829) to head (4a6665d). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
controller/linodemachine_controller_helpers.go 66.66% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #540      +/-   ##
==========================================
- Coverage   64.93%   64.91%   -0.02%     
==========================================
  Files          79       79              
  Lines        4309     4298      -11     
==========================================
- Hits         2798     2790       -8     
+ Misses       1242     1239       -3     
  Partials      269      269              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Oct 11 '24 13:10 codecov[bot]