Merge pull request #9884 from Fs00/autoremove-containers

Autoremove containers launched by DockerContainerRunner

.changeset/grumpy-apes-repeat.md

@@ -0,0 +1,5 @@
+---
+'@backstage/backend-common': patch
+---
+
+DockerContainerRunner.runContainer now automatically removes the container when its execution terminates

packages/backend-common/src/util/DockerContainerRunner.test.ts

@@ -115,6 +115,7 @@ describe('DockerContainerRunner', () => {
         Env: envVarsArray,
         WorkingDir: workingDir,
         HostConfig: {
+          AutoRemove: true,
           Binds: expect.arrayContaining([
             `${path.join(rootDir, 'input')}:/input`,
             `${path.join(rootDir, 'output')}:/output`,
@@ -207,6 +208,7 @@ describe('DockerContainerRunner', () => {
       logStream,
       expect.objectContaining({
         HostConfig: {
+          AutoRemove: true,
           Binds: [],
         },
         Volumes: {},

packages/backend-common/src/util/DockerContainerRunner.ts

@@ -105,6 +105,7 @@ export class DockerContainerRunner implements ContainerRunner {
       await this.dockerClient.run(imageName, args, logStream, {
         Volumes,
         HostConfig: {
+          AutoRemove: true,
           Binds,
         },
         ...(workingDir ? { WorkingDir: workingDir } : {}),