-
Notifications
You must be signed in to change notification settings - Fork 274
fix: Add TCX support to the packetparser plugin #1994
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
base: main
Are you sure you want to change the base?
Conversation
|
@microsoft-github-policy-service agree company="Adobe" |
5135bf7 to
6d5f64f
Compare
Signed-off-by: Brian Ugie <brian@ugie.org>
Signed-off-by: Brian Ugie <brian@ugie.org>
e3aa8be to
758bfef
Compare
Signed-off-by: Brian Ugie <brian@ugie.org>
Signed-off-by: Brian Ugie <brian@ugie.org>
|
I am finding it difficult to reproduce the CI errors locally. Could use some guidance.
Could I get some help with how to run CI locally to reproduce that is run via Github Actions to help avoid this commit noise? I am running Ubuntu 24.04 |
|
Hi @bugie Thanks for raising the PR. With TCX available in newer kernels, adding support for it a great idea.
|
Description
This change will add support for TCX in the Packetparser plugin using the ebpf link package.
This PR introduces a new environment variable to enable TCX,
RETINA_FORCE_TCX_MODE. For backward compatibility this must be set to "true" to be enabled. If set to true and TCX is not found to be supported, TC is the fall back.Related Issue
#1958
Checklist
git commit -S -s ...). See this documentation on signing commits.Screenshots (if applicable) or Testing Completed
Please add any relevant screenshots or GIFs to showcase the changes made.
Additional Notes
Add any additional notes or context about the pull request here.
Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.