Friday, 2022-12-16

opendevreviewliuyulong proposed openstack/neutron master: Refactor for ovs qos driver meter limit features  https://review.opendev.org/c/openstack/neutron/+/86076600:55
opendevreviewliuyulong proposed openstack/neutron master: Add meter bandwidth limit support  https://review.opendev.org/c/openstack/neutron/+/86076700:55
opendevreviewliuyulong proposed openstack/neutron master: Add host metadata haproxy manager  https://review.opendev.org/c/openstack/neutron/+/86464901:21
opendevreviewliuyulong proposed openstack/neutron master: Pass physical bridge informations to OVS agent extension API  https://review.opendev.org/c/openstack/neutron/+/86663501:21
*** haleyb_ is now known as haleyb01:58
*** haleyb is now known as haleyb_away01:59
opendevreviewOpenStack Proposal Bot proposed openstack/networking-odl master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/networking-odl/+/86371903:52
opendevreviewyatin proposed openstack/neutron-tempest-plugin master: [DNM] Test stability of workaround  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86765006:13
opendevreviewyatin proposed openstack/neutron-tempest-plugin master: [DNM] Test lower concurrency on non nested provider  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86793406:38
*** ralonsoh__ is now known as ralonsoh07:35
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM Testing deploy_rootwrap  https://review.opendev.org/c/openstack/neutron/+/86794008:38
opendevreviewSlawek Kaplonski proposed openstack/neutron master: DNM Just testing drop of lib/neutron from Devstack  https://review.opendev.org/c/openstack/neutron/+/86582209:04
lajoskatonayatin: Hi, the nested nodes vs  tempest timeouts issue (i.e.: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867320? ) worth to recheck?09:05
lajoskatonayatin: am I understand well that the issue most probably happens on vexxhost nodes?09:05
lajoskatonaykarel, sorry, stil morning it seems... -----^09:06
ykarellajoskatona, yes it's happening only on vexxhost provider. i am testing only master jobs in other patch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86765009:08
ykarelwell there too it's not working 100% of time even dropping concurrency09:09
sahidlajoskatona: Morning, thanks a lot for your review :-)09:19
opendevreviewyatin proposed openstack/neutron-tempest-plugin master: [DNM] Test stability of workaround  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86765009:22
opendevreviewRodolfo Alonso proposed openstack/neutron master: Remove FIPAddDeleteEvent event  https://review.opendev.org/c/openstack/neutron/+/86400009:24
sahido/ quick question regqrding rbac, I'm not able to see where I cqn increase quota for them09:24
sahidreceiving this error: /networking/v2.0/rbac-policies, Quota exceeded for resources: ['rbac_policy'].09:25
opendevreviewArnaud Morin proposed openstack/neutron master: Allow restoration of tun_ofports on agent restart  https://review.opendev.org/c/openstack/neutron/+/86027009:41
ralonsohsahid, https://docs.openstack.org/nova/rocky/admin/quotas.html09:55
amorinhey haleyb_away, ralonsoh I answered to the fullstack testing on https://review.opendev.org/c/openstack/neutron/+/860270/comments/d1f55c90_d984e9fe09:55
ralonsohok09:56
opendevreviewRodolfo Alonso proposed openstack/neutron master: Enable tox4 testing  https://review.opendev.org/c/openstack/neutron/+/86755410:01
opendevreviewRodolfo Alonso proposed openstack/neutron master: Testing https://review.opendev.org/c/openstack/neutron/+/867554  https://review.opendev.org/c/openstack/neutron/+/86761610:05
opendevreviewliuxie proposed openstack/neutron master: [OVN] Update fip while fixed_ip of internal port has changed  https://review.opendev.org/c/openstack/neutron/+/86796410:14
sahidralonsoh: hum i should hqve check better that doc10:16
amorinthanks ralonsoh, I will apply the fixes you requested10:24
opendevreviewRodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Allow only one physical network per bridge  https://review.opendev.org/c/openstack/neutron/+/86325610:27
opendevreviewyatin proposed openstack/neutron-tempest-plugin master: [DNM] Test lower concurrency on non nested provider  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86793410:32
*** kopecmartin is now known as kopecmartin|sick10:45
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Add vnic_type and binding profile capabilities to LSP info  https://review.opendev.org/c/openstack/neutron/+/86735910:52
ralonsohslaweq, lajoskatona obondarev hi folks, please check https://review.opendev.org/c/openstack/neutron/+/86755411:05
ralonsohbefore tox4 is enabled11:05
ralonsohupsss sorry, I need to make this patch dependant on the tox change...11:06
opendevreviewRodolfo Alonso proposed openstack/neutron master: Enable tox4 testing  https://review.opendev.org/c/openstack/neutron/+/86755411:07
opendevreviewRodolfo Alonso proposed openstack/neutron master: Testing https://review.opendev.org/c/openstack/neutron/+/867554  https://review.opendev.org/c/openstack/neutron/+/86761611:08
slaweqralonsoh it seems that zuul isn't happy with this patch11:35
ralonsohslaweq, what one?11:35
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/86755411:35
slaweqyes11:35
ralonsohhttps://zuul.opendev.org/t/openstack/status#86755411:36
ralonsohit is passing11:36
slaweqI see that functional and fullstack jobs are red there11:36
slaweqahh, ok11:36
ralonsohhehehehe11:36
slaweqI just checked in gerrit11:36
slaweqso it's old results there11:36
ralonsohyeah, I've tested several times11:36
ralonsohI then saw the problem with rootwrap11:36
ralonsohand so far, checking the live output of functional11:37
ralonsohseems to be working! (with tox4)11:37
slaweqralonsoh++11:38
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP] [OVN] OVN monitor agent  https://review.opendev.org/c/openstack/neutron/+/86648012:00
sahidI think we are good for this one, https://review.opendev.org/c/openstack/neutron/+/865823 the CI issue has been fixed, I think it's by slaweq, thanks again13:01
opendevreviewLajos Katona proposed openstack/neutron stable/zed: Fix bulk create without mac  https://review.opendev.org/c/openstack/neutron/+/86792013:21
*** dasm|off is now known as dasm13:36
ralonsohslaweq, qq, where in the fullstack job are the test logs_13:46
ralonsoh?13:46
ralonsohhttps://zuul.opendev.org/t/openstack/build/759a4e82d6824c0da585b3a88551dfdd/logs13:47
ralonsohshouldn't we have a gzip file with this info? same as in functional 13:47
slaweqralonsoh probably yes13:47
slaweqbut it wasn't that critical as there's not as many files in this job as in functional13:48
slaweqso it's not that slow to upload all of them to swift13:48
ralonsohslaweq, no no, what I mean is that in other jobs we have them13:49
ralonsohhttps://4e9470b351b17cbb3518-980dc61496a4ee6111bf6d34d70d10e6.ssl.cf5.rackcdn.com/867359/5/check/neutron-fullstack-with-uwsgi/cb1c5ba/controller/logs/index.html13:49
ralonsohdsvm-fullstack-logs.tar.gz13:49
ralonsohbut not with tox413:49
slaweqhmm13:50
slaweqok, this is strange13:50
slaweq2022-12-16 13:21:17.459519 | TASK [prepare_functional_tests_logs : Check if /opt/stack/logs/dsvm-fullstack-logs exists]13:51
slaweq2022-12-16 13:21:18.108830 | controller | ok13:51
slaweq2022-12-16 13:21:18.152987 | TASK [prepare_functional_tests_logs : Prepare logs archive /opt/stack/logs/dsvm-fullstack-logs.tar.gz]13:51
slaweq2022-12-16 13:21:18.227428 | controller | skipping: Conditional result was False13:51
slaweqso there is no logs in that directory13:51
slaweqand because of that there is no tar.gz archive created13:51
ralonsohslaweq, I think I'm going to limit tox to 3 in out CI for now13:53
ralonsohuntil we fix what is happening there13:53
slaweq++13:53
ralonsoh#startmeeting neutron_drivers14:00
opendevmeetMeeting started Fri Dec 16 14:00:17 2022 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'neutron_drivers'14:00
mlavalleo/14:00
ralonsohhello all14:00
lajoskatonao/14:01
slaweqo/14:01
obondarevo/14:02
ralonsohBrian cannot attend today14:02
ralonsohSo we are 5, I think we have quorum14:02
mlavalleyeap14:02
ralonsohwe have two topics today14:02
ralonsohfirst one14:02
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/199860914:02
ralonsohracosta, please, you are welcome to present it14:02
racostaok, thk. This RFE intends to implement distributed routing support for IPv6 only or dual-stack usage scenarios.14:03
racostaThe proposal is to introduce a new NAT rule for the IPv6 GUA addresses that are allocated to VMs.14:04
lajoskatonais it only for OVN?14:04
racostaThe ovn-controller running on the chassis needs this rule to start responding Neighbor Advertisements for IPv614:04
racostaYes, only for OVN driver14:05
lajoskatonathanks14:05
ralonsohwhy core OVN is not implementing this?14:06
* mlavalle likes the pretty drawings14:06
racostaOn the ovn side, the nat rule is the same for ipv4 or ipv614:06
racostaIt would be more of a CMS task to program how ovn handles flows on chassi14:07
ralonsohyeah but not the SB database14:08
ralonsohwe should be modifying only the NB registers14:08
mlavalleyeap14:08
racostano no, just in the NB database. 14:08
racostaThe NAT rule is associated with the router at the northbound database level. 14:09
ralonsohok, is this doing any IPv6 nating?14:10
racostathere is no NAT for IPv6, so this special rule that translates the GUA address to itself is only used to create the flows in the chassis where the VM resides14:10
racostaWithout this rule, the compute node does not know how to respond to that IPv6 GUA and centralize the communication through the router's GW port.14:11
ralonsohok, is it compatible with IPv4 FIP DVR?14:12
racostayes, absolutely. Does not change anything for IPv4 FIP DVR.14:12
ralonsohone question (for drivers): should this be configurable?14:13
slaweqIIUC from the LP description, it requires NAT like IPv6 GUA to the same IPv6 GUA address, will it require any changes in Neutron API, or do You want to make such nat entry for each IPv6 address automatically?14:13
slaweqor enabled/disabled by config option for all ports/IPs14:14
slaweq?14:14
ralonsohright, similar question14:14
racostaI believe it must be configured, because the user may be using n-d-r, for example.14:14
mlavalleso a config knob?14:15
lajoskatonabut no API change or extension for it?14:15
ralonsohthis is backend specific, actually apart from n-d-r, other traffics should work the same in both scenarios14:16
racostaThe proposal is to enable by enable_distributed_ipv6 flag in the ml2_conf.ini file.14:16
ralonsoh(that's what I think)14:16
racostaYes, other backends may need the current centralized behavior.14:17
slaweqok, thx for explanation14:17
*** njohnston_ is now known as njohnston14:18
mlavalleracosta: I assume this comes from a well known use case to you. How much of a gain / benefit does this improvement represent?14:18
mlavallehave you tested it?14:18
racostaYes, I already tested it in a deployment with openstack yoga.14:19
mlavalleand what was gained out of it?14:20
racostaThe benefits of implementing DVR for IPv6 are the same as for IPv4 FIP DVR, it is a more generalized case for dual stack.14:20
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Fix listener provisioning_status after HM created/deleted  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/86797414:20
racostaPerformance gain by distributing north/south traffic per chassis, and configuration gain by reducing dynamic routing complexity in the fabric.14:21
mlavalleno side effects so far? for how long?14:22
racostaNo effect for now, I'm testing dataplane performance with automated shaker tool.14:24
lajoskatonasounds great14:25
obondarevno questions from me, looks reasonable and rather safe14:25
ralonsohdo you think we can vote now?14:25
mlavalleI'm ok with this. 14:25
lajoskatonaWould be good to have some tests finally for it in upstream CI also14:25
lajoskatona+1 from me14:26
ralonsoh+114:26
obondarev+114:26
mlavalleracosta: if you can, please share your shaker results with us14:26
mlavalle+114:26
* mlavalle formalizes his vote14:26
slaweq+114:26
ralonsohthanks! approved then14:26
ralonsohabout the spec, do you think we need a spec for this RFE?14:26
mlavalleno, it's pretty localized14:27
ralonsohI think it should be documented, that's all14:27
mlavalleyeap14:27
obondarevI think there is one already https://launchpadlibrarian.net/639474950/rfe_ovn_ipv6_dvr.rst14:27
mlavalleand hopefully include the testing results in that document14:27
ralonsohracosta, why don't you present it?14:28
ralonsohobondarev, who's this spec?14:28
obondarevit's linked in the bug14:28
obondarevI believe it's from racosta14:28
mlavalleyeap14:28
obondarevin the RFE*14:28
ralonsohahh I didn't see c#314:29
ralonsohracosta, ok, please, propose this spec formally (now you have it written)14:29
ralonsohI'll write in the bug how to do it14:29
mlavalleand if you could add some testing resuls to it, it would be great14:30
mlavalleracosta: thanks for this proposal. Nice!14:30
racostaOk, thank you very much. I will add the test results ;)14:31
ralonsohok, thanks for your proposal. I'll update the bug14:31
ralonsohthe next topic is 14:31
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/199860814:31
ralonsohquick summary14:31
mlavalleracosta: to be clear, it is not a precondition. I just think it would be nice to share it with the community (and I'm curious)14:31
ralonsohwe are "playing" with hardware offload cards with ML2/OVN. The problem we found is that QoS is still not working14:32
ralonsohHWOL drivers do not translate the OVS QoS rules to the interface14:32
ralonsohthis is where I'm trying to create an OVN monitor14:33
ralonsohthat will run on the compute node14:33
ralonsohthis monitor will be generic (new features could be added)14:33
ralonsohby default, there will be NO need to spawn it14:33
slaweqso You want to have yet another neutron-ovn-agent on each compute node14:33
slaweqto do things like that on nodes, right?14:34
ralonsohyes14:34
ralonsohneutron-ovn-monitor-agent14:34
obondarevand new RPC interface, correct?14:34
ralonsohno14:34
ralonsohno RPC, this is something we won't do14:34
slaweqok, dummy question - what about neutron-ovn-metadata-agent then? Can it be combined with this new "generic" agent maybe?14:34
ralonsohobondarev, anything we need should be stored in the OVN/OVS database14:34
ralonsohsame as metadata14:34
mlavallereacting to ovsdb events14:35
obondarevok, so the new agent will talk only to local ovsdb?14:35
ralonsohslaweq, no, metadata is specific for metadata only. Mixing features in this agent (that is also mandatory) is not a good idea14:35
ralonsohobondarev, not local ovsdb only, also remote OVN database14:35
ralonsohsame as metadata agent14:35
obondarevah, got it14:35
slaweqI have one small concern about scalling of this14:36
ralonsohyes, I have this concern too14:36
slaweqwe know that many connections to the ovn dbs can cause issues14:36
mlavalleI think it's a big concern14:36
slaweqand this will be at least yet another connection from each compute node14:36
ralonsohyes, for sure. This is why this agent won't be necessary in all deployments14:36
ralonsohthis monitor is necessary only, for now, for this specific feature14:37
ralonsohQoS with HWOL14:37
lajoskatonabut this is only necessary on hots where hw offload things are located, no?14:37
ralonsohyes14:37
ralonsohand if you want qos14:37
slaweqfor now, but who knows what else we will implement there :p14:37
ralonsohyes, of course14:37
lajoskatonagood comment14:37
ralonsohPOC: (3 patches) https://review.opendev.org/c/openstack/neutron/+/86648014:38
slaweqI'm not against but I just wanted to raise concern which I have :)14:38
mlavalleas long as the trade offs are well documented so deployers can understand the potential cost, I think it would be ok14:38
obondarev+114:38
ralonsohyes but overloading a necessary agent (metadata) that also has a specific task, is not a good idea14:38
ralonsohand, btw, this is a kind of workaround until driver manufacturers fix the drivers14:39
slaweq+1 from me but with proper documentation as mlavalle mentioned :)14:40
mlavalleso: 1) it is optional 2) the performance trade offs are well documented. with this I'm +114:40
ralonsohof course, it will be documented14:40
ralonsohany other question?14:41
lajoskatonaagree with the above limitations14:42
mlavallenone from me14:42
slaweqI'm good14:43
ralonsohso just to make it explicit, can you vote?14:43
mlavalle+114:43
slaweq+114:43
lajoskatona+114:43
obondarev+114:43
ralonsoh(nothing from me, I proposed it)14:44
ralonsohso approved14:44
ralonsohdo I need a spec?14:44
lajoskatonagood question14:45
obondarevNew agent seems a pretty big change, I believe some sort of design doc would be useful14:45
ralonsohI agree14:45
lajoskatonaas it introduces a new agent like thing would be good to write it 14:45
ralonsohI'll push a spec next week14:45
lajoskatonathanks14:45
slaweqthx14:45
mlavalleI'd say given the potential performance concerns at scale, let's give ourseleves and the community some time to think about it and the spec process gives us that14:46
ralonsohthank you folks! I have nothing else in the agenda14:46
ralonsohso have a nice weekend!14:46
slaweqthx, You too14:46
lajoskatonaBye14:46
ralonsoh#endmeeting14:46
opendevmeetMeeting ended Fri Dec 16 14:46:32 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:46
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-12-16-14.00.html14:46
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-12-16-14.00.txt14:46
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-12-16-14.00.log.html14:46
slaweqo/14:46
mlavalleo/14:46
obondarevo/14:46
opendevreviewArnaud Morin proposed openstack/neutron master: Allow restoration of tun_ofports on agent restart  https://review.opendev.org/c/openstack/neutron/+/86027014:57
opendevreviewRodolfo Alonso proposed openstack/neutron master: Limit tox version to <4  https://review.opendev.org/c/openstack/neutron/+/86797714:58
amorinralonsoh ^ last change with all your requests done :)15:00
ralonsohyeah, Im reviewing the patch now15:00
ralonsohlooks fine, waiting for the CI15:00
* amorin crossing fingers15:02
amorinmy bad, I forgot to push constants.py15:04
opendevreviewArnaud Morin proposed openstack/neutron master: Allow restoration of tun_ofports on agent restart  https://review.opendev.org/c/openstack/neutron/+/86027015:05
opendevreviewRodolfo Alonso proposed openstack/neutron master: Limit tox version to <4  https://review.opendev.org/c/openstack/neutron/+/86797715:12
opendevreviewLajos Katona proposed openstack/neutron stable/zed: Fix bulk create without mac  https://review.opendev.org/c/openstack/neutron/+/86792015:21
opendevreviewMerged openstack/networking-odl master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/networking-odl/+/86371915:22
opendevreviewLajos Katona proposed openstack/neutron stable/zed: Fix bulk create without mac  https://review.opendev.org/c/openstack/neutron/+/86792015:23
opendevreviewRoberto Acosta proposed openstack/neutron-specs master: Add spec for the ovn IPv6 DVR RFE  https://review.opendev.org/c/openstack/neutron-specs/+/86797915:25
opendevreviewLajos Katona proposed openstack/neutron stable/yoga: Fix bulk create without mac  https://review.opendev.org/c/openstack/neutron/+/86792115:30
mlavalleslaweq, ralonsoh, lajoskatona: yesterday https://review.opendev.org/c/openstack/devstack/+/866944 merged. So as of today https://zuul.opendev.org/t/openstack/builds?job_name=neutron-ovn-tempest-mariadb-full&branch=master&skip=015:35
mlavallewe can remove that action item from the CI meeting15:36
ralonsohok15:36
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Fix listener provisioning_status after HM created/deleted  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/86797415:40
opendevreviewyatin proposed openstack/neutron-tempest-plugin master: [DNM] Test lower concurrency on non nested provider  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86793415:46
lajoskatonamlavalle: thanks15:47
opendevreviewyatin proposed openstack/neutron-tempest-plugin master: Revert "Update nested-virt testing for the 2023.1 cycle"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86792215:59
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Pin OVS_BRANCH to work with OVN_BRANCH main  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/86798316:32
opendevreviewRodolfo Alonso proposed openstack/neutron master: Limit tox version to <4  https://review.opendev.org/c/openstack/neutron/+/86797716:55
opendevreviewMerged openstack/neutron master: dhcp: fix issue when network is already removed  https://review.opendev.org/c/openstack/neutron/+/86582317:02
*** dasm is now known as dasm|off21:54
*** osmanlicilegi is now known as Guest022:33
*** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack"22:40

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