| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-12 | |||
| 21:30:09 | mriedem | efried: yeah maybe | |
| 21:30:15 | mriedem | that claims one was in the gate b/c my fix was merged | |
| 21:30:25 | mriedem | *before | |
| 21:30:39 | efried | #3 hasn't failed yet, #4 is still queued | |
| 21:31:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Correctly handle non-CPU flag traits https://review.opendev.org/681932 | |
| 21:31:13 | efried | so the question is, do we recheck 1&2, hoping 3&4 pass, and save their place in the queue | |
| 21:31:13 | efried | or do we rebase, and lose 3&4's place in the queue | |
| 21:31:13 | sean-k-mooney | if we are going to rebase the LM series now is proably better then in a few hours | |
| 21:31:29 | efried | artom: your call | |
| 21:31:38 | stephenfin | mriedem: I'll whip up a functional test for that tomorrow. It's too late to face into that now | |
| 21:31:41 | mriedem | ha, got a node after 11 hours, then failed | |
| 21:31:51 | stephenfin | but unit test is fixed and I've linked the bug | |
| 21:31:55 | mriedem | stephenfin: sure, i was just going to push a simple devstack patch to show the failure | |
| 21:32:03 | artom | efried, does it really matter? I'm assuming if I rebase now, I'll get administrative +Ws | |
| 21:32:04 | efried | artom: first option we have to wait until gate posts actual failure, which will be a while. IMO second option would be best | |
| 21:32:14 | efried | artom: yes | |
| 21:32:15 | artom | It's not like we need this by tomorrow | |
| 21:32:40 | stephenfin | giant chain of patches | |
| 21:32:41 | stephenfin | wheee | |
| 21:32:44 | mriedem | artom: sorry but my +Ws on that series were only good through yesterday | |
| 21:32:44 | artom | It can wait the weekend or w/e, when the load on the gate isn't as high | |
| 21:32:51 | artom | mriedem, you sonnova | |
| 21:32:57 | mriedem | i thought you saw the disclaimer? | |
| 21:33:18 | sean-k-mooney | artom: we are better off just rebasing it now i think | |
| 21:33:30 | efried | artom: by rebasing now, you kick four patches out of the queue, leaving room for other things. | |
| 21:33:55 | sean-k-mooney | if we can line ups the LM serise then put PCPUs on top the vPMEM i think that is the best path forward | |
| 21:34:31 | stephenfin | sean-k-mooney: As things would have it, I've got a PCPUs on vPMEM rebase just waiting to go | |
| 21:35:11 | sean-k-mooney | stephenfin: then after artom pushs can you rebase on top? | |
| 21:35:21 | efried | don't do that ^ | |
| 21:35:21 | stephenfin | yarp | |
| 21:35:25 | efried | cause it'll kick vpmem out | |
| 21:35:25 | stephenfin | or not | |
| 21:35:30 | stephenfin | actually, yeah, no | |
| 21:35:40 | mriedem | are these all conflicting? | |
| 21:35:43 | efried | minorly | |
| 21:35:51 | stephenfin | PCPU doesn't conflict with LM | |
| 21:35:58 | efried | and I un-conflicted vpmem with lm | |
| 21:35:59 | mriedem | i really only care about numa lm | |
| 21:36:03 | efried | yeah we know | |
| 21:36:05 | artom | mriedem, <3 | |
| 21:36:14 | stephenfin | VPMEM conflicted with LM but efried fixed it | |
| 21:36:20 | mriedem | ...because it's been a thing people have wanted forever | |
| 21:36:23 | stephenfin | PCPU and VPMEM is a horror show | |
| 21:36:40 | sean-k-mooney | mriedem: sicne we intoduced numa in like juno | |
| 21:36:42 | artom | mriedem, and here I thought it was because of my pretty eyes | |
| 21:36:42 | stephenfin | in terms of number of merge conflicts, not complexity | |
| 21:36:54 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827 | |
| 21:36:54 | artom | Incoming | |
| 21:36:54 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: LM: Use Claims to update numa-related XML on the source https://review.opendev.org/635229 | |
| 21:36:55 | mriedem | artom: you do have nice brown (?) eyes | |
| 21:36:55 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606 | |
| 21:36:55 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021 | |
| 21:36:56 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | |
| 21:36:58 | mriedem | i got lost in them | |
| 21:37:09 | efried | hazel is my vote | |
| 21:37:18 | mriedem | +Wing | |
| 21:37:20 | stephenfin | I could have worked on k8s... | |
| 21:37:35 | sean-k-mooney | stephenfin: i hear you like yaml | |
| 21:37:45 | sean-k-mooney | stephenfin: becasue that is all k8s is as a user | |
| 21:38:45 | mriedem | soon we'll all only code in yaml | |
| 21:39:07 | efried | mriedem: "I'll push a devstack change to put a required trait that I know will be on all libvirt computes and that can show the failure for us quicker than writing some functional libvirt test." I assume this is a joke. | |
| 21:39:22 | mriedem | efried: nope | |
| 21:39:29 | mriedem | it's a 2 line change in devstack, | |
| 21:39:33 | mriedem | yes i know it will take awhile to get a node | |
| 21:39:36 | efried | yeah, and an 11h... | |
| 21:39:44 | mriedem | but as melwitt pointed out earlier, nova gets punished by zuul | |
| 21:39:52 | mriedem | devstack probably doesn't | |
| 21:39:55 | efried | true story | |
| 21:40:03 | melwitt | yeah, a devstack change might make it in a few hours. or less | |
| 21:40:11 | sean-k-mooney | or you can cheat. notice that all my jobs completed in like 2 yours | |
| 21:40:13 | efried | I thought you meant push a change to nova/devstack | |
| 21:40:19 | mriedem | i heard fungi takes a devstack change manually and runs it on his own hardware :) | |
| 21:40:43 | sean-k-mooney | the FN special lables im using have there own pool and if you put them in the experimental queue run almost right away | |
| 21:40:58 | melwitt | let's cheat | |
| 21:41:55 | fungi | mriedem: i take devstack changes amd manually rub them on my feet. it's better than a pedicure | |
| 21:42:19 | mriedem | i was not expecting that response | |
| 21:42:39 | melwitt | that is some fungi CI indeed | |
| 21:42:53 | mriedem | now i just need to look up the magic incantation to configure a flavor with a required trait | |
| 21:42:53 | fungi | i no longer have a basement full of server racks | |
| 21:42:57 | sean-k-mooney | melwitt: i mean if we want to repoduce quickly we could. also im ment to be wringin a greneade job.... | |
| 21:42:59 | fungi | nor, you know, a basement | |
| 21:43:33 | sean-k-mooney | efried: stephenfin so what patches are where currently | |
| 21:43:39 | sean-k-mooney | vpmem is pending | |
| 21:43:47 | efried | mriedem: extra spec trait:YOUR_TRAIT_HERE='required' ? | |
| 21:43:49 | sean-k-mooney | and stephenfin you have a version of pcpu on that | |
| 21:43:58 | mriedem | efried: WRONG | |
| 21:44:02 | mriedem | trait:<trait>=required | |
| 21:44:06 | mriedem | quotes would kill you | |
| 21:44:15 | melwitt | sean-k-mooney: I only vaguely understand what y'all were talking about earlier. was just saying funny [to me] unhelpful things | |
| 21:44:20 | efried | mriedem: I was being syntax-y | |
| 21:44:34 | stephenfin | sean-k-mooney: I do, yeah. Just running tests locally before I push it up | |
| 21:44:35 | efried | if you're doing it on an osc cli, the quotes will go away | |
| 21:44:59 | efried | s/n osc// | |
| 21:45:25 | mriedem | https://review.opendev.org/681938 | |
| 21:45:44 | sean-k-mooney | melwitt: :) i mean if one queue is slow and the other is fast sometimes cheating in the gate is for the greater good | |
| 21:46:18 | efried | sean-k-mooney: keeping in mind that shoving stuff in the queue from another project will push all the nova stuff down a slot too. | |
| 21:46:54 | sean-k-mooney | ya i know that is why os-vif is generaly way quicker to test stuff in the nova | |
| 21:46:54 | fungi | yep. the change scheduling round-robin allocates nodes to changes by project, so the more changes there are for a given project requesting resources, the longer the later ones will wait for a turn | |
| 21:46:59 | mriedem | https://bugs.launchpad.net/nova/+bug/1843836 tagged for train-rc-potential so we can start using that tag | |
| 21:47:00 | openstack | Launchpad bug 1843836 in OpenStack Compute (nova) "Failure to schedule if flavor contains non-CPU flag traits" [Undecided,In progress] - Assigned to Stephen Finucane (stephenfinucane) | |