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.

Sign up for GitHub

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

Jump to bottom

feat: use security-group module instead of resource #68

Merged
merged 19 commits into from
Jun 15, 2021
Merged

feat: use security-group module instead of resource #68

merged 19 commits into from
Jun 15, 2021

Conversation

SweetOps
Copy link
Contributor

what

  • use security-group module instead of resource
  • update tests

why

  • more flexible than current implementation
  • bring configuration of security group/rules to one standard

references

  • CPCO-409

@SweetOps SweetOps requested review from a team as code owners May 25, 2021 16:40
@SweetOps SweetOps requested review from Gowiem and 3h4x May 25, 2021 16:40
@SweetOps SweetOps marked this pull request as draft May 25, 2021 16:40
@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test bats

@SweetOps
Copy link
Contributor Author

/test bats

1 similar comment
@SweetOps
Copy link
Contributor Author

/test bats

@SweetOps
Copy link
Contributor Author

/test bats

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps
Copy link
Contributor Author

/test all

@SweetOps SweetOps marked this pull request as ready for review May 28, 2021 11:38
nitrocode
nitrocode reviewed Jun 7, 2021
View reviewed changes
outputs.tf Show resolved Hide resolved
@nitrocode nitrocode merged commit 6f4211d into master Jun 15, 2021
@nitrocode nitrocode deleted the CPCO-409 branch June 15, 2021 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nitrocode nitrocode nitrocode approved these changes

@Gowiem Gowiem Awaiting requested review from Gowiem

@3h4x 3h4x Awaiting requested review from 3h4x

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SweetOps @nitrocode @cloudpossebot