cdklabs / cdk-monitoring-constructs

Easy-to-use CDK constructs for monitoring your AWS infrastructure
https://constructs.dev/packages/cdk-monitoring-constructs
Apache License 2.0
480 stars 70 forks source link

[opensearchserverless] Add support for monitoring OpenSearch Serverless #335

Open ajay340 opened 1 year ago

ajay340 commented 1 year ago

Feature scope

Opensearch Serverless

Describe your suggested feature

cdk-monitoring-constructs currently supports monitoring Opensearch and Elasticsearch clusters, but not Opensearch serverless. I am requesting you add support to monitor an Opensearch serverless.

dhanushuUzumaki commented 3 weeks ago

Hi, I'd like to work on this. On a high level, I am planning to add monitorOpenSearchServerlessCollection to facade. With following metrics and alarm support on these metrics. Since OCU is on account level, I am also considering having a boolean param to specify whether to include OCU metrics as part of this segment instance.

echeung-amzn commented 3 weeks ago

Since OCU is on account level, I am also considering having a boolean param to specify whether to include OCU metrics as part of this segment instance.

It'd probably make more sense to separate it such that there's monitorOpenSearchServerlessCollection that monitors a collection and something like monitorOpenSearchServerless that monitors higher-level metrics that aren't at a collection/instance level.

dhanushuUzumaki commented 3 weeks ago

That makes sense. I will start with monitorOpenSearchServerless for metrics on account level. IndexingOCU and SearchOCU