On startup, only attach to logs for running containers, and remove any containers marked as dead

Closes #611
Change-Type: patch
Signed-off-by: Pablo Carranza Velez <pablo@resin.io>

src/application-manager.coffee

@@ -266,6 +266,9 @@ module.exports = class ApplicationManager extends EventEmitter
 						delete @_containerStarted[step.current.containerId]
 						if step.options?.removeImage
 							@images.removeByDockerId(step.current.image)
+			remove: (step) =>
+				# Only called for dead containers, so no need to take locks or anything
+				@services.remove(step.current)
 			updateMetadata: (step, { force = false, skipLock = false } = {}) =>
 				skipLock or= checkTruthy(step.current.labels['io.resin.legacy-container'])
 				@_lockingIfNecessary step.current.appId, { force, skipLock: skipLock or step.options?.skipLock }, =>
@@ -678,6 +681,10 @@ module.exports = class ApplicationManager extends EventEmitter
 			# There is already a kill step in progress for this service, so we wait
 			return { action: 'noop' }
 
+		if current?.status == 'Dead'
+			# Dead containers have to be removed
+			return serviceAction('remove', current.serviceId, current)
+
 		needsDownload = !_.some availableImages, (image) =>
 			image.dockerImageId == target?.image or @images.isSameImage(image, { name: target.imageName })
 

src/compose/service-manager.coffee

@@ -78,6 +78,15 @@ module.exports = class ServiceManager extends EventEmitter
 	kill: (service, { removeContainer = true, wait = false } = {}) =>
 		@_killContainer(service.containerId, service, { removeContainer, wait })
 
+	remove: (service) =>
+		@logger.logSystemEvent(logTypes.removeDeadService, { service })
+		@get(service)
+		.then (existingService) =>
+			@docker.getContainer(existingService.containerId).remove(v: true)
+		.catchReturn(NotFoundError, null)
+		.tapCatch (err) =>
+			@logger.logSystemEvent(logTypes.removeDeadServiceError, { service, error: err })
+
 	getAllByAppId: (appId) =>
 		@getAll("io.resin.app-id=#{appId}")
 
@@ -278,5 +287,6 @@ module.exports = class ServiceManager extends EventEmitter
 	attachToRunning: =>
 		@getAll()
 		.map (service) =>
+			if service.status == 'Running'
 				@logger.logSystemEvent(logTypes.startServiceNoop, { service })
 				@logger.attach(@docker, service.containerId, service)

src/compose/service.coffee

@@ -429,6 +429,8 @@ module.exports = class Service
 			status = 'Running'
 		else if container.State.Status == 'created'
 			status = 'Installed'
+		else if container.State.Status == 'dead'
+			status = 'Dead'
 		else
 			status = 'Stopped'
 

src/lib/log-types.coffee

@@ -15,6 +15,13 @@ module.exports =
 		eventName: 'Service stop error'
 		humanName: 'Failed to kill service'
 
+	removeDeadService:
+		eventName: 'Remove dead container'
+		humanName: 'Removing dead container'
+	removeDeadServiceError:
+		eventName: 'Remove dead container error'
+		humanName: 'Error removing dead container'
+
 	downloadImage:
 		eventName: 'Docker image download'
 		humanName: 'Downloading image'