Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-26
09:47:53 bauzas jaypipes: of course
09:47:58 bauzas sorry if I was unclear
09:48:04 jaypipes since available is a calculated value
09:48:15 bauzas I just spotted that as being a "left", not a "total"
09:48:17 jaypipes bauzas: yup, that's totally cool with me (pun intended)
09:48:24 bauzas English subtilities FTW
09:48:40 jaypipes stephenfin: done
09:48:54 jaypipes bauzas: or even subtleties :)
09:49:05 stephenfin jaypipes: Ta :)
09:49:27 bauzas jaypipes: man, I should seriously consider being the Master of the World
09:49:38 bauzas so, languages wouldn't be a problem
09:49:50 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: properly decode error message from qemu guest agent https://review.openstack.org/511459
09:49:56 jaypipes bauzas: :)
09:50:11 gibi jaypipes: so _create_resource_provider() will always return an RP that contains the parent_provider_uuid but the _get_resource_provider only does the same is the placement latest version is bigger than 1.11.
09:50:17 jaypipes stephenfin: I suppose that's better than "sub titties".
09:50:34 sahid bauzas: in you implmentation, you want to also allocate the mdev?
09:50:41 bauzas sahid: nope
09:50:51 bauzas sahid: not in that change
09:51:08 gibi jaypipes: s/is/if/ , s/bigger/bigger or equal/
09:51:08 sahid so why you don't len(allocated_mdev) ?
09:51:25 sahid to get the total?
09:51:30 sahid allocated does not mean assigned to a guest
09:51:50 bauzas sahid: in that change, I'll focus on reporting the total number of possible mediated devices, which will be the sum up of what availableInstances reports + the sum of the mediated devices that belong to the same physical device
09:51:52 gibi jaypipes: do we assume that the user of the report clean alway tries creating thing first and bails out if placement is older than 1.11 ?
09:52:11 jaypipes gibi: yeah, I need to change that to have get_resource_provider() send 1.11 too
09:52:14 bauzas sahid: oh my bad, a misunderstanding
09:52:14 gibi jaypipes: s/clean/client/
09:52:39 bauzas sahid: yeah, I want to get the total number of possible mediated devices to be created, including the allocated mdevs
09:52:53 gibi jaypipes: OK, then it would be easier to set a minimum version for the whole report client
09:52:58 bauzas sahid: the fact they are assigned to the guest is unrelated with that change
09:53:08 jaypipes gibi: ack, as edleafe said.
09:53:40 sahid bauzas: so you just have to ask libvirt to get list the mdev devices, filter them by mdev_type
09:53:48 sahid and use len() to get the total
09:54:25 bauzas sahid: yup
09:54:30 gibi jaypipes: will you do that in the current patch or as a followup? (/me triest to figure out what to vote on the current patch)
09:54:33 bauzas for the allocated part
09:54:34 sahid bauzas: what you are trying to do with availableinstance does not work
09:54:48 bauzas available will give me the left amount
09:54:57 sahid off course not
09:55:06 jaypipes gibi: doing it now...
09:55:15 sahid since when operator will allocate all the mdev, that value will be 0
09:55:21 bauzas what I need to report from the inventory is the total amount of VGPUs a specific resource provider can create
09:55:23 gibi jaypipes: colle, then I will check back later
09:55:27 sahid you do not need of it
09:55:34 gibi jaypipes: s/colle/cool/
09:55:45 bauzas sahid: ah, I see your point
09:55:57 bauzas sahid: you were expecting the mdev to be manually created by the operator ?
09:56:16 sahid bauzas: well that one was question at the beginning
09:56:17 bauzas sahid: I was rather thinking of libvirt doing the syscall
09:56:39 sahid bauzas: libvirt does not provide any api to allocate mdev
09:56:50 bauzas sahid: we could make the sys call
09:56:58 bauzas not by libvirt of course
09:57:00 bauzas in the virt driver
09:57:21 sahid oh ok so you want to also manage the allocation of mdev
09:57:26 bauzas that's right
09:57:43 bauzas because mdevs don't support reboots
09:57:49 sahid basically that was my first question: 11:50 < sahid> bauzas: in you implmentation, you want to also allocate the mdev?
09:57:58 sahid but it's better like that for sure
09:58:00 sahid +1
09:58:17 bauzas sahid: sorry about the confusion, the 'allocation' word is heavily used in my mind :p
09:58:58 bauzas tbc, the workflow I see is that when you ask for an instance, the virt driver will lookup the allocations and see there is one for VGPUs
09:59:22 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
09:59:24 jaypipes gibi: done, sir ^
09:59:30 bauzas then, it will call the system to create the mdev, and then create the XML with the according UUID
09:59:46 gibi jaypipes: lookgin
10:00:14 bauzas at init (when we reboot), we could also lookup the instances and see if they have mdevs in their domain XML, so we could recreate it on the fly
10:01:00 sahid bauzas: well the uuid will be different
10:01:29 bauzas sahid: you can create a mdev by passing a specific UUID right?
10:01:43 sahid bauzas: yes you right, my mistake
10:02:05 bauzas anyway, I need to fix a problem with my testing machine
10:02:17 bauzas can't see the created mdevs for some reason
10:02:22 bauzas very frustrating
10:10:36 gibi jaypipes: thanks, I'm +2
10:11:36 openstackgerrit Radoslav Gerganov proposed openstack/nova master: VMware: implement get_inventory() driver method https://review.openstack.org/506175
10:28:42 efried jaypipes You're up early
10:36:42 efried dansmith FYI, this is what we were discussing yesterday: https://review.openstack.org/#/c/515223/
10:39:20 efried alex_xu yt?
10:39:43 openstackgerrit Merged openstack/nova master: Remove usage of kwarg retry_on_request in API https://review.openstack.org/501073
11:14:25 openstackgerrit Chris Dent proposed openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798
11:24:23 openstackgerrit Yikun Jiang proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
11:36:48 openstackgerrit Yikun Jiang proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
12:02:02 avolkov jaypipes: hi, if not allowing to delete RP with associated traits is error or feature?
12:02:05 efried cdent FWIW, my IDE complains when I use set([]) instead of {} for set literals. I agree it's tougher to read (though it's fewer characters, which is all-important in python :) Maybe I'll switch that IDE warning off.
12:02:39 cdent yeah about that “fewer characters” thing,wait for my review on the next in the seriues
12:03:07 Dinesh_Bhor jaypipes, bauzas: Hi, could you please give your opinion on this: https://review.openstack.org/#/c/512990/ when you get time.
12:03:15 cdent efried: in that particular case since it is a doc string you can do whatever you like
12:03:18 jmccarthy There seems to be an issue with console logging (in horizon) on xen compute, looking at the instance xml - I don't find any of the path .. console.log type definitions (as seen on other computes) ? Looking at https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py for clues ..
12:03:44 efried cdent Roger wilco.
12:05:19 alex_xu efried: hi, i'm here
12:05:47 jmccarthy Any suggestions ? Editing the xml by hand works to a point (to try and find out what actually works), but changes it doesn't like don't seem to be applied
12:05:53 efried alex_xu Hey, so I was thinking perhaps you and I should try to do some collaboration on these placement changes, edleafe style.
12:07:52 alex_xu efried: yea, that's good idea
12:08:02 efried alex_xu E.g. I'd like to feel at liberty to write tests and make tweaks, but sometimes it makes a lot more sense to do it in the same change set rather than trying to do separate ones with commits based on each other and whatnot.
12:09:13 efried alex_xu Here's the edleafe reference: https://blog.leafe.com/pair-development/
12:09:47 alex_xu give me 10 mins to ready edleafe's blog :)
12:10:25 alex_xu s/ready/read/
12:10:46 jaypipes avolkov: the user should be able to delete a provider unless that provider has allocations against it. nothing to do with traits.
12:10:53 jaypipes avolkov: does that answer your question?
12:11:19 efried Dinesh_Bhor FYI, spec freeze has passed - is this proposed for Rocky?
12:12:12 avolkov jaypipes: Yes, I file a bug then

Earlier   Later