opendevreview | renliang proposed openstack/neutron stable/yoga: Update the Ethernet card information https://review.opendev.org/c/openstack/neutron/+/863790 | 01:38 |
---|---|---|
opendevreview | renliang proposed openstack/neutron stable/xena: Update the Ethernet card information https://review.opendev.org/c/openstack/neutron/+/863791 | 01:38 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 08:01 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 08:01 |
QG | Hi | 08:52 |
QG | I was wondering how it works when creating a router and adding or removing an external gateway. | 08:53 |
QG | When 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 |
QG | Of 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 |
QG | Do you know why the schedule agent part is redone when adding or removing the external gateway ? | 08:53 |
isabek | Hi 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-int | 08:54 |
sahid | lajoskatona: 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 |
lajoskatona | sahid: yes, I worked on it Friday, but no success, so I have to check what I had to adjust :-( | 09:09 |
sahid | oh ok so at least i'm not the only one | 09:10 |
sahid | i's already a good positive for me :-) | 09:10 |
sahid | will make an other try | 09:13 |
opendevreview | Fernando Royo proposed openstack/neutron stable/yoga: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863275 | 09:25 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables https://review.opendev.org/c/openstack/neutron/+/759874 | 09:43 |
ralonsoh | QG, do you have HA configured? | 09:47 |
QG | ralonsoh: Yes i have | 09:47 |
ralonsoh | then check l3_hamode_db | 09:47 |
ralonsoh | when the router is updated, the scheduling is called | 09:47 |
ralonsoh | I would need to check that | 09:48 |
ralonsoh | but this should be related to "get_l3_agents_hosting_routers" | 09:48 |
ralonsoh | when you remove the GW port, you are removing one interface | 09:48 |
ralonsoh | that will remove some bindings | 09:49 |
QG | i was thinking may be it's when the notify is called : | 09:49 |
QG | https://github.com/openstack/neutron/blob/master/neutron/db/l3_db.py#L2142-L2143 | 09:49 |
ralonsoh | but I would need to check that | 09:49 |
QG | we don't specify the param `schedule_routers=False` to the routers_updated function | 09:49 |
ralonsoh | this is the call from the server to the agents | 09:49 |
ralonsoh | what does it means you don't specify this paremeter? | 09:51 |
QG | https://github.com/openstack/neutron/blob/master/neutron/api/rpc/agentnotifiers/l3_rpc_agent_api.py#L86 | 09:52 |
QG | In the _notification function the param schedule_routers is set to true so when we notify the agent a schedule is trigger | 09:52 |
ralonsoh | yes | 09:53 |
ralonsoh | and i don't see in the code any place where that is set to false, to be honest | 09:53 |
ralonsoh | isabek, can you reformulate this question? what do you mean? that you need a different way to connect a tap interface? | 09:53 |
QG | no I don't either, so with each action he has to redo the schedule | 09:54 |
QG | Do we need to do a schedule when we add or remove an external gateway ? | 09:54 |
ralonsoh | QG so far yes, as commented, when you remove the GW interface, you are removing one interface of the router | 09:54 |
ralonsoh | if I'm not wrong, I remember other related improvement related to this... but my memory is short | 09:55 |
ralonsoh | let me find this | 09:55 |
QG | so when we remove one interface of the router we want to choose new agent and do no configure them | 09:55 |
QG | cool thanks ! | 09:55 |
sahid | ralonsoh: 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' unavailable | 09:58 |
sahid | looks stange it's probably something tiny that is missing | 09:59 |
sahid | i think ci already have everysting related to database installed, so it's perhpas a package missing with configure_for_func_testing.sh | 10:00 |
ralonsoh | sahid, in the local.conf of devstack | 10:02 |
ralonsoh | try specifying both db types | 10:02 |
ralonsoh | DATABASE_TYPE: postgresql | 10:02 |
ralonsoh | DATABASE_TYPE: mysql | 10:02 |
ralonsoh | try with both | 10:02 |
ralonsoh | (not at the same time) | 10:02 |
ralonsoh | 1) add this value, 2) re-execute the script configure_for_func_testing | 10:02 |
ralonsoh | 3) execute tox fullstack | 10:03 |
sahid | ack, let me try | 10:03 |
opendevreview | Fernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863852 | 10:08 |
ralonsoh | QG no, there is nothing related to this | 10:10 |
ralonsoh | if you think that this could be an improvement, you can open a launchpad bug and push a path for this | 10:10 |
QG | ralonsoh: ok thank i will do ! | 10:10 |
sahid | no luck... | 10:11 |
sahid | is there a place where I can see the package installed in a CI mahcine? | 10:11 |
ralonsoh | $pip freeze | 10:12 |
ralonsoh | for python packages | 10:12 |
sahid | i'm talking about an upstream ci machine | 10:12 |
ralonsoh | in the node logs | 10:12 |
ralonsoh | for example | 10:13 |
sahid | good point | 10:13 |
ralonsoh | https://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.txt | 10:13 |
sahid | thank you for the link | 10:13 |
sahid | i have diff, for example mysqlclient is not instqlled on my machine | 10:14 |
sahid | wait... I was looking into my local machine :-) | 10:14 |
opendevreview | Fernando Royo proposed openstack/neutron stable/victoria: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863861 | 10:15 |
opendevreview | Fernando Royo proposed openstack/neutron stable/train: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863862 | 10:19 |
opendevreview | Fernando Royo proposed openstack/neutron stable/ussuri: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863863 | 10:20 |
opendevreview | Rodolfo Alonso proposed openstack/neutron-specs master: Strict minimum bandwidth support for tunnelled networks https://review.opendev.org/c/openstack/neutron-specs/+/860859 | 11:00 |
opendevreview | Fernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863852 | 12:10 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: test: fix docstring validation function https://review.opendev.org/c/openstack/neutron/+/863870 | 12:21 |
sahid | lajoskatona, ralonsoh ^ an interesting one :-) | 12:21 |
sahid | noticed 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.html | 12:21 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 12:27 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 12:27 |
opendevreview | Merged openstack/neutron master: Fix some pylint indentation warnings https://review.opendev.org/c/openstack/neutron/+/863674 | 12:28 |
lajoskatona | sahid: thanks | 12:28 |
crohmann | coreycb: 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 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables https://review.opendev.org/c/openstack/neutron/+/759874 | 12:41 |
opendevreview | Merged openstack/neutron master: Fix some pylint indentation warnings https://review.opendev.org/c/openstack/neutron/+/863672 | 12:45 |
opendevreview | Roberto 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/+/863708 | 12:47 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix address_scope calculation https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863708 | 13:08 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix address_scope calculation https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863708 | 13:08 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables https://review.opendev.org/c/openstack/neutron/+/759874 | 13:49 |
*** dasm|off is now known as dasm | 14:11 | |
opendevreview | Rodolfo 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/+/863880 | 14:14 |
opendevreview | Lajos Katona proposed openstack/neutron master: Fix bulk create without mac https://review.opendev.org/c/openstack/neutron/+/863881 | 14:15 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Support for minimum bandwidth rules in tunnelled networks https://review.opendev.org/c/openstack/neutron/+/860639 | 14:16 |
lajoskatona | slaweq: 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/1995732 | 14:18 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863713 | 14:33 |
ralonsoh | lajoskatona, some comments there, but I think the implementation is correct | 14:33 |
lajoskatona | ralonsoh: thanks, I have to check testing also :-) | 14:37 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863713 | 14:38 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863713 | 14:39 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863713 | 14:59 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Allow shared net to be added on router https://review.opendev.org/c/openstack/neutron/+/863885 | 15:26 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Allow shared net to be added on router https://review.opendev.org/c/openstack/neutron/+/863886 | 15:27 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Allow shared net to be added on router https://review.opendev.org/c/openstack/neutron/+/863887 | 15:27 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: Allow shared net to be added on router https://review.opendev.org/c/openstack/neutron/+/863888 | 15:27 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: Allow shared net to be added on router https://review.opendev.org/c/openstack/neutron/+/863889 | 15:28 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/train: Allow shared net to be added on router https://review.opendev.org/c/openstack/neutron/+/863890 | 15:28 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Support for minimum bandwidth rules in tunnelled networks https://review.opendev.org/c/openstack/neutron/+/860639 | 16:06 |
*** sfinucan is now known as stephenfin | 16:10 | |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863713 | 16:14 |
opendevreview | Merged openstack/neutron master: test: fix docstring validation function https://review.opendev.org/c/openstack/neutron/+/863870 | 16:24 |
opendevreview | Fernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863852 | 16:39 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][POC] NFTables https://review.opendev.org/c/openstack/neutron/+/759874 | 17:04 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM Test FT timeout with oslo master https://review.opendev.org/c/openstack/neutron/+/863315 | 17:13 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863713 | 17:33 |
opendevreview | Fernando Royo proposed openstack/neutron stable/wallaby: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863852 | 18:30 |
opendevreview | Roberto Bartzen Acosta proposed openstack/neutron-dynamic-routing master: Fix sql queries when using ovn plugin https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/863713 | 19:22 |
opendevreview | Brian Haley proposed openstack/neutron master: Fix some pylint indentation warnings https://review.opendev.org/c/openstack/neutron/+/863928 | 19:54 |
*** dasm is now known as dasm|off | 22:27 | |
opendevreview | Brian Haley proposed openstack/neutron master: Use --exclude-regex in tox.ini https://review.opendev.org/c/openstack/neutron/+/863950 | 23:43 |
opendevreview | Brian Haley proposed openstack/neutron master: Use --exclude-regex in tox.ini https://review.opendev.org/c/openstack/neutron/+/863950 | 23:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!