| opendevreview | Miro Tomaska proposed openstack/neutron master: FRR driver for a EVPN router https://review.opendev.org/c/openstack/neutron/+/988158 | 01:09 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: WIP: Add project_id attributes for neutron-lib change https://review.opendev.org/c/openstack/neutron/+/986962 | 02:02 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: WIP: Tenant_id has left the building, part 1 https://review.opendev.org/c/openstack/neutron-lib/+/986960 | 02:08 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix issues in _check_router_interface_not_in_use() https://review.opendev.org/c/openstack/neutron/+/991002 | 02:43 |
| opendevreview | Helen Chen proposed openstack/neutron master: Created Single VxLAN Device management for EVPN https://review.opendev.org/c/openstack/neutron/+/990169 | 03:58 |
| opendevreview | Merged openstack/neutron master: bgp: Create new InterconnectBridge https://review.opendev.org/c/openstack/neutron/+/988701 | 05:01 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change _set_router_states() to use a single DB transaction https://review.opendev.org/c/openstack/neutron/+/991015 | 05:43 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: WIP: neutron-ovn-db-sync-util plugin integration https://review.opendev.org/c/openstack/ovn-octavia-provider/+/990918 | 05:44 |
| opendevreview | Merged openstack/neutron master: Fix race condition in worker lifecycle probe https://review.opendev.org/c/openstack/neutron/+/990917 | 06:32 |
| *** ralonsoh_ is now known as ralonsoh | 07:05 | |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2026.1: [OVS FW] Clean invalid conntrack entries after OF rules are applied https://review.opendev.org/c/openstack/neutron/+/991021 | 07:29 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2025.2: [OVS FW] Clean invalid conntrack entries after OF rules are applied https://review.opendev.org/c/openstack/neutron/+/991022 | 07:30 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2025.1: [OVS FW] Clean invalid conntrack entries after OF rules are applied https://review.opendev.org/c/openstack/neutron/+/991023 | 07:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add ``update_l3-router`` policy https://review.opendev.org/c/openstack/neutron/+/990928 | 08:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add ``l3-agent-scheduler-ha-priority`` extension for ML2/OVN https://review.opendev.org/c/openstack/neutron/+/982792 | 08:00 |
| ralonsoh | hi folks, a document patch: https://review.opendev.org/c/openstack/neutron/+/990906 | 08:04 |
| ralonsoh | it updates the OVN extension list | 08:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Implement router_route quota using the Neutron quota engine https://review.opendev.org/c/openstack/neutron/+/991024 | 08:09 |
| opendevreview | Merged openstack/neutron master: ovn: Add subnet_onboard API extension support https://review.opendev.org/c/openstack/neutron/+/990904 | 09:19 |
| opendevreview | Merged openstack/neutron master: Add docs and typing to agent extension events https://review.opendev.org/c/openstack/neutron/+/990992 | 09:19 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 09:29 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [Tap-as-a-Service] Adding 2 Tap as a service test cases. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/989931 | 09:30 |
| opendevreview | Merged openstack/neutron master: ovn: batch subnet queries in _get_nets_and_ipv6_ra_confs_for_router_port https://review.opendev.org/c/openstack/neutron/+/990477 | 10:12 |
| opendevreview | Merged openstack/neutron master: Fix create_agent policy operations path https://review.opendev.org/c/openstack/neutron/+/990295 | 10:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Implement router_route quota using the Neutron quota engine https://review.opendev.org/c/openstack/neutron/+/991024 | 11:11 |
| opendevreview | Merged openstack/neutron stable/2026.1: Fix tunnel network detection to use tunnel type driver ranges https://review.opendev.org/c/openstack/neutron/+/990602 | 11:12 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [Tap-as-a-Service] Adding VM migration scenario test. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991062 | 11:21 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Fixture for FrrService https://review.opendev.org/c/openstack/neutron/+/989475 | 11:32 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Reqs: Add dependent libs to requirements.txt https://review.opendev.org/c/openstack/tap-as-a-service/+/990970 | 11:41 |
| opendevreview | Merged openstack/neutron master: Use models_v2.Port.id in port query https://review.opendev.org/c/openstack/neutron/+/990866 | 12:20 |
| opendevreview | Merged openstack/neutron master: Use Neutron FIP UUID as OVN ``NAT`` row UUID https://review.opendev.org/c/openstack/neutron/+/984408 | 12:22 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add whitebox-neutron-tempest-plugin tests to BGP job https://review.opendev.org/c/openstack/neutron/+/990644 | 12:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas master: Introduce OVO objects representing FWaaS resources https://review.opendev.org/c/openstack/neutron-fwaas/+/990930 | 12:36 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 12:45 |
| haleyb | #startmeeting networking | 13:00 |
| opendevmeet | Meeting started Tue Jun 2 13:00:27 2026 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| opendevmeet | The meeting name has been set to 'networking' | 13:00 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, slaweq, ykarel, lajoskatona, jlibosva, haleyb, ralonsoh | 13:00 |
| mlavalle | \o | 13:00 |
| mtomaska | o/ | 13:00 |
| lajoskatona | o/ | 13:01 |
| haleyb | alright we can get started | 13:02 |
| haleyb | #topic announcements | 13:02 |
| haleyb | We are in week R-17 of Hibiscus | 13:02 |
| haleyb | #link https://releases.openstack.org/hibiscus/schedule.html | 13:02 |
| haleyb | reminder to please use the priority dashboard for anything neutron/networking related | 13:02 |
| haleyb | #link https://tinyurl.com/59z278km | 13:03 |
| slaweq | o/ | 13:03 |
| haleyb | RP +1 for "ready to merge" changes, RP +2 for gate blockers and similar | 13:03 |
| haleyb | there is one RP+2 in that list for 2024.1 that has merged in other branches | 13:03 |
| haleyb | there is another ~10 RP +1 there as well | 13:05 |
| haleyb | that was all i had for announcements, is there anything else? | 13:05 |
| ralonsoh | hello | 13:05 |
| haleyb | hi ralonsoh | 13:06 |
| haleyb | ok, we can move onto bugs | 13:06 |
| haleyb | #topic bugs | 13:06 |
| haleyb | elvira was the deputy last week, her report is at | 13:06 |
| haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/YIDSM5MOJ5LXPST4BBWLSRXYDAN566OP/ | 13:06 |
| haleyb | most have patches proposed | 13:07 |
| haleyb | the only one unassigned is this one | 13:08 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2154368 | 13:08 |
| haleyb | Migrate remaining Neutron extensions from ``ExtensionDescriptor`` to ``APIExtensionDescriptor`` | 13:08 |
| ralonsoh | not important | 13:08 |
| ralonsoh | nice to have but low priority | 13:08 |
| haleyb | that was it for new bugs, are there any other to discuss? | 13:10 |
| ralonsoh | yes, one sec | 13:10 |
| ralonsoh | If you want to use it: https://github.com/ralonsoh/cursor-config/tree/main/skills | 13:10 |
| ralonsoh | skill openstack-neutron-lp-bug-triage | 13:10 |
| ralonsoh | for lazy people (like me) | 13:10 |
| ralonsoh | try it: "do a bug triage from May 25 to May 30" | 13:11 |
| ralonsoh | that's all | 13:11 |
| haleyb | thanks i will take a look, and this probably explains the 100 bugs that were closed over the past 1-2 weeks? | 13:11 |
| ralonsoh | I used another skill | 13:12 |
| ralonsoh | just to find released, stuck or unatended bugs | 13:12 |
| haleyb | it is good to be under 600 bugs | 13:14 |
| haleyb | so this week slaweq is the bug deputy, i will be next week | 13:14 |
| haleyb | still ok for you slaweq ? | 13:14 |
| ralonsoh | I'll ping him later today | 13:15 |
| haleyb | ack, thanks | 13:15 |
| slaweq | yeap, all good | 13:15 |
| haleyb | great | 13:15 |
| slaweq | (I replied in other channel for that :P) | 13:15 |
| ralonsoh | hehehe | 13:16 |
| haleyb | #topic specs | 13:16 |
| haleyb | #link https://review.opendev.org/q/project:openstack/neutron-specs+status:open | 13:16 |
| haleyb | ralonsoh: i did respond to your comment in this one | 13:17 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron-specs/+/987786 | 13:17 |
| haleyb | Add spec for Port Groups for Security Group Rules | 13:17 |
| ralonsoh | yeah, I think a CLI "compress" tool would be nice | 13:17 |
| ralonsoh | or we can even create another command | 13:18 |
| haleyb | but would that fix the "create" side? | 13:18 |
| ralonsoh | I think there is no issue in the create side | 13:18 |
| ralonsoh | there are 2 problems in this spec: | 13:18 |
| ralonsoh | the OVN optimization: very cool, nice to have | 13:18 |
| ralonsoh | the user handling of SG rules, associated to the same protocol, direction, etc but with multiple ports or port ranges | 13:19 |
| ralonsoh | so, IMO, we can implement this spec in 2 parts: OVN and CLI/user friendly optimization | 13:19 |
| ralonsoh | but I would prefer not modifying the DB | 13:20 |
| ralonsoh | in particular to NOT include this string field with port + ranges | 13:20 |
| haleyb | but there was a part of the spec discussing adding ports/port-ranges i thought, or did i mis-understand? | 13:20 |
| ralonsoh | yes, to create a new table, something like ports and port-ranges | 13:21 |
| ralonsoh | and these to be resusable for the SG rules | 13:21 |
| ralonsoh | so you can create {port=1, port=2, port=10-20} | 13:21 |
| ralonsoh | and then use these 3 registers for multiple SG rules | 13:21 |
| ralonsoh | that's the second alternative | 13:22 |
| ralonsoh | the first is to create a SG rule with [string]1,2,10-20 | 13:22 |
| ralonsoh | that's ^^ horrible from a DB point of view | 13:22 |
| slaweq | I need to read that specs again but at first glance I like the idea of having new API resource "PortGroup" | 13:22 |
| haleyb | right, but does that create multiple SG rules? | 13:22 |
| slaweq | we already have "AddressGroup" IIRC so PortGroup would be something similar and IMHO complementatory to AddressGroup | 13:22 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [Tap-as-a-Service] Adding VM migration scenario test. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991062 | 13:23 |
| ralonsoh | haleyb, no, the point of having this PortGroup is to have 1 single SG rule associated to multiple PortGroups | 13:23 |
| ralonsoh | and the PortGroups being available for multuple SG rules | 13:24 |
| ralonsoh | instead of defining the port ranges every time | 13:24 |
| ralonsoh | I'm ok with this optimization | 13:24 |
| ralonsoh | (that implies refactoring the OVS backend, unraveling the SG rules in multiple SG rules) | 13:25 |
| haleyb | ralonsoh: but that requires a DB change, correct? | 13:25 |
| ralonsoh | yes | 13:25 |
| ralonsoh | but not doing SG ports --> [string]1,2,10-20 | 13:25 |
| ralonsoh | but instead SG ports --> PortGroup1, PortGroup2, etc | 13:25 |
| haleyb | ok, so maybe we can add these comments to the spec | 13:28 |
| haleyb | and get other reviews | 13:28 |
| ralonsoh | I'll do | 13:28 |
| haleyb | thanks | 13:29 |
| haleyb | ralonsoh: and i saw you had a small update to the SG default statefulness spec | 13:29 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron-specs/+/990606 | 13:29 |
| haleyb | slaweq: can you look at that one as well ^^ | 13:30 |
| ralonsoh | yeah, because I'm implementing that, so it makes sense to have it in the spec | 13:30 |
| ralonsoh | slaweq, added this comment, I think he will be ok with it | 13:30 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 13:30 |
| haleyb | ack, thanks | 13:30 |
| haleyb | any other specs to discuss? | 13:31 |
| slaweq | @ralonsoh: you mean https://review.opendev.org/c/openstack/neutron-specs/+/990606 ? | 13:31 |
| ralonsoh | yes | 13:31 |
| 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 | 13:31 |
| slaweq | yeah, it's good IMO | 13:31 |
| slaweq | thx | 13:31 |
| haleyb | #link https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 13:31 |
| haleyb | i do see there is one item for discussion this friday | 13:32 |
| ralonsoh | I added a topic, not urgent at all | 13:32 |
| ralonsoh | we can postpone it if needed | 13:32 |
| ralonsoh | there are patches in gerrit if you want to check it (neutron, sdk and cli) | 13:32 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2026489 | 13:33 |
| ralonsoh | and #link https://review.opendev.org/q/topic:%22bug/2026489%22 | 13:33 |
| haleyb | ok, let's see if there are other RFEs to discuss, that looks like a quick one | 13:34 |
| haleyb | #topic community goals | 13:35 |
| haleyb | there are three active series | 13:35 |
| haleyb | #link https://review.opendev.org/q/topic:%22sdk_for_neutron%22 | 13:35 |
| haleyb | #link https://review.opendev.org/q/topic:%2522migrate_stadium_osc%2522 | 13:35 |
| haleyb | #link https://review.opendev.org/q/topic:%22neutron_openstacksdk%22 | 13:36 |
| lajoskatona | small progress last week/this week for the 1st and 3rd one | 13:36 |
| ralonsoh | I'll check the openstackclient and neutronclient ones tomorrow morning | 13:36 |
| lajoskatona | and for 3rd topic for migrating Heat to openstackdk instead of neutronclient, I had a short chat with tkajinam and he try to have some time to review those patches in this cycle | 13:37 |
| lajoskatona | ralonsoh: thanks | 13:37 |
| lajoskatona | so for Heat I also try to push more patches to and hope for review | 13:38 |
| lajoskatona | that's it from me for the migrate away from neutronclient topic | 13:38 |
| haleyb | ack will try and take a look at those | 13:39 |
| lajoskatona | thanks | 13:39 |
| lajoskatona | small thing for the Heat topic: | 13:39 |
| lajoskatona | if we cant have team for Heat, as a community we should decide to have something instead of it, like ORC (https://github.com/k-orc/openstack-resource-controller ) but that is k8s based (kind of a love project between k8s and Openstack) and I am not sure if that is production ready | 13:41 |
| lajoskatona | if you check i.e. networking a lot of things are not yet ready | 13:41 |
| lajoskatona | but if most people use openstack together with k8s perhaps that's the way forward | 13:41 |
| lajoskatona | perhpas worth a question in the user survey :-) | 13:42 |
| ralonsoh | https://github.com/openstack-k8s-operators | 13:42 |
| ralonsoh | ^^ check this if you like OS to deploy OpenStack | 13:42 |
| lajoskatona | thanks, I will check it | 13:42 |
| slaweq | if you want to play with openstack-k8s-operators in "devstack style", you can check https://github.com/openstack-k8s-operators/install_yamls/ :) | 13:44 |
| haleyb | lajoskatona: is it just that things take a long time to merge in heat? | 13:44 |
| slaweq | it will help you install CRC and then that openstack on top of it | 13:44 |
| lajoskatona | haleyb: if you check the statistics for Heat tkajinam is the most active maintainer who works o zillions of other projects | 13:45 |
| lajoskatona | and perhaps one more guy sometimes go to do small things | 13:45 |
| opendevreview | Helen Chen proposed openstack/neutron master: Integrated SVD into OVN Agent EVPN Extension https://review.opendev.org/c/openstack/neutron/+/991099 | 13:45 |
| haleyb | ah, ok | 13:45 |
| haleyb | ok, we can move on | 13:46 |
| haleyb | #topic on-demand | 13:46 |
| ralonsoh | all good, thanks | 13:47 |
| slaweq | just a quick one | 13:47 |
| slaweq | I also wanted to share skill which I am using since few days already: https://github.com/slawqo/ansible/blob/ai_tools/roles/ai_tools/templates/skills/make-reviews-plan/SKILL.md.j2 - this helps me to maintain todo list of the patches to review with some priorities set. Maybe will be useful for someone :) | 13:47 |
| ralonsoh | ++ | 13:48 |
| lajoskatona | thanks | 13:48 |
| haleyb | thanks | 13:49 |
| haleyb | i had one topic as well | 13:49 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/989565 | 13:50 |
| haleyb | this is something one of my colleagues has been working on | 13:50 |
| haleyb | Denote highest priority chassis after crash and BFD failover | 13:50 |
| haleyb | this is something we are testing in a large caracal deployment | 13:51 |
| ralonsoh | I would need to review that with care | 13:52 |
| haleyb | ralonsoh: yeah, i didn't know how that fit into your recent work in the area | 13:52 |
| ralonsoh | so, if I'm not wrong, that changes the priorities once failed | 13:53 |
| ralonsoh | that's a change in the current behaviour | 13:53 |
| ralonsoh | because we expect the LRP to go back to the previously assigned chassis | 13:54 |
| ralonsoh | but we can discuss it in the patch | 13:54 |
| haleyb | yes, it demotes so that things don't automatically fail-back, causing a hiccup | 13:55 |
| haleyb | but yes, we can discuss in the patch | 13:55 |
| haleyb | alright, any other topics? | 13:56 |
| haleyb | ok, thanks for attending everyone, have a good week | 13:57 |
| haleyb | #endmeeting | 13:57 |
| opendevmeet | Meeting ended Tue Jun 2 13:57:05 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:57 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2026/networking.2026-06-02-13.00.html | 13:57 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2026/networking.2026-06-02-13.00.txt | 13:57 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2026/networking.2026-06-02-13.00.log.html | 13:57 |
| mlavalle | \o | 13:57 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Implement multiple segments per host for routed networks https://review.opendev.org/c/openstack/neutron/+/975111 | 13:57 |
| lajoskatona | o/ | 13:58 |
| slaweq | o/ | 13:59 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Agent extension for segment bridges https://review.opendev.org/c/openstack/neutron/+/982018 | 14:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Implement router_route quota using the Neutron quota engine https://review.opendev.org/c/openstack/neutron/+/991024 | 14:10 |
| opendevreview | Merged openstack/tap-as-a-service master: Reqs: Add dependent libs to requirements.txt https://review.opendev.org/c/openstack/tap-as-a-service/+/990970 | 14:12 |
| zigo | ralonsoh: Replied to your review for https://review.opendev.org/c/openstack/neutron/+/836140 | 14:12 |
| zigo | Let me know. | 14:12 |
| ralonsoh | slaweq, I think it was slaweq | 14:16 |
| slaweq | zigo: I just replied to your comments there | 14:20 |
| opendevreview | Pierre Riteau proposed openstack/neutron-tempest-plugin master: WIP: Use branch constraints for tempest venv on stable/2025.1 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991115 | 14:21 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 14:27 |
| stephenfin | cardoe: I was out Friday and Monday. You can use py.typed even if it's not fully typed, yes, and you probably *want* to do that as without it, none of the hints will be picked up elsewhere. | 14:33 |
| stephenfin | SDK has a py.typed file since 2019 | 14:34 |
| stephenfin | ...and that's still not fully typed (but we're close) | 14:35 |
| zigo | slaweq: Thanks, though I have no idea what that line is for: | 14:44 |
| zigo | keepalived_state_change_env: EnvFilter, env, root, PROCESS_TAG=, neutron-keepalived-state-change | 14:44 |
| zigo | What does PROCESS_TAG= does?!? | 14:44 |
| zigo | (I just removed line 54 as you suggested) | 14:45 |
| ralonsoh | zigo, btw, now I realize, this patch (or a follow-up one), should handle the upgrade process: that means deleting any copy of the python script | 14:46 |
| ralonsoh | we discussed about it in the Neutron meeting but I never added this comment in the patch | 14:46 |
| zigo | How do you suggest this would be done? | 14:46 |
| opendevreview | Thomas Goirand proposed openstack/neutron master: Replace keepalived-state-change by a dash script https://review.opendev.org/c/openstack/neutron/+/836140 | 14:47 |
| ralonsoh | zigo, hmmm no idea to be honest. But we need somehow to, when the L3 agent is restarted with the new code, be aware of any present python running script and stop it | 14:48 |
| zigo | I'm not sure how to detect the old process type, unless doing a very weak ps | grep python3 something. | 14:48 |
| ralonsoh | by default, if the L3 agent is stopped correctly, the python script should be stopped | 14:48 |
| zigo | Nope. | 14:48 |
| ralonsoh | but we sometimes don't stop gracefully | 14:48 |
| zigo | I don't think that's the case. | 14:48 |
| zigo | Not with KillMode=process in the .service, which is what I've done in my packaging. | 14:49 |
| zigo | Without that, then yeah, it kills all the processes, but that includes keepalived, which is *very* bad in production. | 14:49 |
| zigo | I believe Ubuntu lacks the feature btw. | 14:50 |
| ralonsoh | ah nonono, we don't stop the keepalived nor the keepalived-state-change scripts | 14:50 |
| ralonsoh | so in this case it is mandatory to have "something" to monitor for the old python script | 14:51 |
| ralonsoh | and stop them | 14:51 |
| zigo | Right, but system does if you don't have KillMode=process in the .service: systemd kills the whole cgroup, which is what is happening under Ubuntu. | 14:51 |
| slaweq | zigo: that is EnvFilter which is used to allow passing additonal env vars to the process, see https://docs.openstack.org/oslo.rootwrap/5.4.0/usage.html#envfilter - so that rule is just to allow passing PROCESSTAG env var to that process. IMHO this can be removed or you should keep/adjust it and add support for that PROCESSTAG variable in your script. | 14:51 |
| zigo | slaweq: Ok, thanks. FYI, I just removed it. | 14:52 |
| ralonsoh | zigo, I would need to check that locally | 14:52 |
| zigo | slaweq: Do you have any idea how we could find out if we're running under the old python3 script ? | 14:52 |
| ralonsoh | in any case, this is not happening in every OS, I guess | 14:53 |
| ralonsoh | the PID should be present in the PID file | 14:53 |
| zigo | ralonsoh: Not under Debian, because we had the issue in production, and I had to fix it ! :) | 14:53 |
| slaweq | but so far it seems to be used only by TripleO so maybe it is not needed anymore | 14:53 |
| slaweq | To better understand what that var can be used for you can check patch https://github.com/openstack/neutron/commit/3d575f8bd066ce2eb46353a49a8c6850ba9e4387 which added it | 14:53 |
| ralonsoh | so if you read the PID file and this PID is still alive, before starting the new bash script, then we need to kill it | 14:53 |
| zigo | Yeah, the PID file would be there, but how to check what interpreter is in use? | 14:54 |
| zigo | Oh, it wont start a new process if the old Python3 script is running. | 14:54 |
| ralonsoh | yeah, if we use process monitor | 14:54 |
| zigo | That's the fun part: after an upgrade, the python3 script still runs from the server memory, even though the /usr/bin script is gone. | 14:54 |
| ralonsoh | zigo, this is what claude says: | 14:57 |
| ralonsoh | https://paste.opendev.org/show/bzrEtbQTaGu7xaqVHhDi/ | 14:57 |
| opendevreview | Pierre Riteau proposed openstack/neutron-tempest-plugin master: WIP: Use branch constraints for tempest venv on stable/2025.1 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991115 | 14:58 |
| zigo | slaweq: ralonsoh: Would that work if I was to provide a script for operator that would simply kill all currently running keepalive-state-change script? | 14:59 |
| zigo | The l3-agent will automatically respawn a new one, that'd be IMO ok. | 14:59 |
| zigo | What I want to avoid: making a loop that is always constantly checking if we have upgraded... | 15:00 |
| ralonsoh | we can have this check for 4 releases, 2 years | 15:00 |
| ralonsoh | I would prefer the check and not always restart the script | 15:00 |
| ralonsoh | it would be faster | 15:00 |
| zigo | Yeah, but it ony needs to be ran *ONCE* after the upgrade. | 15:00 |
| ralonsoh | no, any time the agent is restarted for 4 releases | 15:01 |
| zigo | Oh, I get it: you wish me to only check at startup ? | 15:01 |
| ralonsoh | to guarantee that anyone that upgrades the agent kills the python scrupt | 15:01 |
| ralonsoh | yes | 15:01 |
| zigo | Makes sense, would work. I'll see what I can do. Any pointer where in the source tree I should do it? | 15:01 |
| zigo | (if you don't have it from top of your head, I'll find out) | 15:02 |
| ralonsoh | I would need to check the code, I can give you an answer tomorrow, I'll need to leave in 5 mins | 15:03 |
| zigo | Don't bother, I'll find out myself ! :) | 15:03 |
| zigo | Thanks for your feedback. | 15:03 |
| opendevreview | Michal Nasiadka proposed openstack/neutron stable/2025.1: DNM: test 2025.1 https://review.opendev.org/c/openstack/neutron/+/990240 | 15:09 |
| opendevreview | Michal Nasiadka proposed openstack/neutron-tempest-plugin master: Pin 2025.1 jobs to tempest 46.3.0 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991129 | 15:15 |
| opendevreview | Mohsen Sepandar proposed openstack/neutron-fwaas master: Enable fwaas_v2 driver in DVR mode https://review.opendev.org/c/openstack/neutron-fwaas/+/989857 | 15:22 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [Tap-as-a-Service] Adding VM migration scenario test. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991062 | 15:23 |
| opendevreview | Maor Blaustein proposed openstack/neutron-tempest-plugin master: Migrate QoS with placement tests from tempest https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984558 | 15:27 |
| opendevreview | Thomas Goirand proposed openstack/neutron master: Replace keepalived-state-change by a dash script https://review.opendev.org/c/openstack/neutron/+/836140 | 15:58 |
| zigo | Did it, now need to test it, that's going to be for my tomorrow! :) | 15:59 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Periodic: remove py310 job https://review.opendev.org/c/openstack/tap-as-a-service/+/990972 | 16:11 |
| lajoskatona | haleyb, ralonsoh, mtomaska: if you have time could you please check these taas patches: https://review.opendev.org/q/project:openstack/tap-as-a-service+status:open+-age:2week (only 2 and one is just job twidling) | 16:13 |
| opendevreview | Fiorella Yanac proposed openstack/neutron-tempest-plugin master: Add PVLAN extension API tempest tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/990739 | 16:21 |
| mtomaska | lajoskatona: ACK | 16:21 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add devstack_controller option for remote tempest runners https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/990953 | 16:32 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add devstack_controller option for remote tempest runners https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/990953 | 16:51 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add whitebox-neutron-tempest-plugin tests to BGP job https://review.opendev.org/c/openstack/neutron/+/990644 | 16:51 |
| opendevreview | Fiorella Yanac proposed openstack/neutron-tempest-plugin master: Add PVLAN extension API tempest tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/990739 | 17:22 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Implement peer-to-peer flows on Interconnect Bridge https://review.opendev.org/c/openstack/neutron/+/988830 | 17:29 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement DB model for evpn_vni attribute https://review.opendev.org/c/openstack/neutron/+/987250 | 17:34 |
| opendevreview | Ghanshyam Maan proposed openstack/neutron-tempest-plugin master: Use compatible tempest tag for 2025.2 ubuntu jammy jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991165 | 18:23 |
| opendevreview | Ghanshyam Maan proposed openstack/neutron-tempest-plugin master: Use compatible tempest tag for 2025.1 ubuntu jammy jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991165 | 18:23 |
| haleyb | ralonsoh: if you're still around can you look at https://review.opendev.org/c/openstack/neutron-vpnaas/+/989371 ? it's actually fixing CI as py310 job shouldn't be running | 18:26 |
| opendevreview | Merged openstack/neutron master: Remove python 3.10 references https://review.opendev.org/c/openstack/neutron/+/989026 | 18:36 |
| opendevreview | Merged openstack/neutron master: Consume DEVICE_OWNER_VPN_ROUTER_GW constant from neutron-lib https://review.opendev.org/c/openstack/neutron/+/984211 | 18:37 |
| opendevreview | Merged openstack/neutron master: Remove update_router_static_routes() https://review.opendev.org/c/openstack/neutron/+/984256 | 18:44 |
| opendevreview | Merged openstack/neutron master: Fixture for FrrService https://review.opendev.org/c/openstack/neutron/+/989475 | 18:44 |
| opendevreview | Ghanshyam Maan proposed openstack/neutron-tempest-plugin master: Test tempest fix https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991181 | 19:11 |
| opendevreview | Ghanshyam Maan proposed openstack/neutron-tempest-plugin master: Use compatible tempest tag for 2025.1 ubuntu jammy jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991165 | 19:12 |
| opendevreview | Merged openstack/neutron-specs master: Allow project managers in SG default statefulness spec https://review.opendev.org/c/openstack/neutron-specs/+/990606 | 19:56 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Add generic RangeAllocator and VNI/VLAN allocation https://review.opendev.org/c/openstack/neutron/+/990759 | 20:44 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement DB model for evpn_vni attribute https://review.opendev.org/c/openstack/neutron/+/987250 | 21:05 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Add generic RangeAllocator and VNI/VLAN allocation https://review.opendev.org/c/openstack/neutron/+/990759 | 21:05 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement OVN portion https://review.opendev.org/c/openstack/neutron/+/989626 | 21:05 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: FRR driver for a EVPN router https://review.opendev.org/c/openstack/neutron/+/988158 | 21:26 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 21:33 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Implement peer-to-peer flows on Interconnect Bridge https://review.opendev.org/c/openstack/neutron/+/988830 | 21:43 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement OVN portion https://review.opendev.org/c/openstack/neutron/+/989626 | 22:23 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 23:14 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!