search_opts parameter of volumes_types.list() is not used

[Andrey Pavlov]

volume_types.list() method has a search_opts parameter, which is not used

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/248350
Reason: Abandoning per comments. If there is some reason to continue with this, feel free to unabandon and address the concerns.

[Sheel Rana (ranasheel2000)]

I am marking this as incomplete.

Please provide some more details about why search_opts parameter of volumes_types.list() should not be kept in code.

[Vitalii Gridnev (vgridnev)]

Sheel, who said that it should be removed? We have this parameter, but it should USED, so we need to invent an way how to pass that to Cinder API if such way exists.

[Sheel Rana (ranasheel2000)]

>Sheel, who said that it should be removed?
vitaly Gridnev, In proposed patchset this was suggested which seemed wrong to me.
Refer patch set extracts
def list(self, search_opts=None, is_public=None): -> def list(self, is_public=None):

That is why the query came, why to remove this parameter, better to keep this and update the way it should be used.
I think we are same page.

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/344172

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Yuriy Nesenenko (<email address hidden>) on branch: master
Review: https://review.openstack.org/344172

[OpenStack Infra (hudson-openstack)]

Change abandoned by Yuriy Nesenenko (<email address hidden>) on branch: master
Review: https://review.openstack.org/344256
Reason: search_opts parameter is used at deleting volume-type by name