Skip to content
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

aya: Return error messages from netlink #690

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dave-tucker
Copy link
Member

This returns error strings from netlink since they are more informative than the raw os error. For example:

"Device or Resource Busy" vs. "XDP program already attached".

@netlify
Copy link

netlify bot commented Jul 26, 2023

Deploy Preview for aya-rs-docs failed.

Name Link
🔨 Latest commit ad961db
🔍 Latest deploy log https://app.netlify.com/sites/aya-rs-docs/deploys/64c19aa09df0e70008f3c6f3

aya/src/sys/netlink.rs Outdated Show resolved Hide resolved
aya/src/sys/netlink.rs Outdated Show resolved Hide resolved
aya/src/sys/netlink.rs Show resolved Hide resolved
aya/src/sys/netlink.rs Show resolved Hide resolved
This returns error strings from netlink since they are more informative
than the raw os error. For example:

"Device or Resource Busy" vs. "XDP program already attached".

Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
@mergify
Copy link

mergify bot commented Jul 31, 2023

@dave-tucker, this pull request is now in conflict and requires a rebase.

@mergify mergify bot added needs-rebase aya This is about aya (userspace) labels Jul 31, 2023
@tamird
Copy link
Member

tamird commented Jul 31, 2023

The second commit should probably go first? or they should be squashed?

@mergify mergify bot removed the needs-rebase label Sep 14, 2023
@mergify
Copy link

mergify bot commented Sep 14, 2023

@dave-tucker, this pull request is now in conflict and requires a rebase.

@mergify mergify bot added the needs-rebase label Sep 14, 2023
Copy link

mergify bot commented Feb 6, 2024

@dave-tucker, this pull request is now in conflict and requires a rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aya This is about aya (userspace) needs-rebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants