)]}'
{"neutron/agent/ovn/metadata/ovsdb.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"82df4b123c65773a4c08f9a9f30e6913e7ab5b8e","unresolved":true,"context_lines":[{"line_number":58,"context_line":"    @tenacity.retry("},{"line_number":59,"context_line":"        wait\u003dtenacity.wait_exponential("},{"line_number":60,"context_line":"            max\u003dconfig.get_ovn_ovsdb_retry_max_interval()),"},{"line_number":61,"context_line":"        reraise\u003dTrue)"},{"line_number":62,"context_line":"    def start(self):"},{"line_number":63,"context_line":"        LOG.info(\u0027Getting OvsdbSbOvnIdl for MetadataAgent with retry\u0027)"},{"line_number":64,"context_line":"        conn \u003d connection.Connection("}],"source_content_type":"text/x-python","patch_set":2,"id":"bc7a5a45_f4236c1a","line":61,"updated":"2021-07-02 08:58:43.000000000","message":"FWIW, we use a similar retry mechanism in the mech driver [0] (see the from_worker() method)\n\n[0] https://github.com/openstack/neutron/blob/b08310df9f9061cfdcbc1d4428739db6de3e0e7f/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/impl_idl_ovn.py#L222-L230","commit_id":"95d80a2757e23b6e8cdb44576f3913eedb400810"}],"neutron/agent/ovn/metadata/server.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"00a11f231cc22aa12272498e062783df3d57056d","unresolved":true,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @property"},{"line_number":54,"context_line":"    def sb_idl(self):"},{"line_number":55,"context_line":"        self._post_fork_event.wait()"},{"line_number":56,"context_line":"        return self._sb_idl"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    @sb_idl.setter"},{"line_number":59,"context_line":"    def sb_idl(self, val):"}],"source_content_type":"text/x-python","patch_set":1,"id":"2bdf59bc_19554d00","line":56,"range":{"start_line":55,"start_character":8,"end_line":56,"end_character":27},"updated":"2021-06-16 15:08:17.000000000","message":"Instead of checking \"self._post_fork_event\" always, it could be faster to check only for the \"self._sb_idl\" value\n\n  def __init__(self, conf, chassis):\n    self._sb_idl \u003d None\n\n    @property\n    def sb_idl(self):\n        if not self._sb_idl\n          self._post_fork_event.wait()\n       \n        return self._sb_idl","commit_id":"b608415b36b59e724e93f1a1dcc68a6c6cf7a253"},{"author":{"_account_id":10366,"name":"Hemanth N","email":"hemanth.nakkina@canonical.com","username":"Hemanth"},"change_message_id":"8e5b639fdcee1dc8ce314c4aad6bf31d2a260c82","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @property"},{"line_number":54,"context_line":"    def sb_idl(self):"},{"line_number":55,"context_line":"        self._post_fork_event.wait()"},{"line_number":56,"context_line":"        return self._sb_idl"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    @sb_idl.setter"},{"line_number":59,"context_line":"    def sb_idl(self, val):"}],"source_content_type":"text/x-python","patch_set":1,"id":"010f5270_b7fa03f0","line":56,"range":{"start_line":55,"start_character":8,"end_line":56,"end_character":27},"in_reply_to":"2bdf59bc_19554d00","updated":"2021-06-17 03:14:40.000000000","message":"Done","commit_id":"b608415b36b59e724e93f1a1dcc68a6c6cf7a253"}]}