-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add cilium bpf nodeid list to bugtool and print nodeid in hex in ipcache dump #26130
Conversation
Sorry, pushed to the same PR other change (dump the nodeid in hex in the IPcache output) |
Marking for backports everywhere. I think the risk is very low given it only affects CLI and bugtool and it's fairly trivial. I also don't expect any conflicts for the CLI changes. |
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.
Looks harmless enough. Leading zeroes would still be truncated, which seems fine.
To help to detect when IPcache is out of sync with locally stored Node IDs. Signed-off-by: Martynas Pumputis <m@lambda.lt>
The Node ID is used in SKB mark used by XFRM policies. The latter print it in hex. So, let's reduce a mental strain by a bit when debugging IPsec issues. Signed-off-by: Martynas Pumputis <m@lambda.lt>
224765d
to
6cd0413
Compare
This isn't covered by end-to-end CI tests. It's reviewed and all other tests passed so marking ready to merge. |
To help to detect when IPcache is out of sync with locally stored Node IDs.