Skip to content

Improve ECR Workflows#75

Merged
cabutlermit merged 2 commits intomainfrom
in-1732-update-deploy-workflows
Apr 7, 2026
Merged

Improve ECR Workflows#75
cabutlermit merged 2 commits intomainfrom
in-1732-update-deploy-workflows

Conversation

@cabutlermit
Copy link
Copy Markdown
Contributor

@cabutlermit cabutlermit commented Apr 6, 2026

Why these changes are being introduced

The work on TIMDEX and AOSS has highlighted the need to accelerate the IN-1481 Epic (migrating to ECR workflows that can handle AMD64 AND ARM64 architectures).

The Dev workflow was tested earlier today with Christopher & Graham in both the timdex-index-manager and the browsertrix-harvester repositories with no issues at all.

How this addresses that need

  • Update the three multi-arch workflows by including the "prep" job that had been in the caller workflow until now
  • Organize the workflow into three jobs for clearer separation of steps
  • Update the README

Side effects of this change

No impactful side effects. For caller applications that have already updated to these workflows, the "prep" job will run twice, but this will have no impact on the build/push/deploy steps.

Relevant ticket(s)

@qltysh
Copy link
Copy Markdown

qltysh Bot commented Apr 6, 2026

All good ✅

Comment thread .github/workflows/ecr-multi-arch-deploy-dev.yml
Comment thread .github/workflows/ecr-multi-arch-deploy-dev.yml
Comment thread .github/workflows/ecr-multi-arch-deploy-stage.yml
Comment thread .github/workflows/ecr-multi-arch-deploy-stage.yml
Comment thread .github/workflows/ecr-multi-arch-promote-prod.yml
Comment thread .github/workflows/ecr-multi-arch-promote-prod.yml
Why these changes are being introduced:
The work on TIMDEX and AOSS has highlighted the need to accelerate the
IN-1481 Epic (migrating to ECR workflows that can handle AMD64 AND ARM64
architectures).

How this addresses that need:
* Update the three multi-arch workflows by including the "prep" job that
had been in the caller workflow until now
* Organize the workflow into three jobs for clearer separation of steps
* Update the README

Side effects of this change:
None, hopefully.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/IN-1732
@cabutlermit cabutlermit force-pushed the in-1732-update-deploy-workflows branch from 916125a to 6608c3b Compare April 7, 2026 18:12
@cabutlermit cabutlermit marked this pull request as ready for review April 7, 2026 18:14
@cabutlermit cabutlermit requested review from a team as code owners April 7, 2026 18:14
Comment thread .github/workflows/ecr-multi-arch-deploy-dev.yml
Comment thread README.md
Copy link
Copy Markdown

@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

Note: See my inline comments in the PR diff.

Some additional in-line documentation in the workflow files
so help a reviewer better understand what's happening.

Minor tweaks to the main README.
@cabutlermit cabutlermit requested a review from vab April 7, 2026 20:37
Copy link
Copy Markdown

@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 5ac3905 into main Apr 7, 2026
1 check passed
@cabutlermit cabutlermit deleted the in-1732-update-deploy-workflows branch May 4, 2026 20:26
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