matrix-org / matrix-org/complement
container logs not flushed if there is a timeout spinning up a container
オープン
まだ誰も着手していません。
- 主要言語
- Go
- スター
- 99
- フォーク
- 72
- 平均マージ
- 4日 1時間
- マージ済み PR(30日)
- 8
説明
#392 added a ContainerStop to Deployer.Destroy, so that the container has a chance to gracefully shut down and flush its logs before we print them.
However, we print the logs in a couple of other situations:
- in
Deployer.Deploy, if there is a timeout waiting for the container to be ready. (In this case it looks like we don't destroy the container at all?) - in
Builder.ConstructBlueprint(viaBuilder.construct).
In both these cases, the logs are printed before the container is stopped.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず Builder.construct を介した Deployer.Deploy と Builder.ConstructBlueprint の処理を読み、次にそれらのログ処理を Deployer.Destroy とその ContainerStop の動作と比較します。ログがいつ出力されるかを確認するため、タイムアウト経路と構築経路を再現します。両方のケースでログが出力される前にコンテナが停止され、さらにタイムアウト経路でもコンテナのライフサイクルが正しく処理されれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- go
- 領域
- testing
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100