| opendevreview | liuyulong proposed openstack/neutron master: Remove block flow when port UP https://review.opendev.org/c/openstack/neutron/+/838444 | 01:28 |
|---|---|---|
| opendevreview | liuyulong proposed openstack/neutron master: Remove useless function _add_port_tag_info https://review.opendev.org/c/openstack/neutron/+/838445 | 01:28 |
| opendevreview | OpenStack Proposal Bot proposed openstack/neutron master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron/+/838078 | 03:39 |
| slaweq | #endmeeting | 06:54 |
| opendevmeet | Meeting ended Wed Apr 20 06:54:38 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 06:54 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-04-19-15.01.html | 06:54 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-04-19-15.01.txt | 06:54 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-04-19-15.01.log.html | 06:54 |
| ralonsoh | that was a very long meeting | 06:56 |
| opendevreview | Zhang Hua proposed openstack/neutron stable/xena: Fix list of DNS extensions supported by OVN https://review.opendev.org/c/openstack/neutron/+/838650 | 07:44 |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: WIP: check master status https://review.opendev.org/c/openstack/networking-odl/+/838451 | 07:47 |
| opendevreview | Merged openstack/neutron stable/yoga: ovn: Wait for northd in functional tests https://review.opendev.org/c/openstack/neutron/+/837792 | 08:57 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Keep "connectivity" VIF details parameter in migration https://review.opendev.org/c/openstack/neutron/+/838537 | 09:06 |
| ralonsoh | folks, if you have some mins: https://review.opendev.org/c/openstack/neutron-lib/+/837590 | 09:10 |
| ralonsoh | thanks in advance | 09:10 |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: WIP: check master status https://review.opendev.org/c/openstack/networking-odl/+/838451 | 09:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DHCP] Break reference chain to any Exception object when resync https://review.opendev.org/c/openstack/neutron/+/838492 | 09:29 |
| lajoskatona | ralonsoh: done | 09:43 |
| ralonsoh | lajoskatona, thanks! | 09:43 |
| lajoskatona | counter review if you have some time: these are for zuul cfg errors for taas: https://review.opendev.org/q/project:openstack/tap-as-a-service+status:open+label:Verified%253D%252B1+-age:15d | 09:43 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: tests: Use Zed job definitions https://review.opendev.org/c/openstack/tap-as-a-service/+/834821 | 09:46 |
| ralonsoh | lajoskatona, in https://review.opendev.org/q/project:openstack/tap-as-a-service+status:open+label:Verified%253D%252B1+-age:15d | 09:55 |
| ralonsoh | ussuri is missing | 09:55 |
| ralonsoh | is that correct? | 09:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][L3] QoS inherit network https://review.opendev.org/c/openstack/neutron/+/819147 | 10:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Implement GW IP network QoS inheritance https://review.opendev.org/c/openstack/neutron/+/838678 | 10:15 |
| *** ralonsoh_ is now known as ralonsoh | 10:15 | |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Keep "connectivity" VIF details parameter in migration https://review.opendev.org/c/openstack/neutron/+/838538 | 10:30 |
| opendevreview | Merged openstack/neutron stable/xena: Retry port_update in the OVN if revision mismatch during live-migration https://review.opendev.org/c/openstack/neutron/+/837678 | 10:30 |
| opendevreview | Merged openstack/neutron stable/victoria: Retry port_update in the OVN if revision mismatch during live-migration https://review.opendev.org/c/openstack/neutron/+/837707 | 10:47 |
| opendevreview | Merged openstack/neutron stable/xena: [API] Return 403 for POST requests when user is not authorized https://review.opendev.org/c/openstack/neutron/+/837488 | 10:48 |
| opendevreview | Merged openstack/neutron stable/wallaby: [API] Return 403 for POST requests when user is not authorized https://review.opendev.org/c/openstack/neutron/+/837489 | 10:48 |
| lajoskatona | ralonsoh: yeah, there is no ussuri branch for taas, it was a surprise for me also | 11:08 |
| ralonsoh | ahh ok | 11:08 |
| opendevreview | Merged openstack/neutron stable/yoga: Retry port_update in the OVN if revision mismatch during live-migration https://review.opendev.org/c/openstack/neutron/+/837677 | 11:35 |
| opendevreview | Merged openstack/neutron stable/wallaby: Retry port_update in the OVN if revision mismatch during live-migration https://review.opendev.org/c/openstack/neutron/+/837679 | 12:21 |
| opendevreview | Merged openstack/neutron master: Remove block flow when port UP https://review.opendev.org/c/openstack/neutron/+/838444 | 12:21 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: Remove unimplemented api-def: BFD monitors https://review.opendev.org/c/openstack/neutron-lib/+/838720 | 12:33 |
| bkra | hi there, im having a hard time finding the config param to set neturon ML2 endpoint, can somebody help me out please? :) | 12:45 |
| opendevreview | Merged openstack/neutron stable/wallaby: When creating a VXLAN interface, a device is mandatory https://review.opendev.org/c/openstack/neutron/+/833379 | 12:46 |
| bkra | ml2_conf.ini seems not to contain to set a northbound api endpoint | 12:47 |
| opendevreview | Merged openstack/neutron stable/yoga: Force security_group_id uuid validation of sg rules https://review.opendev.org/c/openstack/neutron/+/837308 | 12:50 |
| lajoskatona | bkra: Hi, it is ovn_nb_connection, for example here: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8e6/838445/2/experimental/neutron-ovn-tempest-with-neutron-lib-master/8e66581/controller/logs/etc/neutron/plugins/ml2/ml2_conf.ini | 13:15 |
| bkra | thanks! | 13:45 |
| *** dasm|off is now known as dasm | 14:03 | |
| bcafarel | lajoskatona: o/ I was checking pending xena/wallaby/victoria backports, list is almost empty right now, may be a good time to update https://review.opendev.org/c/openstack/releases/+/837728 | 14:06 |
| lajoskatona | bcafarel: thanks I will update | 14:27 |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: WIP: check master status https://review.opendev.org/c/openstack/networking-odl/+/838451 | 14:41 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/yoga: Handle properly InvalidScope exceptions to not return error 500 https://review.opendev.org/c/openstack/neutron/+/838697 | 14:51 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Handle properly InvalidScope exceptions to not return error 500 https://review.opendev.org/c/openstack/neutron/+/838698 | 14:52 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Handle properly InvalidScope exceptions to not return error 500 https://review.opendev.org/c/openstack/neutron/+/838699 | 14:52 |
| slaweq | lajoskatona: mlavalle ralonsoh hi, when You will have some time, please check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/838460 | 14:59 |
| slaweq | thx in advance | 14:59 |
| ralonsoh | sure | 14:59 |
| mlavalle | slaweq: will do | 14:59 |
| slaweq | thx | 15:00 |
| lajoskatona | slaweq: I just opened it :-) | 15:02 |
| ralonsoh | hi folks, let's merge this now Zed is starting | 15:10 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-lib/+/828738 | 15:10 |
| ralonsoh | if there is any problem, we can always revert it | 15:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Implement GW IP network QoS inheritance https://review.opendev.org/c/openstack/neutron/+/838678 | 15:20 |
| opendevreview | Merged openstack/neutron master: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/834613 | 17:18 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-specs master: Add spec for DNS subdomain support in OVN https://review.opendev.org/c/openstack/neutron-specs/+/832658 | 17:19 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-specs master: Add spec for DNS subdomain support in OVN https://review.opendev.org/c/openstack/neutron-specs/+/832658 | 17:23 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-specs master: Add spec for DNS subdomain support in OVN https://review.opendev.org/c/openstack/neutron-specs/+/832658 | 17:25 |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: WIP: check master status https://review.opendev.org/c/openstack/networking-odl/+/838451 | 17:33 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/yoga: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/838702 | 17:36 |
| lajoskatona | mnaser: Hi, do you have some news for neutron-vpnaas or do you need some help? | 17:39 |
| mnaser | lajoskatona: i have been trying to replicate the functional tests locally but that has been my main struggle is how to run the func tests locally :\ | 17:40 |
| mnaser | if there is more straight forward docs, that would help, but it's mostly trial and error right now for me | 17:41 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-specs master: Add spec for DNS subdomain support in OVN https://review.opendev.org/c/openstack/neutron-specs/+/832658 | 17:44 |
| opendevreview | Merged openstack/neutron master: [DHCP] Break reference chain to any Exception object when resync https://review.opendev.org/c/openstack/neutron/+/838492 | 17:48 |
| opendevreview | Merged openstack/neutron-lib master: Add "neutron-tempest-plugin-jobs" to CI https://review.opendev.org/c/openstack/neutron-lib/+/837590 | 18:21 |
| lajoskatona | mnaser: I would start with this: https://docs.openstack.org/neutron/latest/contributor/testing/testing.html#id3 and combining it with the script for vpnaas: https://opendev.org/openstack/neutron-vpnaas/src/branch/master/tools/configure_for_vpn_func_testing.sh | 18:38 |
| lajoskatona | mnaser: running Neutron functional workes locally quite well, but never tried vpnaas functional tests myself, but the concept originally was the same I suppose | 18:39 |
| mnaser | lajoskatona: ok that will be a useful doc to go over, i didnt see then neutron doc | 19:03 |
| mnaser | ran into another devstack issue now bc of aarch64 :) | 19:03 |
| lajoskatona | mnaser: ok, hope it will help | 19:04 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: ovn: revert to stateful dnat_and_snat https://review.opendev.org/c/openstack/neutron/+/838776 | 19:15 |
| opendevreview | Carlos Eduardo proposed openstack/neutron-dynamic-routing master: Register common config options by default https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/838780 | 19:39 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/yoga: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/838702 | 19:48 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/xena: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/838783 | 19:56 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/wallaby: Fix setting table monitoring conditions https://review.opendev.org/c/openstack/neutron/+/838784 | 20:03 |
| mnaser | lajoskatona: i am feeling this might be related - https://github.com/openstack/neutron-vpnaas/commit/a46f7d601568a56437a0cc84c32917da9dc591bc | 20:21 |
| mnaser | im seeing a bunch of interfaces going up with 4095 | 20:22 |
| mnaser | ok looks like .plug was moved to `neutron_lib.agent.linux import interface` recently too | 20:24 |
| mnaser | https://github.com/openstack/neutron/commit/0ddca284542aed89df4a22607a2da03f193f083c#diff-2071c4bffc631f327efb008589a0e849fc3dbea31d202a80833a60eb6b9ff3c4 | 20:27 |
| mnaser | looks like the code in that commit added more clearing to attribute | 20:27 |
| mnaser | im wondering if there is a cleaner way of resetting this using a helper rather than adding those exgtra lines | 20:27 |
| mnaser | yup, that's exactly it | 20:35 |
| mnaser | https://github.com/openstack/neutron/commit/0ddca284542aed89df4a22607a2da03f193f083c change broke vpnaas most likely, but im trying to think of a cleaner way than adding a few lines like it was before | 20:35 |
| mnaser | ok, TestIPSecBase is inherited from BaseSudoTestCase.. it shoudl be `L3AgentTestFramework` which covers all this stuff | 20:38 |
| mnaser | time to hack hack hack | 20:38 |
| *** dasm is now known as dasm|off | 20:53 | |
| opendevreview | Mohammed Naser proposed openstack/neutron-vpnaas master: tests: fix functional tests https://review.opendev.org/c/openstack/neutron-vpnaas/+/838789 | 21:18 |
| mnaser | lajoskatona: ^ works locally, lets hope it passes and then we can catch up on the backlog | 21:18 |
| opendevreview | Mohammed Naser proposed openstack/neutron-vpnaas master: tests: fix functional tests https://review.opendev.org/c/openstack/neutron-vpnaas/+/838789 | 21:54 |
| opendevreview | Mohammed Naser proposed openstack/neutron-vpnaas master: tests: fix functional tests https://review.opendev.org/c/openstack/neutron-vpnaas/+/838789 | 23:21 |
| opendevreview | Mohammed Naser proposed openstack/neutron-vpnaas master: ci: temporarily disable functional tests https://review.opendev.org/c/openstack/neutron-vpnaas/+/838795 | 23:21 |
| opendevreview | Mohammed Naser proposed openstack/neutron-vpnaas master: Register common config options by default https://review.opendev.org/c/openstack/neutron-vpnaas/+/838796 | 23:21 |
| opendevreview | Mohammed Naser proposed openstack/neutron-vpnaas master: tests: restore functional testing https://review.opendev.org/c/openstack/neutron-vpnaas/+/838797 | 23:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!