We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
AlertCondition.threshold
Currently AlertCondition.threshold is defined as a number in the spec but an int in the Oslo struct.
number
int
I can see use cases of wanting to threshold on more than 100%, 200%, 300%, etc, such as 150%.
I don't know if "number" in our definition would support a value of 1.5, or whether it needs to change.
1.5
Once we've clarified the meaning in the spec, a follow-on issue would be to adjust the Oslo structs to align with it.
Field is a float to support decimal values.
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Summary
Currently
AlertCondition.threshold
is defined as anumber
in the spec but anint
in the Oslo struct.I can see use cases of wanting to threshold on more than 100%, 200%, 300%, etc, such as 150%.
I don't know if "number" in our definition would support a value of
1.5
, or whether it needs to change.Once we've clarified the meaning in the spec, a follow-on issue would be to adjust the Oslo structs to align with it.
Possible fixes
Field is a float to support decimal values.
The text was updated successfully, but these errors were encountered: