| *** dcbw has quit IRC | 00:31 | |
| *** bdonnahue1 has quit IRC | 02:13 | |
| *** bdonnahue1 has joined #openvswitch | 02:14 | |
| *** rcernin has quit IRC | 02:35 | |
| *** rcernin has joined #openvswitch | 02:42 | |
| *** dholler has quit IRC | 02:49 | |
| *** dholler has joined #openvswitch | 03:03 | |
| *** rcernin has quit IRC | 03:59 | |
| *** rcernin has joined #openvswitch | 04:00 | |
| *** bdonnahue1 has quit IRC | 04:13 | |
| *** bdonnahue2 has joined #openvswitch | 04:22 | |
| *** bdonnahue3 has joined #openvswitch | 04:25 | |
| *** bdonnahue2 has quit IRC | 04:28 | |
| *** terdei has quit IRC | 04:30 | |
| *** terdei has joined #openvswitch | 04:33 | |
| *** slaweq has joined #openvswitch | 05:55 | |
| *** ralonsoh has joined #openvswitch | 06:25 | |
| *** rcernin has quit IRC | 07:04 | |
| *** mdgray has joined #openvswitch | 07:14 | |
| *** eelco has joined #openvswitch | 07:34 | |
| *** anilvenkata has joined #openvswitch | 07:47 | |
| *** jaicaa has quit IRC | 08:02 | |
| *** jaicaa has joined #openvswitch | 08:04 | |
| *** jaicaa has quit IRC | 08:11 | |
| *** jaicaa has joined #openvswitch | 08:14 | |
| *** amorenoz_ has joined #openvswitch | 08:17 | |
| *** amorenoz has quit IRC | 08:18 | |
| *** links has joined #openvswitch | 08:43 | |
| *** elvira has quit IRC | 08:43 | |
| *** istokes has joined #openvswitch | 08:53 | |
| *** elvira has joined #openvswitch | 09:00 | |
| *** anilvenkata has quit IRC | 09:35 | |
| *** anilvenkata has joined #openvswitch | 09:36 | |
| *** dalvarez has quit IRC | 09:36 | |
| *** psahoo has joined #openvswitch | 09:54 | |
| *** tbachman has quit IRC | 10:43 | |
| *** tbachman has joined #openvswitch | 10:43 | |
| *** tbachman has quit IRC | 11:07 | |
| *** bostondriver has joined #openvswitch | 12:34 | |
| *** tbachman has joined #openvswitch | 12:40 | |
| *** jraju__ has joined #openvswitch | 13:13 | |
| *** links has quit IRC | 13:13 | |
| *** aconole has joined #openvswitch | 13:22 | |
| *** amorenoz__ has joined #openvswitch | 13:32 | |
| *** amorenoz_ has quit IRC | 13:34 | |
| *** dcbw has joined #openvswitch | 13:57 | |
| *** istokes has quit IRC | 14:22 | |
| *** istokes has joined #openvswitch | 14:28 | |
| *** links has joined #openvswitch | 14:44 | |
| *** jraju__ has quit IRC | 14:44 | |
| *** eelco has quit IRC | 15:03 | |
| *** psahoo has quit IRC | 15:06 | |
| *** psahoo has joined #openvswitch | 15:19 | |
| *** dalvarez has joined #openvswitch | 16:00 | |
| *** thaller has quit IRC | 16:29 | |
| *** dalvarez has quit IRC | 16:31 | |
| *** psahoo has quit IRC | 16:48 | |
| *** avishnoi has quit IRC | 16:48 | |
| *** bdonnahue3 has quit IRC | 16:58 | |
| *** psahoo has joined #openvswitch | 17:00 | |
| *** avishnoi has joined #openvswitch | 17:03 | |
| mmichelson | Hi everyone, I'll get the meeting started in just a minute | 17:14 |
|---|---|---|
| mmichelson | #startmeeting ovn_community_development_meeting | 17:15 |
| openstack | Meeting started Thu Apr 29 17:15:27 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 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 17:15 |
| openstack | The meeting name has been set to 'ovn_community_development_meeting' | 17:15 |
| *** zhouhan has joined #openvswitch | 17:16 | |
| mmichelson | I can get things started. I have my latest version of the ARP/FIP fixes up for review. I believe numans had comments on it, and I'll need to check that out. Looking into an interesting issue where it appears that expression parsing in certain cases can result in repeated conjunction actions | 17:16 |
| mmichelson | Other than that, it's been code reviews from me. | 17:17 |
| mmichelson | Reminder: soft freeze is in 8 days. So if you have features you're working on for 21.06.0, please get those submitted before then. | 17:17 |
| mmichelson | And that's all from me. | 17:17 |
| _lore_ | hi all | 17:17 |
| zhouhan | "expression parsing in certain cases can result in repeated conjunction actions" -> interesting... | 17:17 |
| zhouhan | mmichelson: could you share if you have the test case to reproduce? | 17:18 |
| mmichelson | zhouhan, sure, just a sec. I'll link a bug report, and a script | 17:19 |
| zhouhan | I can go next | 17:19 |
| mmichelson | zhouhan, https://bugzilla.redhat.com/show_bug.cgi?id=1955161 and https://pastebin.com/fjxEN1JU | 17:20 |
| openstack | bugzilla.redhat.com bug 1955161 in OVN "OVN Expression parser generates repeated conjunctions" [Unspecified,New] - Assigned to ovnteam | 17:20 |
| zhouhan | Firstly, thanks Mark Gray's review to my "flow explosion" series. I applied the first 4 patches, but left the last one since he request another look from someone else | 17:20 |
| mmichelson | zhouhan, the pastebin is a script I run in the sandbox | 17:20 |
| zhouhan | thanks mmichelson | 17:20 |
| zhouhan | numans please also take a look if you can: #link https://patchwork.ozlabs.org/project/ovn/list/?series=241037 | 17:21 |
| numans | zhouhan, ack. | 17:21 |
| zhouhan | I also reviewed some patches including numans's revision of split I-P logical/physical. I will still need to look at the ct.inv patch | 17:22 |
| zhouhan | I need more time for the ddlog part :) | 17:22 |
| mmichelson | zhouhan, I think we all have the problem right now :) | 17:23 |
| zhouhan | Then I submitted some minor fixes. Thanks folks for reviewing. Most of them were merged, but there was one I thought was small but caused CI broken - the warning log change. Sorry for that. | 17:23 |
| zhouhan | The warning log change triggered a discussion regarding how should we monitor DPG changes. imaximets, dumitru and I are still discussing on it. | 17:24 |
| zhouhan | Also, checking with numans on the assertion problem in incremental flow installation. | 17:25 |
| zhouhan | That's all I have to report | 17:25 |
| mmichelson | Yeah I was curious about that assertion problem too | 17:25 |
| numans | zhouhan, I just replied with my latest findings | 17:25 |
| numans | rather a while back | 17:25 |
| zhouhan | thanks numans I will check | 17:25 |
| imaximets | zhouhan, we probably need to apply revert for the sake of CI while we're discussing on a list. To unblock ovsrobot and other patches. | 17:26 |
| *** ralonsoh has quit IRC | 17:26 | |
| mmichelson | zhouhan, I made an error in the first pastebin I put up earlier. Here's a corrected version: https://pastebin.com/YvU09Yuw | 17:26 |
| zhouhan | imaximets I have reverted it (before sending the reply - also mentioned in the reply :) | 17:27 |
| imaximets | zhouhan, ack. thanks. :) I didn't read t yet. | 17:27 |
| zhouhan | imaximets in fact it's in the one you already replied (the last section :D) | 17:28 |
| zhouhan | mmichelson: got it, thx | 17:28 |
| mmichelson | Anyone else want to share today? | 17:29 |
| imaximets | zhouhan, oh, I missed the second part of that email. Oops. :) | 17:29 |
| *** psahoo has quit IRC | 17:29 | |
| imaximets | mmichelson, fyi, we discussed last week that the name of the meeting suddenly changed few months ago from 'ovn_community_development_discussion' to 'ovn_community_development_meeting', so the link for logs from the channel topic doesn't lead to new logs. | 17:29 |
| imaximets | I have a very small update | 17:30 |
| mmichelson | imaximets, whoops. That probably coincides with when I switched laptops. | 17:30 |
| imaximets | It was a busy week, but I finished all the code changes for ovsdb 2-tier deployment patch set. Adding some documentation now. Not sure if I will send it today, but it should be ready tomorrow. | 17:31 |
| zhouhan | imaximets: awesome | 17:32 |
| imaximets | I also sent a small patch today to fix permissions for system test logs in GHA. | 17:32 |
| imaximets | #link https://patchwork.ozlabs.org/project/ovn/patch/20210429121143.2430530-1-i.maximets@ovn.org/ | 17:32 |
| imaximets | currently we have no logs for system tests in artifacts. | 17:33 |
| imaximets | that's it from me. | 17:33 |
| mmichelson | Thanks imaximets | 17:34 |
| mmichelson | Anyone else? | 17:34 |
| _lore_ | can I go next? | 17:34 |
| mmichelson | go for it | 17:34 |
| _lore_ | quite fast..this week I worked on the CoPP | 17:34 |
| _lore_ | as discussed with dumitru, I removed the per-port metering (we have just per-dp metering now) | 17:35 |
| _lore_ | I addressed some comments on rfc and I added system-tests and unit-tests | 17:35 |
| _lore_ | I added some leftover metering for new actions (e.g. bfd or acl/lb reject) | 17:35 |
| _lore_ | I posted v1 just few min ago | 17:36 |
| _lore_ | #link http://patchwork.ozlabs.org/project/ovn/list/?series=241400 | 17:37 |
| mmichelson | Thanks _lore_ | 17:37 |
| imaximets | _lore_, out of curiosity, are you configuring burst size for meters? | 17:37 |
| _lore_ | moreover I posted a trivial patch to add some debug logs for bfd in ovn-controller | 17:37 |
| _lore_ | #link http://patchwork.ozlabs.org/project/ovn/patch/3a9ca2d94478a3b2db51d719233c9a28cafcf75b.1619268349.git.lorenzo.bianconi@redhat.com/ | 17:37 |
| _lore_ | imaximets: it is up to the CMS to configure the meter | 17:38 |
| _lore_ | do you mean in system-tests? | 17:38 |
| imaximets | _lore_, ack. no, in general. | 17:38 |
| _lore_ | in general the workflow is: | 17:39 |
| _lore_ | - CMS create a given meter (rate + w or w/o burst) and associate it to given protocol per dp | 17:39 |
| _lore_ | like: | 17:39 |
| _lore_ | I create a meter to limit arps on this logical router | 17:40 |
| _lore_ | during testing I do not set the burst for the meter | 17:40 |
| _lore_ | since it is easier for testing | 17:40 |
| imaximets | _lore_, ok. Thanks for explanation. | 17:41 |
| _lore_ | that's all from my sice | 17:41 |
| _lore_ | imaximets: yw :) | 17:41 |
| _lore_ | btw to mark g. for the reviews | 17:41 |
| _lore_ | *thx | 17:41 |
| mmichelson | Anyone else? | 17:42 |
| numans | I can go real quick | 17:42 |
| numans | I did few reviews. | 17:42 |
| numans | But mostly looking into the assert issue | 17:42 |
| numans | Looks like I'm able to reproduce the assert issue with my split flow table patch. | 17:43 |
| numans | I'll continue debugging on it. | 17:43 |
| numans | That's it from me. | 17:43 |
| mmichelson | Thanks numans | 17:44 |
| mmichelson | All right, thanks everyone. Have a good day! | 17:45 |
| mmichelson | #endmeeting | 17:45 |
| openstack | Meeting ended Thu Apr 29 17:45:16 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:45 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-04-29-17.15.html | 17:45 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-04-29-17.15.txt | 17:45 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ovn_community_development_meeting/2021/ovn_community_development_meeting.2021-04-29-17.15.log.html | 17:45 |
| _lore_ | bye all | 17:46 |
| zhouhan | Thanks, bye all | 17:46 |
| *** links has quit IRC | 18:24 | |
| *** tbachman has quit IRC | 18:28 | |
| *** Oriz has joined #openvswitch | 18:48 | |
| *** mdgray has quit IRC | 18:54 | |
| *** dalvarez has joined #openvswitch | 19:00 | |
| *** fbl has quit IRC | 19:06 | |
| *** zhouhan has quit IRC | 19:13 | |
| *** istokes has quit IRC | 19:14 | |
| *** frigo has joined #openvswitch | 19:39 | |
| *** thaller has joined #openvswitch | 19:55 | |
| *** tbachman has joined #openvswitch | 19:57 | |
| *** jobewan has quit IRC | 20:03 | |
| *** frigo has quit IRC | 20:51 | |
| *** aconole has quit IRC | 21:02 | |
| *** dcbw has quit IRC | 21:15 | |
| *** _lore_ has quit IRC | 21:19 | |
| *** slaweq has quit IRC | 21:21 | |
| *** bostondriver has quit IRC | 21:41 | |
| *** rcernin has joined #openvswitch | 21:44 | |
| *** rcernin has quit IRC | 21:56 | |
| *** rcernin has joined #openvswitch | 22:02 | |
| *** _lore_ has joined #openvswitch | 22:22 | |
| *** rcernin has quit IRC | 22:37 | |
| *** acidfoo has joined #openvswitch | 23:02 | |
| *** rcernin has joined #openvswitch | 23:19 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!