opendevreview | Miro 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/+/922431 | 01:30 |
---|---|---|
ralonsoh | slaweq, hello! If you have 5 mins (not more): https://review.opendev.org/c/openstack/neutron-lib/+/914473 | 06:10 |
ralonsoh | thanks a lot | 06:10 |
*** liuxie is now known as liushy | 06:13 | |
slaweq | ralonsoh looking right now | 06:50 |
ralonsoh | slaweq++ | 06:50 |
slaweq | ralonsoh I commented one thing there | 06:57 |
ralonsoh | thanks, checking now | 06:57 |
slaweq | please let me know if that makes sense for you | 06:57 |
ralonsoh | slaweq, replied, I'm considering the case that we explicitly pass no actions, thus this method (as implemented now) is returning empty value | 07:05 |
ralonsoh | when that could be needed? No idea, but we provide this possibility | 07:05 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: DNM Just testing u/s change with our operator changes together https://review.opendev.org/c/openstack/neutron/+/922783 | 07:07 |
*** gaudenz_ is now known as gaudenz | 07:17 | |
slaweq | thx ralonsoh, I +2 it | 07:18 |
ralonsoh | thanks | 07:18 |
slaweq | but I will let lajoskatona to have final vote as he had some comments too | 07:18 |
opendevreview | Merged openstack/os-ken master: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922622 | 07:24 |
lajoskatona | slaweq: which patch are we talking about? sorry just connected to net.... | 07:29 |
ralonsoh | lajoskatona, https://review.opendev.org/c/openstack/neutron-lib/+/914473 | 07:30 |
opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: Bump Neutron to 25.0.0.0b1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/922784 | 07:51 |
opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Bump Neutron to 25.0.0.0b1 https://review.opendev.org/c/openstack/neutron-vpnaas/+/922785 | 07:53 |
opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Improve Process fixture service restart handling https://review.opendev.org/c/openstack/neutron/+/922786 | 07:56 |
sahid | o/ hello guys | 07:58 |
sahid | any chance to get the final review for that one: https://review.opendev.org/c/openstack/neutron/+/907250 | 07:58 |
sahid | the context is really limited to rpn and multi-segs | 07:59 |
opendevreview | yatin proposed openstack/neutron master: Improve Process fixture service restart handling https://review.opendev.org/c/openstack/neutron/+/922786 | 07:59 |
ykarel | gaudenz, thx, just updated commit message to fix bug link | 07:59 |
opendevreview | Maximilian Sesterhenn proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver https://review.opendev.org/c/openstack/networking-bgpvpn/+/883060 | 08:21 |
opendevreview | liuyulong proposed openstack/neutron master: Fullstack case for metadata path https://review.opendev.org/c/openstack/neutron/+/888098 | 08:23 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Don't send project_id as QoS rule attribute in the tests https://review.opendev.org/c/openstack/neutron/+/922790 | 08:46 |
opendevreview | Slawek 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/+/921649 | 08:46 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: DNM Test neutron-lib fix with sqlalchemy-master unit test job https://review.opendev.org/c/openstack/neutron/+/921650 | 08:47 |
opendevreview | Lajos 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/+/922791 | 09:07 |
sahid | thanks a lot neutron team! :-) | 09:13 |
ralonsoh | haleyb, lajoskatona: https://review.opendev.org/c/openstack/releases/+/922792 | 09:15 |
lajoskatona | sahid: done, thanks for working on that issue | 09:17 |
lajoskatona | ralonsoh: thanks | 09:17 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "subnet-external-network" extension to "subnet" resource https://review.opendev.org/c/openstack/neutron/+/907313 | 09:48 |
opendevreview | Merged openstack/neutron-lib master: Add the field "rbac_actions" to the model query hooks https://review.opendev.org/c/openstack/neutron-lib/+/914473 | 10:07 |
*** mgoddard- is now known as mgoddard | 10:12 | |
opendevreview | Rodolfo 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/+/922711 | 10:49 |
opendevreview | Merged 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/+/921491 | 11:11 |
ralonsoh | slaweq, 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 method | 11:13 |
ralonsoh | thanks! | 11:13 |
opendevreview | Lajos 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/+/922794 | 11:15 |
opendevreview | Luis 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/+/922799 | 11:57 |
opendevreview | Luis 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/+/922800 | 11:57 |
opendevreview | Lajos 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/+/922801 | 12:02 |
opendevreview | Lajos Katona proposed openstack/os-ken unmaintained/zed: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922809 | 12:22 |
opendevreview | Lajos Katona proposed openstack/os-ken unmaintained/yoga: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922810 | 12:24 |
opendevreview | Merged openstack/os-ken stable/2024.1: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922791 | 12:29 |
opendevreview | Lajos Katona proposed openstack/os-ken unmaintained/xena: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922814 | 12:31 |
opendevreview | Lajos Katona proposed openstack/os-ken unmaintained/wallaby: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922816 | 12:46 |
opendevreview | Lajos Katona proposed openstack/os-ken unmaintained/victoria: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922817 | 12:47 |
*** kleini_ is now known as kleini | 12:47 | |
opendevreview | Merged 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/+/922800 | 13:33 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove the Windows libraries https://review.opendev.org/c/openstack/neutron/+/922821 | 13:36 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove maintenance task "update_port_virtual_type" https://review.opendev.org/c/openstack/neutron/+/922823 | 13:43 |
opendevreview | Maximilian Sesterhenn proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver https://review.opendev.org/c/openstack/networking-bgpvpn/+/883060 | 14:27 |
opendevreview | Merged openstack/os-ken stable/2023.2: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922794 | 14:39 |
opendevreview | Merged openstack/os-ken stable/2023.1: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922801 | 14:39 |
slaweq | hi gmann , how are You? | 15:00 |
slaweq | gmann may I have question about S-RBAC to you? | 15:00 |
slaweq | I started looking today into neutron's "use service token when communicating with other services" task | 15:01 |
slaweq | we 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 that | 15:01 |
slaweq | or should we just change in e.g. devstack so that 'nova' user created there will not have 'admin' role but rather 'service' role | 15:01 |
gmann | slaweq: hi | 19:05 |
gmann | slaweq: 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 that | 19:06 |
gmann | slaweq: I am going to prepare a job for that and see how much fix we need | 19:06 |
gmann | slaweq: let me start changes on nova side about 'using both token service and user' and a devstack job to test those | 19:07 |
gmann | I will ping you once they are up, will try to update my patches this week or so | 19:07 |
opendevreview | Elod Illes proposed openstack/os-ken unmaintained/yoga: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/os-ken/+/922874 | 19:28 |
opendevreview | Maximilian Sesterhenn proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver https://review.opendev.org/c/openstack/networking-bgpvpn/+/883060 | 19:36 |
opendevreview | Merged openstack/os-ken unmaintained/zed: Raise ValueError in case unpack_from returns zero length https://review.opendev.org/c/openstack/os-ken/+/922809 | 20:30 |
opendevreview | Brian Haley proposed openstack/neutron-lib unmaintained/yoga: Update .gitreview for unmaintained/yoga https://review.opendev.org/c/openstack/neutron-lib/+/907859 | 20:30 |
*** dmellado0755 is now known as dmellado075 | 21:06 | |
slaweq | gmann: 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 |
opendevreview | Merged openstack/networking-bagpipe master: rest_attach: encode POST body in UTF-8 https://review.opendev.org/c/openstack/networking-bagpipe/+/916718 | 22:32 |
opendevreview | Merged openstack/networking-bagpipe master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/networking-bagpipe/+/917546 | 22:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!