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

ipam: add method to get IP owner per pool #24358

Merged
merged 1 commit into from Mar 14, 2023

Conversation

tklauser
Copy link
Member

Avoid accessing a nil map once we use more than just the default pool.

Fixes: e3f1a22 ("ipam: track IP owner per pool")

@tklauser tklauser added release-note/misc This PR makes changes that have no direct user impact. sig/ipam IP address management, including cloud IPAM labels Mar 14, 2023
@tklauser tklauser requested a review from gandro March 14, 2023 09:05
@tklauser tklauser requested a review from a team as a code owner March 14, 2023 09:05
Avoid accessing a nil map once we use more than just the default pool.

Fixes: e3f1a22 ("ipam: track IP owner per pool")
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser force-pushed the pr/tklauser/ipam-owner-pool-fix branch from 036b433 to 6e1baee Compare March 14, 2023 09:12
@tklauser
Copy link
Member Author

/test

@tklauser
Copy link
Member Author

gateway-api-conformance-test failed on #24217. All other tests passed and review is in. Marking as ready to merge.

@tklauser tklauser added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 14, 2023
@borkmann borkmann merged commit 945cc9b into master Mar 14, 2023
@borkmann borkmann deleted the pr/tklauser/ipam-owner-pool-fix branch March 14, 2023 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/ipam IP address management, including cloud IPAM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants