Thursday, 2020-10-08

*** rcernin has quit IRC00:48
*** rcernin has joined #openvswitch00:56
*** zhouhan_ has quit IRC01:02
*** rcernin has quit IRC01:08
*** JamesBenson has joined #openvswitch01:20
*** JamesBenson has quit IRC01:25
*** rcernin has joined #openvswitch01:26
*** rcernin has quit IRC01:39
*** zhouhan has joined #openvswitch01:52
*** JamesBenson has joined #openvswitch02:01
*** JamesBenson has quit IRC02:06
*** rcernin has joined #openvswitch02:13
*** JamesBenson has joined #openvswitch02:14
*** JamesBenson has quit IRC02:18
*** dholler has quit IRC02:39
*** JamesBenson has joined #openvswitch02:39
*** JamesBenson has quit IRC02:44
*** dholler has joined #openvswitch02:52
*** JamesBenson has joined #openvswitch03:05
*** JamesBen_ has joined #openvswitch03:09
*** JamesBenson has quit IRC03:09
*** armax has quit IRC03:26
*** JamesBen_ has quit IRC04:02
*** JamesBenson has joined #openvswitch04:07
*** JamesBenson has quit IRC04:10
*** psahoo has joined #openvswitch04:16
*** JamesBenson has joined #openvswitch04:16
*** JamesBenson has quit IRC04:27
*** JamesBenson has joined #openvswitch04:50
*** JamesBenson has quit IRC04:55
*** links has joined #openvswitch05:05
*** psahoo has quit IRC05:07
*** psahoo has joined #openvswitch05:07
*** anilvenkata has joined #openvswitch05:55
*** eelco has joined #openvswitch05:56
*** ralonsoh has joined #openvswitch06:37
*** slaweq has joined #openvswitch07:01
*** anilvenkata_ has joined #openvswitch07:11
*** anilvenkata has quit IRC07:13
*** thaller has joined #openvswitch07:21
*** fab23 has quit IRC07:33
*** tamas_erdei has quit IRC07:33
*** balkamos has quit IRC07:33
*** terdei has joined #openvswitch07:33
*** fab23 has joined #openvswitch07:33
*** balkamos has joined #openvswitch07:38
*** jraju__ has joined #openvswitch07:57
*** rcernin has quit IRC07:57
*** links has quit IRC07:57
*** ktraynor_ has quit IRC08:07
*** ktraynor has joined #openvswitch08:08
*** behzad1992 has joined #openvswitch08:25
*** behzad1992 has left #openvswitch08:26
*** behzad1992 has joined #openvswitch08:27
*** behzad19921 has joined #openvswitch09:05
*** behzad19921 has left #openvswitch09:05
*** behzad1992 has quit IRC09:08
*** anilvenkata_ has quit IRC09:12
*** anilvenkata_ has joined #openvswitch09:17
*** d1rewolf has quit IRC09:26
*** d1rewolf has joined #openvswitch09:28
*** blahdodo has quit IRC09:32
*** oanson has joined #openvswitch09:45
*** dalvarez has quit IRC09:45
*** dalvarez has joined #openvswitch09:49
*** armax has joined #openvswitch09:54
*** rcernin has joined #openvswitch10:29
*** rcernin has quit IRC10:31
*** tbachman has quit IRC10:57
*** JamesBenson has joined #openvswitch11:13
*** thaller_ has joined #openvswitch11:15
*** thaller has quit IRC11:18
*** rebrec has joined #openvswitch11:56
*** tamas_erdei has joined #openvswitch12:01
*** terdei has quit IRC12:04
*** rebrec has quit IRC12:16
*** rebrec has joined #openvswitch12:17
*** tbachman has joined #openvswitch12:51
*** bostondriver has joined #openvswitch12:51
*** dcbw has joined #openvswitch13:32
*** ihrachys has joined #openvswitch13:53
*** larsks has quit IRC13:58
*** larsks has joined #openvswitch14:18
*** larsks is now known as Guest5045114:18
*** Guest50451 has quit IRC14:32
*** fbl has quit IRC14:32
*** fbl has joined #openvswitch14:32
*** slaweq has quit IRC14:38
*** slaweq has joined #openvswitch14:42
*** psahoo has quit IRC14:59
*** thaller_ is now known as thaller15:02
*** larsks has joined #openvswitch15:15
*** larsks is now known as Guest8441215:15
*** Guest84412 has quit IRC15:15
*** Guest84412 has joined #openvswitch15:15
*** Guest84412 is now known as larsks15:15
*** jraju__ has quit IRC15:16
*** eelco has quit IRC15:20
*** links has joined #openvswitch15:24
*** links has quit IRC15:29
*** larsks has quit IRC15:30
*** larsks has joined #openvswitch15:31
*** zhouhan_ has joined #openvswitch16:22
*** blahdodo has joined #openvswitch16:24
*** zhouhan has quit IRC16:25
*** kklimonda has quit IRC16:30
*** kklimonda_ has joined #openvswitch16:30
*** dceara has joined #openvswitch17:08
mmichelsonHi everyone17:15
mmichelsonI figure I'll get this thing started17:16
mmichelsonHopefully this time there won't be a catastrophic netsplit that prevents me from ending the meeting17:16
mmichelson#startmeeting ovn_community_development_discussion17:17
openstackMeeting started Thu Oct  8 17:17:08 2020 UTC and is due to finish in 60 minutes.  The chair is mmichelson. Information about MeetBot at http://wiki.debian.org/MeetBot.17:17
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:17
openstackThe meeting name has been set to 'ovn_community_development_discussion'17:17
numansHello17:17
mmichelsonHi numans17:17
mmichelsonI figure I can go first.17:18
dcearaHi17:18
mmichelsonI've been working on adding some unit test infrastructure to OVN. This way it's easier to test smaller units (e.g. functions). I've added a couple of small example unit tests and I plan to submit an RFC patch series before the end of the week.17:19
mmichelsonOther than that, I've been doing reviews. And that's all from me.17:19
numansI guess I can go next.17:21
_lore_hi all17:22
numansI worked on the reject action and submitted the patches for review17:22
numansthis reduces a lot of logical flows now.17:22
numansI have started working on reducing the load balancer hairpin lflows.17:23
numansI'm hoping I'll submit the patches for review early next week.17:23
numansI couldn't do much review this week.17:23
numansThat's it from me.17:23
*** zhouhan_ has quit IRC17:23
dcearanumans: Thanks for taking care of the hairpin lflows, I wanted to have a go at it for a while but didn't get a chance to.17:24
*** zhouhan has joined #openvswitch17:24
dcearanumans: Looking forward to the patches :)17:24
numansdceara, welcome17:24
numansI was able to reduce 50% of the patches.17:24
numansIf ovs could support something like - ct(commit, nat(dst=reg0)) that would have been great.17:25
numansbut looks like it doesn't support.17:25
numansbecause of which we still need to add an lflow to do ct_snat for each vip.17:25
zhouhannumans: 50% of the patches? or you meant flows :)17:25
numanszhouhan, :)17:25
numanss/patches/lflows.17:25
numansmy bad :)17:25
zhouhanalthough 50% patches would be awesome, too17:25
numanszhouhan, I was talking about the hairpin load balancer lflows.17:26
mmichelsonnumans, why doesn't that ct() action work? Does nat not accept a register as an rvalue?17:26
numanszhouhan, FYI - I've submitted the reject action patch series.17:26
numansmmichelson, I didn't support that way.17:26
zhouhannumans: thanks! It is in my review list17:27
numansMay be we can enhance ovs for that. I'm planing to give a shot at that, if its possible.17:27
numansnot sure if imaximets is here.. do you think ovs supports something like - ct(nat(dst=reg0))17:27
numanssince it supports the zone value from a register.17:27
imaximetsnumans, I'm here, and I'm not sure. :)17:28
numansimaximets, I tried it isn't supported.17:28
numansimaximets, it does support ct(zone=NXM_NX_REG0,...)17:28
numansso I wonder ovs could support something similar for nat too17:29
numansOk. that's it from me. If someone wants to go next.17:29
zhouhanI can go quickly17:29
zhouhanI did some reviews. Why reviewing dceara's patch for "persistent conflict flows", I figured out 2 problems of my code and I submitted the fixes.17:31
zhouhanI am waiting for the review and also waiting for next version from dceara.17:31
zhouhanThat's it from me.17:31
dcearazhouhan: I just reviewed the 2 patches today.  I acked the first one but for the second one I'm a bit confused.17:32
dcearazhouhan: It seems to me we're not changing behavior at all.  I replied on the ML though.17:32
dcearazhouhan: Re the "predictable conflict resolution", I'll try to send a patch soon.17:32
zhouhandceara: ok, thanks. I haven't checked ML yet.17:32
dcearas/a patch/a new revision17:33
imaximetsnumans, according to docs, nat src and dst only supports addresses and ports as a value.17:33
numansimaximets, ack.17:33
imaximetszhouhan, I looked through old patches once again and found this: https://patchwork.ozlabs.org/project/openvswitch/patch/1564100775-80657-1-git-send-email-hzhou8@ebay.com/17:34
zhouhanimaximets: oh, this was never applied?17:36
imaximetszhouhan, nope17:37
zhouhanimaximets: oh, that's why I kept wondering how would the bug reported by Mark happened again. I remembered that I fixed it long ago ...17:38
zhouhanMark = Mark Gray17:38
mmichelsonzhouhan, btw, I got word from Mark that he's planning to do a v2 next week. He couldn't make this meeting17:38
imaximetszhouhan, I think you need to discuss with him, which approach is better.17:39
zhouhanmmichelson: imaximets Ok, thanks for this information. I will check it again with him17:40
imaximetsbtw, I do not have anything else for this meeting today.17:42
dcearaI can go next if that's OK.17:42
zhouhandceara: Mark Gray talked about "There is a failure on one ovn UT with this change. It gets resolved by the fix that Dumitru suggests." Do you know what's that?17:43
dcearazhouhan: It's related to chassis/chassis_private.  The simplest way to reproduce is to start an OVN sandbox and "ovn-sbctl destroy chassis ."17:43
dcearazhouhan: With Mark's latest patch applied ^17:44
zhouhandceara: ok, thanks, I will try it. What's the suggested fix then?17:44
zhouhanI still doesn't understand "Essentially *_is_new() will never return true for rows inserted locally.", and how destroying a chassis would trigger the _is_new() call.17:45
zhouhanMaybe you could reply in the email?17:46
dcearazhouhan: Sure, I'll try to detail it there.17:46
zhouhanthanks a lot dceara. (please continue your update)17:46
dcearaWe had a bug reported by OpenStack related to IGMP Snoop and localnet.  I sent the fix for it (http://patchwork.ozlabs.org/project/ovn/patch/1602164887-24755-1-git-send-email-dceara@redhat.com/) but while testing I realized there's another problem with IGMP Relay and localnet causing multicast packets to be duplicated sometimes.  I don't know how to fix it yet or if it's possible to but I added a TODO entry and am looking forward to17:48
dcearasuggestions. :)17:48
*** anilvenkata_ has quit IRC17:48
* numans waves and disappears 17:49
*** anilvenkata has joined #openvswitch17:49
dcearaAnd more of an OVS issue but spotted with an OVN deployment, there's a bug in the OVS kernel module when doing DNAT.  I sent a patch for it on the netdev mailing list.  The issue was seen very often with OVN-k8s.17:50
dcearaThat's it from my side today, thanks!17:50
dcearaOh, link for the conntrack issue: https://bugzilla.redhat.com/187712817:51
openstackbugzilla.redhat.com bug 1877128 in openvswitch2.13 "OVS complains Invalid Argument on TCP packets going into conntrack" [Urgent,Post] - Assigned to dceara17:51
zhouhandceara: thanks, could you share the link to the patch as well?17:52
dcearazhouhan: Sure: https://patchwork.kernel.org/patch/11820857/17:53
zhouhanthanks dceara17:55
mmichelsonAnybody else going to give an update?17:57
_lore_I can update very fast17:58
_lore_I am working on adding renew/rebind msg support to IPv6 Prefix Delegation17:58
_lore_I guess I will post v1 by the end of the week17:59
_lore_that's all from my side18:00
mmichelsonOK, anyone else?18:01
mmichelsonAll right everyone, thanks! Have a good day/afternoon/night!18:02
dcearaBye!18:03
imaximetsBye!18:03
mmichelson#endmeeting18:04
openstackMeeting ended Thu Oct  8 18:04:17 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:04
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-10-08-17.17.html18:04
zhouhanbye18:04
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-10-08-17.17.txt18:04
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-10-08-17.17.log.html18:04
mmichelsonphew, it worked this week :)18:04
_lore_bye18:07
*** dceara has quit IRC18:25
*** larsks has quit IRC19:22
*** larsks has joined #openvswitch19:34
*** larsks is now known as Guest2571819:34
*** Guest25718 has quit IRC19:37
*** Guest25718 has joined #openvswitch19:37
*** Guest25718 is now known as larsks19:37
*** larsks has quit IRC19:44
*** larsks has joined #openvswitch19:45
*** ralonsoh has quit IRC19:50
*** ihrachys has quit IRC20:24
*** slaweq has quit IRC20:26
*** dcbw has quit IRC21:15
*** rebrec has quit IRC22:24
*** imaximets has quit IRC22:25
*** imaximets has joined #openvswitch22:25
*** larsks has quit IRC22:31
*** bostondriver has quit IRC22:31
*** larsks has joined #openvswitch22:37
*** rcernin has joined #openvswitch22:47
*** fbl has quit IRC23:25
*** JamesBenson has quit IRC23:48
*** JamesBenson has joined #openvswitch23:48
*** armax has quit IRC23:59

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