Monday, 2022-11-07

opendevreviewrenliang proposed openstack/neutron stable/yoga: Update the Ethernet card information  https://review.opendev.org/c/openstack/neutron/+/86379001:38
opendevreviewrenliang proposed openstack/neutron stable/xena: Update the Ethernet card information  https://review.opendev.org/c/openstack/neutron/+/86379101:38
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint  https://review.opendev.org/c/openstack/neutron/+/84145908:01
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host  https://review.opendev.org/c/openstack/neutron/+/84183808:01
QGHi08:52
QGI was wondering how it works when creating a router and adding or removing an external gateway.08:53
QGWhen I set neutron-api in debug and try to add or remove an external gateway, neutron will always choose new agents and try to bind the router to them. 08:53
QGOf course it will realize afterwards that the router already has agents and that it should not do it but it takes a lot of time to choose the agents and try to bind the router on them.08:53
QGDo you know why the schedule agent part is redone when adding or removing the external gateway ? 08:53
isabekHi team, can we send precreated ovs bridge with port vif details to nova to connect tap device to this bridge? Now by default tap device connecting to br-int08:54
sahidlajoskatona: morning... sorry i'm going to annoy you again, have you had chance to look at how to build fullstack on a local VM i/'m still struggling with this 09:07
lajoskatonasahid: yes, I worked on it Friday, but no success, so I have to check what I had to adjust :-(09:09
sahidoh ok so at least i'm not the only one09:10
sahidi's already a good positive for me :-)09:10
sahid will make an other try09:13
opendevreviewFernando Royo proposed openstack/neutron stable/yoga: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86327509:25
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables  https://review.opendev.org/c/openstack/neutron/+/75987409:43
ralonsohQG, do you have HA configured?09:47
QGralonsoh: Yes i have09:47
ralonsohthen check l3_hamode_db09:47
ralonsohwhen the router is updated, the scheduling is called09:47
ralonsohI would need to check that09:48
ralonsohbut this should be related to "get_l3_agents_hosting_routers"09:48
ralonsohwhen you remove the GW port, you are removing one interface09:48
ralonsohthat will remove some bindings09:49
QGi was thinking may be it's when the notify is called :09:49
QGhttps://github.com/openstack/neutron/blob/master/neutron/db/l3_db.py#L2142-L214309:49
ralonsohbut I would need to check that09:49
QGwe don't specify the param  `schedule_routers=False` to the routers_updated function09:49
ralonsohthis is the call from the server to the agents09:49
ralonsohwhat does it means you don't specify this paremeter?09:51
QGhttps://github.com/openstack/neutron/blob/master/neutron/api/rpc/agentnotifiers/l3_rpc_agent_api.py#L8609:52
QGIn the _notification function the param schedule_routers is set to true so when we notify the agent a schedule is trigger09:52
ralonsohyes09:53
ralonsohand i don't see in the code any place where that is set to false, to be honest09:53
ralonsohisabek, can you reformulate this question? what do you mean? that you need a different way to connect a tap interface?09:53
QGno I don't either, so with each action he has to redo the schedule09:54
QGDo we need to do a schedule when we add or remove an external gateway ?09:54
ralonsohQG so far yes, as commented, when you remove the GW interface, you are removing one interface of the router09:54
ralonsohif I'm not wrong, I remember other related improvement related to this... but my memory is short09:55
ralonsohlet me find this09:55
QGso when we remove one interface of the router we want to choose new agent and do no configure them09:55
QGcool thanks !09:55
sahidralonsoh: o/ just in case that make you think about something, when tring to execute tox -e dvsm-fullstack I have this error:09:58
sahid AssertionError: backend 'mysql' unavailable09:58
sahidlooks stange it's probably something tiny that is missing09:59
sahidi think ci already have everysting related to database installed, so it's perhpas a package missing with configure_for_func_testing.sh10:00
ralonsohsahid, in the local.conf of devstack10:02
ralonsohtry specifying both db types10:02
ralonsohDATABASE_TYPE: postgresql10:02
ralonsohDATABASE_TYPE: mysql10:02
ralonsohtry with both10:02
ralonsoh(not at the same time)10:02
ralonsoh1) add this value, 2) re-execute the script configure_for_func_testing10:02
ralonsoh3) execute tox fullstack10:03
sahidack, let me try10:03
opendevreviewFernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86385210:08
ralonsohQG no, there is nothing related to this10:10
ralonsohif you think that this could be an improvement, you can open a launchpad bug and push a path for this10:10
QGralonsoh:  ok thank i will do !10:10
sahidno luck...10:11
sahidis there a place where I can see the package installed in a CI mahcine?10:11
ralonsoh$pip freeze10:12
ralonsohfor python packages10:12
sahidi'm talking about an upstream ci machine10:12
ralonsohin the node logs10:12
ralonsohfor example10:13
sahidgood point10:13
ralonsohhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_9e2/759874/12/check/neutron-linuxbridge-tempest-plugin-nftables/9e2da6d/controller/logs/pip3-freeze.txt10:13
sahidthank you for the link10:13
sahidi have diff, for example mysqlclient is not instqlled on my machine10:14
sahidwait... I was looking into my local machine :-)10:14
opendevreviewFernando Royo proposed openstack/neutron stable/victoria: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86386110:15
opendevreviewFernando Royo proposed openstack/neutron stable/train: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86386210:19
opendevreviewFernando Royo proposed openstack/neutron stable/ussuri: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86386310:20
opendevreviewRodolfo Alonso proposed openstack/neutron-specs master: Strict minimum bandwidth support for tunnelled networks  https://review.opendev.org/c/openstack/neutron-specs/+/86085911:00
opendevreviewFernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86385212:10
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: test: fix docstring validation function  https://review.opendev.org/c/openstack/neutron/+/86387012:21
sahidlajoskatona, ralonsoh ^ an interesting one :-)12:21
sahidnoticed that here: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_751/841459/51/check/neutron-functional-with-uwsgi/7511c92/testr_results.html12:21
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint  https://review.opendev.org/c/openstack/neutron/+/84145912:27
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host  https://review.opendev.org/c/openstack/neutron/+/84183812:27
opendevreviewMerged openstack/neutron master: Fix some pylint indentation warnings  https://review.opendev.org/c/openstack/neutron/+/86367412:28
lajoskatonasahid: thanks12:28
crohmanncoreycb: Could I bother you about an SRU regarding new packages for neutron-vpnaas no Xena, Yoga and maybe even Zed? https://bugs.launchpad.net/cloud-archive/+bug/1995861 12:29
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables  https://review.opendev.org/c/openstack/neutron/+/75987412:41
opendevreviewMerged openstack/neutron master: Fix some pylint indentation warnings  https://review.opendev.org/c/openstack/neutron/+/86367212:45
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix in the iteration to obtain address_scope linked to a subnet. A network can be linked to more than one subnet (ipv4 and ipv6), but if one of them does not have an address_scope, a null object element access failure occurs. Change-Id: I9a944c516d4f09e8d3fdf6d1ca7914504a693c28  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86370812:47
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix address_scope calculation  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86370813:08
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix address_scope calculation  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86370813:08
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables  https://review.opendev.org/c/openstack/neutron/+/75987413:49
*** dasm|off is now known as dasm14:11
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Skip "test_port_resource_request_no_provider_net_conflict"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/86388014:14
opendevreviewLajos Katona proposed openstack/neutron master: Fix bulk create without mac  https://review.opendev.org/c/openstack/neutron/+/86388114:15
opendevreviewRodolfo Alonso proposed openstack/neutron master: Support for minimum bandwidth rules in tunnelled networks  https://review.opendev.org/c/openstack/neutron/+/86063914:16
lajoskatonaslaweq: Hi, could you please check https://review.opendev.org/c/openstack/neutron/+/863881 ? It is related to bulk port create and https://bugs.launchpad.net/neutron/+bug/199573214:18
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86371314:33
ralonsohlajoskatona, some comments there, but I think the implementation is correct14:33
lajoskatonaralonsoh: thanks, I have to check testing also :-)14:37
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86371314:38
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86371314:39
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86371314:59
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: Allow shared net to be added on router  https://review.opendev.org/c/openstack/neutron/+/86388515:26
opendevreviewRodolfo Alonso proposed openstack/neutron stable/xena: Allow shared net to be added on router  https://review.opendev.org/c/openstack/neutron/+/86388615:27
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: Allow shared net to be added on router  https://review.opendev.org/c/openstack/neutron/+/86388715:27
opendevreviewRodolfo Alonso proposed openstack/neutron stable/victoria: Allow shared net to be added on router  https://review.opendev.org/c/openstack/neutron/+/86388815:27
opendevreviewRodolfo Alonso proposed openstack/neutron stable/ussuri: Allow shared net to be added on router  https://review.opendev.org/c/openstack/neutron/+/86388915:28
opendevreviewRodolfo Alonso proposed openstack/neutron stable/train: Allow shared net to be added on router  https://review.opendev.org/c/openstack/neutron/+/86389015:28
opendevreviewRodolfo Alonso proposed openstack/neutron master: Support for minimum bandwidth rules in tunnelled networks  https://review.opendev.org/c/openstack/neutron/+/86063916:06
*** sfinucan is now known as stephenfin16:10
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86371316:14
opendevreviewMerged openstack/neutron master: test: fix docstring validation function  https://review.opendev.org/c/openstack/neutron/+/86387016:24
opendevreviewFernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86385216:39
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables  https://review.opendev.org/c/openstack/neutron/+/75987417:04
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM Test FT timeout with oslo master  https://review.opendev.org/c/openstack/neutron/+/86331517:13
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86371317:33
opendevreviewFernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface  https://review.opendev.org/c/openstack/neutron/+/86385218:30
opendevreviewRoberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/86371319:22
opendevreviewBrian Haley proposed openstack/neutron master: Fix some pylint indentation warnings  https://review.opendev.org/c/openstack/neutron/+/86392819:54
*** dasm is now known as dasm|off22:27
opendevreviewBrian Haley proposed openstack/neutron master: Use --exclude-regex in tox.ini  https://review.opendev.org/c/openstack/neutron/+/86395023:43
opendevreviewBrian Haley proposed openstack/neutron master: Use --exclude-regex in tox.ini  https://review.opendev.org/c/openstack/neutron/+/86395023:46

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