-
Notifications
You must be signed in to change notification settings - Fork 94
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
chore(prog): Add autoattach api #406
Conversation
Add it! |
@chentao-kernel thank you for your contribution. Would you mind creating a selftest for this new API? You could follow this lead: https://github.com/aquasecurity/libbpfgo/tree/main/selftest/set-attach |
hi,geyslan, no problem, i will do it latter, thank you for your reply! |
3a256f4
to
7279938
Compare
done, i also add
done! |
Signed-off-by: Tao Chen <chen.dylane@gmail.com> Co-authored-by: Geyslan Gregório <geyslan@gmail.com>
@chentao-kernel I've requested a pull from you in your own branch (changing the way it tests). Do you mind update it and push force? Thanks. |
|
19fbcd7
to
fefd228
Compare
@chentao-kernel I think you miss that. After signing the CLA I can merge the PR. |
Excuse me, now finished. |
hi,geyslan, i saw the libbpf released some new feature, so libbpfgo developed base on the latest libbpf version or |
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.
LGTM
Feel free to pick some open issue or bring us new API functionalities. Thank you for your contribution. ❤️ |
No description provided.