testcontainers / testcontainers/testcontainers-java

Use of links between containers should be deprecated

Offen
#465 17 Kommentare 4 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

As discussed in #463, docker networks seem mature enough to use, and links are officially not recommended any more.

We'd like to deprecate use of links; off the top of my head I believe we ought to do the following things:

  • Obviously, mark the container addLink method as deprecated with a clear comment
  • Links aren't currently documented very well, but update the docs with a clear enough description of the networking support, so that ad-hoc linking can be done easily by users
  • Modify Webdriver VNC recorder sidecar container to remove use of links
  • Modify Docker Compose support; can we remove use of the ambassador container now?

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 damit, die Unterstützung für Docker Compose und deren Verwendung des ambassador-Containers nachzuverfolgen; untersuche anschließend die zugehörigen Tests und das aktuelle Netzwerkverhalten. Ermittle, ob der ambassador entfernt werden kann, ohne auf Compose basierende Tests zu beschädigen; abgeschlossen ist die Aufgabe, wenn dort keine links mehr erforderlich sind und die bestehende Compose-Unterstützung weiterhin durch erfolgreiche Tests abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
docker, docker-compose, java
Bereich
devops, infrastructure, testing
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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