Skip to content

Stage-to-Main: Update browsertrix-harvester Outputs#72

Merged
cabutlermit merged 3 commits intomainfrom
stage
Oct 2, 2025
Merged

Stage-to-Main: Update browsertrix-harvester Outputs#72
cabutlermit merged 3 commits intomainfrom
stage

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 template for the Makefile outputs
  • Update the outputs for browsertrix-harvester to use the new template files

Helpful background context

This is being done as part of regular maintenance by DataEng for the browsertrix-harvester application. See

What are the relevant tickets?

Requires Database Migrations?

NO

Includes new or updated dependencies?

NO

cabutlermit and others added 3 commits October 2, 2025 14:45
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
Dev-to-Stage: Update browsertrix-harvester Outputs
@cabutlermit cabutlermit marked this pull request as ready for review October 2, 2025 19:43
@cabutlermit cabutlermit requested a review from a team as a code owner October 2, 2025 19:43
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 7c88fdd into main 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