Skip to content

fix: increase memory of stac-browser BucketDeployment - #288

Merged
jjfrench merged 3 commits into
mainfrom
fix/stac-browser-deployment
Aug 7, 2026
Merged

fix: increase memory of stac-browser BucketDeployment#288
jjfrench merged 3 commits into
mainfrom
fix/stac-browser-deployment

Conversation

@jjfrench

@jjfrench jjfrench commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

⚠️ Checklist if your PR is changing anything else than documentation

  • Posted the link to a successful manually triggered deployment workflow (successful including the resources destruction)

Merge request description

  • Increase the memory limits of the stac-browser BucketDeployment. Initial deployment shows a timeout with near max memory usage

    Duration: 900000.00 ms Billed Duration: 900000 ms Memory Size: 128 MB Max Memory Used: 126 MB Status: timeout

  • Added stac-browser to integration build test
  • Added auto_delete_objects for stac-browser s3 bucket

@jjfrench
jjfrench requested a review from hrodmn August 6, 2026 05:01
@jjfrench jjfrench self-assigned this Aug 6, 2026
@jjfrench
jjfrench marked this pull request as ready for review August 6, 2026 13:18

@hrodmn hrodmn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's add a STAC Browser construct to the integration test app so we test the deployment on each PR!

@jjfrench

jjfrench commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

https://github.com/developmentseed/eoapi-cdk/actions/runs/31118723461/job/92717439125

Will have to take a closer look into the initial deployment failures in the health check, but it seems to have passed this time

@jjfrench

jjfrench commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

Not entirely sure if the memory issue is reproducible exactly, when we're tearing down the stack each time, but on fresh deployments, the stac-browser deploys successfully in little to no time.

I've also added the option for auto_delete_objects on the stac-browser bucket - great for our ephemeral stack and user test environments where they may want to delete and redeploy stacks without manually going to empty and delete the bucket.

@jjfrench
jjfrench requested a review from hrodmn August 7, 2026 00:16
@jjfrench
jjfrench merged commit 55f0259 into main Aug 7, 2026
6 of 16 checks passed
@jjfrench
jjfrench deleted the fix/stac-browser-deployment branch August 7, 2026 14:07
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