| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-07 | |||
| 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 ? | |
| 14:20:40 | bauzas | jianghuaw_: I think we should discuss maybe later (I'm a but rushing this afternoon tbh) about how to have multiple supported types | |
| 14:20:52 | bauzas | for the moment, let's merge only one per compute in Queens | |
| 14:21:03 | jianghuaw_ | bauzas, sure. | |
| 14:21:08 | bauzas | we also need to have flavor traits for that, remember | |
| 14:21:35 | jianghuaw_ | bauzas, yes. that's not ready for go. | |
| 14:22:27 | jianghuaw_ | bauzas, Thanks for the comment on https://review.openstack.org/#/c/516899/ | |
| 14:22:41 | jianghuaw_ | I uploaded a new ps. | |
| 14:22:52 | efried | jaypipes I don't see a way around it - I'm going to inject series https://review.openstack.org/#/c/521605/ into the main nrp stack. | |
| 14:23:06 | jaypipes | efried: k | |
| 14:25:01 | mriedem | bauzas: i'm planning on backporting | |
| 14:27:29 | bauzas | mriedem: k, ping me when you're done | |
| 14:31:26 | bauzas | jianghuaw_: wait, just saw https://review.openstack.org/#/c/516899/30/nova/tests/unit/virt/xenapi/test_driver.py@400 | |
| 14:31:59 | bauzas | snap, missed https://review.openstack.org/#/c/516217/24/nova/virt/xenapi/driver.py@457 | |
| 14:32:17 | dansmith | jaypipes: that wasn't me dude.. I appreciate the undue credit and all, but ... :P | |
| 14:33:06 | jianghuaw_ | bauzas, yes. That's because XenServer can only support one VGPU per VM. | |
| 14:33:14 | bauzas | but in libvirt, max_unit = max(vgpus) | |
| 14:33:34 | bauzas | jianghuaw_: I think there is a confusion | |
| 14:33:44 | bauzas | ah ? | |
| 14:33:51 | jianghuaw_ | I saw that difference and I asked if libvirt can support multiple VGPUs. | |
| 14:34:04 | bauzas | in theory, one can create multiple mdevs in libvirt | |
| 14:34:19 | jianghuaw_ | There is no way to create multiple VGPU in XenServer at the moment. | |
| 14:34:33 | bauzas | jianghuaw_: that said, I haven't tested it yet | |
| 14:34:51 | bauzas | multiple VGPUs per instance, right? | |
| 14:36:16 | jianghuaw_ | bauzas, seems yes:-) | |
| 14:36:36 | jianghuaw_ | I asked XenServer developer. They said that restriction is in the tool stack - XAPI so it may work for libvirt. | |
| 14:37:06 | bauzas | jianghuaw_: given it's experimental for libvirt, I probably need to test that once I'm done with implementing it | |
| 14:37:34 | bauzas | but you're fortunate, you're at least 3 people working on the xen side, compared to one on libvirt :p | |
| 14:37:45 | bauzas | one being overcommitted :p | |
| 14:38:37 | jianghuaw_ | bauzas, That's true. We had 3 people. | |
| 14:38:43 | jianghuaw_ | :-) | |
| 14:38:57 | bauzas | you *had* ? | |
| 14:40:09 | jianghuaw_ | I hope we can go further to support multiple vGPU types and traits. But it still depends on n-rp and traits works which are not ready yet. So moving some effort to other feature. | |
| 14:40:20 | jianghuaw_ | that's why I said *had* | |
| 14:42:10 | bauzas | k | |
| 14:44:02 | jianghuaw_ | bauzas, but I do wish very very much to get the patches on single vGPU can be merged soon. Particularly the long vacation is approaching:-) | |
| 14:46:22 | jianghuaw_ | bauzas, but feel free to let me know if further change needed for https://review.openstack.org/#/c/516899/ | |