)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"3579a4899492bb6c4eba5a7ed5eaa635245ff553","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"56bb54de_a973950b","updated":"2021-12-06 19:24:55.000000000","message":"Looks reasonable to me","commit_id":"c59224d715a21998f40f72cf4e37efdc990e4d7e"}],"nova/tests/functional/libvirt/test_numa_servers.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"092a9f1f6908f7e95dc5a93bc160a927691e9dd1","unresolved":true,"context_lines":[{"line_number":929,"context_line":"        self._assert_pinned_cpus(src_host, 2)"},{"line_number":930,"context_line":"        self._assert_pinned_cpus(dst_host, 0)"},{"line_number":931,"context_line":""},{"line_number":932,"context_line":"    def test_resize_dedicated_policy_race_on_dest_bug_1953359(self):"},{"line_number":933,"context_line":""},{"line_number":934,"context_line":"        self.flags(cpu_dedicated_set\u003d\u00270-2\u0027, cpu_shared_set\u003dNone,"},{"line_number":935,"context_line":"                   group\u003d\u0027compute\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e31c7dcd_fcb885d2","line":932,"updated":"2021-12-07 10:40:46.000000000","message":"well, this could have been a regression test [1] but that\u0027s a nit.\n\n\n[1] https://github.com/openstack/nova/tree/master/nova/tests/functional/regressions","commit_id":"c59224d715a21998f40f72cf4e37efdc990e4d7e"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"092a9f1f6908f7e95dc5a93bc160a927691e9dd1","unresolved":true,"context_lines":[{"line_number":979,"context_line":"            # we expect it that CPU pinning fails on the destination node"},{"line_number":980,"context_line":"            # as the resource_tracker will use the source node numa_topology"},{"line_number":981,"context_line":"            # and that does not fit to the dest node as pcpu 0 in the dest"},{"line_number":982,"context_line":"            # is already occupied."},{"line_number":983,"context_line":""},{"line_number":984,"context_line":"        # TODO(stephenfin): The mock of \u0027migrate_disk_and_power_off\u0027 should"},{"line_number":985,"context_line":"        # probably be less...dumb"}],"source_content_type":"text/x-python","patch_set":1,"id":"dd6f290f_04bc087c","line":982,"updated":"2021-12-07 10:40:46.000000000","message":"ok, so you only verify that the race goes when running the periodic time, but fair enough.","commit_id":"c59224d715a21998f40f72cf4e37efdc990e4d7e"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"092a9f1f6908f7e95dc5a93bc160a927691e9dd1","unresolved":true,"context_lines":[{"line_number":1009,"context_line":"            \u0027a subset of free CPU set [1]\u0027,"},{"line_number":1010,"context_line":"            log,"},{"line_number":1011,"context_line":"        )"},{"line_number":1012,"context_line":""},{"line_number":1013,"context_line":""},{"line_number":1014,"context_line":"class NUMAServerTestWithCountingQuotaFromPlacement(NUMAServersTest):"},{"line_number":1015,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"46392b7d_e4b49e4e","line":1012,"updated":"2021-12-07 10:40:46.000000000","message":"as a regression test, I\u0027d have preferred to see within a comment what would be the good behaviour, but meh, I already know it ;)","commit_id":"c59224d715a21998f40f72cf4e37efdc990e4d7e"}]}