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

0.40.0 Missing arguments to pass existing security groups #123

Closed
z0rc opened this issue Jun 22, 2021 · 4 comments
Closed

0.40.0 Missing arguments to pass existing security groups #123

z0rc opened this issue Jun 22, 2021 · 4 comments
Labels
bug 🐛 An issue with the system

Comments

@z0rc
Copy link

z0rc commented Jun 22, 2021

#119 completely removed option to pass existing security groups to cluster, it just allows to create a new SG. Please revert this decision.

@z0rc z0rc added the bug 🐛 An issue with the system label Jun 22, 2021
@nitrocode
Copy link
Member

Please see comment #119 (comment)

@z0rc
Copy link
Author

z0rc commented Jun 22, 2021

Okay, it's called security_groups now. Why this kind of change happened in minor release is unexciting.

@nitrocode
Copy link
Member

@z0rc This is all part of standardizing usage of the security group module across all of the cloudposse modules. We had similar growing pains with bringing context.tf. We'll work on our upgrade process more going forward. If you have suggestions, please let us know in our slack community.

If other issues come up, please file a new ticket.

@Nuru
Copy link
Sponsor Contributor

Nuru commented Jun 28, 2021

@z0rc We are taking feedback like yours into account and have marked v0.40.0 pre-release. We recommend using 0.39.0 for now and waiting for a later release which will provide better backward compatibility and clearer migration instructions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system
Projects
None yet
Development

No branches or pull requests

3 participants