)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0f2deadeb7b8b82d7ef920bc0f8206960cd7b104","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a1be5c6b_763d8399","updated":"2026-03-19 14:12:39.000000000","message":"Odd it didn\u0027t throw a pep8 error when it was proposed, I\u0027m assuming there isn\u0027t something wrong with our check queue?","commit_id":"aa7aa135b0ccae77fb077f4d30d13172ab5173b3"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"0e5aa1a4d9903f58111909393b492a0e5c9b2774","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"44c15a81_666691d3","in_reply_to":"0657a59c_aa7a9342","updated":"2026-03-19 15:58:55.000000000","message":"I thought it was intentional :)","commit_id":"aa7aa135b0ccae77fb077f4d30d13172ab5173b3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"bd28da79ce1c8c7dbd29449c497973c099639c15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0657a59c_aa7a9342","in_reply_to":"6515f3b2_f34ec0a9","updated":"2026-03-19 15:30:08.000000000","message":"I thought I fixed it so we did? hmm","commit_id":"aa7aa135b0ccae77fb077f4d30d13172ab5173b3"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6687e17a0526e1239e1295d8b1fedf1bf7b411a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6515f3b2_f34ec0a9","in_reply_to":"a1be5c6b_763d8399","updated":"2026-03-19 15:04:17.000000000","message":"I think we do not run pep8 checks on the testing code","commit_id":"aa7aa135b0ccae77fb077f4d30d13172ab5173b3"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"cd95b885c5114e13cf0240baeec18b19803c9d9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b60dc583_4e61078d","updated":"2026-03-19 15:21:45.000000000","message":"well there were some errors the pep8 job didn\u0027t catch on the prod code too","commit_id":"9809f3171d92ad45dd80aae0fd889697921f0950"}],"neutron/tests/functional/services/bgp/test_commands.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"c4cc2edccecdf978c5de52eebfd4227648c308d8","unresolved":true,"context_lines":[{"line_number":1344,"context_line":"            main_router_name).execute(check_error\u003dTrue)"},{"line_number":1345,"context_line":"        for policy in policies:"},{"line_number":1346,"context_line":"            if (policy.match \u003d\u003d expected_match and"},{"line_number":1347,"context_line":"                    policy.priority \u003d\u003d (constants."},{"line_number":1348,"context_line":"                                        LR_BGP_TO_CHASSIS_POLICY_PRIORITY)):"},{"line_number":1349,"context_line":"                self.fail("},{"line_number":1350,"context_line":"                    \"Policy with match %r should not exist after delete\" % ("},{"line_number":1351,"context_line":"                        expected_match,))"}],"source_content_type":"text/x-python","patch_set":1,"id":"b4ba6146_33a4154a","line":1348,"range":{"start_line":1347,"start_character":0,"end_line":1348,"end_character":76},"updated":"2026-03-19 12:43:29.000000000","message":"I think the correct fix is just to move L1348 one block to the left, so it\u0027s\n```\n            if (policy.match \u003d\u003d expected_match and\n                    policy.priority \u003d\u003d\n                    constants.LR_BGP_TO_CHASSIS_POLICY_PRIORITY):\n```","commit_id":"e62b398196b18ad574e2379ac328a2a0bd643a1e"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"861f8c42ad19c1e2005ba0bc91b3443be8af4c1c","unresolved":false,"context_lines":[{"line_number":1344,"context_line":"            main_router_name).execute(check_error\u003dTrue)"},{"line_number":1345,"context_line":"        for policy in policies:"},{"line_number":1346,"context_line":"            if (policy.match \u003d\u003d expected_match and"},{"line_number":1347,"context_line":"                    policy.priority \u003d\u003d (constants."},{"line_number":1348,"context_line":"                                        LR_BGP_TO_CHASSIS_POLICY_PRIORITY)):"},{"line_number":1349,"context_line":"                self.fail("},{"line_number":1350,"context_line":"                    \"Policy with match %r should not exist after delete\" % ("},{"line_number":1351,"context_line":"                        expected_match,))"}],"source_content_type":"text/x-python","patch_set":1,"id":"e6a9439a_2e65ace2","line":1348,"range":{"start_line":1347,"start_character":0,"end_line":1348,"end_character":76},"in_reply_to":"b4ba6146_33a4154a","updated":"2026-03-19 12:50:09.000000000","message":"Done","commit_id":"e62b398196b18ad574e2379ac328a2a0bd643a1e"}]}