-
Notifications
You must be signed in to change notification settings - Fork 4.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
syscall: add getifaddrs to os syscalls #18513
Conversation
Signed-off-by: Jose Nino <jnino@lyft.com>
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 for working on this. Should there be some test for this?
/wait
AFAICT there is no direct testing class for OsSysCalls, but instead the functions are indirectly covered by tests that use them. In this case the test that covers |
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
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.
on the basis of tacit approval from @mattklein123 - lgtm
This reverts commit fc04382. Signed-off-by: Mike Schore <mike.schore@gmail.com>
Commit Message: syscall - add getifaddrs to os syscalls
Additional Description: formalize getifaddrs into os syscalls.
Risk Level: low
Testing: existing calling code and UT over that code.