-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Feature: alert rearm setting which allows periodic resending of alert messages. #631
Conversation
logger.info("Alert %d new state: %s", alert.id, new_state) | ||
old_state = alert.state | ||
alert.update_instance(state=new_state) | ||
alert.update_instance(state=new_state, last_triggered_at=datetime.datetime.now()) |
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.
I think that if you use the utcnow
function from redash.utils
module here and on line 246, you won't need to replace the timezone.
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.
@nielsdraaisma can you make this change?
@arikfr thanks for the PR, sorry I just noticed it. @nielsdraaisma is busy with his new role. |
…chat Feature: send alert notifications to HipChat or web hook
@blu35ky thanks! do you mind also rebasing it w/ latest master, so I can easily merge it? |
@arikfr np, done. 👍 |
Feature: alert rearm setting which allows periodic resending of alert messages.
Merged now. Apologies that it took so long. |
Feature: alert rearm setting which allows periodic resending of alert messages.
This pull request adds a setting which matches against a rearm timer after query execution, this allows users to receive daily emails for alerts that are trigger over a longer period of time.