Skip to content

Commit ce0e09f

Browse files
feat(redshift-alpha): add support for RA3.large node type (#31637)
### Issue # (if applicable) Closes #31634. ### Reason for this change add support for RA3.large node type ### Description of changes extended the NodeType enum in ClusterProps.nodeType to include RA3.large ### Description of how you validated changes ran all tests ### Checklist - [ x ] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 57cbd86 commit ce0e09f

12 files changed

+1876
-2
lines changed

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

+1
Original file line numberDiff line numberDiff line change
@@ -700,6 +700,7 @@ as it's a fast operation and typically completes in minutes. Elastic resize is o
700700
* dc2.8xlarge
701701
* ds2.xlarge
702702
* ds2.8xlarge
703+
* ra3.large
703704
* ra3.xlplus
704705
* ra3.4xlarge
705706
* ra3.16xlarge

packages/@aws-cdk/aws-redshift-alpha/lib/cluster.ts

+5
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,11 @@ export enum NodeType {
4949
*/
5050
DC2_8XLARGE = 'dc2.8xlarge',
5151

52+
/**
53+
* ra3.large
54+
*/
55+
RA3_LARGE = 'ra3.large',
56+
5257
/**
5358
* ra3.xlplus
5459
*/

packages/@aws-cdk/aws-redshift-alpha/test/cluster.test.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -734,7 +734,7 @@ describe('multi AZ cluster', () => {
734734
masterPassword: cdk.SecretValue.unsafePlainText('tooshort'),
735735
},
736736
vpc,
737-
nodeType: NodeType.RA3_XLPLUS,
737+
nodeType: NodeType.RA3_LARGE,
738738
multiAz: true,
739739
});
740740

@@ -747,7 +747,7 @@ describe('multi AZ cluster', () => {
747747
AutomatedSnapshotRetentionPeriod: 1,
748748
Encrypted: true,
749749
NumberOfNodes: 2,
750-
NodeType: 'ra3.xlplus',
750+
NodeType: 'ra3.large',
751751
DBName: 'default_db',
752752
PubliclyAccessible: false,
753753
ClusterSubnetGroupName: { Ref: 'RedshiftSubnetsDFE70E0A' },

packages/@aws-cdk/aws-redshift-alpha/test/integ.cluster-nodetype.js.snapshot/RA3LargeNodeIntegTestDefaultTestDeployAssert1EF91947.assets.json

+19
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk/aws-redshift-alpha/test/integ.cluster-nodetype.js.snapshot/RA3LargeNodeIntegTestDefaultTestDeployAssert1EF91947.template.json

+36
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk/aws-redshift-alpha/test/integ.cluster-nodetype.js.snapshot/cdk.out

+1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk/aws-redshift-alpha/test/integ.cluster-nodetype.js.snapshot/integ.json

+12
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk/aws-redshift-alpha/test/integ.cluster-nodetype.js.snapshot/manifest.json

+277
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)