Skip to content

Conversation

flora-five
Copy link
Contributor

Description

Generate and add wrappers.

Motivation and Context

Useful to create and manage multiple clusters with Terragrunt.

Breaking Changes

None

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

Copy link
Member

@antonbabenko antonbabenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't mind having wrappers in modules, but I wonder whether terragrunt has added a capability to handle multiple identical modules transparently? It would be great to have our own codebase that is always smaller.

@flora-five Maybe you know?
@bryantbiggs Merge when/if you decide.

@bryantbiggs
Copy link
Member

I would prefer that we didn't have them (as Anton said, if this was supported natively either through Terragrunt or Terraform), but I am ok with adding them if its for the right reasons. I'm a bit skeptical here because it appears we are just doing a bit of code bowling and adding things for the sake of adding commits (I could be wrong)
image

@antonbabenko
Copy link
Member

@flora-five, could you please investigate and confirm whether it is already possible to achieve the same result in Terragrunt?

Let's merge it anyway, since it doesn't require a lot of maintenance from our side.

@bryantbiggs bryantbiggs merged commit ad0ce14 into terraform-aws-modules:master Sep 30, 2025
15 checks passed
antonbabenko pushed a commit that referenced this pull request Sep 30, 2025
## [9.16.0](v9.15.0...v9.16.0) (2025-09-30)

### Features

* Add wrappers ([#520](#520)) ([ad0ce14](ad0ce14))
@antonbabenko
Copy link
Member

This PR is included in version 9.16.0 🎉

@flora-five
Copy link
Contributor Author

Thank you for reviewing and merging it!

@flora-five, could you please investigate and confirm whether it is already possible to achieve the same result in Terragrunt?

@antonbabenko, unfortunately it is not possible (yet) to do it easily in Terragrunt, with HCL. This is an old answer, from when the term Terragrunt module was being used: Using for_each to call a module multiple times #413. There is a proposal from the community/discussion/work to add support for this in Terragrunt stacks and units: gruntwork-io/terragrunt#4504.

@bryantbiggs I understand that my recent pull requests might give the impression that I'm just doing useless work. I made them because in our projects we usually deploy and use sets of resources instead of single resources. For many resources we already use the wrappers that exist in other modules. The Elasticache and RDS modules that we are using too don't have the wrappers, and I thought that, besides being useful to us, they might be useful to other users too. The PR to fix the ALB example was made after we have encountered the same error about rule conditions when upgrading to the latest version of terraform-aws-alb. I don't have important/useful PRs, but I tried to contribute when I could, for example in terraform-aws-modules/terraform-aws-route53#72.

@bryantbiggs bryantbiggs changed the title feat: Add wrappers feat: Add Terragrunt wrappers Oct 1, 2025
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.

3 participants