Skip to content

Conversation

peternied
Copy link
Member

Description

We have received reports that the solutions project AMI need to be updated due to security issues that have been discovered in the older AMI images the bootstrap process uses. I've updated all images to the latest release, including gov regions.

I have also changed the source for the images, previous we would use the first image returned by searching the EC2 AMI catalog, instead we are calling the Amazon managed parameter that holds these details for all regions.

I have also updated the script to detect if there are missing regions that need to be opted into since we saw a new region arrive "ap-east-2", this should prevent folks that run this script in the future from missing any existing or new regions.

Finally I've also sorted the generated list, which should make it clearer to see any updates in the future.

Testing

Tested deployment in gov region, no issues.

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.

We've recieved reports that the solutions project AMI need to be updatd
due to some security issues that have been discovered.  I've updated all
images to the latest release, including gov regions.

I've also changed the source for the images, previous we would use the
first image returned by searching the AMI catalog, instead we are
calling the Amazon managed parameter that holds these details for all
regions.

I've also updated the script to detect if there are missing regions that
need to be opted into since we saw a new region arrive "ap-east-2", this
should prevent folks that run this script in the future from missing any
existing or new regions.

Finally I've also sorted the generated list, which should make it
clearer to see any updates in the future.

Signed-off-by: Peter Nied <[email protected]>
Signed-off-by: Peter Nied <[email protected]>
Copy link
Collaborator

@jugal-chauhan jugal-chauhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for these changes

@peternied peternied merged commit ac1150f into opensearch-project:main Oct 7, 2025
57 of 58 checks passed
@peternied peternied deleted the refresh-solution-amis branch October 7, 2025 18:31
Copy link

codecov bot commented Oct 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (7d8d015) to head (9c25e60).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff      @@
##   main   #1871   +/-   ##
============================
============================

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants