atomicapp icon indicating copy to clipboard operation
atomicapp copied to clipboard

Should the Docker provider remove the deployed containers on failure?

Open cdrage opened this issue 10 years ago • 7 comments

When deploying using the Docker provider, you may run into the issues of a container already being deployed on a current port.

Unfortunatley, even if it fails, the containers you deployed up until that point are lingering on.

Should the Docker provider go through all the deployed containers thus far and remove them? Or is this too dangerous?

cdrage avatar Nov 24 '15 21:11 cdrage

we will be adding some tooling around "removing" containers sometime soon as part of https://github.com/projectatomic/atomicapp/pull/308 .. Let's revisit after we have the full lifecyle in place..

We may decide that if a part of it fails then it is reasonable to have the user do a clean or equivalent before they start to run it again.

dustymabe avatar Nov 24 '15 21:11 dustymabe

It's a generic problem, and not just for Docker. As we had already discussed in the past, IIRC, deployments should be atomic. If the deployment of a component fails, we should rollback the deployment of any other deployed component.

rtnpro avatar Nov 25 '15 13:11 rtnpro

@cdrage this is also #319

dustymabe avatar Nov 25 '15 13:11 dustymabe

Ok let's get it in for GA then

dustymabe avatar Nov 25 '15 14:11 dustymabe

This issue should be renamed to Atomic deployments, I think :)

rtnpro avatar Dec 15 '15 06:12 rtnpro

Fixed in #456

rtnpro avatar Feb 09 '16 15:02 rtnpro

Removing the blocker label from this. We won't be doing this before GA for several reasons.

  • #456 is a risky change that is a "nice to have"
  • There are issues with removing a docker container from within atomicapp prior to docker 1.9

dustymabe avatar Feb 11 '16 16:02 dustymabe