opendevreview | ZhouHeng proposed openstack/neutron-tempest-plugin master: list floating ip api contains port forwarding id https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/855608 | 00:57 |
---|---|---|
*** ramishra_ is now known as ramishra | 02:56 | |
opendevreview | Merged openstack/neutron master: Fix test_nova_send_events_* tests https://review.opendev.org/c/openstack/neutron/+/856034 | 06:36 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add new role "prepare_functional_tests_logs" https://review.opendev.org/c/openstack/neutron/+/855868 | 07:30 |
ykarel | slaweq, oslo db release also broke some functional tests https://bugs.launchpad.net/neutron/+bug/1989057 | 07:33 |
slaweq | ykarel: ouch | 07:45 |
opendevreview | Felix Huettner proposed openstack/neutron master: Cleanup fanout queues on ovs agent stop (part 2) https://review.opendev.org/c/openstack/neutron/+/856411 | 07:46 |
slaweq | will You take care of it? Or do You want me to check that bug? (I have env to run such tests already) | 07:46 |
ykarel | slaweq, i am looking, but not sure yet where to fix | 07:56 |
slaweq | ykarel: ok | 07:59 |
opendevreview | Matus Brandys proposed openstack/neutron-lib master: Added new DH groups to vpn to also supports well know EC algotithms. https://review.opendev.org/c/openstack/neutron-lib/+/828276 | 08:25 |
slaweq | ykarel: I have a patch, I will send it in 5 minutes to review | 08:35 |
slaweq | I'm not sure it's 100% correct but it makes tests to be green at least :) | 08:35 |
ykarel | slaweq, i also got it green locally, and preparing for review :) | 08:36 |
ykarel | not sure if it's same as yours | 08:36 |
slaweq | ykarel: ok, so please send Yours first :) | 08:36 |
slaweq | I just added self.context.session.commit() after each self.l3_plugin._get_dvr_subnet_ids_on_host_query() in that test | 08:37 |
ykarel | i created a new method _get_dvr_subnet_ids_on_host_query which will return subnet ids instead of the query | 08:37 |
ykarel | s/_get_dvr_subnet_ids_on_host | 08:38 |
slaweq | that makes sense :) | 08:38 |
slaweq | I think your will be better | 08:38 |
opendevreview | Matus Brandys proposed openstack/neutron-vpnaas master: Added new DH groups to vpn to also supports well know EC algotithms. Changes also needed to be merged in neutron_lib library and neutron package https://review.opendev.org/c/openstack/neutron-vpnaas/+/828275 | 08:39 |
opendevreview | Yifan Li proposed openstack/neutron master: [OVN] Adding support for VNIC type virtio-forwarder. https://review.opendev.org/c/openstack/neutron/+/856441 | 08:51 |
opendevreview | yatin proposed openstack/neutron master: Add a method to get dvr subnet_ids on host https://review.opendev.org/c/openstack/neutron/+/856444 | 08:55 |
ykarel | slaweq, lajoskatona ^ | 08:55 |
lajoskatona | ykarel: looking, this is for oslo.db release? | 08:55 |
ykarel | lajoskatona, yes | 08:55 |
ykarel | new oslo.db broke these tests | 08:55 |
opendevreview | Lajos Katona proposed openstack/networking-odl master: DNM: test master https://review.opendev.org/c/openstack/networking-odl/+/856448 | 09:24 |
opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: DNM: Check master with fresh oslo.db https://review.opendev.org/c/openstack/networking-bagpipe/+/856449 | 09:28 |
opendevreview | Lajos Katona proposed openstack/networking-sfc master: DNM: test master with fresh oslo.db https://review.opendev.org/c/openstack/networking-sfc/+/856450 | 09:32 |
opendevreview | liujinxin proposed openstack/neutron master: For DvrEdgeRouter, snat namespace should not be created in initialize. https://review.opendev.org/c/openstack/neutron/+/855995 | 09:33 |
opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: DNM: test master with fresh oslo.db https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/856451 | 09:35 |
opendevreview | Lajos Katona proposed openstack/neutron-vpnaas master: DNM: test master with fresh oslo.db https://review.opendev.org/c/openstack/neutron-vpnaas/+/856452 | 09:38 |
lajoskatona | slaweq, ykarel:FYI: fwaas seems to be affected by oslo.db release also | 09:40 |
opendevreview | Lajos Katona proposed openstack/neutron master: Doc: Refresh NDP proxy doc https://review.opendev.org/c/openstack/neutron/+/856274 | 09:50 |
opendevreview | Lajos Katona proposed openstack/neutron master: Doc: Refresh NDP proxy doc https://review.opendev.org/c/openstack/neutron/+/856274 | 09:54 |
opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: Use n-lib registry decorator https://review.opendev.org/c/openstack/neutron/+/856456 | 09:59 |
opendevreview | Mamatisa Nurmatov proposed openstack/neutron master: Use n-lib registry decorator https://review.opendev.org/c/openstack/neutron/+/856456 | 10:01 |
opendevreview | Felix Huettner proposed openstack/neutron master: Cleanup fanout queues on ovs agent stop (part 2) https://review.opendev.org/c/openstack/neutron/+/856411 | 10:07 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Functional] Close session explicitly after checking dvr subnets https://review.opendev.org/c/openstack/neutron/+/856461 | 10:38 |
slaweq | ykarel: obondarev lajoskatona ^^ alternative fix for the functional tests with oslo_db 12.1.0 | 10:39 |
slaweq | please check when You will have time | 10:39 |
ykarel | lajoskatona, slaweq ack | 10:40 |
ykarel | check obondarev comments now | 10:40 |
ykarel | s/check/checking | 10:40 |
lajoskatona | slaweq: checking | 11:16 |
frickler | lajoskatona: slaweq: that ndp proxy fix broke OSC testing, likely needs to add address-scope added, maybe one of you has time for that? https://zuul.opendev.org/t/openstack/build/96c981b70b5640c5ba2c21642fcf2b59 yangjianfeng seems to have left IRC again | 11:33 |
lajoskatona | frickler: cool :-) thanks for checking it | 12:06 |
*** dasm|off is now known as dasm | 12:56 | |
opendevreview | yatin proposed openstack/neutron master: Make _get_dvr_subnet_ids_on_host_query return subnet ids https://review.opendev.org/c/openstack/neutron/+/856444 | 13:06 |
lajoskatona | frickler: https://review.opendev.org/c/openstack/python-openstackclient/+/856521 | 13:16 |
opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: Adopt to latest VlanManager and oslo.db changes https://review.opendev.org/c/openstack/neutron-fwaas/+/855891 | 14:20 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Reset device namespace when adding to the namespace fails https://review.opendev.org/c/openstack/neutron/+/856261 | 14:59 |
opendevreview | yatin proposed openstack/neutron master: Make _get_dvr_subnet_ids_on_host_query return subnet ids https://review.opendev.org/c/openstack/neutron/+/856444 | 15:50 |
opendevreview | yatin proposed openstack/neutron master: Make _get_dvr_subnet_ids_on_host_query return subnet ids https://review.opendev.org/c/openstack/neutron/+/856444 | 15:58 |
ykarel | slaweq, i think can go with your patch but need to fix the comment and other failures | 16:11 |
*** gibi is now known as gibi_off | 16:16 | |
ykarel | ahhk seems you left, will update your patch, so we have results | 16:19 |
opendevreview | yatin proposed openstack/neutron master: [Functional] Fix tests for oslo.db>=12.1.0 https://review.opendev.org/c/openstack/neutron/+/856461 | 16:35 |
ykarel | done, /me also leaves now | 16:36 |
ykarel | lajoskatona, obondarev_ please check ^ once you around | 16:36 |
*** ykarel is now known as ykarel|away | 16:37 | |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Functional] Fix tests for oslo.db>=12.1.0 https://review.opendev.org/c/openstack/neutron/+/856461 | 17:48 |
*** dasm is now known as dasm|off | 21:13 | |
opendevreview | Merged openstack/neutron master: Doc: Refresh NDP proxy doc https://review.opendev.org/c/openstack/neutron/+/856274 | 21:48 |
opendevreview | Merged openstack/neutron master: [Functional] Fix tests for oslo.db>=12.1.0 https://review.opendev.org/c/openstack/neutron/+/856461 | 22:29 |
opendevreview | Merged openstack/neutron master: Add dsvm-functional-gate tox environment https://review.opendev.org/c/openstack/neutron/+/856262 | 23:16 |
opendevreview | Merged openstack/neutron stable/xena: [ovn] Specify port type if it's a router port when updating https://review.opendev.org/c/openstack/neutron/+/855063 | 23:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!