Thursday, 2020-08-27

*** acidfoo has quit IRC00:25
*** zhouhan_ has joined #openvswitch00:31
*** zhouhan has quit IRC00:34
*** acidfoo has joined #openvswitch00:36
*** acidfoo has quit IRC00:42
*** acidfoo has joined #openvswitch01:07
*** armax has quit IRC01:41
*** armax has joined #openvswitch02:03
*** Madkiss_ has quit IRC02:28
*** Madkiss has joined #openvswitch02:28
*** Madkiss has joined #openvswitch02:28
*** rcernin has quit IRC02:29
*** links has joined #openvswitch02:29
*** armax has quit IRC02:35
*** rcernin has joined #openvswitch02:52
*** rcernin has quit IRC02:56
*** rcernin has joined #openvswitch02:56
*** armax has joined #openvswitch03:03
*** acidfoo has quit IRC03:06
*** acidfoo has joined #openvswitch03:17
*** armax has quit IRC03:19
*** acidfoo has quit IRC03:21
*** psahoo has joined #openvswitch03:57
*** anilvenkata_ has quit IRC04:23
*** psahoo has quit IRC04:42
*** psahoo has joined #openvswitch04:55
*** psahoo has quit IRC05:18
*** anilvenkata_ has joined #openvswitch05:26
*** psahoo has joined #openvswitch05:35
*** eelco has joined #openvswitch05:52
*** ralonsoh has joined #openvswitch06:14
*** dholler has joined #openvswitch06:26
*** slaweq has joined #openvswitch06:32
*** psahoo has quit IRC06:43
*** psahoo has joined #openvswitch06:49
*** yamamoto has quit IRC07:12
*** yamamoto has joined #openvswitch07:13
*** __lore__ is now known as _lore_07:22
*** rcernin has quit IRC07:28
*** dholler has quit IRC07:37
*** dholler has joined #openvswitch07:50
*** yamamoto has quit IRC08:15
*** yamamoto has joined #openvswitch08:25
*** anilvenkata_ has quit IRC08:46
*** anilvenkata_ has joined #openvswitch08:47
*** anilvenkata_ has quit IRC09:19
*** mishiranu has quit IRC09:23
*** yamamoto has quit IRC09:24
*** mishiranu has joined #openvswitch09:29
*** yamamoto has joined #openvswitch09:53
*** yamamoto has quit IRC10:05
*** anilvenkata_ has joined #openvswitch10:14
*** dswebb has joined #openvswitch10:21
*** yamamoto has joined #openvswitch10:36
*** yamamoto has quit IRC10:40
*** yamamoto has joined #openvswitch10:40
*** jaicaa has quit IRC10:41
*** jaicaa has joined #openvswitch10:44
*** yamamoto has quit IRC11:01
*** yamamoto has joined #openvswitch11:12
*** yamamoto has joined #openvswitch11:35
*** thaller_ is now known as thaller12:21
*** acidfoo has joined #openvswitch13:14
*** yamamoto has quit IRC13:23
*** yangyapeng has quit IRC14:14
*** yangyapeng has joined #openvswitch14:28
*** anilvenkata_ has quit IRC14:36
*** dcbw has joined #openvswitch14:40
*** armax has joined #openvswitch14:45
*** links has quit IRC14:53
*** yangyapeng has quit IRC15:05
*** yangyapeng has joined #openvswitch15:09
*** eelco has quit IRC15:15
*** yamamoto has joined #openvswitch15:24
*** yamamoto has quit IRC15:29
*** dholler has quit IRC15:36
*** psahoo has quit IRC16:02
*** JamesBenson has joined #openvswitch16:06
*** fbl_ has joined #openvswitch16:07
*** fbl has quit IRC16:08
*** fbl_ has quit IRC16:15
*** fbl_ has joined #openvswitch16:15
*** fbl_ has quit IRC16:15
*** fbl has joined #openvswitch16:16
*** zhouhan_ has quit IRC16:30
*** zhouhan has joined #openvswitch16:31
*** dceara has joined #openvswitch16:38
*** yamamoto has joined #openvswitch16:48
*** yamamoto has quit IRC16:53
*** ralonsoh has quit IRC17:12
imaximetsHi.17:17
zhouhanHelloe17:17
dcearaHi17:17
imaximetsmmichelson and, probably, numans are not here today.17:18
_lore_hi all17:18
imaximetsSo, I'll host this meeting.17:18
imaximets#startmeeting ovn_community_development_discussion17:18
openstackMeeting started Thu Aug 27 17:18:55 2020 UTC and is due to finish in 60 minutes.  The chair is imaximets. Information about MeetBot at http://wiki.debian.org/MeetBot.17:18
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:18
openstackThe meeting name has been set to 'ovn_community_development_discussion'17:18
imaximetsI do not have much to say.  Whou wants to go first?17:19
dcearaI can start17:19
imaximetsdceara, ok.17:20
dcearathanks zhouhan for reviewing the conntrack bypass patch, I'll probably not have time for a v3 until next week though.17:20
dcearaexcept for that I wanted to schedule a run on our scale setup with zhouhan17:21
dceara's I-P patches17:21
dcearahopefully I can manage to do that tomorrow.17:21
zhouhandceara: thanks dceara17:21
dcearaAnd I sent a couple of bug fix patches today. One of which should probably be backported all the way until 20.03.17:21
dcearaThat's it on my side, thanks!17:22
imaximetsdceara, thanks!17:22
zhouhanI can go next17:23
imaximetszhouhan, sure.17:23
zhouhanI sent the series of incremental processing for flow installation: #link https://patchwork.ozlabs.org/project/openvswitch/list/?series=19700917:23
zhouhanThe CPU cost reduced around 40% for a scale of 1200 HV with 12K ports17:24
zhouhanIt also solves a bug when conjunction combination is used.17:24
zhouhan(that may need back port as well)17:25
zhouhanI also did more scale test for 3k HV with 30k ports.17:25
dcearazhouhan: regarding the conjunction bug (I didn't look at the patches yet), but would it be possible to move it earlier in the series to make backporting easier?17:26
zhouhanIt ran successfully. However, current ovn-nbctl --wait=hv mechanism is not accurate for measuring the end to end latency, because the updates of nbcfg from all HVs actually contributes the most cost.17:27
zhouhandceara: I think the earlier patches are required by the bug fix. (The bug fix is actually a big part of the series)17:28
dcearazhouhan: ack, thanks, i'll try to have a closer look too.17:28
zhouhanTo measure the latency more accurately, I think I need to improve the nb_cfg mechanism, to include a timestamp field. I will work on it.17:29
zhouhanBut overall, by manually checking the latency, it seems a port binding can finish within 4 - 5 sec at that scale.17:30
zhouhanIn addition, I did some code reviews. imaximets: could you take a look at this one as well?17:31
zhouhanhttps://patchwork.ozlabs.org/project/openvswitch/patch/20200813205259.5036-1-zhewang@nvidia.com/17:31
zhouhanThat's it from me17:31
dcearazhouhan: in our tests we wait until the port can ping its gateway (or an external host) and we see it taking >10sec in some cases. I didn't try with your patches yet though.17:32
imaximetszhouhan, yeah, I looked a this patch and I'm thinking if it's possible to fix the issues from the inside of idl/jsonrpc, without requirement for CMS to call special functions.17:33
zhouhandceara: do you ping from all the VMs? I guess that action itself may take a lot of overhead.17:33
dcearazhouhan: only from the new fake vm (netns) until it is successful17:34
zhouhanimaximets: that would be better, if it can be supported.17:34
zhouhandceara: but there will be 30k of them? And we need to make sure the slowest one can ping ...17:35
zhouhandceara: or do you just ping from a random VM and assume most of the VMs got the flow installed at similar latency?17:36
dcearazhouhan: in our tests we don't advance to create the next port until the current one can ping its own gateway.17:36
dcearazhouhan: We also don't batch port add operations. This in order to try to see the worst case scenario latency for flow installation.17:37
zhouhandceara: Oh, I think that's a different scenario. I am testing when the whole scale is built up, then create and bind a new port, and see how long it takes for this new change to get processed in all the HVs (meaning the new port can reach all other ports)17:37
dcearazhouhan: I see, ok, I can set try to set our scenario in a similar way too, thanks.17:38
dcearas/can set try/can try/17:38
imaximetszhouhan, I do not know yet, how to make re-balancing of connections work from the inside of idl, I will likely reply to ML with some ideas a bit later, if any.17:39
zhouhandceara: to make sure *all* HVs has processed the new change, I am utilizing the --wait=hv feature. Now I realized that this mechanism itself was a bottleneck (even after solving the flooding problem).17:39
dcearazhouhan: ack, we decided to go for the ping approach exactly to avoid --wait=hv17:40
zhouhandceara: So I am thinking about posting a timestamp from each ovn-controller while reporting the nb_cfg number it processed, so the nbctl can finally rely on the timestamp to calculate the time spent for the slowest HV17:41
zhouhanimaximets: ok, thanks! But do you think that could be a follow-up improvement, independent of the command provided by that patch?17:42
zhouhan(of course, with that improvement, the current patch provided won't be as useful any more)17:42
imaximetszhouhan, in general, I'd like to avoid introduction of new commands if possible, especially if we can fix the issue in general.17:43
imaximetszhouhan, how these commands supposed to be used?  Will CMS just re-disribute all the clients by itself, or will nominate only part of them for re-connection?17:44
zhouhanimaximets: agree in general. But I feel this command does provide some value for operational need.17:45
*** Ankur1 has joined #openvswitch17:45
Ankur1Hi17:45
zhouhanimaximets: I think the typical case is when failover happend and the node recovered, the newly recovered node has no connections. So operator can use the command to instruct some of the HVs to connect back to the recovered node.17:46
*** Ankur171 has joined #openvswitch17:46
Ankur1hi17:47
Ankur171ab17:47
*** Ankur1 has left #openvswitch17:47
*** Ankur171 has quit IRC17:47
zhouhanimaximets: but it may also be useful if someone wants to adjust (fine tune) the load of different servers but moving clients from one server to another.17:47
imaximetszhouhan, I see. Let me think a little bit.  I will reply on ML, or just apply the patch if there will be no clever ideas from my side. :)17:48
zhouhans/but moving/by moving17:48
zhouhanthanks imaximets17:48
dcearazhouhan: I think there were some similar discussions on the ML at some point about adding a timestamp to the sync mechanism. We can probably continue the chassis.nb_cfg discussion there.17:49
zhouhandceara: sure17:49
zhouhandceara: maybe I will try a POC first17:50
dcearazhouhan: cool17:50
imaximetsOK.  Anyone else wants to share some updates?17:53
imaximetsSo, I think, we could call it now.17:55
imaximets#endmeeting17:56
openstackMeeting ended Thu Aug 27 17:56:31 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:56
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-08-27-17.18.html17:56
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-08-27-17.18.txt17:56
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-08-27-17.18.log.html17:56
zhouhanbye17:56
imaximetsBye.17:57
_lore_bye17:58
dcearaBye17:59
*** acidfoo has quit IRC18:03
*** dceara has quit IRC18:20
*** thaller has quit IRC18:50
*** thaller has joined #openvswitch18:50
*** fbl has quit IRC19:19
*** fbl has joined #openvswitch19:23
*** acidfoo has joined #openvswitch20:11
*** fbl has quit IRC20:56
*** slaweq has quit IRC21:22
*** yamamoto has joined #openvswitch21:30
*** JamesBenson has quit IRC21:42
*** imaximets has quit IRC22:01
*** imaximets has joined #openvswitch22:01
*** yamamoto has quit IRC22:12
*** rcernin has joined #openvswitch22:45
*** armax has quit IRC22:52
*** yangyapeng has quit IRC23:00
*** yangyapeng has joined #openvswitch23:04
*** armax has joined #openvswitch23:08

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!