)]}'
{"nova/tests/fixtures.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0868c90393c89ddd5ecfcbdc2fa55679748524e0","unresolved":false,"context_lines":[{"line_number":3186,"context_line":"                        meth, poison_configure(meth, why)))"},{"line_number":3187,"context_line":"            except ImportError:"},{"line_number":3188,"context_line":"                self.useFixture(fixtures.MonkeyPatch("},{"line_number":3189,"context_line":"                    meth, poison_configure(meth, why)))"}],"source_content_type":"text/x-python","patch_set":2,"id":"386c670e_7ca916af","line":3189,"updated":"2021-03-09 15:24:27.000000000","message":"This is necessary because some tests start the virt driver as part of setUp, but doing that with the libvirt driver enabled requires mocking out libvirt first. Calling this means we end up setting the \u0027os.uname()\u0027 mock to return a valid value, but we then go and override it again with the generic poison fixture.","commit_id":"452d2fb3a03cff3afb99a6d0b3fe34ceaa6c74eb"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"753a2908eb10f066d9227740f8a5826d99ebd87b","unresolved":false,"context_lines":[{"line_number":3186,"context_line":"                        meth, poison_configure(meth, why)))"},{"line_number":3187,"context_line":"            except ImportError:"},{"line_number":3188,"context_line":"                self.useFixture(fixtures.MonkeyPatch("},{"line_number":3189,"context_line":"                    meth, poison_configure(meth, why)))"}],"source_content_type":"text/x-python","patch_set":2,"id":"55773d7f_d962b1d3","line":3189,"in_reply_to":"386c670e_7ca916af","updated":"2021-03-11 11:07:41.000000000","message":"Thnaks for the comment, maybe it should also be good to have a explanation in there and not only in the Gerrit change (that\u0027s a nit for a FUP)","commit_id":"452d2fb3a03cff3afb99a6d0b3fe34ceaa6c74eb"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"d9366ce938b1dd8cc00a2f0f68ff068add48afca","unresolved":true,"context_lines":[{"line_number":4712,"context_line":"        extra_flags \u003d set([flag.lower() for flag in"},{"line_number":4713,"context_line":"            CONF.libvirt.cpu_model_extra_flags])"},{"line_number":4714,"context_line":""},{"line_number":4715,"context_line":"        if not arch:"},{"line_number":4716,"context_line":"            caps \u003d self._host.get_capabilities()"},{"line_number":4717,"context_line":"            arch \u003d caps.host.cpu.arch"},{"line_number":4718,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fbc544a9_d8c93957","line":4715,"range":{"start_line":4715,"start_character":11,"end_line":4715,"end_character":20},"updated":"2021-03-11 09:54:02.000000000","message":"nit - arch is None","commit_id":"452d2fb3a03cff3afb99a6d0b3fe34ceaa6c74eb"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"753a2908eb10f066d9227740f8a5826d99ebd87b","unresolved":true,"context_lines":[{"line_number":4712,"context_line":"        extra_flags \u003d set([flag.lower() for flag in"},{"line_number":4713,"context_line":"            CONF.libvirt.cpu_model_extra_flags])"},{"line_number":4714,"context_line":""},{"line_number":4715,"context_line":"        if not arch:"},{"line_number":4716,"context_line":"            caps \u003d self._host.get_capabilities()"},{"line_number":4717,"context_line":"            arch \u003d caps.host.cpu.arch"},{"line_number":4718,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"36c37cb6_ab9e6ca6","line":4715,"range":{"start_line":4715,"start_character":11,"end_line":4715,"end_character":20},"in_reply_to":"fbc544a9_d8c93957","updated":"2021-03-11 11:07:41.000000000","message":"LGTM for a follow-up.","commit_id":"452d2fb3a03cff3afb99a6d0b3fe34ceaa6c74eb"}]}