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

Change the ec2 instance ami to AL2 #139

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

rishabh6788
Copy link
Collaborator

Description

We recently started seeing regression in our nightly benchmarks across all tracking OS versions, even for 2.17 which is GA since Sept-12.
So it couldn't be something changed on the OpenSearch side, since we use snapshot for data restore, it could not be corpus or index mapping getting changed, and I have OSB version also pinned to 1.8.0 for a few months, so it couldn't be something changed on OSB side.
After digging further I looked into underlying OS, which is AL2023 and it recently had new ami released (Oct-10).
I was looking at change logs but nothing jumped out of the ordinary. Just a few hours ago another user reported latency regression after switching to Al2023 and the root cause seems to be the OpenSSL library version. See amazonlinux/amazon-linux-2023#819 After going into the rabbit hole i found some other issues as well.
aws/aws-lambda-base-images#154
openssl/openssl#17064

This PR updates the ec2 instance ami to us AL2.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

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.

Signed-off-by: Rishabh Singh <sngri@amazon.com>
@rishabh6788 rishabh6788 merged commit b4c314d into opensearch-project:main Oct 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants