| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-05 | |||
| 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: update client to set parent provider https://review.openstack.org/385693 | |
| 12:52:15 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 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 | jaypipes | correct | |
| 13:45:05 | cdent | s/no resource/no results/ | |
| 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: Add a new check to volume attach https://review.openstack.org/525622 | |
| 14:20:49 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | |
| 14:21:54 | ildikov | mriedem: first attempt ^^ | |
| 14:22:36 | mriedem | ack | |
| 14:22:51 | ildikov | mriedem: I might've bumped the service version too early... :/ | |
| 14:23:53 | mriedem | yup | |
| 14:23:55 | mriedem | https://review.openstack.org/#/c/525622/1/nova/objects/service.py shouldn't be in there | |