testcontainers / testcontainers/testcontainers-java
How to make sure there's no stray container not removed when using withDockerfileFromBuilder?
Nessuno ha ancora preso questa issue.
- Lingua principale
- Java
- Stelle
- 8.7k
- Fork
- 1.9k
- Merge medio
- 2g 17h
- PR unite (30g)
- 9
Descrizione
Hi guys,
On XWiki, and since the upgrade to TC 1.15.0 (might not be related but didn't notice it before), I've noticed on our CI agents, that there are some TC-created containers that are not always removed on agents. For example:
These containers are created with the following:
LOGGER.info("(*) Build a dedicated image embedding LibreOffice...");
// The second argument of the ImageFromDockerfile is here to indicate we won't delete the image
// at the end of the test container execution.
container = new XWikiLocalGenericContainer(new ImageFromDockerfile(imageName, false)
.withDockerfileFromBuilder(builder -> {
builder
.from(baseImageName)
.user("root")
.env("LIBREOFFICE_VERSION", officeVersion)
.env("LIBREOFFICE_DOWNLOAD_URL", "https://downloadarchive.documentfoundation.org/"
+ "libreoffice/old/$LIBREOFFICE_VERSION/deb/x86_64/"
+ "LibreOffice_${LIBREOFFICE_VERSION}_Linux_x86-64_deb.tar.gz")
// Note that we expose libreoffice /usr/local/libreoffice so that it can be found by
// JODConverter: https://bit.ly/2w8B82Q
.run("apt-get update && "
+ "apt-get --no-install-recommends -y install curl unzip procps libxinerama1 "
+ "libdbus-glib-1-2 libcairo2 libcups2 libsm6 && "
+ "rm -rf /var/lib/apt/lists/* /var/cache/apt/* && "
+ "wget --no-verbose -O /tmp/libreoffice.tar.gz $LIBREOFFICE_DOWNLOAD_URL && "
+ "mkdir /tmp/libreoffice && "
+ "tar -C /tmp/ -xvf /tmp/libreoffice.tar.gz && "
+ "cd /tmp/LibreOffice_${LIBREOFFICE_VERSION}_Linux_x86-64_deb/DEBS && "
+ "dpkg -i *.deb && "
+ "ln -fs `ls -d /opt/libreoffice*` /opt/libreoffice")
// Increment the image version whenever a change is brought to the image so that it can
// reconstructed on all machines needing it.
.label(OFFICE_IMAGE_VERSION_LABEL, imageVersion);
if (this.testConfiguration.getServletEngine() == ServletEngine.JETTY) {
// Create the right jetty user directory since it doesn't exist
builder.run("mkdir -p /home/jetty && chown jetty:jetty /home/jetty")
// Put back the user as jetty since it's a best practice to not execute the container as
// root.
.user("jetty");
}
builder.build();
}));
Would you have any idea what could be happening and what we could do?
Thanks a lot
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia da ServletContainerExecutor.java intorno alla riga 286 e segui l’uso di withDockerfileFromBuilder mostrato nell’issue. Riproduci lo scenario del CI-agent con il container indicato e analizza il comportamento di cleanup di Testcontainers. Il lavoro è completato quando hai identificato perché i container creati rimangono presenti e hai documentato o implementato una correzione di cleanup verificata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- docker, java
- Ambito
- devops, testing
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100