)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0600db6ec94a4a8ecd9bc17fbb0a4f2a45c13de2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9ad789ab_c2c98a30","updated":"2023-05-24 14:05:38.000000000","message":"That deserves a FT test in ``TestLogicalRouterPortOps``, to avoid a regression.","commit_id":"c39a6c3c31a80379f50d8a33113d1367666aba4a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"5bfd29a61d2fd79eef27973d5b62283771ce594b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fafa2b92_112d82b7","in_reply_to":"9ad789ab_c2c98a30","updated":"2023-05-24 14:51:38.000000000","message":"Ok, we are already testing that (with and without passing peers). The new PS is fixing the broken CI behaviour.","commit_id":"c39a6c3c31a80379f50d8a33113d1367666aba4a"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"c55a0ab512173da43065d06212100181ee12a45b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9451c77d_aa8dc023","updated":"2023-05-30 13:23:40.000000000","message":"thanks!","commit_id":"267286976ea62a2372b774379b45d6d4923b6f24"}],"ovsdbapp/schema/ovn_northbound/commands.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"b8fc3c22da24221d4cd7a7e2bd364dfd74d299cb","unresolved":true,"context_lines":[{"line_number":848,"context_line":""},{"line_number":849,"context_line":"class LrpAddCommand(cmd.BaseCommand):"},{"line_number":850,"context_line":"    def __init__(self, api, router, port, mac, networks,"},{"line_number":851,"context_line":"                 peer\u003d[], may_exist\u003dFalse, **columns):"},{"line_number":852,"context_line":"        self.mac \u003d str(netaddr.EUI(mac, dialect\u003dnetaddr.mac_unix_expanded))"},{"line_number":853,"context_line":"        self.networks \u003d [str(netaddr.IPNetwork(net)) for net in networks]"},{"line_number":854,"context_line":"        self.router \u003d router"}],"source_content_type":"text/x-python","patch_set":1,"id":"e45622f2_5eb67a60","line":851,"updated":"2023-05-24 12:12:32.000000000","message":"Initializing this attribute with a mutable object is essentially creating a global variable named peer. If the code ever appends to it, it is carried over into other calls.","commit_id":"63232ddd18f9fae504ad74bb8c5d375b6cd79bca"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"8bcc520f9d09aa7944eac4baeb1c4844c950fe0d","unresolved":false,"context_lines":[{"line_number":848,"context_line":""},{"line_number":849,"context_line":"class LrpAddCommand(cmd.BaseCommand):"},{"line_number":850,"context_line":"    def __init__(self, api, router, port, mac, networks,"},{"line_number":851,"context_line":"                 peer\u003d[], may_exist\u003dFalse, **columns):"},{"line_number":852,"context_line":"        self.mac \u003d str(netaddr.EUI(mac, dialect\u003dnetaddr.mac_unix_expanded))"},{"line_number":853,"context_line":"        self.networks \u003d [str(netaddr.IPNetwork(net)) for net in networks]"},{"line_number":854,"context_line":"        self.router \u003d router"}],"source_content_type":"text/x-python","patch_set":1,"id":"06ec6f16_68bf76eb","line":851,"in_reply_to":"e45622f2_5eb67a60","updated":"2023-05-24 14:03:01.000000000","message":"nice catch!","commit_id":"63232ddd18f9fae504ad74bb8c5d375b6cd79bca"}]}