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

bpf: tests: pktgen infra for tunneling + GENEVE-DSR test #26301

Merged
merged 5 commits into from Jun 26, 2023

Conversation

julianwiedmann
Copy link
Member

@julianwiedmann julianwiedmann commented Jun 16, 2023

Add the necessary bits so that pktgen can build tunnel encapsulation headers (UDP, VXLAN, GENEVE). Exercise it with some initial tests - more to come.

@julianwiedmann julianwiedmann added sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/ci This PR makes changes to the CI. labels Jun 16, 2023
@julianwiedmann julianwiedmann force-pushed the 1.14-bpf-tests-tunnel-mode branch 3 times, most recently from dbf569d to 08e8eef Compare June 16, 2023 12:57
@julianwiedmann
Copy link
Member Author

/test

@julianwiedmann julianwiedmann changed the title 1.14 bpf tests tunnel mode bpf: tests: pktgen infra for tunneling + GENEVE-DSR test Jun 16, 2023
@julianwiedmann julianwiedmann added the area/CI Continuous Integration testing issue or flake label Jun 16, 2023
@julianwiedmann julianwiedmann marked this pull request as ready for review June 16, 2023 14:17
@julianwiedmann julianwiedmann requested review from a team as code owners June 16, 2023 14:17
bpf/tests/pktgen.h Outdated Show resolved Hide resolved
bpf/tests/pktgen.h Show resolved Hide resolved
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 22, 2023
Introduce pktgen support for pushing a UDP header. Add one UDP test that
uses it.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
This requires moving the tunnel-related definitions into a separate header
file, so that it can be included from pktgen.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
This will see more usage in subsequent patches.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Test the usual scenario in a config with DSR-Geneve dispatch. The tests for
the other scenarios (backend processing / bpf_host) will come at a later
time.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Note that we can't bump this any further, the test programs get too big.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 26, 2023
@julianwiedmann
Copy link
Member Author

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 26, 2023
@julianwiedmann julianwiedmann merged commit b730e28 into cilium:main Jun 26, 2023
65 checks passed
@julianwiedmann julianwiedmann deleted the 1.14-bpf-tests-tunnel-mode branch June 26, 2023 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants