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

Alerting: Allow any "evaluate for" value >=0 in the alert rule form #35807

Merged
merged 2 commits into from Jun 21, 2021

Conversation

domasx2
Copy link
Contributor

@domasx2 domasx2 commented Jun 16, 2021

What this PR does / why we need it:

It updates "evaluate for" field validator in the rule form to allow any value >= 0. Previously it would allow min 10 seconds.

0-for.mp4

Which issue(s) this PR fixes:

Fixes #35756

Special notes for your reviewer:

@domasx2 domasx2 added this to the 8.1.0 milestone Jun 16, 2021
@domasx2 domasx2 requested review from peterholmberg, nathanrodman, a team and tskarhed and removed request for a team June 16, 2021 10:17
@domasx2 domasx2 added this to In progress in Alerting frontend backlog via automation Jun 16, 2021
@domasx2 domasx2 removed the request for review from tskarhed June 16, 2021 10:20
@domasx2
Copy link
Contributor Author

domasx2 commented Jun 17, 2021

warning

@domasx2 domasx2 moved this from In progress to Review in progress in Alerting frontend backlog Jun 17, 2021
Copy link
Contributor

@davidmparrott davidmparrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

Alerting frontend backlog automation moved this from Review in progress to Reviewer approved Jun 17, 2021
@domasx2 domasx2 merged commit 781ab83 into main Jun 21, 2021
Alerting frontend backlog automation moved this from Reviewer approved to Done Jun 21, 2021
@domasx2 domasx2 deleted the domas/alerting-update-for-validation branch June 21, 2021 12:54
@osg-grafana osg-grafana changed the title Alerting: allow any "evaluate for" value >=0 in the alert rule form Alerting: Allow any "evaluate for" value >=0 in the alert rule form Jul 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Alerting: grafana managed rule form should allow any non negative number for "for"
2 participants