Skip to content

Conversation

@cabutlermit
Copy link
Contributor

@cabutlermit cabutlermit commented Mar 18, 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 a new set of ECR-related resources for the s3-bagit-validator application that will be part of CDPS
  • Remove the overwrite = true lines from the Terraform managed ssm_outputs (that line is deprecated in the current version of the AWS Provider)
  • Clean up the README

Helpful background context

As part of CDPS and directly related to the Isilon Replacement project, this starts the build out of a new app for validating AIPs stored in S3 as part of the Archivematica implementation.

What are the relevant tickets?

Requires Database Migrations?

NO

Includes new or updated dependencies?

NO

cabutlermit and others added 2 commits March 18, 2025 13:04
Why these changes are being introduced:
As part of CDPS and directly related to the Isilon Replacement project,
this starts the build out of a new app for validating AIPs stored in
S3 as part of the Archivematica implementation.

How this addresses that need:
* Create a new set of ECR-related resources for the s3-bagit-validator
application that will be part of CDPS
* Remove the `overwrite = true` lines from the Terraform managed
ssm_outputs (that line is deprecated in the current version of the AWS
Provider)

Side effects of this change:
None.

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/IR-177
Dev-to-Stage: Create ECR for s3-bagit-validator
@cabutlermit cabutlermit marked this pull request as ready for review March 18, 2025 19:55
@cabutlermit cabutlermit requested a review from a team as a code owner March 18, 2025 19:55
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 f7e99ac into main Mar 19, 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.

3 participants