Monday, 2023-02-13

opendevreviewZhouHeng proposed openstack/neutron master: [ovn]l3 plugin support floating ip distributed attribues  https://review.opendev.org/c/openstack/neutron/+/85695502:29
opendevreviewZhouHeng proposed openstack/neutron master: [ovn]l3 plugin support floating ip distributed attribues  https://review.opendev.org/c/openstack/neutron/+/85695503:15
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: Remove FIPAddDeleteEvent event  https://review.opendev.org/c/openstack/neutron/+/86400003:43
opendevreviewMiguel Lavalle proposed openstack/neutron-tempest-plugin master: Use OVN version 22.06  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/84819804:20
opendevreviewZhouHeng proposed openstack/neutron master: [ovn]l3 plugin support floating ip distributed attribues  https://review.opendev.org/c/openstack/neutron/+/85695505:50
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: Remove FIPAddDeleteEvent event  https://review.opendev.org/c/openstack/neutron/+/86400007:04
lajoskatonaykarel, ralonsoh, slaweq: Hi, oslo-master job on py310 is failing (https://96a075deff7520d193c6-7b9374470c0105283123d1c69dc66ebe.ssl.cf5.rackcdn.com/periodic/opendev.org/openstack/neutron/master/openstack-tox-py310-with-oslo-master/cc807e5/job-output.txt )08:30
lajoskatona ykarel, ralonsoh, slaweq: I think this can fix it: https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/87349808:30
lajoskatonaykarel, ralonsoh, slaweq: though I don't know what else can g wrong around other openstack projects08:30
slaweqlajoskatona hi, I will look at it now08:31
lajoskatonaslaweq: thanks, it is just to change the parent of the openstack-tox-with-oslo-master-base job to py39 instead of py2708:32
ykarellajoskatona, ack, looks good08:32
ykareljust not sure if parent should be py310 or py3908:33
lajoskatonaykarel: yes, I had the same concern :-) as we have py39 also as sibling, I thought it is better to select py3908:34
ykarellajoskatona, reading Python 3.10 (available as default in Ubuntu 22.04)08:35
ykarelhttps://governance.openstack.org/tc/reference/runtimes/2023.1.html08:35
ykarelsame for https://governance.openstack.org/tc/reference/runtimes/2023.2.html08:36
*** ralonsoh_ is now known as ralonsoh08:37
ralonsohlajoskatona, let me check08:38
ralonsohlajoskatona, I don't know why that started to fail now, to be honest08:42
ralonsohthe patch looks correct08:43
ykarellajoskatona, may be better to switch those oslo master jobs same as neutron-lib master ones and rely on same version openstack-tox base jobs08:43
ykarelthe duplication part can be handled with yaml aliases08:43
ykarelralonsoh, for reference reason is https://review.opendev.org/c/openstack/project-config/+/872476 dropped python2.7 from nodepool images08:46
ralonsohwell, that makes sense08:46
ralonsohand we should not have py27 based jobs anymore08:46
ykarelexcept some stable branch like train08:47
ralonsohright08:47
lajoskatonaykarel: ok, I missed the exact reason for the failure09:00
slaweqralonsoh hi, just FYI, on Friday I updated Your patches https://review.opendev.org/c/openstack/neutron/+/873247/ and https://review.opendev.org/c/openstack/neutron/+/873296/ to fix UT failures09:15
ralonsohsure, let me check09:16
ralonsohslaweq, thanks a lot! On friday I was in PTO09:16
slaweqyeah, I know. That's why I did it :)09:17
ralonsohby the way, I forgot to send the mail aboyt the drivers meeting09:17
ralonsohI'll need to apologise about this during the next meeting09:17
slaweqno worries :)09:17
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add debugging exception messages in ``test_ovs_and_ovs_events``  https://review.opendev.org/c/openstack/neutron/+/87312610:37
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add debugging exception messages in ``test_ovs_and_ovs_events``  https://review.opendev.org/c/openstack/neutron/+/87312610:37
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi  https://review.opendev.org/c/openstack/neutron/+/87329810:37
amorinhello team, if dvr_snat is not supported on a compute node (https://review.opendev.org/c/openstack/neutron/+/801503), should we remove the devstack multinode tests that rely on it?11:03
amorinSome context: I am working on this patch: https://review.opendev.org/c/openstack/neutron/+/86974111:04
amorinwhich is setting the network node qr- interface down on the backup node11:04
amorinsome tests are randomly failing, depending on the instances scheduling in the devstack multinode11:04
amorinif they are scheduled on a network node with the qr- interface down, then the test is failing11:05
amorinI was expecting that this should not be tested, as it is not supported11:05
ralonsohamorin, right, actually we should check if there is any missing bit left in the code. This mode is not supported11:06
ralonsohamorin, do you mind commenting on https://bugs.launchpad.net/neutron/+bug/1934666?11:06
ralonsohand setting the status to "new", to continue fixing what is missing11:07
amorinI can, I dont understand exactly what you expect from me? Should I explain that devstack multinode is relying on network+compute collocated?11:07
ralonsohamorin, if there is any test using this mode in neutron tempest plugin, for example11:08
ralonsohwhat is exactly failing11:08
amorinI think there is no tests failing right now, but my patch is introducing one11:08
amorinthe issues described by weronika in the bug report is not tested I think11:09
ralonsohit is not because is not supported11:09
amorinI dont have rights to change the bug in new11:13
ralonsohamorin, what is the link [1]11:13
ralonsoh?11:13
amorinyup, edited :)11:14
amorinrefresh11:14
ralonsohOk, I'll set this bug to new11:14
ralonsohto address this new comment11:14
amorinack thanks11:17
amorinI think it's only a matter on how multinode is configured11:17
amorinI can maybe help on this, is it relying in the neutron source code?11:17
amorinlike in devstack/* folder?11:17
ralonsohamorin, no, this is configured in the zuul job definition, in zuul.d/tempest-multinode11:19
ralonsohname: neutron-ovs-tempest-dvr-ha-multinode-full11:19
amorinack, will check then11:22
opendevreviewRodolfo Alonso proposed openstack/neutron master: Increase port name size and type to internal  https://review.opendev.org/c/openstack/neutron/+/87311811:36
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi  https://review.opendev.org/c/openstack/neutron/+/87097911:37
opendevreviewRodolfo Alonso proposed openstack/neutron master: Increase port name size and type to internal  https://review.opendev.org/c/openstack/neutron/+/87311811:50
opendevreviewRodolfo Alonso proposed openstack/neutron master: Increase tox-py311 timeout to 3600 seconds  https://review.opendev.org/c/openstack/neutron/+/87355311:58
*** priteau_ is now known as priteau12:25
ralonsohslaweq, obondarev_ lajoskatona please check https://review.opendev.org/c/openstack/neutron/+/87312612:31
ralonsohthere is a DNM patch on top of this one. The affected test is passing in all the FT jobs12:31
ralonsoh(and the only errors should be solved in https://review.opendev.org/c/openstack/neutron/+/873118)12:31
lajoskatonaralonsoh: some of the results in functional seems related: ie.: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b87/870979/7/check/neutron-functional-with-uwsgi-2/b877d06/testr_results.html12:35
ralonsohlajoskatona, yes, this is the other patch12:35
ralonsohI fixing this problem now12:35
lajoskatonaralonsoh: ahh, ok12:35
*** dmellado_ is now known as dmellado12:58
*** obondarev_ is now known as obondarev13:14
opendevreviewMerged openstack/neutron stable/train: [Train Only] Drop networking-ovn jobs  https://review.opendev.org/c/openstack/neutron/+/87290413:23
*** dasm|off is now known as dasm13:35
*** haleyb_ is now known as haleyb13:36
amorinhey again, do you know where the role mutli-node-setup is defined?13:57
amorinhttps://github.com/openstack/neutron/blob/master/playbooks/dvr-multinode-scenario-pre-run.yaml#L313:57
ralonsohamorin, neutron-tempest-plugin/roles13:58
amorinack thanks13:58
opendevreviewRodolfo Alonso proposed openstack/ovsdbapp master: Add Interface paramteres to ``OvsdbIdl.add_port`` method.  https://review.opendev.org/c/openstack/ovsdbapp/+/87356614:28
opendevreviewRodolfo Alonso proposed openstack/neutron master: Increase port name size and type to internal  https://review.opendev.org/c/openstack/neutron/+/87311814:32
opendevreviewLuis Tomas Bolivar proposed openstack/neutron master: [OVN] Ensure traffic for provider vlan networks is not tunneled  https://review.opendev.org/c/openstack/neutron/+/87125214:37
opendevreviewDmitrii Shcherbakov proposed openstack/neutron-lib master: ext-gw-multihoming: api-def and api-ref  https://review.opendev.org/c/openstack/neutron-lib/+/87088714:51
opendevreviewLajos Katona proposed openstack/neutron master: WIP: desperate try to make DVR functional tests more stable  https://review.opendev.org/c/openstack/neutron/+/87311115:24
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [WIP] Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359315:31
opendevreviewMerged openstack/neutron-dynamic-routing master: Use SLQAlchemy ORM "relationship" instead of "relation"  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/87238415:41
opendevreviewMerged openstack/neutron stable/zed: Do not ignore attributes in bulk port create  https://review.opendev.org/c/openstack/neutron/+/87288015:42
opendevreviewMerged openstack/neutron master: Add debugging exception messages in ``test_ovs_and_ovs_events``  https://review.opendev.org/c/openstack/neutron/+/87312615:42
opendevreviewMerged openstack/neutron-tempest-plugin master: [Stateless SG] Fix usage of the default stateless SG  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87337916:26
ralonsohmlavalle, if you have some mins, please check https://review.opendev.org/c/openstack/neutron/+/873247/316:27
ralonsohthanks in advance16:27
opendevreviewRodolfo Alonso proposed openstack/neutron master: Increase port name size and type to internal  https://review.opendev.org/c/openstack/neutron/+/87311816:27
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - TESTING PATCH: neutron-functional-with-uwsgi  https://review.opendev.org/c/openstack/neutron/+/87097916:27
mlavalleralonsoh: will look at by my afternoon16:29
ralonsohmlavalle, thanks!16:29
opendevreviewRodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Add missing LSP device_owner info in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87360816:46
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Add missing LSP device_owner info in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87360916:46
opendevreviewRodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Add missing LSP device_owner info in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87361016:47
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Add missing LSP device_owner info in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87361116:47
opendevreviewRodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Bump the port revision number in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87361216:48
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Bump the port revision number in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87361316:48
opendevreviewRodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Bump the port revision number in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87361416:49
opendevreviewBrian Haley proposed openstack/neutron master: DNM: Add tox-py311 job override into job-templates.yaml  https://review.opendev.org/c/openstack/neutron/+/87361516:49
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Bump the port revision number in trunk driver  https://review.opendev.org/c/openstack/neutron/+/87361616:50
opendevreviewRodolfo Alonso proposed openstack/neutron stable/zed: Improve scheduling L3/DHCP agents, missing lower binding indexes  https://review.opendev.org/c/openstack/neutron/+/87361916:53
jamesdenton_Hi - quick question about OVN DHCP+baremetal. Made an interesting observation during PXE boot today and I could use a hand in tracking down why OVN is using a particular IP during DHCP reply, especially when the IP is not part of the allocation pool. https://paste.opendev.org/show/bBh2pnQlp84W9nDHsSWV/16:56
*** jamesdenton_ is now known as jamesdenton16:56
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: Improve scheduling L3/DHCP agents, missing lower binding indexes  https://review.opendev.org/c/openstack/neutron/+/87362216:56
ralonsohjamesdenton, you'd better ask lucas gomes (not here now)17:01
jamesdentonsure - maybe easier to send to the mailing list, you think?17:01
ralonsohjamesdenton, maybe better a launchpad bug, that will be triaged tomorrow17:02
jamesdentonok sure, i can do that. thanks17:02
opendevreviewMiguel Lavalle proposed openstack/neutron master: [WIP][DNM][OVN] Change the default firewall policy  https://review.opendev.org/c/openstack/neutron/+/83906617:03
opendevreviewRodolfo Alonso proposed openstack/neutron stable/xena: Improve scheduling L3/DHCP agents, missing lower binding indexes  https://review.opendev.org/c/openstack/neutron/+/87362617:08
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: Improve scheduling L3/DHCP agents, missing lower binding indexes  https://review.opendev.org/c/openstack/neutron/+/87362717:08
opendevreviewRodolfo Alonso proposed openstack/neutron stable/victoria: Improve scheduling L3/DHCP agents, missing lower binding indexes  https://review.opendev.org/c/openstack/neutron/+/87362817:09
opendevreviewRodolfo Alonso proposed openstack/neutron stable/ussuri: Improve scheduling L3/DHCP agents, missing lower binding indexes  https://review.opendev.org/c/openstack/neutron/+/87362917:09
opendevreviewMiguel Lavalle proposed openstack/neutron master: [WIP][DNM][OVN] Change the default firewall policy  https://review.opendev.org/c/openstack/neutron/+/83906617:13
opendevreviewLajos Katona proposed openstack/neutron master: WIP: desperate try to make DVR functional tests more stable  https://review.opendev.org/c/openstack/neutron/+/87311117:51
*** jgwentworth is now known as melwitt18:33
*** pvxe8 is now known as pvxe20:19
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [WIP] Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359320:44
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [WIP] Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359320:54
opendevreviewBrian Haley proposed openstack/neutron master: Add tox-py311 job override into job-templates.yaml  https://review.opendev.org/c/openstack/neutron/+/87361520:57
*** iurygregory_ is now known as iurygregory21:22
opendevreviewMerged openstack/neutron stable/xena: Filter subnets by "enable_dhcp" flag using the correct type  https://review.opendev.org/c/openstack/neutron/+/87306921:34
opendevreviewMerged openstack/neutron stable/wallaby: Filter subnets by "enable_dhcp" flag using the correct type  https://review.opendev.org/c/openstack/neutron/+/87309021:34
opendevreviewMerged openstack/neutron stable/yoga: Filter subnets by "enable_dhcp" flag using the correct type  https://review.opendev.org/c/openstack/neutron/+/87306821:34
*** dmellado_ is now known as dmellado22:05
*** dmellado_ is now known as dmellado22:39
*** dasm is now known as dasm|off22:54
*** dmellado_ is now known as dmellado23:02
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [WIP] Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359323:35

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