Skip to content

#374 を実装する(スプリント m8d) - #487

Merged
takemi-ohama merged 2 commits into
sprint/m8dfrom
feat/issue-374-m8d
Oct 14, 2026
Merged

takemi-ohama merged 2 commits into
sprint/m8dfrom
feat/issue-374-m8d

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

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 skipped
  • 実機での devbase up は打っていない(作業の制約)

残した事項

  • 経過時間で打ち切る形への変更(と制限の値の見直し)は行っていない

モード: standard / 通した工程: 実装 → 完了判定 → Pull Request

🤖 Generated with Claude Code

takemi-ohama and others added 2 commits October 14, 2026 10:13
数え方(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
takemi-ohama marked this pull request as ready for review October 14, 2026 01:17
@takemi-ohama
takemi-ohama merged commit 789fa5a into sprint/m8d Oct 14, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant