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

[opensearchserverless] Add support for monitoring OpenSearch Serverless #335

Open
ajay340 opened this issue Mar 1, 2023 · 3 comments
Open
Labels
feature-request New feature

Comments

@ajay340
Copy link

ajay340 commented Mar 1, 2023

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.

@ajay340 ajay340 added the feature-request New feature label Mar 1, 2023
@echeung-amzn echeung-amzn changed the title Add support for monitoring Opensearch Serverless [opensearchserverless] Add support for monitoring OpenSearch Serverless Apr 4, 2023
@dhanushuUzumaki
Copy link

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.

  • IngestionDocumentErrors
  • IngestionRequestErrors
  • IngestionRequestLatency
  • SearchRequestErrors
  • SearchRequestLatency
  • StorageUsedInS3
  • IndexingOCU
  • SearchOCU

@echeung-amzn
Copy link
Member

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
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature
Projects
None yet
Development

No branches or pull requests

3 participants