Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-11-13
11:49:19 sean-k-mooney personally woudl have modele this very slightly diffent. the placemetn resouces manage by a compuate agent are a forest fo trees not a signel tree. so i would not have accpeted a list of compute nodes however you wuld still have needed a list a of root for sharing resouce providers, i think if we upleveled the compute nodes to a seperate map of cn uuid to provider tree the
11:49:21 sean-k-mooney lock in the provder tree coudl use the cn.uuid but the map woudl still likely need to ahve a single lock
11:52:11 sean-k-mooney we did refactor this a bit but we never actully turnd it into a forest of trees
11:56:37 opendevreview sean mooney proposed openstack/nova master: ensure correct cleanup of multi-attach volumes https://review.opendev.org/c/openstack/nova/+/916322
11:57:49 opendevreview sean mooney proposed openstack/nova master: ensure correct cleanup of multi-attach volumes https://review.opendev.org/c/openstack/nova/+/916322
12:28:41 gibi stephenfin: replied in the commit
12:32:22 gibi sean-k-mooney: I'm questioning the need of a named and therefore global lock for any instances of ProviderTree in the process. Even if it is as simple libvirt compute with just a single root provider. I don't see why we need to ensure that two copies of the ProviderTree is not accessed at the same time. I totally understand why we would use a threading.Lock per ProviderTree instance so concurrent
12:32:28 gibi access to the data is synchronized.
12:33:46 gibi but anyhow I keep the global named lock for the ProviderTree in https://review.opendev.org/c/openstack/nova/+/956091 I just feel like it is totally unnecessary
12:33:49 sean-k-mooney yep i realsise that. im just wondering is there any shareing here btween the reshaps that can happen due ot OTU devices and the perodics
12:34:02 sean-k-mooney i dont think we share the same PT between treads today
12:34:09 sean-k-mooney so a simpel lock is likely enough
12:34:52 gibi when we update placement we update that based on a specific ProviderTree instance. A copy of that object does not matter as it is an independent copy
12:34:58 sean-k-mooney the real qustion for me is do we ever store this PT in a shared location like the resouce tracker
12:35:08 sean-k-mooney or is it only constucted in side a function
12:35:21 gibi we pass around PTs
12:35:34 gibi and we alos copy them around in some cases
12:35:48 gibi because we pass arond a threading.lock is needed
12:35:52 sean-k-mooney right so its pass down the call stack but not in shared mutable storage either because of a copy or how its constucted
12:36:32 sean-k-mooney im not directly seeing a need for it to be a named lock either for what its worth
12:36:33 gibi even if we pass to a shared mutable storage, we pass an instance there with a threading lock is enough, we don't need to share the lock across the copies because they are independent deep copies
12:37:15 gibi actually in nova-compute a named lock is never needed as the compute is a single process
12:37:35 gibi so we only sync across threads so a threading.lock is enough
12:37:54 gibi in the scheduler or conductor a named lock might be needed iff we share data across worker processes
12:38:00 gibi like DB state
12:39:10 gibi anyhow I stop here. I just noticed it and felt strange, but I can ignore it and be safe that it will keep working as before
14:20:03 opendevreview Balazs Gibizer proposed openstack/nova master: Compute manager to use thread pools selectively https://review.opendev.org/c/openstack/nova/+/966016
14:20:04 opendevreview Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467
14:20:04 opendevreview Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949
14:21:22 gibi gmaan: FYI https://review.opendev.org/c/openstack/nova/+/966016 proposes changing the default value of some of our config options. After the discussion of dansmith and sean-k-mooney we think this is OK from compatibility perspective via having an upgrade relnote. Do you agree?
14:32:43 opendevreview Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949
14:32:43 opendevreview Balazs Gibizer proposed openstack/nova master: Compute manager to use thread pools selectively https://review.opendev.org/c/openstack/nova/+/966016
14:32:44 opendevreview Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467
14:33:21 opendevreview Dan Smith proposed openstack/nova master: Test nova-next with >1 parallel migrations https://review.opendev.org/c/openstack/nova/+/966447
15:08:00 gibi stephenfin: after a bit of deliberation and a failed trial I'm OK with the direction in 967005
15:08:04 gibi 967005
15:08:13 gibi https://review.opendev.org/c/openstack/nova/+/967005
15:08:22 gibi (wtf happens with my copy paste buffer)
15:09:35 gibi stephenfin: will you fix the 700 failed unit test or looking for a volunteer to take over?
15:10:07 dansmith gibi: I still haven't looked at your patch to fix.. are you saying that one is not working and this is the one we need to pursue?
15:10:23 gibi nope, this is a followup top of mine
15:10:31 dansmith okay
15:10:36 gibi to prevent later patches re-introducing the issue
15:11:01 gibi ~ the generic issue of calling driver methods before driver.init_host
15:11:04 stephenfin gibi: sounds good: I had tried an alternative involving a metaclass but it was feeling...complicated
15:11:55 gibi metaclass, I haven't thought about it. Woudl that sidestep our inheritance pattern?
15:12:22 gibi anyhow
15:12:28 gibi I'm OK not going there :)
15:12:32 opendevreview Stephen Finucane proposed openstack/nova master: virt: Ensure init_host is always called https://review.opendev.org/c/openstack/nova/+/967051
15:12:42 stephenfin gibi: that's the patch (incomplete)
15:14:23 gibi ahh OK, so that basically automating injecting a wrapper to the normal virt calls. I think that will not work as the virt interface function is not called by the implementation as most of the virt inteface calls raising NotImplementedError
15:15:37 stephenfin yeah, that's what I'd figured out before lunch: we'd need to use the metaclass on the actual implementation rather than the base class. And ideally only wrap methods defined in the base class (not that drivers should have other public methods)
15:15:48 gibi yeah
15:16:04 gibi then I think we felt the same pain
15:16:37 gibi anyhow let me know if you need help with the unit test or need a re-review if you jump on the unit test by yourself
16:07:51 vsaienko hey sean-k-mooney: maybe it will be interesting there are performance tests for different scenarios (asap2, vdpa with packed ring 8 queues, virtio single queue, virtio packed + 8 queues) https://jumpshare.com/s/xHU2UTQoKoeKwAWDFsa6 https://jumpshare.com/s/mLQ1u29xOyMftaULc0lJ so far VDPA does not show super significant improvements on 10Gb.
16:08:25 vsaienko scenarios are iperf tests, arguments in each scenario description
16:18:40 sean-k-mooney i think that is in line with expectation
16:19:38 vsaienko I thought that vdpa will be near same as sriov
16:19:41 sean-k-mooney for tcp at least
16:20:02 vsaienko maybe it will be closer on higher rates 50Gbps or 100Gbps
16:20:19 sean-k-mooney vdpa can by connectx6 dx is not doign vdpa fully in hardware
16:21:11 vsaienko nvidiay says they support hardware vdpa on connect 6dx
16:21:18 sean-k-mooney can you expalin the labelign by the way? when you saw swtich deve you mean hardware offloead ovs with a driect assigned VF i assume
16:22:05 sean-k-mooney vsaienko: yes but its not done fully in an asic it does soem fo the data trasformation using firware/cpu comptue on the nic
16:23:08 vsaienko switchdev - sriov + asap2 hardware offload, vdpa-packed - its packed ring + 8 queues, virtio def - is default flavor, no multiqueue or packed ring, virtio-packed - packed ring + 8 queues
16:23:20 vsaienko vdpa with hardware offload as well
16:23:45 vsaienko between hosts VXLAN network, mtu 1450 (no jumboframes) on all configurations
16:23:46 sean-k-mooney ok because the defautl is usign vdpa so its not correct to call it virtio
16:24:06 sean-k-mooney ah you also adding vxlan overhead
16:24:45 sean-k-mooney so is this inter host traffic i.e. 2 vms on diffent hosts or 2 vms on the same host
16:25:01 vsaienko virtio def is default neutron port =normal which is virtio
16:25:16 sean-k-mooney its technaily kernel vhost
16:25:18 vsaienko all scenarios between VMs on different hosts
16:26:01 sean-k-mooney virtio woudl be the virto stack in ovs without the kernel vhost-net module offloading it
16:27:29 sean-k-mooney this looks to me like you are hittig a code path that is not supproted for hardware offload
16:28:00 sean-k-mooney for example the conenction tracker used for security groups is not fully offloadable to hardware
16:28:08 sean-k-mooney are you using ml2/ovs?
16:28:35 vsaienko its ovs, no qos no security groups, in TC I see that traffic is offloaded
16:29:25 sean-k-mooney well in https://jumpshare.com/s/xHU2UTQoKoeKwAWDFsa6
16:29:50 sean-k-mooney you are not hitting line rate env in the swtichdev case which is effectivly jsut using sriov for the dataplane
16:29:59 sean-k-mooney with ovs providing the contol plane
16:30:10 vsaienko yes
16:30:29 sean-k-mooney that to me suggest you are hittign bottel necks in teh trafic generateion as well
16:31:13 vsaienko not sure between hosts on same hypervisor iperf can show more, its new version with multithreads
16:31:35 sean-k-mooney well ierf it not really godo for doing real testing
16:31:43 sean-k-mooney you can use -p to have it use multipel cores
16:32:02 sean-k-mooney but if you want ot test this properly you eigher need a hardware traffic generator or somethign like trex
16:32:17 vsaienko yes, there are scenarios with p1 - its -P 1, p30 - -P 30
16:32:23 sean-k-mooney https://trex-tgn.cisco.com/
16:33:13 sean-k-mooney right but for 1400 by packets you shoudl be able to hit 10G even with 1 thread
16:34:03 sean-k-mooney if the hsot are other wise idel with 1400 by packets even standar kernel ovs shoudl be able to hit 10G
16:34:26 sean-k-mooney so to me the performac eof this host/vms is very much below what i woudl expect
16:36:00 sean-k-mooney vsaienko: what version of opesntack are you using
16:36:20 vsaienko but why with sriov it provides 6+gbps
16:36:27 vsaienko its epoxy
16:36:42 sean-k-mooney so 6GBPS is also too low for sriov
16:36:42 vsaienko 3.x openvswitch
16:37:01 vsaienko maybe indeed its a limitation of iperf
16:37:26 sean-k-mooney so if your using epoxy you shoudl have https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/ovs.py#L107-L126

Earlier   Later