-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Clean up deprecated and unused IPCache APIs after FQDN transition to asynchronous APIs #29657
Clean up deprecated and unused IPCache APIs after FQDN transition to asynchronous APIs #29657
Conversation
Nice! |
8e5ab89
to
eed791c
Compare
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got pinged as tophat for pkg/ip/
, could that one be assigned a codeowner?
Ref. #29657 (review) Signed-off-by: Tobias Klauser <tobias@cilium.io>
Opened #29669 to assign to @cilium/sig-agent. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! ♻️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧹
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than https://github.com/cilium/cilium/pull/29657/files#r1417759146, /LGTM.
Commit 6af2b78 ("ipcache/md: Make IPCache.metadata.remove() generic") changed the method name. Signed-off-by: Tobias Klauser <tobias@cilium.io>
This method is deprecated. Since commit 507259f ("fqdn: move to asynchronous ipcache APIs") its only user is in pkg/ipcache. Unexport it so no new code will accidentally use it. Signed-off-by: Tobias Klauser <tobias@cilium.io>
The AllocateCIDRsForIPs and ReleaseCIDRIdentitiesByID methods are deprecated API and not used anymore since commit 507259f ("fqdn: move to asynchronous ipcache APIs"). Signed-off-by: Tobias Klauser <tobias@cilium.io>
eed791c
to
74c8f16
Compare
/test |
Ref. #29657 (review) Signed-off-by: Tobias Klauser <tobias@cilium.io>
Looks like Chris already covered review for ci-structure. |
Ref. #29657 (review) Signed-off-by: Tobias Klauser <tobias@cilium.io>
Ref. cilium#29657 (review) Signed-off-by: Tobias Klauser <tobias@cilium.io>
Ref. cilium#29657 (review) Signed-off-by: Tobias Klauser <tobias@cilium.io>
Clean up some deprecated/unused ipcache API following #29036 (fixing issue #28930). I've noticed these could be dropped while re-reviewing that PR after merge.
See commits for details.