)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"change_message_id":"01704e6cab3ec40c3081d784bba386c1e0a42698","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eric Fried \u003copenstack@fried.cc\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-06-10 16:59:53 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fup: Remove unused mock in libvirt functional base"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Followon for [1]: remove an unused mock from"},{"line_number":10,"context_line":"libvirt.base.ServersTestBase, and convert an existing one from"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9fb8cfa7_f998a3c0","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"updated":"2019-06-11 08:53:19.000000000","message":"Nit: Let\u0027s please use \"fixup\" in formal commit messages.\n(Also as noted elsewhere: https://review.opendev.org/#/c/664036/2//COMMIT_MSG)","commit_id":"e4cf955d27a0383c2bfa706708b10adcb1cd33d9"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"42c855de6c4b02410b8b25a8f5318249994da312","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eric Fried \u003copenstack@fried.cc\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-06-10 16:59:53 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fup: Remove unused mock in libvirt functional base"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Followon for [1]: remove an unused mock from"},{"line_number":10,"context_line":"libvirt.base.ServersTestBase, and convert an existing one from"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9fb8cfa7_08051f46","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"9fb8cfa7_f998a3c0","updated":"2019-06-11 13:46:28.000000000","message":"Done","commit_id":"e4cf955d27a0383c2bfa706708b10adcb1cd33d9"},{"author":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"change_message_id":"8828e8544cc5aa7c055f63ab1fbd5c90fe086c89","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eric Fried \u003copenstack@fried.cc\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-06-10 16:59:53 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fup: Remove unused mock in libvirt functional base"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Followon for [1]: remove an unused mock from"},{"line_number":10,"context_line":"libvirt.base.ServersTestBase, and convert an existing one from"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9fb8cfa7_595e8f76","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"9fb8cfa7_f998a3c0","updated":"2019-06-11 09:19:59.000000000","message":"Thinko: I meant, *not* use. :D","commit_id":"e4cf955d27a0383c2bfa706708b10adcb1cd33d9"}],"nova/tests/functional/libvirt/base.py":[{"author":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"change_message_id":"8828e8544cc5aa7c055f63ab1fbd5c90fe086c89","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        # Mock the \u0027get_connection\u0027 function, as we\u0027re going to need to provide"},{"line_number":55,"context_line":"        # custom capabilities for each test"},{"line_number":56,"context_line":"        self.mock_conn \u003d self.useFixture(fixtures.MockPatch("},{"line_number":57,"context_line":"            \u0027nova.virt.libvirt.host.Host.get_connection\u0027)).mock"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def _setup_compute_service(self):"},{"line_number":60,"context_line":"        # NOTE(stephenfin): We don\u0027t start the compute service here as we wish"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_19d9f7f1","line":57,"updated":"2019-06-11 09:19:59.000000000","message":"Failures seem to be (from http://logs.openstack.org/49/664449/1/check/openstack-tox-pep8/00a68d4/ara-report/):\n\n...\nRunning flake8 on all files\n./nova/tests/functional/libvirt/base.py:18:1: F401 \u0027mock\u0027 imported but unused\n./nova/tests/functional/libvirt/base.py:20:1: F401 \u0027nova.objects.fields as obj_fields\u0027 imported but unused\nERROR: InvocationError for command /bin/bash tools/flake8wrap.sh (exited with code 1)\n...","commit_id":"e4cf955d27a0383c2bfa706708b10adcb1cd33d9"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"42c855de6c4b02410b8b25a8f5318249994da312","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        # Mock the \u0027get_connection\u0027 function, as we\u0027re going to need to provide"},{"line_number":55,"context_line":"        # custom capabilities for each test"},{"line_number":56,"context_line":"        self.mock_conn \u003d self.useFixture(fixtures.MockPatch("},{"line_number":57,"context_line":"            \u0027nova.virt.libvirt.host.Host.get_connection\u0027)).mock"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def _setup_compute_service(self):"},{"line_number":60,"context_line":"        # NOTE(stephenfin): We don\u0027t start the compute service here as we wish"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_088affbd","line":57,"in_reply_to":"9fb8cfa7_19d9f7f1","updated":"2019-06-11 13:46:28.000000000","message":"Done","commit_id":"e4cf955d27a0383c2bfa706708b10adcb1cd33d9"}]}