Skip to content

Pin localstack to a community image#20967

Merged
andrross merged 1 commit intoopensearch-project:mainfrom
andrross:localstack-fix
Mar 23, 2026
Merged

Pin localstack to a community image#20967
andrross merged 1 commit intoopensearch-project:mainfrom
andrross:localstack-fix

Conversation

@andrross
Copy link
Member

Localstack has changed to require authenticated access. See this blog post for details. In the short term this commit will pin our usage to the last community release that supported unauthenticated access.

Check List

  • Functionality includes testing.

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.

Localstack has changed to require authenticated access. See [this blog
post][1] for details. In the short term this commit will pin our usage
to the last community release that supported unauthenticated access.

[1]: https://blog.localstack.cloud/localstack-single-image-next-steps/

Signed-off-by: Andrew Ross <andrross@amazon.com>
@andrross andrross requested a review from a team as a code owner March 23, 2026 16:29
@github-actions
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 PR contains tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ Recommended focus areas for review

Pinned Version

The image is pinned to localstack/localstack:4.14.0. This is intentional as a short-term fix, but this specific version will not receive security patches or bug fixes. There should be a plan to migrate to the authenticated community image to avoid accumulating technical debt.

localstack = new LocalStackContainer(DockerImageName.parse("localstack/localstack:4.14.0")).withEnv(

@andrross
Copy link
Member Author

FYI @msfroh @varunbharadwaj

Copy link
Member

@cwperks cwperks 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 fixing this. I ran into this earlier this morning.

@reta
Copy link
Contributor

reta commented Mar 23, 2026

Thanks @andrross !

@github-actions
Copy link
Contributor

❌ Gradle check result for 17adf19: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

✅ Gradle check result for 17adf19: SUCCESS

@codecov
Copy link

codecov bot commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.23%. Comparing base (fb5d661) to head (17adf19).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #20967      +/-   ##
============================================
- Coverage     73.24%   73.23%   -0.01%     
+ Complexity    72510    72501       -9     
============================================
  Files          5819     5819              
  Lines        331373   331373              
  Branches      47882    47882              
============================================
- Hits         242707   242692      -15     
+ Misses        69201    69145      -56     
- Partials      19465    19536      +71     

☔ 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.

@andrross andrross merged commit 0537405 into opensearch-project:main Mar 23, 2026
42 of 45 checks passed
@andrross andrross deleted the localstack-fix branch March 23, 2026 19:19
@reta reta added the backport 3.5 Backport to 3.5 branch label Mar 23, 2026
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 23, 2026
Localstack has changed to require authenticated access. See [this blog
post][1] for details. In the short term this commit will pin our usage
to the last community release that supported unauthenticated access.

[1]: https://blog.localstack.cloud/localstack-single-image-next-steps/

Signed-off-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 0537405)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 3.5 Backport to 3.5 branch skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants