)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"31a8722f_843aaaf7","updated":"2023-10-16 18:36:49.000000000","message":"It probably could be outside of patch\u0027s scope, but AFAIK, it is generally recommended to start new comment with capitalized letter.","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"364f2f95fbdc6d548e7d6a83f3b8ce7962d12fec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e3eff096_a19f6337","updated":"2023-12-13 19:35:12.000000000","message":"recheck post failure is likely becasue of a roge vm using an ip it shoudl not be...","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"aed76e59b80530296806115bb9254a061ddbeca8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cad8177d_57308d9d","updated":"2023-12-13 14:18:55.000000000","message":"recheck tempest-integrated-compute-enforce-scope-new-defaults timed out","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5caca48ba54bfea67d0a8df59f97b3576805b96e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"de186117_84843dd7","in_reply_to":"31a8722f_843aaaf7","updated":"2023-11-21 17:23:52.000000000","message":"that is not a rule we follow in OpenStack\nand unless it\u0027s enforced by a tool it\u0027s not one I would like to introduce.","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}],"nova/tests/fixtures/nova.py":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":true,"context_lines":[{"line_number":990,"context_line":"        loader \u003d wsgi.Loader().load_app(service_name)"},{"line_number":991,"context_line":"        app \u003d lambda: loader"},{"line_number":992,"context_line":""},{"line_number":993,"context_line":"        # reuse service setup code from wsgi_app to register"},{"line_number":994,"context_line":"        # service, which is looked for in some tests"},{"line_number":995,"context_line":"        wsgi_app._setup_service(CONF.host, service_name)"},{"line_number":996,"context_line":"        intercept \u003d interceptor.RequestsInterceptor(app, url\u003dendpoint)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b3da26b6_59969b0f","line":993,"range":{"start_line":993,"start_character":10,"end_line":993,"end_character":15},"updated":"2023-10-16 18:36:49.000000000","message":"and here","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}],"nova/tests/functional/libvirt/test_numa_servers.py":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":true,"context_lines":[{"line_number":849,"context_line":"            self.assertEqual(2, len(src_numa_topology.cells[0].pinned_cpus))"},{"line_number":850,"context_line":"            self.assertEqual(2, len(dst_numa_topology.cells[0].pinned_cpus))"},{"line_number":851,"context_line":""},{"line_number":852,"context_line":"            # before continuing with the actually confirm process"},{"line_number":853,"context_line":"            return orig_confirm(*args, **kwargs)"},{"line_number":854,"context_line":""},{"line_number":855,"context_line":"        self.stub_out("}],"source_content_type":"text/x-python","patch_set":1,"id":"f4efc246_dd975f9b","line":852,"range":{"start_line":852,"start_character":14,"end_line":852,"end_character":20},"updated":"2023-10-16 18:36:49.000000000","message":"And here","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}],"nova/tests/functional/libvirt/test_pci_in_placement.py":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":true,"context_lines":[{"line_number":1419,"context_line":"        # * one type-PFs (slot 2) with 1 type-VFs"},{"line_number":1420,"context_line":"        compute2_pci_info \u003d fakelibvirt.HostPCIDevicesInfo("},{"line_number":1421,"context_line":"            num_pci\u003d2, num_pfs\u003d1, num_vfs\u003d1)"},{"line_number":1422,"context_line":"        # the config matches the PCI devs and the PF but not the VFs"},{"line_number":1423,"context_line":"        compute2_device_spec \u003d self._to_list_of_json_str("},{"line_number":1424,"context_line":"            ["},{"line_number":1425,"context_line":"                {"}],"source_content_type":"text/x-python","patch_set":1,"id":"5b703e39_06d21d22","line":1422,"range":{"start_line":1422,"start_character":10,"end_line":1422,"end_character":13},"updated":"2023-10-16 18:36:49.000000000","message":"And here","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}],"nova/tests/unit/compute/test_compute.py":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":true,"context_lines":[{"line_number":5904,"context_line":"        updated_topology \u003d objects.NUMATopology.obj_from_primitive("},{"line_number":5905,"context_line":"            jsonutils.loads(cn.numa_topology))"},{"line_number":5906,"context_line":""},{"line_number":5907,"context_line":"        # after confirming resize all cpus on current host must be free"},{"line_number":5908,"context_line":"        self.assertEqual(2, len(updated_topology.cells))"},{"line_number":5909,"context_line":"        for cell in updated_topology.cells:"},{"line_number":5910,"context_line":"            self.assertEqual(set(), cell.pinned_cpus)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e94506b0_618b677d","line":5907,"range":{"start_line":5907,"start_character":10,"end_line":5907,"end_character":15},"updated":"2023-10-16 18:36:49.000000000","message":"It is probably better to start with an upper case","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}],"nova/tests/unit/conductor/tasks/test_live_migrate.py":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":true,"context_lines":[{"line_number":858,"context_line":"                supp_binding_ext_retval"},{"line_number":859,"context_line":"            self.task.instance.pci_requests \u003d instance_pci_reqs"},{"line_number":860,"context_line":"            self.task._check_can_migrate_pci(\"Src\", \"Dst\")"},{"line_number":861,"context_line":"            # in case we managed to get away without raising, check mocks"},{"line_number":862,"context_line":"            if instance_pci_reqs:"},{"line_number":863,"context_line":"                mock_supp_port_binding_ext.assert_called_once_with("},{"line_number":864,"context_line":"                    self.context)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a9744ac1_34c0c0e8","line":861,"range":{"start_line":861,"start_character":14,"end_line":861,"end_character":16},"updated":"2023-10-16 18:36:49.000000000","message":"Probably makes sense to capitalize","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}],"nova/tests/unit/virt/libvirt/test_driver.py":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":true,"context_lines":[{"line_number":22794,"context_line":"                                                         mock_get_disk_info):"},{"line_number":22795,"context_line":"        # Check error on resize root disk down for rbd."},{"line_number":22796,"context_line":"        # The difference is that get_instance_disk_info always returns"},{"line_number":22797,"context_line":"        # an empty list for rbd."},{"line_number":22798,"context_line":"        # Ephemeral size is not changed in this case (otherwise other check"},{"line_number":22799,"context_line":"        # will raise the same error)."},{"line_number":22800,"context_line":"        self.flags(images_type\u003d\u0027rbd\u0027, group\u003d\u0027libvirt\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f714e616_094b8d14","line":22797,"range":{"start_line":22797,"start_character":10,"end_line":22797,"end_character":12},"updated":"2023-10-16 18:36:49.000000000","message":"Probably should start with an upper case.","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}],"nova/tests/unit/virt/vmwareapi/test_vm_util.py":[{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"7c6b47ac146f3882f87f5ef076d146dfed2d95dc","unresolved":true,"context_lines":[{"line_number":1025,"context_line":"        \"\"\"Ensure we warn when create_vm() fails after we passed an"},{"line_number":1026,"context_line":"        unrecognised guestId"},{"line_number":1027,"context_line":"        \"\"\""},{"line_number":1028,"context_line":"        # avoid real sleeps during test due to the retry decorator on create_vm"},{"line_number":1029,"context_line":"        self.useFixture(oslo_svc_fixture.SleepFixture())"},{"line_number":1030,"context_line":""},{"line_number":1031,"context_line":"        found \u003d [False]"}],"source_content_type":"text/x-python","patch_set":1,"id":"cc287e98_3f9200fb","line":1028,"range":{"start_line":1028,"start_character":10,"end_line":1028,"end_character":15},"updated":"2023-10-16 18:36:49.000000000","message":"Shouldn\u0027t we capitalize it?","commit_id":"2232ca95f22478bc90da5cb2d99cb0a4570fbc96"}]}