)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"aae3f070a2a03fdd9d632dc5e888ac71c3746e2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"471fbff1_804a3ff5","updated":"2025-02-24 05:56:21.000000000","message":"PS4 resolved a merge conflict","commit_id":"6d5296c6a317c641c8cca1ea6b557d0d09a64227"}],"zuul/driver/gerrit/gerritconnection.py":[{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"8fd80d091108cd27ac3a3710e5e195dcf2f26c3b","unresolved":true,"context_lines":[{"line_number":852,"context_line":"                    dep not in needed_by_changes):"},{"line_number":853,"context_line":"                    git_needed_by_changes.append(dep_key.reference)"},{"line_number":854,"context_line":"                    needed_by_changes.add(dep_key.reference)"},{"line_number":855,"context_line":"            except GerritEventProcessingException:"},{"line_number":856,"context_line":"                raise"},{"line_number":857,"context_line":"            except Exception:"},{"line_number":858,"context_line":"                log.exception(\"Failed to get git-needed change %s,%s\","}],"source_content_type":"text/x-python","patch_set":2,"id":"77c8345a_96cf2431","side":"PARENT","line":855,"updated":"2025-02-20 08:54:44.000000000","message":"As discussed out-of-band, we should add a comment here on why we are ignoring the `GerritEventProcessingException` here and in line 884","commit_id":"ce4063690b1df9141e35f1da1e7052d07ddae854"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"dbe63044d7b1ffb9d543d756273609cee0fe3ae0","unresolved":false,"context_lines":[{"line_number":852,"context_line":"                    dep not in needed_by_changes):"},{"line_number":853,"context_line":"                    git_needed_by_changes.append(dep_key.reference)"},{"line_number":854,"context_line":"                    needed_by_changes.add(dep_key.reference)"},{"line_number":855,"context_line":"            except GerritEventProcessingException:"},{"line_number":856,"context_line":"                raise"},{"line_number":857,"context_line":"            except Exception:"},{"line_number":858,"context_line":"                log.exception(\"Failed to get git-needed change %s,%s\","}],"source_content_type":"text/x-python","patch_set":2,"id":"a5b5fa83_2194250c","side":"PARENT","line":855,"in_reply_to":"77c8345a_96cf2431","updated":"2025-02-20 10:16:10.000000000","message":"Done","commit_id":"ce4063690b1df9141e35f1da1e7052d07ddae854"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"f76ff1287c043cc2c50700e011ab0e858ed4d349","unresolved":true,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class HTTPNotFoundException(Exception):"},{"line_number":94,"context_line":"    message \u003d \"Received response 404\""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"class GerritEventProcessingException(Exception):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa9e1a45_852bfbec","line":94,"updated":"2025-02-20 06:42:31.000000000","message":"nit: I think this message is not used","commit_id":"5c423e65581f8c6ecc240909c71232ad6c5e88ef"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"dbe63044d7b1ffb9d543d756273609cee0fe3ae0","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class HTTPNotFoundException(Exception):"},{"line_number":94,"context_line":"    message \u003d \"Received response 404\""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"class GerritEventProcessingException(Exception):"}],"source_content_type":"text/x-python","patch_set":2,"id":"b0aeca16_6f1c0108","line":94,"in_reply_to":"fa9e1a45_852bfbec","updated":"2025-02-20 10:16:10.000000000","message":"Done","commit_id":"5c423e65581f8c6ecc240909c71232ad6c5e88ef"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"f76ff1287c043cc2c50700e011ab0e858ed4d349","unresolved":true,"context_lines":[{"line_number":1350,"context_line":"                # do not retry on 404 results, instead skip further event"},{"line_number":1351,"context_line":"                # processing since we can\u0027t load a corresponding change"},{"line_number":1352,"context_line":"                raise GerritEventProcessingException("},{"line_number":1353,"context_line":"                    f\"Did not find change for number {number}\", e)"},{"line_number":1354,"context_line":"            except Exception:"},{"line_number":1355,"context_line":"                self.log.exception(\"Failed to query change.\")"},{"line_number":1356,"context_line":"                if attempt \u003c max_attempts:"}],"source_content_type":"text/x-python","patch_set":2,"id":"21e2e8d5_4c140d17","line":1353,"updated":"2025-02-20 06:42:31.000000000","message":"nit: you could do a `GerritEventProcessingException from e` to preserve the traceback in a better way than to pass it to the constructor.","commit_id":"5c423e65581f8c6ecc240909c71232ad6c5e88ef"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"dbe63044d7b1ffb9d543d756273609cee0fe3ae0","unresolved":false,"context_lines":[{"line_number":1350,"context_line":"                # do not retry on 404 results, instead skip further event"},{"line_number":1351,"context_line":"                # processing since we can\u0027t load a corresponding change"},{"line_number":1352,"context_line":"                raise GerritEventProcessingException("},{"line_number":1353,"context_line":"                    f\"Did not find change for number {number}\", e)"},{"line_number":1354,"context_line":"            except Exception:"},{"line_number":1355,"context_line":"                self.log.exception(\"Failed to query change.\")"},{"line_number":1356,"context_line":"                if attempt \u003c max_attempts:"}],"source_content_type":"text/x-python","patch_set":2,"id":"a1d4c761_ec96fc27","line":1353,"in_reply_to":"21e2e8d5_4c140d17","updated":"2025-02-20 10:16:10.000000000","message":"Done","commit_id":"5c423e65581f8c6ecc240909c71232ad6c5e88ef"}]}
