-
Notifications
You must be signed in to change notification settings - Fork 53
e2e failures when curling for 502 #552
Comments
Logs look fine to me. Are all those failures that you were waiting for 502 but did not get it within the period? There are a few of these peppered around the router logs:
|
I think so but will have to defer to @slack -- the nuances between both issues have me a little lost. |
So routeability checking from #551 won't help in this particular case. What is happening here.
We are asserting via a router 502 that we have set the replica count to 0. Two things appear to be happening:
I think that's why we are hitting a 10s timeout, curl is doing a bunch of TCP retries and never sees a 5XX. @smothiki and @vdice are working on changing these sets of tests so that we wait after the scale down event, validate the replica count and then make the curl. That, paired with the post-beta router change in deis/router#153 should allow us to have more stable set of ps tests. |
Thanks, @slack. By the way, the test change you referenced is no longer scheduled. For beta, we are marking said tests as |
Sounds good! |
Just a refresh on this issue. It still exists, just not visible as tests the would otherwise experience this behavior are marked (See https://github.com/deis/workflow-e2e/blob/master/tests/ps_test.go#L161-L163, https://github.com/deis/workflow-e2e/blob/master/tests/ps_test.go#L90-L92 and https://github.com/deis/workflow-e2e/blob/master/tests/builds_test.go#L143) |
This behavior has not been seen lately; closing. |
https://ci.deis.io/job/workflow-beta1-test-e2e is consistently catching the following errors when running tests in which app url is curled and a
502
is expected. Specifically, no response is received from said curl command within the specified10s
timeoutFull set looks like:
Logs from the most recent build hitting these can be found here
The text was updated successfully, but these errors were encountered: