)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"1bae6d0c2c7f5c706fb792349f1412dc4141b8b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"56d29efe_33cee17b","updated":"2023-06-02 13:45:48.000000000","message":"Looks great!","commit_id":"cbe1e16d42be2f38d0398880b0d28355117c83ef"}],"ovn_bgp_agent/drivers/openstack/watchers/base_watcher.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9c1ed178aece8bc5984c91e765144407a3aae309","unresolved":true,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class PortBindingChassisEvent(row_event.RowEvent):"},{"line_number":23,"context_line":"    def __init__(self, bgp_agent, events):"},{"line_number":24,"context_line":"        self.agent \u003d bgp_agent"}],"source_content_type":"text/x-python","patch_set":1,"id":"33292483_255dc198","line":21,"updated":"2023-06-02 13:20:27.000000000","message":"We could even have a simple custom RowEvent here\n\n```\nclass Event(row_event.RowEvent):\n    def run(self, *args, **kwargs):\n        try:\n            self._run(*args, **kwargs)\n        except Exception:\n            LOG.exception(\"Unexpected exception while running the event action\") \n```\n\nThan just inherit it below instead of creating multiple run()\u0027s","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9920eee78ebf33c0403ababe05eef60086e203f4","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class PortBindingChassisEvent(row_event.RowEvent):"},{"line_number":23,"context_line":"    def __init__(self, bgp_agent, events):"},{"line_number":24,"context_line":"        self.agent \u003d bgp_agent"}],"source_content_type":"text/x-python","patch_set":1,"id":"52ffa445_c9645fac","line":21,"in_reply_to":"33292483_255dc198","updated":"2023-06-02 13:26:49.000000000","message":"Done","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9c1ed178aece8bc5984c91e765144407a3aae309","unresolved":true,"context_lines":[{"line_number":19,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class PortBindingChassisEvent(row_event.RowEvent):"},{"line_number":23,"context_line":"    def __init__(self, bgp_agent, events):"},{"line_number":24,"context_line":"        self.agent \u003d bgp_agent"},{"line_number":25,"context_line":"        table \u003d \u0027Port_Binding\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"39419a46_d9ccb458","line":22,"range":{"start_line":22,"start_character":30,"end_line":22,"end_character":48},"updated":"2023-06-02 13:20:27.000000000","message":"Inherit here from our custom Event class","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9920eee78ebf33c0403ababe05eef60086e203f4","unresolved":false,"context_lines":[{"line_number":19,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class PortBindingChassisEvent(row_event.RowEvent):"},{"line_number":23,"context_line":"    def __init__(self, bgp_agent, events):"},{"line_number":24,"context_line":"        self.agent \u003d bgp_agent"},{"line_number":25,"context_line":"        table \u003d \u0027Port_Binding\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"08a53a5d_6a2c3414","line":22,"range":{"start_line":22,"start_character":30,"end_line":22,"end_character":48},"in_reply_to":"39419a46_d9ccb458","updated":"2023-06-02 13:26:49.000000000","message":"Done","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9c1ed178aece8bc5984c91e765144407a3aae309","unresolved":true,"context_lines":[{"line_number":34,"context_line":"        try:"},{"line_number":35,"context_line":"            self._run(*args, **kwargs)"},{"line_number":36,"context_line":"        except Exception:"},{"line_number":37,"context_line":"            LOG.error(\"Unexpected exception while running the event action\")"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"class OVNLBMemberEvent(row_event.RowEvent):"}],"source_content_type":"text/x-python","patch_set":1,"id":"a4f6f52a_6b9aa7cc","line":37,"updated":"2023-06-02 13:20:27.000000000","message":"exception instead of error ? Or we will lose the real error if not logged","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9920eee78ebf33c0403ababe05eef60086e203f4","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        try:"},{"line_number":35,"context_line":"            self._run(*args, **kwargs)"},{"line_number":36,"context_line":"        except Exception:"},{"line_number":37,"context_line":"            LOG.error(\"Unexpected exception while running the event action\")"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"class OVNLBMemberEvent(row_event.RowEvent):"}],"source_content_type":"text/x-python","patch_set":1,"id":"34aaaad5_cb808bd9","line":37,"in_reply_to":"a4f6f52a_6b9aa7cc","updated":"2023-06-02 13:26:49.000000000","message":"Done","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9c1ed178aece8bc5984c91e765144407a3aae309","unresolved":true,"context_lines":[{"line_number":37,"context_line":"            LOG.error(\"Unexpected exception while running the event action\")"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"class OVNLBMemberEvent(row_event.RowEvent):"},{"line_number":41,"context_line":"    def __init__(self, bgp_agent, events):"},{"line_number":42,"context_line":"        self.agent \u003d bgp_agent"},{"line_number":43,"context_line":"        table \u003d \u0027Load_Balancer\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"b44d16e3_dfa84bc4","line":40,"range":{"start_line":40,"start_character":23,"end_line":40,"end_character":41},"updated":"2023-06-02 13:20:27.000000000","message":"ditto","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9920eee78ebf33c0403ababe05eef60086e203f4","unresolved":false,"context_lines":[{"line_number":37,"context_line":"            LOG.error(\"Unexpected exception while running the event action\")"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"class OVNLBMemberEvent(row_event.RowEvent):"},{"line_number":41,"context_line":"    def __init__(self, bgp_agent, events):"},{"line_number":42,"context_line":"        self.agent \u003d bgp_agent"},{"line_number":43,"context_line":"        table \u003d \u0027Load_Balancer\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"45bb2b28_6c3c0116","line":40,"range":{"start_line":40,"start_character":23,"end_line":40,"end_character":41},"in_reply_to":"b44d16e3_dfa84bc4","updated":"2023-06-02 13:26:49.000000000","message":"Done","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"}],"ovn_bgp_agent/drivers/openstack/watchers/bgp_watcher.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9c1ed178aece8bc5984c91e765144407a3aae309","unresolved":true,"context_lines":[{"line_number":464,"context_line":"            self.agent.sync()"},{"line_number":465,"context_line":""},{"line_number":466,"context_line":""},{"line_number":467,"context_line":"class ChassisCreateEventBase(row_event.RowEvent):"},{"line_number":468,"context_line":"    table \u003d None"},{"line_number":469,"context_line":""},{"line_number":470,"context_line":"    def __init__(self, bgp_agent):"}],"source_content_type":"text/x-python","patch_set":1,"id":"4ae3036e_0d2d94b5","line":467,"range":{"start_line":467,"start_character":29,"end_line":467,"end_character":47},"updated":"2023-06-02 13:20:27.000000000","message":"Just inherit it from the custom Event() from previous comment","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9920eee78ebf33c0403ababe05eef60086e203f4","unresolved":false,"context_lines":[{"line_number":464,"context_line":"            self.agent.sync()"},{"line_number":465,"context_line":""},{"line_number":466,"context_line":""},{"line_number":467,"context_line":"class ChassisCreateEventBase(row_event.RowEvent):"},{"line_number":468,"context_line":"    table \u003d None"},{"line_number":469,"context_line":""},{"line_number":470,"context_line":"    def __init__(self, bgp_agent):"}],"source_content_type":"text/x-python","patch_set":1,"id":"4820b555_e2cb9579","line":467,"range":{"start_line":467,"start_character":29,"end_line":467,"end_character":47},"in_reply_to":"4ae3036e_0d2d94b5","updated":"2023-06-02 13:26:49.000000000","message":"Done","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"}],"ovn_bgp_agent/drivers/openstack/watchers/evpn_watcher.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9c1ed178aece8bc5984c91e765144407a3aae309","unresolved":true,"context_lines":[{"line_number":215,"context_line":"            self.agent.sync()"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"class ChassisCreateEventBase(row_event.RowEvent):"},{"line_number":219,"context_line":"    table \u003d None"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def __init__(self, bgp_agent):"}],"source_content_type":"text/x-python","patch_set":1,"id":"b5ae0411_90f51e05","line":218,"range":{"start_line":218,"start_character":29,"end_line":218,"end_character":47},"updated":"2023-06-02 13:20:27.000000000","message":"ditto","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"9920eee78ebf33c0403ababe05eef60086e203f4","unresolved":false,"context_lines":[{"line_number":215,"context_line":"            self.agent.sync()"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"class ChassisCreateEventBase(row_event.RowEvent):"},{"line_number":219,"context_line":"    table \u003d None"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def __init__(self, bgp_agent):"}],"source_content_type":"text/x-python","patch_set":1,"id":"b9996988_3b7f59bf","line":218,"range":{"start_line":218,"start_character":29,"end_line":218,"end_character":47},"in_reply_to":"b5ae0411_90f51e05","updated":"2023-06-02 13:26:49.000000000","message":"Done","commit_id":"b0085222506226f3e9924a2146365adbfd83c087"}]}