Skip to content

on update of a LoadBalancer service, untag old ip addresses, if any exist#111

Merged
Gerrit91 merged 3 commits intomasterfrom
fix-lb-ip-change-bug
Apr 1, 2025
Merged

on update of a LoadBalancer service, untag old ip addresses, if any exist#111
Gerrit91 merged 3 commits intomasterfrom
fix-lb-ip-change-bug

Conversation

@mwennrich
Copy link
Contributor

Description

Closes #110

@mwennrich mwennrich force-pushed the fix-lb-ip-change-bug branch from f23f686 to a8ec6d1 Compare March 28, 2025 14:53
@mwennrich
Copy link
Contributor Author

212.34.83.21    ephemeral       metallb-cc246  internet               cd4eac58-46a5-4a31-b59f-2ec207baa817    service:23f10cce-4a84-4430-bb47-805b2c66be8d/default/echoserver
212.34.83.24    static          test           internet               cd4eac58-46a5-4a31-b59f-2ec207baa817

2m28s       Normal    LoadbalancerIP           service/echoserver            212.34.83.21 -> 212.34.83.24

212.34.83.24    static          test           internet               cd4eac58-46a5-4a31-b59f-2ec207baa817    service:23f10cce-4a84-4430-bb47-805b2c66be8d/default/echoserver

@mwennrich mwennrich marked this pull request as ready for review March 31, 2025 05:57
@mwennrich mwennrich requested a review from a team as a code owner March 31, 2025 05:57
Copy link
Contributor

@Gerrit91 Gerrit91 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@Gerrit91 Gerrit91 merged commit 228d67b into master Apr 1, 2025
1 check passed
@Gerrit91 Gerrit91 deleted the fix-lb-ip-change-bug branch April 1, 2025 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ip leakage when loadbalancer IP gets updated

2 participants