Repository navigation
#5545 - Split core eBPF and network extension tests - #5575
Open
VINOD KLSWAMY (kumarvin123) wants to merge 38 commits into
Open
VINOD KLSWAMY (kumarvin123) wants to merge 38 commits into
VINOD KLSWAMY (kumarvin123) wants to merge 38 commits into
Conversation
VINOD KLSWAMY (kumarvin123)
force-pushed
the
user/vinodko/Convert-to-sampleext
branch
from
September 25, 2026 20:16
cb17af7 to
dba606b
Compare
VINOD KLSWAMY (kumarvin123)
marked this pull request as ready for review
September 25, 2026 22:50
VINOD KLSWAMY (kumarvin123)
requested review from
Alan Jowett (Alan-Jowett),
Andrew Beltrano (abeltrano),
Dave Thaler (dthaler),
Laksh (lakshk98),
Matthew Ige (matthewige),
Maksim S. (maxsud),
D. Michael Agun (mikeagun),
Michael Friesen (mtfriesen),
poornagmsft,
Anurag Saxena (saxena-anurag) and
Shankar Seal (shankarseal)
as code owners
September 25, 2026 22:50
Added netebpfext load tests project to README. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Move bpf_get_process_start_key_* and bpf_get_thread_start_time_* test cases (and their helpers send_traffic, try_get_process_start_key, run_process_start_key_test, run_thread_start_time_test) from api_test.cpp to netebpfext_load_test.cpp, since they use CGROUP_INET4/6_CONNECT hooks that require netebpfext. Move perform_socket_bind from api_test_jit.cpp/h to socket_helper.h/cpp so it is available as a shared test utility to both api_test and netebpfext tests. Addresses review comments: - bpf_get_process_start_key/bpf_get_thread_start_time tests moved to netebpf-related tests - perform_socket_bind moved to netebpf test utility (socket_helper) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Shankar Seal (shankarseal)
approved these changes
Nov 11, 2026
Shankar Seal (shankarseal)
left a comment
Shankar Seal (shankarseal)
left a comment
Collaborator
There was a problem hiding this comment.
Approving - but there will be more changes to ensure any remaining tests that need to be moved to sample extension is done.
Tracked by #5704
Laksh (lakshk98)
approved these changes
Nov 11, 2026
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Describe the purpose of and changes within this Pull Request.
Splits the Unit tests and driver tests to remove the dependency on netebpfext so that they can be run independently.
Please reference an issue with a keyword such as Fixes #abc, Closes #xyz, etc., so the work can be tracked.
Fixes #5545
Testing
Ran the Tests manually and will watch the CI/CD pipeline to succeed
Do any existing tests cover this change? Are new tests needed?
Yes
If new tests were added:
Documentation
Is there any documentation impact for this change?
None
Installation
Is there any installer impact for this change?
No