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

treewide: Switch ipcache interface to netip.Prefix #21586

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

joestringer
Copy link
Member

The netip.Prefix provides a much richer IP prefix type that's easier to
work with. Switch all of the remaining users of the ipcache CIDR
allocation to this new type.

The netip.Prefix provides a much richer IP prefix type that's easier to
work with. Switch all of the remaining users of the ipcache CIDR
allocation to this new type.

Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer requested review from a team as code owners October 4, 2022 21:49
@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Oct 4, 2022
@joestringer joestringer requested review from a team as code owners October 4, 2022 21:49
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Oct 4, 2022
@joestringer
Copy link
Member Author

joestringer commented Oct 4, 2022

/test

Job 'Cilium-PR-K8s-1.25-kernel-net-next' failed:

Click to show.

Test Name

K8sDatapathConfig Check BPF masquerading with ip-masq-agent VXLAN

Failure Output

FAIL: Failed to add ip route

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-1.25-kernel-net-next so I can create one.

@joestringer
Copy link
Member Author

Hit only #21120 , reviews are in, merging.

@joestringer joestringer merged commit 379a57d into cilium:master Oct 5, 2022
@joestringer joestringer deleted the submit/moar-netip-prefix branch October 5, 2022 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants