| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-12 | |||
| 14:48:17 | dansmith | apparently kills you faster than cigarettes, which is pretty impressive | |
| 14:48:30 | donnyd | mriedem: the juul virginia tobacco is pretty good | |
| 14:48:54 | mriedem | i did hear a pretty funny quote from trump on his ban on flavored vaping, | |
| 14:49:10 | mriedem | something like, "the kids, they're coming home, and they're saying to their mom, 'hey, i want to vape!', and it's bad" | |
| 14:49:18 | mriedem | like, what kid tells their parents they want to smoke? | |
| 14:49:32 | mriedem | duh | |
| 14:49:32 | mriedem | they just do it and then tell their parents they were at their friend's house, the friend whose parents smoke | |
| 14:50:40 | stephenfin | I see you've done this before | |
| 14:50:59 | mriedem | i was at jared's house if my wife asks | |
| 14:53:33 | dansmith | I like how mriedem has to fly under his wife's radar just like he had to with his parents | |
| 14:54:40 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] numa + pcpus in placment live migration tests https://review.opendev.org/681771 | |
| 14:54:55 | sean-k-mooney | donnyd: is FN currenly active? | |
| 14:55:02 | donnyd | no | |
| 14:55:11 | sean-k-mooney | ok then ^ will fail | |
| 14:55:11 | donnyd | https://review.opendev.org/#/c/681731/ | |
| 14:55:26 | sean-k-mooney | that is the tweek to the multi numa job | |
| 14:55:36 | sean-k-mooney | ill start on the vexhost single numa version now | |
| 15:02:15 | bauzas | mriedem: ack, reviewing those now | |
| 15:02:20 | dansmith | sean-k-mooney: thanks | |
| 15:02:42 | donnyd | sean-k-mooney: https://review.opendev.org/#/c/681773/ | |
| 15:07:09 | stephenfin | alex_xu: If you're still about, could you look at https://review.opendev.org/#/c/681750/ ? | |
| 15:09:33 | luyao | stephenfin: he's away, but will be back some time(he said, but not sure) | |
| 15:10:38 | aspiers | kashyap: is the reference to vexpress-a15 on https://docs.openstack.org/glance/latest/admin/useful-image-properties.html really correct? | |
| 15:11:13 | aspiers | feels like that doc is at best incomplete | |
| 15:20:18 | bauzas | gibi: +Wd with comment https://review.opendev.org/#/c/676980/22/nova/compute/manager.py@4439 | |
| 15:20:36 | bauzas | gibi: I'm not opposed to using a deprecated method as it's an easy path | |
| 15:21:10 | bauzas | gibi: but I feel this old API is broken and we should bump the version very soon | |
| 15:21:59 | gibi | bauzas: the comment about the usage explains that it is there until RPC version 6.0 | |
| 15:22:08 | bauzas | I know | |
| 15:22:27 | gibi | bauzas: but until that, I _have to_ use the deprecated conversion methos | |
| 15:22:28 | gibi | d | |
| 15:22:30 | bauzas | I'm just saying this RPC bump is quite important | |
| 15:22:33 | gibi | bauzas: I agree we shoudl make a bump | |
| 15:22:39 | bauzas | gibi: yeah | |
| 15:23:02 | bauzas | gibi: you could technically use another helper method | |
| 15:23:17 | bauzas | but this one is convenient, I don't disagree | |
| 15:25:56 | gibi | bauzas: ack, I made a TODO for myself about RPC bump for Ussuri but I'm sure I will need help doing that properly | |
| 15:29:09 | dansmith | gibi: we have a doc about how to do it actually | |
| 15:29:10 | dansmith | it's been a while | |
| 15:30:03 | dansmith | gibi: https://wiki.openstack.org/wiki/RpcMajorVersionUpdates | |
| 15:31:11 | dansmith | not sure we've done it since we've had auto version pinning so we'd need to think through that a bit | |
| 15:31:13 | bauzas | gibi: yeah follow dansmith's point, I made it for the scheduler and I can give you a ton of code snippets | |
| 15:31:26 | bauzas | dansmith: oh good point | |
| 15:31:37 | bauzas | it's been a while since I've seen a major bump | |
| 15:31:56 | bauzas | (but it's been a while since I followed changes like I was before) | |
| 15:32:10 | gibi | dansmith: thanks. I've annotated my TODO with the wiki link | |
| 15:33:50 | sean-k-mooney | whats the the synatax for resouce request again "resouces:PCPU=2" | |
| 15:34:39 | sean-k-mooney | that is correct yes? | |
| 15:34:50 | sean-k-mooney | in the flavor | |
| 15:35:11 | dansmith | sean-k-mooney: with proper spelling, I think so | |
| 15:35:25 | sean-k-mooney | ill copy it form functional test to get it right | |
| 15:35:27 | bauzas | sean-k-mooney: yeah, if you don't want more | |
| 15:36:10 | gibi | mriedem: thanks for fixing the race and fixing up the series and the reviews! | |
| 15:36:45 | bauzas | sean-k-mooney: example for VGPU : https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#configure-a-flavor-controller | |
| 15:36:58 | sean-k-mooney | ya i found an example | |
| 15:37:05 | mriedem | gibi: np | |
| 15:37:07 | bauzas | FWIW, pidgin makes it a joke | |
| 15:37:19 | bauzas | or a smiley rather | |
| 15:38:43 | mriedem | dansmith: i think you did the last compute rpc api bump to 5.0 in queens | |
| 15:39:00 | mriedem | which i'm pretty sure was after the auto stuff | |
| 15:39:15 | dansmith | yeah I was looking.. I think so too | |
| 15:40:21 | mriedem | sean-k-mooney: i said this the other day, but i'd think you'd want a flavor with PCPU=1 (i guess if you test resize you need an alternate flavor, but that could also be PCPU=1 and a different ram/disk) and you'll want to tell tempest to run the tests in serial or at least --concurrency=2 | |
| 15:40:45 | mriedem | i think, because otherwise i'd expect novalidhost errors while concurrent tests are consuming all the PCPU inventory on the single node | |
| 15:41:17 | bauzas | gibi: just a nit with +Wd https://review.opendev.org/#/c/679019/14/releasenotes/notes/support-cold-migrating-neutron-ports-with-resource-request-6d23be654a253625.yaml | |
| 15:41:19 | mriedem | i'd probably also just restrict tempest to tempest.api.compute tests to start | |
| 15:42:16 | donnyd | sean-k-mooney: You should be good to go now | |
| 15:45:13 | gibi | bauzas: ack, thanks | |
| 15:47:30 | bauzas | gibi: mriedem: efried_afk: I'm glad we can say support-move-ops-with-qos-ports as approved | |
| 15:47:43 | bauzas | let's now wait for the gate to claim it as implemented | |
| 15:47:55 | mriedem | cool, thanks for hitting those | |
| 15:48:13 | bauzas | mriedem: you did the most | |
| 15:48:59 | bauzas | efried_afk: sean-k-mooney: what's the status with vpmem ? | |
| 15:49:03 | bauzas | can i help ? | |
| 15:49:38 | bauzas | stephenfin: FWIW, reviewing back https://review.opendev.org/#/c/671801/48 | |
| 15:50:31 | stephenfin | bauzas: I'm going through vPMEM while my local hosts are deploying for manual upgrade testing of PCPU | |
| 15:50:40 | stephenfin | So it _should_ be fine | |
| 15:50:51 | stephenfin | We're going to need to rebase this whole thing into one giant chain once it's one though | |
| 15:51:20 | stephenfin | because vPMEM, VCPU and NUMA-aware LM series all conflict with each other | |
| 15:51:30 | bauzas | \o/ | |
| 15:51:39 | stephenfin | so much fun :) | |
| 15:51:45 | stephenfin | I can do that once everything is approved, I guess | |
| 15:52:05 | stephenfin | hopefully the conflicts are trivial (I suspect they will be) | |
| 15:52:47 | gibi | dddf | |
| 15:54:09 | gibi | sorry | |
| 15:54:23 | gibi | bauzas: thanks! | |
| 15:55:09 | gibi | bauzas: to be precise the cold migrate and resize part of support-move-ops-with-qos-ports is now approved. I have to continue working on evacuate, live migrate, and unshelve in Ussuri | |
| 15:55:23 | bauzas | oh correct of course | |
| 15:55:28 | bauzas | I was short in explanation :) | |
| 15:55:34 | gibi | :) | |
| 15:55:40 | bauzas | but we stick with the plan you wrote | |
| 15:55:53 | gibi | bauzas: yeah, there was no way to fitt everything | |
| 15:55:54 | bauzas | FWIW, I wish I could have had time for this with VGPU support... | |
| 15:56:06 | gibi | bauzas: let's try that again in U | |
| 15:56:17 | stephenfin | melwitt: ta for https://review.opendev.org/681374 (y) | |
| 15:56:20 | bauzas | every cycle, I make myself a promise | |
| 15:56:29 | bauzas | I stopped doing it | |
| 15:57:36 | luyao | stephenfin: reminder for vpmems series. :) | |
| 15:57:51 | stephenfin | luyao: https://review.opendev.org/#/c/678452/32 :) | |
| 15:58:04 | bauzas | stephenfin: I'm balancing in my head the need for defaulting disable_fallback_pcpu_query to True as default | |
| 15:58:28 | melwitt | stephenfin: np, good call on keeping the quota tests with it and moving the other unrelated func test to the next patch | |