testcontainers / testcontainers/testcontainers-java
GenericContainer#isHealthy throws NPE if the healthcheck is not set
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 8.7k
- Forks
- 1.9k
- Ø Merge
- 2 T. 17 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Looks like health state is no longer part of the docker inspect response (api v1.40).
Looks like docker-java is mistakenly returning it in the response so we can't rely on that:
https://github.com/docker-java/docker-java/blob/066f268fa4f88687a9e7a41ca5eeaa85d63d8455/docker-java-api/src/main/java/com/github/dockerjava/api/command/InspectContainerResponse.java#L336-L340
Reproducer: https://github.com/testcontainers/testcontainers-java/pull/2204
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit GenericContainer#isHealthy und der im Issue beschriebenen Verarbeitung der Docker inspect-Antwort und führe anschließend den Reproducer in Pull Request #2204 aus oder untersuche ihn. Vergleiche das Verhalten, wenn kein healthcheck konfiguriert ist, mit der Antwort der Docker API v1.40. Erledigt bedeutet, dass GenericContainer#isHealthy in diesem Fall keine NullPointerException mehr auslöst.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- docker, java
- Bereich
- infrastructure, testing
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100