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

There is already a Construct with name 'ConsoleMeSpoke' in App #9248

Open
Rayv1 opened this issue Sep 22, 2021 · 3 comments · May be fixed by #9269
Open

There is already a Construct with name 'ConsoleMeSpoke' in App #9248

Rayv1 opened this issue Sep 22, 2021 · 3 comments · May be fixed by #9269

Comments

@Rayv1
Copy link

Rayv1 commented Sep 22, 2021

Describe the bug
Hi,
Iam trying to deploy ConsoleMe with CDK, but when there are two spoke Accounts, cdk fails with error:
'There is already a Construct with name 'ConsoleMeSpoke' in App'

The main issue here is, that the ConsolemeSpokeAccountsStack Construct use the constant SPOKE_BASE_NAME, which is always 'ConsoleMeSpoke'. This should be something like SPOKE_BASE_NAME + Account Id.

To Reproduce
Deploy with two spoke Accounts.

Expected behavior
Working Deployment

Is there something that i have missed?

@castrapel
Copy link
Contributor

Hello! The CDK deployment strategy is deployed by the community. Looping in @avishayil, who may have some ideas here.

@avishayil
Copy link
Contributor

Hi @Rayv1 good point, the solution shouldn't be difficult like you mentioned. Care to PR or I'll do it?

@Rayv1
Copy link
Author

Rayv1 commented Sep 24, 2021

@avishayil could you please do it? thanks!

@connelldave connelldave linked a pull request Oct 22, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants