-
Notifications
You must be signed in to change notification settings - Fork 177
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
Implement CHART source alert (scheduled task) #108
Conversation
🦋 Changeset detectedLatest commit: 22b4432 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
}) => { | ||
const searchQuery = alert.groupBy |
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.
move this to buildLogEventSlackMessage
count(*) as data, | ||
toUnixTimestamp(toStartOfInterval(timestamp, INTERVAL ?)) as ts_bucket |
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.
to be consistent with the output of getLogsChart
. eventually this method will be replaced by it
const mrkdwn = [ | ||
`*<${buildChartLink({ | ||
dashboardId: dashboard.id, | ||
endTime, | ||
granularity, | ||
startTime, | ||
})} | Alert for "${dashboard.chart.name}" in "${dashboard.name}">*`, | ||
`${totalCount} lines found, expected ${ | ||
alert.type === 'presence' ? 'less than' : 'greater than' | ||
} ${alert.threshold} lines`, | ||
].join('\n'); |
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.
Lmk if we need to add more details to the alert UI
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.
🚢🚢🚢🚢!!!!
No description provided.