| opendevreview | Merged openstack/neutron stable/2026.1: Fix PF GET/PUT parent floating IP validation https://review.opendev.org/c/openstack/neutron/+/990125 | 01:44 |
|---|---|---|
| opendevreview | JayGue Lee proposed openstack/neutron master: Reject duplicate gateway ports on the same subnet https://review.opendev.org/c/openstack/neutron/+/990232 | 04:49 |
| opendevreview | JayGue Lee proposed openstack/neutron master: Reject duplicate gateway ports on the same subnet https://review.opendev.org/c/openstack/neutron/+/990232 | 04:53 |
| opendevreview | Jimin Shin proposed openstack/neutron-lib stable/2025.1: Fix address‑groups API path and request body format https://review.opendev.org/c/openstack/neutron-lib/+/990237 | 05:39 |
| opendevreview | Michal Nasiadka proposed openstack/neutron stable/2025.1: DNM: test 2025.1 https://review.opendev.org/c/openstack/neutron/+/990240 | 06:07 |
| opendevreview | Michal Nasiadka proposed openstack/neutron stable/2025.1: DNM: test 2025.1 https://review.opendev.org/c/openstack/neutron/+/990240 | 06:31 |
| 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 | 06:47 |
| opendevreview | Rico Lin proposed openstack/neutron master: [OVN] Mark agents down when Chassis is deleted but Chassis_Private remains https://review.opendev.org/c/openstack/neutron/+/984495 | 06:50 |
| opendevreview | JayGue Lee proposed openstack/neutron master: Reject duplicate gateway ports on the same subnet https://review.opendev.org/c/openstack/neutron/+/990232 | 07:01 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2025.1: DNM: test 2025.1 https://review.opendev.org/c/openstack/neutron/+/990247 | 07:18 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2025.1: DNM: test 2025.1 https://review.opendev.org/c/openstack/neutron/+/990247 | 07:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Group periodic thread workers with selective wrapping https://review.opendev.org/c/openstack/neutron/+/990002 | 07:48 |
| opendevreview | Merged openstack/neutron stable/2025.2: Fix PF GET/PUT parent floating IP validation https://review.opendev.org/c/openstack/neutron/+/990126 | 07:58 |
| opendevreview | JayGue Lee proposed openstack/neutron master: Fix SNAT ambiguity for routers with multiple gateway ports https://review.opendev.org/c/openstack/neutron/+/990232 | 08:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: doc: Document policy scope types and default roles https://review.opendev.org/c/openstack/neutron/+/990069 | 08:39 |
| slaweq | ralonsoh bcafarel lajoskatona: please check those small backports https://review.opendev.org/q/I396d200b1ed61479c6c3f766adbe1c15585b0fe4 when You will have some time, thx in advance | 08:50 |
| ralonsoh | let me check | 08:51 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas stable/2025.2: Pass router_id to the _cleanup_nflog_process method https://review.opendev.org/c/openstack/neutron-fwaas/+/982258 | 08:55 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas stable/2025.1: Pass router_id to the _cleanup_nflog_process method https://review.opendev.org/c/openstack/neutron-fwaas/+/982259 | 08:55 |
| bcafarel | slaweq: sure will look after meetings | 09:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use Neutron FIP UUID as OVN ``NAT`` row UUID https://review.opendev.org/c/openstack/neutron/+/984408 | 09:03 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 09:08 |
| opendevreview | Kyuyeong Lee proposed openstack/neutron master: Add neutron-ovn-migrate-metadata-mtu script https://review.opendev.org/c/openstack/neutron/+/990270 | 10:10 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas master: Extract DB model classes into separete module https://review.opendev.org/c/openstack/neutron-fwaas/+/988594 | 10:22 |
| sean-k-mooney | ralonsoh: https://bugs.launchpad.net/neutron/+bug/1797575 is an old one as is https://bugs.launchpad.net/neutron/+bug/1909100 | 10:34 |
| sean-k-mooney | the nova side fo the cybrog manged nics propals never happend | 10:35 |
| sean-k-mooney | but it may be someitn i revive now that im activly contibutign to cybrog | 10:35 |
| sean-k-mooney | it wont be this cycle but may in the next 12-18 months depenidn on apitite/priority | 10:36 |
| ralonsoh | sean-k-mooney, I did a bug clean-up today. The VNIC types are added in the neutron code | 10:36 |
| ralonsoh | the sriov bug was not attended and at least from neutron point of view, we will do nothing | 10:37 |
| opendevreview | Merged openstack/neutron-fwaas stable/2026.1: Ensure default fw group, policies and rules exists https://review.opendev.org/c/openstack/neutron-fwaas/+/990090 | 10:37 |
| opendevreview | Merged openstack/neutron-fwaas stable/2025.2: Ensure default fw group, policies and rules exists https://review.opendev.org/c/openstack/neutron-fwaas/+/990091 | 10:37 |
| ralonsoh | unless you consider this is something to be considered | 10:37 |
| sean-k-mooney | no the sriov one i dont thinkw e can really do anything | 10:37 |
| ralonsoh | yeah, that's what I think | 10:38 |
| sean-k-mooney | the cybrog one https://bugs.launchpad.net/neutron/+bug/1909100 is only parlty done by the way | 10:38 |
| ralonsoh | this is why I closed a 2018 bug | 10:38 |
| sean-k-mooney | i belive we added it to neutron lib | 10:38 |
| ralonsoh | yes, what is still needed in Neutron? | 10:38 |
| sean-k-mooney | but we dint end up suppritng it in the neutron ml2 dirver or in nova | 10:38 |
| sean-k-mooney | i think if we decied to revi the supprot for accelerator-direct | 10:38 |
| sean-k-mooney | we can propos a new tracker and review it as a new RFE bug | 10:39 |
| ralonsoh | so at this point I prefer you to create a new bug when needed | 10:39 |
| ralonsoh | yes, exactly | 10:39 |
| sean-k-mooney | ya and we dont need ot create that now | 10:39 |
| sean-k-mooney | we can do that if we ever get back to that | 10:39 |
| ralonsoh | perfect for mw | 10:39 |
| sean-k-mooney | in the next cycle or two im hopign to get teh pci driver in cybrog to have more or less partity with the pci supprot in nova, ocne that is doen we may look at the neuton port side of things. | 10:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use Neutron FIP UUID as OVN ``NAT`` row UUID https://review.opendev.org/c/openstack/neutron/+/984408 | 10:41 |
| sean-k-mooney | there are some gpu/ai networkign usecase taht coudl be intersting in the future | 10:41 |
| opendevreview | Merged openstack/neutron-fwaas stable/2025.1: Ensure default fw group, policies and rules exists https://review.opendev.org/c/openstack/neutron-fwaas/+/990092 | 10:41 |
| ralonsoh | yeah, that will be a "funny" work to be done | 10:41 |
| sean-k-mooney | if all we wanted to do was take the accoated nic (PF or VF) and attach it to a shared provider network it would not be hard to do in theory. nova/cybrog would jsut need to provide the approate pci adress but if we want to actully be able to do this via a neutron port then its more complicated to do the coralation. | 10:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix port device_owner RBAC to require network owner manager https://review.opendev.org/c/openstack/neutron/+/990273 | 10:52 |
| ralonsoh | haleyb, slaweq lajoskatona ^^^ please check this patch. If approved, I'll backport it right now | 10:52 |
| ralonsoh | bug under embargo now | 10:53 |
| sean-k-mooney | am... | 10:54 |
| sean-k-mooney | you realise that you are not allwoed ot push pathces related to an embargod bug | 10:54 |
| sean-k-mooney | you have to attach them to the launchpad | 10:55 |
| ralonsoh | the patch has been discussed inside the LP bug | 10:55 |
| sean-k-mooney | that cannot be submited to gerrit or that break teh embargo | 10:55 |
| ralonsoh | and the proposal accepted | 10:55 |
| sean-k-mooney | ok but the patch push is ment ot only happen after the bug is made public | 10:55 |
| ralonsoh | this bug should be opened because the fix is a matter of updating the policy files | 10:56 |
| ralonsoh | so if we make this public, it will be easy to fix any environment, just by updating the policies | 10:56 |
| sean-k-mooney | sure im just saying under the vmt process you are not allowed to push the patch to gerrit until after it has been made public | 10:57 |
| sean-k-mooney | lookign at the patch | 10:57 |
| sean-k-mooney | it mostly looks correct to me | 10:58 |
| sean-k-mooney | however base.NET_OWNER_MEMBER should not be able to set the device owner or id in general | 10:59 |
| sean-k-mooney | and if base.NET_OWNER_MEMBER is listed the hn base.NET_OWNER_MANAGER is not requried | 10:59 |
| sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/990273/1/neutron/conf/policies/port.py#104 | 10:59 |
| sean-k-mooney | the manger role impleis member | 10:59 |
| sean-k-mooney | so if member can do it manger can too | 10:59 |
| ralonsoh | sean-k-mooney, the string check is an "OR" check | 11:01 |
| ralonsoh | we need the manager to be also the net owner | 11:01 |
| sean-k-mooney | yes | 11:01 |
| sean-k-mooney | but my point was | 11:01 |
| sean-k-mooney | if you have the manger role you also alwasy uncondtionlly have member | 11:01 |
| sean-k-mooney | so if you meet the requiremets for NET_OWNER_MANAGER you also will meet NET_OWNER_MEMBER | 11:02 |
| sean-k-mooney | and since its an OR | 11:02 |
| sean-k-mooney | NET_OWNER_MANAGER is therefor redundent | 11:02 |
| sean-k-mooney | so the fix is either to remove base.PROJECT_MANAGER entirly and not replace it with anything. or replace it with NET_OWNER_MANAGER and remove NET_OWNER_MEMBER | 11:03 |
| ralonsoh | let me check | 11:04 |
| ralonsoh | sean-k-mooney, sorry, I don't understand | 11:07 |
| ralonsoh | the role is different, we need to check both | 11:07 |
| ralonsoh | role manager doesn't implies role member | 11:08 |
| sean-k-mooney | yes it does | 11:08 |
| sean-k-mooney | that is part of the SRBAC work | 11:08 |
| sean-k-mooney | admin implies manager wich imples member which implies reader | 11:08 |
| ralonsoh | yes, you are right | 11:09 |
| ralonsoh | admin -> manager -> member -> reader | 11:09 |
| ralonsoh | I'll check all Neutron policies after this patch | 11:10 |
| sean-k-mooney | governance.openstack.org/tc/goals/selected/consistent-and-secure-rbac.html#project-manager is one of the better refences for that | 11:11 |
| sean-k-mooney | ah https://github.com/openstack/keystone-specs/blob/master/specs/keystone/yoga/default-manager-role.rst | 11:11 |
| sean-k-mooney | that is the spec on the keystone side | 11:11 |
| ralonsoh | yes, is written here: | 11:12 |
| ralonsoh | "The project-manager needs to be added in the role implication so that the admin role implies manager, the manager role implies member, the member role implies reader. This needs the modification in the already merged keystone specification." | 11:12 |
| opendevreview | Merged openstack/neutron master: bgp: Always assume just one provider switch https://review.opendev.org/c/openstack/neutron/+/988539 | 11:12 |
| opendevreview | Merged openstack/neutron stable/2026.1: Fix VLAN network detection to use VlanTypeDriver ranges https://review.opendev.org/c/openstack/neutron/+/990054 | 11:12 |
| opendevreview | Merged openstack/neutron stable/2025.2: Fix VLAN network detection to use VlanTypeDriver ranges https://review.opendev.org/c/openstack/neutron/+/990055 | 11:12 |
| opendevreview | Merged openstack/neutron master: Add PVLAN service plugin https://review.opendev.org/c/openstack/neutron/+/978798 | 11:12 |
| lajoskatona | ralonsoh, sean-k-mooney: checking | 11:19 |
| ralonsoh | lajoskatona, I need to push a new PS | 11:21 |
| ralonsoh | slaweq, found some missing rules | 11:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix port RBAC policies to require network ownership https://review.opendev.org/c/openstack/neutron/+/990273 | 11:21 |
| *** ralonsoh is now known as ralonsoh_lunch | 11:23 | |
| lajoskatona | ralonsoh: ack | 11:35 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas master: Remove nested DB context decorators from private getter methods https://review.opendev.org/c/openstack/neutron-fwaas/+/990119 | 12:27 |
| *** ralonsoh_lunch is now known as ralonsoh | 12:48 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Align QoS rule RBAC with parent-owner member checks https://review.opendev.org/c/openstack/neutron/+/990287 | 13:08 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas master: Remove nested DB context decorators from private getter methods https://review.opendev.org/c/openstack/neutron-fwaas/+/990119 | 13:12 |
| opendevreview | Michal Nasiadka proposed openstack/neutron stable/2025.1: DNM: test 2025.1 https://review.opendev.org/c/openstack/neutron/+/990240 | 13:17 |
| 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 | 13:18 |
| 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 | 13:21 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: doc: Add OVN BGP admin guide for spine-and-leaf fabrics https://review.opendev.org/c/openstack/neutron/+/989128 | 13:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix create_agent policy operations path https://review.opendev.org/c/openstack/neutron/+/990295 | 14:02 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Add common constants https://review.opendev.org/c/openstack/neutron/+/990136 | 14:04 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement DB model for evpn_vni attribute https://review.opendev.org/c/openstack/neutron/+/987250 | 14:04 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement OVN portion https://review.opendev.org/c/openstack/neutron/+/989626 | 14:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix port RBAC policies to require network ownership https://review.opendev.org/c/openstack/neutron/+/990273 | 14:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Align QoS rule RBAC with parent-owner member checks https://review.opendev.org/c/openstack/neutron/+/990287 | 14:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use BaseChecks from neutron-lib https://review.opendev.org/c/openstack/neutron/+/990302 | 14:24 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Add common constants https://review.opendev.org/c/openstack/neutron/+/990136 | 14:28 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement DB model for evpn_vni attribute https://review.opendev.org/c/openstack/neutron/+/987250 | 14:28 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Implement OVN portion https://review.opendev.org/c/openstack/neutron/+/989626 | 14:28 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Add tempest test for multiple segments per host on routed provider networks https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/988332 | 14:52 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-lib master: Trivial: Fix PVLAN_PLUGIN underscore https://review.opendev.org/c/openstack/neutron-lib/+/990307 | 14:57 |
| opendevreview | Merged openstack/neutron master: Add worker lifecycle DEBUG logging decorator https://review.opendev.org/c/openstack/neutron/+/990114 | 15:31 |
| opendevreview | Merged openstack/neutron master: doc: Add OVN BGP admin guide for spine-and-leaf fabrics https://review.opendev.org/c/openstack/neutron/+/989128 | 15:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Group periodic thread workers with selective wrapping https://review.opendev.org/c/openstack/neutron/+/990002 | 17:49 |
| opendevreview | Merged openstack/neutron master: Fix port RBAC policies to require network ownership https://review.opendev.org/c/openstack/neutron/+/990273 | 18:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2026.1: Fix port RBAC policies to require network ownership https://review.opendev.org/c/openstack/neutron/+/990353 | 19:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.2: Fix port RBAC policies to require network ownership https://review.opendev.org/c/openstack/neutron/+/990356 | 19:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2026.1: Skip OVNL3 post-fork init in non-API workers https://review.opendev.org/c/openstack/neutron/+/990357 | 19:15 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: evpn: Add common constants https://review.opendev.org/c/openstack/neutron/+/990136 | 20:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Group periodic thread workers with selective wrapping https://review.opendev.org/c/openstack/neutron/+/990002 | 20:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use Neutron FIP UUID as OVN ``NAT`` row UUID https://review.opendev.org/c/openstack/neutron/+/984408 | 20:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Add security-groups-default-statefulness API extension https://review.opendev.org/c/openstack/neutron/+/984356 | 20:50 |
| * gouthamr is looking for rodolfo's nick | 21:08 | |
| opendevreview | JayGue Lee proposed openstack/neutron master: Fix SNAT ambiguity for routers with multiple gateway ports https://review.opendev.org/c/openstack/neutron/+/990232 | 23:48 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!