Thursday, 2022-09-08

opendevreviewZhouHeng proposed openstack/neutron-tempest-plugin master: list floating ip api contains port forwarding id  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/85560800:57
*** ramishra_ is now known as ramishra02:56
opendevreviewMerged openstack/neutron master: Fix test_nova_send_events_* tests  https://review.opendev.org/c/openstack/neutron/+/85603406:36
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add new role "prepare_functional_tests_logs"  https://review.opendev.org/c/openstack/neutron/+/85586807:30
ykarelslaweq, oslo db release also broke some functional tests https://bugs.launchpad.net/neutron/+bug/198905707:33
slaweqykarel: ouch07:45
opendevreviewFelix Huettner proposed openstack/neutron master: Cleanup fanout queues on ovs agent stop (part 2)  https://review.opendev.org/c/openstack/neutron/+/85641107:46
slaweqwill You take care of it? Or do You want me to check that bug? (I have env to run such tests already)07:46
ykarelslaweq, i am looking, but not sure yet where to fix07:56
slaweqykarel: ok07:59
opendevreviewMatus 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/+/82827608:25
slaweqykarel: I have a patch, I will send it in 5 minutes to review08:35
slaweqI'm not sure it's 100% correct but it makes tests to be green at least :)08:35
ykarelslaweq, i also got it green locally, and preparing for review :)08:36
ykarelnot sure if it's same as yours08:36
slaweqykarel: ok, so please send Yours first :)08:36
slaweqI just added self.context.session.commit() after each self.l3_plugin._get_dvr_subnet_ids_on_host_query() in that test08:37
ykareli created a new method _get_dvr_subnet_ids_on_host_query which will return subnet ids instead of the query08:37
ykarels/_get_dvr_subnet_ids_on_host08:38
slaweqthat makes sense :)08:38
slaweqI think your will be better08:38
opendevreviewMatus 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/+/82827508:39
opendevreviewYifan Li proposed openstack/neutron master: [OVN] Adding support for VNIC type virtio-forwarder.  https://review.opendev.org/c/openstack/neutron/+/85644108:51
opendevreviewyatin proposed openstack/neutron master: Add a method to get dvr subnet_ids on host  https://review.opendev.org/c/openstack/neutron/+/85644408:55
ykarelslaweq, lajoskatona ^08:55
lajoskatonaykarel: looking, this is for oslo.db release?08:55
ykarellajoskatona, yes08:55
ykarelnew oslo.db broke these tests08:55
opendevreviewLajos Katona proposed openstack/networking-odl master: DNM: test master  https://review.opendev.org/c/openstack/networking-odl/+/85644809:24
opendevreviewLajos Katona proposed openstack/networking-bagpipe master: DNM: Check master with fresh oslo.db  https://review.opendev.org/c/openstack/networking-bagpipe/+/85644909:28
opendevreviewLajos Katona proposed openstack/networking-sfc master: DNM: test master with fresh oslo.db  https://review.opendev.org/c/openstack/networking-sfc/+/85645009:32
opendevreviewliujinxin proposed openstack/neutron master: For DvrEdgeRouter, snat namespace should not be created in initialize.  https://review.opendev.org/c/openstack/neutron/+/85599509:33
opendevreviewLajos Katona proposed openstack/neutron-dynamic-routing master: DNM: test master with fresh oslo.db  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/85645109:35
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: DNM: test master with fresh oslo.db  https://review.opendev.org/c/openstack/neutron-vpnaas/+/85645209:38
lajoskatonaslaweq, ykarel:FYI: fwaas seems to be affected by oslo.db release also09:40
opendevreviewLajos Katona proposed openstack/neutron master: Doc: Refresh NDP proxy doc  https://review.opendev.org/c/openstack/neutron/+/85627409:50
opendevreviewLajos Katona proposed openstack/neutron master: Doc: Refresh NDP proxy doc  https://review.opendev.org/c/openstack/neutron/+/85627409:54
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: Use n-lib registry decorator  https://review.opendev.org/c/openstack/neutron/+/85645609:59
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: Use n-lib registry decorator  https://review.opendev.org/c/openstack/neutron/+/85645610:01
opendevreviewFelix Huettner proposed openstack/neutron master: Cleanup fanout queues on ovs agent stop (part 2)  https://review.opendev.org/c/openstack/neutron/+/85641110:07
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [Functional] Close session explicitly after checking dvr subnets  https://review.opendev.org/c/openstack/neutron/+/85646110:38
slaweqykarel: obondarev lajoskatona ^^ alternative fix for the functional tests with oslo_db 12.1.010:39
slaweqplease check when You will have time10:39
ykarellajoskatona, slaweq ack10:40
ykarelcheck obondarev comments now10:40
ykarels/check/checking10:40
lajoskatonaslaweq: checking11:16
fricklerlajoskatona: 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 again11:33
lajoskatonafrickler: cool :-) thanks for checking it12:06
*** dasm|off is now known as dasm12:56
opendevreviewyatin proposed openstack/neutron master: Make _get_dvr_subnet_ids_on_host_query return subnet ids  https://review.opendev.org/c/openstack/neutron/+/85644413:06
lajoskatonafrickler: https://review.opendev.org/c/openstack/python-openstackclient/+/85652113:16
opendevreviewLajos Katona proposed openstack/neutron-fwaas master: Adopt to latest VlanManager and oslo.db changes  https://review.opendev.org/c/openstack/neutron-fwaas/+/85589114:20
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Reset device namespace when adding to the namespace fails  https://review.opendev.org/c/openstack/neutron/+/85626114:59
opendevreviewyatin proposed openstack/neutron master: Make _get_dvr_subnet_ids_on_host_query return subnet ids  https://review.opendev.org/c/openstack/neutron/+/85644415:50
opendevreviewyatin proposed openstack/neutron master: Make _get_dvr_subnet_ids_on_host_query return subnet ids  https://review.opendev.org/c/openstack/neutron/+/85644415:58
ykarelslaweq, i think can go with your patch but need to fix the comment and other failures16:11
*** gibi is now known as gibi_off16:16
ykarelahhk seems you left, will update your patch, so we have results16:19
opendevreviewyatin proposed openstack/neutron master: [Functional] Fix tests for oslo.db>=12.1.0  https://review.opendev.org/c/openstack/neutron/+/85646116:35
ykareldone, /me also leaves now16:36
ykarellajoskatona, obondarev_ please check ^ once you around16:36
*** ykarel is now known as ykarel|away16:37
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [Functional] Fix tests for oslo.db>=12.1.0  https://review.opendev.org/c/openstack/neutron/+/85646117:48
*** dasm is now known as dasm|off21:13
opendevreviewMerged openstack/neutron master: Doc: Refresh NDP proxy doc  https://review.opendev.org/c/openstack/neutron/+/85627421:48
opendevreviewMerged openstack/neutron master: [Functional] Fix tests for oslo.db>=12.1.0  https://review.opendev.org/c/openstack/neutron/+/85646122:29
opendevreviewMerged openstack/neutron master: Add dsvm-functional-gate tox environment  https://review.opendev.org/c/openstack/neutron/+/85626223:16
opendevreviewMerged openstack/neutron stable/xena: [ovn] Specify port type if it's a router port when updating  https://review.opendev.org/c/openstack/neutron/+/85506323:39

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!