Skip to content

Remove pytest packet noise, fix HW test side-effects#682

Merged
guvenc merged 1 commit intomainfrom
fix/pytest_packet_noise
Jun 6, 2025
Merged

Remove pytest packet noise, fix HW test side-effects#682
guvenc merged 1 commit intomainfrom
fix/pytest_packet_noise

Conversation

@PlagueCZ
Copy link
Copy Markdown
Contributor

While using --graphtrace in dpservice PyTest suite, I noticed I am getting unwanted packets:

2025-05-29 23:29:07.369 3(worker) D GRAPH: drop        #0:   1 >> DROP          : 22:22:22:22:22:00 -> 33:33:00:00:00:16 / 0:0:0:0:0:0:0:0 -> ff02:0:0:0:0:0:0:16

This seems to be some IPv6 autoconfiguration/nd stuff. But when testing we do not even need the kernel IPv6 stack, so the best solution seemed to be just disabling it altogether.

That would of course break your host configuration in real HW tests, so I changed the code to only do it for TAP devices. This also fixed an old bug where HW tests removed the IPv6 address from the real HW card.

@PlagueCZ PlagueCZ requested a review from a team as a code owner May 29, 2025 23:37
@github-actions github-actions bot added bug Something isn't working size/XS labels May 29, 2025
@guvenc guvenc linked an issue Jun 4, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@byteocean byteocean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

localtests and benchmarktests still work. LGTM.

@PlagueCZ PlagueCZ force-pushed the fix/pytest_packet_noise branch from 4737294 to fa019b7 Compare June 5, 2025 22:04
@guvenc guvenc merged commit fa019b7 into main Jun 6, 2025
5 checks passed
@guvenc guvenc deleted the fix/pytest_packet_noise branch June 6, 2025 07:45
@github-project-automation github-project-automation bot moved this to Done in Roadmap Jun 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking bug Something isn't working size/XS

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove pytest packet noise, fix HW test side-effects

4 participants