test_runner sometimes fail due to "socket.send() raised exception" in stderr #13579

1. 
   MarcoFalke commented at 9:05 AM on July 1, 2018: member

   Some tests that use P2PConnection might fail after the switch to asyncio. The test itself is successful, but the stderr is non-empty. I believe this is an internal race in asyncio, where the remote node disconnected from our P2PConnection, but asyncio didn't realize it and is still sending (potentially buffered) bytes.

   One solution would be to just strip the stderr of "socket.send() raised exception".

   One example on osx, but can probably reproduced locally with ./test/functional/test_runner.py -j 16 --extended:

   feature_assumevalid.py failed, Duration: 13 s
   stdout:
   2018-06-30T17:10:31.246000Z TestFramework (INFO): Initializing test directory /var/folders/bb/n7t3rs157850byt_jfdcq9k80000gn/T/bitcoin_test_runner_20180630_170848/feature_assumevalid_87
   2018-06-30T17:10:43.682000Z TestFramework.mininode (WARNING): Connection lost to 127.0.0.1:11698 due to [Errno 32] Broken pipe
   2018-06-30T17:10:43.994000Z TestFramework (INFO): Stopping nodes
   2018-06-30T17:10:44.370000Z TestFramework (INFO): Cleaning up /var/folders/bb/n7t3rs157850byt_jfdcq9k80000gn/T/bitcoin_test_runner_20180630_170848/feature_assumevalid_87 on exit
   2018-06-30T17:10:44.370000Z TestFramework (INFO): Tests successful
   stderr:
   socket.send() raised exception.
   socket.send() raised exception.
   socket.send() raised exception.
   socket.send() raised exception.
   

2. MarcoFalke added the label Tests on Jul 1, 2018
3. MarcoFalke added the label good first issue on Jul 1, 2018
4. MarcoFalke added this to the milestone 0.17.0 on Jul 1, 2018
5. MarcoFalke closed this on Jul 19, 2018

   ---

6. MarcoFalke referenced this in commit e1260a798d on Jul 19, 2018
7. domob1812 referenced this in commit 4728553290 on Jul 23, 2018
8. domob1812 referenced this in commit 64b9f27e0e on Jul 23, 2018
9. MarcoFalke referenced this in commit ab28b5b527 on Jul 23, 2018
10. Bushstar referenced this in commit 4cf7d34a20 on Jul 24, 2018
11. HashUnlimited referenced this in commit abcd4e4bb1 on Sep 12, 2018
12. joemphilips referenced this in commit 44001f95b7 on Nov 9, 2018
13. jfhk referenced this in commit b8a0a20775 on Nov 14, 2018
14. furszy referenced this in commit 4331a67772 on Jun 4, 2021
15. furszy referenced this in commit 1460e5b812 on Jun 9, 2021
16. furszy referenced this in commit c9f265c7aa on Jun 10, 2021
17. furszy referenced this in commit 7038ae0bcc on Jun 12, 2021
18. furszy referenced this in commit 649cc97ea5 on Jun 12, 2021
19. furszy referenced this in commit 37aa223224 on Jun 14, 2021
20. furszy referenced this in commit 6ca86b36b0 on Jun 22, 2021
21. furszy referenced this in commit 77b9a4394b on Jun 22, 2021
22. furszy referenced this in commit 0433f2380e on Jun 24, 2021
23. furszy referenced this in commit db28a539ee on Jun 28, 2021
24. furszy referenced this in commit 2f49b8e44c on Jun 28, 2021
25. UdjinM6 referenced this in commit 62a51281a3 on Jun 29, 2021
26. UdjinM6 referenced this in commit cf0d69952f on Jun 29, 2021
27. UdjinM6 referenced this in commit 16fac5ab9a on Jun 29, 2021
28. UdjinM6 referenced this in commit 010ff7d499 on Jun 29, 2021
29. UdjinM6 referenced this in commit 6371bd96b3 on Jul 1, 2021
30. UdjinM6 referenced this in commit 1395a6a01f on Jul 1, 2021
31. UdjinM6 referenced this in commit fa3e3b4c07 on Jul 2, 2021
32. UdjinM6 referenced this in commit 5f50a8c8ad on Jul 2, 2021
33. UdjinM6 referenced this in commit 6f3d5a3e1b on Jul 2, 2021
34. UdjinM6 referenced this in commit ea58f30a6f on Jul 2, 2021
35. DrahtBot locked this on Sep 8, 2021
36. gades referenced this in commit 0323213381 on May 4, 2022

Milestone
0.17.0