)]}'
{"cinder/api/openstack/volume/snapshots.py":[{"author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"change_message_id":"ab3ef7e1872686d8812ca451f23ede4db6b247b7","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from cinder.api.openstack import common"},{"line_number":22,"context_line":"from cinder.api.openstack import wsgi"},{"line_number":23,"context_line":"from cinder.api.openstack import xmlutil"},{"line_number":24,"context_line":"from cinder.api.openstack.volume.volumes import remove_invalid_options"},{"line_number":25,"context_line":"from cinder import exception"},{"line_number":26,"context_line":"from cinder import flags"},{"line_number":27,"context_line":"from cinder.openstack.common import log as logging"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2Fgl4%3D","line":24,"updated":"2012-09-05 15:05:47.000000000","message":"HACKING: import modules not methods.","commit_id":"88f99b664f16062f39fa7fb4985b30c72b2b3ae5"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"95e8e9dceaf0fee1abc02ac54ce7673878790467","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from cinder.api.openstack import common"},{"line_number":22,"context_line":"from cinder.api.openstack import wsgi"},{"line_number":23,"context_line":"from cinder.api.openstack import xmlutil"},{"line_number":24,"context_line":"from cinder.api.openstack.volume.volumes import remove_invalid_options"},{"line_number":25,"context_line":"from cinder import exception"},{"line_number":26,"context_line":"from cinder import flags"},{"line_number":27,"context_line":"from cinder.openstack.common import log as logging"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2Fgjk%3D","line":24,"in_reply_to":"AAAALH%2F%2Fgl4%3D","updated":"2012-09-05 15:23:01.000000000","message":"Done","commit_id":"88f99b664f16062f39fa7fb4985b30c72b2b3ae5"}],"cinder/api/openstack/volume/volumes.py":[{"author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"change_message_id":"ab3ef7e1872686d8812ca451f23ede4db6b247b7","unresolved":false,"context_lines":[{"line_number":315,"context_line":"    log_msg \u003d _(\"Removing options \u0027%(bad_options)s\u0027 from query\") % locals()"},{"line_number":316,"context_line":"    LOG.debug(log_msg)"},{"line_number":317,"context_line":"    for opt in unknown_options:"},{"line_number":318,"context_line":"        del search_options[opt]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2Fglg%3D","line":318,"updated":"2012-09-05 15:05:47.000000000","message":"just curious why the change here?","commit_id":"88f99b664f16062f39fa7fb4985b30c72b2b3ae5"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"95e8e9dceaf0fee1abc02ac54ce7673878790467","unresolved":false,"context_lines":[{"line_number":315,"context_line":"    log_msg \u003d _(\"Removing options \u0027%(bad_options)s\u0027 from query\") % locals()"},{"line_number":316,"context_line":"    LOG.debug(log_msg)"},{"line_number":317,"context_line":"    for opt in unknown_options:"},{"line_number":318,"context_line":"        del search_options[opt]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2FgkU%3D","line":318,"in_reply_to":"AAAALH%2F%2Fglg%3D","updated":"2012-09-05 15:23:01.000000000","message":"in one iteration I was passing in the webob multidict directly, and pop just pulls off the first value for that key.  Deleting the key seems to better reflect the desired/correct behavior (which is to remove this param as a valid filter).","commit_id":"88f99b664f16062f39fa7fb4985b30c72b2b3ae5"}]}