Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

CFN template for "launch vpc wizard" invalid #23

Open
treaz opened this issue Feb 8, 2019 · 2 comments
Open

CFN template for "launch vpc wizard" invalid #23

treaz opened this issue Feb 8, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@treaz
Copy link

treaz commented Feb 8, 2019

Describe the bug
The CFN template created for the "launch vpc wizard" is invalid.

Steps to reproduce the behavior:
0. Start the recorder

  1. Go to VPC
  2. Click on 'Launch vpc wizard'. Don't change anything, just give the name and click "create VPC"
  3. Stop recording and try to create a CFN stack with the output.
  4. CFN cannot create the stack because of the same id for all the resources.

Expected behavior
The CFN stack is successfully created.

Additional context
I've attached the invalid cfn template that was generated by the recorder.
console_cfn.txt

@iann0036 iann0036 added the bug Something isn't working label Feb 9, 2019
iann0036 added a commit that referenced this issue Feb 9, 2019
@iann0036
Copy link
Owner

iann0036 commented Feb 9, 2019

Hi Horia,

I've got it to a point where it should create a valid template for the first VPC wizard only (ready for use in 0.3.10). They are incredibly difficult to do the mappings for so might be some time.

Thanks again for helping find these issues!

@treaz
Copy link
Author

treaz commented Feb 9, 2019

Wow, that's indeed difficult and cumbersome. You could just remove the functionality. Maybe get the plugin to show a popup that it's not a supported feature.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants