)]}'
{"cinder/api/v2/volumes.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"27fd74282bf6242827240c98054b207cc6a4b2e5","unresolved":false,"context_lines":[{"line_number":37,"context_line":"from cinder.i18n import _"},{"line_number":38,"context_line":"from cinder.image import glance"},{"line_number":39,"context_line":"from cinder.objects import fields"},{"line_number":40,"context_line":"from cinder import objects"},{"line_number":41,"context_line":"from cinder import utils"},{"line_number":42,"context_line":"from cinder import volume as cinder_volume"},{"line_number":43,"context_line":"from cinder.volume import volume_types"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_3113a25e","line":40,"updated":"2020-07-08 18:13:59.000000000","message":"pep8: H306: imports not in alphabetical order (cinder.objects.fields, cinder.objects)","commit_id":"6b184f019f31952f72ede5dc340b020c98c5cc1e"}],"cinder/db/sqlalchemy/api.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"92ffb8a6ca747d7b8367f1c885ce40be18741854","unresolved":false,"context_lines":[{"line_number":2508,"context_line":"            col_gl_attr \u003d models.Volume.volume_glance_metadata"},{"line_number":2509,"context_line":"            for k, v in value.items():"},{"line_number":2510,"context_line":"                query \u003d query.filter(col_gl_attr.any(key\u003dk, value\u003dv))"},{"line_number":2511,"context_line":"        elif key \u003d\u003d \u0027status\u0027 and value \u003d\u003d \u0027error\u0027:"},{"line_number":2512,"context_line":"            # (haixin)the function of get_volume_status in api layer, it"},{"line_number":2513,"context_line":"            # change error_managing to error for API compatibility, so we"},{"line_number":2514,"context_line":"            # need to include error_managing here."}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_8ddc4d19","line":2511,"updated":"2020-05-06 13:57:26.000000000","message":"Are you sure this doesn\u0027t affect any other calls than the one described in the bug?","commit_id":"1ad87676e8e94a7c56b5705eb5a2b257e10b1d2c"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"8ecbf23f1be734a513817687c95ef12047b57f44","unresolved":false,"context_lines":[{"line_number":2508,"context_line":"            col_gl_attr \u003d models.Volume.volume_glance_metadata"},{"line_number":2509,"context_line":"            for k, v in value.items():"},{"line_number":2510,"context_line":"                query \u003d query.filter(col_gl_attr.any(key\u003dk, value\u003dv))"},{"line_number":2511,"context_line":"        elif key \u003d\u003d \u0027status\u0027 and value \u003d\u003d \u0027error\u0027:"},{"line_number":2512,"context_line":"            # (haixin)the function of get_volume_status in api layer, it"},{"line_number":2513,"context_line":"            # change error_managing to error for API compatibility, so we"},{"line_number":2514,"context_line":"            # need to include error_managing here."}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_d9a56949","line":2511,"in_reply_to":"1f493fa4_8ddc4d19","updated":"2020-05-07 02:33:11.000000000","message":"this change only affect query volume list. because the function of _process_volume_filters() only be called in query volume list.","commit_id":"1ad87676e8e94a7c56b5705eb5a2b257e10b1d2c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"92ffb8a6ca747d7b8367f1c885ce40be18741854","unresolved":false,"context_lines":[{"line_number":2516,"context_line":"                                     models.Volume.status \u003d\u003d \u0027error_managing\u0027))"},{"line_number":2517,"context_line":"        elif key \u003d\u003d \u0027status\u0027 and value \u003d\u003d \u0027creating\u0027:"},{"line_number":2518,"context_line":"            query \u003d query.filter(or_(models.Volume.status \u003d\u003d \u0027creating\u0027,"},{"line_number":2519,"context_line":"                                     models.Volume.status \u003d\u003d \u0027managing\u0027))"},{"line_number":2520,"context_line":"        elif key \u003d\u003d \u0027status\u0027 and value \u003d\u003d \u0027deleting\u0027:"},{"line_number":2521,"context_line":"            query \u003d query.filter(or_("},{"line_number":2522,"context_line":"                models.Volume.status \u003d\u003d \u0027deleting\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_ad5e6973","line":2519,"updated":"2020-05-06 13:57:26.000000000","message":"This isn\u0027t described in the commit message.","commit_id":"1ad87676e8e94a7c56b5705eb5a2b257e10b1d2c"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"8ecbf23f1be734a513817687c95ef12047b57f44","unresolved":false,"context_lines":[{"line_number":2516,"context_line":"                                     models.Volume.status \u003d\u003d \u0027error_managing\u0027))"},{"line_number":2517,"context_line":"        elif key \u003d\u003d \u0027status\u0027 and value \u003d\u003d \u0027creating\u0027:"},{"line_number":2518,"context_line":"            query \u003d query.filter(or_(models.Volume.status \u003d\u003d \u0027creating\u0027,"},{"line_number":2519,"context_line":"                                     models.Volume.status \u003d\u003d \u0027managing\u0027))"},{"line_number":2520,"context_line":"        elif key \u003d\u003d \u0027status\u0027 and value \u003d\u003d \u0027deleting\u0027:"},{"line_number":2521,"context_line":"            query \u003d query.filter(or_("},{"line_number":2522,"context_line":"                models.Volume.status \u003d\u003d \u0027deleting\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_1965111e","line":2519,"in_reply_to":"1f493fa4_ad5e6973","updated":"2020-05-07 02:33:11.000000000","message":"oh, it is same as described above, and i will add describe in   cinder-spec.","commit_id":"1ad87676e8e94a7c56b5705eb5a2b257e10b1d2c"}]}