| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-07 | |||
| 10:14:32 | bauzas | I'm thinking of a problem about allocations with multiple possible types | |
| 10:15:03 | bauzas | for the moment, we're sure that the compute node only support one type, so when creating the VGPU (ie. the mdev for libvirt) I can ask for that type | |
| 10:15:14 | bauzas | now, say we have more than one type | |
| 10:15:32 | bauzas | we should then try to understand which type is related to the allocation | |
| 10:32:29 | sean-k-mooney2 | stephenfin: wait what we should never load the glance image into ram in one go. where? | |
| 10:32:59 | stephenfin | sean-k-mooney2: The 'download' function modified here https://review.openstack.org/526328 | |
| 10:35:02 | sean-k-mooney2 | on line 318? | |
| 10:36:44 | stephenfin | Actually, higher https://review.openstack.org/#/c/526328/1/nova/image/glance.py@179 | |
| 10:39:32 | sean-k-mooney2 | stephenfin: i was just going to say i really hope self._client.call(context, 2, 'data', image_id) is retruning an asyc stream to the data form the glance api but that call to list is painful | |
| 10:42:29 | sean-k-mooney2 | stephenfin: i take it we dont have an fuctional test that i dont know validate a 1TB qcow image of all 0 bytes do we because we shoudl remove if on line 176 and add one | |
| 10:43:27 | tetsuro | Hi sean, sorry for cutting in. Jay is asking to you about this patch, so could you response? https://review.openstack.org/#/c/465160/ , | |
| 10:43:42 | mdbooth | stephenfin: https://review.openstack.org/#/c/526328/1 | |
| 10:43:47 | sean-k-mooney2 | tetsuro: sure ill take a look now | |
| 10:44:10 | tetsuro | thanks in advance | |
| 10:44:18 | stephenfin | mdbooth: Fair. I'm reporting that as a bug now | |
| 10:52:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle images with no data https://review.openstack.org/526329 | |
| 10:54:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle images with no data https://review.openstack.org/526329 | |
| 11:15:55 | sean-k-mooney2 | bauzas: if your device support multiple mdev types you need to have multiple resouce provider pools. | |
| 11:16:28 | sean-k-mooney2 | bauzas: the tricky bit is doing the accounting correctly | |
| 11:17:34 | sean-k-mooney2 | when you claim the mdev you will have to revailidate the available_instances for all other mdev types on the same Physical device | |
| 11:24:29 | openstackgerrit | Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603 | |
| 11:24:30 | openstackgerrit | Matthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr https://review.openstack.org/524167 | |
| 11:24:30 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599 | |
| 11:24:31 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | |
| 11:24:31 | openstackgerrit | Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346 | |
| 11:28:31 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance https://review.openstack.org/526348 | |
| 12:04:47 | openstackgerrit | Chris Dent proposed openstack/nova stable/pike: [placement] re-use existing conf with auth token middleware https://review.openstack.org/526351 | |
| 12:10:18 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | |
| 12:13:07 | openstackgerrit | Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395 | |
| 12:33:08 | openstackgerrit | garyk proposed openstack/nova master: Conductor: ensure build cleanup works with cell https://review.openstack.org/526356 | |
| 12:36:04 | jaypipes | sean-k-mooney2: I want that original pug. | |
| 12:37:37 | sean-k-mooney2 | jaypipes: did you see my latest responce | |
| 12:37:55 | jaypipes | sean-k-mooney2: ya, reading it now :) | |
| 12:40:56 | sean-k-mooney2 | jaypipes: for rodolfos path by the way im happy to enable the fuctional gate jobs but do you know if infra allow enableing new legacy jobs or do i need to swap to zuul v3 in repo job config first? | |
| 12:41:29 | jaypipes | sean-k-mooney2: no idea... we should ask clarkb or fungi | |
| 12:42:19 | sean-k-mooney2 | jaypipes: i have been meaning to get linuxbridge gate jobs running of osvif for a while too so ill reach out to them and see how much work the v3 migration would be | |
| 12:42:33 | jaypipes | sean-k-mooney2: sounds good, thanks man | |
| 12:45:56 | sean-k-mooney2 | for tetsuro path since the intel nfv ci is still offline(ment to be fixed in january) we do not have any gate jobs for any cpu pinning, hugepage, emulator pin or numa feautres bar unit test which kind of sucks right now. | |
| 12:46:07 | sean-k-mooney2 | s/path/patch | |
| 12:50:49 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | |
| 12:50:50 | openstack | bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 12:56:47 | openstackgerrit | garyk proposed openstack/nova master: Conductor: ensure build cleanup works with cell https://review.openstack.org/526356 | |
| 12:59:17 | jaypipes | sean-k-mooney2: right, and I'm not really willing to approve any but the easiest/simplest patches that touch NUMA/CPU pinning/huge pages until that CI is green. | |
| 13:01:13 | efried | alex_xu We took a vote and settled on in_tree=<uuid> | |
| 13:02:04 | sean-k-mooney2 | jaypipes: the reason i created that patch originally was to allow hugepages, numa topology and emulator pinning to be testing in the upstream gate but the current version does not enable that | |
| 13:04:38 | jaypipes | efried: mornin. | |
| 13:04:44 | efried | jaypipes Howdy! | |
| 13:05:09 | jaypipes | efried: that's right... I forgot you're in Texas. :) | |
| 13:05:18 | jaypipes | pistols at dawn and all that. :P | |
| 13:06:12 | jaypipes | efried: though frankly, Florida might have just as many guns as Texas. | |
| 13:06:36 | efried | I haven't taken my pistol out of the safe in months. You, on the other hand, are living the Florida stereotype to a "tee". | |
| 13:06:37 | jaypipes | efried: my favorite thing about Sarasota is the ammo/gun store directly across from the courthouse called "The Bullet Hole" | |
| 13:06:46 | efried | Heh | |
| 13:06:48 | jaypipes | efried: touche | |
| 13:07:38 | jaypipes | efried: dansmith introduced me to https://www.reddit.com/r/FloridaMan/ about a year ago, much to my delight. | |
| 13:08:55 | efried | wow | |
| 13:11:34 | jaypipes | efried: you can thank dansmith for that :) there goes an hour of work time. | |
| 13:11:48 | efried | jaypipes It's better than The Onion. | |
| 13:12:20 | jaypipes | efried: one of my favorites from this past year: https://nypost.com/2017/05/11/man-drowns-in-freak-car-crash-with-fire-hydrant/ | |
| 13:12:42 | efried | Hah, it's like that scene from one of the old Superman movies. | |
| 13:12:54 | jaypipes | yep. only "Florida style" | |
| 13:14:50 | jaypipes | efried: ok, so back to work stuff... how we doing on the n-r-p series. I see only the bottom patch merged. | |
| 13:15:01 | efried | Yup | |
| 13:15:20 | efried | I rebased (and slightly reworked) the pile of patches on top of that. | |
| 13:15:33 | efried | And also resurrected (and corrected) this series: https://review.openstack.org/#/c/521605/ | |
| 13:16:06 | efried | ...which puts caching of aggs & traits into ProviderTree and out of the report client. | |
| 13:16:11 | jaypipes | ack | |
| 13:16:29 | jaypipes | OK, let me review that as soon as I comment on sean-k-mooney2's virt_type patch | |
| 13:16:30 | efried | Not sure if I should fold those guys back into the main series or not. We're going to need them all before those top three WIPs will work. | |
| 13:16:38 | jaypipes | right | |
| 13:16:42 | efried | But if we can get 'em merged fairly quick... | |
| 13:18:09 | efried | jaypipes I'm working today and first half of tomorrow, then out til Jan. My goal is to have at least some of those WIPs un-WIP'd and close enough for someone (cdent volunteered) to shepherd them while I'm gone. | |
| 13:18:28 | jaypipes | efried: ack, that would be great. | |
| 13:19:03 | efried | jaypipes Though realistically, I'm not sure how much we can expect to merge the last two weeks of December. Do lots of people disappear? | |
| 13:19:21 | jaypipes | efried: yes | |
| 13:19:25 | efried | k | |
| 13:51:07 | openstackgerrit | Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799 | |
| 13:59:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/523194 | |
| 13:59:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027 | |
| 14:00:10 | mriedem | alex_xu: removed that file finally, thanks for pointing that out again ^ | |
| 14:01:14 | mriedem | sdague: can we get https://review.openstack.org/523194 in so i can get it backported to newton before eol | |
| 14:07:46 | sdague | mriedem: +2 from me | |
| 14:10:31 | alex_xu | mriedem: np, +w | |
| 14:10:41 | mriedem | thanks | |
| 14:12:26 | mriedem | the functional py35 logs are unreadable due to deprecation warnigns | |
| 14:12:28 | mriedem | *warnings | |
| 14:12:32 | mriedem | from oslo.context and oslo.policy | |
| 14:13:14 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | |
| 14:13:35 | jianghuaw_ | bauzas, I think we can only enable one type for each RP (pgpu for libvirt; gpu group for xenapi). Then we can mapping the allocation to the single enabled type in that resource provider. | |
| 14:13:37 | fungi | jaypipes: wow, you almost make north carolina sound tame by comparison ;) | |
| 14:13:59 | fungi | (though in full disclosure, i was born in florida so can't really judge) | |
| 14:14:07 | jaypipes | fungi: :) | |
| 14:15:06 | mriedem | bauzas: https://review.openstack.org/#/c/508262/ - i think we should get that in and backported to avoid the tons of excessive logging about what state the computes are in when we're doing anything in the resource tracker | |
| 14:15:25 | jianghuaw_ | bauzas, it should work if we can mapping back from RP uuid to the PGPU/gpu group. | |
| 14:15:36 | mriedem | when i compared logs, it went from logging nearly 400 times to 35 | |
| 14:16:07 | jianghuaw_ | bauzas, for XenAPI we plan to use the gpu group's uuid as the VGPU RP's uuid. That should work. | |
| 14:19:22 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556 | |
| 14:19:30 | jianghuaw_ | bauzas, see this patch: https://review.openstack.org/#/c/521717 | |
| 14:19:38 | bauzas | mriedem: agreed | |
| 14:20:03 | bauzas | mriedem: planning to backport ? or want me to do such thing ? | |