Posted	Nick	Remark
#openstack-nova - 2017-12-05
11:25:40	openstackgerrit	Chris Dent proposed openstack/nova master: DNM: Test devstack fix for process restart https://review.openstack.org/525534
11:30:59	openstackgerrit	Merged openstack/nova master: Get original image_id from volume for volume-backed instance rebuild https://review.openstack.org/521391
11:31:05	openstackgerrit	Merged openstack/nova master: [placement] Add aggregate link note in API ref https://review.openstack.org/521502
11:31:12	openstackgerrit	Merged openstack/nova master: [placement] Fix foreign key constraint error https://review.openstack.org/516880
11:43:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add PowerVM to compute_driver options https://review.openstack.org/523978
11:52:10	openstackgerrit	wanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached https://review.openstack.org/525540
11:55:39	openstackgerrit	Merged openstack/nova master: Make _Provider really private https://review.openstack.org/523932
11:55:45	openstackgerrit	Merged openstack/nova master: ProviderTree.get_provider_uuids() https://review.openstack.org/520243
12:36:10	cdent	stephenfin, bauzas : are you guys able to comment whether specless is okay for this blueprint? https://blueprints.launchpad.net/nova/+spec/vmware-boot-uefi there’s already an implementation and Ken’ichi (on the review) seemed to think so.
12:37:43	openstackgerrit	Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603
12:38:34	openstackgerrit	Hengqing Hu proposed openstack/nova master: Remove build_sphinx section from setup.cfg https://review.openstack.org/525587
12:44:28	openstackgerrit	Chen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model https://review.openstack.org/525055
12:48:53	openstackgerrit	Alexandru Muresan proposed openstack/nova master: Fix driver tests on Windows https://review.openstack.org/518525
12:52:14	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
12:52:15	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
12:52:15	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
12:52:16	openstackgerrit	Jay Pipes proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643
12:52:30	jaypipes	alex_xu: ^^ added additional func test that includes resources filter along with tree filter
12:53:11	jaypipes	alex_xu: I prefer just "tree" to "in_tree", but if everyone else wants to change it, I can
12:59:38	cdent	jaypipes, alex_xu : It is somewhat confusing, but I’m not sure what’s right. There’s a part of me that thinks ‘tree’ actually ought to be a boolean: give me the results and anything else which is a relative in the trees of any results
13:00:28	openstackgerrit	Jay Pipes proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643
13:00:39	dmsimard	kashyap: ping ?
13:01:11	openstackgerrit	Merged openstack/nova master: Fix wrong argument order in functional test https://review.openstack.org/525363
13:01:23	dmsimard	Was wondering if you had come across my question yesterday: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-12-04.log.html#t2017-12-04T17:26:46
13:04:01	kashyap	dmsimard: Hey
13:04:35	kashyap	dmsimard: I did have my proxy catch it, I just didn't get around to it.
13:07:07	openstackgerrit	Matthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599
13:08:15	dmsimard	kashyap: thanks, much appreciated.
13:08:24	kashyap	dmsimard: Reading the bug report, VM building can hang w/ libguestfs & TCG
13:08:38	kashyap	dmsimard: Still catching up on the bug report & libguestfs mailing list thread linked in there
13:09:09	dmsimard	Yeah.. I summarized the issue in the eavesdrop link
13:09:56	kashyap	dmsimard: Ah-ha! So, TCG didn't fully take effect
13:14:35	kashyap	dmsimard: Related aside: Do we have any plans (at least it has been talked about since Paris Summit) to test in a portion of Infra with actual Nested KVM?
13:15:59	kashyap	dmsimard: In the past, the argument was: "It's a bit unstable / buggy". But no bugs were filed. Probably it'd be useful to think about enabling actual nVMX (Intel) / nSVM (AMD) for a portion of the Infra Cloud.
13:16:18	dmsimard	Not sure, I'd love to of course but my opinion doesn't really matter :D
13:16:38	dmsimard	It's about nodepool providers supporting it or not
13:16:40	openstackgerrit	Merged openstack/nova master: Fix some incorrect option references for scheduler filters https://review.openstack.org/521645
13:16:47	openstackgerrit	Merged openstack/nova master: Deprecate the IronicHostManager https://review.openstack.org/521648
13:16:53	openstackgerrit	Merged openstack/nova master: Fix invalid minRam error message https://review.openstack.org/523142
13:17:01	openstackgerrit	Merged openstack/nova master: Add description for resource class creation https://review.openstack.org/508083
13:18:15	kashyap	dmsimard: Yeah, I know it's not in your hands; it's with node providers; Rackspace Infra uses Xen, so that's out of scope.
13:19:26	openstackgerrit	Matthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr https://review.openstack.org/524167
13:22:02	dmsimard	kashyap: need to brb
13:22:25	kashyap	dmsimard: It's fine; I too need to be briefly AFK to finish some paperwork
13:22:32	kashyap	(Not now, in a few)
13:32:28	openstackgerrit	Merged openstack/nova master: Add debug information to metadata requests https://review.openstack.org/511895
13:32:39	openstackgerrit	Merged openstack/nova master: [placement]Enhance doc for placement allocation list https://review.openstack.org/511715
13:32:56	openstackgerrit	Merged openstack/nova master: Fix TypeError in nova-manage db archive_deleted_rows https://review.openstack.org/520765
13:38:24	alex_xu	cdent: so "tree=True&resources=...." means that give me the tree which match those resources?
13:38:51	stephenfin	cdent: I think it best to ask mriedem that
13:39:54	alex_xu	jaypipes: thanks
13:39:56	cdent	alex_xu: sort of: for the resource providers that satisfy the resources request, return those resource providers and their trees
13:40:25	jaypipes	cdent: what do you mean by "and their trees"?
13:41:00	cdent	jaypipes: if rp X has a parent of Y and a sibling of Z, all of X Y and Z would return
13:41:10	cdent	(where X satisfies resources)
13:41:35	jaypipes	cdent: the problem with that is due to GET /resource_providers not returning information about inventory or allocations, you would have no idea *which* of the resource providers met the resources query.
13:42:17	jaypipes	cdent: for GET /allocation_candidates, we will return the provider information (in the provider summaries section) for all providers in the tree of resource providers included in allocation requests.
13:42:34	cdent	yeah, I know. It’s how tree “feels” as a param though
13:42:54	cdent	the meaning you’ve made is “limit to tree”, yes?
13:42:59	jaypipes	cdent: do you prefer "in_tree" then? or something else entirely?
13:43:21	cdent	I can’t decide. I’m trying to talk out loud to see if anything reasonable leaks out
13:43:26	jaypipes	cdent: the meaning of "tree=X" is "get the root provider UUID of X and return all providers in that tree"
13:43:56	jaypipes	cdent: so yeah, "limit to this provider's tree" is what the filter says.
13:44:16	jaypipes	cdent: just want to be clear that "X" doesn't need to be the root provider UUID.
13:44:34	jaypipes	cdent: it can be any old resource provider UUID. we look up that provider's root UUID.
13:44:54	cdent	and if some other parameter (like resources) is present, and X isn’t in the resource satisfyng rps, no resource, right?
13:45:05	cdent	s/no resource/no results/
13:45:05	jaypipes	correct
13:45:09	efried_cya_wed	I was thinking ?tree=X&resources=Y would mean, "find me only the providers from within tree X that have resources Y"
13:45:19	cdent	efried_cya_wed: it is not wed, go away
13:45:23	efried_cya_wed	I.e. explicitly *not* the whole tree.
13:45:24	jaypipes	efried_cya_wed: that is precisely what it means.
13:45:46	jaypipes	efried_cya_wed: filters are "ANDed" together...
13:46:12	cdent	I reckon in_tree is better
13:46:15	cdent	but not hugely so
13:46:54	alex_xu	cdent: X isn't in the resource satisfying rps, there may have result, for the case, the other rps match the resource in the tree
13:48:10	jaypipes	alex_xu: yes, that's true. if X is a grandchild and Y is a child, and Y has all the resources needed, then Y would be returned, yes.
13:48:45	alex_xu	jaypipes: yea
13:50:00	jaypipes	alex_xu: we could call the filter 木 :)
13:50:12	cdent	so a) in_tree is beginning to sound better to me, b) what’s the use case? when does a client want to do this?
13:51:31	jaypipes	cdent: this is primarily going to be called by the scheduler report client's get_providers_in_tree() method which will populate a ProviderTree structure that is passed to the virt driver to populate
13:51:45	alex_xu	jaypipes: you mean tree? it should be 树，木 is wood :)
13:51:59	cdent	jaypipes: so in that case only the tree param is used, yes?
13:52:12	jaypipes	alex_xu: crap! there's like 15 symbols that are "tree" in Google translate ;)
13:52:18	alex_xu	haha
13:53:15	jaypipes	cdent: yeah, in that case, only the tree filter is used
13:53:40	alex_xu	the chinese version 'GET /资源_提供者?树=...&资源=...'
13:59:23	alex_xu	jaypipes: I guess cdent is asking the use case of tree+resources
13:59:53	cdent	not really. I agree that if we have filters, they should all be allowed and all should be and-ed
13:59:59	cdent	I also agree that some combinations are weird
14:00:11	cdent	but as long as we are and-ing correctly it is okay
14:00:28	cdent	I don’t want us to be saying that some filter combinations are disallowed
14:00:54	efried_cya_wed	++
14:01:21	efried_cya_wed	Realistically, there are combinations consumers won't use because they don't make any sense. And that should be fine.
14:02:00	alex_xu	cdent: +1
14:17:03	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: [placement] Fix foreign key constraint error https://review.openstack.org/525620
14:20:48	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
14:20:49	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
14:20:49	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Add a new check to volume attach https://review.openstack.org/525622
14:21:54	ildikov	mriedem: first attempt ^^

Earlier   Later