Posted	Nick	Remark
#openstack-nova - 2017-10-26
08:16:09	openstackgerrit	jichenjc proposed openstack/nova master: Add description for resource class creation https://review.openstack.org/508083
08:20:45	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
08:22:18	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
08:24:13	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
08:24:59	bauzas	naichuans: jianghuaw: could you please upload new revisions in a series by rebasing on the existing changes?
08:25:55	bauzas	naichuans: jianghuaw: https://review.openstack.org/#/c/512580/ is no longer dependent on https://review.openstack.org/#/c/514942/
08:26:28	bauzas	naichuans: jianghuaw: I'll rebase those
08:28:31	bauzas	jianghuaw: thanks for your comment https://review.openstack.org/#/c/514763/ you're right
08:29:40	jianghuaw	bauzas, u r welcome. Have you rebased the patch set. Otherwise Naichuan can do it now.
08:33:41	kashyap	bauzas: stephenfin: Hi there, what is the way to restore a specification?
08:35:26	stephenfin	kashyap: In Gerrit? If it's not your spec, you need a core to do it I think? bauzas can
08:35:37	kashyap	stephenfin: Yeah, it's not my specc.
08:35:59	bauzas	stephenfin: kashyap: which spec?
08:36:04	kashyap	Context is, I was talking to KVM / QEMU developers at KVM Forum here. Incidentally Markus Zoeller was also here
08:36:12	kashyap	And I'd like to work on this spec -- https://review.openstack.org/#/c/230968/
08:36:15	kashyap	bauzas: ^
08:36:23	kashyap	"libvirt: iothreads for disk devices"
08:36:57	bauzas	kashyap: you know it'll be a rocky spec, right?
08:37:03	kashyap	It's a future thing: since it's a performance feature, need to backup any claims in the spec with a couple of real benchmarks.
08:37:08	kashyap	bauzas: Yep, sure.
08:37:19	bauzas	kashyap: okay, so restoring
08:37:23	kashyap	bauzas: No urgency right now. Just want to chug along at it, as I get time
08:37:26	kashyap	Thank you
08:37:53	bauzas	restored
08:38:35	kashyap	bauzas: Merci beaucoup. (/me heads out for a break)
08:49:45	openstackgerrit	Naichuan Sun proposed openstack/nova master: VGPU_support: add enabled white list https://review.openstack.org/512580
08:50:30	jianghuaw	bauzas, ^
08:53:40	bauzas	jianghuaw: ta
08:55:30	jianghuaw	bauzas, how to get the allocation for vgpu in driver?
08:58:35	bauzas	jianghuaw: I'm working on your comment
08:58:52	bauzas	basically, I should just add availableInstances + mdevs
08:59:14	jianghuaw	yeah, it should work.
09:00:12	jianghuaw	bauzas, I'm wondering how to get the allocation in driver at spawn.
09:01:01	bauzas	jianghuaw: the allocations are not passed by the parameter in spawn
09:01:04	bauzas	now*
09:01:09	bauzas	s/not/now
09:02:11	jianghuaw	bauzas, where? I didn't find it.
09:03:11	jianghuaw	ah, yes. I see it.
09:03:42	jianghuaw	that's great. I was in a old branch.
09:14:52	openstackgerrit	jichenjc proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219
09:16:57	openstackgerrit	jichenjc proposed openstack/nova stable/pike: propagate OSError to MigrationPreCheckError https://review.openstack.org/515330
09:17:45	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: block_device_mapping_v2.bus_type is missing from api-ref https://review.openstack.org/499488
09:18:26	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190
09:20:54	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138
09:20:55	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
09:20:56	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
09:20:56	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
09:20:58	jaypipes	gibi: ^
09:24:56	sahid	bauzas: you don't want to deal with availableInstances
09:25:48	sahid	that is informining of the number of a specific mdev type can be allocated by operators
09:26:24	sahid	in this implmentation you are not going to allocate the mdev devices
09:26:31	sahid	the mdev devices are going to be pre-allocated
09:26:44	sahid	what you want is to list the allocated mdev devices
09:27:07	sahid	calling libvirt with list_devices('mdevs')
09:28:38	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff https://review.openstack.org/515149
09:28:39	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
09:32:42	stephenfin	jaypipes, bauzas: Trivial "move nova-network stuff around" patch here, if you want it? https://review.openstack.org/#/c/499168/
09:32:57	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Move additional nova-net opts to 'network' https://review.openstack.org/499168
09:33:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Move additional nova-net opts to 'network' https://review.openstack.org/499168
09:35:34	openstackgerrit	jichenjc proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219
09:38:11	gibi	jaypipes: thanks, the server side patches look good to me. Now reading the report client patch
09:40:43	jaypipes	gibi: cheers
09:41:20	gibi	jaypipes: Do I understand correctly that you will change the versioning still in the report client, based on your answer in https://review.openstack.org/#/c/385693/54/nova/scheduler/client/report.py ?
09:41:27	bauzas	jaypipes: I have a design question for you
09:42:05	bauzas	jaypipes: do you think it's acceptable to have an inventory of some resource class that is based on the sum of "left available resources" + "existing resources" ?
09:42:16	jaypipes	gibi: see latest patch on that. yeah, I set it to always pass 1.11 and bomb if server doesn't support that.
09:42:20	bauzas	jaypipes: speaking of the total value of course
09:42:45	jaypipes	bauzas: no
09:42:59	gibi	jaypipes: so the always doesn't mean that it is set in the __init__ of the client
09:43:00	bauzas	jaypipes: what makes me a sad panda is that I misread some documentation in https://libvirt.org/drvnodedev.html#MDEVCap
09:43:30	jaypipes	gibi: no, just that that method always passes 1.11
09:43:42	jaypipes	gibi: because it expects to operate on the n-r-p attrs
09:43:52	bauzas	jaypipes: I thought "availableInstances" was the total capacity of a specific device for creating vGPUs, but that's actually just an integer which decrements every time you create a mediated device
09:44:16	jaypipes	bauzas: that is available i.e. total - used.
09:44:28	bauzas	jaypipes: yeah, my point
09:44:37	bauzas	jaypipes: so I can't use it for reporting the inventory
09:44:42	jaypipes	bauzas: it's a pointless attribute if you already have total and used counts.
09:44:59	bauzas	libvirt doesn't give me the API for the total number
09:45:10	jaypipes	bauzas: oh well, that's a flaw in libvirt.
09:45:38	bauzas	the "still" word in "how many instances of the given type can still be successfully created on the physical device" is just pissing me off
09:46:07	openstackgerrit	Michael Still proposed openstack/nova master: Convert IVS VIF plugging / unplugging to privsep. https://review.openstack.org/515196
09:46:07	openstackgerrit	Michael Still proposed openstack/nova master: Move infiniband vif plugging to privsep. https://review.openstack.org/515197
09:46:08	openstackgerrit	Michael Still proposed openstack/nova master: Move midonet vif plugging to privsep. https://review.openstack.org/515198
09:46:08	openstackgerrit	Michael Still proposed openstack/nova master: Move plumgrid vif plugging to privsep. https://review.openstack.org/515336
09:46:45	bauzas	jaypipes: so seriously you would nack some reporting that would sum up that value with the existing already created mediated devices?
09:47:19	bauzas	I know, it's something racy racy
09:47:22	jaypipes	bauzas: no, I would be fine if you wanted to calculate total from used and available.
09:47:34	bauzas	ah ok, you make my day then
09:47:43	jaypipes	bauzas: just saying don't pass available as total :)
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

Earlier   Later