This fix tries to clean up the containers during the tests
so that no pre-existing volumes left in docker integration tests.
This fix adds `-v` when removing containers, and makes sure
containers launched in non-daemon mode are removed.
This fix is realted to moby PR 36292
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
This fix adds the volume id to the list so that it could be
cleaned up on test teardown.
The issue was originally from https://github.com/moby/moby/pull/36292
where an additional `somevolume` pre-exists in tests.
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Seems like this is pretty much ignored by Docker, so it wasn't
causing any visible issues, except when a volume name was used
instead of a path.
Also, added integration tests.
Ref #1380
Signed-off-by: Ben Firshman <ben@firshman.co.uk>