| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-12 | |||
| 14:40:53 | sean-k-mooney | the functionla patch we are leaving to not need to rebase all the rest | |
| 14:40:55 | dansmith | sean-k-mooney: that means you can depends-on it in your gate jobs yeah? or are you going to rebase all of pcpu and push it up? | |
| 14:41:04 | sean-k-mooney | so that will be rebased when the stuff is merged | |
| 14:41:16 | mriedem | gibi: bauzas: melwitt: i'm +2 up the bw provider migrate/resize series now https://review.opendev.org/#/q/topic:bp/support-move-ops-with-qos-ports+status:open | |
| 14:41:23 | mriedem | just a few changes need a +W but they aren't too hard | |
| 14:41:39 | sean-k-mooney | ya i just need to depnd on the second form last patch in artoms serie | |
| 14:41:53 | sean-k-mooney | e.g. the one before the the functionl test | |
| 14:42:08 | sean-k-mooney | but i can have the job that is testing artoms stuff depend on stephens change too | |
| 14:42:20 | sean-k-mooney | and it should all merge fine | |
| 14:42:38 | sean-k-mooney | did i mention that zuul is awsome recently | |
| 14:42:40 | artom | Famous last words ;) | |
| 14:43:40 | sean-k-mooney | i did the merge locally so it should be fine | |
| 14:44:41 | sean-k-mooney | it will be a 4 way merge however (master+numa+cpu+ci) patches | |
| 14:45:04 | dansmith | nothing has even entered the gate yet | |
| 14:45:14 | dansmith | I mean, nothing at all | |
| 14:45:19 | mriedem | check queue is ~200 deep so nothing approved today is going to merge today | |
| 14:45:21 | sean-k-mooney | yep | |
| 14:45:21 | mriedem | for nova anyway | |
| 14:46:01 | dansmith | mriedem: don't burn your cards | |
| 14:46:19 | mriedem | hey, i'm a professional | |
| 14:46:54 | mriedem | oh wait, | |
| 14:47:02 | mriedem | can i switch cigarette for flavored vape? | |
| 14:47:08 | mriedem | i want to be young and hip | |
| 14:48:07 | dansmith | mriedem: vaping kills, haven't you heard? | |
| 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 | they just do it and then tell their parents they were at their friend's house, the friend whose parents smoke | |
| 14:49:32 | mriedem | duh | |
| 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 | donnyd | https://review.opendev.org/#/c/681731/ | |
| 14:55:11 | sean-k-mooney | ok then ^ will fail | |
| 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 | |