Add etcd integrations tests

[mrjana]

- Added etcd integration test for overlay
- Added etcd integration test for multinode
  with mock test driver suitable for circleci
- Added multinode tests for zookeeper
- Made the script smart enough to only start
  data stores necessary for the requested suites

Signed-off-by: Jana Radhakrishnan mrjana@docker.com

[chenchun]

Looks good than my PR. Closed mine.

[abronan]

Maybe a small nit on the port choice (why 42000 and not 4001?).

But LGTM. Great to have tests for the 3 stores now.

[tomdee]

Yes big 👍 to having tests for all three stores. Thanks for getting these tests in.

[mrjana]

@abronan I chose 42000 since 4001 is a common etcd port which folks might be using in their dev machine (if they are playing with etcd separately) and we start the etcd container in host namespace. So just to avoid that case

[mavenugo]

thanks @mrjana . huge 👍 from me. LGTM.

[abronan]

@mrjana Ok makes sense ;) then you might probably want to change the consul port to something else than 8500 to make sure it does not end up to the state you described.

[mrjana]

@abronan Yeah that makes sense too but since I start consul in bridge network somehow didn't pay attention to the port but we are really mapping container port 8500 to host port 8500. Will do that in another PR. Thanks :-)