Thursday, 2021-03-18

*** bostondriver has quit IRC00:22
*** HongKong has joined #openvswitch00:25
*** dcbw has quit IRC01:43
*** acidfoo has quit IRC02:03
*** rcernin_ has quit IRC02:21
*** rcernin_ has joined #openvswitch02:48
*** rcernin_ has quit IRC02:53
*** rcernin has joined #openvswitch02:54
*** zhouhan has joined #openvswitch03:10
*** HongKong has quit IRC03:47
*** teardown has quit IRC04:27
*** teardown has joined #openvswitch04:39
*** anilvenkata has joined #openvswitch04:48
*** teardown has quit IRC05:43
*** jaicaa has quit IRC06:31
*** jaicaa has joined #openvswitch06:33
*** ricolin has quit IRC06:48
*** dholler has joined #openvswitch06:52
*** eelco has joined #openvswitch06:57
*** ralonsoh has joined #openvswitch07:18
*** crabbey has quit IRC07:19
*** afics has joined #openvswitch07:39
*** ebail has joined #openvswitch07:49
*** mdgray has joined #openvswitch08:12
*** elvira has joined #openvswitch08:30
*** rcernin has quit IRC08:37
*** rcernin has joined #openvswitch09:05
*** ktraynor has joined #openvswitch09:19
*** rcernin has quit IRC09:19
*** ebail has quit IRC09:48
*** rcernin has joined #openvswitch10:10
*** rcernin has quit IRC10:40
*** tbachman has quit IRC11:01
*** tbachman_ has joined #openvswitch11:01
*** warewolf has quit IRC12:04
*** slaweq has quit IRC12:22
*** rcernin has joined #openvswitch12:22
*** rcernin has quit IRC12:24
*** rcernin has joined #openvswitch12:24
*** slaweq has joined #openvswitch12:25
*** bostondriver has joined #openvswitch12:33
*** ricolin has joined #openvswitch12:44
*** tbachman_ has quit IRC12:46
*** aconole has quit IRC12:46
kklimonda_numans: Hi, if you have a minute, I've been writing some code to fix the packet loop when router has SNAT enabled and packet is sent to router's IP, mostly to get a better understanding of how ovn internals work. I've been trying to reproduce that issue by writing a testcase, but I'm unsure how to test behavior of a packet arriving from external network.13:22
kklimonda_is there some other test I could take a look at to see how such test is done?13:23
*** acidfoo has joined #openvswitch13:27
numanskklimonda_, Hi13:58
numanskklimonda_, we do have lot of tests in tests/ovn.at and in tests/system-ovn.at for such scenarios.13:59
numansI can point you a test case13:59
kklimonda_ooh, for some reason I didn't look into system-ovn.at  - it seems "ovn -- multiple gateway routers, SNAT and DNAT" is more or less what I'm looking for, right?14:02
numanskklimonda_, I'd suggest to look into system-ovn.at14:02
numansit should have cases where we inject the pkt from the provider bridge...14:02
numanskklimonda_, you could create an ovs port on a provider bridge (say br-phys), associate to a namespace and inject the pkt. the pkt should reach ovn pipeline into the br-int via the patch ports.14:03
kklimonda_thanks, it seems system-ovn.at has a lot of examples that fit what I need14:05
*** rcernin has quit IRC14:35
*** stand has joined #openvswitch14:45
*** tabakhase has quit IRC14:51
*** tabakhase has joined #openvswitch14:55
*** dholler has quit IRC15:29
*** dholler has joined #openvswitch15:43
*** acidfoo has quit IRC16:03
*** acidfoo has joined #openvswitch16:04
*** eelco has quit IRC16:31
*** acidfoo has quit IRC16:56
*** acidfoo has joined #openvswitch17:00
*** blp has joined #openvswitch17:00
blpHello everyone!17:00
*** dholler has quit IRC17:02
mmichelsonHi blp17:04
*** imaximets has quit IRC17:04
blpmmichelson: I have good news for the meeting!17:05
*** imaximets has joined #openvswitch17:05
blp(but not enough time to have sent out patches)17:05
mmichelsonblp, always intriguing when you say you have good news. I think this marks 2 weeks in a row.17:09
*** dceara has joined #openvswitch17:13
mmichelsonI think it's about time to start. I'll issue the command...17:14
blpmmichelson: I'm also in another call at the same time so I might be a bit flaky in responding.17:14
mmichelson#startmeeting ovn_community_development_meeting17:15
openstackMeeting started Thu Mar 18 17:15:06 2021 UTC and is due to finish in 60 minutes.  The chair is mmichelson. Information about MeetBot at http://wiki.debian.org/MeetBot.17:15
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:15
openstackThe meeting name has been set to 'ovn_community_development_meeting'17:15
mmichelsonIt's time once again for the weekly OVN meeting.17:15
mmichelsonThanks to everyone who helped out, we released OVN 21.03.0 last week. I have a PR up for the ovn-website that adds the release and release notes. I don't think it's been merged yet though.17:15
mmichelsonI've mentioned in here a couple of times work I was doing regarding an oddball floating IP scenario that was not working for OpenStack. numans and blp gave reviews to my two-patch series. numans pointed out there's a failing test, and I'm working on getting that fixed now.17:16
mmichelsonI hope to have a new version of that up before the end of the day but might have to wait until tomorrow if it's tougher to fix than I had expected.17:17
blpMost of my reviews these days are about ddlog though.17:17
mmichelsonblp, yeah, in my case you just ensured the C and DDLog were equivalent.17:17
blpRight...17:17
mmichelsonWe have a bit of a problem with tests being inconsistent in whether they pass or fail. blp has done a good job during DDLog development of fixing up some of them.17:18
blpI've got some more of those coming up.17:18
mmichelsonI'm planning to get some issues filed on redhat's bugzilla for each of the "flaky" tests so we can get to the bottom of them. We can determine if the tests are flaky or if OVN is flaky.17:18
mmichelsonblp, oh nice.17:18
mmichelsonblp, there may be some overlap there then.17:19
mmichelsonIf we can't trust CI to be correct, then it's doing us no good.17:19
mmichelsonSo we need to ensure that a red run actually means there is a problem, and not get in the habit of constantly rechecking or ignoring failures.17:19
mmichelsonThanks to imaximets for fixing the weirdness with flake8 checking17:19
mmichelsonThat's a big help17:20
mmichelsonAnd I think that's all I had wanted to mention.17:20
mmichelsonWhoever wants to go next, feel free.17:20
mmichelsonI know we're unlikely to hear from dceara since this is a bad time for him, and _lore_ is on PTO today.17:20
imaximetsI have a small update.  Hi.17:21
imaximetsI tried to review ovsdb-idl fixes from dceara.  It's on v3 now.  I have couple of comments to this patch-set.  Will send them shortly.17:22
imaximetsOtherwise, worked a bit on GHA with its wierd python configurations.17:22
imaximetsthat's it from me.17:23
blpI have an update.17:23
blpI received some assistnace from Leonid on optimizing the ddlog implementaiton of northd.17:24
blpWe looked at the benchmark from Numan.17:24
blpI did some work on the benchmark itself17:24
blpand some work on ovn-sbctl17:24
blpAnd managed to make the benchmark run several times faster without ddlog17:25
blpand something like 3x faster than that with ddlog17:25
blpNow the timing on my laptop is something like 115 seconds without ddlog and 35 seconds with it.17:25
mmichelsonblp, what sort of changes did you make in ovn-sbctl?17:25
blpI haven't managed to fit together the full patch series yet, but that will be coming up when I have some actual time outside of meetings.17:26
blpI added the same daemon feature that ovn-nbctl has.17:26
blpand made the benchmark use the daemon for both utilities.17:26
mmichelsonblp, interesting. I thought that since ovn-sbctl was selective with which tables/columns it was interested in, it didn't have the same scaling issues as ovn-nbctl. Or at least the scaling issues wouldn't be nearly as dire. It's interesting to learn this.17:27
blpIt made a huge difference.17:27
blpEvery iteration in the benchmark runs ovn-nbctl once and ovn-sbctl twice. The cost of ovn-sbctl overwhlemed the rest of the test.17:27
blpWell, in the fomr that Numan sent it, the benchmark ran ovn-nbctl dozens of times, but it didn't need to, so I merged into just one.17:28
mmichelsonblp, that is some good news. You mentioned during my update that you had more test fixes coming up. Are those related to your ddlog optimizations or are they a separate patch series?17:29
blpThey're in my series. They aren't huge things, just minor improvements to tests.17:30
mmichelsonblp, ack17:30
blpI get lots of test failures while checking ddlog, and I try to eliminate them when they're races.17:30
blp(usually, they are races)17:30
mmichelsonblp, I suspect that's the case for most of the tests.17:32
blpI would welcome more challenge benchmarks if anyone wants to send me them.17:33
*** aconole has joined #openvswitch17:33
imaximetsI guess, daemon mode for sbctl was not implemented just because users normally doesn't need it.17:33
blpIt doesn't cost us much.17:33
mmichelsonIt's used way less frequently than nbctl17:33
mmichelsonMy most typical use of sbctl is to inspect logical flows17:34
mmichelsonAnyone else present? Anyone else want to give a report?17:34
blpCurrently my daemon mode is cut-and-paste from ovn-nbctl, but I might work to factor it into common code.17:35
mmichelsonblp, that would probably be for the best17:35
mmichelson(factoring, I mean)17:35
* blp nods17:35
blpThat's my whole report.17:36
mmichelsonOK, last chance for someone to speak up. Otherwise this meeting is OVER17:37
mmichelsonI certainly don't mind a short meeting :)17:37
blpSHORT MEETING FTW17:37
flaviofHi folks! I have a --kind of a-- question for the IP gurus (like zhouhan). :)17:38
flaviofI seem to have hit a bug where rate in the meter band gets updated in the NB+SB, but17:38
flaviofovn-controller is failing to act on it, even when calling "ovn-appctl -t ovn-controller recompute".17:38
flaviofref link: https://bugzilla.redhat.com/show_bug.cgi?id=193952417:38
flaviofIs there anything special when it comes to meter tables for the incremental processing logic?17:38
flaviofNo worries if you don't know from the top of your head. Just something to be worked on I17:38
openstackbugzilla.redhat.com bug 1939524 in ovn2.13 "Changing rate on meter does not take effect unless ovn-controller is restarted" [High,New] - Assigned to ovnteam17:38
flaviofwanted to make you aware of.17:38
zhouhanflaviof: I can look at it17:39
blpflaviof: Does it work if you restart ovn-controller? I mean, is it a general bug as opposed to a I-changed-something-and-it-didn't-update bug?17:39
flaviofzhouhan++ yes, it works fine if I restart the ovn-controller17:39
zhouhanflaviof: not sure what's wrong. It is either a missing dependency or something more subtle17:40
flaviofzhouhan: ack.17:40
zhouhanflaviof: let met check more details and come back17:40
zhouhanblp: thanks for the daemon mode work for sbctl, I think it is quite useful, especially for some operational tasks17:41
zhouhanI don't have anything to update. I will get more time on the review next week17:42
blpzhouhan: Glad someone has a positive view :-)17:43
flaviofblp no lack of appreciation for what you do, here!17:44
mmichelsonAll right everyone. I guess that's all for today. Have a nice day/weekend17:44
mmichelson#endmeeting17:44
openstackMeeting ended Thu Mar 18 17:44:56 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:44
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-03-18-17.15.html17:44
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-03-18-17.15.txt17:45
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-03-18-17.15.log.html17:45
blpGoodbye everyone! see you next week!17:45
*** blp has left #openvswitch17:45
*** elvira has quit IRC17:51
*** ALEXEY_ROYTMAN has joined #openvswitch18:44
*** dcbw has joined #openvswitch18:44
*** dceara has quit IRC18:48
*** trident has quit IRC19:01
*** warewolf has joined #openvswitch19:09
*** trident has joined #openvswitch19:19
*** mdgray has quit IRC19:38
*** teardown has joined #openvswitch19:57
*** rcernin has joined #openvswitch20:52
*** rcernin has quit IRC21:06
*** rfolco has quit IRC21:14
*** rcernin has joined #openvswitch21:15
*** ralonsoh has quit IRC21:20
*** rcernin has quit IRC21:22
*** imaximets_ has joined #openvswitch21:30
*** imaximets has quit IRC21:31
*** openstack has joined #openvswitch21:38
*** openstack has joined #openvswitch22:03
*** crabbey has joined #openvswitch22:05
*** ihrachys has quit IRC22:32
*** ihrachys has joined #openvswitch22:32
*** BlackDex_ has quit IRC23:06
*** BlackDex_ has joined #openvswitch23:07
*** tbachman has joined #openvswitch23:35
*** tbachman_ has joined #openvswitch23:39
*** tbachman has quit IRC23:40
*** tbachman_ is now known as tbachman23:40
*** moldorcoder7 has quit IRC23:46
*** moldorcoder7 has joined #openvswitch23:58

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