Skip to content

Conversation

@badmintoncryer
Copy link
Contributor

@badmintoncryer badmintoncryer commented Dec 30, 2025

Issue # (if applicable)

Closes #36416.

Reason for this change

Redshift doesn't support for creating dc2 instance type now. However, AWS CDK uses dc2.large node type as a default value. Therefore, we cannot deploy Redshift cluster with minimum configuration.

declare const vpc: ec2.IVpc;

const cluster = new Cluster(this, 'Redshift', {
  masterUser: {
    masterUsername: 'admin',
  },
  vpc,
});

Description of changes

Update the default node type to ra3.large that is now recommended by AWS.

Describe any new or updated permissions being added

none

Description of how you validated changes

Update existing unit and integ tests.

Checklist


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

BREAKING CHANGE: Updated default nodeType. Previously, DC2.LARGE is used, but after the correction, new default nodeType is RA3.LARGE.

@github-actions github-actions bot added the bug This issue is a bug. label Dec 30, 2025
@aws-cdk-automation aws-cdk-automation requested a review from a team December 30, 2025 13:12
@github-actions github-actions bot added effort/medium Medium work item – several days of effort p2 distinguished-contributor [Pilot] contributed 50+ PRs to the CDK labels Dec 30, 2025
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

(This review is outdated)

@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ☑️SkippedFailed ❌️
Security Guardian Results325 ran324 passed1 failed
TestResult
Security Guardian Results
packages/@aws-cdk/aws-redshift-alpha/test/integ.cluster-loggingbucket.js.snapshot/awscdkredshiftclusterdatabaseredshiftloggingbucketinteg06BC94A7.template.json
s3-encryption-enabled.guard❌ failure

@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ☑️SkippedFailed ❌️
Security Guardian Results with resolved templates325 ran324 passed1 failed
TestResult
Security Guardian Results with resolved templates
packages/@aws-cdk/aws-redshift-alpha/test/integ.cluster-loggingbucket.js.snapshot/awscdkredshiftclusterdatabaseredshiftloggingbucketinteg06BC94A7.template.json
s3-encryption-enabled.guard❌ failure

@aws-cdk-automation aws-cdk-automation dismissed their stale review December 30, 2025 15:18

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

@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Dec 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug This issue is a bug. distinguished-contributor [Pilot] contributed 50+ PRs to the CDK effort/medium Medium work item – several days of effort p2 pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(redshift): cannot deploy cluster with default nodeType setting

2 participants