-
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
[RAI-15505] Configurable default values #77
Conversation
src/code-util.jl
Outdated
@@ -345,3 +345,8 @@ function get_logging_io() | |||
ctx = IOContext(io, stream) | |||
return io, ctx | |||
end | |||
|
|||
default_timeout() = parse(Int64, get(ENV, "TEST_REL_TEST_TIMEOUT", "300")) | |||
default_allowed() = Symbol(get(ENV, "TEST_REL_ALLOWED", "warning")) |
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.
allowed is a little vague here - maybe SEVERITY_ALLOWED
?
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.
beautiful! one naming thought, but otherwise looks excellent
cc @nystrom this was the PR I was mentioning where you could set the default allow level to :none
to get a sense of potential diagnostics changes impact
Add a configurable global default for test timeout and allow_unexpected. Update db and engine names to use a similar approach.
Engine naming now has two ways to be configured - changing the default base name (e.g. test_rel-1, test_rel-2, etc) or replacing the generator entirely (e.g. provision engine then generate a different name if there were problems).