Wait for attachment state after attach/detach

[bertinatto]

Also, upsdate e2e test.

Fixes #38.

/kind feature
/assign @leakingtapan

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bertinatto

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [bertinatto]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[coveralls]

Pull Request Test Coverage Report for Build 55

• 32 of 42 (76.19%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+1.1%) to 49.378%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pkg/cloud/cloud.go	32	42	76.19%

Totals
Change from base Build 53:	1.1%
Covered Lines:	516
Relevant Lines:	1045

---

💛 - Coveralls

[leakingtapan]

Im wondering why need to replace go test with ginkgo?

[bertinatto]

I need to enable Ginkgo's verbosity and I also need to run the test with admin privileges due to the mount command. For some reason, executing go test -ginkgo.v with sudo didn't work for me (I think it's related to go modules), so I called ginkgo directly.

In any case, I improved these bits by compiling the test and then calling it with sudo.

[leakingtapan]

/lgtm