Skip to content

[Add] deployment for MA dashboards on EKS #1879

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 30 commits into from
Oct 30, 2025
Merged

[Add] deployment for MA dashboards on EKS #1879

merged 30 commits into from
Oct 30, 2025

Conversation

@jugal-chauhan
Copy link
Collaborator

@jugal-chauhan jugal-chauhan commented Oct 10, 2025 •
edited
Loading

Description

This PR adds automatic CloudWatch dashboard deployment to the EKS Migration Assistant deployment process via the aws-bootstrap.sh script. Both the Dashboards for monitoring MA (RFS + CaptureReplay) will now be available for a user in their CloudWatch after EKS deployment approach.

Changes made:

  • Added deploy_dashboard() function to aws-bootstrap.sh script
  • Moved dashboard JSON files from the cdk/ directory into the k8s/ directory
  • Updated file paths to now reference k8s location of these JSON files

Issues Resolved

Part 1/2 of MIGRATIONS-2691

Testing

Performed a manual deployment to verify installation of dashboards

Check List

  • New functionality includes testing
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@jugal-chauhan jugal-chauhan changed the title Install dashboards ma eks [Add] deployment for MA dashboards on EKS Oct 10, 2025
@jugal-chauhan jugal-chauhan marked this pull request as ready for review October 14, 2025 16:42
gregschohn
gregschohn reviewed Oct 16, 2025
View reviewed changes
AndreKurait
AndreKurait reviewed Oct 23, 2025
View reviewed changes
AndreKurait
AndreKurait reviewed Oct 23, 2025
View reviewed changes
AndreKurait
AndreKurait reviewed Oct 23, 2025
View reviewed changes
@jugal-chauhan
Merge commit changes
6bd5399 
Signed-off-by: Jugal Chauhan <[email protected]>
@jugal-chauhan
Fix code comments
fbf8c4c 
Signed-off-by: Jugal Chauhan <[email protected]>
@jugal-chauhan jugal-chauhan merged commit ae00428 into opensearch-project:main Oct 30, 2025
141 of 145 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndreKurait AndreKurait AndreKurait approved these changes

@chelma chelma Awaiting requested review from chelma chelma is a code owner

@lewijacn lewijacn Awaiting requested review from lewijacn lewijacn is a code owner

@mikaylathompson mikaylathompson Awaiting requested review from mikaylathompson mikaylathompson is a code owner

@peternied peternied Awaiting requested review from peternied peternied is a code owner

@sumobrian sumobrian Awaiting requested review from sumobrian sumobrian is a code owner

@gregschohn gregschohn Awaiting requested review from gregschohn gregschohn is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jugal-chauhan @gregschohn @AndreKurait