diff options
author | gui-yue <yuemeng.gui@gmail.com> | 2025-05-28 01:17:04 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-05-27 17:17:04 +0000 |
commit | 07468a0f1c6ef07fe056e6a77347d045d34c69ff (patch) | |
tree | ef14805684c524a70767eae65a7c5faa1d42be16 /scripts-dev/build_debian_packages.py | |
parent | fix(device-handler): make _maybe_retry_device_resync thread-safe (#18391) (diff) | |
download | synapse-07468a0f1c6ef07fe056e6a77347d045d34c69ff.tar.xz |
Increase timeout for test_lock_contention on RISC-V (#18430)
This PR addresses a test failure for `tests.handlers.test_worker_lock.WorkerLockTestCase.test_lock_contention` which consistently times out on the RISC-V (specifically `riscv64`) architecture. The test simulates high lock contention and has a default timeout of 5 seconds, which seems sufficient for architectures like x86_64 but proves too short for current RISC-V hardware/environment performance characteristics, leading to spurious `tests.utils.TestTimeout` failures. This fix introduces architecture detection using `platform.machine()`. If a RISC-V architecture is detected: * The timeout for this specific test is increased (e.g., to 15 seconds ). The original, stricter timeout (5 seconds) and lock count (500) are maintained for all other architectures to avoid masking potential performance regressions elsewhere. This change has been tested locally on RISC-V, where the test now passes reliably, and on x86_64, where it continues to pass with the original constraints. --- ### Pull Request Checklist <!-- Please read https://element-hq.github.io/synapse/latest/development/contributing_guide.html before submitting your pull request --> * [X] Pull request is based on the develop branch *(Assuming you based it correctly)* * [X] Pull request includes a [changelog file](https://element-hq.github.io/synapse/latest/development/contributing_guide.html#changelog). *(See below)* * [X] [Code style](https://element-hq.github.io/synapse/latest/code_style.html) is correct (run the [linters](https://element-hq.github.io/synapse/latest/development/contributing_guide.html#run-the-linters)) *(Please run linters locally)*
Diffstat (limited to 'scripts-dev/build_debian_packages.py')
0 files changed, 0 insertions, 0 deletions