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

Fix ingress security group rule #187

Merged
merged 2 commits into from
Jul 25, 2023
Merged

Fix ingress security group rule #187

merged 2 commits into from
Jul 25, 2023

Conversation

Ganeshrockz
Copy link
Contributor

Changes proposed in this PR:

  • Fixes the existing ingress security group rule for the VPC that gets created for the acceptance tests. Doing so prevents public internet traffic from reaching this vpc network.

How I've tested this PR:

CI

How I expect reviewers to test this PR:

Checklist:

  • Tests added
  • CHANGELOG entry added

@Ganeshrockz Ganeshrockz marked this pull request as ready for review July 25, 2023 08:41
@Ganeshrockz Ganeshrockz merged commit 18871e9 into main Jul 25, 2023
@Ganeshrockz Ganeshrockz deleted the ganeshrockz/fix-ingress branch July 25, 2023 08:42
Ganeshrockz added a commit that referenced this pull request Jul 31, 2023
* Update main.tf

* Fix fmt
Ganeshrockz added a commit that referenced this pull request Aug 1, 2023
* Fix acceptance tests (#186)

* Fix acceptance tests

* Revert 1.16 consul version

* Skip running tests

* Add egress rule

* Fix egress

* Fix fmt

* Fix fmt

* Remove egress

* Fix egress rules

* Fix tests

* Fix

* Add egress rules

* Fix

* Enable more tests

* Add ingress rules

* Bring back HCP tests

* Remove ingress

* Add back ingress

* Enable only HCP

* Enable all tests

* Fix ingress security group rule (#187)

* Update main.tf

* Fix fmt

* Bump consul and envoy versions (#185)

* Bump consul and envoy versions

* Address comments

* Fix consul versions

* Address comments

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

Successfully merging this pull request may close these issues.

2 participants