Remove the sleep-based lease setup from the two Redis reaper tests
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
- 難度
- 2/5
- 預估耗時
- 1-3 小時
- 新手友好度
- 92/100
- Issue 類型
- 重構
- 描述清晰度
- 描述清楚
- 活躍度
- 活躍
- 技術堆疊
- python, redis
- 領域
- backend, testing-qa
研究方向
從 tests/backends/test_redis.py 中的 test_running_reaper__fails_expired_tasks 和 test_stale_acknowledge__is_noop 開始。閱讀模組層級的 helper _expire_lease(backend, task_id) 以及使用它的現有 reap 測試，然後替換兩個測試中基於 sleep 的過期設定。執行上述兩個測試，並確認它們在不使用 1.1 秒 sleep 的情況下通過。
由索引模型根據 Issue 內容生成。
描述
Found during the SuperJoe review of the lease-expiry retry patch (4ec9068..d5a1fdb). It is out of the scope of that patch because both tests predate it.
tests/backends/test_redis.py still expires a lease with a sleep of 1.1 seconds:
# tests/backends/test_redis.py
time.sleep(1.1)
# Run the broker to reap the running set
broker = RedisBroker(backend)
broker.main()
The two tests are test_running_reaper__fails_expired_tasks and test_stale_acknowledge__is_noop.
The patch added the module-level _expire_lease(backend, task_id) helper. The helper sets the running-set score back to 0. Every new reap test already uses it. Both old tests can use it too. Then the suite loses about 2.2 seconds of wall clock and the risk of a timing flake.
- 主要語言
- Python
- 星號
- 21
- 分支
- 2
- 平均合併
- 12 小時 39 分鐘
- 30 天內合併 PR
- 27
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 沒有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
相似的 Issue
-
難度 2/5 1-3 小時 新手友好度 90/100
SublimeText/AdvancedNewFile#211 ·
-
難度 2/5 1-3 小時 新手友好度 73/100
-
難度 2/5 1-3 小時 新手友好度 83/100
Imbad0202/academic-research-skills#987 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 62/100
-
Pull-through from gcr.io returns 500 when pulling using podman/buildah可能已有人在做 關聯的 PR 仍在進行中或已合併。 未關閉Issue Triage-Needed
難度 2/5 1-3 小時 新手友好度 68/100
pulp/pulp_container#2581 ·
維護者通常 1 天內回覆