Skip to content

Conversation

hcwagner
Copy link
Contributor

This PR is focused on adding support for IP Reservation while using NodeBalancers as a Service LoadBalancer. This assume that the customer has:

Reserved an IP through Linode CLI, CM, terraform, etc.
Supplies the reserved IP attached to their account through the provided Annotation

@hcwagner hcwagner self-assigned this Aug 14, 2025
@hcwagner hcwagner added the new-feature for new features in the changelog. label Aug 14, 2025
Copy link

codecov bot commented Aug 14, 2025

Codecov Report

❌ Patch coverage is 90.90909% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.06%. Comparing base (39a1d7b) to head (f875f55).

Files with missing lines Patch % Lines
cloud/linode/loadbalancers.go 90.90% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #437      +/-   ##
==========================================
+ Coverage   71.88%   72.06%   +0.17%     
==========================================
  Files          19       19              
  Lines        3625     3658      +33     
==========================================
+ Hits         2606     2636      +30     
- Misses        779      781       +2     
- Partials      240      241       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@komer3 komer3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hcwagner hcwagner merged commit 7a5d9e7 into main Sep 9, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature for new features in the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants