Skip to content

Commit

Permalink
Bump timeout in mqtt/stomp system tests (#428)
Browse files Browse the repository at this point in the history
- the test seems rather flaky with the 10 seconds
timeout, see pipeline failures:
https://hush-house.pivotal.io/teams/rabbitmq-for-kubernetes/pipelines/operator/jobs/operator-system-tests/builds/670
- use EventuallyWithOffset because this helper function is
called twice in the system tests, and its not obvious which
one it's failing
  • Loading branch information
ChunyiLyu committed Oct 28, 2020
1 parent 3cba654 commit 5d74b52
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions system_tests/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -741,7 +741,7 @@ func publishAndConsumeMQTTMsg(hostname, nodePort, username, password string, ove
c := mqtt.NewClient(opts)

var token mqtt.Token
Eventually(func() bool {
EventuallyWithOffset(1, func() bool {
token = c.Connect()
// Waits for the network request to reach the destination and receive a response
if !token.WaitTimeout(3 * time.Second) {
Expand All @@ -752,7 +752,7 @@ func publishAndConsumeMQTTMsg(hostname, nodePort, username, password string, ove
return true
}
return false
}, 10, 2).Should(BeTrue(), "Expected to be able to connect to MQTT port")
}, 15, 2).Should(BeTrue(), "Expected to be able to connect to MQTT port")

topic := "tests/mqtt"
msgReceived := false
Expand Down

0 comments on commit 5d74b52

Please sign in to comment.