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

Update performance tests. #260

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Update performance tests. #260

wants to merge 1 commit into from

Conversation

msaarine
Copy link
Collaborator

@msaarine msaarine commented Feb 27, 2025

Basically only some small changes to get these 8 tests to work on Lenovo-X1 & Orin-NX as well. Needed to work with firewall settings. Other Devices than Lenovo-X1 or Orin-NX will use the original solution (iptables -F still left as was for the other targets)

Changes:

  • Selection where to connect, if Lenovo or NX -> net-vm, other targets ghaf-host
  • Opened port 5201 from iptables. Originally this was issue since using flushing (iptables -F ) in 'net-vm' got the net-vm into 'totally stucked -state.'
  • Moved starting/stopping of 'iperf server' to Suite SetUp/Suite Teardown ( started/stopped only once -> more robust, and bit faster execution) compared to the original solution where iperf-server was started/stopped in every test separately.
  • In local trials there was some random test failures-> added a keyword to check if the data collection was successfully passed or not. If not -> FAIL with clear statement.
  • Added ' lenovo-x1' tags back for the tests

@msaarine msaarine marked this pull request as draft February 27, 2025 13:56
@msaarine msaarine force-pushed the performance_test_fix branch 3 times, most recently from 41a3115 to 18f33a3 Compare February 28, 2025 12:12
Signed-off-by: Mikko Saarinen <[email protected]>
@msaarine msaarine force-pushed the performance_test_fix branch from 18f33a3 to a91f935 Compare February 28, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant