Repository navigation
[EXP] Increase default remote timeout - #6001
Conversation
|
@pllim - This looks good to me given the assumption this will help the tests pass. If they become unstable again, we can revert the default again. I don't think this need a changelog, and milestone doesn't matter much as the bugfix branch will only have a few CI runs before release. |
|
Well... Look at that... Remote tests failing even here! Maybe this is not the correct solution after all. 🤔 |
|
@pllim - let's hope it was a fluke, only one of the remote ones were failing (now restrted) while previously it was all of them. |
|
That test passed now. Remaining failures unrelated. |
|
There are a few timeouts again, so I'm merging this in the hope it will have have some positive effect for those. |
|
FWIW https://travis-ci.org/astropy/astropy/builds/229908432 is all green except the ufunc thingy. |
|
Lovely, I'll add |
|
Ops, and also this but we'll see if it is consistent or transient. EarthLocation.of_address('1002 Holy Grail Court, St. Louis, MO') |
|
@mhvk , did you post the wrong link? It shows green with |
|
Ah, I hadn't realised that if one restarts a test, the old log gets overwritten... |
Increase default remote timeout to see if that helps with remote tests being flaky. This was part of #5997 but then taken out.
Related discussion: #5994
c/c @mhvk @bsipocz
TODO: