Skip to content

Commit

Permalink
feat(redshift-alpha): issue #31634: add support for RA3.large node type
Browse files Browse the repository at this point in the history
  • Loading branch information
Imtiaz Asad Bin, INI-DNA-DL authored and Imtiaz Asad Bin, INI-DNA-DL committed Oct 3, 2024
1 parent 7a4f865 commit 41b2a51
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
5 changes: 5 additions & 0 deletions packages/@aws-cdk/aws-redshift-alpha/lib/cluster.ts
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,11 @@ export enum NodeType {
*/
DC2_8XLARGE = 'dc2.8xlarge',

/**
* ra3.large
*/
RA3_LARGE = 'ra3.large',

/**
* ra3.xlplus
*/
Expand Down
4 changes: 2 additions & 2 deletions packages/@aws-cdk/aws-redshift-alpha/test/cluster.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -671,7 +671,7 @@ describe('multi AZ cluster', () => {
masterPassword: cdk.SecretValue.unsafePlainText('tooshort'),
},
vpc,
nodeType: NodeType.RA3_XLPLUS,
nodeType: NodeType.RA3_LARGE,
multiAz: true,
});

Expand All @@ -684,7 +684,7 @@ describe('multi AZ cluster', () => {
AutomatedSnapshotRetentionPeriod: 1,
Encrypted: true,
NumberOfNodes: 2,
NodeType: 'ra3.xlplus',
NodeType: 'ra3.large',
DBName: 'default_db',
PubliclyAccessible: false,
ClusterSubnetGroupName: { Ref: 'RedshiftSubnetsDFE70E0A' },
Expand Down
2 changes: 1 addition & 1 deletion packages/aws-cdk-lib/cx-api/FEATURE_FLAGS.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ The following json shows the current recommended set of flags, as `cdk init` wou
"@aws-cdk/aws-s3:keepNotificationInImportedBucket": false,
"@aws-cdk/aws-ecs:reduceEc2FargateCloudWatchPermissions": true,
"@aws-cdk/aws-ec2:ec2SumTImeoutEnabled": true,
"@aws-cdk/aws-appsync:appSyncGraphQLAPIScopeLambdaPermission": true
"@aws-cdk/aws-appsync:appSyncGraphQLAPIScopeLambdaPermission": true,
"@aws-cdk/aws-rds:setCorrectValueForDatabaseInstanceReadReplicaInstanceResourceId": true
}
}
Expand Down

0 comments on commit 41b2a51

Please sign in to comment.