Skip to content

BCSS-23409 - feat(elasticache): create elasticache module#93

Open
nhs-oliverslater wants to merge 6 commits into
mainfrom
feature/BCSS-23409-create-elasticache-module
Open

BCSS-23409 - feat(elasticache): create elasticache module#93
nhs-oliverslater wants to merge 6 commits into
mainfrom
feature/BCSS-23409-create-elasticache-module

Conversation

@nhs-oliverslater

Copy link
Copy Markdown
Contributor

Description

Refactor the ElastiCache module to support multiple deployment modes, simplify security group handling, and enhance logging configuration. Update the gitleaks configuration for improved IPv4 and IPv6 rule handling. Align the context.tf file and update documentation for case sensitivity.

Context

These changes are required to improve the flexibility and usability of the ElastiCache module, allowing for various deployment configurations while ensuring better security practices and logging capabilities.

Type of changes

  • Refactoring (non-breaking change)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would change existing functionality)

Checklist

  • I am familiar with the contributing guidelines
  • I have followed the code style of the project
  • I have added tests to cover my changes
  • I have updated the documentation accordingly
  • This PR is a result of pair or mob programming

Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT include any PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that does contain any sensitive information. We really appreciate your cooperation in this matter.

  • I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

…eployment modes (replication_group, cluster, serverless)

- Updated locals.tf to define deployment modes and naming conventions.
- Enhanced main.tf to configure ElastiCache resources based on deployment mode.
- Added outputs for replication group, cluster, and serverless cache details.
- Removed deprecated readme.md and updated variables.tf with comprehensive input descriptions.
- Improved validation for engine types, deployment modes, and snapshot configurations.
- Adjusted versions.tf to require compatible provider versions.

BREAKING CHANGE: Replaces exisiting module, replaces all explicit and implicit resources and changes upstream module version. All resources will be re-created.
@nhs-oliverslater nhs-oliverslater marked this pull request as ready for review June 23, 2026 09:36
@nhs-oliverslater nhs-oliverslater requested review from a team and saliceti as code owners June 23, 2026 09:36
@nhs-oliverslater nhs-oliverslater enabled auto-merge (squash) June 23, 2026 11:21
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