| opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 00:13 |
|---|---|---|
| *** dmellado17 is now known as dmellado1 | 04:55 | |
| opendevreview | liuyulong proposed openstack/neutron master: Refactor for ovs qos driver meter limit features https://review.opendev.org/c/openstack/neutron/+/860766 | 07:14 |
| opendevreview | liuyulong proposed openstack/neutron master: Add meter bandwidth limit support https://review.opendev.org/c/openstack/neutron/+/860767 | 07:14 |
| lajoskatona | slaweq: Hi, regarding https://review.opendev.org/c/openstack/neutron/+/884613 and port binding API | 07:46 |
| lajoskatona | slaweq: I realized there is no api-ref for it, so I proposed a mostly wip version of it: https://review.opendev.org/c/openstack/neutron-lib/+/884916 | 07:47 |
| slaweq | lajoskatona thx a lot | 07:47 |
| lajoskatona | slaweq: I am not sure I got all the grips of this API, so please check if you have some time | 07:47 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Default SG api rules template - DB and OVO models https://review.opendev.org/c/openstack/neutron/+/883246 | 07:48 |
| slaweq | lajoskatona sure, I will check it | 07:52 |
| slaweq | but I think that also mlavalle needs to look at it as he's author of this API | 07:52 |
| sahid | haleyb: o/ regarding https://review.opendev.org/c/openstack/neutron/+/884674 I have noticed you comment but I'm not sure if you want me to change it or if it was just a remark as you have not voted on it | 07:54 |
| sahid | anychance that you put -1 on it so I can argue and disagree with you ahah ;) | 07:55 |
| sahid | it's just a joke, but let me know if you prefer i change this part | 07:57 |
| lajoskatona | sahid: Hi, haleyb is ~UTC-4, so I suppose will answer only later :-) | 08:10 |
| sahid | lajoskatona: Hello, sure no worries, thank you :-) | 08:19 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add missing port_binding policy https://review.opendev.org/c/openstack/neutron/+/884983 | 08:28 |
| slaweq | lajoskatona ralonsoh amotoki ^^ quick follow up related to port_bindings, please review when You will have time | 08:29 |
| ralonsoh | sure, let me check | 08:29 |
| ralonsoh | slaweq, do we need a backport? | 08:29 |
| slaweq | nope | 08:29 |
| ralonsoh | ok, perfect | 08:29 |
| slaweq | for those port_binding things we decided we don't need backports | 08:29 |
| *** froyo__ is now known as froyo | 08:43 | |
| lajoskatona | ralonsoh, amotoki: let me advertise the api-ref for port bindings API: https://review.opendev.org/c/openstack/neutron-lib/+/884916 (I realized yesterday that there is no api-ref for it) | 09:13 |
| ralonsoh | good to see that, let me check the patch | 09:14 |
| ralonsoh | slaweq, if you have a couple of mins, to start closing specs: https://review.opendev.org/c/openstack/neutron-specs/+/882151 | 09:15 |
| slaweq | ralonsoh sure, I will read it today | 09:17 |
| ralonsoh | lajoskatona, https://review.opendev.org/c/openstack/networking-odl/+/884359 | 09:18 |
| ralonsoh | what do you think? | 09:18 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/xena: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884905 | 09:26 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/wallaby: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884992 | 09:28 |
| lajoskatona | ralonsoh: thanks, we can leave odl in a proper state for the aliens / future generations :-) | 09:29 |
| ralonsoh | hehehe | 09:29 |
| ralonsoh | I've pushed the same patch for Xena and Wallaby | 09:29 |
| ralonsoh | I need the Ussuri one too | 09:29 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/wallaby: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884992 | 09:46 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/zed: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884189 | 09:46 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/yoga: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884190 | 09:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Change RBAC relationship loading method to "joined" https://review.opendev.org/c/openstack/neutron/+/884877 | 10:00 |
| ralonsoh | lajoskatona, slaweq (sorry for being so annoying today): https://review.opendev.org/q/topic:bug%252F1918145 | 10:01 |
| slaweq | ralonsoh sure, I will finish one thing and will start those reviews :) | 10:01 |
| ralonsoh | please check these two patches (that I will backport). The performance improvement with a large number of RBACs goes from 1 minute to 1 second | 10:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-fwaas master: [alembic] Alembic operations require keywords only arguments https://review.opendev.org/c/openstack/neutron-fwaas/+/883342 | 10:05 |
| opendevreview | Merged openstack/networking-odl stable/yoga: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884359 | 10:11 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Default SG rules - use new rules templates to create rules for SGs https://review.opendev.org/c/openstack/neutron/+/884474 | 10:40 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Default SG api rules template - DB and OVO models https://review.opendev.org/c/openstack/neutron/+/883246 | 10:41 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Default SG rules - use new rules templates to create rules for SGs https://review.opendev.org/c/openstack/neutron/+/884474 | 10:41 |
| opendevreview | Merged openstack/neutron-fwaas master: Support standard_attrs for fwaas resources https://review.opendev.org/c/openstack/neutron-fwaas/+/866988 | 10:44 |
| opendevreview | Merged openstack/neutron-specs master: ERSPAN and GRE mirroring for tap-as-a-service https://review.opendev.org/c/openstack/neutron-specs/+/882151 | 11:02 |
| opendevreview | Merged openstack/neutron-fwaas master: notify the plugin only on status updated https://review.opendev.org/c/openstack/neutron-fwaas/+/884335 | 11:05 |
| opendevreview | Merged openstack/neutron-fwaas master: Remove Exception Definition in fwaas_v2 Extension https://review.opendev.org/c/openstack/neutron-fwaas/+/884334 | 11:07 |
| opendevreview | Merged openstack/neutron-vpnaas master: [alembic] Alembic operations require keywords only arguments https://review.opendev.org/c/openstack/neutron-vpnaas/+/883343 | 11:08 |
| opendevreview | Merged openstack/neutron-lib master: api-ref: add api-ref for bindings https://review.opendev.org/c/openstack/neutron-lib/+/884916 | 11:27 |
| sahid | slaweq: o/ I think the ci error is related to your patch here, no https://zuul.opendev.org/t/openstack/build/ff8c6b4c82cd4181b1e2efb0f8654bd1 ? | 11:40 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Adapt bgpvpn scenarios to recent sRBAC changes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838 | 12:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Raise the timeout of "neutron-ovn-rally-task" to 9000 https://review.opendev.org/c/openstack/neutron/+/885045 | 12:58 |
| ralonsoh | ^^^ folks, the rally job has been timing out recently too often | 12:59 |
| ralonsoh | please check this patch | 12:59 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/wallaby: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884992 | 13:05 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/wallaby: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884192 | 13:26 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/xena: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884191 | 13:27 |
| haleyb | sahid: it seemed like you were waiting for other reviews, so i didn't vote either way | 13:47 |
| opendevreview | Merged openstack/networking-odl stable/yoga: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884190 | 13:52 |
| sahid | haleyb: no no i was not I will be glad to adresse any of your reviews/suggestions | 14:00 |
| haleyb | sahid: i guess i would vote for a standalone 'if' for that condition, since it is different from the others. just my opinion | 14:05 |
| haleyb | i'll leave another (unrelated) comment there based on your "i will clean this" comment | 14:05 |
| sahid | oh I see now i understand why you were thinking i was waiting for more comments | 14:07 |
| sahid | yes I can understand your preference having that standalone if condition I will fix this point aswell and see if that suits everyone | 14:09 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/ussuri: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/885051 | 14:17 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/ussuri: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/885051 | 14:19 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/victoria: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/885054 | 14:24 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/ussuri: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/885051 | 14:25 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/xena: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884905 | 14:35 |
| ralonsoh | obondarev, rubasov, if you don't mind, please check https://review.opendev.org/q/topic:bug%252F1918145. These 2 patches are easy to review and provide great performance improvement (when both are applied together) | 14:37 |
| ralonsoh | haleyb, ^ | 14:37 |
| ralonsoh | thanks in advance! btw, I'm going to backport them | 14:37 |
| haleyb | ralonsoh: ack, have a few cycles today | 14:38 |
| ralonsoh | thanks a lot | 14:38 |
| obondarev | sure | 14:38 |
| haleyb | ralonsoh: and don't know if there is a drivers meeting tomorrow, but i am pto | 14:39 |
| ralonsoh | yes, there is a topic | 14:39 |
| ralonsoh | mlavalle, will present it | 14:39 |
| ralonsoh | mlavalle, if you have time today, please check https://review.opendev.org/q/project:openstack/neutron-specs+status:open | 14:42 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/zed: Update the CI requirements and remove grenade/tempest jobs https://review.opendev.org/c/openstack/networking-odl/+/885062 | 15:20 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/victoria: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/885054 | 15:23 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/ussuri: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/885051 | 15:26 |
| opendevreview | yatin proposed openstack/neutron master: [ovn][ipv6] Skip test_update_router_admin_state https://review.opendev.org/c/openstack/neutron/+/885074 | 16:04 |
| opendevreview | Merged openstack/networking-odl stable/wallaby: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884992 | 16:44 |
| opendevreview | Merged openstack/networking-odl stable/wallaby: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884192 | 16:44 |
| opendevreview | Merged openstack/neutron-lib master: Add a "GROUP BY" clause on queries with RBAC entries https://review.opendev.org/c/openstack/neutron-lib/+/884878 | 16:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib stable/2023.1: Add a "GROUP BY" clause on queries with RBAC entries https://review.opendev.org/c/openstack/neutron-lib/+/885080 | 17:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib stable/zed: Add a "GROUP BY" clause on queries with RBAC entries https://review.opendev.org/c/openstack/neutron-lib/+/885081 | 17:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib stable/yoga: Add a "GROUP BY" clause on queries with RBAC entries https://review.opendev.org/c/openstack/neutron-lib/+/885082 | 17:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib stable/xena: Add a "GROUP BY" clause on queries with RBAC entries https://review.opendev.org/c/openstack/neutron-lib/+/885083 | 17:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib stable/wallaby: Add a "GROUP BY" clause on queries with RBAC entries https://review.opendev.org/c/openstack/neutron-lib/+/885084 | 17:16 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/victoria: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884193 | 18:54 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-odl stable/zed: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884189 | 18:55 |
| opendevreview | Merged openstack/networking-odl stable/xena: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/884905 | 19:01 |
| opendevreview | Merged openstack/neutron master: [ovn][ipv6] Skip test_update_router_admin_state https://review.opendev.org/c/openstack/neutron/+/885074 | 19:21 |
| opendevreview | Merged openstack/neutron master: Raise the timeout of "neutron-ovn-rally-task" to 9000 https://review.opendev.org/c/openstack/neutron/+/885045 | 19:21 |
| opendevreview | Merged openstack/networking-odl stable/victoria: Fix tox4 issues and define project upper constraints https://review.opendev.org/c/openstack/networking-odl/+/885054 | 19:37 |
| opendevreview | Merged openstack/networking-odl stable/victoria: Remove periodic-stable-jobs template https://review.opendev.org/c/openstack/networking-odl/+/884193 | 19:43 |
| opendevreview | Merged openstack/neutron master: Implement ``get_subnets_address_scopes`` method https://review.opendev.org/c/openstack/neutron/+/882540 | 21:30 |
| *** JayF is now known as Guest1962 | 21:37 | |
| *** JasonF is now known as JayF | 21:37 | |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 23:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!