)]}'
{"nova/virt/libvirt/utils.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"6bcbf072fcf18cd3f766c9da96558f48f6d8294e","unresolved":false,"context_lines":[{"line_number":225,"context_line":"                                 \u0027 Only %(free_space)db available,\u0027"},{"line_number":226,"context_line":"                                 \u0027 but %(size)db required\u0027"},{"line_number":227,"context_line":"                                 \u0027 by volume %(lv)s.\u0027)"},{"line_number":228,"context_line":"                               % {\u0027vg\u0027: vg,"},{"line_number":229,"context_line":"                                  \u0027free_space\u0027: free_space,"},{"line_number":230,"context_line":"                                  \u0027size\u0027: size,"},{"line_number":231,"context_line":"                                  \u0027lv\u0027: lv})"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2Fx9k%3D","line":228,"updated":"2013-05-23 07:05:47.000000000","message":"I think we prefer % to be on the previous line... I know pep8 highly suggests splitting after operators not before.. and I think % qualifies?","commit_id":"81f5fb92b0acb9bbe60aa885236803f95fee7aea"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"6bcbf072fcf18cd3f766c9da96558f48f6d8294e","unresolved":false,"context_lines":[{"line_number":239,"context_line":"                          \u0027 Virtual volume size is %(size)db,\u0027"},{"line_number":240,"context_line":"                          \u0027 but free space on volume group is\u0027"},{"line_number":241,"context_line":"                          \u0027 only %(free_space)db.\u0027)"},{"line_number":242,"context_line":"                        % {\u0027vg\u0027: vg,"},{"line_number":243,"context_line":"                           \u0027free_space\u0027: free_space,"},{"line_number":244,"context_line":"                           \u0027size\u0027: size,"},{"line_number":245,"context_line":"                           \u0027lv\u0027: lv})"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2Fx9g%3D","line":242,"updated":"2013-05-23 07:05:47.000000000","message":"This one can be its own argument.. or did we change our mind on which way we should go with logging calls?","commit_id":"81f5fb92b0acb9bbe60aa885236803f95fee7aea"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"80f7cd9f36163d6499cf5b269ab831324c0dbc44","unresolved":false,"context_lines":[{"line_number":239,"context_line":"                          \u0027 Virtual volume size is %(size)db,\u0027"},{"line_number":240,"context_line":"                          \u0027 but free space on volume group is\u0027"},{"line_number":241,"context_line":"                          \u0027 only %(free_space)db.\u0027)"},{"line_number":242,"context_line":"                        % {\u0027vg\u0027: vg,"},{"line_number":243,"context_line":"                           \u0027free_space\u0027: free_space,"},{"line_number":244,"context_line":"                           \u0027size\u0027: size,"},{"line_number":245,"context_line":"                           \u0027lv\u0027: lv})"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2Fxr8%3D","line":242,"in_reply_to":"AAAAPH%2F%2Fx9g%3D","updated":"2013-05-23 09:45:39.000000000","message":"Sorry, this is a brain fart. I should have comma\u0027d these. I shall fix.","commit_id":"81f5fb92b0acb9bbe60aa885236803f95fee7aea"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"1e8a34ad00a957ace518e8960feb60837ad832af","unresolved":false,"context_lines":[{"line_number":225,"context_line":"                                 \u0027 Only %(free_space)db available,\u0027"},{"line_number":226,"context_line":"                                 \u0027 but %(size)db required\u0027"},{"line_number":227,"context_line":"                                 \u0027 by volume %(lv)s.\u0027),"},{"line_number":228,"context_line":"                               {\u0027vg\u0027: vg,"},{"line_number":229,"context_line":"                                \u0027free_space\u0027: free_space,"},{"line_number":230,"context_line":"                                \u0027size\u0027: size,"},{"line_number":231,"context_line":"                                \u0027lv\u0027: lv})"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2FwhA%3D","line":228,"updated":"2013-05-23 18:27:42.000000000","message":"This one is a raise(), not a LOG call, so I think it needs the %... but I\u0027d put it on the line before the dict.","commit_id":"8233e9c22c98642eda0e47a9be064e07b6adcffc"}]}