-
Notifications
You must be signed in to change notification settings - Fork 412
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: Move helpers package from libbpfgo to local repository #4090
chore: Move helpers package from libbpfgo to local repository #4090
Conversation
Copied helpers package from libbpfgo to local repo to avoid dependency issues. This eliminates frequent updates of libbpfgo and simplifies our workflow. Allows faster bug fixes and updates, improving development efficiency.
Ensure safe type assertions in kernel config parsing to prevent potential panics. Used type switch assignment and added type checks for value comparisons.
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.
Awesome.
This is the initial part to close aquasecurity/libbpfgo#243 and aquasecurity/libbpfgo#297.
P.S.: The libbpfgo incoming code lacks assertion checks which is enforced in Tracee via revive: https://github.com/aquasecurity/tracee/actions/runs/9305562614/job/25612793173?pr=4090#step:5:15
ALMALINUX_MANTISBT_PROJECT="AlmaLinux-8" | ||
ALMALINUX_MANTISBT_PROJECT_VERSION="8.7" | ||
REDHAT_SUPPORT_PRODUCT="AlmaLinux" | ||
REDHAT_SUPPORT_PRODUCT_VERSION="8.7" |
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.
nit: came with the missing blank end line.
Ensure errors from file closures are handled to prevent potential issues. - Added deferred anonymous functions to check and log errors from f.Close(). - Improved logging for better error tracking.
This commit integrates the libbpfgo packages into the tracee repository. These packages were previously copied over from libbpfgo in a separate PR (aquasecurity#4090) to resolve dependency issues. Now, tracee is utilizing these packages, providing better compatibility and ensuring smoother development.
This commit integrates the libbpfgo packages into the tracee repository. These packages were previously copied over from libbpfgo in a separate PR (aquasecurity#4090) to resolve dependency issues. Now, tracee is utilizing these packages, providing better compatibility and ensuring smoother development.
This commit integrates the libbpfgo packages into the tracee repository. These packages were previously copied over from libbpfgo in a separate PR (#4090) to resolve dependency issues. Now, tracee is utilizing these packages, providing better compatibility and ensuring smoother development.
This commit integrates the libbpfgo packages into the tracee repository. These packages were previously copied over from libbpfgo in a separate PR (aquasecurity#4090) to resolve dependency issues. Now, tracee is utilizing these packages, providing better compatibility and ensuring smoother development.
This commit integrates the libbpfgo packages into the tracee repository. These packages were previously copied over from libbpfgo in a separate PR (#4090) to resolve dependency issues. Now, tracee is utilizing these packages, providing better compatibility and ensuring smoother development.
This commit integrates the libbpfgo packages into the tracee repository. These packages were previously copied over from libbpfgo in a separate PR (aquasecurity#4090) to resolve dependency issues. Now, tracee is utilizing these packages, providing better compatibility and ensuring smoother development.
1. Explain what the PR does
Copied helpers package from libbpfgo to local repo to avoid dependency issues. This eliminates frequent updates of libbpfgo and simplifies our workflow. Allows faster bug fixes and updates, improving development efficiency.
2. Explain how to test it
3. Other comments