Cannot search volumes by meta-data

[Mathieu Gagné]

The metadata parameter is urlencoded (to string) by cinderclient but isn't decoded back to dict by the API.

This prevents the search of volumes by meta-data.

[OpenStack Infra (hudson-openstack)]

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

[Mathieu Gagné (mgagne)]

I wish this bugfix to be backported to Grizzly.

[Seif Lotfy (seif)]

Are you still working on it or can I take over? I have a working which uses your unit tests :D

[Mathieu Gagné (mgagne)]

Sorry for the delay. I'm about to propose a new approach. Feel free to comment or challenge my proposition.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[Seif Lotfy (seif)]

Hey Mathieu,
Thanks a lot for taking up on this, I also think search volumes by meta-data is a crucial feature.
That being said I would like to challenge your approach to this.

sending 'metadata': {'tag': 'looloo', 'desc': 'foobar'}
should be encoded to
?metadata=%7B%27tag%27%3A+%27looloo%27%2C+%27desc%27%3A+%27foobar%27%7D

What is wrong with that. It is easy translated back and the cinder side can handle it
What am I missing

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/38062
Committed: http://github.com/openstack/cinder/commit/76fc407382315a97cc020e8592199609b3a6a8e9
Submitter: Jenkins
Branch: master

commit 76fc407382315a97cc020e8592199609b3a6a8e9
Author: Mathieu Gagné <email address hidden>
Date: Wed Jun 26 16:27:43 2013 -0400

    Fix volumes search by metadata

    The metadata parameter is urlencoded (to string) by cinderclient
    but isn't decoded back to dict by the API.

    The metadata parameter needs to be converted back to dict
    to be of any use by the filters parameter of get_all().

    The original implementation has been done by:
    Mathieu Gagne <email address hidden>

    Fixes: bug #1195015

    Change-Id: I19d7d386afddddc067e9f4ef86967c9b72d9e530