[SES-PHASE-1-7] - PLU-744: add env vars for ses and sqs#1633
Open
m0nggh wants to merge 1 commit into
Open
Conversation
This was referenced May 19, 2026
Contributor
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
9c04394 to
d5647a6
Compare
35706ff to
3db2cbc
Compare
d5647a6 to
19d6d37
Compare
This comment has been minimized.
This comment has been minimized.
19d6d37 to
e533f62
Compare
3db2cbc to
d24c3a0
Compare
ogp-weeloong
left a comment
Contributor
There was a problem hiding this comment.
let's try to roles etc in in our Iac repo as well
e533f62 to
badc3fe
Compare
d24c3a0 to
a60b320
Compare
badc3fe to
a291c21
Compare
9a41983 to
ff1bad2
Compare
983424e to
68d56fe
Compare
ff1bad2 to
a32f27a
Compare
f020b93 to
69fb6dc
Compare
067c7be to
8dbd477
Compare
46fc7be to
4d4108d
Compare
8dbd477 to
1d0d495
Compare
4d4108d to
921f21a
Compare
6b9d2e4 to
08ba295
Compare
921f21a to
aab629b
Compare
08ba295 to
87b8040
Compare
aab629b to
97d3b6f
Compare
87b8040 to
a16af3c
Compare
97d3b6f to
555afa8
Compare
555afa8 to
0ab31fe
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

TL;DR
Adds SES and SQS environment variable references to the ECS configuration.
What changed?
Three new environment variable entries have been added to the ECS environment configuration:
SES_ROLE_ARN— sourced fromplumber-ses-role-arnSES_CONFIGURATION_SET— sourced fromplumber-<ENVIRONMENT>-ses-configuration-setSQS_QUEUE_URL— sourced fromplumber-<ENVIRONMENT>-sqs-queue-urlA missing newline at the end of the file has also been added.
How to test?
Note: Tested on UAT
Deploy to a non-production environment and verify that the SES and SQS environment variables are correctly injected into the running ECS task. Confirm the values resolve properly from their respective secrets/parameter store entries.
Why make this change?
These environment variables are required to support SES email sending (via an assumed IAM role and configuration set) and SQS queue integration within the ECS-hosted service.