Skip to content

Conversation

jschwinger233
Copy link
Member

#35231 will use two new ipcache cmd to temporarily delete ipcache entries for strict mode encryption test.

Suggested-by: Timo Beckers timo@isovalent.com

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 21, 2024
@jschwinger233 jschwinger233 added the release-note/misc This PR makes changes that have no direct user impact. label Oct 21, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 21, 2024
@jschwinger233 jschwinger233 force-pushed the gray/cilium-cli-bpf-ipcache branch from 591cf17 to b323dd9 Compare October 21, 2024 11:28
@jschwinger233
Copy link
Member Author

/test

@jschwinger233 jschwinger233 force-pushed the gray/cilium-cli-bpf-ipcache branch from b323dd9 to ee1c467 Compare October 22, 2024 02:46
@jschwinger233
Copy link
Member Author

/test

@jschwinger233 jschwinger233 marked this pull request as ready for review October 22, 2024 04:56
@jschwinger233 jschwinger233 requested review from a team as code owners October 22, 2024 04:56
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 22, 2024
Copy link
Contributor

@squeed squeed left a comment

Choose a reason for hiding this comment

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

Basic idea is fine, just some cleanups.

Do we need to add any warnings that this is a very low-level tool? Or is it sufficiently hidden that nobody will find it (and break their cluster)?

@maintainer-s-little-helper maintainer-s-little-helper bot removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 22, 2024
$ cilium-dbg bpf ipcache delete 10.244.3.110/32
Deleted entry 10.244.3.110/32

$ cilium-dbg bpf ipcache delete 10.244.3.110/31
Error deleting entry 10.244.3.110/31: unable to delete element 10.244.3.110/31 from map cilium_ipcache: delete: key does not exist

Signed-off-by: gray <gray.liang@isovalent.com>
$ cilium-dbg bpf ipcache update 10.244.3.110/32 --tunnelendpoint 172.21.0.2 --identity 6 --encryptkey 255 --clusterid 0
Updated entry 10.244.3.110/32 => identity=6 encryptkey=255 tunnelendpoint=172.21.0.2 flags=<none>

Signed-off-by: gray <gray.liang@isovalent.com>
Signed-off-by: gray <gray.liang@isovalent.com>
make -C Documentation update-cmdref

Signed-off-by: gray <gray.liang@isovalent.com>
@jschwinger233 jschwinger233 force-pushed the gray/cilium-cli-bpf-ipcache branch from ee1c467 to a47b96f Compare October 23, 2024 07:58
@jschwinger233
Copy link
Member Author

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 23, 2024
@squeed squeed added this pull request to the merge queue Oct 23, 2024
Merged via the queue into cilium:main with commit 5c6bd7c Oct 23, 2024
65 checks passed
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants