| opendevreview | Ghanshyam proposed openstack/networking-bgpvpn master: DNM: Testing tempest pin on stable/ussuri https://review.opendev.org/c/openstack/networking-bgpvpn/+/843087 | 02:35 |
|---|---|---|
| opendevreview | Miro Tomaska proposed openstack/neutron master: Notify mech drivers with original and modified network. https://review.opendev.org/c/openstack/neutron/+/840146 | 02:57 |
| opendevreview | liuxie proposed openstack/neutron master: [ovn_l3]no need to get az list use ml2/ovn mech_driver https://review.opendev.org/c/openstack/neutron/+/843229 | 04:28 |
| opendevreview | liuxie proposed openstack/neutron master: [ovn_l3]no need to get az list use ml2/ovn mech_driver https://review.opendev.org/c/openstack/neutron/+/843229 | 05:53 |
| opendevreview | yatin proposed openstack/neutron master: Switch to cirros uec image in multinode jobs https://review.opendev.org/c/openstack/neutron/+/843237 | 06:28 |
| hjensas | ralonsoh: good morning. I create bug for the segment mapping issue - https://bugs.launchpad.net/neutron/+bug/1975542 | 07:04 |
| ralonsoh | hjensas, let me check | 07:05 |
| hjensas | ralonsoh: deleting and recreating the agent helps. The issue is segment mapping is never created when new networks/segments are added. | 07:05 |
| ralonsoh | hjensas, really? uffff.... | 07:05 |
| hjensas | ralonsoh: I think it may be sqlalchemy 2 related. | 07:05 |
| ralonsoh | hjensas, can I check that in a regular env? | 07:06 |
| ralonsoh | I mean, without any specific backend? | 07:06 |
| hjensas | I think so, - The check that return false and thus terminates the "add segment mapping" is here - https://opendev.org/openstack/neutron/src/branch/master/neutron/services/segments/db.py#L330-L334 | 07:09 |
| opendevreview | Merged openstack/tap-as-a-service master: Add context for all SQL transactions https://review.opendev.org/c/openstack/tap-as-a-service/+/842134 | 07:25 |
| opendevreview | yatin proposed openstack/neutron master: [WIP] Switch fullstack/functional fips jobs to 9-stream https://review.opendev.org/c/openstack/neutron/+/843245 | 07:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [Trunk] Check if parent port belongs to OVN to activate https://review.opendev.org/c/openstack/neutron/+/843247 | 08:34 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] fips fullstack/functional test without dbcounter https://review.opendev.org/c/openstack/neutron/+/843252 | 09:12 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Add baremetal support without Neutron DHCP agent for IPv4 https://review.opendev.org/c/openstack/neutron/+/840316 | 09:16 |
| opendevreview | Henning Eggers proposed openstack/neutron master: Defer flow deletion in openvswitch firewall https://review.opendev.org/c/openstack/neutron/+/843253 | 09:18 |
| ralonsoh | zzzeek_, hi, do you have 5 mins for https://bugs.launchpad.net/neutron/+bug/1975542? I'm checking what hjensas has reported in this LP bug and I can reproduce it always. | 09:33 |
| ralonsoh | The issue I detected is that the affected method is called **inside** a writer context and just after the creation of a new OVO (that creates some DB registers, I think 4) | 09:34 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 09:34 | |
| ralonsoh | I can see that, in "is_session_active", session.new is empty | 09:35 |
| ralonsoh | but session.transaction._new is not | 09:35 |
| ralonsoh | and has those 4 registers I was talking about (4 weakred to a InstanceState | 09:36 |
| ralonsoh | questions: shouldn't session.new read what is in session.transaction._new? | 09:36 |
| ralonsoh | should I change the code in this method? | 09:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Use session.transaction information to decide if active https://review.opendev.org/c/openstack/neutron/+/843256 | 10:10 |
| ralonsoh | zzzeek_, ^^^ | 10:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Skip "PortBindingChassisEvent" if revision number changes https://review.opendev.org/c/openstack/neutron/+/842147 | 10:38 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] fips fullstack/functional test without dbcounter https://review.opendev.org/c/openstack/neutron/+/843252 | 11:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] During the DB migration, allow the DB context to finish https://review.opendev.org/c/openstack/neutron/+/843263 | 11:18 |
| zzzeek_ | ralonsoh: OK i see what you're doing, i need to look more closely in a bit | 13:01 |
| ralonsoh | zzzeek_, thanks in advance | 13:01 |
| opendevreview | Ghanshyam proposed openstack/neutron stable/ussuri: DNM: testing old tempest https://review.opendev.org/c/openstack/neutron/+/838052 | 15:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove session active check in "_add_segment_host_mapping_for_segment" https://review.opendev.org/c/openstack/neutron/+/843294 | 15:15 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: taas: use ssh_key_type from config for taas scenario https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/842113 | 15:25 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: DNM: testing Tempest pin in stable/victoria https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/843299 | 15:49 |
| opendevreview | Ghanshyam proposed openstack/neutron stable/victoria: DNM: testing Tempest pin in stable/victoria https://review.opendev.org/c/openstack/neutron/+/843300 | 15:49 |
| opendevreview | Ghanshyam proposed openstack/neutron master: DNM: testing Tempest pin in stable/victoria https://review.opendev.org/c/openstack/neutron/+/843303 | 15:54 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix way of calculate LB status after HM event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/843308 | 15:55 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] fips fullstack/functional test without dbcounter https://review.opendev.org/c/openstack/neutron/+/843252 | 16:00 |
| opendevreview | Miguel Lavalle proposed openstack/os-vif master: Delete trunk bridges to avoid race with Neutron https://review.opendev.org/c/openstack/os-vif/+/841499 | 17:32 |
| opendevreview | Ghanshyam proposed openstack/neutron stable/ussuri: DNM: testing old tempest https://review.opendev.org/c/openstack/neutron/+/838052 | 20:21 |
| opendevreview | Miguel Lavalle proposed openstack/os-vif master: Delete trunk bridges to avoid race with Neutron https://review.opendev.org/c/openstack/os-vif/+/841499 | 21:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!