[DONTMERGE] Test run with reproducer for forced chunked send #36724

pull optout21 wants to merge 1 commits into bitcoin:master from optout21:2610-net-send-chunks-reproducer changing 1 files +26 −1
  1. optout21 commented at 8:59 AM on November 2, 2026: contributor

    WARNING: NOT INTENDED FOR MERGING!

    Performing a test run with forced chunked send.

    All networking code using sockets must not make assumption about the chunking of the incoming/outgoing data, and should be resilient against any chunking. Normally, the chunking of the sends of the writer dictate the chunking of the receives at the reader, but this is not guaranteed by IP/TCP, intermediate networking elements may change the packet structure.

    This branch contains a stress test reproducer. For every socket send, sending is changed to break up data in small chunks. This forcedly changes chunking of the data stream. The goal is to test code for robustness in face of unexpected chunking of the data.

    This PR will be closed once the test runs complete, and kept for reference (some tests are expected to fail).

    WARNING: NOT INTENDED FOR MERGING!

  2. DONTMERGE! net: Reproducer with forced chunked send
    A stress test reproducer. For every socket send, change sending to break up
    data in small chunks. This forecedly changes chunking of the data stream.
    The goal is to test code for robustness in face of unexpected chunking of the data.
    WARNING: Only for testing purposes, not to be merged!
    b37819f86d
  3. DrahtBot commented at 8:59 AM on November 2, 2026: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--006a51241073e994b41acfe9ec718e94-->

    External sites

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline and AI policy for information on the review process. A summary of reviews will appear here.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  4. DrahtBot added the label CI failed on Nov 2, 2026
  5. DrahtBot commented at 10:08 AM on November 2, 2026: contributor

    <!--85328a0da195eb286784d51f73fa0af9-->

    🚧 At least one of the CI tasks failed. <sub>Task iwyu: https://github.com/bitcoin/bitcoin/actions/runs/41417100283/job/125183540072</sub> <sub>LLM reason (✨ experimental): CI failed because IWYU required adding <thread> to src/util/sock.cpp, leaving a diff that the include-check rejects.</sub>

    <details><summary>Hints</summary>

    Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:

    • Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

    • A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.

    • An intermittent issue.

    Leave a comment here, if you need help tracking down a confusing failure.

    </details>

  6. optout21 commented at 10:38 AM on November 2, 2026: contributor

    CI runs have completed, with some failures. An issue has been raised to document the failed tests -- #36726 . Closing this PR (was not intended for merging, kept for reference).

  7. optout21 closed this on Nov 2, 2026


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-11-03 23:51 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me