)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"c5d4e16a992045ddd6f1561374932488b84082c5","unresolved":true,"context_lines":[{"line_number":13,"context_line":"adds 3 seconds of slowness to the test execution."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Conflicts:"},{"line_number":16,"context_line":"  nova/tests/functional/regressions/test_bug_1896463.py due to"},{"line_number":17,"context_line":"  I84c58de90dad6d86271767363aef90ddac0f1730 and"},{"line_number":18,"context_line":"  I8c96b337f32148f8f5899c9b87af331b1fa41424 is not in stable/train."},{"line_number":19,"context_line":"  Also I had to make the test code python 2 compatible."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"30971f50_bc9a5b50","line":16,"updated":"2022-06-21 17:57:01.000000000","message":"These merged in the meantime, so this is now a clean cherry-pick.","commit_id":"50164f840f1d9dc650ca8f7ebe6b479c8077b384"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"f890eb7bb7acf42e91cdfa10b70d82e310f318ed","unresolved":true,"context_lines":[{"line_number":13,"context_line":"adds 3 seconds of slowness to the test execution."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Conflicts:"},{"line_number":16,"context_line":"  nova/tests/functional/regressions/test_bug_1896463.py due to"},{"line_number":17,"context_line":"  I84c58de90dad6d86271767363aef90ddac0f1730 and"},{"line_number":18,"context_line":"  I8c96b337f32148f8f5899c9b87af331b1fa41424 is not in stable/train."},{"line_number":19,"context_line":"  Also I had to make the test code python 2 compatible."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a7837492_3db839bf","line":16,"in_reply_to":"30971f50_bc9a5b50","updated":"2022-11-25 08:30:52.000000000","message":"I\u0027m not seeing I84c58de90dad6d86271767363aef90ddac0f1730 being in stable/train.","commit_id":"50164f840f1d9dc650ca8f7ebe6b479c8077b384"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"c5d4e16a992045ddd6f1561374932488b84082c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2362b9e1_7729c633","updated":"2022-06-21 17:57:01.000000000","message":"The changes pointed","commit_id":"50164f840f1d9dc650ca8f7ebe6b479c8077b384"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"fe90bf7c15dc8e9300efd3cffb5ace31da68c4fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ad593b02_e294e09a","updated":"2022-05-31 08:13:44.000000000","message":"recheck lower-constraints is dropped","commit_id":"50164f840f1d9dc650ca8f7ebe6b479c8077b384"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f39637c36ad7fb9a06b3a1724171ea4373f24cb9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"43163405_9e91f338","updated":"2022-11-25 08:48:14.000000000","message":"this is not perfect but if you adress the nits inline im ok with rervieing this\noverall its equvalent if not identical to the ussuri version so im happy with the overall content","commit_id":"e558aa2046b649696b69551657e5d429e8117427"}],"nova/tests/functional/regressions/test_bug_1896463.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f39637c36ad7fb9a06b3a1724171ea4373f24cb9","unresolved":true,"context_lines":[{"line_number":54,"context_line":"            api_version\u003d\u0027v2.1\u0027))"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        self.admin_api \u003d self.api_fixture.admin_api"},{"line_number":57,"context_line":"        self.admin_api \u003d self.api_fixture.admin_api"},{"line_number":58,"context_line":"        self.admin_api.microversion \u003d \u0027latest\u0027"},{"line_number":59,"context_line":"        self.api \u003d self.admin_api"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"713b11e1_edb83671","line":57,"updated":"2022-11-25 08:48:14.000000000","message":"nit: you repeated this line.","commit_id":"e558aa2046b649696b69551657e5d429e8117427"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f39637c36ad7fb9a06b3a1724171ea4373f24cb9","unresolved":true,"context_lines":[{"line_number":121,"context_line":"            image_uuid\u003d\u0027155d900f-4e14-4e4c-a73d-069cbf4541e6\u0027,"},{"line_number":122,"context_line":"            networks\u003d[{\u0027port\u0027: self.neutron.sriov_port[\u0027id\u0027]}])"},{"line_number":123,"context_line":"        server_req[\u0027availability_zone\u0027] \u003d \u0027nova:host1\u0027"},{"line_number":124,"context_line":"        created_server \u003d self.api.post_server({\u0027server\u0027: server_req})"},{"line_number":125,"context_line":"        server \u003d self._wait_for_state_change("},{"line_number":126,"context_line":"            self.admin_api, created_server, \u0027ACTIVE\u0027)"},{"line_number":127,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"133b5887_d8aaa8e9","line":124,"updated":"2022-11-25 08:48:14.000000000","message":"hear you have adapted to the lack of _create_server \nand used tha az hack to specify the host rahter then the host parmater on the create request but functionally it will be the same in this case.","commit_id":"e558aa2046b649696b69551657e5d429e8117427"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f39637c36ad7fb9a06b3a1724171ea4373f24cb9","unresolved":true,"context_lines":[{"line_number":124,"context_line":"        created_server \u003d self.api.post_server({\u0027server\u0027: server_req})"},{"line_number":125,"context_line":"        server \u003d self._wait_for_state_change("},{"line_number":126,"context_line":"            self.admin_api, created_server, \u0027ACTIVE\u0027)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        self._assert_pci_device_allocated(server[\u0027id\u0027], self.compute1_id)"},{"line_number":129,"context_line":"        self._assert_pci_device_allocated("},{"line_number":130,"context_line":"            server[\u0027id\u0027], self.compute2_id, num\u003d0)"}],"source_content_type":"text/x-python","patch_set":3,"id":"c46cbcfa_76c3c1c4","line":127,"updated":"2022-11-25 08:48:14.000000000","message":"and you have to explcitly wait since you are not using the helper.","commit_id":"e558aa2046b649696b69551657e5d429e8117427"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f39637c36ad7fb9a06b3a1724171ea4373f24cb9","unresolved":true,"context_lines":[{"line_number":222,"context_line":"        # the already started evacuation"},{"line_number":223,"context_line":"        ctx \u003d context.get_admin_context()"},{"line_number":224,"context_line":"        self.compute1.manager.update_available_resource(ctx)"},{"line_number":225,"context_line":"        self.compute2.manager.update_available_resource(ctx)"},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"        self._wait_for_server_parameter("},{"line_number":228,"context_line":"            self.admin_api,"}],"source_content_type":"text/x-python","patch_set":3,"id":"8b9942fc_95355ff0","line":225,"updated":"2022-11-25 08:48:14.000000000","message":"so this seams to be the only real delta\n\nin ussuri you call _run_periodics and train you just call update_avilable_resource directly since we dont have that helper.\n\nthis is eqiuvalent sincce that the only periodic we care about so im fine with this change","commit_id":"e558aa2046b649696b69551657e5d429e8117427"}]}