Skip to content

Conversation

@bryantbiggs
Copy link
Member

Description

  • Update examples to use latest engine versions and security group module version (redis does not support latest 7.x versions)
  • Update examples to use data source for computing AZs and CIDR ranges
  • Deploy examples into eu-west-1 consistent with other module examples (where supported)
  • Rename complete example to redis to better reflect intent. Both examples are technically "complete" but demonstrate either engine option
  • engine is now required; this has been added to the redis example
  • The user names were made unique to allow for both examples to be provisioned at same time, otherwise there was a duplicate resource name conflict

Motivation and Context

  • Up to date and consistent

Breaking Changes

  • No

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

@magreenbaum magreenbaum left a comment

Choose a reason for hiding this comment

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

Thank you!

@bryantbiggs bryantbiggs merged commit 7a994d3 into terraform-aws-modules:master Nov 5, 2025
7 checks passed
@bryantbiggs bryantbiggs deleted the chore/example-update branch November 5, 2025 20:27
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