Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

static ip in exclude-ips can be allocated normally when subnet's avai… #3031

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

hongzhen-ma
Copy link
Collaborator

@hongzhen-ma hongzhen-ma commented Jul 12, 2023

…lableIPs is 0

What type of this PR

  • Bug fixes

Which issue(s) this PR fixes:

Fixes #3030

WHAT

🤖 Generated by Copilot at 9de9b2c

Enable static IP allocation for pods from excluded IPs. Refactor pod.go to handle subnets with no free IPs.

🤖 Generated by Copilot at 9de9b2c

Oh we're the coders of the pod.go file
And we don't like to waste a single IP
So we'll allocate them from the exclude list
Heave away, me hearties, heave away

HOW

🤖 Generated by Copilot at 9de9b2c

  • Enable allocating static IPs from exclude-ips list of a subnet (link, link)

@hongzhen-ma hongzhen-ma added bug Something isn't working need backport labels Jul 12, 2023
@hongzhen-ma hongzhen-ma added this to In progress in 2023-6 via automation Jul 12, 2023
@hongzhen-ma hongzhen-ma merged commit 97a3e1b into master Jul 12, 2023
59 checks passed
2023-6 automation moved this from In progress to Done Jul 12, 2023
@hongzhen-ma hongzhen-ma deleted the static branch July 12, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need backport
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

static ip in exclude-ips can be allocated normally when subnet's availableIPs is 0
2 participants