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

Add a new console variable to control length of test #26

Merged
merged 7 commits into from
Jun 3, 2022

Conversation

lmbr-pip
Copy link
Contributor

@lmbr-pip lmbr-pip commented May 24, 2022

Add a new cvar variable to control length of test. If cvar is 0 or missing, tests run in old indefinite behaviour, with cvar tests run up to length specified, then we dump stats and exit.

Based on this old PR: #15 but uses friendlier exit method,

Tested

  • Windows without variable
  • Windows with variable set to zero
  • Windows with variable set to 500ms
  • Same checks on Linux (Note: NetSoakTest does not run on Linux currently)

lmbr-pip and others added 4 commits May 11, 2022 17:15
Signed-off-by: Potter <rppotter@amazon.com>
Signed-off-by: Potter <rppotter@amazon.com>
Signed-off-by: Potter <rppotter@amazon.com>
Signed-off-by: Pip Potter <61438964+lmbr-pip@users.noreply.github.com>
@lmbr-pip lmbr-pip requested review from AMZN-Gene, AMZN-puvvadar and a team May 24, 2022 20:12
@AMZN-puvvadar
Copy link
Contributor

Looks good though seems like there's a merge that needs to happen

Signed-off-by: Pip Potter <61438964+lmbr-pip@users.noreply.github.com>
Signed-off-by: Pip Potter <61438964+lmbr-pip@users.noreply.github.com>
@lmbr-pip lmbr-pip requested a review from Kadino May 27, 2022 23:10
Signed-off-by: Pip Potter <61438964+lmbr-pip@users.noreply.github.com>
@lmbr-pip lmbr-pip merged commit 9e76820 into o3de:development Jun 3, 2022
@lmbr-pip lmbr-pip deleted the timed_exit branch June 3, 2022 05:12
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.

6 participants