fluentd log driver: default retry wait is passed in nanoseconds
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 25/100
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Go
- Estrellas
- 10.5k
- Forks
- 839
- Merge medio
- 3 d 18 h
- PR fusionados (30 d)
- 40
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de containerd/nerdctl
-
Test and examples: update wordpress images (to silence potential false security alerts)Quizá libre de nuevo @ayushpratap27 la tomó hace 14 días y no hay ningún pull request abierto. Abiertoarea/ci documentation priority/low
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
containerd/nerdctl#5283 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
-
`nerdctl kill` shouldn't set `containerd.io/restart.explicitly-stopped` label for non-stop signalsQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
containerd/nerdctl#5171 · 3 comentarios ·
Los mantenedores suelen responder en 2 días
-
`nerdctl kill -s <signal>` removes the network of a container that continues to runPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
containerd/nerdctl#5327 ·
Los mantenedores suelen responder en 2 días
-
`nerdctl stop` on an exited container frees its IP but keeps its /etc/hosts entry, so a later container can lose its namePosiblemente ocupada @papidb la tomó hace 1 día. Abiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
containerd/nerdctl#5323 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
compose: only the last volume's create_host_path directories are createdPosiblemente ocupada @ogulcanaydogan la tomó hace 4 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
containerd/nerdctl#5317 ·
Los mantenedores suelen responder en 2 días
Todos los issues de containerd/nerdctl
Issues similares
-
Cluster details: the Control Plane version field has a different border color than its icon boxAbiertokind/design sig/ui
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
kubermatic/dashboard#8449 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
filecoin-project/curio#1632 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
-
compose: derived node options overwrite each other's targetsPosiblemente ocupada @yangjj-iso la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día