Feature: Ryuk support in ComposeContainer class (like in Java)
Maintainer antworten meist innerhalb von 12 Tagen
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 68/100
- Issue-Typ
- Feature
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- docker, docker-compose, python
- Bereich
- devops, testing-qa
Rechercherichtung
Beginne in src/testcontainers/compose/compose.py, insbesondere bei der ComposeContainer-Klasse rund um die referenzierte start()-Methode, und vergleiche sie mit den verknüpften Implementierungen von Java ComposeContainer und ComposeDelegate. Verfolge zunächst die bestehenden APIs zur Registrierung von Python Ryuk, bevor du entscheidest, wie Labels für Compose-Projekte registriert werden sollten. Erledigt ist die Aufgabe, wenn von ComposeContainer gestartete Container nach Abstürzen oder erzwungenen Stopps sicher bereinigt werden, mit entsprechender Abdeckung dieses Verhaltens.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
What are you trying to do?
When using testcontainers for Python in a CI/CD environment containers started using the ComposeContainer class are not stopped in crash or force-stop scenarios. The same class in Java cleaned up the started Containers via Ryuk.
It would be nice if the Python class ComposeContainer provides the same functionality of cleaning up the containers it started in a safe manner. (for example with Ryuk)
Safe cleanup of containers seems to be a core feature of the other *Container classes and is missing for compose files using the ComposeContainer class.
Here is a brief summary of how it is implemented in Java and potential changes:
ComposeContainer class in Java:
In the "start()" method "registerContainersForShutdown" is called. See:
In "registerContainersForShutdown"
the line ".registerLabelsFilterForCleanup(Collections.singletonMap("com.docker.compose.project", project));" effectivly adds a label filter to Ryuk using the official Docker property com.docker.compose.project with a random project ID. (see constructor of ComposeDelegate for "project" property)
Python doesn't have this functionality, see https://github.com/testcontainers/testcontainers-python/blob/a3476488ac1f7526a9c22d65251c8639d7089326/src/testcontainers/compose/compose.py#L84 for the analog of the ComposeContainer class in Python and https://github.com/testcontainers/testcontainers-python/blob/a3476488ac1f7526a9c22d65251c8639d7089326/src/testcontainers/compose/compose.py#L318 for the "start()" method that has no code to interact with Ryuk.
Why should it be done this way?
Please provide the same or similar functionality to allow safe usage of ComposeContainer class like in Java.
Thank you.
- Vorherrschende Sprache
- Python
- Sterne
- 2.3k
- Forks
- 390
- Ø Merge
- 11 T. 16 Std.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus testcontainers/testcontainers-python
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
testcontainers/testcontainers-python#1086 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 12 Tagen
-
Bug: [minio] _healthcheck calls for nested wait_container_is_ready resulting in long wait timeOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
testcontainers/testcontainers-python#578 · 5 Kommentare ·
Maintainer antworten meist innerhalb von 12 Tagen
-
Bug: `Reaper.delete_instance()` raises 409 and keeps the dead reaper when ryuk was killed from outsideEvtl. vergeben @RitiGrover hat das vor 12 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 76/100
testcontainers/testcontainers-python#1125 ·
Maintainer antworten meist innerhalb von 12 Tagen
-
Bug: MongoDB container is reported ready before the server accepts connectionsEvtl. vergeben @RitiGrover hat das vor 16 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 65/100
testcontainers/testcontainers-python#1122 ·
Maintainer antworten meist innerhalb von 12 Tagen
-
Bug: Reaper reports success while the session filter never reaches Ryuk (silent leak on Linux/docker-proxy)Evtl. vergeben @RitiGrover hat das vor 16 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
testcontainers/testcontainers-python#1114 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 12 Tagen
Alle Issues in testcontainers/testcontainers-python
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
EvaluationSuite.run fails with default args_for_task and mutates supplied kwargsEvtl. vergeben @ktz03 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
huggingface/evaluate#825 ·
Maintainer antworten meist innerhalb von 1 Tag
-
dependencies feature github_actions good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
wemake-services/wemake-django-template#3149 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[request] vsg/1.1.16Offenupstream update
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
conan-io/conan-center-index#31142 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:core bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag