-
Notifications
You must be signed in to change notification settings - Fork 51
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
Bpfd integration #121
Bpfd integration #121
Conversation
3692af0
to
a0ec143
Compare
a0ec143
to
b1e8a06
Compare
TODO: I need follow up issues for
|
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.
Looking good, glad to see us adding bpfd!
Mostly small comments.
This commit makes the necessary changes which allow the bpf programs to be build with btf. It requires: - The `fix-di` branch of bpf-linker - A patched version of aya (https://github.com/aya-rs/aya/tree/fix-verifier-blixt) - A patched version of bpfd (https://github.com/bpfd-dev/bpfd/tree/blixt-custom-build) Additionally it adds a check in the loader program to check if the blixt programs are installed by bpfd. Signed-off-by: astoycos <[email protected]>
Add new config to allow: - deploying bpfd (with custom blixt settings) - Run integrations on the blixt + bpfd setup - Activate all of this in CI - Allow building of new blixt program bytecode images Signed-off-by: astoycos <[email protected]>
b1e8a06
to
593b04b
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.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: astoycos, shaneutt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This sets up the integration with bpfd
Additionally it adds a deployment scenario to the integration tests so that we run with and without bpfd.
To test it out locally