WHY IS THE TEST FAILING ON CI I DON’T UNDERSTAND
# developmenta
average-vr-56795
11/14/2018, 1:33 PMWHY IS THE TEST FAILING ON CI I DON’T UNDERSTAND
😢 2
r
red-balloon-89377
11/14/2018, 5:07 PMThe elves, it’s always the elves.
a
average-vr-56795
11/14/2018, 5:58 PMGenuinely travis’s stty size changed in the 14 hours between me pushing the PR and it merging.
https://github.com/pantsbuild/pants/pull/6768
average-vr-56795
11/14/2018, 6:10 PM@witty-crayon-22786 You seemed slightly in disbelief when I mentioned the avenue I was investigating… 😛
w
witty-crayon-22786
11/14/2018, 6:11 PMhah, yes. because i didn't understand why that would cause a failure for the mixed test, but not the other!
witty-crayon-22786
11/14/2018, 6:12 PM(i would still be curious about that)
a
average-vr-56795
11/14/2018, 6:12 PMThe three tests were in different shards
average-vr-56795
11/14/2018, 6:12 PMMy assumption is that one shard got the new config and the other two got the old one
average-vr-56795
11/14/2018, 6:13 PMYay implicit dependencies!
w
witty-crayon-22786
11/14/2018, 6:13 PMwowie.
a
average-vr-56795
11/14/2018, 6:14 PMAll three now consistently fail on travis with the default stty, and consistently pass with the fixed stty