Wait for volume to become available

[bertinatto]

Is this a bug fix or adding new feature?

/kind bug

What is this PR about? / Why do we need it?

This patch waits for the volume to become available when CreateVolume
is called. This prevents the CO from trying to attach the volume when
it's still being created.

Fixes #65.

/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 186

• 4 of 6 (66.67%) changed or added relevant lines in 1 file are covered.
• 13 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-1.08%) to 50.181%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pkg/cloud/cloud.go	4	6	66.67%

Files with Coverage Reduction	New Missed Lines	%
pkg/cloud/cloud.go	13	66.67%

Totals
Change from base Build 185:	-1.08%
Covered Lines:	556
Relevant Lines:	1108

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 232

• 8 of 9 (88.89%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.4%) to 50.352%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pkg/cloud/cloud.go	8	9	88.89%

Totals
Change from base Build 231:	0.4%
Covered Lines:	572
Relevant Lines:	1136

---

💛 - Coveralls

[bertinatto]

/hold

This can only be merged after #122 because there's conflict with CSI v0.

[bertinatto]

/hold cancel

Actually, no. The real problem was that gomock wasn't generating the correct import paths.

[bertinatto]

/retest

[bertinatto]

I just changed the base branch for this PR from master to next.

[leakingtapan]

Just a minor comment, did we use a different version of MockGen so that the content are different from the previous generated code? Which MockGen are you using? Could you add some documentation about this in docs folder. Or we can do this in a separate PR

[bertinatto]

I think it was the same version as before, but apparently it had some incompatibilities with go modules. I had to update the imports of this file manually (not good).

I updated the deps to their last minor versions and now it works fine.

[leakingtapan]

/lgtm