fluentd log driver: default retry wait is passed in nanoseconds
メンテナーはふだん 2 日以内に返信
評価
調査の方向性
The bug is in pkg/logging/fluentd_logger.go, where defaultRetryWait is converted with int(defaultRetryWait) instead of milliseconds, and the existing tests in the same package assert that wrong value. Start by reading how retryWait feeds the fluent-logger Config and the fluentd-retry-wait handling that already uses Milliseconds(). Done means the default is passed in milliseconds and the tests check that value. Pull request #5321 is already open against this issue, so check it before starting.
索引モデルが issue の本文から書いたものです。
説明
In pkg/logging/fluentd_logger.go the default is defaultRetryWait = 1000 * time.Millisecond and retryWait := int(defaultRetryWait), which is 1000000000. fluent-logger-golang reads Config.RetryWait as milliseconds, and the code already converts a user-set fluentd-retry-wait with temp.Milliseconds().
Without fluentd-retry-wait the logger therefore gets about 11 days of retry wait. Its backoff caps the wait at MaxRetryWait (60000 ms), so a reconnect waits a minute instead of the intended one second.
Using defaultRetryWait.Milliseconds() fixes it. The existing tests compared against int(defaultRetryWait), so they matched the bug. I have a patch that changes them to check the value in milliseconds.
- 主要言語
- Go
- スター
- 10.5k
- フォーク
- 839
- 平均マージ
- 3日 18時間
- マージ済み PR（30日）
- 40
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
containerd/nerdctl のほかの issue
-
Test and examples: update wordpress images (to silence potential false security alerts)再び着手できるかも @ayushpratap27 が 15 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンarea/ci documentation priority/low
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
containerd/nerdctl#5283 · コメント 2 件 ·
メンテナーはふだん 2 日以内に返信
-
`nerdctl kill` shouldn't set `containerd.io/restart.explicitly-stopped` label for non-stop signals再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
containerd/nerdctl#5171 · コメント 3 件 ·
メンテナーはふだん 2 日以内に返信
-
`nerdctl kill -s <signal>` removes the network of a container that continues to run対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
containerd/nerdctl#5327 ·
メンテナーはふだん 2 日以内に返信
-
`nerdctl stop` on an exited container frees its IP but keeps its /etc/hosts entry, so a later container can lose its name対応中かも @papidb が 2 日前に担当しました。 オープンbug
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
containerd/nerdctl#5323 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
compose: only the last volume's create_host_path directories are created対応中かも @ogulcanaydogan が 5 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 25/100
containerd/nerdctl#5317 ·
メンテナーはふだん 2 日以内に返信
containerd/nerdctl の issue をすべて見る
似ている issue
-
[Bug] Custom tool truncation leaves unused stream copies open対応中かも @OUAO-FRANK が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
[v3] KeyBindingManager calls (Add/Remove) are not converted into a case-insensitive accelerator.KeyオープンBug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
crmne/hyprmoncfg#114 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
runtime: usleep sleeps nanoseconds instead of microseconds on linux/mips and linux/mips64対応中かも @BRobertson7 が今日担当しました。 オープンcompiler/runtime
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100