Skip to content

Infrastructure/ Improvements#203

Open
Jarek-Rolski wants to merge 26 commits into
no10ds:mainfrom
co-cddo:infra/improve-providers
Open

Infrastructure/ Improvements#203
Jarek-Rolski wants to merge 26 commits into
no10ds:mainfrom
co-cddo:infra/improve-providers

Conversation

@Jarek-Rolski

@Jarek-Rolski Jarek-Rolski commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

A few infrastructure improvements:

  • Additional permissions to kms key - after failed deployment it wasn't possible to recreate resource due to lack of permissions
  • Updated aws_region attribute name replaced with region - removed depreciated warning
  • Added providers blocks in modules - allows to pass providers to module, it is necessary when provider uses assume role during deployment
  • Added aws_route53_record.copy-ns-records resource - copies subdomain hosted zone NS records to parent subdomain, without this deployment fails
  • Updated random_password.password resource - now it is consistent with cognito password requirements
  • Replaced null_resource with terraform_data
  • Added AWS_ROLE_ARN_TO_ASSUME variable to UI module and updated script uploading files to s3 - the change allows to execute script in case rAPId module is deployed with assume role
  • Added 4 variables to managage invitation emails: invite_message_email_message, invite_message_email_subject, sender_display_name, reply_to_email_address

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.

1 participant