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.

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

Enable backward compat tests #2747

Closed
saxena-anurag opened this issue Aug 10, 2023 · 1 comment · Fixed by #3421
Closed

Enable backward compat tests #2747

saxena-anurag opened this issue Aug 10, 2023 · 1 comment · Fixed by #3421
Assignees
Labels
blocked Blocked on another issue that must be done first enhancement New feature or request P1 tests triaged Discussed in a triage meeting
Milestone

Comments

@saxena-anurag
Copy link
Contributor

saxena-anurag commented Aug 10, 2023

Describe the feature you'd like supported

To ensure eBPF platform is backward compatible, we need to enable tests for backward compatibility. Currently xdp-for-windows uses the test binaries from a previous release to execute on the latest bits, to ensure there is no regression.

We cannot do the same as most of our tests are unit_tests, which means they statically compile with the ebpfcore, api, etc. So there is no way to run the older tests against new binaries.

Currently there is a minimal test which loads a 0.9 native driver using latest drivers.

Proposed solution

The solution has basically 2 parts:

Part 1
We currently have some kernel tests. Automate the existing kernel tests for regression tests -- test binaries from previous releases should pass using the latest eBPF binaries. This can possibly be run as scheduled task.

Part 2
Refactor all the unit tests which use public eBPF APIs so that they can be compiled both for unit tests and for kernel tests.
Once this is done, automate these new kernel tests also for regression tests.

Additional context

No response

@saxena-anurag saxena-anurag added the enhancement New feature or request label Aug 10, 2023
@dahavey dahavey added the triaged Discussed in a triage meeting label Aug 14, 2023
@dahavey dahavey modified the milestones: 2308, 2309 Aug 14, 2023
@dahavey dahavey modified the milestones: 2309, 2310 Sep 18, 2023
@shankarseal shankarseal modified the milestones: 2310, 2311 Oct 23, 2023
@shankarseal shankarseal modified the milestones: 2311, 2401 Nov 28, 2023
@saxena-anurag saxena-anurag modified the milestones: 2401, 2402 Jan 29, 2024
@saxena-anurag
Copy link
Contributor Author

The regression tests should replace the following from a "previous" release artifacts:

  1. export_program_info.exe
  2. export_program_info_sample.exe
  3. sample_ext.sys
  4. native programs, .o files
  5. api_test.exe

@Alan-Jowett Alan-Jowett modified the milestones: 2402, 2403 Feb 20, 2024
@dahavey dahavey added blocked Blocked on another issue that must be done first tests labels Mar 6, 2024
@dahavey dahavey modified the milestones: 2403, 2404 Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked on another issue that must be done first enhancement New feature or request P1 tests triaged Discussed in a triage meeting
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants