Thursday, 2021-01-07

*** acidfu has quit IRC00:01
*** bostondriver has quit IRC00:03
*** anilvenkata has joined #openvswitch00:34
*** moldorcoder7 has quit IRC01:06
*** moldorcoder7 has joined #openvswitch01:06
*** moldorcoder7 has quit IRC01:12
*** moldorcoder7 has joined #openvswitch01:16
*** spatel has joined #openvswitch01:16
*** armax has quit IRC01:40
*** acidfu has joined #openvswitch01:52
*** spatel has quit IRC02:03
*** moldorcoder7 has quit IRC02:15
*** moldorcoder7 has joined #openvswitch02:17
*** spatel has joined #openvswitch02:18
*** acidfu has quit IRC02:36
*** acidfu has joined #openvswitch02:38
*** acidfu has quit IRC02:43
*** acidfu has joined #openvswitch02:55
*** spatel has joined #openvswitch03:13
*** spatel has quit IRC06:54
*** spatel has joined #openvswitch06:55
*** spatel has quit IRC06:55
*** spatel has joined #openvswitch06:56
*** spatel has quit IRC07:01
*** dholler has joined #openvswitch07:01
*** rcernin has quit IRC07:07
*** ralonsoh has joined #openvswitch07:30
*** elvira has joined #openvswitch07:44
*** zenkuro has joined #openvswitch08:04
*** slaweq has joined #openvswitch08:06
*** eelco has joined #openvswitch08:13
*** BlackDex has joined #openvswitch09:06
*** zenkuro has quit IRC09:41
*** zenkuro has joined #openvswitch09:42
*** zhouhan has joined #openvswitch10:30
*** blahdodo has quit IRC10:48
*** blahdodo has joined #openvswitch10:53
*** spatel has joined #openvswitch10:57
*** spatel has quit IRC11:02
*** acidfu has quit IRC11:12
*** acidfu has joined #openvswitch11:21
*** elvira has quit IRC12:50
*** elvira has joined #openvswitch12:52
*** dcbw has joined #openvswitch13:21
*** spatel has joined #openvswitch14:03
*** oanson has quit IRC14:15
*** oanson has joined #openvswitch14:17
*** bostondriver has joined #openvswitch14:19
*** acidfoo has joined #openvswitch14:31
*** acidfu has quit IRC14:33
*** fdangelo has joined #openvswitch14:35
*** slaweq has quit IRC14:37
*** slaweq has joined #openvswitch14:40
*** armax has joined #openvswitch15:09
*** slaweq has quit IRC15:22
*** slaweq has joined #openvswitch15:28
*** eduardas has joined #openvswitch15:37
*** dholler has quit IRC16:55
*** mdgray has joined #openvswitch16:56
*** eelco has quit IRC17:05
*** eduardas_ has joined #openvswitch17:12
*** eduardas has quit IRC17:13
*** eduardas_ has quit IRC17:20
*** eduardas has joined #openvswitch17:20
*** ralonsoh has quit IRC17:45
mmichelsonHi everyone18:15
mmichelsonI'm going to get the OVN meeting started18:16
mmichelson#startmeeting ovn_community_development_discussion18:16
openstackMeeting started Thu Jan  7 18:16:14 2021 UTC and is due to finish in 60 minutes.  The chair is mmichelson. Information about MeetBot at http://wiki.debian.org/MeetBot.18:16
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.18:16
openstackThe meeting name has been set to 'ovn_community_development_discussion'18:16
mmichelsonSo is anyone else here for the OVN meeting?18:16
*** blp has joined #openvswitch18:17
blphi18:17
mmichelsonhi blp18:17
mmichelsonblp, I just started the meeting. Not sure who all is actually here this week18:17
blpI've been cocooned in staycation mode for a couple of weeks. Now I have emerged as a beautiful butterfly and ready to do some work.18:17
mmichelsonyay!18:17
blpI found the linking bug that some people were experiencing trying to build ovn+ddlog. That will be fixed in the next version.18:18
blpI need to do a new rev of that. I'll post it when it's ready.18:18
mmichelsonblp, what's the status of the IDL patches for OVS that DDLog is dependent on that?18:18
mmichelsons/that//18:18
blpmmichelson: I posted v4. Haven't seen any reviews.18:19
_lore_hi all18:19
blpmmichelson: I'll rebase it against master to make sure that no changes are needed.18:19
mmichelsonblp, OK cool.18:19
blpand if any are, I'll post a v5.18:19
mmichelsonblp, When the new version of the ddlog patches are posted, I'm tempted to say we should get them merged as soon as we can18:19
mmichelsonBut if the IDL split patches are also needed, then it seems silly to merge the DDLog patches too18:20
blpThe IDL split is a prereq.18:20
mmichelsonok, that's what I thought18:20
blp(It wasn't originally, but I got feedback that there was too much code duplication without the split.)18:20
mmichelsonblp, okie-dokie18:21
blp(which was fair)18:21
imaximetsblp, mmichelson: I'll take another look at IDL split v4 tomorrow or early next week.18:21
mmichelsonimaximets, excellent18:21
blpimaximets: thanks!18:21
blpI'm a little shaken today with political events from yesterday.18:21
mmichelsonWe're about 4 weeks away from the proposed OVN soft freeze date for 21.0318:21
mmichelsonSo getting more time with DDLog baked in is a good thing18:21
blpmmichelson: yes18:22
mmichelsonOn my side of things, I've been working up a document that discusses pain points and possible solutions for ovn-controller code.18:23
mmichelsonAt this point, the document isn't ready to be shared publicly18:23
blpmmichelson: OK.18:23
blpmmichelson: I've found the ovn-controller code pretty impenetrable lately.18:24
mmichelsonBut the gist of the doc is that ovn-controller has reached a point where it is becoming more and more common that we try to add something new but end up breaking something as a result18:24
blp(partly my fault of course)18:24
mmichelsonblp, it's all our faults18:24
mmichelsonI was kind of hoping zhouhan, blp, and others might be able to chime in with quick high-level ideas for what y'all think are the sticking points in ovn-controller, and if you have ideas for how to fix them.18:25
mmichelsonI'd like to ensure that whatever document I publish takes into account the viewpoints of everyone and not just Red Hat people.18:25
blpWell, ultimately I think ddlog might help? It's very good at making incremental computation easily understood.18:26
zhouhanmmichelson: The lately problem was mostly related to the changes related to change-handler of local interface and port-bindings. There were some temporary code which numans planned to rework. numans also mentioned that's the next thing he will work on.18:27
zhouhanblp: in general I believe ddlog is the better approach18:28
zhouhanblp: I wonder if it is even better for ovn-controller to directly consume the NB DB, skipping the logical flow stage. Would that result in simpler implementation?18:29
mmichelsonHm18:30
zhouhan(of course this is a bigger architecture change)18:30
zhouhanbut in reality maybe it is not harder than converting existing ovn-controller to ddlog.18:31
blpzhouhan: It's an option! I would like to think that doing the nb->sb transformation in one place does a lot of work in one place that otherwise would have to happen in many places.18:31
blpzhouhan: I don't have measurements to back that up.18:31
mmichelsonMy initial thought was that a lot of the work would be duplicated if ovn-controller is handling NB directly18:31
blpmmichelson: that's what i mean, yes18:32
mmichelsonBut the thing is, if logical flows aren't being generated18:32
mmichelsonThen that could mean that the duplicated work isn't happening18:32
blpyes, that's possible too18:32
mmichelsonAnyway, nice to hear some input18:35
zhouhanI would like to try this with ddlog, but I will still need to learn ddlog first :)18:35
mmichelsonI should have a document ready for publication next week18:35
blpmmichelson: great!18:35
zhouhanI guess most part of the current northd ddlog would be reused for that.18:35
mmichelsonI probably won't be suggesting DDLog as the solution, BUT, what I suggest may pave the way for DDLog to slot in more easily at some point?18:35
blpmmichelson: makes sense18:36
mmichelsonAnyway that's all from me18:36
_lore_can I go next? very fast18:38
_lore_this week I worked on v8 of bfd support for ovn (posted upstream)18:38
_lore_moreover I posted some ovn-trace fixes18:39
blp_lore_: all done?18:41
blpAnyone else?18:42
imaximetsSmall update from me18:43
_lore_blp: yes sorry18:43
_lore_thx18:43
blp_lore_: np18:43
imaximetsFor the 'diff's as a file transactions in ovsdb:  I'm going to add "downgrade" related documentation and send v2 somewhere soon.18:44
blpimaximets: great! This change has great potential.18:45
imaximetsAnd OVS is in a soft-freeze now, but I think we still could accept IDL split and ovsdb "diff"s since those were discussed and reviewed mostly.18:46
imaximetsThat's it from me.18:46
blpI think IDL split is ready; it just needs a final look from someone.18:46
imaximetsblp, sure.18:47
blpAre we done for today?18:51
mmichelsonI suppose so?18:51
mmichelsonAll right thanks everyone18:52
mmichelson#endmeeting18:52
openstackMeeting ended Thu Jan  7 18:52:04 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:52
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2021/ovn_community_development_discussion.2021-01-07-18.16.html18:52
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2021/ovn_community_development_discussion.2021-01-07-18.16.txt18:52
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2021/ovn_community_development_discussion.2021-01-07-18.16.log.html18:52
*** blp has left #openvswitch18:54
*** mdgray has quit IRC19:00
*** ktraynor has quit IRC19:15
*** eduardas has quit IRC19:32
*** mdgray has joined #openvswitch19:38
*** elvira has quit IRC19:56
*** mdgray has quit IRC20:53
*** zhouhan has quit IRC21:49
*** rcernin has joined #openvswitch22:19
*** dcbw has quit IRC22:29
*** bostondriver has quit IRC22:33
*** fdangelo has quit IRC22:59
*** zenkuro has quit IRC23:03
*** spatel has quit IRC23:07

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