Skip to content

Commit 526a289

Browse files
committed
ADR for Multi-Region Update
Why these changes are being introduced: Once again, the CDPS project is pushing us to update one of our core infrastructure repositories to handle deployment of resources to multiple regions. How this addresses that need: * Create and ADR for this decision * Minor formatting change to the PR template Side effects of this change: None. Relevant ticket(s): * https://mitlibraries.atlassian.net/browse/IR-238
1 parent 5203cac commit 526a289

File tree

2 files changed

+29
-6
lines changed

2 files changed

+29
-6
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,30 @@
1-
#### Developer Checklist
1+
## Developer Checklist
22

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

7-
#### What does this PR do?
7+
## What does this PR do?
88

99
A few sentences describing the overall goals of the pull request's commits.
1010
Why are we making these changes? Is there more work to be done to fully
1111
achieve these goals?
1212

13-
#### Helpful background context
13+
## Helpful background context
1414

1515
Describe any additional context beyond what the PR accomplishes if it is likely
1616
to be useful to a reviewer.
1717

1818
Delete this section if it isn't applicable to the PR.
1919

20-
#### What are the relevant tickets?
20+
## What are the relevant tickets?
2121

2222
Include links to Jira Software and/or Jira Service Management tickets here.
2323

24-
#### Requires Database Migrations?
24+
## Requires Database Migrations?
2525

2626
YES | NO
2727

28-
#### Includes new or updated dependencies?
28+
## Includes new or updated dependencies?
2929

3030
YES | NO
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# 4. Make repository mulit-region capable
2+
3+
Date: 2025-05-06
4+
5+
## Status
6+
7+
Proposed
8+
9+
## Context
10+
11+
We are now at the place in our infrastructure that we need to be able to deploy ECR repositories in multiple AWS regions so that we can deploy containerized applications (Fargate & Lambda) in mulitple regions. This is primarly driven by the CDPS project, but will be available to any other project that expects containers in more than just `us-east-1`.
12+
13+
As we extend this to multiple regions, it is **very import** that we do not modify any of the existing outputs from this repository, either to SSM Parameter Store or to Terraform Cloud outputs -- too many other repositories are already dependent on those values and would all need refactoring if any of the outputs change.
14+
15+
## Decision
16+
17+
1. Add additional provider blocks in the root of the repository, as needed.
18+
1. Update the embedded `ecr` module to handle ECR repository creation for containers that need to be deployed in multiple AWS regions.
19+
1. Update the generated GHA workflows and Makefile outputs to support multiple AWS regions.
20+
21+
## Consequences
22+
23+
If this is done correctly, there will be no consequences.

0 commit comments

Comments
 (0)