)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e449194fe204fa1112cd33d111685ff8d7ec2bd0","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Currently, if live migration failed, nova compute did not send"},{"line_number":12,"context_line":"notification, this caused other service such as nova-scheduler"},{"line_number":13,"context_line":"has no chance to do some operation based on the result of live"},{"line_number":14,"context_line":"migration."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The fix add notification logic for live_migration like other"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAOn%2F%2FY4s%3D","line":13,"updated":"2013-05-11 16:36:58.000000000","message":"Change \"this caused other service such as nova-scheduler has no chance to do some operation\" to \"this causes other services such as nova-scheduler to have no chance to do some operation\".","commit_id":"ce30c952043849dd9c359507cee2be0665f4d439"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"34d3b7709c828d16bdc283c34f0f2c98a24328e0","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Currently, if live migration failed, nova compute did not send"},{"line_number":12,"context_line":"notification, this caused other service such as nova-scheduler"},{"line_number":13,"context_line":"has no chance to do some operation based on the result of live"},{"line_number":14,"context_line":"migration."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The fix add notification logic for live_migration like other"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAOn%2F%2FU78%3D","line":13,"in_reply_to":"AAAAOn%2F%2FY4s%3D","updated":"2013-05-14 15:32:02.000000000","message":"Done","commit_id":"ce30c952043849dd9c359507cee2be0665f4d439"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c9017c9935f9afd9c0fa1c337d028a7691911d58","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Currently, if live migration failed, nova compute did not send"},{"line_number":12,"context_line":"notification, this causes other services such as nova-scheduler"},{"line_number":13,"context_line":"to have no chance to do some operation_rollback_live_migration"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The fix add notification logic for live_migration like other"},{"line_number":16,"context_line":"VM instance operations such as resize, run instance etc."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAOn%2F%2FUtw%3D","line":13,"updated":"2013-05-14 17:56:27.000000000","message":"I\u0027m confused by \"operation_rollback_live_migration\" here. Your original commit message had this:\n\n\"no chance to do some operation based on the result of live migration.\"\n\nI prefer that unless it no longer makes sense in the context of this fix.","commit_id":"70da60e46b4e464ecd8082f241920a70b13d0a94"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"2a6b706dd8e389c4f691b1516899f0f33cc144e7","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Currently, if live migration failed, nova compute did not send"},{"line_number":12,"context_line":"notification, this causes other services such as nova-scheduler"},{"line_number":13,"context_line":"to have no chance to do some operation_rollback_live_migration"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The fix add notification logic for live_migration like other"},{"line_number":16,"context_line":"VM instance operations such as resize, run instance etc."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAOn%2F%2FTwM%3D","line":13,"in_reply_to":"AAAAOn%2F%2FUtw%3D","updated":"2013-05-15 03:54:02.000000000","message":"My fault, updated according to your comments for patch set 2. Thanks!","commit_id":"70da60e46b4e464ecd8082f241920a70b13d0a94"}],"nova/compute/manager.py":[{"author":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"change_message_id":"a8122c3c262de056ffdc38f2d3d645f0040612dd","unresolved":false,"context_lines":[{"line_number":3218,"context_line":"                            context, instance, bdms\u003dbdms)"},{"line_number":3219,"context_line":""},{"line_number":3220,"context_line":"        network_info \u003d self._get_instance_nw_info(context, instance)"},{"line_number":3221,"context_line":"        self._notify_about_instance_usage("},{"line_number":3222,"context_line":"                     context, instance, \"pre.live_migration.start\","},{"line_number":3223,"context_line":"                     network_info\u003dnetwork_info)"},{"line_number":3224,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2FY7I%3D","line":3221,"updated":"2013-05-11 14:49:35.000000000","message":"Naming it \"live_migration.pre.start\" seems more consistent with other notifications.  (Same for all the other notifs.)","commit_id":"bb6d2d4ac1a53af352632a6325aa5efa0db3fb49"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"9198d853d137b10f3abd8daa049f0080a4f5868f","unresolved":false,"context_lines":[{"line_number":3218,"context_line":"                            context, instance, bdms\u003dbdms)"},{"line_number":3219,"context_line":""},{"line_number":3220,"context_line":"        network_info \u003d self._get_instance_nw_info(context, instance)"},{"line_number":3221,"context_line":"        self._notify_about_instance_usage("},{"line_number":3222,"context_line":"                     context, instance, \"pre.live_migration.start\","},{"line_number":3223,"context_line":"                     network_info\u003dnetwork_info)"},{"line_number":3224,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2FY6U%3D","line":3221,"in_reply_to":"AAAAOn%2F%2FY7I%3D","updated":"2013-05-11 15:13:01.000000000","message":"Done","commit_id":"bb6d2d4ac1a53af352632a6325aa5efa0db3fb49"}],"nova/tests/compute/test_compute.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"2d5489aa31f372b56131ff7316bbe12c10541646","unresolved":false,"context_lines":[{"line_number":3464,"context_line":"                                              block_migration\u003dFalse,"},{"line_number":3465,"context_line":"                                              migrate_data\u003dmigrate_data)"},{"line_number":3466,"context_line":"        self.assertEqual(ret, None)"},{"line_number":3467,"context_line":"        self.assertEquals(len(test_notifier.NOTIFICATIONS), 2)"},{"line_number":3468,"context_line":"        msg \u003d test_notifier.NOTIFICATIONS[0]"},{"line_number":3469,"context_line":"        self.assertEquals(msg[\u0027event_type\u0027],"},{"line_number":3470,"context_line":"                          \u0027compute.instance.live_migration.pre.start\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAO3%2F%2F%2BXw%3D","line":3467,"updated":"2013-05-15 19:44:53.000000000","message":"assertEquals is deprecated; use assertEqual instead.","commit_id":"a680f70b6567589ddb2a73cd0f5d7c5822dce55e"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"0b67e1b13f39fb93a1bd2d14108070afdc3433b2","unresolved":false,"context_lines":[{"line_number":3464,"context_line":"                                              block_migration\u003dFalse,"},{"line_number":3465,"context_line":"                                              migrate_data\u003dmigrate_data)"},{"line_number":3466,"context_line":"        self.assertEqual(ret, None)"},{"line_number":3467,"context_line":"        self.assertEquals(len(test_notifier.NOTIFICATIONS), 2)"},{"line_number":3468,"context_line":"        msg \u003d test_notifier.NOTIFICATIONS[0]"},{"line_number":3469,"context_line":"        self.assertEquals(msg[\u0027event_type\u0027],"},{"line_number":3470,"context_line":"                          \u0027compute.instance.live_migration.pre.start\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAO3%2F%2F9MU%3D","line":3467,"in_reply_to":"AAAAO3%2F%2F%2BXw%3D","updated":"2013-05-16 06:21:21.000000000","message":"Done","commit_id":"a680f70b6567589ddb2a73cd0f5d7c5822dce55e"}]}