| sean-k-mooney | haleyb: for what its worht in most project i have interacted with pep8 has alwasy been enforced for tests too | 00:15 |
|---|---|---|
| haleyb | sean-k-mooney: right, and i think i fixed that in neutron last cycle, just seeing some oddities with indentation, and not just in tests, but no pep8 failures | 00:16 |
| sean-k-mooney | the only thing that is generaly skip for it should be the security checks and even then that just so it dose not warn ofn passwords and such | 00:16 |
| sean-k-mooney | so ye ar thinking its because of https://github.com/openstack/neutron/blob/master/.pylintrc | 00:16 |
| sean-k-mooney | i have not seen that in other repos | 00:16 |
| haleyb | i have no idea what's causing it at the moment, kuba was just seeing things fail downstream and pushed up some changes | 00:17 |
| haleyb | i mean, i could have messed something up | 00:17 |
| sean-k-mooney | im not seeing any filtering in the pre-commit def https://github.com/openstack/neutron/blob/master/.pre-commit-config.yaml#L47-L56 and your pyproject.toml looks pretty normal | 00:18 |
| haleyb | sean-k-mooney: i basically copied it from nova | 00:18 |
| sean-k-mooney | haleyb: im not seein ganytin obvious | 00:18 |
| sean-k-mooney | well byond .pylintrc | 00:19 |
| sean-k-mooney | ye also have a https://github.com/openstack/neutron/blob/master/.coveragerc | 00:19 |
| sean-k-mooney | those are the tow things that look diffent form nova at a glance | 00:19 |
| haleyb | sean-k-mooney: thanks for looking, what it is with .pylintrc that you see? | 00:19 |
| sean-k-mooney | the fact it exists | 00:20 |
| sean-k-mooney | nova does not have one | 00:20 |
| sean-k-mooney | https://github.com/openstack/neutron/blob/master/.pylintrc#L8 | 00:20 |
| sean-k-mooney | specificly im wondering ifn the ignore paths has any effect | 00:20 |
| haleyb | we do run pylint separately from a tools script | 00:21 |
| sean-k-mooney | hum well if i remove those fiels and run pre-commit nothing changes in the repo | 00:25 |
| sean-k-mooney | so that does not appre to be related | 00:26 |
| sean-k-mooney | super odd i can confim that pep8 issues dont appear to be caught in the test modules | 00:32 |
| sean-k-mooney | oh actully that not in the test it in neutron/service.py | 00:33 |
| haleyb | we borked something up | 00:36 |
| sean-k-mooney | [00:40:39]> uvx autopep8 -d -r neutron -v | 00:41 |
| sean-k-mooney | read config path: /home/smooney/repos/neutron/setup.cfg | 00:41 |
| sean-k-mooney | read config path: /home/smooney/repos/neutron/tox.ini | 00:41 |
| sean-k-mooney | enable config: section=flake8, key=select, value=H,N | 00:41 |
| sean-k-mooney | enable config: section=flake8, key=ignore, value=H405,N530 | 00:41 |
| sean-k-mooney | enable config: section=flake8, key=exclude, value=./.*,build,dist,doc | 00:41 |
| sean-k-mooney | why do you have exclude = ./.*,build,dist,doc | 00:42 |
| sean-k-mooney | in tox.ini | 00:42 |
| haleyb | it's always been there so ? | 00:43 |
| sean-k-mooney | oh no that not the issue | 00:43 |
| sean-k-mooney | so the issue si | 00:44 |
| * haleyb waits for the bomb to drop :) | 00:44 | |
| sean-k-mooney | [flake8] | 00:44 |
| sean-k-mooney | # We only enable the hacking (H) and neutron (N) checks | 00:44 |
| sean-k-mooney | select = H,N | 00:44 |
| sean-k-mooney | you only neable the hacking and N which i guess is neutron check | 00:44 |
| sean-k-mooney | that also contols autopep8 | 00:44 |
| sean-k-mooney | your flake8 section is disbaling everything else which is fine if your using ruff for formating | 00:45 |
| sean-k-mooney | but neutron currently is not | 00:45 |
| haleyb | i thought it was | 00:45 |
| sean-k-mooney | ruff check is enabled | 00:45 |
| sean-k-mooney | but not ruff format | 00:45 |
| sean-k-mooney | if i comment out the select line autopep8 starts fixing a bunch of stuff | 00:46 |
| haleyb | wtf | 00:46 |
| sean-k-mooney | autopep8 reads the flake8 secotion of tox ini to know what rules to enforce | 00:46 |
| sean-k-mooney | nova does not use select https://github.com/openstack/nova/blob/master/tox.ini#L318-L344 because nova is still using autopep8 to format things | 00:48 |
| sean-k-mooney | so the fix is either to adopt ruff for formating | 00:48 |
| sean-k-mooney | or to remove the select and keep autopep8 for now | 00:48 |
| haleyb | so that change was in be77ed3a813709e by $someoneelse, so might have been an oversight | 00:49 |
| sean-k-mooney | ya at this point i woudl proably jsut add ruff here https://github.com/openstack/neutron/blob/master/.pre-commit-config.yaml#L47-L51 | 00:49 |
| sean-k-mooney | https://github.com/openstack/cyborg/blob/master/.pre-commit-config.yaml#L33-L38 | 00:50 |
| sean-k-mooney | i did that in cyborg this cyle | 00:50 |
| sean-k-mooney | the select thing is correct if you have adoptied ruff but when you do that you also remove autopep8 | 00:50 |
| haleyb | so i don't see ruff in nova, guess it does things differently? | 00:50 |
| sean-k-mooney | nova has not adopted ruff for formatting as some peopel stongly dislike its style | 00:51 |
| sean-k-mooney | so its still entirly pep8 | 00:51 |
| sean-k-mooney | there is a series up to adopt it but it not a battle im currently planning to fight | 00:52 |
| sean-k-mooney | watcher is in the transation like neutron | 00:52 |
| sean-k-mooney | with ruff-check for extra linters and autopep8 for formating https://github.com/openstack/watcher/blob/master/.pre-commit-config.yaml#L44-L52 | 00:53 |
| haleyb | sean-k-mooney: well thanks for finding the issue, luckily i don't think it's caused any issues | 00:53 |
| haleyb | oh boy, ruff is pretty aggressive | 00:53 |
| sean-k-mooney | but again importally no select line https://github.com/openstack/watcher/blob/master/tox.ini#L134-L141 | 00:53 |
| sean-k-mooney | ruff has diffent defaults | 00:54 |
| haleyb | i think i'll just remove the select line | 00:54 |
| sean-k-mooney | if you want to be concervitive i woudl say ya remove the select line and let autopep8 fix things for now | 00:54 |
| sean-k-mooney | and then consider moving to ruff next release | 00:54 |
| haleyb | it is the next release :) | 00:55 |
| sean-k-mooney | well true | 00:55 |
| haleyb | but yeah, i'll wait a while for that as it touched over 400 files | 00:55 |
| sean-k-mooney | i was thinking you might want to remove the select on the stabel branch and just apply autopep8 on the affected stabel branches | 00:55 |
| sean-k-mooney | ya the cyborg diff was pretty massive | 00:56 |
| sean-k-mooney | haleyb: my advice is done try to cherry pick the fix | 00:56 |
| sean-k-mooney | jsut do a stabel only change to remove the select on the affected stable branches | 00:57 |
| sean-k-mooney | and let auto pep8 adress the relevent fixes for each branch | 00:57 |
| haleyb | well i don't think the changes impact anything, so i wouldn't touch stable, mostly is missing newlines and multi-line wrap stuff | 00:58 |
| sean-k-mooney | well yes | 00:59 |
| haleyb | but kuba will be happy he doesn't have to look into it further | 00:59 |
| sean-k-mooney | you coudl leave it as is but it will be anoying when you backprot | 00:59 |
| sean-k-mooney | stephen skiped a sepc in https://github.com/openstack/neutron/commit/be77ed3a813709eebb2659b5473b890b5978c743#diff-ef2cef9f88b4fe09ca3082140e67f5ad34fb65fb6e228f119d3812261ae51449 | 00:59 |
| sean-k-mooney | my guess is an elerive version also enabeld ruff formating | 01:00 |
| sean-k-mooney | and when swaping to autopep8 that select didnt get removed | 01:00 |
| sean-k-mooney | ah https://review.opendev.org/c/openstack/neutron/+/950640/2 was abandoned | 01:00 |
| sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/950640/2/.pre-commit-config.yaml | 01:01 |
| sean-k-mooney | so yes there orgial plan was to also apply ruff format but that never happend | 01:01 |
| sean-k-mooney | "show all 1440 files" ... ya im not going to click that | 01:02 |
| haleyb | yeah, he was the $someoneelse, i didn't want to blame him as i probably approved it | 01:02 |
| sean-k-mooney | well its not really blaming htem it has 4 +2s :) | 01:02 |
| sean-k-mooney | but i know they have doen the conversion on seveal other repo but few have doen it in mulitple steps where they ran with auto pep8 + ruff check for protacted peroids | 01:03 |
| sean-k-mooney | so this was just an oversight | 01:03 |
| sean-k-mooney | anyway i better go to sleep if you need any help fixing it let me know | 01:04 |
| haleyb | no, i've been done for a while, just saw the ping, have a good one and thanks for finding it! | 01:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Fix an issue in the L3 OVN scheduler (3) https://review.opendev.org/c/openstack/neutron/+/981211 | 08:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == DNM == OVN L3 scheduler test (3) https://review.opendev.org/c/openstack/neutron/+/981212 | 08:23 |
| opendevreview | Harald Jensås proposed openstack/neutron stable/2026.1: Include dynamic segments in NetworkContext https://review.opendev.org/c/openstack/neutron/+/981456 | 08:24 |
| opendevreview | Harald Jensås proposed openstack/neutron stable/2026.1: Add docstrings for filter_dynamic parameter https://review.opendev.org/c/openstack/neutron/+/981457 | 08:25 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: Enable additional ruff rules https://review.opendev.org/c/openstack/neutron/+/981471 | 10:04 |
| opendevreview | Merged openstack/neutron master: bgp: Fix more pep8 errors https://review.opendev.org/c/openstack/neutron/+/981215 | 10:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Fix an issue in the L3 OVN scheduler (3) https://review.opendev.org/c/openstack/neutron/+/981211 | 11:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == DNM == OVN L3 scheduler test (3) https://review.opendev.org/c/openstack/neutron/+/981212 | 11:58 |
| opendevreview | Merged openstack/neutron stable/2026.1: Include dynamic segments in NetworkContext https://review.opendev.org/c/openstack/neutron/+/981456 | 13:04 |
| haleyb | ralonsoh, lajoskatona: just an fyi that stable/2026.1 still isn't running the full set of zuul tests. i have a patch up for tempest which is blocked but can't remember why things like functional aren't running there | 13:10 |
| lajoskatona | haleyb: ack, thanks for checking | 13:12 |
| haleyb | there must be something over in the openstack-zuul-jobs repo or something ? | 13:12 |
| haleyb | i.e. a stable/2026.1 regex | 13:12 |
| lajoskatona | haleyb: https://review.opendev.org/c/openstack/neutron/+/981131 , ykarel wrote that not all branches were cut | 13:13 |
| haleyb | lajoskatona: right, but what about functional? | 13:14 |
| lajoskatona | haleyb: +1, and I hope we have the branches now | 13:15 |
| elodilles | devstack and requirements repositories not branched yet, but all the others had | 13:18 |
| lajoskatona | for functional that is really strange | 13:18 |
| haleyb | i just don't remember doing anything special for 2025.2, maybe it's just devstack then | 13:18 |
| elodilles | haleyb: btw, do you plan to update neutron-tempest-plugin latest release patch? | 13:19 |
| lajoskatona | elodilles: but than we cant run tempest jobs without devstack, am I right? | 13:19 |
| lajoskatona | haleyb: true, for functional we also neew devstack, so that can be the culprit | 13:19 |
| lajoskatona | s/neew/need | 13:19 |
| haleyb | elodilles: hey, might have just forgot, let me find it and i can ping people while they're online | 13:20 |
| elodilles | +1 | 13:21 |
| elodilles | lajoskatona: i think the bigger problem would be if devstack + requirements branched, but the others are not. though it is possible that the not yet existing stable/2026.1 of devstack causing issue on neutron's stable/2026.1 branch | 13:23 |
| haleyb | well that isn't happening since we're only running pep8, unit and docs jobs :-o | 13:23 |
| elodilles | the general template for python unittest jobs should run at least: https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/976619 | 13:25 |
| elodilles | i mean, the template is updated, so those jobs should run | 13:26 |
| haleyb | right, i think that's why we see unit tests run, functional would rely on devstack :( | 13:30 |
| lajoskatona | strange because nothing is in the job definition that hardcodes devstack branch | 13:31 |
| * haleyb shrugs | 13:32 | |
| haleyb | elodilles: i updated to the latest hash for the n-t-p release, think it should be good just con't confirm since jobs are not running there | 13:37 |
| elodilles | haleyb: ACK | 13:47 |
| haleyb | not sure if we'll have quorum for drivers meeting but will see | 14:00 |
| haleyb | #startmeeting neutron_drivers | 14:00 |
| opendevmeet | Meeting started Fri Mar 20 14:00:33 2026 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 'neutron_drivers' | 14:00 |
| haleyb | Ping list: ykarel, mlavalle, mtomaska, slaweq, tobias-urdin, lajoskatona, haleyb, ralonsoh | 14:00 |
| mtomaska | hi | 14:01 |
| ichen | hello | 14:01 |
| slaweq | hi | 14:01 |
| lajoskatona | o/ | 14:02 |
| haleyb | hi everyone, and i guess with 3 we have quorum? i can't remember what we reset that to | 14:02 |
| haleyb | there was only one item on the agenda today from ichen | 14:04 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2144617 | 14:04 |
| slaweq | according to https://launchpad.net/~neutron-drivers/+members#active there are 5 members in total, so 3 is quorum | 14:04 |
| lajoskatona | +1 | 14:04 |
| haleyb | thanks for looking | 14:05 |
| ichen | Should I start? | 14:05 |
| lajoskatona | so this is an extension for the feature on which jlibosva is working/ just finished? | 14:05 |
| haleyb | ichen: yes, please | 14:05 |
| ichen | Yes, this RFE is to extend the BGP service plugin that jlibosva started. | 14:05 |
| ichen | It’s to support EVPN type-5 route. | 14:06 |
| lajoskatona | ack | 14:06 |
| ichen | I forgot to mention it in the RFE, but after some prototyping work, the first phase will only support centralized routing. OVN currently doesn’t make it easy to support DVR. | 14:07 |
| lajoskatona | and for that you need FRR am I right? | 14:08 |
| ichen | Yes. Jlibosva’s BGP service plugin uses FRR, and configures one BGP instance. EVPN extension will reuse the same FRR/BGP setup, except that that it’ll use l2vpn evpn address family. | 14:09 |
| ichen | More precisely, *add* l2vpn evpn address family. | 14:09 |
| lajoskatona | ack, thanks for clarification | 14:10 |
| haleyb | and a type-5 route is for a prefix? | 14:10 |
| ichen | Additionally, each VNI will have a corresponding VRF and BGP vrf instance. | 14:10 |
| ralonsoh | hello | 14:11 |
| haleyb | o/ | 14:11 |
| ichen | Yes, type-5 route is only IP prefix, including /32. The plan is to advertise the VM host routes. | 14:11 |
| haleyb | ichen: and an entire private subnet i guess? | 14:12 |
| ichen | haleyb: do you mean the subnet the hosts are on? I haven’t checked if OVN supports both subnet and host routes simultaneously. | 14:13 |
| haleyb | ichen: i guess i thought that would just be possible, instead of a /32 a type-5 could be a /28 or /24 or... | 14:14 |
| ichen | Yes, that is possible. | 14:15 |
| ichen | I meant that I haven’t tried to advertise both the network and the /32. I personally think that advertising just the network makes sense without type-2, but I tried the host routes because I got the question explicitly. | 14:16 |
| haleyb | is that part of this RFE, or just changing to use a type-5 for /32 | 14:16 |
| ichen | I consider it part of this RFE, and whether we advertise /32 or the network is a design decision. | 14:18 |
| lajoskatona | do we need to set it in a design doc/spec ? | 14:18 |
| ralonsoh | yes, I think so | 14:19 |
| ichen | The way to advertise a route is for Neutron to set an option in the NB ovsdb, and I was planning to call out what option Neutron will set. | 14:19 |
| ichen | Here are the possible options: https://github.com/ovn-org/ovn/blob/4bab00d57e5cc3d8d7a6670660fc8f6d63c2bdf9/ovn-nb.xml#L3319 | 14:20 |
| ichen | This is the host route: https://github.com/ovn-org/ovn/blob/4bab00d57e5cc3d8d7a6670660fc8f6d63c2bdf9/ovn-nb.xml#L3327 | 14:20 |
| ichen | There are more following those two. | 14:21 |
| ichen | Sorry, yes, what ralonsho said. The design doc/spec should call out the options. | 14:22 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: Enable additional ruff rules https://review.opendev.org/c/openstack/neutron/+/981471 | 14:22 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: Re-enable flake8 https://review.opendev.org/c/openstack/neutron/+/981521 | 14:22 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: ruff: Enable preview rules https://review.opendev.org/c/openstack/neutron/+/981522 | 14:22 |
| lajoskatona | ichen, ralonsoh: +1 for spec | 14:23 |
| stephenfin | haleyb: sean-k-mooney: ^ I didn't know so many of the ruff rules were behind the preview flag. Sorry about that :( | 14:23 |
| haleyb | ichen: yes a spec would be good since there are options on what to support | 14:24 |
| haleyb | stephenfin: thanks, will look in a bit | 14:25 |
| sean-k-mooney | stephenfin: oh cool ill take a look at those when i can | 14:25 |
| haleyb | anyway, are there more questions about the RFE? | 14:25 |
| ichen | The spec is almost ready, just need to incorporate some corrections based on the prototype. | 14:25 |
| sean-k-mooney | thanks for spliting thos up it will make the review eaiser | 14:25 |
| ralonsoh | ichen, will that be an independent extension? Or just an update of the ovn-bgp feature? | 14:26 |
| slaweq | no questions from me about this RFE | 14:26 |
| ichen | Just an update to what jlibosva created. | 14:26 |
| ralonsoh | can both bgp and evpn work together? (if that is even necessary) | 14:26 |
| mtomaska | BGP is a mechanism to EVPN working, they have to work together for the overall solution to work. | 14:28 |
| ichen | Yes. The plan is to reuse the same BGP instance and the same peer(s), with additional EVPN configurations. (The additional EVPN configurations are in the form of BGP vrf instance, so it sounds like another instance, but everything will be advertised through the same BGP peer(s).) | 14:28 |
| ralonsoh | ^ that was the answer I was looking for | 14:29 |
| ichen | The same BGP peer can advertise regular routes and evpn routes. | 14:29 |
| ralonsoh | cool | 14:29 |
| jlibosva | I thought it was more independent and OVN steers the traffic to the tunnel already from the Neutron router | 14:29 |
| jlibosva | so so none of the underlaying BGP resources was gonna be used in such case | 14:30 |
| ichen | I assume that OVN would respect the VRF routing. The regular routes will be routing in Linux 42. The EVPN routes will be routing based on their respective VRF. | 14:31 |
| ichen | Sorry, not “Linux 42”, but “Linux VRF table 42”, which is what the existing BGP service plugin uses. | 14:32 |
| ralonsoh | ok, so that needs to be clarified in the spec too | 14:33 |
| ichen | I will incorporate a discussion of regular address family and EVPN address family in the spec. | 14:34 |
| ralonsoh | because how the architecture jlibosva designed, I don't see how the traffic will sent to the bgp peers | 14:34 |
| jlibosva | I think ichen means if we have a remote workload running in the remote cloud then it's gonna use tunnel created by evpn. If it's some east-west or provider north-south it's gonna be using the underlaying core BGP | 14:36 |
| ichen | Additionally, a workload is either within an EVPN VRF or not. If the workload is within an EVPN VRF, then the forwarding is based on the EVPN VRF. | 14:38 |
| haleyb | ok, so some things to clarify in the spec | 14:38 |
| haleyb | any other questions or should we vote? | 14:38 |
| haleyb | ok, i am a +1 | 14:39 |
| slaweq | +1 from me | 14:39 |
| lajoskatona | +1 from me with spec | 14:40 |
| ralonsoh | +1 | 14:40 |
| haleyb | ok, i will approve it, thanks everyone | 14:41 |
| ichen | Thanks, everyone! | 14:41 |
| haleyb | ichen: i think we have to create a 2026.2 directory in the specs repo, will check | 14:41 |
| ichen | ack | 14:42 |
| haleyb | that was the only thing on the agenda, anything else? | 14:42 |
| haleyb | ichen: actually looks like it's there | 14:43 |
| lajoskatona | nope | 14:43 |
| haleyb | ok, thanks everyone for attending, have a nice weekend! | 14:43 |
| haleyb | remember the neutron and ci meetings next week have moved an hour for DST | 14:43 |
| slaweq | have a nice one | 14:43 |
| haleyb | #endmeeting | 14:43 |
| opendevmeet | Meeting ended Fri Mar 20 14:43:38 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:43 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-03-20-14.00.html | 14:43 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-03-20-14.00.txt | 14:43 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-03-20-14.00.log.html | 14:43 |
| slaweq | o/ | 14:43 |
| lajoskatona | o/ | 14:43 |
| mtomaska | o/ | 14:43 |
| ralonsoh | bye | 14:43 |
| ralonsoh | ichen, https://review.opendev.org/c/openstack/neutron-specs/+/980305 | 14:44 |
| ralonsoh | that was merged 1 week ago | 14:44 |
| zigo | Has this been implemented in Gazpacho, or not yet ? | 14:51 |
| zigo | https://opendev.org/openstack/neutron-specs/src/branch/master/specs/2025.2/ovn-bgp-integration.rst | 14:51 |
| ralonsoh | jlibosva, ^ | 14:52 |
| ralonsoh | there are some missing bits that are almost merged | 14:52 |
| zigo | ralonsoh: Is there hope to have them in a Gazpacho point release then? | 15:12 |
| haleyb | zigo: that is the plan, we (canonical) want them too | 15:13 |
| zigo | Sounds gr8 ! :) | 15:13 |
| ralonsoh | haleyb, hi! So I have the 3rd patch for the L3 OVN scheduler. And the upper patch is testing both affected UTs | 15:39 |
| ralonsoh | and I've also increased the coverage of one of these tests | 15:40 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/981211/ | 15:40 |
| ralonsoh | that needs to be backported to 2026.1 | 15:40 |
| haleyb | ralonsoh: ack, will look | 15:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Update the parent hostname in the virtual ports https://review.opendev.org/c/openstack/neutron/+/981540 | 16:41 |
| ildikov_ | Hi Neutron team | 16:57 |
| ildikov_ | I'm reaching out with a quick question, favor to ask | 16:58 |
| ildikov_ | On the last OpenStack Ops Radio Hour folks brought up OVN as a topic area they would like to learn and talk more about | 16:58 |
| ildikov_ | The next call is on March 27, next Friday, at 1300 UTC | 16:59 |
| ildikov_ | Would any of y'all be available to join the meeting? | 16:59 |
| opendevreview | Chris Buggy proposed openstack/neutron master: Add logging decorator for OVN maintenance methods https://review.opendev.org/c/openstack/neutron/+/981556 | 17:22 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-lib master: Add PVLAN api extension https://review.opendev.org/c/openstack/neutron-lib/+/975931 | 17:58 |
| elodilles | haleyb: it seems that now that devstack, grenade and requirements have branched, the gate is back to how it should look like. see the job list: https://zuul.opendev.org/t/openstack/status?change=981131 | 18:02 |
| haleyb | elodilles: great, jobs are good :) | 18:03 |
| elodilles | :] | 18:03 |
| haleyb | hopefully we didn't merge a bad one | 18:04 |
| elodilles | it will turn out soon :S if needed (let's hope not :S fingers crossed) then a DNM empty patch can be proposed to execercise a bit the gate and see that every job is still passing | 18:06 |
| opendevreview | Merged openstack/neutron unmaintained/2024.1: [OVN] Add a maintenace method to create the AS for each AG https://review.opendev.org/c/openstack/neutron/+/977907 | 20:07 |
| ichen | ralonsoh: thanks for the reference. I’ll take a look. | 20:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: All service plugins should inherit from ``ServicePluginBase`` https://review.opendev.org/c/openstack/neutron/+/978898 | 20:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Fix an issue in the L3 OVN scheduler (3) https://review.opendev.org/c/openstack/neutron/+/981211 | 21:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == DNM == OVN L3 scheduler test (3) https://review.opendev.org/c/openstack/neutron/+/981212 | 21:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "Mark OVN L3 test unstable" https://review.opendev.org/c/openstack/neutron/+/980397 | 21:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "Mark OVN L3 test unstable" https://review.opendev.org/c/openstack/neutron/+/980397 | 21:20 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove sort_ips_by_version() https://review.opendev.org/c/openstack/neutron/+/981614 | 22:48 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove remove_interface_suffix() from ip_lib https://review.opendev.org/c/openstack/neutron/+/981615 | 22:50 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Don't process BGP events without the lock https://review.opendev.org/c/openstack/neutron/+/981616 | 23:04 |
| opendevreview | Merged openstack/neutron master: Use ovn-db-sync-util related constants from neutron-lib https://review.opendev.org/c/openstack/neutron/+/978060 | 23:33 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!