Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vpc property maxAZs is ignored #3237

Closed
1 of 5 tasks
Console32 opened this issue Jul 8, 2019 · 1 comment
Closed
1 of 5 tasks

Vpc property maxAZs is ignored #3237

Console32 opened this issue Jul 8, 2019 · 1 comment
Labels
needs-triage This issue or PR still needs to be triaged.

Comments

@Console32
Copy link

Console32 commented Jul 8, 2019

  • I'm submitting a ...

    • πŸͺ² bug report
    • πŸš€ feature request
    • πŸ“š construct library gap
    • ☎️ security issue or vulnerability => Please see policy
    • ❓ support request => Please see note at the top of this template.
  • What is the current behavior?
    If the current behavior is a πŸͺ²bugπŸͺ²: Please provide the steps to reproduce

If I create a brand new stack, specify a region that has at least 3 AZs (ex: eu-central-1) and create a Vpc.

new MaxAzTestStack(app, 'MaxAzTestStack', { 
  env: {
    region: 'us-east-1'
  }
});
export class MaxAzTestStack extends cdk.Stack {
  constructor(scope: cdk.Construct, id: string, props?: cdk.StackProps) {
    super(scope, id, props);

    new ec2.Vpc(this,'test', { })
  }
}

The Vpc spans two AZ and creates 4 Subnets

  • What is the expected behavior (or behavior of feature suggested)?

Deploying the stack should create a VPC that spans 3 AZs as described in the propeprty maxAZs of the Vpc, but the value is ignored (even if set to 99)

  • What is the motivation / use case for changing the behavior or adding this feature?

The maxAZs property is required to create a vpc network that needs more than 2 AZs

  • Please tell us about your environment:

    • CDK CLI Version: 0.37.0
    • Module Version: 0.37.0
    • OS: Windwos 10
    • Language: TypeScript
  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. associated pull-request, stackoverflow, gitter, etc)

@Console32 Console32 added the needs-triage This issue or PR still needs to be triaged. label Jul 8, 2019
@mhuebner
Copy link

mhuebner commented Jul 8, 2019

You have to set account in your env. See #3083

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

No branches or pull requests

2 participants