)]}'
{"cinder/tests/brick/test_brick_lvm.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9b35c8e82b8c7cfb12dc650f864fc71cf83dbf0b","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                data \u003d \u0027  wi-a-\u0027"},{"line_number":94,"context_line":"            else:"},{"line_number":95,"context_line":"                data \u003d \u0027  owi-a-\u0027"},{"line_number":96,"context_line":"        elif \u0027pvs, --noheadings\u0027 and \u0027fake-volumes\u0027 in cmd_string:"},{"line_number":97,"context_line":"            data \u003d \"  fake-volumes:/dev/sda:10.00g:8.99g\\n\""},{"line_number":98,"context_line":"        elif \u0027pvs, --noheadings\u0027 in cmd_string:"},{"line_number":99,"context_line":"            data \u003d \"  fake-volumes:/dev/sda:10.00g:8.99g\\n\""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2FZE%3D","side":"PARENT","line":96,"updated":"2014-01-04 16:41:28.000000000","message":"This if \"x\" and \"y\" in cmd_string construct is wrong.  It means the pvs output is being passed for executes of lvcreate,  too because it matches anything containing \u0027fake-volumes\u0027.","commit_id":"1aa01b3b79847ebeecc0b39e3af6cb09b8ea6e19"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"b9568126bd1c7d71cd95b3faa73332a955af1320","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                   \"kVxztV-dKpG-Rz7E-xtKY-jeju-QsYU-SLG6Z1\\n\""},{"line_number":81,"context_line":"            if \u0027fake-volumes\u0027 in cmd_string:"},{"line_number":82,"context_line":"                return (data, \"\")"},{"line_number":83,"context_line":"            data +\u003d \"  fake-volumes-2:11.00:11.00:0:\"\\"},{"line_number":84,"context_line":"                    \"lWyauW-dKpG-Rz7E-xtKY-jeju-QsYU-SLG7Z2\\n\""},{"line_number":85,"context_line":"            data +\u003d \"  fake-volumes-3:12.00:12.00:0:\"\\"},{"line_number":86,"context_line":"                    \"mXzbuX-dKpG-Rz7E-xtKY-jeju-QsYU-SLG8Z3\\n\""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2Fec%3D","line":83,"updated":"2014-01-04 15:16:41.000000000","message":"what\u0027s the value of this change?","commit_id":"acb9a019e495412514e55393272273b38436df50"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"b9568126bd1c7d71cd95b3faa73332a955af1320","unresolved":false,"context_lines":[{"line_number":82,"context_line":"                return (data, \"\")"},{"line_number":83,"context_line":"            data +\u003d \"  fake-volumes-2:11.00:11.00:0:\"\\"},{"line_number":84,"context_line":"                    \"lWyauW-dKpG-Rz7E-xtKY-jeju-QsYU-SLG7Z2\\n\""},{"line_number":85,"context_line":"            data +\u003d \"  fake-volumes-3:12.00:12.00:0:\"\\"},{"line_number":86,"context_line":"                    \"mXzbuX-dKpG-Rz7E-xtKY-jeju-QsYU-SLG8Z3\\n\""},{"line_number":87,"context_line":"        elif \u0027lvs, --noheadings, --unit\u003dg, -o, vg_name,name,size\u0027\\"},{"line_number":88,"context_line":"                in cmd_string:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2FeY%3D","line":85,"updated":"2014-01-04 15:16:41.000000000","message":"Same here, why is \"3:12.00:12\" better than 3:10.00:10\" ????","commit_id":"acb9a019e495412514e55393272273b38436df50"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9b35c8e82b8c7cfb12dc650f864fc71cf83dbf0b","unresolved":false,"context_lines":[{"line_number":82,"context_line":"                return (data, \"\")"},{"line_number":83,"context_line":"            data +\u003d \"  fake-volumes-2:11.00:11.00:0:\"\\"},{"line_number":84,"context_line":"                    \"lWyauW-dKpG-Rz7E-xtKY-jeju-QsYU-SLG7Z2\\n\""},{"line_number":85,"context_line":"            data +\u003d \"  fake-volumes-3:12.00:12.00:0:\"\\"},{"line_number":86,"context_line":"                    \"mXzbuX-dKpG-Rz7E-xtKY-jeju-QsYU-SLG8Z3\\n\""},{"line_number":87,"context_line":"        elif \u0027lvs, --noheadings, --unit\u003dg, -o, vg_name,name,size\u0027\\"},{"line_number":88,"context_line":"                in cmd_string:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2F%2FaQ%3D","line":85,"in_reply_to":"AAAAUH%2F%2F%2FeY%3D","updated":"2014-01-04 16:41:28.000000000","message":"Just changed to make it easier to tell where parsing is failing, so they don\u0027t all have the same values.  Not strictly required.","commit_id":"acb9a019e495412514e55393272273b38436df50"}]}