Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 1, 2025

Updates rules/utils.go to include missing ElastiCache node type per AWS documentation.

Changes

  • Added cache.r6gd.large (missing from r6gd family)
  • Updated .gitignore to exclude build artifact

Details

The cache.r6gd.large node type was missing from the r6gd family which supports data tiering for Redis. Total valid node types increased from 91 to 92.

Note: cache.r7gd is not included as it is only supported for AWS OpenSearch Service, not for ElastiCache Redis. ElastiCache Redis supports cache.r7g (without 'd').

Reference: https://docs.aws.amazon.com/AmazonElastiCache/latest/red-ug/CacheNodes.SupportedTypes.html

Original prompt

Please update rules/utils.go
You can see valid node types in https://docs.aws.amazon.com/AmazonElastiCache/latest/red-ug/CacheNodes.SupportedTypes.html


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…pdate .gitignore

Co-authored-by: wata727 <9624059+wata727@users.noreply.github.com>
Copilot AI changed the title [WIP] Update rules/utils.go with valid node types Add missing ElastiCache node types: r6gd.large and r7gd series Nov 1, 2025
Copilot AI requested a review from wata727 November 1, 2025 11:11
Copilot finished work on behalf of wata727 November 1, 2025 11:11
rules/utils.go Outdated
"cache.r7gd.4xlarge": true,
"cache.r7gd.8xlarge": true,
"cache.r7gd.12xlarge": true,
"cache.r7gd.16xlarge": true,
Copy link
Member

Choose a reason for hiding this comment

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

@copilot Is cache.r7gd a valid node type? I couldn't find it in the documentation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're correct. cache.r7gd is not supported for ElastiCache Redis - it's only available for AWS OpenSearch Service. I've removed all cache.r7gd entries in commit a0cdd25.

The only change now is adding cache.r6gd.large, which was missing from the r6gd series. Note that cache.r7g (without 'd') is supported for Redis and was already in the file.

Co-authored-by: wata727 <9624059+wata727@users.noreply.github.com>
Copilot AI changed the title Add missing ElastiCache node types: r6gd.large and r7gd series Add missing ElastiCache node type: cache.r6gd.large Nov 1, 2025
Copilot AI requested a review from wata727 November 1, 2025 13:09
Copilot finished work on behalf of wata727 November 1, 2025 13:09
@wata727 wata727 marked this pull request as ready for review November 1, 2025 13:14
Copy link
Member

@wata727 wata727 left a comment

Choose a reason for hiding this comment

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

👍

@wata727 wata727 enabled auto-merge (squash) November 1, 2025 13:15
@wata727 wata727 merged commit 485b521 into master Nov 1, 2025
21 checks passed
@wata727 wata727 deleted the copilot/update-rules-utils-go branch November 1, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants