-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Add intended timestamp #191717
Add intended timestamp #191717
Conversation
/ci |
/ci |
/ci |
/ci |
2 similar comments
/ci |
/ci |
@elasticmachine merge upstream |
/ci |
…nto intentend-timestamp
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.
obs-ux-management change LGTM
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.
response ops changes LGTM
@elasticmachine merge upstream |
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.
Looks good to me! I just had one followup question about processes related to new field additions. Thanks Nikita!
@@ -153,6 +153,7 @@ function alertsAreTheSame(alertsA: any[], alertsB: any[]): void { | |||
'kibana.alert.rule.uuid', | |||
'kibana.alert.rule.execution.uuid', | |||
'kibana.alert.rule.execution.timestamp', | |||
'kibana.alert.intended_timestamp', |
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 job adding these "omission" entries where necessary. Is this something that you just knew to do, or do we have some kind of documentation pointing developers to these blocklists when new fields are added?
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 mostly follow the recent kibana.alert.rule.execution.timestamp
changes, didn't see any docs
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
## Add new field to alert Add optional `kibana.alert.intended_timestamp`. For scheduled rules it has the same values as ALERT_RULE_EXECUTION_TIMESTAMP (`kibana.alert.rule.execution.timestamp`) for manual rule runs (backfill) it - will get the startedAtOverridden For example if i have event at 14:30 And if we run manual rule run from 14:00-15:00, then alert will have `kibana.alert.intended_timestamp` at 15:00 --------- Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Add new field to alert
Add optional
kibana.alert.intended_timestamp
. For scheduled rules it has the same values as ALERT_RULE_EXECUTION_TIMESTAMP (kibana.alert.rule.execution.timestamp
)for manual rule runs (backfill) it - will get the startedAtOverridden
For example if i have event at 14:30
And if we run manual rule run from 14:00-15:00, then alert will have
kibana.alert.intended_timestamp
at 15:00