testcontainers / testcontainers/testcontainers-java

[Bug]: [MariaDB] getLivenessCheckPortNumbers incorrectly implemented

Offen
#10,359 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

type/bug
Vorherrschende Sprache
Java
Sterne
8.7k
Forks
1.9k
Ø Merge
2 T. 17 Std.
Gemergte PRs (30 T.)
9

Beschreibung

Module

MariaDB

Testcontainers version

1.21.1

Problem at least present since 1.16.0

Using the latest Testcontainers version?

Yes

What happened?

https://github.com/testcontainers/testcontainers-java/blob/7d8301903a6b45591605fd376702d24fc878a61d/modules/mariadb/src/main/java/org/testcontainers/containers/MariaDBContainer.java#L64-L66

The MAPPED port is NOT returned, which causes the HostPortWaitStrategy to fail.

Why does this method exists in the first place? I couldn't find why it even exists. The super Method should work just fine.

Additional Information

This is part of a general performance regression in JDBC containers (waitUntilContainerStarted), that I'm currently addressing. I will create some follow up issues too describe the problem/s in more detail soon.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne in modules/mariadb/src/main/java/org/testcontainers/containers/MariaDBContainer.java bei getLivenessCheckPortNumbers und vergleiche die Methode anschließend mit der Methode der Superklasse und dem Verhalten von HostPortWaitStrategy. Bestätige, dass der gemappte Port konsistent behandelt wird, und überprüfe, dass waitUntilContainerStarted für den MariaDB-Container erfolgreich ausgeführt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
docker, java, mariadb
Bereich
databases, testing
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.