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
RDS: Support VPC #113
RDS: Support VPC #113
Conversation
af463f9
to
46a090e
Compare
46a090e
to
2a86846
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! I'll check my PR to see if there's something that I have added. The only worry I have is regarding EC2-VPC (new) vs EC2-Classic (legacy) accounts. We had this issue with my PR, it's difficult to test if you don't have both type of accounts :/.
2a86846
to
eea168e
Compare
b88e3b5
to
a6c8d9c
Compare
This introduced a regression: networks on old-style security groups now have an error unless they specify an empty list of vpc security groups. |
Create resources for subnet groups to allow RDS instances to be spawned in a VPC. Closes #57, #13, #6.
cc @RaitoBezarius -- I didn't think to look for your PR before I wrote this (sorry :() but also that one is pre-py3, etc.