| *** JamesBenson has joined #openvswitch | 00:17 | |
| *** JamesBenson has quit IRC | 00:22 | |
| *** JamesBenson has joined #openvswitch | 00:35 | |
| *** rcernin has quit IRC | 02:26 | |
| *** rcernin has joined #openvswitch | 02:50 | |
| *** |subz3r0| has quit IRC | 03:16 | |
| *** |subz3r0| has joined #openvswitch | 03:17 | |
| *** |subz3r0| has joined #openvswitch | 03:17 | |
| *** fdangelo has quit IRC | 03:47 | |
| *** mmichelson_ has joined #openvswitch | 04:53 | |
| *** mmichelson has quit IRC | 04:56 | |
| *** ClaesN has joined #openvswitch | 04:58 | |
| *** jgjl has quit IRC | 05:12 | |
| *** jaicaa has quit IRC | 06:00 | |
| *** jaicaa has joined #openvswitch | 06:02 | |
| *** eelco has joined #openvswitch | 06:13 | |
| *** slaweq has joined #openvswitch | 06:36 | |
| *** ralonsoh has joined #openvswitch | 06:53 | |
| *** rcernin has quit IRC | 07:35 | |
| *** rcernin_ has joined #openvswitch | 07:35 | |
| *** rcernin_ has quit IRC | 07:42 | |
| *** dmellado has quit IRC | 07:43 | |
| *** dmellado has joined #openvswitch | 07:47 | |
| *** rcernin_ has joined #openvswitch | 09:56 | |
| *** thaller_afk is now known as thaller | 10:15 | |
| *** rcernin_ has quit IRC | 10:25 | |
| *** rcernin_ has joined #openvswitch | 10:42 | |
| *** tbachman has quit IRC | 11:02 | |
| *** rcernin_ has quit IRC | 11:06 | |
| *** grive has quit IRC | 11:48 | |
| *** rcernin_ has joined #openvswitch | 12:00 | |
| *** grive has joined #openvswitch | 12:22 | |
| *** fdangelo has joined #openvswitch | 12:23 | |
| *** rcernin_ has quit IRC | 14:27 | |
| *** fdangelo has quit IRC | 14:28 | |
| *** eelco has quit IRC | 15:31 | |
| *** tbachman has joined #openvswitch | 15:32 | |
| *** donhw has quit IRC | 15:48 | |
| *** donhw has joined #openvswitch | 15:53 | |
| *** eelco has joined #openvswitch | 15:53 | |
| *** eelco has quit IRC | 16:03 | |
| *** donhw has quit IRC | 16:06 | |
| *** donhw has joined #openvswitch | 16:09 | |
| *** tryauuum has joined #openvswitch | 16:27 | |
| *** dceara has joined #openvswitch | 17:02 | |
| *** mmichelson_ has quit IRC | 17:19 | |
| *** mmichelson has joined #openvswitch | 17:20 | |
| mmichelson | Hi everyone | 17:20 |
|---|---|---|
| mmichelson | I'm a bit late to the party today. Had some IRC issues | 17:20 |
| imaximets | hi | 17:21 |
| mmichelson | #startmeeting ovn_community_development_discussion | 17:21 |
| openstack | Meeting started Thu Oct 15 17:21:40 2020 UTC and is due to finish in 60 minutes. The chair is mmichelson. Information about MeetBot at http://wiki.debian.org/MeetBot. | 17:21 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 17:21 |
| openstack | The meeting name has been set to 'ovn_community_development_discussion' | 17:21 |
| mmichelson | I can start things off, but there's not much for me to report. | 17:22 |
| dceara | Hi | 17:22 |
| mmichelson | Biggest time sink of mine this week has been reviews. I still have more in my queue to look at | 17:22 |
| mmichelson | I'm also writing up a post that will hopefully go on the OVN website that gives an overview of changes in 20.09 | 17:23 |
| mmichelson | Nothing too deep, but more than just what's in the NEWS file. | 17:23 |
| mmichelson | And that's all for me to report | 17:23 |
| imaximets | I have a very little update. | 17:26 |
| imaximets | This week I tried to run OVN tests with clang MemorySanitizer and found on issue with ic. | 17:27 |
| imaximets | It hashes struct v46_ip with uninitialized memory inside. | 17:27 |
| mmichelson | I think I saw a patch from Han with a fix for that | 17:27 |
| imaximets | Yep, I sent a bug report to maillist and zhouhan sent a patch for this. | 17:28 |
| imaximets | That's it from my side for OVN this week. | 17:28 |
| _lore_ | hi all | 17:29 |
| _lore_ | can I go next? | 17:29 |
| _lore_ | very quick | 17:29 |
| mmichelson | go for it | 17:29 |
| _lore_ | this week I posted: | 17:29 |
| _lore_ | 1- iPXE support for ovn | 17:29 |
| _lore_ | tested by OSP folks | 17:29 |
| _lore_ | 2- Renew/Rebind support for IPv6 PD | 17:30 |
| _lore_ | 3- now I stared working on a BFD implementation in OVN to enable link failure detection for emcp routing | 17:30 |
| _lore_ | that's all from my side | 17:30 |
| dceara | I have a quick update too if I may go next. | 17:31 |
| mmichelson | yeah I like these quick updates :) | 17:32 |
| dceara | Thanks to zhouhan and Mark Gray for reviews on the ofctrl predictable conflict resolution patches! I sent a v6 and today a v7: http://patchwork.ozlabs.org/project/ovn/patch/1602753150-31918-1-git-send-email-dceara@redhat.com/ | 17:32 |
| zhouhan | dceara: thank you! I am reviewing now. | 17:33 |
| dceara | zhouhan: Sorry, I think we had some confusion at some point, but I hope it's all sorted out now. | 17:33 |
| dceara | Except for that I did some reviews this week and I wanted to bring up to attention numans' question on patch http://patchwork.ozlabs.org/project/ovn/patch/20201014091525.3391981-1-numans@ovn.org/ | 17:34 |
| dceara | That's it from my side, thanks! | 17:35 |
| numans | dceara, thanks for bringing it up. | 17:35 |
| dceara | numans: I didn't know if you'd join :) | 17:35 |
| numans | I can go real quick too. | 17:35 |
| * numans just joined | 17:35 | |
| numans | It would be great if mmichelson zhouhan and others can provide their comments to the link dceara shared. | 17:36 |
| numans | based on that, I'll spin up next version of reject patch series. | 17:36 |
| zhouhan | numans: is the question about whether we should swap IPs and ETHs together? | 17:36 |
| numans | other than that I worked for reducing the hairpin flows. | 17:36 |
| numans | zhouhan, yes | 17:37 |
| numans | zhouhan, or versus whether to go another ovn field ip.src <-> ip.dst | 17:37 |
| numans | which results in another packet-in. | 17:37 |
| mmichelson | numans, I'll have a look after this meeting. | 17:38 |
| numans | mmichelson, thanks. | 17:38 |
| zhouhan | another packet-in sounds bad :( | 17:38 |
| mmichelson | yeah the issue is that the ip.src <-> ip.dst results in a packet in, and then the reject {} does a second packet in. | 17:38 |
| numans | zhouhan, which means we need to swap eth src and eth dst and ip src and ip dst | 17:38 |
| numans | mmichelson, zhouhan FYI we do the same for icmp4.frag_mtu too | 17:39 |
| zhouhan | numans: you mean two packet-ins for frag_mtu? | 17:40 |
| numans | I'm fine to implicitly swap these fields in reject pkt-in, but normally in OVN, we specify the inner actions to any OVN action | 17:40 |
| numans | zhouhan, the usage of it is - icmp4 { eth.src <-> eth.dst ,...., icmp4.frag_mtu = 1500 ...} | 17:40 |
| numans | zhouhan, this results in packetin for icmp4 action | 17:40 |
| dceara | numans: In my opinion "reject" is explicit enough to imply that it will generate a packet with addresses swapped. | 17:41 |
| numans | and then again to set icmp4.frag_mtu (with pause flag set for this one) | 17:41 |
| numans | zhouhan, that was suggested by Ben when I worked on it while back | 17:41 |
| numans | I'm fine with both the approaches. I'll respin the patches based on the comments :) | 17:42 |
| numans | That's it from my side. | 17:42 |
| zhouhan | numans: thanks for the information. I will check it offline. | 17:42 |
| numans | zhouhan, welcome | 17:42 |
| zhouhan | May I go next? | 17:43 |
| zhouhan | I want to briefly discuss the _is_new problem, if Mark Gray is here? | 17:43 |
| mmichelson | Hm, if he's in here he's using a nick I'm not familiar with | 17:44 |
| zhouhan | dceara: thanks for sharing the code that relies on _is_new() for a row inserted in current transaction | 17:44 |
| zhouhan | dceara: I think there is some confusion for those interfaces _is_new(), _is_modified() ... | 17:45 |
| dceara | zhouhan: I agree. But we do use them for locally inserted records too. We could make them work in that case too. Which is what Mark's patch does, IIUC. | 17:45 |
| zhouhan | dceara: for my understanding these are for tracked changes only. But it seems it is used for checking a row added in current transaction (not committed yet) | 17:45 |
| zhouhan | dceara: yes, it is definitely possible to change it. | 17:46 |
| dceara | zhouhan: I think the patch will not break the tracked changes functionality so I think it would be good to have. | 17:47 |
| zhouhan | dceara: but if we change, we should make sure it is changed for modify and delete, not only just for insert. (I am not 100% sure if there is any potential problems, need to study more on this) | 17:47 |
| dceara | But I'm not extremely familiar with that code so I might be wrong. | 17:47 |
| dceara | zhouhan: OK, I see. | 17:48 |
| zhouhan | dceara: let's check more and discuss with Mark Gray. | 17:48 |
| dceara | zhouhan: Ack. | 17:48 |
| zhouhan | I don't have any other updates except the bug fix and some reviews | 17:49 |
| zhouhan | thanks | 17:49 |
| mmichelson | Okie-dokie. Anyone else have updates? | 17:50 |
| *** donhw has quit IRC | 17:51 | |
| *** donhw_ has joined #openvswitch | 17:51 | |
| mmichelson | I guess that's it then. Thanks everyone! | 17:52 |
| mmichelson | #endmeeting | 17:52 |
| openstack | Meeting ended Thu Oct 15 17:52:28 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:52 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-10-15-17.21.html | 17:52 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-10-15-17.21.txt | 17:52 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-10-15-17.21.log.html | 17:52 |
| dceara | Bye! | 17:53 |
| *** donhw_ has quit IRC | 17:58 | |
| *** donhw has joined #openvswitch | 18:03 | |
| *** dceara has quit IRC | 18:05 | |
| *** mdgray has joined #openvswitch | 18:10 | |
| *** oanson has quit IRC | 18:18 | |
| *** oanson has joined #openvswitch | 18:19 | |
| *** ralonsoh has quit IRC | 18:22 | |
| *** donhw has quit IRC | 18:59 | |
| *** donhw has joined #openvswitch | 18:59 | |
| *** gregwork has quit IRC | 19:26 | |
| *** imaximets has quit IRC | 19:28 | |
| *** imaximets has joined #openvswitch | 19:28 | |
| *** fdangelo has joined #openvswitch | 19:40 | |
| *** oanson has quit IRC | 20:15 | |
| *** oanson has joined #openvswitch | 20:15 | |
| *** mdgray has quit IRC | 20:17 | |
| *** jobewan has quit IRC | 21:14 | |
| *** dcbw has quit IRC | 21:15 | |
| *** slaweq has quit IRC | 21:31 | |
| *** rcernin_ has joined #openvswitch | 22:03 | |
| *** rcernin_ has quit IRC | 22:19 | |
| *** slaweq has joined #openvswitch | 22:22 | |
| *** slaweq has quit IRC | 22:38 | |
| *** donhw has quit IRC | 23:43 | |
| *** zhouhan_ has joined #openvswitch | 23:44 | |
| *** donhw has joined #openvswitch | 23:46 | |
| *** zhouhan has quit IRC | 23:47 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!