[18.02] revert attach: Use ContainerWait instead of Inspect

[andrewhsu]

revert:

• attach: Use ContainerWait instead of Inspect docker/cli#696 attach: Use ContainerWait instead of Inspect

to address failures like:

panic: DockerSuite.TestAttachAfterDetach test timed out after 5m0s [recovered]
	panic: DockerSuite.TestAttachAfterDetach test timed out after 5m0s

reverting git commits 3acd490 1a65d7d:

$ git revert 3acd490 1a65d7d

only conflict on cli/command/container/attach_test.go which i removed to resolve

[GordonTheTurtle]

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "qq" git@github.com:andrewhsu/docker-ce.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354562416
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

[thaJeztah]

LGTM for 18.02

[mlaventure]

Given that it didn't occur until now, there may be a regression somewhere else 😇

[thaJeztah]

@mlaventure Yes, we should look into that; I discussed with @andrewhsu, and given that the issue it's fixing was already in previous releases, we deemed it was better to revert for this release, and look into the issue for the next release