| opendevreview | ZhouHeng proposed openstack/neutron master: Improve ACL comparison efficiency https://review.opendev.org/c/openstack/neutron/+/885449 | 01:00 |
|---|---|---|
| opendevreview | Jay Jahns proposed openstack/neutron master: Add network dns_domain to dns assignment if present https://review.opendev.org/c/openstack/neutron/+/920459 | 04:47 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 04:55 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 04:56 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 04:58 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 06:42 |
| *** elodilles_pto is now known as elodilles | 07:26 | |
| opendevreview | Merged openstack/neutron master: Fix trunk test_subport_delete functional test https://review.opendev.org/c/openstack/neutron/+/921296 | 07:28 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 07:49 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 07:49 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix dhcp cleaning stale devices process when enable action https://review.opendev.org/c/openstack/neutron/+/907250 | 08:14 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2024.1: Ensure cr-lrp ports are exposed https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921667 | 08:19 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Ensure cr-lrp ports are exposed https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921668 | 08:21 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: Handle missing vlan for egress flow setup https://review.opendev.org/c/openstack/networking-sfc/+/921514 | 11:37 |
| opendevreview | Fernando Royo proposed openstack/neutron master: [OVN] Bump revision number after update_virtual_port_host https://review.opendev.org/c/openstack/neutron/+/921759 | 12:30 |
| *** ykarel_ is now known as ykarel | 13:10 | |
| opendevreview | Mohammed Naser proposed openstack/neutron stable/2023.1: [ML2/OVN] Add gateway_port support for FIP https://review.opendev.org/c/openstack/neutron/+/921034 | 13:57 |
| opendevreview | Mohammed Naser proposed openstack/neutron unmaintained/zed: [ML2/OVN] Add gateway_port support for FIP https://review.opendev.org/c/openstack/neutron/+/921035 | 13:57 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Jun 11 14:00:27 2024 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'networking' | 14:00 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh | 14:00 |
| mlavalle | \o | 14:00 |
| slaweq | o/ | 14:00 |
| ralonsoh | hello | 14:00 |
| ihrachys | o/ | 14:00 |
| ykarel | o/ | 14:00 |
| lajoskatona | o/ | 14:01 |
| obondarev | o/ | 14:01 |
| haleyb | looks like we have quorum | 14:02 |
| haleyb | #topic announcements | 14:02 |
| haleyb | We are now in Dalmatian release week (R - 16) | 14:02 |
| haleyb | #link https://releases.openstack.org/dalmatian/schedule.html | 14:02 |
| frickler | o/ | 14:02 |
| rubasov | o/ | 14:02 |
| haleyb | we did do a neutron-lib release last week for some things, so it is being used in the gate, the 'bump' patch hasn't merged yet though | 14:03 |
| haleyb | Reminder if you have a topic for the drivers meeting on Fridays, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:04 |
| haleyb | and just a reminder that if you have a feature planned and haven't started work on it, please start working on it | 14:05 |
| haleyb | i will go through the ptg notes this week as i had some items in there for this cycle | 14:06 |
| * slaweq needs to do the same | 14:06 | |
| haleyb | that was all i had | 14:06 |
| haleyb | oh, and i see ralonsoh is back - welcome back! hope you had a good leave | 14:07 |
| ralonsoh | thank you! | 14:07 |
| mlavalle | +1 | 14:07 |
| lajoskatona | +1 | 14:07 |
| haleyb | we can move onto bugs | 14:08 |
| haleyb | #topic bugs | 14:08 |
| haleyb | jlibosva was the deputy last week | 14:08 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/W2TC6LKDC6FYKJK2MD22PUP3PGAT2HWM/ | 14:08 |
| haleyb | there was one un-assigned bug | 14:09 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2067851 | 14:09 |
| haleyb | Packets dropped in br-int bridge | 14:09 |
| haleyb | large E-W burst shows packet drops, but drop counters don't show it | 14:10 |
| haleyb | i think if we just added some notes on where else to look it would be helpful | 14:10 |
| ralonsoh | (recommendation: not to use hybrid plug --> qvoxxx but native) | 14:10 |
| slaweq | IMHO this is more likely to be asked on the ovs mailing list for example | 14:11 |
| haleyb | slaweq: right, and hybrid plug as well, hadn't noticed that | 14:11 |
| haleyb | i will add a quick note and mention the ovs list | 14:12 |
| slaweq | I don't think hybrid plug is issue here as qvo is veth end which is in the br-int | 14:12 |
| slaweq | so it probably wouldn't change if they would try non hybrid plug | 14:12 |
| ralonsoh | but the linux bridge could be the one dropping the packages | 14:13 |
| slaweq | oh, maybe | 14:13 |
| slaweq | yeah | 14:13 |
| slaweq | so that is one thing we can recommend them to check | 14:13 |
| slaweq | if not then go to the ovs community IMO | 14:13 |
| haleyb | it also didn't mention which ovs version, a 3.x release would be desired | 14:13 |
| haleyb | all other bugs have assignees, i did have a question about one though | 14:15 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2068761 | 14:15 |
| haleyb | mtomaska: this is a revert, but in the bug you mention 'selectin' | 14:16 |
| slaweq | I think that the idea here is to do revert first and backport this revert to stable branches too | 14:18 |
| slaweq | and later change it to use 'selectin' in master branch | 14:18 |
| ralonsoh | ^^ agree (and the test this new option) | 14:18 |
| haleyb | i don't know the back story of the original patch myself, but since i have a 'selectin' change up for review didn't want to miss this in my change | 14:18 |
| slaweq | original patch was proposed long time ago when dansmith raised to us issues that neutron is making A LOT of the sql queries during e.g. tempest job run | 14:19 |
| ralonsoh | but we can revert (including stable branches) and then apply your patch | 14:19 |
| haleyb | i have a selectin discussion in on-demand | 14:19 |
| slaweq | with ralonsoh we were trying to optimize it and the only thing we found was that query for tags | 14:19 |
| ihrachys | selectin is unrelated to revert; but yes, will have to pick up the change to not miss this instance of subquery | 14:20 |
| ralonsoh | the only goal was that, to reduce the queries in the logs | 14:20 |
| slaweq | but now it seems it caused performance regression in case when there is a lot of tags associated with port | 14:20 |
| haleyb | slaweq: ack, just wanted to make sure ihrachys was happy as well since there was a -1 there | 14:20 |
| ihrachys | I am not happy with the commit message | 14:20 |
| ihrachys | 1. it's malformed - there's a single line that spands hundreds of characters. | 14:21 |
| ihrachys | (which is minor but must be fixed) | 14:21 |
| ihrachys | 2. it doesn't document what everyone claims about the number of logs being irrelevant to actual performance. which may be true - I don't question it - but then if that's the rationale, document it. it shouldn't be hard to make this claim in writing in commit message. | 14:22 |
| ihrachys | once these are handled, I will not -1 it. I will happily rely on others' knowledge about the number of logs not being a real issue. | 14:23 |
| haleyb | ihrachys: ok, let's get the message fixed and better document the issue, not sure if mtomaska is around today | 14:23 |
| ralonsoh | it's around and checking now | 14:23 |
| ihrachys | he is, seen him in slack | 14:23 |
| slaweq | just to be clear - it wasn't about number of logs | 14:24 |
| ihrachys | "The initial reason for this change was to reduce amount of SELECT logs" | 14:24 |
| slaweq | dansmith may have more details but IIRC he was trying to optimize i/o load made by ci jobs on the ci infra | 14:24 |
| ihrachys | quoting. so let's make the commit message correct, whatever the correct is | 14:24 |
| ykarel | +1 | 14:25 |
| slaweq | and he came to the conclusion that neutron is far ahead of every other project with making sql queries | 14:25 |
| slaweq | so we have tried to optimize that somehow | 14:25 |
| dansmith | whatever patch wasn't from me was it? I was raising the flag about the query load yeah | 14:25 |
| dansmith | but yeah, neutron was doing like an order of magnitude more select calls for a given run than other services | 14:26 |
| slaweq | ihrachys I know that this is what current commit message says and I already explained to mtomaska that this is not correct | 14:26 |
| ihrachys | ok. then we shouldn't +2 patches with incorrect commit messages! :) | 14:26 |
| slaweq | dansmith patch was from me :) | 14:26 |
| dansmith | ack :) | 14:26 |
| slaweq | ihrachys you're right, my bad. I just removed my vote | 14:27 |
| ihrachys | thanks. I think the next steps are clear - fix / clarify commit message. maybe if we still are not sure the revert is totally innocent, report a LP to follow up on it. | 14:28 |
| haleyb | +1 | 14:28 |
| haleyb | are there any other bugs to discuss? | 14:29 |
| haleyb | This week obondarev is the bug deputy, next week will be isabek | 14:30 |
| haleyb | obondarev: is that good for you? | 14:30 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 14:31 |
| haleyb | and since i have not seen isabek in a while, does anyone here work with him to ping him? | 14:31 |
| haleyb | ok, i'll try and follow-up offline | 14:32 |
| haleyb | #topic specs | 14:33 |
| haleyb | #link https://review.opendev.org/q/project:openstack%252Fneutron-specs+status:open | 14:33 |
| obondarev | haleyb: yes sure | 14:33 |
| haleyb | slaweq: i think we can merge the pre-commit one, will look after meeting | 14:33 |
| haleyb | obondarev: thanks! | 14:33 |
| slaweq | thx | 14:34 |
| haleyb | there is some discussion in the other | 14:34 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron-specs/+/920681 | 14:34 |
| haleyb | i have not reviewed yet | 14:34 |
| haleyb | #topic community-goals | 14:36 |
| haleyb | lajoskatona: i see the horizon change is green, hopefully you get some core reviews | 14:37 |
| lajoskatona | I have to ping the horizon folks again to have attention | 14:37 |
| haleyb | slaweq: i don't seem to have your current community goal in the wiki, will need to add it | 14:39 |
| slaweq | thx | 14:39 |
| slaweq | I still need to finish some internal tasks and then get back to the SRBAC stuff | 14:40 |
| haleyb | still has service role, which is complete | 14:40 |
| haleyb | ack | 14:40 |
| lajoskatona | is there a consensus for the drop eventlet topic? | 14:40 |
| slaweq | yes, but we need now to use service token while communicating with e.g. nova | 14:40 |
| ihrachys | lajoskatona consensus where? does neutron have any specific plans? | 14:41 |
| slaweq | and test that (I need to sync with @gmann on that part at some point) | 14:41 |
| lajoskatona | I mean the community as a whole (for this one: https://review.opendev.org/c/openstack/governance/+/902585 ) | 14:41 |
| haleyb | the timeline i had (from the ptg) was | 14:42 |
| haleyb | 2025.2 for co-existence | 14:42 |
| haleyb | 2027.2 for migration | 14:42 |
| haleyb | personally, i think it's a good plan in the governance doc | 14:43 |
| haleyb | but neutron does not have an owner for any work at the moment | 14:44 |
| lajoskatona | haleyb: ack, we go for asyncio? yes the last update is more concrete perhaps | 14:44 |
| ralonsoh | did we check the tasks to fix that in neutron? | 14:44 |
| opendevreview | Merged openstack/neutron-specs master: Add pre-commit configuration https://review.opendev.org/c/openstack/neutron-specs/+/918306 | 14:44 |
| ralonsoh | for example, remove evenlet from agents, fix OVN with wsgi, etc | 14:44 |
| opendevreview | Jay Jahns proposed openstack/neutron-tempest-plugin master: Add check to pick up network's dns_domain if present https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/921772 | 14:45 |
| ralonsoh | haleyb, right, that needs an owner, first of all | 14:45 |
| haleyb | ralonsoh: the OVN with wsgi is still an issue | 14:45 |
| ralonsoh | the bug is still open | 14:46 |
| ralonsoh | I'll ping Lucas to check the status | 14:47 |
| haleyb | yes, and last i asked about it the work was non-trivial | 14:47 |
| ihrachys | so who's the owner? :) | 14:49 |
| ralonsoh | I can take it and do my best | 14:49 |
| ralonsoh | but for sure I'll need your help | 14:50 |
| ihrachys | thank you. I think we will need to read on it, I for one am not well versed in asyncio. | 14:50 |
| haleyb | ralonsoh: thanks, and I will help where i can | 14:50 |
| haleyb | let's move on | 14:51 |
| haleyb | #topic on-demand | 14:51 |
| haleyb | i had added two topics, but want to see if others have any | 14:52 |
| mlavalle | none from me | 14:52 |
| haleyb | alright, my first was regarding 'selectin' load strategy | 14:52 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/920933 | 14:52 |
| haleyb | once neutron-lib was released that started to pass | 14:53 |
| haleyb | but it will need a rebase on the revert we mentioned earlier | 14:53 |
| haleyb | and i now wonder if we should add a hacking check to not allow lazy='subquery' any longer | 14:53 |
| ihrachys | yes pls | 14:53 |
| ykarel | haleyb, nice, btw so you observed that it also helped in memory usage with cover jobs? | 14:54 |
| lajoskatona | good idea | 14:54 |
| haleyb | the reason to merge that sooner than later was to get some cycles testing with it | 14:54 |
| haleyb | ykarel: no, didn't seem to help memory consumption, but will try again with new neutron-lib | 14:55 |
| ykarel | ohkk, CI should already have newer neutron-lib | 14:55 |
| haleyb | ykarel: right, and i'll run some tests locally when a fresh VM, but 'coverage' tool itself seems to be the memory hog here | 14:56 |
| ykarel | ack | 14:57 |
| haleyb | ykarel: i will have to look at your changes regarding this, i did not forget | 14:57 |
| haleyb | i.e. swap, etc | 14:57 |
| ykarel | +1 | 14:57 |
| haleyb | my other topic was regarding a ptg goal - migrate from wsgi scripts to python module paths | 14:58 |
| haleyb | there were two reviews posted right after ptg from stephenfin (thanks!) - i have since rebased and will do again if necessary, but need some reviews | 14:58 |
| haleyb | https://review.opendev.org/c/openstack/neutron/+/916406 | 14:59 |
| haleyb | https://review.opendev.org/c/openstack/neutron/+/916407 | 14:59 |
| haleyb | so if anyone has some cycles, thanks in advance | 14:59 |
| haleyb | and we are out of time | 15:00 |
| lajoskatona | do we have/do we need devstack support for this? | 15:00 |
| haleyb | ci meeting starts now, video today | 15:00 |
| haleyb | lajoskatona: i think there was a patch for that, don't know if we need more for neutron, good question | 15:01 |
| frickler | lajoskatona: there should be related devstack patches, yes | 15:01 |
| haleyb | https://review.opendev.org/q/topic:%22remove-wsgi_scripts%22 | 15:01 |
| lajoskatona | haleyb, frickler: thanks | 15:01 |
| haleyb | thanks for coming everyone | 15:01 |
| haleyb | #endmeeting | 15:01 |
| opendevmeet | Meeting ended Tue Jun 11 15:01:46 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-06-11-14.00.html | 15:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-06-11-14.00.txt | 15:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-06-11-14.00.log.html | 15:01 |
| lajoskatona | o/ | 15:01 |
| ralonsoh | bye | 15:01 |
| frickler | https://review.opendev.org/c/openstack/devstack/+/916412 | 15:01 |
| ykarel | #startmeeting neutron_ci | 15:02 |
| opendevmeet | Meeting started Tue Jun 11 15:02:49 2024 UTC and is due to finish in 60 minutes. The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:02 |
| ykarel | Ping list: bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:02 |
| mlavalle | \o | 15:02 |
| ykarel | This will be video meeting this time: https://meetpad.opendev.org/neutron-ci-meetings | 15:02 |
| ralonsoh | (sorry I can' | 15:03 |
| lajoskatona | o/ | 15:03 |
| ralonsoh | I can't attend today | 15:03 |
| slaweq | o/ | 15:03 |
| ykarel | #topic Actions from previous meetings | 15:04 |
| ykarel | lajoskatona to check sfc failures | 15:04 |
| lajoskatona | https://bugs.launchpad.net/neutron/+bug/2068727 | 15:04 |
| ykarel | #link https://bugs.launchpad.net/neutron/+bug/2068727 | 15:06 |
| ykarel | mlavalle to check failures in neutron-functional | 15:06 |
| ykarel | slaweq to include ovn grenade jobs in check queue | 15:08 |
| ykarel | #action slaweq to add ovn grenade to check queue | 15:11 |
| ykarel | ykarel to include neutron unit-test job in neutron-lib to catch such issues | 15:12 |
| ykarel | https://review.opendev.org/c/openstack/neutron-lib/+/921422 | 15:12 |
| ykarel | ykarel to check failures in neutron-fullstack | 15:12 |
| ykarel | ykarel to push patch to drop centos 8-stream jobs | 15:15 |
| ykarel | https://review.opendev.org/c/openstack/neutron/+/921517 | 15:15 |
| ykarel | #topic Stable branches | 15:15 |
| ykarel | all green | 15:15 |
| ykarel | #topic Stadium projects | 15:16 |
| ykarel | #topic Rechecks | 15:16 |
| ykarel | #topic fullstack/functional | 15:17 |
| ykarel | test_direct_route_for_address_scope | 15:17 |
| ykarel | 3 packets transmitted, 0 received, 100% packet loss, time 2049ms | 15:17 |
| ykarel | https://4ae2de5d02c97f2930b7-ff2d888c3307ad9718a14b305143c473.ssl.cf2.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-oslo-master/419cce2/testr_results.html | 15:17 |
| ykarel | #action mlavalle to check failures in test_direct_route_for_address_scope | 15:18 |
| ykarel | test_metadata_proxy_rate_limiting_ipv6 | 15:18 |
| ykarel | ; Stdout: ; Stderr: Cannot open network namespace "qrouter-0156eaec-06cc-43a7-83cb-1a72504f7624": No such file or directory | 15:19 |
| ykarel | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_08a/885449/22/check/neutron-functional-with-uwsgi/08af95d/testr_results.html | 15:19 |
| ykarel | #action ykarel to check test_metadata_proxy_rate_limiting_ipv6 | 15:20 |
| ykarel | test_configurations_are_synced_towards_placement | 15:20 |
| ykarel | neutron.common.utils.WaitTimeout: Timed out after 47 seconds | 15:20 |
| ykarel | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_66c/periodic/opendev.org/openstack/neutron/master/neutron-fullstack-with-uwsgi-fips/66c69d3/testr_results.html | 15:20 |
| ykarel | #action lajoskatona to check fullstack failure test_configurations_are_synced_towards_placement | 15:21 |
| ykarel | #topic Tempest/Scenario | 15:21 |
| ykarel | neutron-tempest-plugin-openvswitch-enforce-scope-old-defaults | 15:21 |
| ykarel | https://551b5d3d5ab1e9429cee-63329dcd236a98c48b3784d0d458e269.ssl.cf5.rackcdn.com/921309/5/gate/neutron-tempest-plugin-openvswitch-enforce-scope-old-defaults/a4ace52/testr_results.html | 15:21 |
| ykarel | https://3fd7427f13d5af8cb7bc-eca1b8de8e7b31708d9cffbfd5030903.ssl.cf1.rackcdn.com/921309/7/gate/neutron-tempest-plugin-openvswitch-enforce-scope-old-defaults/2ca2d39/testr_results.html | 15:21 |
| ykarel | https://f38d3934203b991a7ebe-d701f6a98461967df274f184d5a7d3cd.ssl.cf2.rackcdn.com/921526/1/check/neutron-ovs-tempest-multinode-full/2dc0e58/testr_results.html | 15:21 |
| ykarel | #action ykarel to check tempest failures in enforce-scope job | 15:22 |
| ykarel | #topic Grafana | 15:23 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:23 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921566 | 15:23 |
| ykarel | #topic On Demand | 15:26 |
| ykarel | #endmeeting | 15:27 |
| opendevmeet | Meeting ended Tue Jun 11 15:27:57 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-06-11-15.02.html | 15:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-06-11-15.02.txt | 15:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-06-11-15.02.log.html | 15:27 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 15:52 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/2024.1: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921567 | 16:31 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/2023.2: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921568 | 16:31 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/2023.1: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921569 | 16:32 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/zed: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921780 | 16:41 |
| opendevreview | Merged openstack/neutron-lib master: Add neutron unit test job in check queue https://review.opendev.org/c/openstack/neutron-lib/+/921422 | 17:35 |
| opendevreview | Merged openstack/neutron master: [OVN] Fix virtual parent match for PortBindingUpdateVirtualPortsEvent https://review.opendev.org/c/openstack/neutron/+/921659 | 18:04 |
| opendevreview | Merged openstack/neutron master: Bump neutron-lib to 3.13.0 https://review.opendev.org/c/openstack/neutron/+/921533 | 18:04 |
| opendevreview | Miro Tomaska proposed openstack/neutron unmaintained/zed: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921787 | 18:16 |
| opendevreview | Miro Tomaska proposed openstack/neutron unmaintained/yoga: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921788 | 18:17 |
| opendevreview | Miro Tomaska proposed openstack/neutron unmaintained/wallaby: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921791 | 18:25 |
| opendevreview | Miro Tomaska proposed openstack/neutron unmaintained/wallaby: Revert "Use HasStandardAttributes as parent class for Tags DB model" https://review.opendev.org/c/openstack/neutron/+/921791 | 18:32 |
| opendevreview | Jay Jahns proposed openstack/ovn-bgp-agent master: Announce lrp ip if advertisement method is subnet https://review.opendev.org/c/openstack/ovn-bgp-agent/+/921516 | 18:34 |
| opendevreview | Jay Jahns proposed openstack/neutron-tempest-plugin master: Add check to pick up network's dns_domain if present https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/921772 | 18:52 |
| opendevreview | Brian Haley proposed openstack/neutron master: Improve ACL comparison efficiency https://review.opendev.org/c/openstack/neutron/+/885449 | 20:51 |
| haleyb | ihrachys: i think this one is ready for review - https://review.opendev.org/c/openstack/neutron/+/907504 :) | 20:59 |
| ihrachys | haleyb commented; main question is if fallback to empty set - and not "just the 1st level subnets" - is the right approach. otherwise, just nits. | 21:30 |
| ihrachys | the patch is cursed | 21:31 |
| haleyb | ihrachys: i thought our last conversation was "do no SNAT" if the recursion failed, will look in the time machine | 21:31 |
| haleyb | "could we just not configure nested if there are too many nested, at all? so not leave it to chance but just do nothing (except the warning)? that would be consistent (order independent)." | 21:33 |
| haleyb | sigh, maybe i interpreted how i wanted to see it, especially since having just 1st level is harder to do with a single function doing all the work | 21:36 |
| ihrachys | haleyb I was vague; but also only when I saw your latest version, I realized we are going to affect a (unknown, probably minuscule) number of existing users with convoluted chains of networks. | 21:55 |
| haleyb | but it's very determinate - "you'll get nothing and like it", and it covers the case where someone creates 100 1st level subnets | 21:57 |
| haleyb | ihrachys: i can hack at it one more time, going to make the testing harder i think | 21:58 |
| ihrachys | it's definitely deterministic now. just not backwards compat. :) | 21:58 |
| * haleyb will get the machete out tomorrow | 22:01 | |
| opendevreview | Merged openstack/neutron master: [OVN] Bump revision number after update_virtual_port_host https://review.opendev.org/c/openstack/neutron/+/921759 | 22:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!