)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c6d33289a19978418f73b65f30b24e85c355de2b","unresolved":true,"context_lines":[{"line_number":32,"context_line":"NOTE: This patch had to be slightly re-worked for back-porting: namely"},{"line_number":33,"context_line":"thread_pool_factory does not exist, so the previous method of using"},{"line_number":34,"context_line":"nova.utils.{executor,spawn_on,etc} is used."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Related-Bug: 2162924"},{"line_number":37,"context_line":"Generated-By: Cursor sonnet 4.6"},{"line_number":38,"context_line":"Change-Id: I36163947a09b82cfc40ab4b522d82ffb503a0b84"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3279d4fd_42eec19d","line":35,"updated":"2026-10-16 20:27:10.000000000","message":"+1\nthis is needed bcause these are helper we intoduced for eventlet removal so as we go backwards we expect to need to update these as we have been consolidating and refining those helper each release","commit_id":"caa28ac71b9407360950215c471da77c8c7ee323"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":14228,"name":"Clif Houck","email":"me@clifhouck.com","username":"clif_h"},"change_message_id":"d506968320ebff1ad78e7824b3eb88f4d27d176a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"82828322_13596f7c","updated":"2026-09-14 15:04:55.000000000","message":"recheck tests pass but overall result times out","commit_id":"caa28ac71b9407360950215c471da77c8c7ee323"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"cbe3ebe446d58b220465290abd38cc1075db65d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a8079592_68b122b6","updated":"2026-10-02 15:16:47.000000000","message":"recheck tests pass but overall result times out","commit_id":"caa28ac71b9407360950215c471da77c8c7ee323"},{"author":{"_account_id":14228,"name":"Clif Houck","email":"me@clifhouck.com","username":"clif_h"},"change_message_id":"5064cd19866711e8f0ff060b93a5bf5b470f35bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"891e76c0_9fdfde8e","updated":"2026-10-06 17:49:58.000000000","message":"recheck, CI passed but timed out","commit_id":"caa28ac71b9407360950215c471da77c8c7ee323"}],"nova/compute/manager.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c6d33289a19978418f73b65f30b24e85c355de2b","unresolved":true,"context_lines":[{"line_number":671,"context_line":"        self._syncs_in_progress: set[str] \u003d set()"},{"line_number":672,"context_line":"        self._syncs_in_progress_lock \u003d threading.Lock()"},{"line_number":673,"context_line":"        self._update_resources_executor \u003d nova.utils.create_executor("},{"line_number":674,"context_line":"            max_workers\u003dCONF.compute.update_resources_max_workers)"},{"line_number":675,"context_line":"        self.send_instance_updates \u003d ("},{"line_number":676,"context_line":"            CONF.filter_scheduler.track_instance_changes)"},{"line_number":677,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"99e5c2ec_dac7cc4f","line":674,"updated":"2026-10-16 20:27:10.000000000","message":"+1 this is close to an eairer revision fo the master change\nhttps://review.opendev.org/c/openstack/nova/+/980679/9/nova/compute/manager.py#673\n\nbefore we adapted it for the new thread_pool_factory that is not present on stable","commit_id":"caa28ac71b9407360950215c471da77c8c7ee323"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c6d33289a19978418f73b65f30b24e85c355de2b","unresolved":true,"context_lines":[{"line_number":11581,"context_line":"                    context, nodename, startup\u003dstartup),"},{"line_number":11582,"context_line":"            )"},{"line_number":11583,"context_line":"            for nodename in nodenames"},{"line_number":11584,"context_line":"        ]"},{"line_number":11585,"context_line":"        # Drain all futures so errors are re-raised in this thread rather"},{"line_number":11586,"context_line":"        # than being silently lost. Keep draining after the first failure"},{"line_number":11587,"context_line":"        # so all scheduled per-node updates finish before we preserve the"}],"source_content_type":"text/x-python","patch_set":1,"id":"fb5dfb7e_74879d07","line":11584,"updated":"2026-10-16 20:27:10.000000000","message":"and this is the equivlent logic from v9 which is compatible with v17\n\nhttps://review.opendev.org/c/openstack/nova/+/980679/9/nova/compute/manager.py#11476","commit_id":"caa28ac71b9407360950215c471da77c8c7ee323"}],"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c6d33289a19978418f73b65f30b24e85c355de2b","unresolved":false,"context_lines":[{"line_number":562,"context_line":"                self.context, nodename, startup\u003dTrue)"},{"line_number":563,"context_line":"        for fut in futures:"},{"line_number":564,"context_line":"            fut.result.assert_called_once_with()"},{"line_number":565,"context_line":""},{"line_number":566,"context_line":"    @mock.patch(\u0027nova.utils.spawn_on\u0027)"},{"line_number":567,"context_line":"    @mock.patch.object(fake_driver.FakeDriver, \u0027get_available_nodes\u0027)"},{"line_number":568,"context_line":"    @mock.patch.object(manager.ComputeManager, \u0027_get_compute_nodes_in_db\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"77aafc72_e53cbaf2","line":565,"updated":"2026-10-16 20:27:10.000000000","message":"again these are bsically the same with minor adoptation to the slightly different executor interface in 2026.1","commit_id":"caa28ac71b9407360950215c471da77c8c7ee323"}]}
