On 4/17/20 5:45 AM, Han Zhou wrote: > Hi Ben > > We talked in a previous week's OVN meeting. I think you reviewed this patch > already and was about to merge it. Would you like to merge it when you get > time? Please also backport to 2.12 if no issues. > > Thanks, > Han
Hi, Ben. Just wanted to check what is the status of this patch? Last time you said that you will take care of it, but it was long time ago. These probes are causing issues during scale testing so I'm curious if you have some time to revisit/apply this patch? The patch itself looks fine to me and I could apply it myself in case you have no time for that. Best regards, Ilya Maximets. > On Mon, Mar 30, 2020 at 5:21 PM Zhen Wang <[email protected]> wrote: > >> With the scale test of 640 nodes k8s cluster, raft DB nodes' jsonrpc >> session got closed due to the timeout of default 5 seconds probe. >> It will cause disturbance of the raft cluster. Since we already have >> the heartbeat for RAFT, just disable the probe between the servers >> to avoid the unnecessary jsonrpc inactivity probe. >> >> Acked-by: Han Zhou <[email protected]> >> Signed-off-by: Zhen Wang <[email protected]> >> --- >> ovsdb/raft.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/ovsdb/raft.c b/ovsdb/raft.c >> index 6391eeb13..c974e91b6 100644 >> --- a/ovsdb/raft.c >> +++ b/ovsdb/raft.c >> @@ -938,6 +938,7 @@ raft_add_conn(struct raft *raft, struct >> jsonrpc_session *js, >> &conn->sid); >> conn->incoming = incoming; >> conn->js_seqno = jsonrpc_session_get_seqno(conn->js); >> + jsonrpc_session_set_probe_interval(js, 0); >> } >> >> /* Starts the local server in an existing Raft cluster, using the local >> copy of >> -- >> 2.20.1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
