microversion 3.0's upload-to-image should not support “--visibility" and "--protected"

[Cao ShuFeng]

when "--os-volume-api-version 3.0" is passed to "cinder upload-to-image" command, cinderclient should not access cinder-api with argument "visibility" and "protected", but currently it does pass these two arguments in request url.

http://paste.openstack.org/show/497962/

This is bug is caused by another bug:

https://bugs.launchpad.net/python-cinderclient/+bug/1573414

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Cao ShuFeng (<email address hidden>) on branch: master
Review: https://review.openstack.org/336767
Reason: This is done here:
https://review.openstack.org/#/c/309283/

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/309283
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=09a76374c38e217f29520be5dd898993f2fba37f
Submitter: Jenkins
Branch: master

commit 09a76374c38e217f29520be5dd898993f2fba37f
Author: Cao ShuFeng <email address hidden>
Date: Fri Apr 22 14:46:23 2016 +0800

    Fix useless api_version of Manager class

    Manager's api_version property is used by api_version.wraps. It
    should not be an "empty" class and the real api_version can be
    read from Client class.

    The 3.0 version's upload-to-image doesn't work as excepted because
    of this useless api_version of VolumeManager class. This patch also
    fix it and update some unit tests for it, because the changes are
    co-dependent on one another.

    Co-Authored-By: Nate Potter <email address hidden>

    Change-Id: I398cbd02b61f30918a427291d1d3ae00435e0f4c
    Closes-Bug: #1573414
    Closes-Bug: #1589040

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/python-cinderclient 1.9.0 release.