-
Notifications
You must be signed in to change notification settings - Fork 2.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
docs: Add developers guide page about BPF testing framework #20165
Conversation
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.
I've got a few suggestions, and tons of nitpicks; but great work overall!
70c1f82
to
a38f084
Compare
Ah, I see my English is failing me. Thanks, all of them good suggestions. |
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.
Few more nits, but thanks a lot for addressing all the previous batch!
a38f084
to
a2fcea9
Compare
a2fcea9
to
f7e31ed
Compare
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.
One left, but all good for me otherwise. Thanks!
f7e31ed
to
412c359
Compare
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 writing a detailed guide! I only followed the first half of the guide.
412c359
to
857d087
Compare
This commits adds a new page to the "For developers/Testing" section. The page covers how to run and create BPF tests within the framework that was added in cilium#20017 Signed-off-by: Dylan Reimerink <dylan.reimerink@isovalent.com>
857d087
to
c1468d2
Compare
This PR adds a new page to the "For developers/Testing" section.
The page covers how to run and create BPF tests within the framework
that was added in #20017