Skip to content

Commit

Permalink
Merge pull request #331 from rowanjacobs/patch-2
Browse files Browse the repository at this point in the history
Allow security groups to pass on NSX-T [Finishes #160180588]
  • Loading branch information
staylor14 authored Aug 31, 2018
2 parents 1ec903c + 5bb3d0d commit 1b9296c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions security_groups/running_security_groups.go
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ var _ = SecurityGroupsDescribe("App Instance Networking", func() {

By("Testing that external connectivity to a private ip is not refused (but may be unreachable for other reasons)")
catnipCurlResponse = testAppConnectivity(clientAppName, privateAddress, 80)
Expect(catnipCurlResponse.Stderr).To(ContainSubstring("Connection timed out after"), "wide-open ASG configured but app is still refused by private ip")
Expect(catnipCurlResponse.Stderr).To(MatchRegexp("Connection timed out after|No route to host"), "wide-open ASG configured but app is still refused by private ip")

By("adding policy")
workflowhelpers.AsUser(TestSetup.AdminUserContext(), Config.DefaultTimeoutDuration(), func() {
Expand Down Expand Up @@ -263,7 +263,7 @@ var _ = SecurityGroupsDescribe("App Instance Networking", func() {

By("Testing that external connectivity to a private ip is refused")
catnipCurlResponse = testAppConnectivity(clientAppName, privateAddress, 80)
Expect(catnipCurlResponse.Stderr).To(ContainSubstring("refused"))
Expect(catnipCurlResponse.Stderr).To(MatchRegexp("refused|No route to host"))

By("deleting policy")
workflowhelpers.AsUser(TestSetup.AdminUserContext(), Config.DefaultTimeoutDuration(), func() {
Expand Down

0 comments on commit 1b9296c

Please sign in to comment.