-
Notifications
You must be signed in to change notification settings - Fork 0
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
Permit configuring maximum number of seconds to wait for test command #42
Conversation
Benchmarks:
Fuzz result sizes:
|
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.
nice!
Adds a `--startup-timeout-seconds` flag (and `ABQ_STARTUP_TIMEOUT_SECONDS` env var) to configure the maximum amount of time to wait for the native test process to startup.
Co-authored-by: Michael Glass <me@mike.is>
fabf9ab
to
e97944b
Compare
@@ -3924,7 +3924,7 @@ fn kill_on_early_startup_timeout_seconds() { | |||
let simulation = [ | |||
Connect, | |||
// | |||
Sleep(Duration::from_millis(100)), | |||
Sleep(Duration::MAX), |
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.
will def timeout
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.
the intention is that we kill it before then
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.
Sorry. Color commentary.
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.
Missing 🤣 or something
Benchmarks:
Fuzz result sizes:
|
Adds a
--startup-timeout-seconds
flag (andABQ_STARTUP_TIMEOUT_SECONDS
env var) to configure the maximum amount of time to wait for the native test process to startup.