Thursday, 2021-05-06

*** deadalnix has quit IRC01:15
*** tabakhase__ has joined #openvswitch01:51
*** ktraynor has quit IRC01:52
*** tamas_erdei has quit IRC01:52
*** gadago has quit IRC01:52
*** vancz has quit IRC01:52
*** otherwiseguy has quit IRC01:52
*** numans has quit IRC01:52
*** erig has quit IRC01:52
*** tabakhase has quit IRC01:53
*** ktraynor has joined #openvswitch01:54
*** tamas_erdei has joined #openvswitch01:54
*** gadago has joined #openvswitch01:54
*** vancz has joined #openvswitch01:54
*** otherwiseguy has joined #openvswitch01:54
*** numans has joined #openvswitch01:54
*** erig has joined #openvswitch01:54
*** markmcclain has quit IRC01:55
*** markmcclain has joined #openvswitch01:57
*** tabakhase__ has left #openvswitch01:58
*** tabakhase has joined #openvswitch01:58
*** rcernin has quit IRC04:35
*** rcernin has joined #openvswitch04:38
*** _lore_ has quit IRC04:54
*** _lore_ has joined #openvswitch05:00
*** anilvenkata has quit IRC05:48
*** anilvenkata has joined #openvswitch05:52
*** psahoo has joined #openvswitch06:00
*** dholler has joined #openvswitch06:02
*** psahoo has quit IRC06:03
*** slaweq has joined #openvswitch06:15
*** ralonsoh has joined #openvswitch06:31
*** anilvenkata has quit IRC06:41
*** anilvenkata has joined #openvswitch06:42
*** anilvenkata has quit IRC06:45
*** mdgray has joined #openvswitch07:20
*** rcernin has quit IRC07:36
*** elvira has joined #openvswitch07:36
*** jaicaa has quit IRC07:43
*** jaicaa has joined #openvswitch07:44
*** rcernin has joined #openvswitch08:01
*** rcernin has quit IRC08:07
*** rcernin has joined #openvswitch08:20
*** rcernin has quit IRC08:26
*** rcernin has joined #openvswitch08:34
*** anilvenkata has joined #openvswitch08:44
*** istokes has joined #openvswitch08:53
*** rcernin has quit IRC09:32
*** psahoo has joined #openvswitch09:51
*** deadalnix has joined #openvswitch10:15
*** rcernin has joined #openvswitch10:24
*** rcernin has quit IRC10:25
*** rcernin has joined #openvswitch10:25
*** deadalnix has quit IRC10:29
*** anilvenkata has quit IRC10:35
*** anilvenkata has joined #openvswitch10:43
*** anilvenkata has quit IRC10:44
*** anilvenkata has joined #openvswitch10:44
*** istokes has quit IRC10:52
*** istokes has joined #openvswitch10:55
*** thaller has quit IRC10:56
*** thaller has joined #openvswitch10:57
*** deadalnix has joined #openvswitch10:58
*** istokes has quit IRC11:00
*** istokes has joined #openvswitch11:10
tryauuumHow to make traffic to vtep port be processed on a local hypervisor?11:23
tryauuumI have a following scheme |client_switch| → |client_router| → |switch_with_particular_vtep_port|. It does work, but traffic (+application of dnat_and_snat rules) is processed on a hypervisor where client router resides.11:24
*** rcernin has quit IRC11:24
*** mdgray has quit IRC11:26
*** deadalnix has quit IRC11:28
*** deadalnix_ has joined #openvswitch11:28
*** istokes has left #openvswitch11:46
*** istokes has joined #openvswitch11:49
*** mdgray has joined #openvswitch12:12
*** thaller has quit IRC12:18
*** |subz3r0| has joined #openvswitch12:18
*** |subz3r0| has joined #openvswitch12:18
*** thaller has joined #openvswitch12:18
*** istokes has left #openvswitch12:21
*** ihrachys has joined #openvswitch12:23
*** deadalnix_ is now known as deadalnix12:28
*** bostondriver has joined #openvswitch12:28
tryauuumand another question — do I have to set "external-ids:ovn-encap-type=geneve,vxlan" on chassis for VXLAN ("vtep port") connection to TOR to work? If I do not, ovn-controller-vtep emits message "VTEP tunnel encap on chassis (%s) not found". But I am scared by information in the following release, which says that total number of network is reduced to 4096 if any chassis has VXLAN-encap enabled13:05
*** fdangelo has joined #openvswitch13:11
*** istokes1 has joined #openvswitch13:21
*** moldorcoder7 has quit IRC13:41
*** moldorcoder7 has joined #openvswitch13:44
*** istokes has joined #openvswitch13:44
*** istokes1 has quit IRC13:45
*** dcbw has joined #openvswitch13:46
*** eduardas has joined #openvswitch13:55
*** istokes has left #openvswitch14:16
*** istokes has joined #openvswitch14:20
*** mdgray has quit IRC14:34
*** mdgray has joined #openvswitch14:55
*** deadalnix_ has joined #openvswitch15:01
*** deadalnix has quit IRC15:02
*** istokes has left #openvswitch15:19
*** rcernin has joined #openvswitch15:20
*** rcernin has quit IRC15:25
*** istokes has joined #openvswitch15:25
*** istokes has quit IRC15:38
*** istokes has joined #openvswitch15:44
*** dholler has quit IRC16:11
*** istokes has left #openvswitch16:18
*** istokes has joined #openvswitch16:28
*** istokes has left #openvswitch16:28
*** ralonsoh has quit IRC16:58
*** dceara has joined #openvswitch17:10
*** blp has joined #openvswitch17:13
*** zhouhan has joined #openvswitch17:16
imaximetshello everyone.17:16
blphi!17:16
imaximetsmmichelson is not here today.17:16
blpI am sorry that I disappeared for a few weeks. I am back to update my patch series.17:17
_lore_hi all17:17
blpYou should do the startmeeting thing17:17
zhouhanHi17:17
imaximetsok, let me be the host.17:17
imaximets#startmeeting ovn_community_development_meeting17:17
openstackMeeting started Thu May  6 17:17:31 2021 UTC and is due to finish in 60 minutes.  The chair is imaximets. 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_meeting'17:17
blpThanks imaximets17:17
imaximetsI have some updates.17:17
imaximetsFirst is that OVSDB 2-Tier patch-set finally posted17:18
imaximets#link https://patchwork.ozlabs.org/project/openvswitch/list/?series=24164217:18
imaximetsI also sent today a patch for raft17:19
imaximetsto transfer leadership before starting database compaction17:19
imaximets#link https://patchwork.ozlabs.org/project/openvswitch/patch/20210506124731.3599531-1-i.maximets@ovn.org/17:19
imaximetsthe  problem is that database compaction/snapshotting could take a lot of time if database is big17:20
blpIt's a good idea. Thanks for writing it up.17:20
imaximetse.g 200-500MB of on-disk data.  It takes 2-15 seconds to create a snapshot17:20
imaximetsand cluster is not functional if the server that creates a snapshot is a leader.17:21
imaximetstransferring the leadership makes the system more stable.17:21
imaximetsand the last update:17:21
blpWow, that's a lot. I never did measurements.17:21
imaximetsI also worked few weeks ago on a patch for OVN to collapse ARP responder flows.17:23
blpExcellent.17:23
imaximetsOn some setups these flows could take half of the southbound db.17:23
blpwow17:23
imaximetsIn my case it was almost 250MB out of 500MB database.17:23
imaximetsAnd the change actually reduced the db size in half.17:24
numansimaximets++17:24
numansthat's great17:24
imaximetsI have a patch, but I don't have a ddlog part yet.17:24
imaximetsit looks like this: https://github.com/igsilya/ovn/commit/732365025682cdb2987d601d3caedee1a94dfcf717:25
imaximetsthe idea is to simplify the actions (i.e. remove dependency from addresses) and put all load balancer IPs into one lflow instead of creating a separate lflow for all of them.17:26
imaximetsAlternatively this could be done with the address set.  I'll probably will go with the address set for the final patch.17:26
blpWonderful.17:27
imaximetsIIRC, tomorrow is the deadline to submit v1 for 20.06?17:27
imaximets*21.0617:27
imaximetsSo, I will post some version of the patch tomorrow.  And will work it out later if will not be able to figure out ddlog parts until then.17:28
zhouhanimaximets combining flows may have some implication on I-P processing (maybe that's not a problem for this case, but we need to be careful)17:28
blpI upgraded to Fedora 34 and therefore GCC 11 and I'm getting some new warnings for OVS. I didn't notice anyone posting patches for these already, has anyone else?17:29
imaximetszhouhan, sure.  But in this case shrinking the database in half is kind of important too, because ovsdb-server just cant handle it.17:29
imaximetsblp, I have a 'may be uninitialized' in netdev-dummy.17:30
blpOK, this is different. I'll fix and post.17:30
zhouhanimaximets yes. CPU & MEM sometimes need trade-offs17:30
imaximetsblp, thanks!17:31
imaximetsok.  that was it from my side.17:31
imaximetswho will go next?17:31
blpI just have a quick report17:31
imaximetsblp, go ahead.17:32
blpwhich is that I'm getting back to fixing up my patch series and will repost.17:32
blpSorry I disappeared for a while.17:32
_lore_can I go next? quite fast17:33
imaximets_lore_, sure, if that's all from blp?17:33
_lore_blp: yes, sorry17:33
blpOh, I think that's all.17:33
blpIf I think of more, I'll add it.17:34
blpOh, one thing is that we got a paper about OVS accepted at SIGCOMM.17:34
zhouhancongrats!17:34
imaximetsblp, cool!17:34
imaximetsok, _lore_ , your turn.17:35
_lore_ok, last week I worked on CoPP series, I added unit/system tests and remove per-port metering, posted v1 upstream17:35
_lore_http://patchwork.ozlabs.org/project/ovn/list/?series=24140017:35
_lore_I guess we can continue discussing from v1 about possible changes on API and so on17:36
_lore_I added a fix for localport --> localnet traffic17:36
_lore_http://patchwork.ozlabs.org/project/ovn/patch/8008fa9867d210cf18ad31f912535f2c14e85c43.1620151078.git.lorenzo.bianconi@redhat.com/17:36
_lore_and I have a pending debug patch out for review17:37
_lore_http://patchwork.ozlabs.org/project/ovn/patch/3a9ca2d94478a3b2db51d719233c9a28cafcf75b.1619268349.git.lorenzo.bianconi@redhat.com/17:37
_lore_that's all from my side, thx17:37
zhouhanI can go next17:38
imaximets_lore_, thanks!17:38
imaximetszhouhan, sure.17:38
zhouhanThis patch still needs review: https://patchwork.ozlabs.org/project/ovn/patch/20210504013332.1689887-1-hzhou@ovn.org/17:38
zhouhanThe last version addressed Mark M17:38
zhouhan's comments about test case improvement17:39
mmichelsonzhouhan, I'll have a look at that this afternoon. I plan to review that plus a patch of numans's17:39
mmichelson(btw, I'm back now)17:39
zhouhanmmichelson cool. thanks a lot!17:39
zhouhanI was also checking DP group. I think there is a small problem on the MC_UNKNOWN flow handling. I have a fix and will send out today17:40
imaximetszhouhan, we're not using DP groups for multicast groups.  Or the issue is that we are doing that somewhere?17:41
zhouhanimaximets yes, I think it is just missed for the MC_Unknown group. It is using DPG :)17:42
zhouhanthe fix is simply not using DPG for the flow17:42
imaximetszhouhan, oh, ok.  Sounds good. :)17:42
zhouhanIt also seems that DP group could have a higher CPU cost in ovn-controller when dealing with port group changes for ACL flows, because each change would have be bigger impact with the DPG enabled. But I will study more on this.17:43
* imaximets thinks that we still need to fix dependencies somehow between multicast groups and lflows.17:43
zhouhanimaximets also, any plan to make DPG work in DDlog version? (Or is it already supported but I missed somehow?)17:43
mmichelsonI thought it was supported there17:44
imaximetszhouhan, should be supported.  There is a test for them.  blp should know, I think.17:44
mmichelsonWhen you run a test with OVN_FOR_EACH_NORTHD now, it will run four versions: C with DP groups, C without DP groups, DDLog with DP groups, and DDLog without DP groups17:44
imaximetszhouhan, "ovn -- logical gatapath groups" should work for all implementations, so it should be supported.17:45
blpdatapath groups? I think that they are supported.17:46
blpin ddlog17:46
*** zhouhan56 has joined #openvswitch17:46
zhouhan56test17:46
blpWow, that's a lot of variations, 4x for each of those?17:46
mmichelsonYep17:47
imaximetsblp, unit tests are a bit slow now. :)  but that's for good.17:47
blpThe worst part of the OVN_FOR_EACH_NORTHD stuff is that it doesn't report line numbers very well.17:47
blpIf anyone figures out how to do that better, that'd be great.17:47
*** zhouhan has quit IRC17:48
blpzhouhan56: we see you17:48
*** zhouhan has joined #openvswitch17:48
zhouhanhi I'm back17:48
zhouhanzhouhan56 cannot talk :(17:48
imaximetszhouhan, hi. :)17:49
zhouhansorry I was disconnected for a while17:49
imaximetszhouhan, we concluded that DPG supported in ddlog.17:49
blpzhouhan: welcome back17:49
zhouhanimaximets oh, great. Sorry I missed that17:49
imaximetszhouhan, for the performance, IIRC, there was a lot of issues with I-P engine, so I had to sacrifice some performance on the ovn-controller side.17:50
imaximetsi.e. I-P is not ideal with DPG.17:50
zhouhanimaximets: yes there is some challenge, but I will see if there is a way to make DPG work well with I-P17:51
*** psahoo has quit IRC17:51
imaximetszhouhan, that would be great.  thanks!17:51
zhouhanMEM and CPU are both important for some of our environment :)17:51
zhouhanthe last thing I said before disconnecting was: imaximets the dependency part I think I also have a fix in mind, will work on it as well17:52
zhouhanApart from these, I did review of numans CT patches17:52
imaximetszhouhan, nice.  this also would be great to have.17:52
zhouhanThanks numans for the I-P improvement, but I'd like to discuss more on that patch17:53
zhouhanthat's my update17:53
imaximetsok.  Who wants to go next?17:53
mmichelsonI can go next17:53
imaximetsmmichelson, all yours.17:54
mmichelsonI have two main things I've worked on recently. First is the ARP-related patch series. I posted a v7 of it where I ended up whitelisting a warning message from ovn-controller that had been causing systems tests to fail17:54
mmichelsonI've talked that particular issue over with imaximets and dceara and dceara suggested a possible workaround that I'm going to look into.17:55
mmichelsonv7 also triggered 0-day robot since the line length checks are working again in checkpatch.py17:55
mmichelsonThe other thing I'm working on is an issue where the same conjunction() action can appear multiple times in a flow. I figured out the issue, it's a one-line fix in expression code. The explanation of the issue and the fix will take probably 50x more lines than the actual fix.17:56
mmichelsonThat particular fix should hit the mailing list later today17:56
mmichelsonOther than that, we're approaching soft freeze tomorrow. So any patches that you  want included in 21.06.0 should be posted by tomorrow.17:57
mmichelsonThat's all from me.17:57
blpI posted my GCC 11 series: https://patchwork.ozlabs.org/project/openvswitch/list/?series=24251117:57
blp(It's for OVS, not OVN.)17:57
*** zhouhan has quit IRC17:57
*** zhouhan has joined #openvswitch17:58
imaximetsblp, thanks, I'll take a look.17:58
mmichelsonblp, is there a pending OVN patch, too?17:58
zhouhanmmichelson shall we release 20.03.1 anytime soon?17:58
zhouhansorry, 21.03.117:59
mmichelsonzhouhan, sure, we can do that when we release 21.06.017:59
blpmmichelson: I'll look at OVN now.18:00
zhouhanmmichelson ok, thanks!18:00
imaximetsGood.  Anyone else has updates?18:01
imaximetsOK.  In this case we can close this meeting.  Have a nice day!18:02
imaximets#endmeeting18:02
*** zhouhan56 has quit IRC18:02
openstackMeeting ended Thu May  6 18:02:39 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:02
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-05-06-17.17.html18:02
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-05-06-17.17.txt18:02
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-05-06-17.17.log.html18:02
mmichelsonBye everyone!18:02
zhouhanbye!18:02
*** elvira has quit IRC18:17
*** blp has quit IRC18:30
*** zhouhan has quit IRC18:33
*** dceara has quit IRC18:37
*** mdgray has quit IRC18:57
*** eduardas has quit IRC20:14
*** elvira has joined #openvswitch20:26
*** slaweq has quit IRC20:38
*** dcbw has quit IRC20:46
*** bostondriver has quit IRC21:01
*** rcernin has joined #openvswitch21:36
*** deadalnix_ has quit IRC21:40
*** elvira has quit IRC21:44
*** rcernin has quit IRC21:58
*** znf has quit IRC22:05
*** znf has joined #openvswitch22:06
*** deadalnix has joined #openvswitch22:17
*** rcernin has joined #openvswitch23:02
*** dceara has joined #openvswitch23:10
*** dceara has quit IRC23:11

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