| opendevreview | Merged openstack/neutron master: ovn: Clean up code to handle old versions with no stateless acl support https://review.opendev.org/c/openstack/neutron/+/927634 | 04:59 |
|---|---|---|
| sahid | o/ | 07:20 |
| sahid | quick question guys, do you know in which case the code is trying to get local vlan tag from other_config? | 07:21 |
| lajoskatona | sahid: Hi, you mean for OVS driver? I just quickly grepped so I am sure not a full answer but I found these methods that can fetch tag from other-config: https://opendev.org/openstack/neutron/src/commit/76a1aa36f059fc312fc76380b8bc15fb0f357661/neutron/agent/common/ovs_lib.py#L1271-L1288 | 07:52 |
| lajoskatona | bcafarel: Hi, I found this one (and minustwo) among the grafana yamls: https://opendev.org/openstack/project-config/src/branch/master/grafana/neutron-stable-minusone.yaml , do we need these grafana boards or we have redundant ones somewhere? | 08:10 |
| sahid | lajoskatona: yep thank you, for ovs | 08:23 |
| sahid | i can also see them but I'm not sure in which context they are used | 08:23 |
| sahid | i don't see when we set that in other_config and why | 08:23 |
| ralonsoh | hello lajoskatona! 2 quick reviews, if you have less than 1 min | 09:08 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/930889 | 09:08 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-lib/+/930716 | 09:08 |
| ralonsoh | thanks! | 09:08 |
| opendevreview | Elod Illes proposed openstack/neutron-fwaas unmaintained/zed: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-fwaas/+/921527 | 09:56 |
| opendevreview | Takashi Natsume proposed openstack/neutron master: Fix a wrong assertion method https://review.opendev.org/c/openstack/neutron/+/930995 | 11:03 |
| opendevreview | Merged openstack/neutron master: Disable the OVN/OVS WGSI experimental jobs https://review.opendev.org/c/openstack/neutron/+/930889 | 11:57 |
| opendevreview | Merged openstack/neutron-lib master: Do not define the executor of the RPC server https://review.opendev.org/c/openstack/neutron-lib/+/930716 | 12:20 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: Use tuple type for SUPPORTED_LOGGING_TYPES https://review.opendev.org/c/openstack/neutron/+/929865 | 12:51 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/* https://review.opendev.org/c/openstack/neutron/+/929866 | 12:51 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/db/l3* https://review.opendev.org/c/openstack/neutron/+/929867 | 12:52 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: use dataclass for AgentConsumer https://review.opendev.org/c/openstack/neutron/+/929868 | 12:52 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/extensions/* https://review.opendev.org/c/openstack/neutron/+/929869 | 12:52 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/agent/* https://review.opendev.org/c/openstack/neutron/+/929870 | 12:52 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: mypy: neutron/objects/* https://review.opendev.org/c/openstack/neutron/+/929871 | 12:52 |
| ihrachys | haleyb: what blocks https://review.opendev.org/c/openstack/neutron/+/618208 from merging? I think you said you want this merged, but you haven't voted. | 12:57 |
| haleyb | ihrachys: i haven't voted since i've done all the latest updates | 13:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: [WSGI] Move all OVN jobs to use WSGI API module https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/930743 | 13:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add Neutron API eventlet experimental jobs https://review.opendev.org/c/openstack/neutron/+/931012 | 13:26 |
| ihrachys | haleyb: ah I haven't noticed :) | 13:27 |
| haleyb | that patch is almost as old as neutron :) | 13:30 |
| fungi | ihrachys: was there a reason you marked https://launchpad.net/bugs/2045785 as a potential security vulnerability? if so, please add a comment explaining the reason (and if not, please set it back). thanks! | 13:31 |
| ihrachys | fungi: oops I think I misclicked something | 13:32 |
| fungi | no worries, just checking! | 13:32 |
| opendevreview | Merged openstack/neutron master: Add initial mypy job https://review.opendev.org/c/openstack/neutron/+/929864 | 13:34 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: tests: Expand coverage for ovn_emit_need_to_frag=True https://review.opendev.org/c/openstack/neutron/+/930757 | 13:35 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Make mypy job voting https://review.opendev.org/c/openstack/neutron/+/931013 | 13:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: WIP == Cleanup the OVS DB and log files during the cleanup process https://review.opendev.org/c/openstack/neutron/+/931014 | 13:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Cleanup the OVS DB and log files during the cleanup process https://review.opendev.org/c/openstack/neutron/+/931015 | 13:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM: test functional jobs https://review.opendev.org/c/openstack/neutron/+/931016 | 13:43 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Oct 1 14:00:43 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 |
| lajoskatona | o/ | 14:00 |
| mlavalle | \o | 14:00 |
| ykarel | o/ | 14:00 |
| ralonsoh | hello | 14:00 |
| ihrachys | o/ | 14:00 |
| elvira | o/ | 14:01 |
| obondarev | o/ | 14:01 |
| rubasov | o/ | 14:01 |
| slaweq | o/ | 14:01 |
| haleyb | #topic announcements | 14:01 |
| haleyb | hi everyone | 14:01 |
| bcafarel | o/ | 14:02 |
| cbuggy | o/ | 14:02 |
| haleyb | Neutron RC2 was released last week, and will be the final Dalmatian RC | 14:02 |
| haleyb | Final 2024.2 Dalmatian release: October 2nd, 2024 (tomorrow) | 14:02 |
| haleyb | so give yourself a high five (my dog just gave me one) | 14:03 |
| slaweq | o/\o | 14:03 |
| ralonsoh | congrats everybody! | 14:03 |
| haleyb | thanks for all the hard work! | 14:04 |
| haleyb | Master branch has switched to 2025.1 Epoxy development | 14:04 |
| lajoskatona | +1 | 14:05 |
| haleyb | Epoxy cycle PTG sessions (October 21-25) | 14:05 |
| haleyb | https://etherpad.opendev.org/p/oct2024-ptg-neutron | 14:05 |
| haleyb | Please add topics and I'll organize them the week before | 14:05 |
| haleyb | I will sign up for time slots this week | 14:06 |
| haleyb | Like last PTG, I will work around TC timeslot, etc | 14:06 |
| haleyb | Will also have a Nova and Ironic interlock, the latter being about adding another dhcp implementation | 14:07 |
| ihrachys | deadline for topics? | 14:07 |
| haleyb | ihrachys: earlier the better, but the Friday before is the deadline for organizational sake. we can have an on-demand section at the end, but sometimes people wander away early | 14:08 |
| haleyb | For your planning purposed, I will stick to the 13:00-17:00 UTC timeframe, and probably just Tuesday/Wednesday/Thursday unless we have a ton of topics | 14:10 |
| haleyb | Reminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:11 |
| haleyb | We do have a topic for this week | 14:12 |
| lajoskatona | haleyb: thanks for PTG scheduling the PTG | 14:12 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2083214 | 14:12 |
| haleyb | lajoskatona: thank me after i remember how to talk to the event bot :) | 14:13 |
| lajoskatona | haleyb :-) | 14:14 |
| haleyb | Let's continue to use the priorities dashboard for patches in the "ready to merge" state. This could be older changes as well as new ones. It seems to have worked out well the past month | 14:14 |
| haleyb | That was all the announcements I had, any others? | 14:15 |
| haleyb | #topic bugs | 14:16 |
| haleyb | obondarev was the deputy this week, and we seemed to have a lot of bugs :( | 14:16 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/WHX3AS35SFFFDGRREWHYMOKKYI2IQFUM/ | 14:16 |
| haleyb | ralonsoh: thanks for picking up a lot of them | 14:17 |
| haleyb | lets talk about the unassigned ones | 14:17 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2081868 | 14:17 |
| haleyb | unit test bug in logging api | 14:18 |
| ralonsoh | I think slaweq was planning to push a patch | 14:18 |
| ralonsoh | at least adding more information in the exception | 14:18 |
| ralonsoh | instead of the webob.exc.HTTPClientError vague exception | 14:19 |
| slaweq | yes but only to log a bit more about the original response | 14:19 |
| slaweq | maybe that will help understand what happened there | 14:19 |
| lajoskatona | we discussed this one yesterday on the CI meeting or am I mess it with something else? | 14:19 |
| slaweq | lajoskatona yes, we did | 14:19 |
| haleyb | ack, i was not able to attend, the first three might have been discussed there | 14:20 |
| haleyb | next one then | 14:20 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2081870 | 14:20 |
| haleyb | functional test failure respawning children | 14:20 |
| haleyb | i don't see that in the CI minutes | 14:21 |
| ykarel | yes only 1 of those 3 was discussed in CI meeting | 14:22 |
| haleyb | ykarel: thanks | 14:23 |
| haleyb | looking at opensearch link, this one has only happened twice in the past 30 days | 14:23 |
| ralonsoh | this is the same execution, only once | 14:24 |
| ralonsoh | I would lower a bit the importance | 14:25 |
| haleyb | ok, it showed 2 hits at the top | 14:25 |
| haleyb | alright, will monitor that one | 14:26 |
| haleyb | next one | 14:26 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2081871 | 14:27 |
| haleyb | fullstack test failure | 14:27 |
| ralonsoh | I'll check the logs | 14:27 |
| haleyb | thanks ralonsoh | 14:28 |
| haleyb | i think all the rest on the list have owners and/or patches, so thanks everyone for that | 14:29 |
| haleyb | the one new RFE there - new DHCP backend for Ironic - I might delay until PTG interlock | 14:30 |
| haleyb | any other bugs to discuss? | 14:30 |
| haleyb | This week ralonsoh is the bug deputy, next week will be lajoskatona | 14:30 |
| haleyb | Is that good for both? | 14:31 |
| ralonsoh | I didn't check the list, thanks for the heads-up | 14:31 |
| haleyb | Current bug count this week: 734, up 12 from last week, so trending up slightly | 14:32 |
| lajoskatona | +1 from me | 14:32 |
| haleyb | great, thanks | 14:33 |
| haleyb | #topic community-goals | 14:33 |
| haleyb | eventlet deprecation | 14:33 |
| haleyb | #link https://review.opendev.org/q/topic:%22bug/2069581%22 | 14:33 |
| ralonsoh | I'm doing an internal document for Red Hat | 14:34 |
| ralonsoh | related to this topic | 14:34 |
| ralonsoh | once I have it and because all these topics should be addressed in U/S, I'll share it | 14:34 |
| ralonsoh | I have one big concern: os-ken, that is 100% based in eventlet | 14:34 |
| ralonsoh | (that affects the ovs-agent) | 14:34 |
| ralonsoh | I'll also add the corresponding topics in the PTG agenda | 14:35 |
| ralonsoh | that's all | 14:35 |
| haleyb | ralonsoh: thanks. are the patches in that topic ready to merge once they're green? | 14:35 |
| ralonsoh | yes | 14:35 |
| haleyb | ok, am watching them, thanks for the work | 14:36 |
| haleyb | neutronclient deprecation | 14:37 |
| haleyb | #link https://review.opendev.org/q/topic:%22bug/1999774%22 | 14:37 |
| haleyb | lajoskatona has been working through this | 14:38 |
| lajoskatona | I have to jump on it again, but yes I am planning to allocate time to it in this cycle also | 14:38 |
| haleyb | lajoskatona: thanks - i see your last comment - "if you have time don't hesitate to check and fix the failures" | 14:39 |
| haleyb | if i have time before next meeting i'll see if i can pull and poke at a unit test failure | 14:40 |
| haleyb | #topic on-demand | 14:41 |
| haleyb | there was nothing in the wiki, but if you have something to discuss now is the time | 14:41 |
| haleyb | i will give a plug to the mypy changes ihrachys is working on | 14:42 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/929864 | 14:42 |
| haleyb | there is a number in that series, good to get started on that | 14:43 |
| haleyb | that was all i had | 14:43 |
| ihrachys | this is just getting mypy covering all files. but we can now start adding hints to new code or code we touch. | 14:43 |
| ralonsoh | +1 to this (better late than never) | 14:43 |
| ihrachys | per se without the latter step, the job won't give too much value :) | 14:43 |
| ihrachys | I can do the same for neutron-lib in next week or two, I have some old patches somewhere, was just hoping Miro would take care of it since he expressed interest in the prev ptg | 14:44 |
| haleyb | ihrachys: there was an initial series linked from the meeting wiki if that helps | 14:45 |
| ihrachys | I can check; just give me a nod that I should take it over ;) | 14:45 |
| haleyb | i also had a larger RST linting patch that should get merged, it just needs a rebase as it touches a lot of the doc/* files | 14:46 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/930089 | 14:46 |
| haleyb | will update after meeting | 14:46 |
| ihrachys | thanks and sorry for the hassle; will +2 asap | 14:46 |
| haleyb | ihrachys: np, some of those code-style types were incorrect, even if they all seem to render similarly | 14:47 |
| ihrachys | yeah but I guess we may handle any other minor things when the bulk lands. | 14:47 |
| haleyb | luckily rebases are pretty easy on docs | 14:48 |
| haleyb | ok, we can get :12 minutes back, thanks for attending everyone | 14:49 |
| ihrachys | o/ | 14:49 |
| haleyb | #endmeeting | 14:49 |
| opendevmeet | Meeting ended Tue Oct 1 14:49:12 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:49 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-10-01-14.00.html | 14:49 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-10-01-14.00.txt | 14:49 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-10-01-14.00.log.html | 14:49 |
| ralonsoh | bye | 14:49 |
| ykarel | o/ | 14:49 |
| mlavalle | \o | 14:49 |
| lajoskatona | o/ | 14:49 |
| obondarev | o/ | 14:49 |
| ihrachys | ykarel: ralonsoh since you are the CI czars ;) please take a look at https://review.opendev.org/c/openstack/neutron/+/927843 and let me know if you think this is helpful. tl;dr I'd like to surface integration issues with ovs / ovn master patches earlier through periodics (at least) | 14:49 |
| slaweq | o/ | 14:50 |
| ralonsoh | let me check | 14:50 |
| ralonsoh | ihrachys, yes, I agree with this change, that will help us with integration issues with ovs/ovn | 14:51 |
| ykarel | ihrachys, LGTM, that was the plan with original change when those were added to experimental | 14:51 |
| lajoskatona | ihrachys: +1, good idea | 14:51 |
| ihrachys | yay, thank you guys!! | 14:53 |
| ykarel | lajoskatona, i did some comments on too many open files debug patches, if you can try those | 14:55 |
| lajoskatona | ykarel: thanks, I check it | 14:59 |
| opendevreview | Brian Haley proposed openstack/neutron master: Add RST linting to neutron https://review.opendev.org/c/openstack/neutron/+/930089 | 15:21 |
| opendevreview | Merged openstack/neutron master: Enable liberal TCP connection tracking for SNAT namespaces https://review.opendev.org/c/openstack/neutron/+/618208 | 15:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Cleanup the OVS DB and log files during the cleanup process https://review.opendev.org/c/openstack/neutron/+/931015 | 16:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM: test functional jobs https://review.opendev.org/c/openstack/neutron/+/931016 | 16:07 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add logging details of the http response in the unit tests https://review.opendev.org/c/openstack/neutron/+/931036 | 16:11 |
| opendevreview | Merged openstack/neutron master: zuul: Move ovs-master and ovs-lts jobs to periodic queue https://review.opendev.org/c/openstack/neutron/+/927843 | 17:02 |
| ihrachys | have we seen unit test timeouts lately in zuul? I just hit one for py312 and it seems like a legit one - test run took 1h+ https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_09a/930608/2/check/openstack-tox-py312/09a3686/job-output.txt | 17:04 |
| haleyb | ihrachys: i haven't seen one yet, but might need to tweak job-templates.yaml where we override the timeout for that job, think it should be 3600 seconds currently | 17:59 |
| haleyb | i see it runs it on Ubuntu Noble as well, so more than just a python change | 18:13 |
| opendevreview | Merged openstack/neutron-fwaas unmaintained/zed: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-fwaas/+/921527 | 21:05 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Set distributed flag to NB_Global https://review.opendev.org/c/openstack/neutron/+/931067 | 21:32 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Use set_nb_global_options in maintenance task https://review.opendev.org/c/openstack/neutron/+/931068 | 21:35 |
| opendevreview | Merged openstack/tap-as-a-service master: Generate plugin config file https://review.opendev.org/c/openstack/tap-as-a-service/+/929655 | 21:59 |
| opendevreview | Merged openstack/neutron master: tests: Expand coverage for ovn_emit_need_to_frag=True https://review.opendev.org/c/openstack/neutron/+/930757 | 21:59 |
| opendevreview | Merged openstack/neutron master: Add logging details of the http response in the unit tests https://review.opendev.org/c/openstack/neutron/+/931036 | 21:59 |
| opendevreview | Merged openstack/neutron unmaintained/zed: Handle creation of Port_Binding with chassis set https://review.opendev.org/c/openstack/neutron/+/926666 | 23:03 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!