[ci/docker] stop removing containers to avoid disrupting chrome (#85425)

Co-authored-by: spalger <spalger@users.noreply.github.com>
This commit is contained in:
Spencer 2020-12-09 12:23:38 -07:00 committed by GitHub
parent 159eab7f30
commit 075810bcd3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -124,7 +124,11 @@ export class DockerServersService {
lifecycle.cleanup.add(() => {
try {
execa.sync('docker', ['kill', containerId]);
execa.sync('docker', ['rm', containerId]);
// we don't remove the containers on CI because removing them causes the
// network list to be updated and aborts all in-flight requests in Chrome
if (!process.env.CI) {
execa.sync('docker', ['rm', containerId]);
}
} catch (error) {
if (
error.message.includes(`Container ${containerId} is not running`) ||