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

issue MarcoFalke opened this issue on July 1, 2018
  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. marcoagner cross-referenced this on Jul 19, 2018 from issue tests: fixes mininode's P2PConnection sending messages on closing transport by marcoagner
  6. MarcoFalke closed this on Jul 19, 2018

  7. MarcoFalke referenced this in commit e1260a798d on Jul 19, 2018
  8. marcoagner cross-referenced this on Jul 23, 2018 from issue mininode tests broken with Python 3.4.2 (Debian Jessie) by domob1812
  9. domob1812 referenced this in commit 4728553290 on Jul 23, 2018
  10. domob1812 cross-referenced this on Jul 23, 2018 from issue tests: Skip P2PConnection's is_closing() check when not available by domob1812
  11. domob1812 referenced this in commit 64b9f27e0e on Jul 23, 2018
  12. MarcoFalke referenced this in commit ab28b5b527 on Jul 23, 2018
  13. Bushstar referenced this in commit 4cf7d34a20 on Jul 24, 2018
  14. HashUnlimited referenced this in commit abcd4e4bb1 on Sep 12, 2018
  15. HashUnlimited cross-referenced this on Sep 12, 2018 from issue Skip is_closing() check when not available. by HashUnlimited
  16. HashUnlimited cross-referenced this on Sep 12, 2018 from issue Skip is_closing() check when not available. by HashUnlimited
  17. joemphilips referenced this in commit 44001f95b7 on Nov 9, 2018
  18. jfhk referenced this in commit b8a0a20775 on Nov 14, 2018
  19. furszy referenced this in commit 4331a67772 on Jun 4, 2021
  20. furszy referenced this in commit 1460e5b812 on Jun 9, 2021
  21. furszy referenced this in commit c9f265c7aa on Jun 10, 2021
  22. furszy referenced this in commit 7038ae0bcc on Jun 12, 2021
  23. furszy referenced this in commit 649cc97ea5 on Jun 12, 2021
  24. furszy referenced this in commit 37aa223224 on Jun 14, 2021
  25. furszy referenced this in commit 6ca86b36b0 on Jun 22, 2021
  26. furszy referenced this in commit 77b9a4394b on Jun 22, 2021
  27. furszy referenced this in commit 0433f2380e on Jun 24, 2021
  28. furszy referenced this in commit db28a539ee on Jun 28, 2021
  29. furszy referenced this in commit 2f49b8e44c on Jun 28, 2021
  30. UdjinM6 referenced this in commit 62a51281a3 on Jun 29, 2021
  31. UdjinM6 referenced this in commit cf0d69952f on Jun 29, 2021
  32. UdjinM6 referenced this in commit 16fac5ab9a on Jun 29, 2021
  33. UdjinM6 referenced this in commit 010ff7d499 on Jun 29, 2021
  34. UdjinM6 referenced this in commit 6371bd96b3 on Jul 1, 2021
  35. UdjinM6 referenced this in commit 1395a6a01f on Jul 1, 2021
  36. UdjinM6 referenced this in commit fa3e3b4c07 on Jul 2, 2021
  37. UdjinM6 referenced this in commit 5f50a8c8ad on Jul 2, 2021
  38. UdjinM6 referenced this in commit 6f3d5a3e1b on Jul 2, 2021
  39. UdjinM6 referenced this in commit ea58f30a6f on Jul 2, 2021
  40. bitcoin locked this on Sep 8, 2021
  41. gades referenced this in commit 0323213381 on May 4, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-19 06:54 UTC