)]}'
{"nova/conductor/manager.py":[{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"2b0929bbb577e9a3cce602a3d34c1568e48c9ea1","unresolved":false,"context_lines":[{"line_number":713,"context_line":"        except Exception as ex:"},{"line_number":714,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":715,"context_line":"                vm_state \u003d instance[\u0027vm_state\u0027]"},{"line_number":716,"context_line":"                if not vm_state:"},{"line_number":717,"context_line":"                    vm_state \u003d vm_states.ACTIVE"},{"line_number":718,"context_line":"                updates \u003d {\u0027vm_state\u0027: vm_state,"},{"line_number":719,"context_line":"                            \u0027task_state\u0027: None}"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2BH7I%3D","line":716,"updated":"2013-12-02 10:50:46.000000000","message":"I don\u0027t believe vm_state is ever None, so not sure why this check is needed.","commit_id":"b7d126ff2df550579af64f7e2675f26f4d792747"}],"nova/tests/conductor/test_conductor.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"22325d2fa5b3a3c00c40f346922769a1ed0938d7","unresolved":false,"context_lines":[{"line_number":1537,"context_line":"            self.conductor.migrate_server, self.context, inst_obj,"},{"line_number":1538,"context_line":"            {\u0027host\u0027: \u0027destination\u0027}, True, False, None, \u0027block_migration\u0027,"},{"line_number":1539,"context_line":"            \u0027disk_over_commit\u0027)"},{"line_number":1540,"context_line":""},{"line_number":1541,"context_line":"    def test_migrate_server_deals_with_unexpected_exceptions(self):"},{"line_number":1542,"context_line":"        instance \u003d fake_instance.fake_db_instance()"},{"line_number":1543,"context_line":"        inst_obj \u003d instance_obj.Instance._from_db_object("}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F%2FqbY%3D","line":1540,"updated":"2013-10-30 17:40:10.000000000","message":"please add a new test for what you are testing, we still need this test case!","commit_id":"3a9f8cd40a6795c910095404cba3631b904d2228"},{"author":{"_account_id":8802,"name":"Vladik Romanovsky","email":"vromanso@redhat.com","username":"vladikr"},"change_message_id":"0388e24211188bce5e461c12ae25cb765c0de65f","unresolved":false,"context_lines":[{"line_number":1537,"context_line":"            self.conductor.migrate_server, self.context, inst_obj,"},{"line_number":1538,"context_line":"            {\u0027host\u0027: \u0027destination\u0027}, True, False, None, \u0027block_migration\u0027,"},{"line_number":1539,"context_line":"            \u0027disk_over_commit\u0027)"},{"line_number":1540,"context_line":""},{"line_number":1541,"context_line":"    def test_migrate_server_deals_with_unexpected_exceptions(self):"},{"line_number":1542,"context_line":"        instance \u003d fake_instance.fake_db_instance()"},{"line_number":1543,"context_line":"        inst_obj \u003d instance_obj.Instance._from_db_object("}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F%2Fl7Y%3D","line":1540,"in_reply_to":"AAAATn%2F%2FqbY%3D","updated":"2013-10-31 18:22:57.000000000","message":"Not sure what are you referring to?\nThis test has been adjusted to the change that I have made, hence:\n1. vm_state wont be set to vm_states.ERROR\n2. MigrationError will be raised if migration has failed.","commit_id":"3a9f8cd40a6795c910095404cba3631b904d2228"}]}