Skip to content

Dev-to-Stage: Update browsertrix-harvester Outputs#71

Merged
cabutlermit merged 2 commits intostagefrom
dev
Oct 2, 2025
Merged

Dev-to-Stage: Update browsertrix-harvester Outputs#71
cabutlermit merged 2 commits intostagefrom
dev

Conversation

@cabutlermit
Copy link
Contributor

@cabutlermit cabutlermit commented Oct 2, 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?

  • Fix the Makefile template based on lessons-learned during the update of the marimo-launcher application
  • Update the browsertrix-harvester outputs to use the new template files for the workflows and Makefile

Helpful background context

This is the second application repository upgrade, so we expect that everything will work smoothly. There are no changes to anything in AWS, just updated Terraform Cloud outputs.

What are the relevant tickets?

Requires Database Migrations?

NO

Includes new or updated dependencies?

NO

Why these changes are being introduced:
After going through the process with the marimo-launcher application,
there were a few minor lessons-learned that are reflected in this
slightly updated Makefile template.

How this addresses that need:
* Fix the tagging for containers generated by the `make publish-dev`
command

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/IN-1481
Why these changes are being introduced:
See IN-1481 and IN-1500 for details. This is the migration to the new
shared workflows.

How this addresses that need:
* Update the Terraform outputs for browsertrix-harvester to pull from
the new template files for the new shared workflows

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/IN-1500
@cabutlermit cabutlermit marked this pull request as ready for review October 2, 2025 19:04
@cabutlermit cabutlermit requested a review from a team as a code owner October 2, 2025 19:04
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 dcf84ae into stage Oct 2, 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