opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add "neutron-ovn-tempest-ovs-release" job to the check and gate queue https://review.opendev.org/c/openstack/neutron/+/860636 | 07:33 |
---|---|---|
opendevreview | Fernando Royo proposed openstack/neutron master: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/859143 | 07:47 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Placement tunnelled networks https://review.opendev.org/c/openstack/neutron/+/860639 | 08:14 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Clean "ovn ipv6 only" jobs templates https://review.opendev.org/c/openstack/neutron/+/860642 | 08:21 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Docs] Remove python version from the ci jobs summary https://review.opendev.org/c/openstack/neutron/+/860643 | 08:21 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Promote neutron-ovn-rally-task job to be voting in check queue https://review.opendev.org/c/openstack/neutron/+/860644 | 08:21 |
slaweq | ralonsoh ykarel lajoskatona "inspired" by yesterday's drama about broken gates I made some small cleaning in our ci jobs ^^ | 08:22 |
slaweq | please check those 3 patches when You will have time | 08:22 |
ralonsoh | looking now | 08:22 |
slaweq | ralonsoh no rush, I would like first to see that all is green there :) | 08:22 |
ykarel | slaweq, ack | 08:23 |
lajoskatona | slaweq: checking | 08:23 |
opendevreview | Lucas Alvares Gomes proposed openstack/networking-ovn stable/train: Split Hash Ring probing from the maintenance task https://review.opendev.org/c/openstack/networking-ovn/+/860647 | 08:49 |
opendevreview | Arnaud Morin proposed openstack/neutron master: Discard port with ofport -1 in _get_ofport_moves https://review.opendev.org/c/openstack/neutron/+/860649 | 09:14 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/860671 | 09:39 |
opendevreview | Merged openstack/neutron stable/ussuri: Convert OvnDbNotifyHandler rows to frozen rows https://review.opendev.org/c/openstack/neutron/+/860248 | 11:08 |
*** dasm|off is now known as dasm | 13:24 | |
opendevreview | Merged openstack/neutron stable/victoria: Split Hash Ring probing from the maintenance task https://review.opendev.org/c/openstack/neutron/+/860566 | 13:24 |
opendevreview | Merged openstack/neutron stable/ussuri: Split Hash Ring probing from the maintenance task https://review.opendev.org/c/openstack/neutron/+/860567 | 13:36 |
opendevreview | Lukas Piwowarski proposed openstack/neutron master: Remove reference to 'all-plugin' tox environment https://review.opendev.org/c/openstack/neutron/+/860700 | 13:55 |
lajoskatona | #startmeeting neutron_drivers | 14:00 |
opendevmeet | Meeting started Fri Oct 7 14:00:24 2022 UTC and is due to finish in 60 minutes. The chair is lajoskatona. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'neutron_drivers' | 14:00 |
lajoskatona | o/ | 14:00 |
mlavalle | o/ | 14:00 |
ralonsoh | hello | 14:00 |
slaweq | o/ | 14:00 |
haleyb | o/ | 14:00 |
obondarev | hi | 14:01 |
amotoki | o/ | 14:01 |
lajoskatona | ok, let's start it | 14:02 |
lajoskatona | We have one RFE for today: | 14:02 |
lajoskatona | [RFE] Strict minimum bandwidth support for tunnelled networks (#link https://bugs.launchpad.net/neutron/+bug/1991965 ) | 14:02 |
lajoskatona | from ralonsoh | 14:02 |
ralonsoh | thanks, I have the speech prepared | 14:02 |
ralonsoh | First of all, this RFE will require a spec | 14:02 |
ralonsoh | The goal of this RFE is to provide a mechanism to model the tunnelled networks in a host, same as we do with the different physical interfaces | 14:02 |
ralonsoh | In this case, this is "easier": for each compute node, there will be just one single "local_ip" (where the backend sends the traffic and its a VTEP of the overlay network) | 14:02 |
ralonsoh | How it will work: | 14:02 |
ralonsoh | 1) The agent will report the node tunnelled network BW | 14:02 |
ralonsoh | 2) The port now will create a "resource_request" even if it belongs to an overlay network (now, if the first network segment has no physical network, the resource request is empty) | 14:02 |
ralonsoh | ^^^ caveat: that will change the current behaviour if an active deployment has ports on tunnelled networks with QoS policies and min-BW associated. Now those rules won't apply (not in the backend enforcement nor in the scheduling via Placement) | 14:02 |
yamamoto | hi | 14:03 |
lajoskatona | Hi | 14:03 |
lajoskatona | for this my feeleing from the RFE that nova change is also necessary, ami wrong? | 14:04 |
ralonsoh | not really, the port resource request will request the new trait added | 14:04 |
opendevreview | Lukas Piwowarski proposed openstack/neutron-fwaas master: Remove reference to 'all-plugin' tox environment https://review.opendev.org/c/openstack/neutron-fwaas/+/860704 | 14:05 |
ralonsoh | and the agent will report, at the init time, the tunnelled network BW | 14:05 |
ralonsoh | so the resource provider will be created | 14:05 |
lajoskatona | and nova can do the placement request based on that without change? | 14:05 |
ralonsoh | I'm currently testing this (with OVN) in https://review.opendev.org/c/openstack/neutron/+/860639 | 14:05 |
ralonsoh | this is just a 30 mins patch | 14:05 |
opendevreview | Merged openstack/ovn-octavia-provider master: Ensure lbs are properly configured for router gateway set/unset https://review.opendev.org/c/openstack/ovn-octavia-provider/+/858363 | 14:06 |
ralonsoh | and works, placement reserves the BW for the tunneled network | 14:06 |
lajoskatona | suprising :-) | 14:06 |
ralonsoh | this is just a matter of requesting the needed trait | 14:06 |
lajoskatona | sounds cool, I will test it with OVS next week than :-) | 14:07 |
ralonsoh | just with a small change in the OVS agent init method | 14:07 |
ralonsoh | and handling the current 1:1 association between the BW reported and the bridge mapping | 14:07 |
ralonsoh | because in this case there is no bridge | 14:07 |
ralonsoh | --> https://review.opendev.org/c/openstack/neutron/+/860639/1/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/extensions/placement.py#266 | 14:08 |
ralonsoh | of course, that will be discussed in the spec | 14:08 |
lajoskatona | +1 | 14:09 |
ralonsoh | ah, this is valid for OVN and OVS | 14:09 |
slaweq | +1 from me :) | 14:09 |
lajoskatona | +1 from me also | 14:09 |
ralonsoh | (slaweq has the credit of this idea) | 14:09 |
lajoskatona | a useful generalization of the original feature | 14:09 |
ralonsoh | yeah, some customers asking for this | 14:10 |
ralonsoh | as most of them they are using only vxlan networks | 14:10 |
opendevreview | Merged openstack/neutron stable/yoga: Split Hash Ring probing from the maintenance task https://review.opendev.org/c/openstack/neutron/+/860563 | 14:10 |
lajoskatona | Anybody else do you have any concerns? | 14:11 |
mlavalle | None | 14:12 |
mlavalle | good explanation ralonsoh | 14:12 |
mlavalle | thanks | 14:12 |
haleyb | none from me | 14:12 |
lajoskatona | ralonsoh: you mentioned spec, you are preparing it? | 14:12 |
ralonsoh | right now | 14:13 |
lajoskatona | ok, thanks | 14:13 |
opendevreview | Merged openstack/neutron stable/zed: Split Hash Ring probing from the maintenance task https://review.opendev.org/c/openstack/neutron/+/860562 | 14:13 |
amotoki | does this just ensure the minimum bw for tunnel only on a compute node, right? | 14:13 |
ralonsoh | yes, any tunnelled traffic in one compute node | 14:13 |
amotoki | thanks | 14:14 |
amotoki | I am okay with this proposal | 14:14 |
yamamoto | nothing from me | 14:14 |
mlavalle | it pays to come to the meeting with a "prepared speech" | 14:15 |
lajoskatona | ok, I think we agreed to continue this and accept the RFE, thanks ralonsoh (and slaweq :)) | 14:15 |
ralonsoh | thank you all | 14:15 |
slaweq | :) | 14:15 |
lajoskatona | #topic On Demand Agenda | 14:15 |
lajoskatona | I have one (it is not in the agenda as it formed for me today) | 14:16 |
lajoskatona | Nova is in a big refactoring of the PCI stuff handling | 14:16 |
lajoskatona | https://specs.openstack.org/openstack/nova-specs/specs/zed/approved/pci-device-tracking-in-placement.html | 14:16 |
lajoskatona | currently it is ready for the pci handling where neutron is not involved, and they are preparing for doing it for neutron managed sriov/smartnic whatever ports also | 14:17 |
lajoskatona | so this is more a FYI to be prepared for it | 14:17 |
ralonsoh | should we add this topic to the PTG? to the x-team meeting | 14:17 |
lajoskatona | in the spec what I copied there is a section for neutron as not covered currently but with some details how they think now | 14:18 |
mlavalle | ralonsoh: you read my mind | 14:18 |
lajoskatona | ralonsoh: +1 | 14:18 |
lajoskatona | that's why I bring this here | 14:18 |
ralonsoh | done, I'll add it (on thursday we'll have those meetings) | 14:18 |
mlavalle | +1 | 14:18 |
lajoskatona | cool | 14:18 |
ralonsoh | (on monday I'll update the scheduling) | 14:18 |
lajoskatona | that's all from me | 14:18 |
lajoskatona | Do you have anything more to discuss? | 14:19 |
slaweq | nope | 14:19 |
mlavalle | so, next drivers meeting will be chaired by ralonsoh | 14:20 |
lajoskatona | yes, good topic :-) | 14:20 |
ralonsoh | yeah, I think it is time | 14:20 |
lajoskatona | this week I forgot to push this on his back :-) | 14:20 |
ralonsoh | I'll take care of this meeting since the next week | 14:20 |
mlavalle | well, thank you for leading us this past year lajoskatona !!! | 14:20 |
ralonsoh | lajoskatona++ thanks a lot! | 14:20 |
mlavalle | you did a great job | 14:20 |
amotoki | lajoskatona: thanks a lot | 14:20 |
lajoskatona | :-) | 14:20 |
lajoskatona | I tried my best | 14:21 |
slaweq | lajoskatona++ thx for leading this meeting for long time | 14:21 |
lajoskatona | it was really good experience | 14:21 |
mlavalle | and welcome to our new Fearless Leader, ralonsoh! | 14:21 |
lajoskatona | Yeayyyy | 14:21 |
ralonsoh | thanks | 14:22 |
lajoskatona | If nothing more we can close the meeting | 14:22 |
slaweq | \o/ | 14:22 |
ralonsoh | have a nice weekend | 14:22 |
lajoskatona | #endmeeting | 14:22 |
opendevmeet | Meeting ended Fri Oct 7 14:22:52 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:22 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-10-07-14.00.html | 14:22 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-10-07-14.00.txt | 14:22 |
opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-10-07-14.00.log.html | 14:22 |
amotoki | o/ | 14:22 |
mlavalle | you too! | 14:22 |
slaweq | have a nice weekend all | 14:22 |
lajoskatona | Bye, have a nice weekend | 14:22 |
mlavalle | o/ | 14:22 |
slaweq | bye | 14:23 |
opendevreview | Merged openstack/neutron stable/xena: Split Hash Ring probing from the maintenance task https://review.opendev.org/c/openstack/neutron/+/860564 | 15:17 |
opendevreview | Merged openstack/neutron master: Remove reference to 'all-plugin' tox environment https://review.opendev.org/c/openstack/neutron/+/860700 | 15:17 |
opendevreview | Merged openstack/neutron stable/zed: Accept a port deletion with missing port binding information https://review.opendev.org/c/openstack/neutron/+/860069 | 16:31 |
opendevreview | Merged openstack/neutron stable/yoga: Accept a port deletion with missing port binding information https://review.opendev.org/c/openstack/neutron/+/860071 | 16:31 |
opendevreview | Merged openstack/neutron stable/xena: Accept a port deletion with missing port binding information https://review.opendev.org/c/openstack/neutron/+/860072 | 16:32 |
opendevreview | Merged openstack/neutron stable/zed: Port provisioning should retry only for VM ports https://review.opendev.org/c/openstack/neutron/+/859914 | 16:32 |
opendevreview | Merged openstack/neutron-lib master: Use oslo_context.from_dict() for context generation https://review.opendev.org/c/openstack/neutron-lib/+/859813 | 16:56 |
ozzzo_work | ralonsoh: Sorry if I wasn't clear. I know why it happened, it was because the customer deleted VMs during a maintenance. I am just trying to figure out how to detach a port from a nonexistent VM | 17:03 |
ozzzo_work | via cli | 17:03 |
ozzzo_work | since I can do it in the GUI, I'm guessing that there must be a way to do it via CLI also | 17:04 |
frickler | ozzzo_work: "openstack port unset --binding-profile"? | 17:11 |
ozzzo_work | frickler: ty! I'll try that the next time it happens | 17:56 |
ozzzo_work | What is <binding-profile-key>, is that going to be binding_host_id? | 17:58 |
*** jlibosva is now known as Guest2583 | 18:18 | |
*** dasm is now known as dasm|off | 19:32 | |
*** EugenMayer6 is now known as EugenMayer | 22:13 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!