Repository navigation
#374 を実装する(スプリント m8d) - #487
Merged
Merged
Conversation
数え方(1 秒休む巡の回数・上限 60)は変えない。StartupFailure に elapsed(既定 0)を足し、 単調な時計で測った経過を整数の秒で入れる。時計を差し替えたテストを足した。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
takemi-ohama
marked this pull request as ready for review
October 14, 2026 01:17
This was referenced Oct 14, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
devbaseの起動でコンテナを待つ処理が時間切れになったとき、文言に待ちの巡の回数と実際に経過した秒数を出すようにしました(#374)。新しい文言は確定仕様(docs/specifications/post-start.md)と CHANGELOG にも書いています。この変更はスプリント m8d のブランチへ集めます。利用者向けの変化
移行の手順
変更の要点
lib/devbase/utils/docker.py: 時間切れの文言に巡の回数と実際の経過の秒数を含めるようにしました。tests/utils/test_wait_for_containers.py: 新しい文言を確かめるテストを足しました。docs/specifications/post-start.md: 確定仕様に時間切れの新しい文言を書きました。CHANGELOG.md: 新しい文言を書きました。テスト
設計と違う点
概要
devbase up/devbase scaleの起動の待ちで時間切れになった番号の行を、timeout (60s) ...からtimeout after 60 rounds (73s elapsed) waiting for the entrypoint to completeの形へ変えた(#374)。2026-10-13 の利用者の決定に従い、数え方(1 秒休む巡の回数・上限 60)と打ち切りの条件は変えていない。
変えたこと
StartupFailureにelapsed: int = 0を足した(既定値つき)。待ちを始めてから時間切れが決まるまでの秒数をtime.monotonicで測り、端数を切り捨てた整数で入れるready・failures、引数と戻り値は変えていないdocs/specifications/post-start.mdの文言の表・例・数え方の説明・クラス図を合わせたCHANGELOG.mdの[Unreleased]の Fixed に書いた設計との違い
設計なし(要求の受け入れ条件どおり)。課題の「直す場所」にある「経過時間で打ち切る」は、利用者の決定により行っていない。
確かめたこと
time.sleepを差し替えたテストを 2 件足した(docker の呼び出し 1 回 0.65 秒で 60 巡 →138s elapsed、端数の切り捨て)。
test_rounds_do_not_exceed_the_limitは変えずに通るenv -u DEVBASE_ROOT python3 -m pytest -q tests: 4389 passed, 32 skippeddevbase upは打っていない(作業の制約)残した事項
モード: standard / 通した工程: 実装 → 完了判定 → Pull Request
🤖 Generated with Claude Code