WEB: 暖気処理のステータスが処理中から変化しないことがある

[xsekiguchi]

closes #2198 .

対応内容

• 暖気処理で発生した例外は Throwable でキャッチする
• 例外をキャッチした場合、ステータスを FAILED に変更する

動作確認・スクリーンショット（任意）

• 暖気処理が終了し、 /warmupStatus を確認
  • すべて正常終了した場合は、 complete (200) になることを確認（status: COMPLETE）
  • Exception がスローされた場合は、 failed (500) になることを確認（status: FAILED）
  • Error がスローされた場合は、 failed (500) になることを確認（status: FAILED）

レビュー観点・補足情報（任意）

• Error がスローされた場合に想定通りの動作となるように修正しています。
• それ以外は、現状と変わらない動作を行います。

[Copilot]

Pull request overview

本PRは Issue #2198「暖気処理中に Error 派生例外がスローされるとステータスが PROCESSING から変化しない」問題に対応し、ウォームアップ非同期タスクのエントリポイントで Throwable を捕捉して失敗時にステータスを FAILED に遷移させることで、処理中ステータスのまま取り残されるケースを防ぐものです。

Changes:

• ウォームアップ実行エントリポイントの例外捕捉を Exception → Throwable に拡張
• 例外捕捉時にステータスを FAILED に変更する処理を追加
• 上記挙動をJavaDocへ追記

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

iplass-web/src/main/java/org/iplass/mtp/impl/web/WarmupStatusServlet.java:179

• [must] catch (Throwable e) 内で changeStatus(FAILED) の取得/更新処理が ServiceConfigrationException などの別例外を投げる可能性があり、その場合に元の Throwable がマスクされてしまいます（結果として原因調査が困難になり、ステータス更新にも失敗し得ます）。ステータス更新は別途 try/catch で保護してログに落とし、元の例外は必ず再スローしてください。併せて、instanceof 後のキャストはパターンマッチングを使うとガイドラインに沿います。

			} catch (Throwable e) {
				logger.error("Warmup failed.", e);
				ServiceRegistry.getRegistry()
						.getService(WarmupService.class)
						.changeStatus(WarmupStatus.FAILED);