)]}'
{"nova/tests/unit/compute/test_tracker.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"78787681a75655a0a524171f54a986b7ac066b85","unresolved":false,"context_lines":[{"line_number":1861,"context_line":"        self.assertEqual(_HOSTNAME, migration.dest_compute)"},{"line_number":1862,"context_line":"        self.assertEqual(_NODENAME, migration.dest_node)"},{"line_number":1863,"context_line":"        self.assertEqual(\"pre-migrating\", migration.status)"},{"line_number":1864,"context_line":"        self.assertEqual(1, len(self.rt.tracked_migrations))"},{"line_number":1865,"context_line":"        mig_save_mock.assert_called_once_with()"},{"line_number":1866,"context_line":"        inst_save_mock.assert_called_once_with()"},{"line_number":1867,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_11a9cc24","line":1864,"updated":"2016-08-10 11:42:10.000000000","message":"nit: this is the kind of assert that we really should have a 3rd param on. 1 !\u003d 0 isn\u0027t super useful to get what failed. \n\nIf this needs to be respun, please add that.","commit_id":"5ffb377c0a543df3a153e6cc40081a8abcaf3061"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"78787681a75655a0a524171f54a986b7ac066b85","unresolved":false,"context_lines":[{"line_number":1863,"context_line":"        self.assertEqual(\"pre-migrating\", migration.status)"},{"line_number":1864,"context_line":"        self.assertEqual(1, len(self.rt.tracked_migrations))"},{"line_number":1865,"context_line":"        mig_save_mock.assert_called_once_with()"},{"line_number":1866,"context_line":"        inst_save_mock.assert_called_once_with()"},{"line_number":1867,"context_line":""},{"line_number":1868,"context_line":""},{"line_number":1869,"context_line":"class TestUpdateUsageFromMigrations(BaseTestCase):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_d179f4c9","line":1866,"updated":"2016-08-10 11:42:10.000000000","message":"so... I get why these are true, but are they actually relevant to the test really?","commit_id":"5ffb377c0a543df3a153e6cc40081a8abcaf3061"},{"author":{"_account_id":14819,"name":"Timofey Durakov","email":"timofei.nd@gmail.com","username":"tdurakov"},"change_message_id":"748441239a7b3555fca9fc95f770e730b9ecc78e","unresolved":false,"context_lines":[{"line_number":1826,"context_line":"            status\u003d\u0027accepted\u0027,"},{"line_number":1827,"context_line":"            migration_type\u003d\u0027evacuation\u0027"},{"line_number":1828,"context_line":"        )"},{"line_number":1829,"context_line":"        instance \u003d objects.Instance("},{"line_number":1830,"context_line":"            id\u003d1,"},{"line_number":1831,"context_line":"            host\u003dNone,"},{"line_number":1832,"context_line":"            node\u003dNone,"},{"line_number":1833,"context_line":"            uuid\u003d\u0027abef5b54-dea6-47b8-acb2-22aeb1b57919\u0027,"},{"line_number":1834,"context_line":"            memory_mb\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027memory_mb\u0027],"},{"line_number":1835,"context_line":"            vcpus\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027vcpus\u0027],"},{"line_number":1836,"context_line":"            root_gb\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027root_gb\u0027],"},{"line_number":1837,"context_line":"            ephemeral_gb\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027ephemeral_gb\u0027],"},{"line_number":1838,"context_line":"            numa_topology\u003dNone,"},{"line_number":1839,"context_line":"            pci_requests\u003dNone,"},{"line_number":1840,"context_line":"            pci_devices\u003dNone,"},{"line_number":1841,"context_line":"            instance_type_id\u003d2,"},{"line_number":1842,"context_line":"            vm_state\u003dvm_states.ACTIVE,"},{"line_number":1843,"context_line":"            power_state\u003dpower_state.RUNNING,"},{"line_number":1844,"context_line":"            task_state\u003dtask_states.REBUILDING,"},{"line_number":1845,"context_line":"            os_type\u003d\u0027fake-os\u0027,"},{"line_number":1846,"context_line":"            project_id\u003d\u0027fake-project\u0027,"},{"line_number":1847,"context_line":"            flavor \u003d _INSTANCE_TYPE_OBJ_FIXTURES[2],"},{"line_number":1848,"context_line":"            old_flavor \u003d _INSTANCE_TYPE_OBJ_FIXTURES[2],"},{"line_number":1849,"context_line":"            new_flavor \u003d _INSTANCE_TYPE_OBJ_FIXTURES[2],"},{"line_number":1850,"context_line":"        )"},{"line_number":1851,"context_line":""},{"line_number":1852,"context_line":"        # not using mock.sentinel.ctx because resize_claim calls #elevated"},{"line_number":1853,"context_line":"        ctx \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_e4a7e5ba","line":1850,"range":{"start_line":1829,"start_character":8,"end_line":1850,"end_character":9},"updated":"2016-08-11 08:33:54.000000000","message":"I\u0027d prefer to put that object to _INSTANCE_FIXTURES, what do you think?","commit_id":"9fff82896870be9c581abd6f556b80db371cda68"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5d4f11deebabbf1e13b1e3fd34ff108bdcf99007","unresolved":false,"context_lines":[{"line_number":1826,"context_line":"            status\u003d\u0027accepted\u0027,"},{"line_number":1827,"context_line":"            migration_type\u003d\u0027evacuation\u0027"},{"line_number":1828,"context_line":"        )"},{"line_number":1829,"context_line":"        instance \u003d objects.Instance("},{"line_number":1830,"context_line":"            id\u003d1,"},{"line_number":1831,"context_line":"            host\u003dNone,"},{"line_number":1832,"context_line":"            node\u003dNone,"},{"line_number":1833,"context_line":"            uuid\u003d\u0027abef5b54-dea6-47b8-acb2-22aeb1b57919\u0027,"},{"line_number":1834,"context_line":"            memory_mb\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027memory_mb\u0027],"},{"line_number":1835,"context_line":"            vcpus\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027vcpus\u0027],"},{"line_number":1836,"context_line":"            root_gb\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027root_gb\u0027],"},{"line_number":1837,"context_line":"            ephemeral_gb\u003d_INSTANCE_TYPE_FIXTURES[2][\u0027ephemeral_gb\u0027],"},{"line_number":1838,"context_line":"            numa_topology\u003dNone,"},{"line_number":1839,"context_line":"            pci_requests\u003dNone,"},{"line_number":1840,"context_line":"            pci_devices\u003dNone,"},{"line_number":1841,"context_line":"            instance_type_id\u003d2,"},{"line_number":1842,"context_line":"            vm_state\u003dvm_states.ACTIVE,"},{"line_number":1843,"context_line":"            power_state\u003dpower_state.RUNNING,"},{"line_number":1844,"context_line":"            task_state\u003dtask_states.REBUILDING,"},{"line_number":1845,"context_line":"            os_type\u003d\u0027fake-os\u0027,"},{"line_number":1846,"context_line":"            project_id\u003d\u0027fake-project\u0027,"},{"line_number":1847,"context_line":"            flavor \u003d _INSTANCE_TYPE_OBJ_FIXTURES[2],"},{"line_number":1848,"context_line":"            old_flavor \u003d _INSTANCE_TYPE_OBJ_FIXTURES[2],"},{"line_number":1849,"context_line":"            new_flavor \u003d _INSTANCE_TYPE_OBJ_FIXTURES[2],"},{"line_number":1850,"context_line":"        )"},{"line_number":1851,"context_line":""},{"line_number":1852,"context_line":"        # not using mock.sentinel.ctx because resize_claim calls #elevated"},{"line_number":1853,"context_line":"        ctx \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_880b8ed5","line":1850,"range":{"start_line":1829,"start_character":8,"end_line":1850,"end_character":9},"in_reply_to":"9ad45d7e_e4a7e5ba","updated":"2016-08-11 12:28:06.000000000","message":"Meh, just for this one test I thought it was easier just to have a local explicit fixture here...","commit_id":"9fff82896870be9c581abd6f556b80db371cda68"}]}