| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: DNM: Run migration tool https://review.opendev.org/c/openstack/neutron-fwaas/+/920672 | 01:57 |
|---|---|---|
| opendevreview | Merged openstack/neutron-lib master: Revert "Always expect project_id and tenant_id to be in the attributes list" https://review.opendev.org/c/openstack/neutron-lib/+/920789 | 02:08 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Fix missing oslo.log options in migration tool https://review.opendev.org/c/openstack/neutron-fwaas/+/920803 | 04:03 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Fix missing oslo.log options in migration tool https://review.opendev.org/c/openstack/neutron-fwaas/+/920803 | 04:04 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: DNM: Run migration tool https://review.opendev.org/c/openstack/neutron-fwaas/+/920672 | 04:06 |
| lajoskatona | ihrachys, haleyb, all: I added the lazy=selectin / coverage job failure topic to the drivers meeting agenda: https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 07:01 |
| opendevreview | Merged openstack/neutron master: Temporarily add '--concurrency 4' to coverage job https://review.opendev.org/c/openstack/neutron/+/920766 | 07:54 |
| opendevreview | Lucas Alvares Gomes proposed openstack/ovn-bgp-agent master: Fix missing EVPN doc indexing https://review.opendev.org/c/openstack/ovn-bgp-agent/+/920813 | 08:26 |
| opendevreview | Lucas Alvares Gomes proposed openstack/ovn-bgp-agent master: Fix missing EVPN doc indexing https://review.opendev.org/c/openstack/ovn-bgp-agent/+/920813 | 08:27 |
| opendevreview | Merged openstack/neutron-fwaas master: create default security group using an independent context https://review.opendev.org/c/openstack/neutron-fwaas/+/916968 | 09:40 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: avoid reconfig bridges/flows when reconnecting to OVS https://review.opendev.org/c/openstack/neutron/+/920819 | 10:17 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: avoid reconfig bridges/flows when reconnecting to OVS https://review.opendev.org/c/openstack/neutron/+/920819 | 10:21 |
| sahid | lajoskatona, haleyb o/ | 12:18 |
| sahid | any chance you have a look at this one https://review.opendev.org/c/openstack/neutron/+/920819 | 12:19 |
| ykarel | lajoskatona, haleyb can you check backports https://review.opendev.org/q/I45c6e6a1c3536cf4f2d90b01a3577eec9eaf3743 | 13:39 |
| ykarel | would be good to get those before ovsdbapp gets updated in u-c https://review.opendev.org/q/topic:%22new-release%22+ovsdbapp+status:open | 13:40 |
| ykarel | thx in advance | 13:40 |
| lajoskatona | ykarel: sure | 13:45 |
| lajoskatona | sahid: sure, I will check your patch | 13:45 |
| sahid | thanks a lot | 14:10 |
| haleyb | ykarel: one of those is marked WIP, was that an accident? | 14:42 |
| haleyb | https://review.opendev.org/c/openstack/neutron/+/920635 | 14:43 |
| ykarel | haleyb, yes mistake, don't know why that got set | 14:43 |
| lajoskatona | I just got to that one :-) | 14:45 |
| *** jlibosva is now known as Guest8068 | 15:06 | |
| opendevreview | Merged openstack/neutron stable/2024.1: [functional tests] compatibility with ovsdbapp>=2.6.1 https://review.opendev.org/c/openstack/neutron/+/920634 | 15:36 |
| opendevreview | Merged openstack/neutron stable/2023.2: [functional tests] compatibility with ovsdbapp>=2.4.2 https://review.opendev.org/c/openstack/neutron/+/920635 | 15:36 |
| opendevreview | Merged openstack/neutron stable/2023.1: [functional tests] compatibility with ovsdbapp>=2.2.2 https://review.opendev.org/c/openstack/neutron/+/920730 | 15:36 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Allow restoration of tun_ofports on agent restart https://review.opendev.org/c/openstack/neutron/+/860270 | 15:38 |
| ihrachys_ | lajoskatona pls refresh +2 here https://review.opendev.org/c/openstack/neutron-lib/+/918149 | 15:49 |
| ihrachys_ | would be nice to also get https://review.opendev.org/c/openstack/neutron-lib/+/917855 reviewed / merged with ^ so that we can cut a neutron-lib release that is netaddr1 compatible (test suite passing) | 15:51 |
| haleyb | ihrachys_: i'm doing some follow-ups on the coverage thread, will send a patch with 'selectin' as well once i create a bug and gather more info | 15:59 |
| ihrachys_ | haleyb ack. Mike said he doesn't expect memory consumption much higher though, right? | 16:00 |
| haleyb | ihrachys_: right. i'm trying different versions of coverage as well to see if that was part of it | 16:01 |
| ihrachys_ | yeah that was a good lead, curious | 16:01 |
| haleyb | not sure 7.4.4 is any better, but i can't tell how far the test is along | 16:03 |
| lajoskatona | haleyb, ihrachys_: as I see this lazy=subquery appears only in Neutron, not in stadiums, so perhaps it will not effect if we change to selectin | 16:04 |
| *** jlibosva is now known as Guest8075 | 16:04 | |
| haleyb | lajoskatona: it will make neutron super-duper fast :) | 16:06 |
| lajoskatona | haleyb: the Holy Grael was under our nose for years and just missed it..... | 16:08 |
| haleyb | lajoskatona: we were using the wrong height for the Staff of Ra? (i'm a dork) | 16:11 |
| lajoskatona | haleyb: :-) | 16:19 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: netaddr1: except TypeError in validate_ip_addr https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/919790 | 16:21 |
| ihrachys_ | haleyb ah off by one errors AGAIN | 16:29 |
| opendevreview | Merged openstack/neutron-fwaas master: Add py312 job to periodic based on testing runtime for 2024.2 https://review.opendev.org/c/openstack/neutron-fwaas/+/914278 | 16:36 |
| opendevreview | Merged openstack/neutron-vpnaas master: Remove executable from python files which don't really needs it https://review.opendev.org/c/openstack/neutron-vpnaas/+/917409 | 16:45 |
| opendevreview | Merged openstack/neutron-vpnaas master: Add pre-commit configuration https://review.opendev.org/c/openstack/neutron-vpnaas/+/917410 | 16:45 |
| opendevreview | Merged openstack/os-ken master: Remove trailing whitespaces and tabs https://review.opendev.org/c/openstack/os-ken/+/918299 | 16:59 |
| opendevreview | Merged openstack/os-ken master: Remove executable from python files which don't really needs it https://review.opendev.org/c/openstack/os-ken/+/918300 | 16:59 |
| opendevreview | Merged openstack/os-ken master: Add pre-commit configuration https://review.opendev.org/c/openstack/os-ken/+/918292 | 16:59 |
| opendevreview | Merged openstack/networking-bagpipe master: Fix trailing whitespaces and replace tabs with 4 spaces https://review.opendev.org/c/openstack/networking-bagpipe/+/917953 | 17:24 |
| mnaser | lajoskatona: if it is ok for you to push an unmaintained branch change - https://review.opendev.org/c/openstack/neutron/+/920610 | 17:25 |
| opendevreview | Merged openstack/networking-bagpipe master: Remove executable from python files which don't really needs it https://review.opendev.org/c/openstack/networking-bagpipe/+/917954 | 17:25 |
| opendevreview | Merged openstack/networking-bagpipe master: Add pre-commit configuration https://review.opendev.org/c/openstack/networking-bagpipe/+/917955 | 17:25 |
| opendevreview | Merged openstack/networking-bgpvpn master: Fix trailing whitespaces and replace tabs with 4 spaces https://review.opendev.org/c/openstack/networking-bgpvpn/+/917957 | 17:25 |
| opendevreview | Merged openstack/networking-bgpvpn master: Remove executable from python files which don't really needs it https://review.opendev.org/c/openstack/networking-bgpvpn/+/917958 | 17:27 |
| lajoskatona | mnaser: Hi, yes, I think I pushed the last one on unmaintained/zed, to tell the truth last 2 weeks I havent checked how those branches work, so fingers crossed :-) | 17:47 |
| opendevreview | Merged openstack/neutron-lib master: api: canonicalize ipv6 addresses in `subnet:allocation_pools` field https://review.opendev.org/c/openstack/neutron-lib/+/918149 | 18:00 |
| opendevreview | Merged openstack/networking-bgpvpn master: Add pre-commit configuration https://review.opendev.org/c/openstack/networking-bgpvpn/+/917959 | 18:55 |
| *** elodilles is now known as elodilles_ooo | 20:04 | |
| opendevreview | Brian Haley proposed openstack/neutron master: Change OVN client to account for nested networks https://review.opendev.org/c/openstack/neutron/+/907504 | 20:42 |
| haleyb | ihrachys_: i think that ^^^ is my final update for that, pending review feedback | 20:43 |
| ihrachys_ | haleyb will check; I already started writing a comment yesterday saying that I don't have preference of behavior, as long as it doesn't depend on randomness (order of ports iterated). will check if this is addressed. | 20:46 |
| ihrachys_ | and thanks for boiling the ocran | 20:46 |
| ihrachys_ | *ocean | 20:49 |
| haleyb | ihrachys_: well, it still depends on order of ports returned when i switched from depth to number, so i'll wait for your review | 20:50 |
| ihrachys_ | so you now limit the amount of work at 50. but the order is still important I think. (it will return 50 subnets max - and configure these 50 but nothing else.) this seems order dependent. | 20:50 |
| ihrachys_ | ok yeah. will check it. but spitballing - 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). | 20:52 |
| haleyb | right, i was trying to protect against someone creating a /8 with all router ports | 20:52 |
| haleyb | ihrachys_: that should be doable I think | 20:53 |
| ihrachys_ | protection is very well welcome there. unlimited recursion can bite us - but all breadth growth, not just depth. so you are onto something with limiting the max number of subnets in addition / instead of recursion depth. | 20:54 |
| ihrachys_ | s/all/also | 20:54 |
| haleyb | i'll wait for any other comments and work on it tomorrow, leaving on-time today :) | 20:55 |
| ihrachys_ | haleyb aye, have a good evening. thanks again. | 20:55 |
| * haleyb gets a large pot ready for a lobster | 20:56 | |
| opendevreview | Jay Jahns proposed openstack/neutron master: Add network dns_domain to dns assignment if present https://review.opendev.org/c/openstack/neutron/+/920459 | 20:57 |
| opendevreview | Merged openstack/neutron-lib master: Catch all exceptions in convert_ip_to_canonical_format https://review.opendev.org/c/openstack/neutron-lib/+/917263 | 21:24 |
| opendevreview | Merged openstack/neutron master: Rename delete_flow_for_ip -> delete_flow_for_ip_and_mac https://review.opendev.org/c/openstack/neutron/+/918143 | 21:28 |
| opendevreview | Merged openstack/neutron stable/2023.2: [stable only] Do not fail on missing logical router ports https://review.opendev.org/c/openstack/neutron/+/920610 | 21:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!