test: Avoid interface_ipc.py Duplicate ID errors #33420

pull ryanofsky wants to merge 1 commits into bitcoin:master from ryanofsky:pr/ipc-pyimport changing 1 files +6 −1
  1. ryanofsky commented at 4:39 PM on September 17, 2025: contributor

    This change should fix issue #33417 reported by zaidmstrr. It's possible to reproduce the mp/proxy.capnp:0: failed: Duplicate ID [@0xcc316e3f71a040fb](/bitcoin-bitcoin/contributor/0xcc316e3f71a040fb/) error by installing libmultiprocess system-wide, or to one of the locations listed in the python test's imports list before the local libmultiprocess subtree, and then running the test.

  2. test: Avoid interface_ipc.py Duplicate ID errors
    This change should fix issue https://github.com/bitcoin/bitcoin/issues/33417
    reported by zaidmstrr. It's possible to reproduce the `mp/proxy.capnp:0:
    failed: Duplicate ID @0xcc316e3f71a040fb` error by installing libmultiprocess
    system-wide, or to one of the locations listed in the python test's `imports`
    list before the local libmultiprocess subtree, and then running the test.
    e9c52272eb
  3. DrahtBot added the label Tests on Sep 17, 2025
  4. DrahtBot commented at 4:40 PM on September 17, 2025: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33420.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK zaidmstrr
    Concept ACK Sjors

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

    LLM Linter (✨ experimental)

    Possible typos and grammar issues:

    • and there are not "failed: Duplicate ID @0xcc316e3f71a040fb" errors. -> and that there are no "failed: Duplicate ID @0xcc316e3f71a040fb" errors. [the clause needs "that" and "no" to be grammatical and clear]

    <sup>drahtbot_id_5_m</sup>

  5. zaidmstrr commented at 4:46 PM on September 17, 2025: contributor

    Tested ACK e9c5227

    This fixes the issue and is also mentioned here.

  6. Sjors commented at 5:07 PM on September 17, 2025: member

    Concept ACK

    That said, I'm unable to reproduce the issue on macOS. I installed libmultiprocess using (sudo) make install (https://github.com/bitcoin-core/libmultiprocess/blob/master/doc/install.md). capnp is installed via Homebrew.

  7. fanquake added the label Needs backport (30.x) on Sep 18, 2025
  8. fanquake commented at 9:53 AM on September 18, 2025: member

    I also haven't reproduced the issue, using the steps provided. i.e installing libmulitprocess system-wide in a new VM, and then building Core & running interface_ipc.py. Can someone list the steps to reach the failure, using a clean VM.

  9. zaidmstrr commented at 10:06 AM on September 18, 2025: contributor

    Here are the steps to reproduce the error in Ubuntu based systems:

    1. Install libmultiprocess library system using sudo make install
    2. Then install capnp system-wide. You can refer this for the installation. After succesfully downloading the capnp library you can check using the which capnp command, which outputs the default location of the installed binary. In most of the cases the location was /usr/local/bin/capnp.
    3. Now run the interface_ipc.py test.
  10. fanquake commented at 10:19 AM on September 18, 2025: member

    Thanks, repro'd:

    Remaining jobs: [interface_ipc.py]
    1/1 - interface_ipc.py failed, Duration: 1 s
    
    stdout:
    2025-09-18T10:18:46.257357Z TestFramework (INFO): PRNG seed is: 5267162394658679552
    2025-09-18T10:18:46.257836Z TestFramework (INFO): Initializing test directory /tmp/test_runner_₿_🏃_20250918_101846/interface_ipc_0
    
    
    stderr:
    terminate called after throwing an instance of 'kj::ExceptionImpl'
      what():  mp/proxy.capnp:0: failed: Duplicate ID [@0xcc316e3f71a040fb](/bitcoin-bitcoin/contributor/0xcc316e3f71a040fb/).
    
    
    
    TEST             | STATUS    | DURATION
    
    interface_ipc.py | ✖ Failed  | 1 s
    
    ALL              | ✖ Failed  | 1 s (accumulated) 
    Runtime: 1 s
    
  11. fanquake merged this on Sep 18, 2025
  12. fanquake closed this on Sep 18, 2025

  13. fanquake referenced this in commit 56fb74ee89 on Sep 18, 2025
  14. fanquake removed the label Needs backport (30.x) on Sep 18, 2025
  15. fanquake commented at 10:36 AM on September 18, 2025: member

    Backported to 30.x in #33424.

  16. ryanofsky commented at 10:50 AM on September 18, 2025: contributor

    I also haven't reproduced the issue, using the steps provided. i.e installing libmulitprocess system-wide in a new VM, and then building Core & running interface_ipc.py.

    Yeah, an additional requirement for the bug to happen is that cap'nproto and libmultiprocess need to be installed in the same prefix. If you are using a system provided cap'n proto installed in /usr and a locally build libmultiprocess installed in /usr/local the bug won't happen. They both need to be installed in /usr or /usr/local or other prefix.

  17. fanquake referenced this in commit 4b02bc1a72 on Sep 23, 2025
  18. fanquake referenced this in commit 72c1f13c33 on Sep 23, 2025
  19. sedited referenced this in commit 3f14dd4b2a on Oct 7, 2025
  20. sedited referenced this in commit 5f9d179b8d on Oct 8, 2025
  21. sedited referenced this in commit 845b93d99e on Oct 8, 2025
  22. yuvicc referenced this in commit ccce70c31a on Oct 8, 2025
  23. sedited referenced this in commit 3b1e26d5d3 on Oct 10, 2025
  24. sedited referenced this in commit 0fa2fdaca0 on Oct 11, 2025
  25. stringintech referenced this in commit f74b275ffc on Oct 14, 2025
  26. stickies-v referenced this in commit a785fe6759 on Nov 5, 2025
  27. morozow referenced this in commit 4ae173cd64 on May 8, 2026
  28. morozow referenced this in commit 6ae5e123cb on May 8, 2026
  29. Kino1994 referenced this in commit 2c5cf87194 on Jun 28, 2026
  30. BigcoinBGC referenced this in commit 080a8c35f2 on Jun 30, 2026
  31. Kino1994 referenced this in commit bfe1c0d4ad on Aug 19, 2026
  32. bitcoin locked this on Sep 18, 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-09-20 20:53 UTC

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