Wednesday, 2024-06-26

opendevreviewMiro Tomaska proposed openstack/neutron stable/2023.1: [OVN] Add the bridge name and datapath type to the port VIF details  https://review.opendev.org/c/openstack/neutron/+/92243101:30
ralonsohslaweq, hello! If you have 5 mins (not more): https://review.opendev.org/c/openstack/neutron-lib/+/91447306:10
ralonsohthanks a lot06:10
*** liuxie is now known as liushy06:13
slaweqralonsoh looking right now06:50
ralonsohslaweq++06:50
slaweqralonsoh I commented one thing there06:57
ralonsohthanks, checking now06:57
slaweqplease let me know if that makes sense for you06:57
ralonsohslaweq, replied, I'm considering the case that we explicitly pass no actions, thus this method (as implemented now) is returning empty value07:05
ralonsohwhen that could be needed? No idea, but we provide this possibility07:05
opendevreviewSlawek Kaplonski proposed openstack/neutron master: DNM Just testing u/s change with our operator changes together  https://review.opendev.org/c/openstack/neutron/+/92278307:07
*** gaudenz_ is now known as gaudenz07:17
slaweqthx ralonsoh, I +2 it07:18
ralonsohthanks07:18
slaweqbut I will let lajoskatona to have final vote as he had some comments too07:18
opendevreviewMerged openstack/os-ken master: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92262207:24
lajoskatonaslaweq: which patch are we talking about? sorry just connected to net....07:29
ralonsohlajoskatona, https://review.opendev.org/c/openstack/neutron-lib/+/91447307:30
opendevreviewRodolfo Alonso proposed openstack/neutron-dynamic-routing master: Bump Neutron to 25.0.0.0b1  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/92278407:51
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas master: Bump Neutron to 25.0.0.0b1  https://review.opendev.org/c/openstack/neutron-vpnaas/+/92278507:53
opendevreviewGaudenz Steinlin proposed openstack/neutron master: Improve Process fixture service restart handling  https://review.opendev.org/c/openstack/neutron/+/92278607:56
sahido/ hello guys07:58
sahidany chance to get the final review for that one: https://review.opendev.org/c/openstack/neutron/+/90725007:58
sahidthe context is really limited to rpn and multi-segs07:59
opendevreviewyatin proposed openstack/neutron master: Improve Process fixture service restart handling  https://review.opendev.org/c/openstack/neutron/+/92278607:59
ykarelgaudenz, thx, just updated commit message to fix bug link07:59
opendevreviewMaximilian Sesterhenn proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver  https://review.opendev.org/c/openstack/networking-bgpvpn/+/88306008:21
opendevreviewliuyulong proposed openstack/neutron master: Fullstack case for metadata path  https://review.opendev.org/c/openstack/neutron/+/88809808:23
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Don't send project_id as QoS rule attribute in the tests  https://review.opendev.org/c/openstack/neutron/+/92279008:46
opendevreviewSlawek Kaplonski proposed openstack/neutron-lib master: Don't add tenant_id to all resources during create request  https://review.opendev.org/c/openstack/neutron-lib/+/92164908:46
opendevreviewSlawek Kaplonski proposed openstack/neutron master: DNM Test neutron-lib fix with sqlalchemy-master unit test job  https://review.opendev.org/c/openstack/neutron/+/92165008:47
opendevreviewLajos Katona proposed openstack/os-ken stable/2024.1: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92279109:07
sahidthanks a lot neutron team! :-)09:13
ralonsohhaleyb, lajoskatona: https://review.opendev.org/c/openstack/releases/+/92279209:15
lajoskatonasahid: done, thanks for working on that issue09:17
lajoskatonaralonsoh: thanks09:17
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add "subnet-external-network" extension to "subnet" resource  https://review.opendev.org/c/openstack/neutron/+/90731309:48
opendevreviewMerged openstack/neutron-lib master: Add the field "rbac_actions" to the model query hooks  https://review.opendev.org/c/openstack/neutron-lib/+/91447310:07
*** mgoddard- is now known as mgoddard10:12
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Add a new client method: "get_loaded_network_extensions"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92271110:49
opendevreviewMerged openstack/ovn-bgp-agent master: Ensure cr-lrp permanent ip nei entry is added on NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92149111:11
ralonsohslaweq, mlavalle hi! Sorry for the respin of https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/922711, but I realize (in the Neutron patch implementing the feature) that this filter was needed only in the specific testing method11:13
ralonsohthanks!11:13
opendevreviewLajos Katona proposed openstack/os-ken stable/2023.2: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92279411:15
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2024.1: Ensure cr-lrp permanent ip nei entry is added on NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92279911:57
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Ensure cr-lrp permanent ip nei entry is added on NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92280011:57
opendevreviewLajos Katona proposed openstack/os-ken stable/2023.1: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92280112:02
opendevreviewLajos Katona proposed openstack/os-ken unmaintained/zed: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92280912:22
opendevreviewLajos Katona proposed openstack/os-ken unmaintained/yoga: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92281012:24
opendevreviewMerged openstack/os-ken stable/2024.1: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92279112:29
opendevreviewLajos Katona proposed openstack/os-ken unmaintained/xena: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92281412:31
opendevreviewLajos Katona proposed openstack/os-ken unmaintained/wallaby: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92281612:46
opendevreviewLajos Katona proposed openstack/os-ken unmaintained/victoria: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92281712:47
*** kleini_ is now known as kleini12:47
opendevreviewMerged openstack/ovn-bgp-agent stable/2023.2: Ensure cr-lrp permanent ip nei entry is added on NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92280013:33
opendevreviewRodolfo Alonso proposed openstack/neutron master: Remove the Windows libraries  https://review.opendev.org/c/openstack/neutron/+/92282113:36
opendevreviewRodolfo Alonso proposed openstack/neutron master: Remove maintenance task "update_port_virtual_type"  https://review.opendev.org/c/openstack/neutron/+/92282313:43
opendevreviewMaximilian Sesterhenn proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver  https://review.opendev.org/c/openstack/networking-bgpvpn/+/88306014:27
opendevreviewMerged openstack/os-ken stable/2023.2: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92279414:39
opendevreviewMerged openstack/os-ken stable/2023.1: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92280114:39
slaweqhi gmann , how are You?15:00
slaweqgmann may I have question about S-RBAC to you?15:00
slaweqI started looking today into neutron's "use service token when communicating with other services" task15:01
slaweqwe discussed it in the last PTG IIRC, but I am not sure if this means that in neutron I should use "service_token" thing provided by the keystoneauth1, in the way like e.g. nova is doing that15:01
slaweqor should we just change in e.g. devstack so that 'nova' user created there will not have 'admin' role but rather 'service' role15:01
gmannslaweq: hi19:05
gmannslaweq: as end goal, yes we should corrrect the service user role in devstack but that can break CI. once we have the services using both service as well as user token in service interaction then we can do that19:06
gmannslaweq: I am going to prepare a job for that and see how much fix we need19:06
gmannslaweq: let me start changes on nova side about 'using both token service and user' and a devstack job to test those19:07
gmannI will ping you once they are up, will try to update my patches this week or so19:07
opendevreviewElod Illes proposed openstack/os-ken unmaintained/yoga: Drop lower-constraints.txt and its testing  https://review.opendev.org/c/openstack/os-ken/+/92287419:28
opendevreviewMaximilian Sesterhenn proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver  https://review.opendev.org/c/openstack/networking-bgpvpn/+/88306019:36
opendevreviewMerged openstack/os-ken unmaintained/zed: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92280920:30
opendevreviewBrian Haley proposed openstack/neutron-lib unmaintained/yoga: Update .gitreview for unmaintained/yoga  https://review.opendev.org/c/openstack/neutron-lib/+/90785920:30
*** dmellado0755 is now known as dmellado07521:06
slaweqgmann: so is my uderstanding correct that this is just about changing role of the 'nova' user which is set in keystone, right. We don't need any changes in neutron for that, correct?21:08
opendevreviewMerged openstack/networking-bagpipe master: rest_attach: encode POST body in UTF-8  https://review.opendev.org/c/openstack/networking-bagpipe/+/91671822:32
opendevreviewMerged openstack/networking-bagpipe master: reno: Update master for unmaintained/zed  https://review.opendev.org/c/openstack/networking-bagpipe/+/91754622:32

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!