Refactor bpf loader
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
make the bpf loader more readable and deduplicate error handling
Which issue(s) this PR fixes: Fixes #858
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Codecov Report
Attention: Patch coverage is 58.33333% with 195 lines in your changes missing coverage. Please review.
Project coverage is 54.58%. Comparing base (
390e845) to head (832964e). Report is 41 commits behind head on main.
| Files with missing lines | Coverage Δ | |
|---|---|---|
| pkg/bpf/utils/bpf_helper.go | 100.00% <100.00%> (ø) |
|
| pkg/bpf/utils/pin.go | 57.57% <100.00%> (ø) |
|
| pkg/cache/v2/maps/cluster.go | 56.09% <ø> (ø) |
|
| pkg/cache/v2/maps/common.go | 56.09% <ø> (ø) |
|
| pkg/controller/client.go | 65.43% <100.00%> (ø) |
|
| pkg/controller/workload/workload_processor.go | 61.76% <ø> (ø) |
|
| pkg/utils/test/bpf_map.go | 37.14% <100.00%> (-4.97%) |
:arrow_down: |
| pkg/controller/controller.go | 0.00% <0.00%> (ø) |
|
| pkg/status/status_server.go | 42.30% <0.00%> (ø) |
|
| pkg/controller/workload/workload_controller.go | 62.74% <0.00%> (ø) |
|
| ... and 9 more |
... and 4 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update bff5b9a...832964e. Read the comment docs.
/assign @lec-bit
cc @nlgwcy
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nlgwcy
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [nlgwcy]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment