Skip to content

Commit

Permalink
Merge pull request #216 from NASA-IMPACT/fix/assume-role-arns
Browse files Browse the repository at this point in the history
Add back read, write assume arns
  • Loading branch information
smohiudd authored Aug 13, 2024
2 parents bbca364 + a1cc41b commit 226a6ed
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions infrastructure/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,8 @@ resource "local_file" "mwaa_variables" {
ecs_cluster_name = module.mwaa.cluster_name
log_group_name = module.mwaa.log_group_name
mwaa_execution_role_arn = module.mwaa.mwaa_role_arn
assume_role_read_arn = var.assume_role_arns[0]
assume_role_write_arn = var.assume_role_arns[0]
account_id = local.account_id
aws_region = local.aws_region
cognito_app_secret = var.workflows_client_secret
Expand Down
2 changes: 2 additions & 0 deletions infrastructure/mwaa_environment_variables.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@
"ACCOUNT_ID": "${account_id}",
"AWS_REGION": "${aws_region}"
},
"ASSUME_ROLE_READ_ARN": "${assume_role_read_arn}",
"ASSUME_ROLE_WRITE_ARN": "${assume_role_write_arn}",
"COGNITO_APP_SECRET": "${cognito_app_secret}",
"STAC_INGESTOR_API_URL": "${stac_ingestor_api_url}",
"STAC_URL": "${stac_url}",
Expand Down

0 comments on commit 226a6ed

Please sign in to comment.