*** acidfoo_ has quit IRC | 01:06 | |
*** dcbw has quit IRC | 01:18 | |
*** duso has joined #openvswitch | 01:39 | |
duso | morning | 01:40 |
---|---|---|
*** dholler has quit IRC | 02:23 | |
*** dholler has joined #openvswitch | 02:36 | |
*** JamesBenson has quit IRC | 02:37 | |
*** yamamoto has quit IRC | 02:54 | |
*** yamamoto has joined #openvswitch | 02:54 | |
*** dqx has quit IRC | 03:05 | |
*** JamesBenson has joined #openvswitch | 03:05 | |
*** dqx has joined #openvswitch | 03:06 | |
*** yamamoto has quit IRC | 03:18 | |
*** _lore_ has quit IRC | 03:23 | |
*** zhouhan_ has quit IRC | 03:23 | |
*** FH_thecat has quit IRC | 03:26 | |
*** yamamoto has joined #openvswitch | 03:31 | |
*** zhouhan has joined #openvswitch | 03:33 | |
*** psahoo has joined #openvswitch | 04:11 | |
*** mb__ has joined #openvswitch | 04:14 | |
*** bern has quit IRC | 04:16 | |
*** JamesBenson has quit IRC | 04:43 | |
*** _lore_ has joined #openvswitch | 04:55 | |
*** mb__ has quit IRC | 06:16 | |
*** yogananth has joined #openvswitch | 06:36 | |
*** yogananth_ has joined #openvswitch | 06:55 | |
*** yogananth has quit IRC | 06:58 | |
*** slaweq has joined #openvswitch | 06:58 | |
*** mbarroso has joined #openvswitch | 07:01 | |
*** eelco has joined #openvswitch | 07:07 | |
*** yamamoto has quit IRC | 07:19 | |
*** yamamoto has joined #openvswitch | 07:23 | |
*** yamamoto has quit IRC | 07:36 | |
*** yamamoto has joined #openvswitch | 07:50 | |
*** mmirecki has joined #openvswitch | 08:23 | |
*** dceara has joined #openvswitch | 08:37 | |
*** duso has quit IRC | 08:52 | |
*** theodotos[m] has quit IRC | 08:56 | |
*** yamamoto has quit IRC | 09:02 | |
*** theodotos[m] has joined #openvswitch | 09:11 | |
*** yamamoto has joined #openvswitch | 09:16 | |
*** yamamoto has quit IRC | 09:19 | |
*** yamamoto has joined #openvswitch | 09:23 | |
*** timothy has joined #openvswitch | 09:33 | |
*** yamamoto has quit IRC | 09:37 | |
*** yamamoto has joined #openvswitch | 10:05 | |
*** rcernin has quit IRC | 11:06 | |
*** FH_thecat has joined #openvswitch | 12:00 | |
*** FH_thecat has quit IRC | 12:06 | |
*** FH_thecat has joined #openvswitch | 12:06 | |
*** ebail has joined #openvswitch | 12:11 | |
*** _lore_ has quit IRC | 12:23 | |
*** yamamoto has quit IRC | 12:23 | |
*** rcernin has joined #openvswitch | 12:31 | |
*** slaweq has quit IRC | 12:35 | |
*** aconole has joined #openvswitch | 12:35 | |
*** slaweq has joined #openvswitch | 12:37 | |
*** slaweq_ has joined #openvswitch | 12:38 | |
*** slaweq has quit IRC | 12:41 | |
*** yamamoto has joined #openvswitch | 12:49 | |
*** yamamoto has quit IRC | 12:50 | |
*** yamamoto has joined #openvswitch | 12:54 | |
*** acidfoo_ has joined #openvswitch | 12:54 | |
*** yamamoto has joined #openvswitch | 12:54 | |
*** bostondriver has joined #openvswitch | 12:57 | |
*** acidfoo_ has quit IRC | 12:58 | |
*** yamamoto has quit IRC | 13:02 | |
*** FH_thecat has quit IRC | 13:06 | |
*** JamesBenson has joined #openvswitch | 13:10 | |
*** FH_thecat has joined #openvswitch | 13:11 | |
*** _lore_ has joined #openvswitch | 13:11 | |
*** mmichelson has quit IRC | 13:19 | |
*** dcbw has joined #openvswitch | 13:24 | |
*** mmichelson has joined #openvswitch | 13:25 | |
*** FH_thecat has quit IRC | 13:26 | |
*** rcernin has quit IRC | 13:29 | |
*** acidfoo_ has joined #openvswitch | 13:37 | |
*** troulouliou_div2 has joined #openvswitch | 14:10 | |
*** emptynine has joined #openvswitch | 14:18 | |
*** mmirecki has quit IRC | 14:35 | |
*** armax has joined #openvswitch | 14:44 | |
*** slaweq_ is now known as slaweq | 15:03 | |
*** mmirecki has joined #openvswitch | 15:03 | |
*** avishnoi has quit IRC | 15:11 | |
*** avishnoi has joined #openvswitch | 15:13 | |
*** mmirecki has quit IRC | 15:19 | |
*** eelco has quit IRC | 15:30 | |
*** balkamos has quit IRC | 15:44 | |
*** dholler has quit IRC | 15:46 | |
*** balkamos has joined #openvswitch | 15:52 | |
*** zhouhan_ has joined #openvswitch | 15:56 | |
*** zhouhan has quit IRC | 16:00 | |
*** balkamos has quit IRC | 16:15 | |
*** aconole has quit IRC | 16:15 | |
*** tbachman has quit IRC | 16:23 | |
*** tbachman has joined #openvswitch | 16:24 | |
*** balkamos has joined #openvswitch | 16:30 | |
*** balkamos has quit IRC | 16:30 | |
*** balkamos has joined #openvswitch | 16:30 | |
*** balkamos has quit IRC | 16:41 | |
*** troulouliou_div2 has quit IRC | 16:44 | |
*** balkamos has joined #openvswitch | 16:56 | |
*** troulouliou_div2 has joined #openvswitch | 16:56 | |
*** balkamos has quit IRC | 17:05 | |
*** psahoo has quit IRC | 17:12 | |
*** balkamos has joined #openvswitch | 17:13 | |
mmichelson | Hey everyone, I'm about to start the weekly OVN meeting | 17:16 |
mmichelson | #startmeeting ovn-community-development-discussion | 17:16 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 17:16 |
openstack | The meeting name has been set to 'ovn_community_development_discussion' | 17:17 |
numans | Hello | 17:17 |
*** zhouhan_ has quit IRC | 17:17 | |
*** zhouhan has joined #openvswitch | 17:18 | |
zhouhan | hi | 17:18 |
mmichelson | Let's get this thing started! | 17:18 |
mmichelson | First 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 |
mmichelson | Tomorrow, I'll send out an e-mail asking people which patches they would like to get merged in for the release of 20.06 | 17:19 |
_lore_ | hi all | 17:19 |
mmichelson | Aside from that, I don't think I have anything to share right now. So whoever would like to go next, feel free. | 17:20 |
numans | I can go real quick | 17:21 |
numans | I did some code reviews this week. | 17:21 |
numans | And pushed out few patches. | 17:21 |
numans | 1. I just rebased the I-P patches as they had some merge conflict (and hoping this would be pop up for review :)) | 17:22 |
zhouhan | numans: so sorry for the delay of review. I was on some other issues. | 17:23 |
numans | 2. 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 |
numans | zhouhan, no worries. I understand :) | 17:23 |
numans | I 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 |
numans | That's it from me. | 17:24 |
*** mbarroso has quit IRC | 17:24 | |
numans | zhouhan, I've addressed the load balancer selection method here - https://patchwork.ozlabs.org/project/openvswitch/patch/20200430172028.1978594-1-numans@ovn.org/ | 17:24 |
numans | request to just go through the commit message. That should give an idea :) | 17:25 |
numans | and let me know if any issues with the approach. | 17:25 |
numans | Request everyone to take a look at the patches. | 17:25 |
numans | That's it from me. | 17:25 |
zhouhan | numans: 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 |
numans | zhouhan, I saw that. The default will be dp_hash | 17:26 |
numans | and it is left for the CMS to decide. | 17:26 |
numans | zhouhan, in the case of octavia lb algorithms there are 2 which we can support - IP_SOURCE and IP_SOURCE_PORT. | 17:26 |
numans | and its possible with this. | 17:27 |
zhouhan | ok, 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 |
numans | zhouhan, agree. | 17:28 |
zhouhan | numans: and I am not sure when should we suggest user to use "hash" instead of "dp_hash". | 17:28 |
numans | zhouhan, it might take a while for the kernel patch to be backported to the distros. | 17:28 |
numans | in the meanwhile cms can use it. | 17:29 |
mmichelson | People should be using bleeding edge at all times | 17:29 |
numans | zhouhan, I'd suggest user to use dp_hash. | 17:29 |
numans | mmichelson, :) | 17:29 |
zhouhan | numans: if they use "hash", performance problems may be reported ... | 17:29 |
numans | zhouhan, agree. So its CMS's discretion. | 17:30 |
numans | zhouhan, I think we should document tht properly. | 17:30 |
zhouhan | And I am also confused about the kernel patch backporting process. Is there anyone taking care of the backportings? What's the practices? | 17:31 |
numans | zhouhan, I mean it takes time for those patches to be in RHEL kernel for example | 17:31 |
numans | I'm not sure about ubuntu and other distros. | 17:31 |
zhouhan | numans: Oh, ok. What I meant is backporting to OVS repo's datapath. | 17:31 |
mmichelson | Yeah, Debian, RHEL, and CentOS are known for being late to the party | 17:31 |
numans | zhouhan, Ok. Most of the distros will have openvswitch kernel module in them | 17:32 |
numans | and RHEL uses the kernel openvswitch datapath | 17:32 |
numans | Ok. I'm done. If someone wants to go next. | 17:33 |
zhouhan | numans: 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 |
numans | zhouhan, Ok. | 17:34 |
dceara | hi all! sorry for joining late | 17:34 |
zhouhan | at least blp should have the answer, but he seems to be absent for several weeks | 17:34 |
numans | zhouhan, from what I know, normally when the patch is merged in kernel, either the author will send it to ovs-dev ML | 17:34 |
numans | or someone from community posts the patch to ovs ML> | 17:35 |
*** yogananth_ has quit IRC | 17:35 | |
zhouhan | numans: ok, this time I pinged the author Tonghao Zhang, and he said he would help the backporting. | 17:35 |
numans | zhouhan, ack. I've one patch experience. I can post it if the author doesn't post :) | 17:36 |
zhouhan | However, 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 |
zhouhan | I can keep on going with my report. | 17:37 |
numans | zhouhan, that's strange bcoz I used the fake-multi-node too and i was able to see the problem | 17:37 |
numans | :) | 17:37 |
zhouhan | numans: using master? | 17:37 |
numans | zhouhan, yes. ovs master | 17:37 |
numans | and fedora 31 | 17:38 |
zhouhan | numans: hmm... that's really strange | 17:38 |
zhouhan | Apart from the dp_hash invesitigation, I was chasing some other problems. | 17:38 |
zhouhan | One of them is that Chassis-redirect-port doesn't work for ingress. | 17:39 |
numans | zhouhan, chassis-redirect-port .. its the distributed gw port with cr- right ? | 17:40 |
zhouhan | When 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 |
zhouhan | numans: yes (cr-) | 17:42 |
numans | zhouhan, ok. | 17:43 |
zhouhan | I will continue work on it. | 17:43 |
zhouhan | that's it form me. | 17:43 |
dceara | I can go next if that's ok | 17:43 |
dceara | zhouhan, 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=173823 | 17:44 |
dceara | zhouhan, it is something that we might see often with ovn-kubernetes when pods are "moved" from one node to another | 17:45 |
dceara | zhouhan, because every node has its own logical switch | 17:45 |
dceara | zhouhan, 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 |
zhouhan | dceara: I see. I think it is better if we treat it as delete and create. | 17:46 |
numans | I agree with zhouhan. | 17:47 |
numans | this would ensure correctness. | 17:47 |
dceara | zhouhan, ok, I need to think about other cases too (e.g., multicast_groups use tunnel_keys too) | 17:47 |
dceara | but that should be fine. I'll try to post a v3 as soon as possible | 17:48 |
zhouhan | From 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 |
dceara | zhouhan, 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 |
zhouhan | dceara: yes, exactly. I think recreating a port with same name should be treated as deletion and recreation. | 17:50 |
dceara | zhouhan, ok, i'll do it like that then | 17:50 |
dceara | apart 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 |
dceara | that's it from me | 17:51 |
zhouhan | dceara: thanks a lot. Looking forward to it | 17:51 |
dceara | zhouhan, np | 17:51 |
ihrachys | ok, maybe me? i'll be quick | 17:52 |
ihrachys | so the multiple localnet ports series is now ready and validated by openstack folks that are primary consumers | 17:52 |
ihrachys | https://patchwork.ozlabs.org/project/openvswitch/list/?series=173328 | 17:53 |
ihrachys | and I hope we can get it in the upcoming release, so reviews appreciated (thanks dceara for the prev round!) | 17:53 |
mmichelson | ihrachys, noted! | 17:53 |
numans | ihrachys, thanks. I've downloaded the patches for review. | 17:53 |
ihrachys | that's it for me. :) | 17:53 |
dceara | ihrachys, np, I'll try to have another look soon too | 17:54 |
* ihrachys bows | 17:54 | |
_lore_ | can I go next? very quick | 17:55 |
_lore_ | this week I worked on adding some missing bits for IPv6 prefix delegation, low-hanging fruits actually | 17:55 |
_lore_ | moreover I reposted the fix for qos meters in ovn addressing comments from zhouhan | 17:56 |
mmichelson | The qos meters fix got merged, right? | 17:56 |
_lore_ | I addressed comments from dceara in NetworkPolicy support for ovn-scale-test | 17:56 |
zhouhan | _lore_: thanks, I will check | 17:56 |
_lore_ | mmichelson: yep | 17: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 file | 17: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 issues | 17:58 |
_lore_ | zhouhan: yes, I just added unit-test respect to the previous one | 17:58 |
zhouhan | great, 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 |
zhouhan | for 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 dceara | 18:00 |
_lore_ | I hope I fixed them :) | 18:00 |
_lore_ | thx, that's all from my side | 18:00 |
mmichelson | OK. Anyone else? | 18:00 |
dceara | _lore_, thanks, i'll have another look at your PR soon | 18:00 |
_lore_ | ack, thx | 18:01 |
mmichelson | All right. Thanks everyone! | 18:03 |
mmichelson | #endmeeting | 18:03 |
openstack | Meeting ended Thu Apr 30 18:03:04 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:03 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-30-17.16.html | 18:03 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-30-17.16.txt | 18:03 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-04-30-17.16.log.html | 18:03 |
numans | Bye | 18:03 |
zhouhan | thanks, bye | 18:04 |
*** donhw_ has quit IRC | 18:05 | |
*** donhw has joined #openvswitch | 18:05 | |
*** thaller_ has quit IRC | 18:06 | |
*** acidfoo_ has quit IRC | 18:12 | |
*** ebail has quit IRC | 18:23 | |
*** fbl has quit IRC | 18:31 | |
*** fbl has joined #openvswitch | 18:32 | |
dceara | zhouhan, 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=173915 | 18:34 |
*** thaller_ has joined #openvswitch | 18:34 | |
*** acidfoo_ has joined #openvswitch | 19:01 | |
*** acidfoo_ has quit IRC | 19:09 | |
*** acidfoo_ has joined #openvswitch | 19:12 | |
*** acidfoo_ has quit IRC | 19:24 | |
*** acidfoo_ has joined #openvswitch | 19:32 | |
*** acidfoo_ has quit IRC | 19:41 | |
*** slaweq has quit IRC | 19:57 | |
*** atpa8a has quit IRC | 19:57 | |
*** slaweq has joined #openvswitch | 19:58 | |
*** acidfoo_ has joined #openvswitch | 20:06 | |
*** acidfoo_ has quit IRC | 20:09 | |
*** atpa8a has joined #openvswitch | 20:10 | |
*** acidfoo_ has joined #openvswitch | 20:11 | |
*** acidfoo_ has quit IRC | 20:25 | |
*** slaweq has quit IRC | 20:29 | |
*** acidfoo_ has joined #openvswitch | 20:32 | |
*** acidfoo_ has quit IRC | 20:38 | |
*** acidfoo_ has joined #openvswitch | 20:41 | |
*** mmirecki has joined #openvswitch | 20:42 | |
*** troulouliou_div2 has quit IRC | 20:46 | |
*** mmirecki has quit IRC | 20:47 | |
*** fbl has quit IRC | 20:50 | |
*** oanson_ has quit IRC | 20:53 | |
*** osmanlicilegi has quit IRC | 20:53 | |
*** jon_d has quit IRC | 20:53 | |
*** jon_d has joined #openvswitch | 20:53 | |
*** oanson has joined #openvswitch | 20:53 | |
*** osmanlicilegi has joined #openvswitch | 20:54 | |
*** acidfu has joined #openvswitch | 21:06 | |
*** acidfoo_ has quit IRC | 21:07 | |
*** dcbw has quit IRC | 21:11 | |
*** dceara has quit IRC | 21:36 | |
*** timothy has quit IRC | 21:56 | |
*** rcernin has joined #openvswitch | 22:04 | |
*** donhw has quit IRC | 22:05 | |
*** donhw has joined #openvswitch | 22:10 | |
*** donhw has quit IRC | 22:11 | |
*** bostondriver has quit IRC | 22:19 | |
*** donhw has joined #openvswitch | 22:30 | |
*** rcernin has quit IRC | 22:33 | |
*** rcernin has joined #openvswitch | 22:34 | |
*** duso has joined #openvswitch | 22:44 | |
*** tbachman has quit IRC | 22:49 | |
*** tbachman has joined #openvswitch | 22:52 | |
*** yamamoto has joined #openvswitch | 23:45 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!