-
Notifications
You must be signed in to change notification settings - Fork 46
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
Convert InfoBond::ArpValidate
into an enum.
#110
Convert InfoBond::ArpValidate
into an enum.
#110
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #110 +/- ##
==========================================
- Coverage 63.97% 63.82% -0.16%
==========================================
Files 140 140
Lines 8617 8653 +36
==========================================
+ Hits 5513 5523 +10
- Misses 3104 3130 +26 ☔ View full report in Codecov by Sentry. |
Also, it seems that these PRs steadily lower the code-coverage. Is there place for tests that open an actual socket and try to create rtnl objects; is this problematic with CI running us in a non-privileged context? |
I have document on how to capture netlink package and convert it into Vec at: https://github.com/rust-netlink/netlink-packet-route/blob/main/README.md You may setup up a bond with arp_validate value changed to non-default one. Then capture the netlink package kernel replied to the request of
For this crate, there is no real netlink communication to kernel. Because this crate only handle parsing and emitting netlink package itself, no socket communication code included. All tests in this crate are unit test, no privilege required. |
Thank you both! Unfortunately life got in the way; |
dbb355f
to
1f51fc8
Compare
Life got out of the way! I've added a test as instructed; dump is taken of I did notice that serializing EDIT: Yep, the kernel-sent message claimed that the length of Stats64 is 196, and omitted the last 4 fields in |
1f51fc8
to
236fbe9
Compare
236fbe9
to
6300363
Compare
6300363
to
d432894
Compare
I have amend the patch:
|
Closes rust-netlink#80 Signed-off-by: Gilad Naaman <[email protected]>
d432894
to
87c4997
Compare
Signed-off-by: Gris Ge <[email protected]>
87c4997
to
e12f9be
Compare
Closes #80