| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-07 | |||
| 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/ | |
| 14:46:33 | jianghuaw_ | I'm happy to rework it. | |
| 15:03:47 | mriedem | gibi: ildikov: replied in https://review.openstack.org/#/c/525787/ | |
| 15:03:53 | mriedem | gibi: thanks for the thorough review as always | |
| 15:04:20 | ildikov | mriedem: Tnx, will check in a bit | |
| 15:04:26 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/526426 | |
| 15:06:16 | mriedem | sdague: dansmith: bauzas: can we get these patches in for the recent cve fix (pike) and then i'll start prepping for a release https://review.openstack.org/#/q/topic:alternate/1664931+status:open | |
| 15:06:35 | dansmith | ya | |
| 15:07:22 | dansmith | mriedem: what is the pike only workaround? that hash in the commit message links to this | |
| 15:07:31 | mriedem | lyarwood: there are several patches with +2s on pike, can you go through those? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike | |
| 15:07:48 | dansmith | oh wait | |
| 15:07:50 | mriedem | dansmith: this https://review.openstack.org/#/c/523213/4/nova/tests/functional/test_servers.py | |
| 15:07:52 | mriedem | the setUp | |
| 15:08:10 | mriedem | i should have said, "in the functional test" or something | |
| 15:08:10 | dansmith | yeah, okay I was expecting something functional | |
| 15:08:12 | dansmith | yeah | |