testcontainers / testcontainers/testcontainers-java

Log the exception before swallowing it

Abierto
#4,859 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

resolution/waiting-for-info
Lenguaje dominante
Java
Estrellas
8.7k
Forks
1.9k
Merge medio
2 d 17 h
PR fusionados (30 d)
9

Descripción

Here the exception is swallowed without any logs. That makes debugging confusing. In my case, the test failed with the error "Docker must be present blah blah", but the actual issue was disk space problem.
Another solution could be to change the level of log in DockerClientFactory#checkDiskSpace to warning.
Let me know if you need the PR.

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza en core/src/main/java/org/testcontainers/DockerClientFactory.java, en el código enlazado alrededor de la línea 133; después, inspecciona DockerClientFactory#checkDiskSpace y las convenciones de logging cercanas. Confirma cómo se ignora la excepción y qué nivel de log se utiliza actualmente. Se considera terminado cuando el fallo de espacio en disco sea visible en los logs en lugar de producir únicamente el mensaje engañoso de que Docker está presente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
docker, java
Área
infrastructure, testing
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
38/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.