| opendevreview | liuyulong proposed openstack/neutron master: Fullstack case for metadata path https://review.opendev.org/c/openstack/neutron/+/888098 | 01:40 |
|---|---|---|
| opendevreview | liuyulong proposed openstack/neutron master: Add devstack plugin to enable ovs metadata_path https://review.opendev.org/c/openstack/neutron/+/928586 | 01:40 |
| opendevreview | liuyulong proposed openstack/neutron master: [doc] Remove port resource change after unplug https://review.opendev.org/c/openstack/neutron/+/941616 | 01:56 |
| opendevreview | liuyulong proposed openstack/neutron master: Remove resource cache query_ids https://review.opendev.org/c/openstack/neutron/+/904169 | 01:58 |
| opendevreview | Merged openstack/neutron master: [eventlet-removal] Remove the usage of eventlet in the Neutron API (2) https://review.opendev.org/c/openstack/neutron/+/941202 | 04:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Revert "Add router ``PUT`` external gateways actions policies" https://review.opendev.org/c/openstack/neutron/+/941624 | 06:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: Revert "Add router ``PUT`` external gateways actions policies" https://review.opendev.org/c/openstack/neutron/+/941625 | 06:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Create a deterministic hash ring node UUID generator https://review.opendev.org/c/openstack/neutron/+/941008 | 06:34 |
| ralonsoh | lajoskatona, slaweq we have drama!! | 08:44 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/2098109/comments/11 | 08:44 |
| ralonsoh | I'm pushing now a fix for the API without the multihoming extension | 08:44 |
| ralonsoh | then we'll need to fix it | 08:45 |
| lajoskatona | ralonsoh: checking | 08:48 |
| slaweq | thx for cleaning this mess @ralol | 08:55 |
| slaweq | *ralonsoh | 08:55 |
| opendevreview | Andrew Bonney proposed openstack/neutron master: ml2/sriov: bind port on segment that subnet belongs https://review.opendev.org/c/openstack/neutron/+/941345 | 10:17 |
| opendevreview | Andrew Bonney proposed openstack/neutron master: ml2/sriov: bind port on segment that subnet belongs https://review.opendev.org/c/openstack/neutron/+/941345 | 10:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Fix the router "external_gateway_info" validation https://review.opendev.org/c/openstack/neutron-lib/+/941631 | 11:11 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: WIP: Remove LinuxBridge related code https://review.opendev.org/c/openstack/networking-bagpipe/+/940765 | 11:15 |
| opendevreview | Merged openstack/neutron stable/2023.2: Revert "Add router ``PUT`` external gateways actions policies" https://review.opendev.org/c/openstack/neutron/+/941624 | 13:15 |
| opendevreview | Merged openstack/neutron stable/2024.1: Revert "Add router ``PUT`` external gateways actions policies" https://review.opendev.org/c/openstack/neutron/+/941625 | 13:23 |
| *** tkajinam is now known as Guest9129 | 13:39 | |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Add IPv6 gateway ips on the vrf interface https://review.opendev.org/c/openstack/ovn-bgp-agent/+/941653 | 13:48 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Add IPv6 gateway ips on the vrf interface https://review.opendev.org/c/openstack/ovn-bgp-agent/+/941653 | 13:49 |
| haleyb | #startmeeting neutron_drivers | 14:00 |
| opendevmeet | Meeting started Fri Feb 14 14:00:31 2025 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, obondarev, tobias-urdin, lajoskatona, amotoki, haleyb, ralonsoh | 14:00 |
| mlavalle | \o | 14:00 |
| obondarev | o/ | 14:00 |
| lajoskatona | o/ | 14:00 |
| slaweq | o/ | 14:01 |
| sahid | o/ | 14:02 |
| haleyb | just waiting for ralonsoh | 14:02 |
| haleyb | the item on the agenda is his change | 14:02 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/941511 | 14:02 |
| ralonsoh | hello | 14:02 |
| ralonsoh | yeah, did you check the mail? | 14:03 |
| haleyb | no, not yet | 14:03 |
| ralonsoh | I commented that it would be better just to take this conversation offline | 14:03 |
| ralonsoh | I commented in the LP bug | 14:03 |
| ralonsoh | one sec | 14:03 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/2098109/comments/11 | 14:03 |
| ralonsoh | so is worst than expected | 14:03 |
| ralonsoh | we have the default API that is currently broken | 14:04 |
| ralonsoh | using "external_gateway_info" | 14:04 |
| ralonsoh | that should be fixed with https://review.opendev.org/c/openstack/neutron-lib/+/941631 | 14:04 |
| ralonsoh | but that requires some additional changes in the tempest tests | 14:04 |
| ralonsoh | and we have the multihoming extension, that uses "external_gateways" | 14:04 |
| ralonsoh | that extension does not enforce at all any policy nor we have any defined | 14:05 |
| ralonsoh | that will take more time | 14:05 |
| ralonsoh | and that's all basically | 14:05 |
| ralonsoh | so I think, as commented in the mail, that we can continue the investigation, testing and reviews offline | 14:06 |
| mlavalle | so no need to meet today? | 14:06 |
| haleyb | ok, i'm still reading the bug comments but we can look offline | 14:06 |
| haleyb | your email came at 2am my time, so i had not seen it | 14:06 |
| ralonsoh | no, no meeting today is needed | 14:06 |
| ralonsoh | yes, I finished the investigation today early (for me) | 14:07 |
| haleyb | does anyone have any questions? | 14:07 |
| ralonsoh | (I do... but I'm trying to find the answers myself) | 14:08 |
| lajoskatona | so we need to disable some tempest tests till the n-lib fix is merged? | 14:08 |
| ralonsoh | no, we'll need to change the client to admin_client | 14:08 |
| lajoskatona | ah, ok, that's good | 14:08 |
| ralonsoh | this is because https://review.opendev.org/c/openstack/neutron-lib/+/941631 CI result, right? | 14:08 |
| ralonsoh | these calls must be done by an admin | 14:08 |
| ralonsoh | for example, create a router with GW and snat flag | 14:09 |
| ralonsoh | (e.g.: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_656/941631/1/check/neutron-tempest-plugin-ovn/65617f7/testr_results.html) | 14:09 |
| ralonsoh | I'm proposing a patch right now. That's all from me! | 14:10 |
| lajoskatona | for tempest this is the patch am I right: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941545 ? | 14:10 |
| ralonsoh | not really, that was for the reverted patch | 14:11 |
| ralonsoh | this one will be needed but in a second phase, once we implement the multihoming policies and API changes | 14:11 |
| lajoskatona | ah, ack | 14:11 |
| haleyb | ralonsoh: one of the other reasons i'm confused is even the openstackclient --enable-snat help text doesn't say admin only, and the api ref doesn't show it as an argument | 14:13 |
| ralonsoh | about the API doc, I need to check that | 14:14 |
| ralonsoh | we have this rule (one sec) | 14:14 |
| ralonsoh | https://github.com/openstack/neutron/blob/master/neutron/conf/policies/router.py#L120-L131 | 14:14 |
| ralonsoh | except for the network_id, any other parameter inside "external_gateway_info" can be modified/created only by an admin | 14:15 |
| ralonsoh | that means: a non-admin user can add an external GW to a router, selecting the network | 14:15 |
| ralonsoh | but cannot to this (or modify) the enable_snat or external_fixed_ips | 14:16 |
| haleyb | the add_external_gateways section needs an update, and probably something needed for external_gateway_info as well, fyi | 14:16 |
| haleyb | and update* of course | 14:17 |
| ralonsoh | these new commands, related to external_gateways, are related to the new extension | 14:17 |
| haleyb | right | 14:17 |
| ralonsoh | and yes, doc, API and policies are in bad shape | 14:17 |
| ralonsoh | (in ml2/ovs this is not used by default, but it is in ml2/ovn) | 14:17 |
| mlavalle | haleyb: do we need to stay in the meeting? Do we have something else to discuss? Or can we continue the conversation in LP and Gerrit? | 14:18 |
| ralonsoh | let's close the meeting | 14:18 |
| haleyb | sure | 14:18 |
| haleyb | #endmeeting | 14:18 |
| opendevmeet | Meeting ended Fri Feb 14 14:18:38 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:18 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2025/neutron_drivers.2025-02-14-14.00.html | 14:18 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2025/neutron_drivers.2025-02-14-14.00.txt | 14:18 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2025/neutron_drivers.2025-02-14-14.00.log.html | 14:18 |
| mlavalle | \o | 14:18 |
| slaweq | have a great weekend :) | 14:18 |
| ralonsoh | haleyb, so I'll start with the patches today | 14:18 |
| mlavalle | have a nice weekend y'all! | 14:19 |
| lajoskatona | o/ | 14:19 |
| haleyb | ralonsoh: thanks, i'll look for them | 14:19 |
| lajoskatona | YEah, have a nice weekend! | 14:19 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Remove Linux Bridge job from experimental queue https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941151 | 14:27 |
| sahid | haleyb, ralonsoh thanks a lot for your review regarding os-ken | 14:29 |
| haleyb | sahid: np, did the final one merge? | 14:29 |
| sahid | do you want me to build a release with this current state? | 14:29 |
| sahid | haleyb: I think, there is just the last one which enables native, that ralonsoh prefer to wait a bit | 14:30 |
| sahid | i understand its fear :-) | 14:30 |
| ralonsoh | we can control this value with a env variable | 14:31 |
| ralonsoh | right now, if we need to test that, we can always define it in the CI jobs | 14:31 |
| ralonsoh | so for now we can wait | 14:31 |
| ralonsoh | until all the removal process is done | 14:31 |
| haleyb | ok, so we can update the hash to tip of master | 14:31 |
| sahid | it's basicall what i have done, ehre no? https://review.opendev.org/c/openstack/neutron/+/941508 | 14:32 |
| haleyb | no, i mean in the os-ken release patch - we want a new version, correct? | 14:33 |
| ralonsoh | if we release what we have now in os-ken, it is possible to switch to native just defining OSKEN_HUB_TYPE in the job | 14:33 |
| ralonsoh | so I would release os-ken as is now | 14:33 |
| sahid | haleyb: correct I will update that | 14:34 |
| sahid | ralonsoh: sure | 14:34 |
| ralonsoh | is there a patch for releases? | 14:34 |
| sahid | https://review.opendev.org/c/openstack/releases/+/941356 | 14:34 |
| sahid | I will update it now | 14:35 |
| ralonsoh | sahid, I would update the minor number | 14:35 |
| ralonsoh | to 2.12 | 14:35 |
| ralonsoh | this is not a minor release\ | 14:35 |
| ralonsoh | actually, haleyb I would bump this to 3.0.0 | 14:35 |
| ralonsoh | now supporting native threads | 14:36 |
| sahid | ralonsoh: i would be to 3 if you switch to native by default, no? | 14:36 |
| sahid | s/you/we | 14:36 |
| sahid | bump... | 14:36 |
| ralonsoh | I think this is not a minor bump and that requires a mayor change | 14:36 |
| sahid | ok sure and how do you think we shoould handle the switch evenlet to native by default? | 14:37 |
| sahid | regarding the version | 14:37 |
| ralonsoh | we can always release a new minor version, 3.1.0, for example | 14:38 |
| ralonsoh | but at least 3.0.0 provides support for this | 14:38 |
| sahid | ok, sounds good for me | 14:38 |
| sahid | if we are all agree I can do that | 14:38 |
| haleyb | +1 | 14:38 |
| sahid | thanks I doing that so | 14:39 |
| sahid | hum I should have perhaps added a release note to mention the new support | 14:41 |
| haleyb | you can add it as a standalone patch | 14:43 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 14:49 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: release: add release note to mention support fo native https://review.opendev.org/c/openstack/os-ken/+/941671 | 14:49 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: release: add release note to mention support for native https://review.opendev.org/c/openstack/os-ken/+/941671 | 14:50 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 14:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Bagpipe: make ``neutron-tempest-plugin-bgpvpn-bagpipe`` non-voting https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941218 | 14:59 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: release: add release note to mention support for native https://review.opendev.org/c/openstack/os-ken/+/941671 | 15:03 |
| lajoskatona | haleyb: bagpipe patch is green: https://review.opendev.org/c/openstack/networking-bagpipe/+/940765 and much lighter than before :-) | 15:34 |
| lajoskatona | haleyb: must check and update all the docs, and find somebody perhaps from the old team to check if it worth to keep what is left in it | 15:35 |
| opendevreview | Merged openstack/os-ken master: release: add release note to mention support for native https://review.opendev.org/c/openstack/os-ken/+/941671 | 15:46 |
| sahid | thank you lajoskatona, ralonsoh for the fast review :-) | 15:56 |
| sahid | I have proposed the change for releases I will ping you once CI green | 15:57 |
| sahid | https://review.opendev.org/c/openstack/releases/+/941356 | 15:57 |
| opendevreview | Merged openstack/neutron master: Update Nova aggregates on changed host mappings https://review.opendev.org/c/openstack/neutron/+/935990 | 16:00 |
| lajoskatona | sahid: cool | 16:11 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2024.2: Update Nova aggregates on changed host mappings https://review.opendev.org/c/openstack/neutron/+/941685 | 16:20 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2024.1: Update Nova aggregates on changed host mappings https://review.opendev.org/c/openstack/neutron/+/941686 | 16:21 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2023.2: Update Nova aggregates on changed host mappings https://review.opendev.org/c/openstack/neutron/+/941688 | 16:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Execute router commands with admin client https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941689 | 16:28 |
| haleyb | lajoskatona: you should just remove the WIP from that bagpipe patch | 17:00 |
| opendevreview | Merged openstack/neutron master: [OVN] Create a deterministic hash ring node UUID generator https://review.opendev.org/c/openstack/neutron/+/941008 | 17:30 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Tap Mirror OVS driver https://review.opendev.org/c/openstack/tap-as-a-service/+/893086 | 17:50 |
| opendevreview | Elod Illes proposed openstack/python-neutronclient unmaintained/victoria: [stable-only] Cap setuptools <71.0.0 https://review.opendev.org/c/openstack/python-neutronclient/+/941706 | 18:22 |
| opendevreview | Vasyl Saienko proposed openstack/neutron-tempest-plugin master: Add network_data validation for trunks https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/941355 | 21:21 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Do not review https://review.opendev.org/c/openstack/neutron/+/941730 | 21:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!