Skip to content

Dev-to-Stage: Add ECR Repository for marimo-launcher#65

Merged
cabutlermit merged 2 commits intostagefrom
dev
Aug 15, 2025
Merged

Dev-to-Stage: Add ECR Repository for marimo-launcher#65
cabutlermit merged 2 commits intostagefrom
dev

Conversation

@cabutlermit
Copy link
Contributor

@cabutlermit cabutlermit commented Aug 14, 2025

Developer Checklist

  • The README contains any additional info needed outside of the terraform docs generated
  • Any special variables have values configured in AWS SSM
  • Stakeholder approval has been confirmed (or is not needed)

What does this PR do?

  • Create an ECR Repository for a new container (see marimo-launcher)
  • Create all the associated outputs to enable automated deployments in dev/stage/prod
  • Update dependencies (pre-commit and AWS Provider)

Helpful background context

Graham and Christopher are working on an MVP for running marimo notebooks in AWS with on-demand "run" access for stakeholders (especially for marimo notebooks that need to access data stored in AWS).

What are the relevant tickets?

Requires Database Migrations?

NO

Includes new or updated dependencies?

NO

Why these changes are being introduced:
Routine maintenance before updating infrastructure.

How this addresses that need:
* Update third-party dependencies in pre-commit configuration
* Update AWS Provider to 5.100

Side effects of this change:
None.
Why these changes are being introduced:
MVP for running marimo notebooks in AWS as on-demand Fargate tasks.
This builds the necessary infrastructure for storing the container
image.

How this addresses that need:
* Create the ECR repository and associated resources

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/IN-1416
@cabutlermit cabutlermit marked this pull request as ready for review August 14, 2025 19:21
@cabutlermit cabutlermit requested a review from a team as a code owner August 14, 2025 19:21
Copy link
Contributor

@vab vab left a comment

Choose a reason for hiding this comment

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

lgtm

@cabutlermit cabutlermit merged commit 5859d87 into stage Aug 15, 2025
5 checks passed
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