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-Xazs: Smaller subnet size options #180

Open
bedge opened this issue Jun 29, 2018 · 1 comment
Open

vpc-Xazs: Smaller subnet size options #180

bedge opened this issue Jun 29, 2018 · 1 comment

Comments

@bedge
Copy link

bedge commented Jun 29, 2018

Re: vpc/vpc-*azs.yaml

Feature suggestion/guidelines for possible patch.

Class B subnet granularity is a lot for a single VPC. If one wants to preserve the option of VPC peering, then using less than a class B may be preferable.

Rather than basing everything off of one param:

CidrBlock: !Sub '10.${ClassB}.0.0/16'

Ideally one could add new params with defaults that provide the same behavior unless specified. However given that arithmetic expressions aren't supported, this implies some preprocessing, or a breaking change that uses different params all together.

The minimum number of params one would need are, I think:

IpBase, 10.x.x.x
PrivateSubnetMask: 16
PublicSubnetMask: 20

To continue with that line of reasoning for a bit, one could do this with some pre-processing of the template, despite the lack of finesse. The custom resource thing just feels way to heavy handed for simple arithmetic (http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/template-custom-resources.html)
Some level of validation to prevent illegal combinations wouldn't be bad either.

Is there an accepted templating format for yaml?

@michaelwittig
Copy link
Contributor

Hi @bedge this is slightly related to #152 which will be fixed in v7. As soon as we have this merged, we can start working on your proposal. At the moment (v6), the ClassB parameter is used by other templates as well, so we don't have much flexibility here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants