)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7e58113b7302800d49f04d3877eb8c7a21caf3f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"65048c9e_76f546ae","updated":"2026-03-20 16:17:08.000000000","message":"Addressing comments now","commit_id":"7a695ca3abf325cc1c4842bc1c0d6d7402dedcba"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"256284a4d09990c528066ff6ad2eb612b5711ff9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b0002daa_8d6dd2dd","updated":"2026-03-23 11:18:25.000000000","message":"nice","commit_id":"ea65a4d83e162231dbd283c8ca52d9c1bc6d2f95"}],"neutron/scheduler/l3_ovn_scheduler.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5abccaeccc262f6b3d1efd54d7f37f1ad62fe38f","unresolved":true,"context_lines":[{"line_number":213,"context_line":"                break"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"            leastload \u003d min(chassis_load.values())"},{"line_number":216,"context_line":"            least_loaded \u003d [c for c, l in chassis_load.items()"},{"line_number":217,"context_line":"                           if l \u003d\u003d leastload]"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"            if len(least_loaded) \u003e 1:"}],"source_content_type":"text/x-python","patch_set":6,"id":"4bac941b_66fba63b","line":216,"range":{"start_line":216,"start_character":37,"end_line":216,"end_character":38},"updated":"2026-03-20 16:03:43.000000000","message":"nit: \u0027l\u0027 is an ambiguous variable name with E741, was trying to clean-up in https://review.opendev.org/c/openstack/neutron/+/981421/ - can you just change this to another letter or \u0027load\u0027? Or I can just fix it in that patch.","commit_id":"7a695ca3abf325cc1c4842bc1c0d6d7402dedcba"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"d2d56de5e657504dca26e5fb7b875b5462264049","unresolved":false,"context_lines":[{"line_number":213,"context_line":"                break"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"            leastload \u003d min(chassis_load.values())"},{"line_number":216,"context_line":"            least_loaded \u003d [c for c, l in chassis_load.items()"},{"line_number":217,"context_line":"                           if l \u003d\u003d leastload]"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"            if len(least_loaded) \u003e 1:"}],"source_content_type":"text/x-python","patch_set":6,"id":"6f0dfe35_8e8a5331","line":216,"range":{"start_line":216,"start_character":37,"end_line":216,"end_character":38},"in_reply_to":"4bac941b_66fba63b","updated":"2026-03-20 21:17:17.000000000","message":"Yeah, Claude is sometime a bit cryptic with the variable names.","commit_id":"7a695ca3abf325cc1c4842bc1c0d6d7402dedcba"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5abccaeccc262f6b3d1efd54d7f37f1ad62fe38f","unresolved":true,"context_lines":[{"line_number":233,"context_line":"                            1 for lrp_name, prio in"},{"line_number":234,"context_line":"                            all_chassis_bindings.get(chassis, [])"},{"line_number":235,"context_line":"                            if prio \u003d\u003d lp)"},{"line_number":236,"context_line":"                        loads.append(-load_at_lp)"},{"line_number":237,"context_line":"                    loads.append(_h ^ hash(chassis))"},{"line_number":238,"context_line":"                    return tuple(loads)"},{"line_number":239,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"588e3d42_b1fa946e","line":236,"updated":"2026-03-20 16:03:43.000000000","message":"okay, so the \u0027-\u0027 is confusing me, are you trying to sort by higher negative number first? But is -200 lower than -100 for load? Maybe just some comments would help.","commit_id":"7a695ca3abf325cc1c4842bc1c0d6d7402dedcba"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"d2d56de5e657504dca26e5fb7b875b5462264049","unresolved":false,"context_lines":[{"line_number":233,"context_line":"                            1 for lrp_name, prio in"},{"line_number":234,"context_line":"                            all_chassis_bindings.get(chassis, [])"},{"line_number":235,"context_line":"                            if prio \u003d\u003d lp)"},{"line_number":236,"context_line":"                        loads.append(-load_at_lp)"},{"line_number":237,"context_line":"                    loads.append(_h ^ hash(chassis))"},{"line_number":238,"context_line":"                    return tuple(loads)"},{"line_number":239,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"2dfd60e2_2b811e60","line":236,"in_reply_to":"588e3d42_b1fa946e","updated":"2026-03-20 21:17:17.000000000","message":"Done","commit_id":"7a695ca3abf325cc1c4842bc1c0d6d7402dedcba"}],"neutron/tests/functional/services/ovn_l3/test_plugin.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5abccaeccc262f6b3d1efd54d7f37f1ad62fe38f","unresolved":true,"context_lines":[{"line_number":320,"context_line":""},{"line_number":321,"context_line":"        This test creates ``num_chassis`` GW chassis and a router with"},{"line_number":322,"context_line":"        ``num_networks`` GW networks. The gateway_chassis registers"},{"line_number":323,"context_line":"        (``num_chassis`` * ``num_networks``) should be balanced."},{"line_number":324,"context_line":"        across all GW chassis."},{"line_number":325,"context_line":"        NOTE: to make a balanced distribution, the relation"},{"line_number":326,"context_line":"        ``num_networks`` / ``num_chassis`` must be an integer."}],"source_content_type":"text/x-python","patch_set":6,"id":"7dda0a72_a1348114","line":323,"range":{"start_line":323,"start_character":63,"end_line":323,"end_character":64},"updated":"2026-03-20 16:03:43.000000000","message":"super-nit: remove period","commit_id":"7a695ca3abf325cc1c4842bc1c0d6d7402dedcba"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"d2d56de5e657504dca26e5fb7b875b5462264049","unresolved":false,"context_lines":[{"line_number":320,"context_line":""},{"line_number":321,"context_line":"        This test creates ``num_chassis`` GW chassis and a router with"},{"line_number":322,"context_line":"        ``num_networks`` GW networks. The gateway_chassis registers"},{"line_number":323,"context_line":"        (``num_chassis`` * ``num_networks``) should be balanced."},{"line_number":324,"context_line":"        across all GW chassis."},{"line_number":325,"context_line":"        NOTE: to make a balanced distribution, the relation"},{"line_number":326,"context_line":"        ``num_networks`` / ``num_chassis`` must be an integer."}],"source_content_type":"text/x-python","patch_set":6,"id":"f76e2c9d_32ab7408","line":323,"range":{"start_line":323,"start_character":63,"end_line":323,"end_character":64},"in_reply_to":"7dda0a72_a1348114","updated":"2026-03-20 21:17:17.000000000","message":"Done","commit_id":"7a695ca3abf325cc1c4842bc1c0d6d7402dedcba"}]}