| *** ramishra_ is now known as ramishra | 05:38 | |
| lajoskatona | ralonsoh: Good morning, could you check this req bump: https://review.opendev.org/c/openstack/neutron/+/809649 , l-c job broke last week, and it seems fixes it | 06:34 |
|---|---|---|
| ralonsoh | lajoskatona, sure,let me check | 06:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Revert "update subport status when trunk/subport create/delete is triggerred" https://review.opendev.org/c/openstack/neutron/+/809550 | 07:21 |
| *** rpittau|afk is now known as rpittau | 07:55 | |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: BFD validator https://review.opendev.org/c/openstack/neutron-lib/+/809978 | 09:07 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/809580 | 09:12 |
| opendevreview | Lajos Katona proposed openstack/neutron master: BFD for Neutron, server side https://review.opendev.org/c/openstack/neutron/+/808046 | 09:13 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron-lib master: Remove deprecated method registry.notify https://review.opendev.org/c/openstack/neutron-lib/+/807869 | 09:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Execute the quota reservation removal in an isolated DB txn https://review.opendev.org/c/openstack/neutron/+/809983 | 09:32 |
| ralonsoh | slaweq, lajoskatona ^^ | 09:32 |
| lajoskatona | ralonsoh: checking | 09:33 |
| ralonsoh | I'm not against https://review.opendev.org/c/openstack/neutron/+/809191, but ^^ isolates the DB transaction as intended in https://review.opendev.org/c/openstack/neutron/+/805031 | 09:33 |
| ralonsoh | I need to change transaction_guard to accept methods outside classes | 09:37 |
| slaweq | ralonsoh: thx | 09:42 |
| lajoskatona | ralonsoh, slaweq: do we have to wait with rc for https://review.opendev.org/c/openstack/neutron/+/809983 , or https://review.opendev.org/c/openstack/neutron/+/809191 is enough ? | 10:05 |
| ralonsoh | lajoskatona, I think https://review.opendev.org/c/openstack/neutron/+/809191 is valid for RC (and tested) | 10:06 |
| ralonsoh | that will partially solve the problem we has before | 10:06 |
| ralonsoh | and then, for Y, I'll propose the other one | 10:06 |
| ralonsoh | I'll +2 https://review.opendev.org/c/openstack/neutron/+/809191 for RC | 10:07 |
| ralonsoh | then we'll test my patch for Y | 10:07 |
| ralonsoh | obondarev, ^^ | 10:08 |
| ralonsoh | I have +2 the patch https://review.opendev.org/c/openstack/neutron/+/809191 | 10:08 |
| ralonsoh | but I'll propose mine for Y | 10:08 |
| lajoskatona | ralonsoh: thanks | 10:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Execute the quota reservation removal in an isolated DB txn https://review.opendev.org/c/openstack/neutron/+/809983 | 10:21 |
| obondarev | ralonsoh: do you think it's safe to rollback transaction not started by current function and proceed execution? | 10:36 |
| ralonsoh | obondarev, from https://review.opendev.org/c/openstack/neutron/+/809191/1/neutron/db/quota/api.py#221 | 10:40 |
| ralonsoh | there are three places | 10:40 |
| ralonsoh | [1], this is safe, that will revert only the reservations deleted | 10:40 |
| ralonsoh | [4] same here | 10:41 |
| ralonsoh | and [5], that will revert the whole SG creation | 10:41 |
| obondarev | ralonsoh: understood, thanks! | 10:41 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: WIP: check master https://review.opendev.org/c/openstack/neutron-lib/+/810003 | 12:23 |
| opendevreview | Balazs Gibizer proposed openstack/neutron master: [DNM]See if tempest passes with the new mod https://review.opendev.org/c/openstack/neutron/+/809170 | 12:25 |
| TheJulia | slaweq: Remember you pointing me at an rfe last week for neutron-openvswitch-agent exploding in startup? | 13:16 |
| TheJulia | slaweq: basically firewall startup in __init__ is failing often enough with devstack that we're likely going to disable neutron firewalling entirely for Ironic jobs at this point just to have a working gate again. | 13:21 |
| slaweq | TheJulia: ok, but please report bug for us with that | 13:24 |
| slaweq | I will try to get to this soon | 13:25 |
| TheJulia | slaweq: okay, will do shortly | 13:27 |
| slaweq | TheJulia: thx | 13:28 |
| TheJulia | slaweq: https://bugs.launchpad.net/neutron/+bug/1944201 | 14:05 |
| slaweq | TheJulia: thx | 14:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Open Yoga DB branch https://review.opendev.org/c/openstack/neutron/+/810036 | 15:09 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: WIP: Bump requirements and disable f-string check https://review.opendev.org/c/openstack/neutron-lib/+/810003 | 15:27 |
| pjakuszew | I wanted to check whether it is possible to split up L3 agent router processing queue into parallel workers, but I see that this process is dealt with via greenlets | 15:53 |
| pjakuszew | so, this is probably not possible to implement? | 15:53 |
| pjakuszew | not sure if I'm understanding the code correctly | 15:54 |
| pjakuszew | what I want to achieve is to make L3 Agent spawn the routers faster | 15:55 |
| pjakuszew | initial idea was to make it spawn multiple routers in parallel, with appropriate sorting so multiple actions regarding the same router wouldn't be incorrectly processed (prevention of race conditions) | 15:57 |
| pjakuszew | I noticed the greenlet part though, and according to its documentation it is deliberately *not* parallel | 15:57 |
| *** rpittau is now known as rpittau|afk | 16:00 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add new indexes to RBAC DB models https://review.opendev.org/c/openstack/neutron/+/810072 | 16:06 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!