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

Simplify default security group lookup #561

Merged
merged 4 commits into from
Oct 13, 2023
Merged

Simplify default security group lookup #561

merged 4 commits into from
Oct 13, 2023

Conversation

shemau
Copy link
Contributor

@shemau shemau commented Oct 12, 2023

Description

To address internal issue 6157. The lookup of the default security group is unnecessarily complex. Avoid looking up all VPCs and finding the right one based on the supplied vpc_id. Just look up the one VPC using the vpc_id.

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content

There are no resource changes, so no impact to customers.

It is possible that the data resource for all VPC may show as being removed from the state file. However, as a data resource there are no resource changes.

Run the pipeline

If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.

Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:

/run pipeline

Checklist for reviewers

  • If relevant, a test for the change is included or updated with this PR.
  • If relevant, documentation for the change is included or updated with this PR.

For mergers

  • Use a conventional commit message to set the release level. Follow the guidelines.
  • Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
  • Use the Squash and merge option.

@shemau
Copy link
Contributor Author

shemau commented Oct 12, 2023

/run pipeline

@ocofaigh
Copy link
Member

@shemau can you resolve conflicts and re-run please? You can ignore the authorization policy error that occurred in last run - there was a parallel test running which caused conflict

@shemau shemau requested a review from jor2 October 12, 2023 15:42
@shemau
Copy link
Contributor Author

shemau commented Oct 12, 2023

/run pipeline

@shemau
Copy link
Contributor Author

shemau commented Oct 13, 2023

/run pipeline

@ocofaigh ocofaigh merged commit dff14e5 into main Oct 13, 2023
2 checks passed
@ocofaigh ocofaigh deleted the simplify branch October 13, 2023 09:26
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 2.12.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants