)]}'
{"nova/virt/block_device.py":[{"author":{"_account_id":14027,"name":"Michal Rostecki","email":"mrostecki@suse.de","username":"mrostecki"},"change_message_id":"1e0ebb3ae5318d563137c5e5439ab8c4f3ffc6fe","unresolved":false,"context_lines":[{"line_number":203,"context_line":"                self._valid_destination):"},{"line_number":204,"context_line":"            LOG.error(\"Failed to transform: source_type: %(source)s, \""},{"line_number":205,"context_line":"                      \"destination_type: %(dest)s\","},{"line_number":206,"context_line":"                      { \u0027source\u0027: self._bdm_obj.source_type,"},{"line_number":207,"context_line":"                        \u0027dest\u0027: self._bdm_obj.destination_type }"},{"line_number":208,"context_line":"                     )"},{"line_number":209,"context_line":"            raise _InvalidType"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_3da4b355","line":206,"updated":"2015-05-06 16:46:21.000000000","message":"PEP8 violation. :( I propose to do something like:\n\n LOG.error(\"Failed to transform: source_type: %(source)s, \"\n           \"destination_type: %(dest)s\",\n           {\u0027source\u0027: self._bdm_obj.source_type,\n            \u0027dest\u0027: self._bdm_obj.destination_type})","commit_id":"00c95c14dcaf255ec0bca1eafc0882117832e1af"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"bdb17731de0a5d47b54408b8e120d51800d81909","unresolved":false,"context_lines":[{"line_number":203,"context_line":"                self._valid_destination):"},{"line_number":204,"context_line":"            LOG.error(\"Failed to transform: source_type: %(source)s, \""},{"line_number":205,"context_line":"                      \"destination_type: %(dest)s\","},{"line_number":206,"context_line":"                      { \u0027source\u0027: self._bdm_obj.source_type,"},{"line_number":207,"context_line":"                        \u0027dest\u0027: self._bdm_obj.destination_type }"},{"line_number":208,"context_line":"                     )"},{"line_number":209,"context_line":"            raise _InvalidType"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_989bac72","line":206,"in_reply_to":"1aca2d91_3da4b355","updated":"2015-05-07 14:22:57.000000000","message":"Whoops, thanks. :)","commit_id":"00c95c14dcaf255ec0bca1eafc0882117832e1af"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"84925d7768caa8da77960716961e41f64adbe2e6","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        if (not self._bdm_obj.source_type \u003d\u003d self._valid_source"},{"line_number":203,"context_line":"                or not self._bdm_obj.destination_type \u003d\u003d"},{"line_number":204,"context_line":"                self._valid_destination):"},{"line_number":205,"context_line":"            LOG.error(_LE(\"Failed to transform: source_type: %(source)s, \""},{"line_number":206,"context_line":"                      \"destination_type: %(dest)s\"),"},{"line_number":207,"context_line":"                      {\u0027source\u0027: self._bdm_obj.source_type,"},{"line_number":208,"context_line":"                       \u0027dest\u0027: self._bdm_obj.destination_type}"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a41bdd9_03bd65b6","line":205,"updated":"2015-07-15 07:59:53.000000000","message":"Should we mention in the message that it was because of a wrong Type?","commit_id":"7a373e1be33879c39fae88fa6020e417abb55847"}]}