feat(elasticache): implement Serverless ElastiCache L2 Construct #35424

maramure · 2025-09-05T15:35:51Z

Issue # (if applicable)

Closes #.

Reason for this change

AWS CDK currently lacks L2 constructs for Serverless ElastiCache and user management features. Customers must work directly with L1 constructs (CfnUser, CfnUserGroup, CfnServerlessCache) which are verbose and lack the convenience, validation, and best practices that L2 constructs provide.

Description of changes

This PR implements the L2 constructs for ElastiCache UserGroup and ServerlessCache as an initial step towards complete Serverless ElastiCache support.

Files added to aws-elasticache package:

common.ts - Shared UserEngine enum (VALKEY, REDIS)
user-group.ts - Complete UserGroup L2 construct with import methods, addUser() functionality, and engine-specific validation
serverless-cache-base.ts - Engine Enum; Interface and Base class for ServerlessCache L2 construct
serverless-cache.ts - Complete ServerlessCache L2 construct with IAM grant methods, comprehensive CloudWatch metrics, networking connections, backup management, usage limits configuration, and robust input validations
user-base.ts - Base class and interface extended by ElastiCache User constructs with shared properties and import methods
password-user.ts- Complete PasswordUser L2 construct with password-based authentication, and proper validations
no-password-user.ts- Complete NoPasswordUser L2 construct without password authentication, and proper validations
iam-user.ts - Complete IamUser L2 construct with IAM authentication, connect permissions, and validations
README.md - Complete user manual describing how to create and configure a user, user group and a serverless cache, explaining all the important features

Describe any new or updated permissions being added

NA

Description of how you validated changes

I implemented unit tests for all the construct files introduced: all the 3 types of users, user-group and serverless-cache.
I also added integration tests.

Checklist

My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

aws-cdk-automation

(This review is outdated)

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts

kumvprat

Added a few inline comments

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache-base.ts

packages/aws-cdk-lib/aws-elasticache/lib/user-group.ts

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts

packages/aws-cdk-lib/aws-elasticache/lib/user-group.ts

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache-base.ts

gasolima

This module needs to ends with -alpha see examples here

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts

mazyu36

Thanks.
I've added some comments, though I haven't completed the full review yet."

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache-base.ts

packages/aws-cdk-lib/aws-elasticache/lib/iam-user.ts

packages/aws-cdk-lib/aws-elasticache/lib/password-user.ts

packages/aws-cdk-lib/aws-elasticache/lib/no-password-user.ts

packages/aws-cdk-lib/aws-elasticache/lib/user-group.ts

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache-base.ts

packages/@aws-cdk/aws-elasticache-alpha/README.md

packages/@aws-cdk/aws-elasticache-alpha/lib/iam-user.ts

packages/@aws-cdk/aws-elasticache-alpha/lib/user-base.ts

packages/@aws-cdk/aws-elasticache-alpha/lib/user-group.ts

packages/@aws-cdk/aws-elasticache-alpha/test/iam-user.test.ts

packages/@aws-cdk/aws-elasticache-alpha/test/no-password-user.test.ts

packages/@aws-cdk/aws-elasticache-alpha/test/password-user.test.ts

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

aws-cdk-automation

(This review is outdated)

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

Pull request has been modified.

aws-cdk-automation

(This review is outdated)

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

...st/integ.distribution-function-runtime.js.snapshot/integ-distribution-function.template.json

packages/@aws-cdk/aws-elasticache-alpha/README.md

packages/@aws-cdk/aws-elasticache-alpha/lib/serverless-cache.ts

packages/@aws-cdk/aws-elasticache-alpha/lib/user-group.ts

packages/@aws-cdk/aws-elasticache-alpha/test/integ.serverless-cache.ts

packages/@aws-cdk/aws-elasticache-alpha/test/serverless-cache-base.test.ts

packages/@aws-cdk/aws-elasticache-alpha/lib/iam-user.ts

packages/@aws-cdk/aws-elasticache-alpha/test/integ.serverless-cache.ts

kumvprat

LGTM

mergify · 2025-09-26T16:55:32Z

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

mergify · 2025-09-26T16:57:53Z

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

mergify · 2025-09-26T17:55:19Z

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #35424 has been dequeued. Mergify failed to merge the pull request. GitHub can't merge the pull request after 15 retries.
Waiting for the branch protection required status checks to be validated.

You can check the last failing draft PR here: #35605.

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

mergify · 2025-09-26T17:57:47Z

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

Pull request has been modified.

ozelalisen · 2025-09-29T09:05:07Z

@Mergifyio refresh

mergify · 2025-09-29T09:05:12Z

refresh

✅ Pull request refreshed

mergify · 2025-09-29T09:07:24Z

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

mergify · 2025-09-29T09:09:48Z

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

github-actions · 2025-09-29T09:10:05Z

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

github-actions bot added the p2 label Sep 5, 2025

aws-cdk-automation requested a review from a team September 5, 2025 15:36

github-actions bot added the beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK label Sep 5, 2025

aws-cdk-automation previously requested changes Sep 5, 2025

View reviewed changes

gasolima added the pr/do-not-merge This PR should not be merged at this time. label Sep 8, 2025

maramure force-pushed the elasticache branch from 9e150a7 to a9191d1 Compare September 9, 2025 17:23

gasolima reviewed Sep 10, 2025

View reviewed changes

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts Outdated Show resolved Hide resolved

maramure force-pushed the elasticache branch 2 times, most recently from 0146d26 to 06537ee Compare September 10, 2025 14:35

kumvprat reviewed Sep 10, 2025

View reviewed changes

otaviomacedo reviewed Sep 11, 2025

View reviewed changes

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts Outdated Show resolved Hide resolved

packages/aws-cdk-lib/aws-elasticache/lib/user-group.ts Outdated Show resolved Hide resolved

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache-base.ts Outdated Show resolved Hide resolved

gasolima previously requested changes Sep 15, 2025

View reviewed changes

go-to-k reviewed Sep 16, 2025

View reviewed changes

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache.ts Outdated Show resolved Hide resolved

mazyu36 suggested changes Sep 16, 2025

View reviewed changes

gasolima reviewed Sep 18, 2025

View reviewed changes

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache-base.ts Show resolved Hide resolved

gasolima reviewed Sep 18, 2025

View reviewed changes

packages/aws-cdk-lib/aws-elasticache/lib/serverless-cache-base.ts Outdated Show resolved Hide resolved

mergify bot mentioned this pull request Sep 18, 2025

Elasticache #35504

Closed

1 task

maramure force-pushed the elasticache branch from 612fba6 to 19245b4 Compare September 19, 2025 16:25

kumvprat reviewed Sep 22, 2025

View reviewed changes

maramure force-pushed the elasticache branch 2 times, most recently from 6632657 to 847097e Compare September 23, 2025 08:14

maramure force-pushed the elasticache branch from 847097e to cf4ff6a Compare September 23, 2025 08:18

aws-cdk-automation previously requested changes Sep 23, 2025

View reviewed changes

maramure force-pushed the elasticache branch from cf4ff6a to 847097e Compare September 23, 2025 08:18

aws-cdk-automation previously requested changes Sep 23, 2025

View reviewed changes

maramure force-pushed the elasticache branch from 8a524cd to 7afbb8d Compare September 23, 2025 11:42

gasolima and others added 3 commits September 23, 2025 19:11

remove unintended changes

b0b6c02

feat(elasticache): fix README error

cdf0e16

feat(elasticache): replace Average metrics

1fdf4a4

maramure force-pushed the elasticache branch from 118c1d1 to 1fdf4a4 Compare September 23, 2025 17:12

build refresh

49aa623

kumvprat reviewed Sep 24, 2025

View reviewed changes

feat(elasticache): add assertions and more detailed documentation

37da1b9

kumvprat reviewed Sep 26, 2025

View reviewed changes

packages/@aws-cdk/aws-elasticache-alpha/test/integ.serverless-cache.ts Show resolved Hide resolved

kumvprat previously approved these changes Sep 26, 2025

View reviewed changes

kumvprat removed the pr/do-not-merge This PR should not be merged at this time. label Sep 26, 2025

mergify bot added the queued label Sep 26, 2025

mergify bot added a commit that referenced this pull request Sep 26, 2025

Merge of #35424

db6c22c

mergify bot mentioned this pull request Sep 26, 2025

merge queue: embarking main (89bfef7) and #35424 together #35605

Closed

30 tasks

mergify bot removed the queued label Sep 26, 2025

Merge branch 'main' into elasticache

e8c96a4

kumvprat approved these changes Sep 29, 2025

View reviewed changes

mergify bot added the queued label Sep 29, 2025

mergify bot merged commit 0e08c8c into aws:main Sep 29, 2025
27 of 28 checks passed

mergify bot removed the queued label Sep 29, 2025

github-actions bot locked as resolved and limited conversation to collaborators Sep 29, 2025

feat(elasticache): implement Serverless ElastiCache L2 Construct #35424

feat(elasticache): implement Serverless ElastiCache L2 Construct #35424

Uh oh!

Conversation

maramure commented Sep 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue # (if applicable)

Reason for this change

Description of changes

Describe any new or updated permissions being added

Description of how you validated changes

Checklist

Uh oh!

aws-cdk-automation left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

kumvprat left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gasolima left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mazyu36 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

aws-cdk-automation left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

aws-cdk-automation left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

maramure commented Sep 5, 2025 •

edited

Loading

aws-cdk-automation left a comment •

edited

Loading

aws-cdk-automation left a comment •

edited

Loading

aws-cdk-automation left a comment •

edited

Loading