)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"43db8392b2b983e2ed8e1323f03de64a619b7a68","unresolved":false,"context_lines":[{"line_number":9,"context_line":"While migration jobs are running it is possible to have"},{"line_number":10,"context_line":"tasks requested against the VM, such as pausing it. This"},{"line_number":11,"context_line":"task processing is handled by a separate inline method."},{"line_number":12,"context_line":"Move this inline method out to be a static method at"},{"line_number":13,"context_line":"in the class, and add explicitly testing of it. This"},{"line_number":14,"context_line":"also adds an explicit warning if it finds an unknown"},{"line_number":15,"context_line":"task"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7aa08908_532d1645","line":12,"updated":"2016-06-13 02:13:53.000000000","message":"s/at in the/in the/","commit_id":"e9fddec21aef929c1b3418e1e3de8b11ac7b3afb"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"43db8392b2b983e2ed8e1323f03de64a619b7a68","unresolved":false,"context_lines":[{"line_number":10,"context_line":"tasks requested against the VM, such as pausing it. This"},{"line_number":11,"context_line":"task processing is handled by a separate inline method."},{"line_number":12,"context_line":"Move this inline method out to be a static method at"},{"line_number":13,"context_line":"in the class, and add explicitly testing of it. This"},{"line_number":14,"context_line":"also adds an explicit warning if it finds an unknown"},{"line_number":15,"context_line":"task"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7aa08908_33320aa8","line":13,"updated":"2016-06-13 02:13:53.000000000","message":"s/explicitly/explicit/","commit_id":"e9fddec21aef929c1b3418e1e3de8b11ac7b3afb"}],"nova/virt/libvirt/migration.py":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"dec685b6989f81121c3c5fbb4344c415f1918e67","unresolved":false,"context_lines":[{"line_number":309,"context_line":"            except Exception as e:"},{"line_number":310,"context_line":"                LOG.warning(_LW(\"Failed to pause instance during \""},{"line_number":311,"context_line":"                                \"live-migration %s\"),"},{"line_number":312,"context_line":"                            e, instance\u003dinstance)"},{"line_number":313,"context_line":"        else:"},{"line_number":314,"context_line":"            LOG.warning(_LW(\"Unknown migration task \u0027%(task)s\u0027\"),"},{"line_number":315,"context_line":"                        {\"task\": task}, instance\u003dinstance)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_dd11d753","line":312,"updated":"2016-06-24 16:00:29.000000000","message":"Just a suggestion, you are passing instance to use if for log, but you can use guest.\n\n  log.warning(..., instance_uuid\u003dguest.uuid) \n\nmaybe\n\n  log.warning(..., instance\u003dguest) would work but I\u0027m not sure.","commit_id":"b26852d330c12a3620ce394998b17a8cde9012ad"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"929d987f3ca6db46faf0f584fea476add8f7b419","unresolved":false,"context_lines":[{"line_number":309,"context_line":"            except Exception as e:"},{"line_number":310,"context_line":"                LOG.warning(_LW(\"Failed to pause instance during \""},{"line_number":311,"context_line":"                                \"live-migration %s\"),"},{"line_number":312,"context_line":"                            e, instance\u003dinstance)"},{"line_number":313,"context_line":"        else:"},{"line_number":314,"context_line":"            LOG.warning(_LW(\"Unknown migration task \u0027%(task)s\u0027\"),"},{"line_number":315,"context_line":"                        {\"task\": task}, instance\u003dinstance)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_655c5319","line":312,"in_reply_to":"3aaa91ec_dd11d753","updated":"2016-06-27 10:51:54.000000000","message":"I quite like just passing in instance like we do elsewhere, for consistency. Although I have a feeling, using our defaults, instance_uuid\u003dguest.uuid might work too.","commit_id":"b26852d330c12a3620ce394998b17a8cde9012ad"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"929d987f3ca6db46faf0f584fea476add8f7b419","unresolved":false,"context_lines":[{"line_number":311,"context_line":"                                \"live-migration %s\"),"},{"line_number":312,"context_line":"                            e, instance\u003dinstance)"},{"line_number":313,"context_line":"        else:"},{"line_number":314,"context_line":"            LOG.warning(_LW(\"Unknown migration task \u0027%(task)s\u0027\"),"},{"line_number":315,"context_line":"                        {\"task\": task}, instance\u003dinstance)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3aaa91ec_05a46f05","line":314,"updated":"2016-06-27 10:51:54.000000000","message":"I like this add.","commit_id":"b26852d330c12a3620ce394998b17a8cde9012ad"}]}