Monday, 2022-07-11

opendevreviewTyler Stachecki proposed openstack/neutron-lib master: model_query: Improve perf with large # of RBAC  https://review.opendev.org/c/openstack/neutron-lib/+/81745902:23
opendevreviewMiro Tomaska proposed openstack/neutron master: Add workaround for eventlet bug[1] in get_hypervisor_hostname  https://review.opendev.org/c/openstack/neutron/+/84912203:09
opendevreviewTakashi Kajinami proposed openstack/neutron-vpnaas-dashboard master: Replace deprecated ugettext_lazy and ungettext_lazy  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84925704:33
opendevreviewTakashi Kajinami proposed openstack/neutron-vpnaas-dashboard master: Replace deprecated django.conf.urls.url()  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84925804:34
opendevreviewTakashi Kajinami proposed openstack/neutron-fwaas-dashboard master: Replace deprecated ugettext_lazy and ungettext_lazy  https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/84925904:35
opendevreviewTakashi Kajinami proposed openstack/neutron-vpnaas-dashboard master: Replace deprecated ugettext_lazy and ungettext_lazy  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84925704:51
opendevreviewTakashi Kajinami proposed openstack/neutron-vpnaas-dashboard master: Replace deprecated django.conf.urls.url()  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84925804:51
opendevreviewTakashi Kajinami proposed openstack/neutron-vpnaas-dashboard master: Update python testing as per zed cycle testing runtime  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84926605:06
opendevreviewTakashi Kajinami proposed openstack/neutron-fwaas-dashboard master: Replace deprecated ugettext_lazy and ungettext_lazy  https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/84925905:42
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: make vlanmanager to handle more vlan mapping per network  https://review.opendev.org/c/openstack/neutron/+/83947907:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: use a local vlan per network/segmentation  https://review.opendev.org/c/openstack/neutron/+/84041507:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp/rpc: retrieve network details with segments  https://review.opendev.org/c/openstack/neutron/+/84041607:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: utils: add option to generate dhcp device id per segmentation  https://review.opendev.org/c/openstack/neutron/+/84041707:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/mech: bind port on segment that subnet belongs  https://review.opendev.org/c/openstack/neutron/+/84041807:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: make device manager to clean only not used devices  https://review.opendev.org/c/openstack/neutron/+/84041907:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: add/use cleanup stale devices API  https://review.opendev.org/c/openstack/neutron/+/84042007:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: support multiple segmentations per network  https://review.opendev.org/c/openstack/neutron/+/84042107:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint  https://review.opendev.org/c/openstack/neutron/+/84145907:30
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host  https://review.opendev.org/c/openstack/neutron/+/84183807:30
lajoskatonaobondarev: Hi, pushed a rally-openstack patch to limit docutils for it to <0.19, can you check it it please when you have few mintues?08:09
lajoskatonaobondarev: the link, sorry : https://review.opendev.org/c/openstack/rally-openstack/+/84928108:10
obondarevlajoskatona: sure, will do08:11
lajoskatonaobondarev: I think the final solution can be done when in docutils the parser will be ready, but perhaps I can send them a mail to ask about how they plan, because now OptionParser is deprecated but no replacement for it, just a hint08:12
obondarevlajoskatona: strange that tests fail due to deprecation, let me check08:15
lajoskatonaobondarev: that can be good direction also, I havent found it :-)08:15
opendevreviewTyler Stachecki proposed openstack/neutron master: external_net_db: Adhere to model_query changes  https://review.opendev.org/c/openstack/neutron/+/81746209:55
sahidHello guys, do we know whehter we have issue with CI running DVR jobs?11:57
sahidhttps://zuul.opendev.org/t/openstack/build/0adaf40a86574a75a8d3a7a6a4c098d9 and https://zuul.opendev.org/t/openstack/build/94705b6010874e07bd782bf4a4bea46611:58
opendevreviewMerged openstack/neutron-dynamic-routing master: [sqlalchemy-20] Add missing DB context decorator  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/84864512:20
lajoskatonasahid: Hi, thanks, I saw some failures this morning, I check if it is general or just occasional12:21
lajoskatonasahid: the job itself looks ok: https://zuul.openstack.org/builds?job_name=neutron-ovs-tempest-dvr-ha-multinode-full in general12:25
sahidlajoskatona: Hello, thanks to have took time looking at it, does that ring a bell https://zuul.opendev.org/t/openstack/build/0adaf40a86574a75a8d3a7a6a4c098d9/log/compute1/logs/screen-q-agt.txt?severity=4 ?12:26
sahidyes... looks like all failing are related to me 12:27
lajoskatonasahid: yes :-(12:30
lajoskatonasahid: the log you pointed previously is due to that q-svc is restarted at that time (grenade most probably doing upgrade at that time of the services): https://428b85e7030640e63a25-64321574d333a1d78009697cef7af9d1.ssl.cf2.rackcdn.com/839479/14/check/neutron-ovs-grenade-dvr-multinode/0adaf40/controller/logs/screen-q-svc.txt12:31
lajoskatonasahid: 10:27:15.550058 - 10:31:18.348196 (the timestamps for the restart of the service based on q-svc log)12:32
lajoskatonasahid: by-the-way: You are the "owner" of the multiple routed provider net feature?12:43
lajoskatonasahid: shall I ask if you can clean the specs? https://review.opendev.org/c/openstack/neutron-specs/+/823823 & https://review.opendev.org/c/openstack/neutron-specs/+/824464 & https://review.opendev.org/c/openstack/neutron-specs/+/824465 ?12:43
lajoskatonasahid: if you need help just ping me (i.e.: to abandon some of these three if you have all the content in one of them....)12:44
*** damiandabrowski[m] is now known as damiandabrowski12:45
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: make vlanmanager to handle more vlan mapping per network  https://review.opendev.org/c/openstack/neutron/+/83947912:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: use a local vlan per network/segmentation  https://review.opendev.org/c/openstack/neutron/+/84041512:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp/rpc: retrieve network details with segments  https://review.opendev.org/c/openstack/neutron/+/84041612:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: utils: add option to generate dhcp device id per segmentation  https://review.opendev.org/c/openstack/neutron/+/84041712:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/mech: bind port on segment that subnet belongs  https://review.opendev.org/c/openstack/neutron/+/84041812:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: make device manager to clean only not used devices  https://review.opendev.org/c/openstack/neutron/+/84041912:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: add/use cleanup stale devices API  https://review.opendev.org/c/openstack/neutron/+/84042012:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: support multiple segmentations per network  https://review.opendev.org/c/openstack/neutron/+/84042112:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint  https://review.opendev.org/c/openstack/neutron/+/84145912:48
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host  https://review.opendev.org/c/openstack/neutron/+/84183812:48
sahidlajoskatona: could you abandon: https://review.opendev.org/c/openstack/neutron-specs/+/824464 and https://review.opendev.org/c/openstack/neutron-specs/+/824465 ?12:50
sahidyes I'm not able to clean them, i'm the new owner12:51
sahidthanks a lot12:51
lajoskatonasahid: done, thanks for checking12:52
sahidI may have find the CI issue, I have https://review.opendev.org/c/openstack/neutron/+/839479/14/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#1953, I removed the "else"12:52
lajoskatonasahid: I will check it13:01
dulekslaweq: Hi! We're working on an OpenShift feature and came across this problem. So technically number of IPs per port is unlimited in Neutron, but I bet there's some practical limit. At what value would you put a safe maximum here?13:35
dulekWe're asking because e.g. in AWS that number is limited to 16.13:35
*** dasm|off is now known as dasm13:51
sahids/CI issue/change in my patch that makes CI failing13:59
lajoskatonaskoech[m]: Hi, regarding https://review.opendev.org/q/topic:cascade_delete_extension15:10
lajoskatonaskoech[m]: thanks for uploading, I think you have to rebase to master as both patches have merge conflict15:10
sahido/ dulek have you considered looking at how we map ips per port in database?15:28
duleksahid: I just assumed it's a regular one-to-many DB relationship? 15:32
duleksahid: It seems so, right: https://github.com/openstack/neutron/blob/182044d628314290a38440bab203d27fb9db755d/neutron/db/models_v2.py#L71-L85 ?15:37
sahidyes looks like it is https://github.com/openstack/neutron/blob/182044d628314290a38440bab203d27fb9db755d/neutron/db/models_v2.py#L11015:49
sahiddoes not look to be right to do not have limitation...15:49
sahidhave you check whehter we don't have quota for that specific case?15:49
sahidlajoskatona: do you have pointers for me to build locally an env to execute neutron-ovs-grenade-dvr-multinode ?15:54
lajoskatonasahid: grenade is an interesting question, I never run grenade locally, but I would start here: https://github.com/openstack/grenade/blob/master/README.rst16:10
lajoskatonasahid: and the job definition is here: https://opendev.org/openstack/neutron/src/branch/master/zuul.d/grenade.yaml#L77 , not sure if you can reproduce it locally based on that16:11
lajoskatonadulek: Hi, slaweq, is on PTO, and to tell the truth I don't think we have specific hardcoded or configured limit for IPs for port16:12
lajoskatonadulek: I would expect 2 limiting factors: * performance after some and * backend limitation perhaps16:12
lajoskatonadulek: but now we have no such test to see if we have such (at least I don't know about it)16:13
lajoskatonadulek: tomorrow we have team meeting at 1400UTC, would be good to discuss your usecase if you think16:14
sahidlajoskatona: can you give me access to one of the job failing<16:14
sahid?16:14
duleklajoskatona: Thanks for help! I've discussed this a bit with ralonsoh and apparently it was tested up to 200 adresses. We'll probably just cap it at 64, which should be more than enough for our use case, while sounds like a safe limit for Neutron.16:15
sahiddulek: there is no quotas for this so? or you havnt checked?16:33
duleksahid: Sorry, I don't think I understand. There doesn't seem to be a DB-induced limit as IPs are a separate table with as relationship to a port.16:36
lajoskatonasahid: I can't give you acces to these nodes, for that some infra people can help, try on #openstack-infra16:39
lajoskatonadulek: ok, good to know16:40
sahiddulek: https://docs.openstack.org/neutron/latest/admin/ops-quotas.html17:08
sahidlajoskatona: ack I think I have found the problem17:09
lajoskatonasahid: +117:09
opendevreviewSharon Koech proposed openstack/neutron master: Network Cascade Deletion Extension  https://review.opendev.org/c/openstack/neutron/+/84903917:28
skoech[m]lajoskatona: Yeah, I forgot to rebase. On  it now, hope the merge conflicts are solved.17:31
opendevreviewSharon Koech proposed openstack/neutron-lib master: Network Cascade Deletion Extension  https://review.opendev.org/c/openstack/neutron-lib/+/84904617:34
opendevreviewBrian Haley proposed openstack/neutron master: Remove import of 'imp' module  https://review.opendev.org/c/openstack/neutron/+/84245018:21
turgleHi everyone, i have an issue with ovn. One of my compute node lost all vlan network18:34
turglei Check no port patch-provnet-*-to-br-int create in br-ex18:34
turgleI deploy openstack victoria at 2021 and ovn 20.0318:35
turgleanyone have idea about this isssue?18:35
turgleovs-ofctl dump-flows br-int table=0 |grep vlan don't show anything18:44
turgleanother compute run fine have flow about vlan18:45
lajoskatonaskoech[m]: +1, good luck with it :-)19:15
skoech[m]Thanks, I need it. :D19:17
skoech[m]s/Thanks/lajoskatona: thanks/19:18
lajoskatonaskoech[m]:  I still see mergeconflicts, if you have problems with it tomorrow morning we can try to push it in a video call19:19
skoech[m]lajoskatona: yeah, that sounds good. What time works for you?19:21
*** dasm is now known as dasm|off22:18

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