Skip to content

Commit

Permalink
Add a restart to multi-port test
Browse files Browse the repository at this point in the history
As part of the zero downtime feature work in CAPI, making changes to an
app's ports now requires a restart for the changes to take effect. This
commit reflects that change by adding a `cf restart` to the "Multiple
App Ports" test.

[#175190769](https://www.pivotaltracker.com/story/show/175190769)

Co-authored-by: Belinda Liu <[email protected]>
Co-authored-by: Sarah Weinstein <[email protected]>
  • Loading branch information
belinda-liu and sweinstein22 committed Jan 22, 2021
1 parent 8bd3c4f commit b28f00d
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions routing/multiple_app_ports.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,8 @@ var _ = RoutingDescribe("Multiple App Ports", func() {
Port: 7777,
}
InsertDestinations(getRouteGuid(secondRouteHostname), []Destination{destination})

Expect(cf.Cf("restart", appName, "--strategy", "rolling").Wait()).To(Exit(0))
})

It("should listen on multiple ports", func() {
Expand Down

0 comments on commit b28f00d

Please sign in to comment.