fake nova_client redefines the volumes while creating fake_nova_client

[Sushil Kumar]

https://github.com/openstack/trove/blob/master/trove/tests/fakes/nova.py#L793 defines
nova_client.volumes = volume_client

also volume in nova_client is defined as https://github.com/openstack/trove/blob/master/trove/tests/fakes/nova.py#L770

This makes volumes APIs with nova_client accessible as nova_client.volumes.volumes

This should be nova_client.volumes, instead.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/80536

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/80536
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=53182ddb06328d7c29f651b405f677a023b4c608
Submitter: Jenkins
Branch: master

commit 53182ddb06328d7c29f651b405f677a023b4c608
Author: Sushil Kumar <email address hidden>
Date: Fri Mar 14 10:48:18 2014 +0000

    Removes volumes redefinition in fake nova_client

    Reasons:
    - The redefinition of nova_client.volumes was making it,
      nova_client.volumes.volumes, if we needed to call the volume APIs
      from nova_client.
    - nova_client.volumes is already defined when defining "class FakeClient".

    Changes:
    - Removes redefinition of nova_client.volumes.

    Change-Id: I5981d461e5a0fd000214c35d5d0be58d86d6d368
    Closes-Bug: #1292457