Craftadoc/clsi
2
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #141 from das7pad/hotfix-container-deletion-locking

Browse Source 
[DockerRunner] destroyOldContainers: normalize the container name
This commit is contained in:
Jakob Ackermann
2020-05-29 12:31:50 +02:00
committed by GitHub
parent 7254a025ae f8cb5e36af
commit 985bbf27c9
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/js/DockerRunner.js
View File

@@ -632,6 +632,9 @@ module.exports = DockerRunner = {
ttl ttl
) { ) {
if (name.slice(0, 9) === '/project-' && ttl <= 0) { if (name.slice(0, 9) === '/project-' && ttl <= 0) {
// strip the / prefix
// the LockManager uses the plain container name
name = name.slice(1)
return jobs.push(cb => return jobs.push(cb =>
DockerRunner.destroyContainer(name, id, false, () => cb()) DockerRunner.destroyContainer(name, id, false, () => cb())
) )

6
test/unit/js/DockerRunnerTests.js
View File

@@ -630,19 +630,19 @@ describe('DockerRunner', function() {
it('should destroy old containers', function() { it('should destroy old containers', function() {
this.DockerRunner.destroyContainer.callCount.should.equal(1) this.DockerRunner.destroyContainer.callCount.should.equal(1)
return this.DockerRunner.destroyContainer return this.DockerRunner.destroyContainer
.calledWith('/project-old-container-name', 'old-container-id') .calledWith('project-old-container-name', 'old-container-id')
.should.equal(true) .should.equal(true)
}) })
it('should not destroy new containers', function() { it('should not destroy new containers', function() {
return this.DockerRunner.destroyContainer return this.DockerRunner.destroyContainer
.calledWith('/project-new-container-name', 'new-container-id') .calledWith('project-new-container-name', 'new-container-id')
.should.equal(false) .should.equal(false)
}) })
it('should not destroy non-project containers', function() { it('should not destroy non-project containers', function() {
return this.DockerRunner.destroyContainer return this.DockerRunner.destroyContainer
.calledWith('/totally-not-a-project-container', 'some-random-id') .calledWith('totally-not-a-project-container', 'some-random-id')
.should.equal(false) .should.equal(false)
}) })