testcontainers / testcontainers/testcontainers-java

Log the exception before swallowing it

Aperta
#4,859 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

resolution/waiting-for-info
Lingua principale
Java
Stelle
8.7k
Fork
1.9k
Merge medio
2g 17h
PR unite (30g)
9

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia in core/src/main/java/org/testcontainers/DockerClientFactory.java, nel codice collegato intorno alla riga 133, quindi esamina DockerClientFactory#checkDiskSpace e le convenzioni di logging nelle vicinanze. Conferma come l’eccezione venga ignorata e quale livello di log venga attualmente utilizzato. Il lavoro è completato quando il problema relativo allo spazio su disco è visibile nei log invece di produrre soltanto il messaggio fuorviante che Docker è presente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
docker, java
Ambito
infrastructure, testing
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
38/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.