GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Retire Elemental MediaStore by viren-nadkarni · Pull Request #13484 · localstack/localstack

This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Conversation

@viren-nadkarni

@viren-nadkarni viren-nadkarni commented Dec 9, 2025

Copy link
Copy Markdown
Member

Changes

This PR removes all code related to Elemental MediaStore.

Related

Closes: PNX-561

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@viren-nadkarni viren-nadkarni self-assigned this Dec 9, 2025
@viren-nadkarni viren-nadkarni added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Dec 9, 2025
@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

S3 Image Test Results (AMD64 / ARM64)

    2 files  ±0      2 suites  ±0   8m 1s ⏱️ -3s
  552 tests ±0    500 ✅ ±0   52 💤 ±0  0 ❌ ±0 
1 104 runs  ±0  1 000 ✅ ±0  104 💤 ±0  0 ❌ ±0 

Results for commit b2dc920. ± Comparison against base commit bca5595.

♻️ This comment has been updated with latest results.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Test Results - Preflight, Unit

22 999 tests   - 1   21 156 ✅  - 1   6m 22s ⏱️ -1s
     1 suites ±0    1 843 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit b2dc920. ± Comparison against base commit bca5595.

This pull request removes 1 test. 
tests.unit.aws.test_connect.TestClientFactory ‑ test_typed_client_creation[mediastore]

♻️ This comment has been updated with latest results.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   2m 49s ⏱️ -8s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit b2dc920. ± Comparison against base commit bca5595.

♻️ This comment has been updated with latest results.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 34m 49s ⏱️
5 521 tests 4 965 ✅ 556 💤 0 ❌
5 527 runs  4 965 ✅ 562 💤 0 ❌

Results for commit b2dc920.

♻️ This comment has been updated with latest results.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@github-actions

github-actions Bot commented Dec 9, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 57m 53s ⏱️ + 1m 59s
5 141 tests ±0  4 745 ✅ ±0  396 💤 ±0  0 ❌ ±0 
5 143 runs  ±0  4 745 ✅ ±0  398 💤 ±0  0 ❌ ±0 

Results for commit b2dc920. ± Comparison against base commit bca5595.

♻️ This comment has been updated with latest results.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@aidehn aidehn added this to the 4.13 milestone Dec 10, 2025
@viren-nadkarni
viren-nadkarni marked this pull request as ready for review December 15, 2025 07:49
@viren-nadkarni
viren-nadkarni requested review from alexrashed, bentsku, k-a-il, sannya-singal, silv-io and thrau as code owners December 15, 2025 07:49
alexrashed
alexrashed approved these changes Dec 15, 2025
View reviewed changes

@alexrashed alexrashed left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Nice! Thanks for the thorough cleanup after the companion in community has already removed the service! 💯 🧹

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@viren-nadkarni
viren-nadkarni requested a review from dominikschubert as a code owner December 15, 2025 09:30
@viren-nadkarni viren-nadkarni closed this Dec 15, 2025
@viren-nadkarni
viren-nadkarni force-pushed the retire-mediastore branch from a4439a1 to bca5595 Compare December 15, 2025 09:32
@viren-nadkarni viren-nadkarni reopened this Dec 15, 2025
@viren-nadkarni
Retire Elemental MediaStore
b2dc920
@viren-nadkarni
viren-nadkarni force-pushed the retire-mediastore branch from 431b081 to b2dc920 Compare December 15, 2025 09:35
@viren-nadkarni
viren-nadkarni merged commit 96eea38 into main Dec 16, 2025
49 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@viren-nadkarni
viren-nadkarni deleted the retire-mediastore branch December 16, 2025 04:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@thrau thrau Awaiting requested review from thrau
@silv-io silv-io Awaiting requested review from silv-io
@k-a-il k-a-il Awaiting requested review from k-a-il k-a-il is a code owner
@bentsku bentsku Awaiting requested review from bentsku
@sannya-singal sannya-singal Awaiting requested review from sannya-singal
@dominikschubert dominikschubert Awaiting requested review from dominikschubert
1 more reviewer
@alexrashed alexrashed alexrashed approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@viren-nadkarni viren-nadkarni

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

4.13

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@viren-nadkarni @alexrashed @aidehn
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.