)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"071a317b15df95cbd3e8d29f242abd655897bb1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e1a0a9a2_722c3808","updated":"2022-02-02 12:32:22.000000000","message":"Not sure if it will fix the underlying issue that caused job timeouts, but ok to clear it.","commit_id":"a2b9f07ca4a6e89aeaffbc503584e2fa1845d86d"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"b02f0aefef8575c74228e9624894250cdc47f9a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"37c74355_420eae4b","updated":"2022-02-03 07:55:05.000000000","message":"Thanks! I hit this issue in other patch sets!","commit_id":"a2b9f07ca4a6e89aeaffbc503584e2fa1845d86d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b0314d905f5af26d355f2c3a1c059acc34bfa0b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8868d5af_9a581e3f","updated":"2022-02-02 12:34:58.000000000","message":"We have fixed (I hope so) all related problems related to DB backends timeouts. This is not needed any more.","commit_id":"a2b9f07ca4a6e89aeaffbc503584e2fa1845d86d"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"b47d2515921272322f94e9f8a4b83a8531f2b1da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cd14e534_d5e18caf","updated":"2022-02-03 07:55:10.000000000","message":"recheck","commit_id":"a2b9f07ca4a6e89aeaffbc503584e2fa1845d86d"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"40728f6cde06f43609bf8192a83f55524ba285c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7a78c086_be3ccc0d","updated":"2022-02-17 10:10:33.000000000","message":"Fullstack failure indeed doesn\u0027t seems to be related to that change.","commit_id":"abc03d755ef9e4a19b159793384f3d3d3102e7c0"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"cb9a166f5aad7df4f118b685d20e3acea94097f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e1b6ada1_4d0fb8d3","updated":"2022-03-17 05:26:23.000000000","message":"The failure in neutron.tests.fullstack.test_l3_agent.TestHAL3Agent.test_ha_router_restart_agents_no_packet_lost looks related, i see it mostly taking more than 180 seconds in success jobs too.","commit_id":"abc03d755ef9e4a19b159793384f3d3d3102e7c0"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c56d651414c1c1e1edefea7d5da266bcfff619ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e962751e_8cc57a88","updated":"2022-02-17 09:35:06.000000000","message":"recheck\nfullstack mysql error?","commit_id":"abc03d755ef9e4a19b159793384f3d3d3102e7c0"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"25d34ebdb483e178d3c756d44091de63fa2d7a8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1cf7dc9f_e1a64f03","updated":"2022-03-18 12:47:57.000000000","message":"Based on the results I start to not understand how OS_TEST_TIMEOUT works. (neutron.tests.fullstack.test_securitygroup.TestSecurityGroupsSameNetwork.test_securitygroup(ovs-openflow) [756.834517s] ???)","commit_id":"b6b363330eca93602a9776f8e0d2182eaed37c3f"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"d7731d95723c9d814689bd3675c8b8378b21978d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e0ee359c_ac731dae","updated":"2022-03-18 16:38:16.000000000","message":"recheck","commit_id":"b6b363330eca93602a9776f8e0d2182eaed37c3f"}],"neutron/agent/common/async_process.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"2ed04ab766f5109913226cdae10acd5566e990dc","unresolved":true,"context_lines":[{"line_number":205,"context_line":"            # like during fullstack execution"},{"line_number":206,"context_line":"            # During fullstack tests for example processes are started as"},{"line_number":207,"context_line":"            # fixtures, and the timeout exception is fixtures.TimeoutException"},{"line_number":208,"context_line":"            except fixtures.TimeoutException:"},{"line_number":209,"context_line":"                LOG.warning(msg)"},{"line_number":210,"context_line":"                return self._kill_process(pid, signal.SIGKILL)"},{"line_number":211,"context_line":"        return True"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    def _kill_process(self, pid, kill_signal):"}],"source_content_type":"text/x-python","patch_set":3,"id":"68f1be2f_784fa914","line":210,"range":{"start_line":208,"start_character":12,"end_line":210,"end_character":62},"updated":"2022-02-16 17:50:07.000000000","message":"I don\u0027t like to have test related code in non-test code, so I will check how to come over this, but as we use the same code path in tests....","commit_id":"39a8ec24c5a41b69acf56e02a41051f9680813db"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"80dab702df3132496c4ba8cb2a7e6543ee4ce86b","unresolved":true,"context_lines":[{"line_number":198,"context_line":"        if self._process:"},{"line_number":199,"context_line":"            try:"},{"line_number":200,"context_line":"                self._process.wait(kill_timeout)"},{"line_number":201,"context_line":"            except subprocess.TimeoutExpired:"},{"line_number":202,"context_line":"                LOG.warning(msg)"},{"line_number":203,"context_line":"                return self._kill_process(pid, signal.SIGKILL)"},{"line_number":204,"context_line":"            # Note(lajoskatona): This can happen only during test runs"},{"line_number":205,"context_line":"            # like during fullstack execution"},{"line_number":206,"context_line":"            # During fullstack tests for example processes are started as"},{"line_number":207,"context_line":"            # fixtures, and the timeout exception is fixtures.TimeoutException"},{"line_number":208,"context_line":"            except fixtures.TimeoutException:"},{"line_number":209,"context_line":"                LOG.warning(msg)"},{"line_number":210,"context_line":"                return self._kill_process(pid, signal.SIGKILL)"},{"line_number":211,"context_line":"        return True"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    def _kill_process(self, pid, kill_signal):"}],"source_content_type":"text/x-python","patch_set":4,"id":"b04084cb_816fe129","line":210,"range":{"start_line":201,"start_character":0,"end_line":210,"end_character":62},"updated":"2022-03-08 07:56:41.000000000","message":"as what is done for both exceptions is the same, why not simply having something like \"except (subprocess.TimeoutExpired, fixtures.TimeoutException)\", of course with the note about the testing problem","commit_id":"abc03d755ef9e4a19b159793384f3d3d3102e7c0"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3831d0a003c4089a8266eea2c75bc0cd23749077","unresolved":true,"context_lines":[{"line_number":198,"context_line":"        if self._process:"},{"line_number":199,"context_line":"            try:"},{"line_number":200,"context_line":"                self._process.wait(kill_timeout)"},{"line_number":201,"context_line":"            except subprocess.TimeoutExpired:"},{"line_number":202,"context_line":"                LOG.warning(msg)"},{"line_number":203,"context_line":"                return self._kill_process(pid, signal.SIGKILL)"},{"line_number":204,"context_line":"            # Note(lajoskatona): This can happen only during test runs"},{"line_number":205,"context_line":"            # like during fullstack execution"},{"line_number":206,"context_line":"            # During fullstack tests for example processes are started as"},{"line_number":207,"context_line":"            # fixtures, and the timeout exception is fixtures.TimeoutException"},{"line_number":208,"context_line":"            except fixtures.TimeoutException:"},{"line_number":209,"context_line":"                LOG.warning(msg)"},{"line_number":210,"context_line":"                return self._kill_process(pid, signal.SIGKILL)"},{"line_number":211,"context_line":"        return True"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    def _kill_process(self, pid, kill_signal):"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff4da402_44f825b9","line":210,"range":{"start_line":201,"start_character":0,"end_line":210,"end_character":62},"in_reply_to":"b04084cb_816fe129","updated":"2022-03-17 13:11:46.000000000","message":"Sorry I missed this comment....\nI would keep the 2 exceptions separated as fixtures.TimeoutException is only for test environments","commit_id":"abc03d755ef9e4a19b159793384f3d3d3102e7c0"}]}