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.

Sign up for GitHub

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

Jump to bottom

Port verifier-test.sh to Go #24522

Closed
ti-mo opened this issue Mar 22, 2023 · 0 comments · Fixed by #24538
Closed

Port verifier-test.sh to Go #24522

ti-mo opened this issue Mar 22, 2023 · 0 comments · Fixed by #24538
Assignees
@ti-mo
Labels
area/CI Continuous Integration testing issue or flake sig/loader Impacts the loading of BPF programs into the kernel.

Comments

@ti-mo
Copy link
Contributor

ti-mo commented Mar 22, 2023

Since 9d3f3125bd, the entry point of the test is already testing.T-based.

This issue is for porting the rest of verifier-test.sh to Go. The majority of the script is dealing with managing existing bpffs state and setting up a dummy interface, which can all be dropped. ELFs can be loaded with pinning disabled and without attaching them to any kernel hooks, making the test completely isolated.
@ti-mo ti-mo self-assigned this Mar 22, 2023
@ti-mo ti-mo added sig/loader Impacts the loading of BPF programs into the kernel. area/CI Continuous Integration testing issue or flake labels Mar 22, 2023
@ti-mo ti-mo mentioned this issue Mar 22, 2023
Closed
@ti-mo ti-mo mentioned this issue Mar 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ti-mo ti-mo

Labels
area/CI Continuous Integration testing issue or flake sig/loader Impacts the loading of BPF programs into the kernel.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bpf: port verifier test to Go cilium/cilium
bpf: port verifier test to Go ti-mo/cilium
1 participant
@ti-mo