)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"393294b469b01d574f585195647b3f821fcd995f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Stephen Finucane \u003csfinucan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-08-15 16:56:49 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"libvirt: Mock libvirt\u0027y things globally"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"While addressing my nit from change"},{"line_number":10,"context_line":"I8e5a122cc547222249973cf595d90c2d8d5658d4, I realized pretty much every"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7faddb67_39ee4522","line":7,"range":{"start_line":7,"start_character":31,"end_line":7,"end_character":39},"updated":"2019-08-15 17:24:58.000000000","message":"This isn\u0027t a great word choice, made me think about actual global mocks, which would be bad.","commit_id":"3f5348a834444623b2471afacd5e53baacbb7a92"}],"nova/tests/functional/libvirt/base.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e3189d43687faa31f1cd07401cefe11d70bb0330","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        self.useFixture(fixtures.MockPatch("},{"line_number":55,"context_line":"            \u0027nova.virt.libvirt.driver.libvirt_utils.is_valid_hostname\u0027,"},{"line_number":56,"context_line":"            return_value\u003dTrue))"},{"line_number":57,"context_line":"        self.useFixture(fixtures.MockPatch("},{"line_number":58,"context_line":"            \u0027nova.virt.libvirt.driver.libvirt_utils.file_open\u0027,"},{"line_number":59,"context_line":"            side_effect\u003d[io.BytesIO(b\u0027\u0027), io.BytesIO(b\u0027\u0027)]))"},{"line_number":60,"context_line":"        self.useFixture(fixtures.MockPatch("},{"line_number":61,"context_line":"            \u0027nova.privsep.utils.supports_direct_io\u0027,"},{"line_number":62,"context_line":"            return_value\u003dTrue))"}],"source_content_type":"text/x-python","patch_set":2,"id":"5faad753_cbae972d","line":59,"range":{"start_line":57,"start_character":8,"end_line":59,"end_character":60},"updated":"2019-09-09 15:49:20.000000000","message":"This means that at most 2 servers can be created per test case that uses this fixture, which is extremely fragile and not obvious to debug when things start failing in weird ways.\n\nI realize this patch just moved things here, but it was bad to begin with.","commit_id":"85ea4f703d802ea9f3f8e40edefe4df2e73c53ad"}]}
