Thursday, 2020-04-30

*** acidfoo_ has quit IRC01:06
*** dcbw has quit IRC01:18
*** duso has joined #openvswitch01:39
dusomorning01:40
*** dholler has quit IRC02:23
*** dholler has joined #openvswitch02:36
*** JamesBenson has quit IRC02:37
*** yamamoto has quit IRC02:54
*** yamamoto has joined #openvswitch02:54
*** dqx has quit IRC03:05
*** JamesBenson has joined #openvswitch03:05
*** dqx has joined #openvswitch03:06
*** yamamoto has quit IRC03:18
*** _lore_ has quit IRC03:23
*** zhouhan_ has quit IRC03:23
*** FH_thecat has quit IRC03:26
*** yamamoto has joined #openvswitch03:31
*** zhouhan has joined #openvswitch03:33
*** psahoo has joined #openvswitch04:11
*** mb__ has joined #openvswitch04:14
*** bern has quit IRC04:16
*** JamesBenson has quit IRC04:43
*** _lore_ has joined #openvswitch04:55
*** mb__ has quit IRC06:16
*** yogananth has joined #openvswitch06:36
*** yogananth_ has joined #openvswitch06:55
*** yogananth has quit IRC06:58
*** slaweq has joined #openvswitch06:58
*** mbarroso has joined #openvswitch07:01
*** eelco has joined #openvswitch07:07
*** yamamoto has quit IRC07:19
*** yamamoto has joined #openvswitch07:23
*** yamamoto has quit IRC07:36
*** yamamoto has joined #openvswitch07:50
*** mmirecki has joined #openvswitch08:23
*** dceara has joined #openvswitch08:37
*** duso has quit IRC08:52
*** theodotos[m] has quit IRC08:56
*** yamamoto has quit IRC09:02
*** theodotos[m] has joined #openvswitch09:11
*** yamamoto has joined #openvswitch09:16
*** yamamoto has quit IRC09:19
*** yamamoto has joined #openvswitch09:23
*** timothy has joined #openvswitch09:33
*** yamamoto has quit IRC09:37
*** yamamoto has joined #openvswitch10:05
*** rcernin has quit IRC11:06
*** FH_thecat has joined #openvswitch12:00
*** FH_thecat has quit IRC12:06
*** FH_thecat has joined #openvswitch12:06
*** ebail has joined #openvswitch12:11
*** _lore_ has quit IRC12:23
*** yamamoto has quit IRC12:23
*** rcernin has joined #openvswitch12:31
*** slaweq has quit IRC12:35
*** aconole has joined #openvswitch12:35
*** slaweq has joined #openvswitch12:37
*** slaweq_ has joined #openvswitch12:38
*** slaweq has quit IRC12:41
*** yamamoto has joined #openvswitch12:49
*** yamamoto has quit IRC12:50
*** yamamoto has joined #openvswitch12:54
*** acidfoo_ has joined #openvswitch12:54
*** yamamoto has joined #openvswitch12:54
*** bostondriver has joined #openvswitch12:57
*** acidfoo_ has quit IRC12:58
*** yamamoto has quit IRC13:02
*** FH_thecat has quit IRC13:06
*** JamesBenson has joined #openvswitch13:10
*** FH_thecat has joined #openvswitch13:11
*** _lore_ has joined #openvswitch13:11
*** mmichelson has quit IRC13:19
*** dcbw has joined #openvswitch13:24
*** mmichelson has joined #openvswitch13:25
*** FH_thecat has quit IRC13:26
*** rcernin has quit IRC13:29
*** acidfoo_ has joined #openvswitch13:37
*** troulouliou_div2 has joined #openvswitch14:10
*** emptynine has joined #openvswitch14:18
*** mmirecki has quit IRC14:35
*** armax has joined #openvswitch14:44
*** slaweq_ is now known as slaweq15:03
*** mmirecki has joined #openvswitch15:03
*** avishnoi has quit IRC15:11
*** avishnoi has joined #openvswitch15:13
*** mmirecki has quit IRC15:19
*** eelco has quit IRC15:30
*** balkamos has quit IRC15:44
*** dholler has quit IRC15:46
*** balkamos has joined #openvswitch15:52
*** zhouhan_ has joined #openvswitch15:56
*** zhouhan has quit IRC16:00
*** balkamos has quit IRC16:15
*** aconole has quit IRC16:15
*** tbachman has quit IRC16:23
*** tbachman has joined #openvswitch16:24
*** balkamos has joined #openvswitch16:30
*** balkamos has quit IRC16:30
*** balkamos has joined #openvswitch16:30
*** balkamos has quit IRC16:41
*** troulouliou_div2 has quit IRC16:44
*** balkamos has joined #openvswitch16:56
*** troulouliou_div2 has joined #openvswitch16:56
*** balkamos has quit IRC17:05
*** psahoo has quit IRC17:12
*** balkamos has joined #openvswitch17:13
mmichelsonHey everyone, I'm about to start the weekly OVN meeting17:16
mmichelson#startmeeting ovn-community-development-discussion17:16
openstackMeeting started Thu Apr 30 17:16:57 2020 UTC and is due to finish in 60 minutes.  The chair is mmichelson. Information about MeetBot at http://wiki.debian.org/MeetBot.17:16
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:16
openstackThe meeting name has been set to 'ovn_community_development_discussion'17:17
numansHello17:17
*** zhouhan_ has quit IRC17:17
*** zhouhan has joined #openvswitch17:18
zhouhanhi17:18
mmichelsonLet's get this thing started!17:18
mmichelsonFirst off, I'd like to remind everyone that tomorrow is soft freeze for OVN 20.06. If you don't have patches for new features up by tomorrow, they'll need to wait.17:19
mmichelsonTomorrow, I'll send out an e-mail asking people which patches they would like to get merged in for the release of 20.0617:19
_lore_hi all17:19
mmichelsonAside from that, I don't think I have anything to share right now. So whoever would like to go next, feel free.17:20
numansI can go real quick17:21
numansI did some code reviews this week.17:21
numansAnd pushed out few patches.17:21
numans1. I just rebased the I-P patches as they had some merge conflict (and hoping this would be pop up for review :))17:22
zhouhannumans: so sorry for the delay of review. I was on some other issues.17:23
numans2. Submitted a patch set to support OF1.5 in OVN and to support load balancer selection fields so tht CMS can decide whether to use dp_hash (which is default) or use hash with specified fields.17:23
numanszhouhan, no worries. I understand :)17:23
numansI also submitted one more patch to configure hwaddr for the br-int. The patch is here - https://patchwork.ozlabs.org/project/openvswitch/patch/20200430081000.751498-1-numans@ovn.org/17:24
numansThat's it from me.17:24
*** mbarroso has quit IRC17:24
numanszhouhan, I've addressed the load balancer selection method here - https://patchwork.ozlabs.org/project/openvswitch/patch/20200430172028.1978594-1-numans@ovn.org/17:24
numansrequest to just go through the commit message. That should give an idea :)17:25
numansand let me know if any issues with the approach.17:25
numansRequest everyone to take a look at the patches.17:25
numansThat's it from me.17:25
zhouhannumans: for 2), not sure if you saw the later discussions on the LB db_hash. I don't think we should switch to using hash instead of dp_hash. It would degrade the performance much.17:26
numanszhouhan, I saw that. The default will be dp_hash17:26
numansand it is left for the CMS to decide.17:26
numanszhouhan, in the case of octavia lb algorithms there are 2 which we can support - IP_SOURCE and IP_SOURCE_PORT.17:26
numansand its possible with this.17:27
zhouhanok, I am ok with your patch that makes it flexible. But I mean the real problem here is the dp_hash implementation. And as imaximets pointed out it is already fixed upstream.17:27
numanszhouhan, agree.17:28
zhouhannumans: and I am not sure when should we suggest user to use "hash" instead of "dp_hash".17:28
numanszhouhan, it might take a while for the kernel patch to be backported to the distros.17:28
numansin the meanwhile cms can use it.17:29
mmichelsonPeople should be using bleeding edge at all times17:29
numanszhouhan, I'd suggest user to use dp_hash.17:29
numansmmichelson, :)17:29
zhouhannumans: if they use "hash", performance problems may be reported ...17:29
numanszhouhan, agree. So its CMS's discretion.17:30
numanszhouhan, I think we should document tht properly.17:30
zhouhanAnd I am also confused about the kernel patch backporting process. Is there anyone taking care of the backportings? What's the practices?17:31
numanszhouhan, I mean it takes time for those patches to be in RHEL kernel for example17:31
numansI'm not sure about ubuntu and other distros.17:31
zhouhannumans: Oh, ok. What I meant is backporting to OVS repo's datapath.17:31
mmichelsonYeah, Debian, RHEL, and CentOS are known for being late to the party17:31
numanszhouhan, Ok. Most of the distros will have openvswitch kernel module in them17:32
numansand RHEL uses the kernel openvswitch datapath17:32
numansOk. I'm done. If someone wants to go next.17:33
zhouhannumans: yes, but we have always been using the ones in OVS repo :) So I am really curious about the maintenance of that part. I thought it was always up-to-date. But this time it didn't happen automatically.17:33
numanszhouhan, Ok.17:34
dcearahi all! sorry for joining late17:34
zhouhanat least blp should have the answer, but he seems to be absent for several weeks17:34
numanszhouhan, from what I know, normally when the patch is merged in kernel, either the author will send it to ovs-dev ML17:34
numansor someone from community posts the patch to ovs ML>17:35
*** yogananth_ has quit IRC17:35
zhouhannumans: ok, this time I pinged the author Tonghao Zhang, and he said he would help the backporting.17:35
numanszhouhan, ack. I've one patch experience. I can post it if the author doesn't post :)17:36
zhouhanHowever, in my testing using fake-multi-node, the problem is solved even without the kernel patch. I didn't understand how. I will do some more investigation, maybe with a real server (instead of fake node)17:36
zhouhanI can keep on going with my report.17:37
numanszhouhan, that's strange bcoz I used the fake-multi-node too and i was able to see the problem17:37
numans:)17:37
zhouhannumans: using master?17:37
numanszhouhan, yes. ovs master17:37
numansand fedora 3117:38
zhouhannumans: hmm... that's really strange17:38
zhouhanApart from the dp_hash invesitigation, I was chasing some other problems.17:38
zhouhanOne of them is that Chassis-redirect-port doesn't work for ingress.17:39
numanszhouhan, chassis-redirect-port .. its the distributed gw port with cr- right ?17:40
zhouhanWhen sending out a packet, the pipeline will redirect packet to the dedicated chassis to do the egress. However, if a packet is sent from LR1 to LR2 and a CRP on LR2 is the ingress port between LR1 to LR2, there is no redirection, and the packet is processed on the current chassis, and it gets dropped because in the admission stage the flows are missing due to the "chassis_resident" check.17:42
zhouhannumans: yes (cr-)17:42
numanszhouhan, ok.17:43
zhouhanI will continue work on it.17:43
zhouhanthat's it form me.17:43
dcearaI can go next if that's ok17:43
dcearazhouhan, regarding the last patch series I sent to make ovn-northd work properly when ports are "moved" from one switch to another https://patchwork.ozlabs.org/project/openvswitch/list/?series=17382317:44
dcearazhouhan, it is something that we might see often with ovn-kubernetes when pods are "moved" from one node to another17:45
dcearazhouhan, because every node has its own logical switch17:45
dcearazhouhan, I think I can make it work so that stale port bindings are deleted instead of being reused but is that a must?17:46
zhouhandceara: I see. I think it is better if we treat it as delete and create.17:46
numansI agree with zhouhan.17:47
numansthis would ensure correctness.17:47
dcearazhouhan, ok, I need to think about other cases too (e.g., multicast_groups use tunnel_keys too)17:47
dcearabut that should be fine. I'll try to post a v3 as soon as possible17:48
zhouhanFrom the logical network's point of view, port shouldn't move between switches. Also, even if the user would like to think so, it has to handle many other things like MAC conflict, etc.17:48
dcearazhouhan, I agree and ovn-kube is not trying to "move" the port, but just deletes the old one and adds a new one with the same name. That is a valid use case in my opinion.17:48
zhouhandceara: yes, exactly. I think recreating a port with same name should be treated as deletion and recreation.17:50
dcearazhouhan, ok, i'll do it like that then17:50
dcearaapart from that, I started on the v5 for the ovsdb-idl fix for monitor_cond_since. I have the code ready but didn't get a chance to test it yet.17:51
dcearathat's it from me17:51
zhouhandceara: thanks a lot. Looking forward to it17:51
dcearazhouhan, np17:51
ihrachysok, maybe me? i'll be quick17:52
ihrachysso the multiple localnet ports series is now ready and validated by openstack folks that are primary consumers17:52
ihrachyshttps://patchwork.ozlabs.org/project/openvswitch/list/?series=17332817:53
ihrachysand I hope we can get it in the upcoming release, so reviews appreciated (thanks dceara for the prev round!)17:53
mmichelsonihrachys, noted!17:53
numansihrachys, thanks. I've downloaded the patches for review.17:53
ihrachysthat's it for me. :)17:53
dcearaihrachys, np, I'll try to have another look soon too17:54
* ihrachys bows17:54
_lore_can I go next? very quick17:55
_lore_this week I worked on adding some missing bits for IPv6 prefix delegation, low-hanging fruits actually17:55
_lore_moreover I reposted the fix for qos meters in ovn addressing comments from zhouhan17:56
mmichelsonThe qos meters fix got merged, right?17:56
_lore_I addressed comments from dceara in NetworkPolicy support for ovn-scale-test17:56
zhouhan_lore_: thanks, I will check17:56
_lore_mmichelson: yep17:56
_lore_zhouhan: not yet yours :(17:57
_lore_Just fixed issues in the current approach, I need to extend it to load conf from a json file17:57
zhouhan_lore_: oh, ok. I was talking about the qos meter fix. But I just saw you mentioned it is merged.17:58
_lore_maybe we can add it as a follow-up patch if the current version fixes the previous issues17:58
_lore_zhouhan: yes, I just added unit-test respect to the previous one17:58
zhouhangreat, thanks!17:58
_lore_zhouhan: what do you think about address your comments as follow-up series? do you prefer to add them in the current one?17:59
zhouhanfor ovn-scale-test? I am ok with a separate one.17:59
_lore_ack, so I will check if there are some leftover issues from dceara18:00
_lore_I hope I fixed them :)18:00
_lore_thx, that's all from my side18:00
mmichelsonOK. Anyone else?18:00
dceara_lore_, thanks, i'll have another look at your PR soon18:00
_lore_ack, thx18:01
mmichelsonAll right. Thanks everyone!18:03
mmichelson#endmeeting18:03
openstackMeeting ended Thu Apr 30 18:03:04 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:03
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-30-17.16.html18:03
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-30-17.16.txt18:03
openstackLog:            http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-30-17.16.log.html18:03
numansBye18:03
zhouhanthanks, bye18:04
*** donhw_ has quit IRC18:05
*** donhw has joined #openvswitch18:05
*** thaller_ has quit IRC18:06
*** acidfoo_ has quit IRC18:12
*** ebail has quit IRC18:23
*** fbl has quit IRC18:31
*** fbl has joined #openvswitch18:32
dcearazhouhan, I sent a v3 as we discussed during the meeting, this now deletes the old PB and creates a new one for "moved" ports: https://patchwork.ozlabs.org/project/openvswitch/list/?series=17391518:34
*** thaller_ has joined #openvswitch18:34
*** acidfoo_ has joined #openvswitch19:01
*** acidfoo_ has quit IRC19:09
*** acidfoo_ has joined #openvswitch19:12
*** acidfoo_ has quit IRC19:24
*** acidfoo_ has joined #openvswitch19:32
*** acidfoo_ has quit IRC19:41
*** slaweq has quit IRC19:57
*** atpa8a has quit IRC19:57
*** slaweq has joined #openvswitch19:58
*** acidfoo_ has joined #openvswitch20:06
*** acidfoo_ has quit IRC20:09
*** atpa8a has joined #openvswitch20:10
*** acidfoo_ has joined #openvswitch20:11
*** acidfoo_ has quit IRC20:25
*** slaweq has quit IRC20:29
*** acidfoo_ has joined #openvswitch20:32
*** acidfoo_ has quit IRC20:38
*** acidfoo_ has joined #openvswitch20:41
*** mmirecki has joined #openvswitch20:42
*** troulouliou_div2 has quit IRC20:46
*** mmirecki has quit IRC20:47
*** fbl has quit IRC20:50
*** oanson_ has quit IRC20:53
*** osmanlicilegi has quit IRC20:53
*** jon_d has quit IRC20:53
*** jon_d has joined #openvswitch20:53
*** oanson has joined #openvswitch20:53
*** osmanlicilegi has joined #openvswitch20:54
*** acidfu has joined #openvswitch21:06
*** acidfoo_ has quit IRC21:07
*** dcbw has quit IRC21:11
*** dceara has quit IRC21:36
*** timothy has quit IRC21:56
*** rcernin has joined #openvswitch22:04
*** donhw has quit IRC22:05
*** donhw has joined #openvswitch22:10
*** donhw has quit IRC22:11
*** bostondriver has quit IRC22:19
*** donhw has joined #openvswitch22:30
*** rcernin has quit IRC22:33
*** rcernin has joined #openvswitch22:34
*** duso has joined #openvswitch22:44
*** tbachman has quit IRC22:49
*** tbachman has joined #openvswitch22:52
*** yamamoto has joined #openvswitch23:45

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