-
Notifications
You must be signed in to change notification settings - Fork 94
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Backport 2.x] Add support for remote monitors (#694)
* changes to add support for remote monitors in alerting (#661) Signed-off-by: Subhobrata Dey <[email protected]> * changes to add support for remote monitors in alerting (#662) * changes to add support for remote monitors in alerting Signed-off-by: Subhobrata Dey <[email protected]> * add tests for moved classes Signed-off-by: Subhobrata Dey <[email protected]> --------- Signed-off-by: Subhobrata Dey <[email protected]> * changes to support generic inputs and triggers in remote monitors (#664) Signed-off-by: Subhobrata Dey <[email protected]> * add remote doc level monitor input (#665) Signed-off-by: Subhobrata Dey <[email protected]> * fix serde of RemoteDocLevelMonitorInput (#666) Signed-off-by: Subhobrata Dey <[email protected]> * fix serde for monitor (#692) Signed-off-by: Subhobrata Dey <[email protected]> --------- Signed-off-by: Subhobrata Dey <[email protected]> Co-authored-by: Subhobrata Dey <[email protected]>
- Loading branch information
1 parent
413ae0f
commit fbb297b
Showing
30 changed files
with
2,413 additions
and
26 deletions.
There are no files selected for viewing
15 changes: 15 additions & 0 deletions
15
src/main/kotlin/org/opensearch/commons/alerting/action/DocLevelMonitorFanOutAction.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
/* | ||
* Copyright OpenSearch Contributors | ||
* SPDX-License-Identifier: Apache-2.0 | ||
*/ | ||
|
||
package org.opensearch.commons.alerting.action | ||
|
||
import org.opensearch.action.ActionType | ||
|
||
class DocLevelMonitorFanOutAction private constructor() : ActionType<DocLevelMonitorFanOutResponse>(NAME, ::DocLevelMonitorFanOutResponse) { | ||
companion object { | ||
val INSTANCE = DocLevelMonitorFanOutAction() | ||
const val NAME = "cluster:admin/opensearch/alerting/monitor/doclevel/fanout" | ||
} | ||
} |
101 changes: 101 additions & 0 deletions
101
src/main/kotlin/org/opensearch/commons/alerting/action/DocLevelMonitorFanOutRequest.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,101 @@ | ||
/* | ||
* Copyright OpenSearch Contributors | ||
* SPDX-License-Identifier: Apache-2.0 | ||
*/ | ||
|
||
package org.opensearch.commons.alerting.action | ||
|
||
import org.opensearch.action.ActionRequest | ||
import org.opensearch.action.ActionRequestValidationException | ||
import org.opensearch.commons.alerting.model.IndexExecutionContext | ||
import org.opensearch.commons.alerting.model.Monitor | ||
import org.opensearch.commons.alerting.model.MonitorMetadata | ||
import org.opensearch.commons.alerting.model.WorkflowRunContext | ||
import org.opensearch.core.common.io.stream.StreamInput | ||
import org.opensearch.core.common.io.stream.StreamOutput | ||
import org.opensearch.core.index.shard.ShardId | ||
import org.opensearch.core.xcontent.ToXContent | ||
import org.opensearch.core.xcontent.ToXContentObject | ||
import org.opensearch.core.xcontent.XContentBuilder | ||
import java.io.IOException | ||
|
||
class DocLevelMonitorFanOutRequest : ActionRequest, ToXContentObject { | ||
val monitor: Monitor | ||
val dryRun: Boolean | ||
val monitorMetadata: MonitorMetadata | ||
val executionId: String | ||
val indexExecutionContext: IndexExecutionContext? | ||
val shardIds: List<ShardId> | ||
val concreteIndicesSeenSoFar: List<String> | ||
val workflowRunContext: WorkflowRunContext? | ||
|
||
constructor( | ||
monitor: Monitor, | ||
dryRun: Boolean, | ||
monitorMetadata: MonitorMetadata, | ||
executionId: String, | ||
indexExecutionContext: IndexExecutionContext?, | ||
shardIds: List<ShardId>, | ||
concreteIndicesSeenSoFar: List<String>, | ||
workflowRunContext: WorkflowRunContext? | ||
) : super() { | ||
this.monitor = monitor | ||
this.dryRun = dryRun | ||
this.monitorMetadata = monitorMetadata | ||
this.executionId = executionId | ||
this.indexExecutionContext = indexExecutionContext | ||
this.shardIds = shardIds | ||
this.concreteIndicesSeenSoFar = concreteIndicesSeenSoFar | ||
this.workflowRunContext = workflowRunContext | ||
require(false == shardIds.isEmpty()) { } | ||
} | ||
|
||
@Throws(IOException::class) | ||
constructor(sin: StreamInput) : this( | ||
monitor = Monitor.readFrom(sin)!!, | ||
dryRun = sin.readBoolean(), | ||
monitorMetadata = MonitorMetadata.readFrom(sin), | ||
executionId = sin.readString(), | ||
shardIds = sin.readList(::ShardId), | ||
concreteIndicesSeenSoFar = sin.readStringList(), | ||
workflowRunContext = if (sin.readBoolean()) { | ||
WorkflowRunContext(sin) | ||
} else { null }, | ||
indexExecutionContext = IndexExecutionContext(sin) | ||
) | ||
|
||
@Throws(IOException::class) | ||
override fun writeTo(out: StreamOutput) { | ||
monitor.writeTo(out) | ||
out.writeBoolean(dryRun) | ||
monitorMetadata.writeTo(out) | ||
out.writeString(executionId) | ||
out.writeCollection(shardIds) | ||
out.writeStringCollection(concreteIndicesSeenSoFar) | ||
out.writeBoolean(workflowRunContext != null) | ||
workflowRunContext?.writeTo(out) | ||
indexExecutionContext?.writeTo(out) | ||
} | ||
|
||
override fun validate(): ActionRequestValidationException? { | ||
var actionValidationException: ActionRequestValidationException? = null | ||
if (shardIds.isEmpty()) { | ||
actionValidationException = ActionRequestValidationException() | ||
actionValidationException.addValidationError("shard_ids is null or empty") | ||
} | ||
return actionValidationException | ||
} | ||
|
||
@Throws(IOException::class) | ||
override fun toXContent(builder: XContentBuilder, params: ToXContent.Params): XContentBuilder { | ||
builder.startObject() | ||
.field("monitor", monitor) | ||
.field("dry_run", dryRun) | ||
.field("execution_id", executionId) | ||
.field("index_execution_context", indexExecutionContext) | ||
.field("shard_ids", shardIds) | ||
.field("concrete_indices", concreteIndicesSeenSoFar) | ||
.field("workflow_run_context", workflowRunContext) | ||
return builder.endObject() | ||
} | ||
} |
92 changes: 92 additions & 0 deletions
92
src/main/kotlin/org/opensearch/commons/alerting/action/DocLevelMonitorFanOutResponse.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,92 @@ | ||
/* | ||
* Copyright OpenSearch Contributors | ||
* SPDX-License-Identifier: Apache-2.0 | ||
*/ | ||
|
||
package org.opensearch.commons.alerting.action | ||
|
||
import org.opensearch.commons.alerting.model.DocumentLevelTriggerRunResult | ||
import org.opensearch.commons.alerting.model.InputRunResults | ||
import org.opensearch.commons.alerting.util.AlertingException | ||
import org.opensearch.core.action.ActionResponse | ||
import org.opensearch.core.common.io.stream.StreamInput | ||
import org.opensearch.core.common.io.stream.StreamOutput | ||
import org.opensearch.core.xcontent.ToXContent | ||
import org.opensearch.core.xcontent.ToXContentObject | ||
import org.opensearch.core.xcontent.XContentBuilder | ||
import java.io.IOException | ||
|
||
class DocLevelMonitorFanOutResponse : ActionResponse, ToXContentObject { | ||
val nodeId: String | ||
val executionId: String | ||
val monitorId: String | ||
val lastRunContexts: MutableMap<String, Any> | ||
val inputResults: InputRunResults | ||
val triggerResults: Map<String, DocumentLevelTriggerRunResult> | ||
val exception: AlertingException? | ||
|
||
@Throws(IOException::class) | ||
constructor(sin: StreamInput) : this( | ||
nodeId = sin.readString(), | ||
executionId = sin.readString(), | ||
monitorId = sin.readString(), | ||
lastRunContexts = sin.readMap()!! as MutableMap<String, Any>, | ||
inputResults = InputRunResults.readFrom(sin), | ||
triggerResults = suppressWarning(sin.readMap(StreamInput::readString, DocumentLevelTriggerRunResult::readFrom)), | ||
exception = sin.readException() | ||
) | ||
|
||
constructor( | ||
nodeId: String, | ||
executionId: String, | ||
monitorId: String, | ||
lastRunContexts: MutableMap<String, Any>, | ||
inputResults: InputRunResults = InputRunResults(), // partial, | ||
triggerResults: Map<String, DocumentLevelTriggerRunResult> = mapOf(), | ||
exception: AlertingException? = null | ||
) : super() { | ||
this.nodeId = nodeId | ||
this.executionId = executionId | ||
this.monitorId = monitorId | ||
this.lastRunContexts = lastRunContexts | ||
this.inputResults = inputResults | ||
this.triggerResults = triggerResults | ||
this.exception = exception | ||
} | ||
|
||
@Throws(IOException::class) | ||
override fun writeTo(out: StreamOutput) { | ||
out.writeString(nodeId) | ||
out.writeString(executionId) | ||
out.writeString(monitorId) | ||
out.writeMap(lastRunContexts) | ||
inputResults.writeTo(out) | ||
out.writeMap( | ||
triggerResults, | ||
StreamOutput::writeString, | ||
{ stream, stats -> stats.writeTo(stream) } | ||
) | ||
out.writeException(exception) | ||
} | ||
|
||
@Throws(IOException::class) | ||
override fun toXContent(builder: XContentBuilder, params: ToXContent.Params): XContentBuilder { | ||
builder.startObject() | ||
.field("node_id", nodeId) | ||
.field("execution_id", executionId) | ||
.field("monitor_id", monitorId) | ||
.field("last_run_contexts", lastRunContexts) | ||
.field("input_results", inputResults) | ||
.field("trigger_results", triggerResults) | ||
.field("exception", exception) | ||
.endObject() | ||
return builder | ||
} | ||
|
||
companion object { | ||
@Suppress("UNCHECKED_CAST") | ||
fun suppressWarning(map: MutableMap<String?, Any?>?): Map<String, DocumentLevelTriggerRunResult> { | ||
return map as Map<String, DocumentLevelTriggerRunResult> | ||
} | ||
} | ||
} |
57 changes: 57 additions & 0 deletions
57
src/main/kotlin/org/opensearch/commons/alerting/model/BucketLevelTriggerRunResult.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,57 @@ | ||
/* | ||
* Copyright OpenSearch Contributors | ||
* SPDX-License-Identifier: Apache-2.0 | ||
*/ | ||
|
||
package org.opensearch.commons.alerting.model | ||
|
||
import org.opensearch.core.common.io.stream.StreamInput | ||
import org.opensearch.core.common.io.stream.StreamOutput | ||
import org.opensearch.core.xcontent.ToXContent | ||
import org.opensearch.core.xcontent.XContentBuilder | ||
import java.io.IOException | ||
|
||
data class BucketLevelTriggerRunResult( | ||
override var triggerName: String, | ||
override var error: Exception? = null, | ||
var aggregationResultBuckets: Map<String, AggregationResultBucket>, | ||
var actionResultsMap: MutableMap<String, MutableMap<String, ActionRunResult>> = mutableMapOf() | ||
) : TriggerRunResult(triggerName, error) { | ||
|
||
@Throws(IOException::class) | ||
@Suppress("UNCHECKED_CAST") | ||
constructor(sin: StreamInput) : this( | ||
sin.readString(), | ||
sin.readException() as Exception?, // error | ||
sin.readMap(StreamInput::readString, ::AggregationResultBucket), | ||
sin.readMap() as MutableMap<String, MutableMap<String, ActionRunResult>> | ||
) | ||
|
||
override fun internalXContent(builder: XContentBuilder, params: ToXContent.Params): XContentBuilder { | ||
return builder | ||
.field(AGG_RESULT_BUCKETS, aggregationResultBuckets) | ||
.field(ACTIONS_RESULTS, actionResultsMap as Map<String, Any>) | ||
} | ||
|
||
@Throws(IOException::class) | ||
@Suppress("UNCHECKED_CAST") | ||
override fun writeTo(out: StreamOutput) { | ||
super.writeTo(out) | ||
out.writeMap(aggregationResultBuckets, StreamOutput::writeString) { | ||
valueOut: StreamOutput, aggResultBucket: AggregationResultBucket -> | ||
aggResultBucket.writeTo(valueOut) | ||
} | ||
out.writeMap(actionResultsMap as Map<String, Any>) | ||
} | ||
|
||
companion object { | ||
const val AGG_RESULT_BUCKETS = "agg_result_buckets" | ||
const val ACTIONS_RESULTS = "action_results" | ||
|
||
@JvmStatic | ||
@Throws(IOException::class) | ||
fun readFrom(sin: StreamInput): TriggerRunResult { | ||
return BucketLevelTriggerRunResult(sin) | ||
} | ||
} | ||
} |
69 changes: 69 additions & 0 deletions
69
src/main/kotlin/org/opensearch/commons/alerting/model/ChainedAlertTriggerRunResult.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,69 @@ | ||
/* | ||
* Copyright OpenSearch Contributors | ||
* SPDX-License-Identifier: Apache-2.0 | ||
*/ | ||
|
||
package org.opensearch.commons.alerting.model | ||
|
||
import org.opensearch.commons.alerting.alerts.AlertError | ||
import org.opensearch.core.common.io.stream.StreamInput | ||
import org.opensearch.core.common.io.stream.StreamOutput | ||
import org.opensearch.core.xcontent.ToXContent | ||
import org.opensearch.core.xcontent.XContentBuilder | ||
import org.opensearch.script.ScriptException | ||
import java.io.IOException | ||
import java.time.Instant | ||
|
||
data class ChainedAlertTriggerRunResult( | ||
override var triggerName: String, | ||
var triggered: Boolean, | ||
override var error: Exception?, | ||
var actionResults: MutableMap<String, ActionRunResult> = mutableMapOf(), | ||
val associatedAlertIds: Set<String> | ||
) : TriggerRunResult(triggerName, error) { | ||
|
||
@Throws(IOException::class) | ||
@Suppress("UNCHECKED_CAST") | ||
constructor(sin: StreamInput) : this( | ||
triggerName = sin.readString(), | ||
error = sin.readException(), | ||
triggered = sin.readBoolean(), | ||
actionResults = sin.readMap() as MutableMap<String, ActionRunResult>, | ||
associatedAlertIds = sin.readStringList().toSet() | ||
) | ||
|
||
override fun alertError(): AlertError? { | ||
if (error != null) { | ||
return AlertError(Instant.now(), "Failed evaluating trigger:\n${error!!.userErrorMessage()}") | ||
} | ||
for (actionResult in actionResults.values) { | ||
if (actionResult.error != null) { | ||
return AlertError(Instant.now(), "Failed running action:\n${actionResult.error.userErrorMessage()}") | ||
} | ||
} | ||
return null | ||
} | ||
|
||
override fun internalXContent(builder: XContentBuilder, params: ToXContent.Params): XContentBuilder { | ||
if (error is ScriptException) error = Exception((error as ScriptException).toJsonString(), error) | ||
return builder | ||
.field("triggered", triggered) | ||
.field("action_results", actionResults as Map<String, ActionRunResult>) | ||
} | ||
|
||
@Throws(IOException::class) | ||
override fun writeTo(out: StreamOutput) { | ||
super.writeTo(out) | ||
out.writeBoolean(triggered) | ||
out.writeMap(actionResults as Map<String, ActionRunResult>) | ||
out.writeStringCollection(associatedAlertIds) | ||
} | ||
|
||
companion object { | ||
@JvmStatic | ||
@Throws(IOException::class) | ||
fun readFrom(sin: StreamInput): TriggerRunResult { | ||
return ChainedAlertTriggerRunResult(sin) | ||
} | ||
} | ||
} |
Oops, something went wrong.