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.

Sign up for GitHub

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

Jump to bottom

Rewrite network multicast stress tests into new API + fix them #440

Closed
pevik opened this issue Dec 5, 2018 · 2 comments
Closed

Rewrite network multicast stress tests into new API + fix them #440

pevik opened this issue Dec 5, 2018 · 2 comments
Labels
network

Comments

@pevik
Copy link
Member

pevik commented Dec 5, 2018

Tests in testcases/network/stress/multicast/ depend on rsh (should use tst_net.sh instead of tools in testcases/network/stress/ns-tools/).

While I rewrote some of them (testcases/network/stress/multicast/grp-operation/) some time ago, most of them are still waiting to be rewritten.

IMHO there is no much point trying to fix some issue before cleanup and rewrite into network API.
@pevik pevik added the network label Dec 5, 2018
This was referenced Dec 5, 2018
Open
Closed
@pevik pevik mentioned this issue Dec 10, 2019
Closed
@pevik
Copy link
Member Author

pevik commented Oct 26, 2020

https://patchwork.ozlabs.org/project/ltp/patch/20201026070006.25298-1-pvorel@suse.cz/
(based on comment https://patchwork.ozlabs.org/project/ltp/patch/1603361548-1534-1-git-send-email-zhufy.jy@cn.fujitsu.com/#2558989)

pevik added a commit that referenced this issue Oct 27, 2020
@pevik
net: Remove legacy tests from runtest
a638361 
Remove all network stress tests which haven't been rewritten to the new
API yet from the runtest file (mostly multicast tests + some route tests).
These tests use check_envval and other deprecated tools.

See also:
#310
#440

Suggested-by: Alexey Kodanev <alexey.kodanev@oracle.com>
Acked-by: Alexey Kodanev <alexey.kodanev@oracle.com>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
@pevik
Copy link
Member Author

pevik commented Jan 22, 2021 •
edited

All tests has been rewritten in: ef227ac (query-flood) and 6d6b9d9 (packet-flood). Although it'd deserve some more cleanup (unify grp-operation with the rest), but that's a separate problem.

@pevik pevik closed this as completed Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
network
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pevik