| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-12 | |||
| 14:35:41 | dansmith | we need to at least make sure we don't regress the stupid behavior | |
| 14:35:41 | mnaser | ill wait until "we need it" | |
| 14:35:41 | sean-k-mooney | but if i use concurancy 1 it wont break anything | |
| 14:35:52 | dansmith | sean-k-mooney: or depends-on artoms? | |
| 14:36:08 | sean-k-mooney | im not sure if there will be a merge conflict | |
| 14:36:11 | sean-k-mooney | ill test that locally | |
| 14:36:15 | dansmith | uh | |
| 14:36:19 | openstackgerrit | Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245 | |
| 14:36:24 | dansmith | well, if there is we should rebase the pcpu stuff now, right? | |
| 14:36:35 | sean-k-mooney | am i guess so | |
| 14:36:50 | dansmith | maybe check locally first | |
| 14:36:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980 | |
| 14:36:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497 | |
| 14:36:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019 | |
| 14:36:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Extract pf$N literals as constants from func test https://review.opendev.org/680991 | |
| 14:36:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Improve dest service level func tests https://review.opendev.org/680998 | |
| 14:36:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Follow up for Ib50b6b02208f5bd2972de8a6f8f685c19745514c https://review.opendev.org/681490 | |
| 14:36:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip querying resource request in revert_resize if no qos port https://review.opendev.org/681513 | |
| 14:40:17 | sean-k-mooney | dansmith: artoms funtional tests still ned to be rebased but if i merge the cpu series with artoms serise without that it works locally | |
| 14:40:28 | sean-k-mooney | so ill do that quickly | |
| 14:40:33 | dansmith | sean-k-mooney: ack | |
| 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 | |