Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate infrastructure code to Terraform 1.3 #50

Merged
merged 4 commits into from
Nov 2, 2023

Conversation

marcransome
Copy link
Contributor

@marcransome marcransome commented Nov 1, 2023

These changes upgrade the codebase to work with Terraform version 1.3 and include the following modifications and improvements:

  • Update Terraform version constraint
  • Update terraform-modules/aws/instance_profile module version
  • Update AWS provider version constraint
  • Replace deprecated AWS provider data source aws_subnet_ids with aws_subnets
  • Reorder input variables, update descriptions for consistency across stacks, and add additional type constraints
  • Replace attribute-only splat expressions (.*) with the new syntax ([*])
  • Simplify log group handling and internal data structure conversion
  • Replace duplicate ternary expressions with single-expression local variable representing resource count
  • Remove unnecessary dynamic blocks
  • Remove empty source files

Related: https://github.com/companieshouse/ci-pipelines/pull/2659.

@marcransome marcransome added the enhancement New feature or request label Nov 1, 2023
@marcransome marcransome changed the title Refactor sourcecode for migration to Terraform 1.3 Migrate infrastructure code to Terraform 1.3 Nov 1, 2023
@marcransome marcransome merged commit 9a6c83f into infrastructure Nov 2, 2023
@marcransome marcransome deleted the terraform-1.13-migration branch November 2, 2023 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants