@abronan @aluzzardi please take a look.

I suggested the same in docker-compose; perhaps also add a label for the version of swarm that's used to set the labels; this allows for migrating existing labels/containers in case the format changes in future.

(Reading on my phone, so it's possible its in there already, but I overlooked)

LGTM

@thaJeztah Yes that would be a good idea, overall I like the idea of interfering with ContainerConfig creation to inject labels needed by Swarm. This way we can inject a label for the Swarm version, one for the container Virtual ID, etc..

I think the store (RequestedState) should be using a cluster.ContainerConfig so we serialize any extra data we might have

I think the store (RequestedState) should be using a cluster.ContainerConfig so we serialize any extra data we might have

And

ClusterConfig is a superset of dockerclient.ContainerConfig.
It contains everything the standard docker configuration contains plus additional fields such as constraints.
ClusterConfig manages to translation to/from dockerclient.ContainerConfig and handles backward compatibility.

👍

I switched to 2 keys com.docker.swarm.affinities and com.docker.swarm.constraints and added a bunch of tests

@aluzzardi can you please take another look and comment on the PR if I forgot something ?

LGTM

There's just a tiny golint error

Is there any docs on this?