)]}'
{"neutron/plugins/ml2/drivers/openvswitch/agent/openflow/native/ofswitch.py":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"36896c345e10749866968031e3b7128878e79336","unresolved":false,"context_lines":[{"line_number":66,"context_line":"            eventlet.sleep(1)"},{"line_number":67,"context_line":"        return dp"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    @tenacity.retry_if_exception_type(ryu.app.ofctl.exception.InvalidDatapath)"},{"line_number":70,"context_line":"    def _send_msg_retry(self, msg, reply_cls, reply_multi):"},{"line_number":71,"context_line":"        return ofctl_api.send_msg(self._app, msg, reply_cls, reply_multi)"},{"line_number":72,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a22e58a_482f7fa1","line":69,"updated":"2017-03-14 06:02:05.000000000","message":"you mean @...retry(retry\u003dretry_if_exception_type(... ?\n\ni guess it\u0027s better to have some wait.","commit_id":"90454a12a7a67550181ba3805c9b39d8d1f14829"},{"author":{"_account_id":9200,"name":"IWAMOTO Toshihiro","email":"iwamoto@valinux.co.jp","username":"toshii"},"change_message_id":"5fd5d0cde5c39866c03045e62783f10e66aaa857","unresolved":false,"context_lines":[{"line_number":66,"context_line":"            eventlet.sleep(1)"},{"line_number":67,"context_line":"        return dp"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    @tenacity.retry_if_exception_type(ryu.app.ofctl.exception.InvalidDatapath)"},{"line_number":70,"context_line":"    def _send_msg_retry(self, msg, reply_cls, reply_multi):"},{"line_number":71,"context_line":"        return ofctl_api.send_msg(self._app, msg, reply_cls, reply_multi)"},{"line_number":72,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a1ff146_2299656f","line":69,"in_reply_to":"5a22e58a_482f7fa1","updated":"2017-03-15 05:42:41.000000000","message":"Done","commit_id":"90454a12a7a67550181ba3805c9b39d8d1f14829"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"0e41f399ec0a11494f2228df01da54ecc2483989","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        return dp"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    @tenacity.retry(wait\u003d1, retry\u003dtenacity.retry_if_exception_type("},{"line_number":70,"context_line":"        ofctl_exc.InvalidDatapath))"},{"line_number":71,"context_line":"    def _send_msg_retry(self, msg, reply_cls, reply_multi):"},{"line_number":72,"context_line":"        return ofctl_api.send_msg(self._app, msg, reply_cls, reply_multi)"},{"line_number":73,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_e059ad5b","line":70,"updated":"2017-03-21 02:55:01.000000000","message":"does this mean retrying forever?\n\ndon\u0027t you need to refresh datapath-id? (restart from _get_dp)","commit_id":"727fe807139563d13a20ce72a4f611221fc6b139"},{"author":{"_account_id":9200,"name":"IWAMOTO Toshihiro","email":"iwamoto@valinux.co.jp","username":"toshii"},"change_message_id":"c7983d72e7aadf124a757c30b888979cce1cd99e","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        return dp"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    @tenacity.retry(wait\u003d1, retry\u003dtenacity.retry_if_exception_type("},{"line_number":70,"context_line":"        ofctl_exc.InvalidDatapath))"},{"line_number":71,"context_line":"    def _send_msg_retry(self, msg, reply_cls, reply_multi):"},{"line_number":72,"context_line":"        return ofctl_api.send_msg(self._app, msg, reply_cls, reply_multi)"},{"line_number":73,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_92880fc7","line":70,"in_reply_to":"1a1ced50_b2c6eb65","updated":"2017-03-22 08:38:41.000000000","message":":O\n\nagreed, but there are a lot of install_xxx methods.","commit_id":"727fe807139563d13a20ce72a4f611221fc6b139"},{"author":{"_account_id":9200,"name":"IWAMOTO Toshihiro","email":"iwamoto@valinux.co.jp","username":"toshii"},"change_message_id":"f6448349f4014c1581ab2354a925678d1574dbe8","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        return dp"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    @tenacity.retry(wait\u003d1, retry\u003dtenacity.retry_if_exception_type("},{"line_number":70,"context_line":"        ofctl_exc.InvalidDatapath))"},{"line_number":71,"context_line":"    def _send_msg_retry(self, msg, reply_cls, reply_multi):"},{"line_number":72,"context_line":"        return ofctl_api.send_msg(self._app, msg, reply_cls, reply_multi)"},{"line_number":73,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_f2006303","line":70,"in_reply_to":"1a1ced50_e059ad5b","updated":"2017-03-22 08:15:16.000000000","message":"Timeout in L.76 will take care of that.\n\nThe log suggests datapath-id aren\u0027t changed and I thought everything is fine after ovs reconnect, but stale datapath object in msg might be an issue.\n\nDo you have an idea?","commit_id":"727fe807139563d13a20ce72a4f611221fc6b139"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"7e9504bc1e283025c21076fb8f848b328d9fd9ee","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        return dp"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    @tenacity.retry(wait\u003d1, retry\u003dtenacity.retry_if_exception_type("},{"line_number":70,"context_line":"        ofctl_exc.InvalidDatapath))"},{"line_number":71,"context_line":"    def _send_msg_retry(self, msg, reply_cls, reply_multi):"},{"line_number":72,"context_line":"        return ofctl_api.send_msg(self._app, msg, reply_cls, reply_multi)"},{"line_number":73,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_b2c6eb65","line":70,"in_reply_to":"1a1ced50_f2006303","updated":"2017-03-22 08:28:04.000000000","message":"i guess it\u0027s safer to restart from the beginning of the outer method. ie. raise an exception here and sprinkle decorator to catch it on methods calling _get_dp.","commit_id":"727fe807139563d13a20ce72a4f611221fc6b139"}]}