| opendevreview | liuyulong proposed openstack/neutron master: [Trivial] speed up l3_db floating IP port janitor DB query https://review.opendev.org/c/openstack/neutron/+/888200 | 02:18 |
|---|---|---|
| ykarel | gmann, ack /me will also check this week what is causing the failure when they run parallel | 04:50 |
| opendevreview | yatin proposed openstack/neutron master: Add unit tests periodic jobs to the experimental queue https://review.opendev.org/c/openstack/neutron/+/887951 | 05:22 |
| opendevreview | Maxim Korezkij proposed openstack/neutron master: [neutron-api] remove leader_only for sb connection https://review.opendev.org/c/openstack/neutron/+/885240 | 06:16 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add initial support for local OVN cluster instead of kernel-networking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/881779 | 06:39 |
| *** elvira1 is now known as elvira | 07:58 | |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add initial support for local OVN cluster instead of kernel-networking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/881779 | 09:36 |
| opendevreview | Bracha Frenkel proposed openstack/neutron-tempest-plugin master: adds a test for SSH connectivity for 3 VMs on the same network https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/888222 | 09:51 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add initial support for local OVN cluster instead of kernel-networking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/881779 | 10:28 |
| opendevreview | Merged openstack/neutron master: [neutron-api] remove leader_only for sb connection https://review.opendev.org/c/openstack/neutron/+/885240 | 10:29 |
| stephenfin | ralonsoh: Just an FYI, I'm still seeing failures on a proposed sqlalchemy-2x job against requirements https://review.opendev.org/c/openstack/requirements/+/879743 | 14:16 |
| stephenfin | I haven't looked into it beyond making sure that it is neutron tests that are failing, so there's a chance it's something _else_ causing the issue, but I'm letting you know since I know you have sqlalchemy jobs that might not be covering everything you think they are | 14:17 |
| ralonsoh | stephenfin, let me check | 14:17 |
| stephenfin | Oh, and I'm talking about the cross-neutron-py310 job naturally https://zuul.opendev.org/t/openstack/build/b86c934c290c43aa8ce47f768a0e7230 | 14:17 |
| ralonsoh | stephenfin, we don't run UTs with sqlalchemy + oslo.db master | 14:22 |
| ralonsoh | I'll check it manually now | 14:22 |
| ralonsoh | and I'll propose a fix (if that is a problem in our UTs) | 14:22 |
| ralonsoh | I'll ping you ASAP | 14:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Network "repr" should be tested with any order https://review.opendev.org/c/openstack/neutron/+/888286 | 15:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Define one DB model per "FromClause.join" clause https://review.opendev.org/c/openstack/neutron/+/888295 | 16:05 |
| ralonsoh | stephenfin, ^^ both patches | 16:05 |
| ralonsoh | we should be good now, I'm locally testing now | 16:06 |
| stephenfin | great, thank you for the quick turnaround! :) | 16:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "neutron-tox-py310-with-sqlalchemy-master" CI job https://review.opendev.org/c/openstack/neutron/+/888298 | 16:39 |
| ralonsoh | rbacosta, hello? | 16:39 |
| rbacosta | Hello ralonsoh | 16:40 |
| rbacosta | ralonsoh, about your comment in the review: https://review.opendev.org/c/openstack/neutron/+/867513/21..22/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py#b2936 | 16:40 |
| ralonsoh | what do you need in L2935? The router ID, right? | 16:41 |
| rbacosta | Can you provide some example of how to do this directly in ovn without taking it from the neutron bank? | 16:41 |
| rbacosta | I need the router name for the NAT rule transaction | 16:42 |
| ralonsoh | so you can list the Logical_Router_Ports stored in the NB DB | 16:42 |
| ralonsoh | and find this one with external_ids:"neutron:subnet_ids" == <subnet_id> | 16:43 |
| ralonsoh | right? | 16:43 |
| rbacosta | List all ports? From what I've seen there's no way to user filter on get Logical_Router_Ports. | 16:44 |
| ralonsoh | no, you should do that manually | 16:44 |
| rbacosta | hummm, okay.I didn't see anyone doing it like that in mech_driver, I'll try. | 16:46 |
| ralonsoh | you have an example in ovsdbapp: QoSDelExtIdCommand | 16:46 |
| ralonsoh | finding the qos register with a specific external_ids register | 16:47 |
| rbacosta | I'll look at this, thank you very much. | 16:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Use the correct OVO field type https://review.opendev.org/c/openstack/neutron/+/888305 | 17:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "neutron-tox-py310-with-sqlalchemy-master" CI job https://review.opendev.org/c/openstack/neutron/+/888298 | 17:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Define one DB model per "FromClause.join" clause https://review.opendev.org/c/openstack/neutron/+/888295 | 17:53 |
| ralonsoh | haleyb, mlavalle, please review these 4 patches (https://review.opendev.org/c/openstack/neutron/+/888298) related to sqlalchemy. The last one introduces the validation with the "neutron-tox-py310-with-sqlalchemy-master" job. | 17:59 |
| ralonsoh | please note the failing UTs jobs are due to an issue in n-lib.master that slaweq is addressing now | 18:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Use the correct OVO field type https://review.opendev.org/c/openstack/neutron/+/888305 | 18:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "neutron-tox-py310-with-sqlalchemy-master" CI job https://review.opendev.org/c/openstack/neutron/+/888298 | 18:01 |
| opendevreview | Merged openstack/neutron master: Add unit tests periodic jobs to the experimental queue https://review.opendev.org/c/openstack/neutron/+/887951 | 19:36 |
| opendevreview | Roberto Acosta proposed openstack/neutron master: [RFE] [OVN] - IPv6 Distributed routing https://review.opendev.org/c/openstack/neutron/+/867513 | 20:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!