Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-13
21:31:44 efried bauzas: It's less of a problem for VGPU, which is single unit
21:31:51 efried bauzas: Oh, no, that's a separate problem.
21:32:19 bauzas sorry, I learnt German and English at college, not Placement :p
21:53:13 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.openstack.org/602474
21:53:59 bauzas oh, we're not advertising on nova-specs changes ?
21:54:22 bauzas I mean, I don't openstackgerrit telling my spec
21:55:43 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.openstack.org/602474
21:56:54 bauzas mmm, strange but ok
21:58:34 openstackgerrit sean mooney proposed openstack/os-vif master: add support for generic tap device plug https://review.openstack.org/602384
21:59:06 bauzas mriedem: given what we said yesterday, shall we put a procedural -2 on https://review.openstack.org/#/c/599208/ and say we can't do it until https://review.openstack.org/#/q/topic:use-nested-allocation-candidates+(status:open+OR+status:merged) is merged ?
22:01:04 mriedem because once the vgpu is on the child provider, vgpu scheduling would be broken?
22:03:42 bauzas mriedem: yup
22:04:00 bauzas no valid hosts
22:04:15 bauzas since it wouldn't find any RP that fulfills the VGPU resource class
22:04:25 mriedem bauzas: i guess? alternatively, we should have a functional test that does the reshape and then tries to schedule to the vgpu in the child - which we said we wanted as part of the checklist for placement extraction
22:04:44 bauzas mriedem: yeah, I discussed that with gibi
22:04:55 bauzas mriedem: the functional test should see that
22:04:56 mriedem we do have functional tests for reshaper with the fake virt driver,
22:05:02 mriedem we just don't try to schedule to that host after the reshape
22:05:08 openstackgerrit Merged openstack/nova stable/rocky: Add functional test for affinity with multiple cells https://review.openstack.org/599731
22:05:17 bauzas mriedem: cool, I know what to do for next week
22:05:52 mriedem the functional test for reshaper mocks out the reshape code in the test itself; i suggested to efried that we move that to a fake virt driver - and if we did that, then i think we can just bolt the post-reshape scheduler request onto the same test
22:06:18 mriedem maybe we don't need to move that into a fake virt driver, idk
22:06:29 mriedem but would be nice if we can just add a few lines to that existing test to see if this passes or fails
22:07:24 bauzas mriedem: lemme test that
22:07:51 mriedem bauzas: https://github.com/openstack/nova/blob/0ba34a818414823eda5e693dc2127a534410b5df/nova/tests/functional/test_servers.py#L1835
22:08:16 mriedem yeah so at the end, just try to create an instance on that host with vgpu and see if it blows up
22:08:33 mriedem if it fails, we can assert it should fail until https://review.openstack.org/#/q/topic:use-nested-allocation-candidates+(status:open+OR+status:merged) fixes it
22:08:37 mriedem and rebase those on top
22:08:41 mriedem LIKE A CHAMP
22:08:53 efried mriedem: What ought to happen is: the fake libvirt driver should (by default? based on fixture setup?) call the real libvirt driver's impl of update_provider_tree, which should be modified to do reshaping for vgpus in this cycle (and maybe other things) so we can actually test those things in functional tests.
22:09:03 bauzas mriedem: https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L1835 ?
22:09:10 bauzas oh shit, sorry
22:09:11 mriedem bauzas: yes just linked that above
22:09:15 mriedem gawd!
22:09:19 efried mriedem: But we should (still) absolutely not move the mocked-up update_provider_tree-of-bizarreness from that mock to the fake libvirt driver. It just doesn't make sense.
22:09:32 mriedem efried: i will cut your damn letter jacket
22:09:32 bauzas I should employ someone to read my IRC
22:09:50 mriedem efried: honestly i don't care how it's done
22:09:51 efried Just try it. Actually, the last guy who said that was also named Matt.
22:09:53 mriedem just that it's tested
22:09:54 openstackgerrit sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.openstack.org/602432
22:15:04 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Refix disk size during live migration with disk over-commit https://review.openstack.org/602477
22:15:12 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: Refix disk size during live migration with disk over-commit https://review.openstack.org/602478
22:21:07 jroll melwitt: https://docs.openstack.org/ironic/latest/install/configure-nova-flavors.html
22:23:42 melwitt jroll: yeah, I'm pretty sure that override will _not_ stop quota counting from considering and summing the values in the flavor
22:25:20 sean-k-mooney mriedem: o/ fixed now on irc
22:26:44 melwitt jroll: we count the 'vcpus' and 'memory_mb' columns in the 'instances' table to count as quota usage. and I don't think the extra spec overrides will affect how those columns are populated
22:26:52 melwitt and those values come from the flavor
22:30:47 mriedem sean-k-mooney: https://etherpad.openstack.org/p/nova-network-removal-rocky
22:31:01 mriedem there are still some apis to investigate there
22:31:46 sean-k-mooney mriedem: thanks i have bookmarked it ill take a look
22:34:28 openstackgerrit Jay Pipes proposed openstack/nova master: remove virt driver requires_allocation_refresh https://review.openstack.org/602481
22:39:15 jroll melwitt: awesome, thanks :)
23:17:52 openstackgerrit Chris Dent proposed openstack/nova-specs master: List resource providers having inventory https://review.openstack.org/600016
23:30:12 openstackgerrit Merged openstack/nova stable/rocky: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/599732
23:35:50 openstackgerrit Merged openstack/nova stable/rocky: Fix DB archiver AttributeError due to wrong table name attribute used https://review.openstack.org/599881
23:37:42 openstackgerrit Sam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list. https://review.openstack.org/526558
#openstack-nova - 2018-09-14
00:24:13 openstackgerrit Tony Breeds proposed openstack/python-novaclient stable/queens: Switch to stestr https://review.openstack.org/601933
00:24:21 openstackgerrit Tony Breeds proposed openstack/python-novaclient stable/queens: import zuul job settings from project-config https://review.openstack.org/601400
01:05:33 naichuans_ bauzas: Sorry, I'm offline last night
01:39:06 openstackgerrit Naichuan Sun proposed openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672
02:00:51 openstackgerrit Merged openstack/nova stable/rocky: Do not dump all instances in the scheduler https://review.openstack.org/601285
02:25:11 openstackgerrit Zhenyu Zheng proposed openstack/nova stable/rocky: Fix service list for disabled compute using MC driver https://review.openstack.org/592310
02:32:14 openstackgerrit huanhongda proposed openstack/nova master: DNM: Test get deleted instances by non-admin https://review.openstack.org/602514
02:56:34 pvc hi guys
02:58:02 openstackgerrit Merged openstack/nova stable/queens: [Stable Only] Remove soft-deleted instances from quota_usages https://review.openstack.org/579093
03:06:47 pvc is tesla p100 supported vgpu?
03:14:57 openstackgerrit Tao Li proposed openstack/python-novaclient master: Remove the unused instance-name https://review.openstack.org/602520
03:22:23 openstackgerrit Tao Li proposed openstack/python-novaclient master: Remove the unused instance-name https://review.openstack.org/602520
03:26:23 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Rename changes-since test sample file https://review.openstack.org/602522
03:26:27 yikun brinzh: ^^^ as we discussed before, I split changes-since rename stuff to a separate patch.
03:27:30 yikun and I will rebase your main patch on it soon. : )
03:28:30 brinzh Ok, thank you.
03:29:32 brinzh This should be merged as quickly as possible
03:39:28 pvc hi guys
03:39:31 pvc yikun
03:39:32 pvc brinzh
03:39:56 yikun yep?
03:47:23 pvc is tesla p100 supports vgpu?
04:51:18 pvc yikun
04:51:21 pvc hello yikun
05:01:14 pvc hi
05:27:09 openstackgerrit Merged openstack/nova stable/queens: Wait for network-vif-plugged before starting live migration https://review.openstack.org/591735
05:42:00 pooja_jadhav mriedem: hello
05:45:32 brinzh pvc: this maybe need the specific driver package with the p100, or others, we alse need this ^^
05:45:58 pvc i have my centos server with p100 on it
05:46:03 pvc but i cannot see any mdev bus
05:46:25 pvc https://docs.openstack.org/nova/queens/admin/virtual-gpu.html
05:48:16 brinzh pvc: We also have been read this, and cannot find the mdev bus.
05:49:23 brinzh pvc:I don't know if I need support from NVIDIA driver.
05:49:45 pvc so how i can enable the vgpu?
05:49:51 pvc if ever i just passthrough it to my instance
05:50:28 brinzh all: If a friend has done something similar, please give guidance.^^
05:51:48 brinzh passthrouth diff with vgpu...
05:54:38 openstackgerrit Merged openstack/nova stable/rocky: libvirt: skip setting rx/tx queue sizes for not virto interfaces https://review.openstack.org/599506
05:54:49 pvc yes
05:55:04 pvc brinzh what hypervisor u are using
05:55:34 pvc im using centos7 with p100
05:56:12 brinzh I am centos7 too, and testing in Rocky version.

Earlier   Later