| opendevreview | Terry Wilson proposed openstack/neutron master: Avoid race condition by using persist_uuid https://review.opendev.org/c/openstack/neutron/+/923971 | 00:38 |
|---|---|---|
| opendevreview | Terry Wilson proposed openstack/neutron master: Avoid race condition by using persist_uuid https://review.opendev.org/c/openstack/neutron/+/923971 | 05:48 |
| opendevreview | Rodolfo Alonso proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 06:35 |
| opendevreview | Rodolfo Alonso proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 06:50 |
| opendevreview | Rodolfo Alonso proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 07:18 |
| opendevreview | Rodolfo Alonso proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 08:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Missing required extension in 'subnet-external-network' https://review.opendev.org/c/openstack/neutron-lib/+/924041 | 08:52 |
| ralonsoh | slaweq, lajoskatona ^^ trivial fix (thanks lajoskatona for commenting that) | 08:53 |
| opendevreview | Rodolfo Alonso proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 09:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "subnet-external-network" extension to "subnet" resource https://review.opendev.org/c/openstack/neutron/+/907313 | 09:28 |
| ralonsoh | slaweq, ^ | 09:29 |
| ralonsoh | lajoskatona, Eureka! https://review.opendev.org/c/openstack/tap-as-a-service/+/923837. I've merged the taas plugin ini file in the ml2_plugin one. By default, we always read the default two files defined in https://github.com/openstack/neutron/blob/master/neutron/server/__init__.py#L50 | 09:59 |
| ralonsoh | because in wsgi we don't explicitly pass any config file, we must add all config there | 09:59 |
| ralonsoh | (or change how are we loading the wsgi module, passing extra files) | 10:00 |
| lajoskatona | ralonsoh: Cool, I check it | 10:02 |
| lajoskatona | ralonsoh: I have patch for a limited OVN driver for taas, I will update the relevant patch for it to make that working also (job config for it: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/886004 ) | 10:04 |
| ralonsoh | ufff a big one, I'll need some time to review it | 10:06 |
| opendevreview | Merged openstack/neutron stable/2023.2: Add policy enforcer for "tags" service plugin https://review.opendev.org/c/openstack/neutron/+/923493 | 10:27 |
| vicent | Good morning! I have two PFs with two VFs each. The PFs are on switchdev mode (I am using sriovnicswitch). The PFs are connected to different networks, so when I create a neutron port with type direct, I need a way to map that port so it uses one of the VFs of a specific PF. I found this document | 10:32 |
| vicent | https://specs.openstack.org/openstack/nova-specs/specs/zed/approved/pci-device-tracking-in-placement.html which uses traits, but I need to set the trait on the port, not on the flavor. Is there a way to do what I am trying to do? | 10:32 |
| vicent | Basically, how can I configure the VFs of different PF in pools, so I have control to which VF pool is used for which network? | 10:34 |
| ralonsoh | vicent, to be honest, I'm not sure how to do this. However, I would suggest to define the resource_provider_bandwidths of both devices (PFs). That will create a resource provider per PF. Then it is possible to add new traits (the physical_network) to each defined resource provider | 10:58 |
| ralonsoh | what I don't know is how Nova request it: for BW the port should have a QoS defined, but if that is not the case, I don't know if nova will use the physical network | 10:59 |
| ralonsoh | please, check that in the Nova channel, the spec has some missing improvements (physical_network, for example) | 10:59 |
| ralonsoh | if that is not the case, Nova folks will tell you how Nova makes this request to placement | 11:00 |
| vicent | I was able to create resource providers with the device_spec. I can see them on the placement. | 11:00 |
| vicent | Ok, I will ask on Nova channel. | 11:00 |
| vicent | Thanks | 11:00 |
| ralonsoh | but that's a resource from Nova | 11:00 |
| ralonsoh | how Nova request a specific VF then? using the PF vendor/product? | 11:01 |
| opendevreview | Chris Buggy proposed openstack/ovn-octavia-provider master: Adding new error log if SSL, ca and cert file are not set. Change-Id: I2a21b94fee03767a5f703486bdab2908cda18746 https://review.opendev.org/c/openstack/ovn-octavia-provider/+/924059 | 11:13 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 12:42 |
| ralonsoh | lajoskatona, ^ the variable TAAS_SERVICE_DRIVER is not defined | 12:45 |
| ralonsoh | this is why I added the string copied from the etc/taas_plugin.ini file | 12:46 |
| ralonsoh | actually in the next lines, if TAAS_SERVICE_DRIVER is defined, if will override the string defined | 12:46 |
| lajoskatona | ralonsoh: hmmmm, and if we add TAAS_SERVICE_DRIVER="...." to the settings file? | 12:52 |
| ralonsoh | IMO, we can add the default string, provided in the taas_plugin.ini file, that I added in PS11 | 12:52 |
| ralonsoh | but anyone can actually override this value defining a value in TAAS_SERVICE_DRIVER | 12:53 |
| ralonsoh | it is used in L41-42 | 12:53 |
| ralonsoh | it comments the default value and adds the manually defined one | 12:53 |
| lajoskatona | ralonsoh: ok, so we have to keep the long string there anyway | 12:55 |
| ralonsoh | yes, that is easier, IMO | 12:55 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 12:56 |
| lajoskatona | ralonsoh: thanks, I reversed back to your version | 12:57 |
| ralonsoh | thank | 12:57 |
| ralonsoh | thanks | 12:57 |
| lajoskatona | ralonsoh, haleyb: I have to fetch my son from the swimming-pool, so I can't join the drivers meeting today. | 13:22 |
| ralonsoh | lajoskatona, haleyb|out is on PTO | 13:25 |
| ralonsoh | hmmm, let's see if there is quorum today | 13:25 |
| mlavalle | ralonsoh: are we meeting today? | 13:57 |
| ralonsoh | yes but, to be honest, I don't know if we have quorum | 13:58 |
| mlavalle | ralonsoh: ack, np I'll be here in case we achieve quorum | 13:59 |
| ralonsoh | #startmeeting neutron_drivers | 14:01 |
| opendevmeet | Meeting started Fri Jul 12 14:01:29 2024 UTC and is due to finish in 60 minutes. The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'neutron_drivers' | 14:01 |
| ralonsoh | hello all | 14:01 |
| mlavalle | \o | 14:01 |
| ralonsoh | Ping list: ykarel, mlavalle, mtomaska, slaweq, obondarev, tobias-urdin, lajoskatona, amotoki, haleyb, ralonsoh | 14:01 |
| obondarev | o/ | 14:02 |
| cbuggy | o/ Hey everyone. | 14:02 |
| ykarel | o/ | 14:02 |
| slaweq | hi, sorry but I can't attend today | 14:04 |
| ralonsoh | sorry folks but we are short of people today | 14:04 |
| ralonsoh | we have one topic from me that could wait | 14:04 |
| ralonsoh | and another one from slaweq that could wait too | 14:05 |
| ralonsoh | the last one could be discussed next tuesday during the team meeting | 14:05 |
| ralonsoh | please check https://bugs.launchpad.net/neutron/+bug/2067183 and the related patch | 14:05 |
| mlavalle | ack | 14:05 |
| ralonsoh | to have better knowledge next day | 14:05 |
| ralonsoh | I'll close this meeting for today | 14:06 |
| ralonsoh | is that ok? | 14:06 |
| mlavalle | yes | 14:06 |
| obondarev | sure | 14:06 |
| ralonsoh | #endmeeting | 14:06 |
| opendevmeet | Meeting ended Fri Jul 12 14:06:15 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:06 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2024/neutron_drivers.2024-07-12-14.01.html | 14:06 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2024/neutron_drivers.2024-07-12-14.01.txt | 14:06 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2024/neutron_drivers.2024-07-12-14.01.log.html | 14:06 |
| ralonsoh | thank you folks, have a nice weekend | 14:06 |
| mlavalle | ralonsoh: good luck on | 14:06 |
| mlavalle | Sunday against England | 14:06 |
| mlavalle | I just jumped on the Spain bandwagon. I read they are the favorites | 14:07 |
| ralonsoh | thanks! | 14:07 |
| opendevreview | Chris Buggy proposed openstack/ovn-octavia-provider master: Error log for missing certs with NB and SB DBs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/924059 | 14:14 |
| opendevreview | Stefan Hoffmann proposed openstack/neutron master: [OVN] Add OVS_HYBRID_PLUG to port binding in db https://review.opendev.org/c/openstack/neutron/+/923962 | 14:55 |
| opendevreview | Stefan Hoffmann proposed openstack/neutron master: [OVN] Wait for ovn-controller at migration https://review.opendev.org/c/openstack/neutron/+/923963 | 14:55 |
| gmann | slaweq: sorry for delay but manager role lgtm, https://review.opendev.org/c/openstack/neutron/+/923578 | 17:15 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: style: Clarify a check for mixed stateful/stateless SGs https://review.opendev.org/c/openstack/neutron/+/924087 | 18:05 |
| opendevreview | Merged openstack/neutron stable/2023.1: [2023.1 Only][OVN] Add the bridge name and datapath type to the port VIF details https://review.opendev.org/c/openstack/neutron/+/922431 | 21:32 |
| opendevreview | Merged openstack/neutron stable/2023.2: Reorder subnet RBAC policy check strings https://review.opendev.org/c/openstack/neutron/+/923494 | 23:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!