-
Notifications
You must be signed in to change notification settings - Fork 117
[feat] Add support for repeated testing through two new options: --reruns and --duration
#2839
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
Conversation
|
Can I test this patch? |
|
ok to test |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #2839 +/- ##
===========================================
+ Coverage 86.84% 86.90% +0.05%
===========================================
Files 60 60
Lines 11414 11496 +82
===========================================
+ Hits 9913 9991 +78
- Misses 1501 1505 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
|
Ok to test |
ekouts
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall it looks good, if you fix the very small comment I made I will approve.
teojgo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only the minor comment by @ekouts needs to be fixed. Otherwise it looks good.
Co-authored-by: Eirini Koutsaniti <eirini.koutsaniti@cscs.ch>
This PR introduces the following two options that are meant to allow users to use reframe for stress testing a target system:
--reruns=Noption will rerun the same test suite N times (N+1 total runs).--duration=TIMEOUToption will run the same test suite repeatedly untilTIMEOUTexpires.Compared to the similar
--max-retriesoption that reruns the failed tests, these options will report statistics as well as failures from all runs.Closes #619.
@brandongc FYI
Todos