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

Enhance per bucket, and per document monitor notification message ctx. #605

Closed
wants to merge 8 commits into from
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,14 @@
return builder
}

fun asTemplateArg(): Map<String, Any?> {
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
return mapOf(
PARENTS_BUCKET_PATH to parentBucketPath,
BUCKET_KEYS to bucketKeys,
BUCKET to bucket

Check warning on line 50 in src/main/kotlin/org/opensearch/commons/alerting/model/AggregationResultBucket.kt

View check run for this annotation

Codecov / codecov/patch

src/main/kotlin/org/opensearch/commons/alerting/model/AggregationResultBucket.kt#L47-L50

Added lines #L47 - L50 were not covered by tests
)
}

companion object {
const val CONFIG_NAME = "agg_alert_content"
const val PARENTS_BUCKET_PATH = "parent_bucket_path"
Expand Down
101 changes: 99 additions & 2 deletions src/main/kotlin/org/opensearch/commons/alerting/model/Alert.kt
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import org.opensearch.core.xcontent.XContentParserUtils.ensureExpectedToken
import java.io.IOException
import java.time.Instant

data class Alert(
open class Alert(
val id: String = NO_ID,
val version: Long = NO_VERSION,
val schemaVersion: Int = NO_SCHEMA_VERSION,
Expand Down Expand Up @@ -591,7 +591,7 @@ data class Alert(
return builder
}

fun asTemplateArg(): Map<String, Any?> {
open fun asTemplateArg(): Map<String, Any?> {
return mapOf(
ACKNOWLEDGED_TIME_FIELD to acknowledgedTime?.toEpochMilli(),
ALERT_ID_FIELD to id,
Expand All @@ -614,4 +614,101 @@ data class Alert(
CLUSTERS_FIELD to clusters?.joinToString(",")
)
}

/**
* Creates a copy of an [Alert] with optionally modified properties
*/
fun copy(
id: String = this.id,
version: Long = this.version,
schemaVersion: Int = this.schemaVersion,
monitorId: String = this.monitorId,
workflowId: String = this.workflowId,
workflowName: String = this.workflowName,
monitorName: String = this.monitorName,
monitorVersion: Long = this.monitorVersion,
monitorUser: User? = this.monitorUser,
triggerId: String = this.triggerId,
triggerName: String = this.triggerName,
findingIds: List<String> = this.findingIds,
relatedDocIds: List<String> = this.relatedDocIds,
state: State = this.state,
startTime: Instant = this.startTime,
endTime: Instant? = this.endTime,
lastNotificationTime: Instant? = this.lastNotificationTime,
acknowledgedTime: Instant? = this.acknowledgedTime,
errorMessage: String? = this.errorMessage,
errorHistory: List<AlertError> = this.errorHistory,
severity: String = this.severity,
actionExecutionResults: List<ActionExecutionResult> = this.actionExecutionResults,
aggregationResultBucket: AggregationResultBucket? = this.aggregationResultBucket,
executionId: String? = this.executionId,
associatedAlertIds: List<String> = this.associatedAlertIds,
clusters: List<String>? = this.clusters
): Alert {
return Alert(
id = id,
version = version,
schemaVersion = schemaVersion,
monitorId = monitorId,
workflowId = workflowId,
workflowName = workflowName,
monitorName = monitorName,
monitorVersion = monitorVersion,
monitorUser = monitorUser,
triggerId = triggerId,
triggerName = triggerName,
findingIds = findingIds,
relatedDocIds = relatedDocIds,
state = state,
startTime = startTime,
endTime = endTime,
lastNotificationTime = lastNotificationTime,
acknowledgedTime = acknowledgedTime,
errorMessage = errorMessage,
errorHistory = errorHistory,
severity = severity,
actionExecutionResults = actionExecutionResults,
aggregationResultBucket = aggregationResultBucket,
executionId = executionId,
associatedAlertIds = associatedAlertIds,
clusters = clusters
)
}

override fun equals(other: Any?): Boolean {
if (this === other) return true
if (javaClass != other?.javaClass) return false

other as Alert

if (id != other.id) return false
if (version != other.version) return false
if (schemaVersion != other.schemaVersion) return false
if (monitorId != other.monitorId) return false
if (workflowId != other.workflowId) return false
if (workflowName != other.workflowName) return false
if (monitorName != other.monitorName) return false
if (monitorVersion != other.monitorVersion) return false
if (monitorUser != other.monitorUser) return false
if (triggerId != other.triggerId) return false
if (triggerName != other.triggerName) return false
if (findingIds != other.findingIds) return false
if (relatedDocIds != other.relatedDocIds) return false
if (state != other.state) return false
if (startTime != other.startTime) return false
if (endTime != other.endTime) return false
if (lastNotificationTime != other.lastNotificationTime) return false
if (acknowledgedTime != other.acknowledgedTime) return false
if (errorMessage != other.errorMessage) return false
if (errorHistory != other.errorHistory) return false
if (severity != other.severity) return false
if (actionExecutionResults != other.actionExecutionResults) return false
if (aggregationResultBucket != other.aggregationResultBucket) return false
if (executionId != other.executionId) return false
if (associatedAlertIds != other.associatedAlertIds) return false
if (clusters != other.clusters) return false

return true
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,74 @@
/*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/

package org.opensearch.commons.alerting.model

/**
* This model is a wrapper for [Alert] that should only be used to create a more
* informative alert object to enrich mustache template notification messages.
*/
data class AlertContext(
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
val alert: Alert,
val associatedQueries: List<DocLevelQuery>? = null,
val sampleDocs: List<Map<String, Any?>>? = null
) : Alert(
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
id = alert.id,
version = alert.version,
schemaVersion = alert.schemaVersion,
monitorId = alert.monitorId,
monitorName = alert.monitorName,
monitorVersion = alert.monitorVersion,
monitorUser = alert.monitorUser,
triggerId = alert.triggerId,
triggerName = alert.triggerName,
state = alert.state,
startTime = alert.startTime,
endTime = alert.endTime,
lastNotificationTime = alert.lastNotificationTime,
acknowledgedTime = alert.acknowledgedTime,
errorMessage = alert.errorMessage,
errorHistory = alert.errorHistory,
severity = alert.severity,
actionExecutionResults = alert.actionExecutionResults,
aggregationResultBucket = alert.aggregationResultBucket,
findingIds = alert.findingIds,
relatedDocIds = alert.relatedDocIds,
executionId = alert.executionId,
workflowId = alert.workflowId,
workflowName = alert.workflowName,
associatedAlertIds = alert.associatedAlertIds,
clusters = alert.clusters
) {

override fun asTemplateArg(): Map<String, Any?> {
val queriesContext = associatedQueries?.map {
mapOf(
DocLevelQuery.QUERY_ID_FIELD to it.id,
DocLevelQuery.NAME_FIELD to it.name,
DocLevelQuery.TAGS_FIELD to it.tags
)
}

// Compile the custom context fields.
val customContextFields = mapOf(
ASSOCIATED_QUERIES_FIELD to queriesContext,
SAMPLE_DOCS_FIELD to sampleDocs
)

// Get the alert template args
val templateArgs = super.asTemplateArg().toMutableMap()

// Add the non-null custom context fields to the alert templateArgs.
customContextFields.forEach { (key, value) ->
if (value !== null) templateArgs[key] = value
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
}
return templateArgs
}

companion object {
const val ASSOCIATED_QUERIES_FIELD = "associated_queries"
const val SAMPLE_DOCS_FIELD = "sample_documents"
}
}
20 changes: 20 additions & 0 deletions src/test/kotlin/org/opensearch/commons/alerting/TestHelpers.kt
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ import org.opensearch.commons.alerting.aggregation.bucketselectorext.BucketSelec
import org.opensearch.commons.alerting.model.ActionExecutionResult
import org.opensearch.commons.alerting.model.AggregationResultBucket
import org.opensearch.commons.alerting.model.Alert
import org.opensearch.commons.alerting.model.AlertContext
import org.opensearch.commons.alerting.model.BucketLevelTrigger
import org.opensearch.commons.alerting.model.ChainedAlertTrigger
import org.opensearch.commons.alerting.model.ChainedMonitorFindings
Expand Down Expand Up @@ -601,3 +602,22 @@ fun randomFinding(
timestamp = timestamp
)
}

fun randomAlertContext(
alert: Alert = randomAlert(),
associatedQueries: List<DocLevelQuery>? = (-1..2).random().takeIf { it != -1 }?.let {
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
(0..it).map { randomDocLevelQuery() }
},
sampleDocs: List<Map<String, Any?>>? = (-1..2).random().takeIf { it != -1 }?.let {
(0..it).map {
// Using 'randomFinding' to mimic documents in an index.
randomFinding().asTemplateArg()
}
}
): AlertContext {
return AlertContext(
alert = alert,
associatedQueries = associatedQueries,
sampleDocs = sampleDocs
)
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
/*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/

package org.opensearch.commons.alerting.model

import org.junit.jupiter.api.Assertions.assertEquals
import org.junit.jupiter.api.BeforeEach
import org.junit.jupiter.api.Test
import org.opensearch.commons.alerting.randomAlertContext

class AlertContextTests {
private var alertContext: AlertContext = randomAlertContext()

@BeforeEach
fun generateRandomData() {
alertContext = randomAlertContext()
AWSHurneyt marked this conversation as resolved.
Show resolved Hide resolved
}

@Test
fun `test AlertContext asTemplateArg`() {
val templateArgs = alertContext.asTemplateArg()

assertEquals(templateArgs[Alert.ALERT_ID_FIELD], alertContext.id, "Template args id does not match")
assertEquals(templateArgs[Alert.ALERT_VERSION_FIELD], alertContext.version, "Template args version does not match")
assertEquals(templateArgs[Alert.STATE_FIELD], alertContext.state.toString(), "Template args state does not match")
assertEquals(templateArgs[Alert.ERROR_MESSAGE_FIELD], alertContext.errorMessage, "Template args error message does not match")
assertEquals(templateArgs[Alert.ACKNOWLEDGED_TIME_FIELD], null, "Template args acknowledged time does not match")
assertEquals(templateArgs[Alert.END_TIME_FIELD], alertContext.endTime?.toEpochMilli(), "Template args end time does not")
assertEquals(templateArgs[Alert.START_TIME_FIELD], alertContext.startTime.toEpochMilli(), "Template args start time does not")
assertEquals(templateArgs[Alert.LAST_NOTIFICATION_TIME_FIELD], null, "Template args last notification time does not match")
assertEquals(templateArgs[Alert.SEVERITY_FIELD], alertContext.severity, "Template args severity does not match")
assertEquals(templateArgs[Alert.CLUSTERS_FIELD], alertContext.clusters?.joinToString(","), "Template args clusters does not match")
val formattedQueries = alertContext.associatedQueries?.map {
mapOf(
DocLevelQuery.QUERY_ID_FIELD to it.id,
DocLevelQuery.NAME_FIELD to it.name,
DocLevelQuery.TAGS_FIELD to it.tags
)
}
assertEquals(templateArgs[AlertContext.ASSOCIATED_QUERIES_FIELD], formattedQueries, "Template associated queries do not match")
assertEquals(templateArgs[AlertContext.SAMPLE_DOCS_FIELD], alertContext.sampleDocs, "Template args sample docs do not match")
}
}
Loading