)]}'
{"nova/tests/unit/virt/libvirt/test_driver.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6b941fdba148b6459278d7f3a67b5e00b2378f90","unresolved":false,"context_lines":[{"line_number":10259,"context_line":"        disk_info[\u0027mapping\u0027].pop(\u0027disk.local\u0027)"},{"line_number":10260,"context_line":""},{"line_number":10261,"context_line":"        with test.nested("},{"line_number":10262,"context_line":"                mock.patch.object(utils, \u0027execute\u0027),"},{"line_number":10263,"context_line":"                mock.patch.object(drvr, \u0027get_info\u0027),"},{"line_number":10264,"context_line":"                mock.patch.object(drvr, \u0027_create_domain_and_network\u0027),"},{"line_number":10265,"context_line":"                mock.patch.object(imagebackend.Image, \u0027verify_base_size\u0027),"},{"line_number":10266,"context_line":"                mock.patch.object(imagebackend.Image, \u0027get_disk_size\u0027)"},{"line_number":10267,"context_line":"        ) as (execute_mock, get_info_mock,"},{"line_number":10268,"context_line":"              create_mock, verify_base_size_mock, disk_size_mock):"},{"line_number":10269,"context_line":"            disk_size_mock.return_value \u003d 0"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_dde4370e","line":10266,"range":{"start_line":10262,"start_character":0,"end_line":10266,"end_character":70},"updated":"2016-06-30 13:49:32.000000000","message":"nit: Does the indentation need to change for this?","commit_id":"426d58bdf4e945b8ace542e11299a13d509474b7"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"660b3f25781e4ced3df90f47e2c560bd991f6782","unresolved":false,"context_lines":[{"line_number":10259,"context_line":"        disk_info[\u0027mapping\u0027].pop(\u0027disk.local\u0027)"},{"line_number":10260,"context_line":""},{"line_number":10261,"context_line":"        with test.nested("},{"line_number":10262,"context_line":"                mock.patch.object(utils, \u0027execute\u0027),"},{"line_number":10263,"context_line":"                mock.patch.object(drvr, \u0027get_info\u0027),"},{"line_number":10264,"context_line":"                mock.patch.object(drvr, \u0027_create_domain_and_network\u0027),"},{"line_number":10265,"context_line":"                mock.patch.object(imagebackend.Image, \u0027verify_base_size\u0027),"},{"line_number":10266,"context_line":"                mock.patch.object(imagebackend.Image, \u0027get_disk_size\u0027)"},{"line_number":10267,"context_line":"        ) as (execute_mock, get_info_mock,"},{"line_number":10268,"context_line":"              create_mock, verify_base_size_mock, disk_size_mock):"},{"line_number":10269,"context_line":"            disk_size_mock.return_value \u003d 0"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_7b4f0892","line":10266,"range":{"start_line":10262,"start_character":0,"end_line":10266,"end_character":70},"in_reply_to":"3aaa91ec_dde4370e","updated":"2016-06-30 15:15:55.000000000","message":"Done","commit_id":"426d58bdf4e945b8ace542e11299a13d509474b7"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"a7ad671adb2f23ba90fe70cfe54bea94ac16c561","unresolved":false,"context_lines":[{"line_number":13308,"context_line":"    def test_shared_storage_detection_easy(self):"},{"line_number":13309,"context_line":"        drvr \u003d libvirt_driver.LibvirtDriver(fake.FakeVirtAPI(), True)"},{"line_number":13310,"context_line":"        self.mox.StubOutWithMock(drvr, \u0027get_host_ip_addr\u0027)"},{"line_number":13311,"context_line":"        self.mox.StubOutWithMock(utils, \u0027execute\u0027)"},{"line_number":13312,"context_line":"        self.mox.StubOutWithMock(os.path, \u0027exists\u0027)"},{"line_number":13313,"context_line":"        self.mox.StubOutWithMock(os, \u0027unlink\u0027)"},{"line_number":13314,"context_line":"        drvr.get_host_ip_addr().AndReturn(\u0027foo\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_fd29e1f3","side":"PARENT","line":13311,"updated":"2016-07-22 12:43:56.000000000","message":"Curious..why is this 3-line deletion part of this patch?","commit_id":"84b0f747f85120acc08ae361281ff6f730bccc58"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"deafe25ae98a1cf329ae9dccb53b8f244668fbcc","unresolved":false,"context_lines":[{"line_number":13308,"context_line":"    def test_shared_storage_detection_easy(self):"},{"line_number":13309,"context_line":"        drvr \u003d libvirt_driver.LibvirtDriver(fake.FakeVirtAPI(), True)"},{"line_number":13310,"context_line":"        self.mox.StubOutWithMock(drvr, \u0027get_host_ip_addr\u0027)"},{"line_number":13311,"context_line":"        self.mox.StubOutWithMock(utils, \u0027execute\u0027)"},{"line_number":13312,"context_line":"        self.mox.StubOutWithMock(os.path, \u0027exists\u0027)"},{"line_number":13313,"context_line":"        self.mox.StubOutWithMock(os, \u0027unlink\u0027)"},{"line_number":13314,"context_line":"        drvr.get_host_ip_addr().AndReturn(\u0027foo\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"bacf61ea_187466fb","side":"PARENT","line":13311,"in_reply_to":"dada55a8_fd29e1f3","updated":"2016-07-29 10:48:36.000000000","message":"This change comes from the work of dims, I don\u0027t know why he removed these 3 lines. When I ran the unit test using testtools.run, it failed for an unknown reason. But when I run the test using testr, the test pass with and without these lines, so I reverted this change (keep the 3 lines).","commit_id":"84b0f747f85120acc08ae361281ff6f730bccc58"}]}