| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-11 | |||
| 15:29:27 | efried | sean-k-mooney, dtroyer: I can't tell by looking at https://github.com/intel/Intel-OpenStack-CI-jobs/pull/1/commits/5d4a998a890fef3a2b0a46f73d26aa1deeb3cd8b -- is this merged? | |
| 15:29:29 | sean-k-mooney | efried in that case they are likely using the git plug not the gitub plugin | |
| 15:30:09 | sean-k-mooney | efried: the zuul git driver does not support pull request but works with any git repo via a timer | |
| 15:30:27 | sean-k-mooney | the gitub one require the zuul app to be installed in the github org | |
| 15:30:35 | sean-k-mooney | but once that is done pull requeist work | |
| 15:30:47 | sean-k-mooney | i suspect intel it dont allow the zuul app | |
| 15:30:49 | efried | sean-k-mooney, dtroyer: I wanted to just recheck one of the top vpmem patches to get new results | |
| 15:31:03 | efried | but no point unless that pr merged | |
| 15:32:04 | mriedem | gibi_fly: | |
| 15:32:05 | mriedem | https://review.opendev.org/#/c/676980/19 | |
| 15:32:11 | gibi_fly | mriedem: looking | |
| 15:33:22 | bauzas | I believe I can gibi_fly | |
| 15:33:32 | bauzas | (sorry) | |
| 15:33:35 | mriedem | get out | |
| 15:35:16 | gibi_fly | mriedem: good points. I will fix it up today. | |
| 15:36:01 | sean-k-mooney | efried its not merged by the way | |
| 15:36:02 | gibi_fly | bauzas: I will change to a bus soon so you were on time with that joke :) | |
| 15:36:22 | bauzas | gibi_fly: do you need help for providing a new revision ? | |
| 15:36:23 | sean-k-mooney | efried: the PR woudl be closed if it was merged | |
| 15:36:39 | bauzas | gibi_fly: since you're a bit on-off | |
| 15:37:04 | bauzas | FWIW, I starbucks'd last Friday and it wasn't a great experience | |
| 15:37:06 | gibi_fly | bauzas: thanks. I will manage | |
| 15:37:18 | gibi_fly | bauzas: I will ping you for review :) | |
| 15:37:21 | bauzas | gibi_fly: cool | |
| 15:37:55 | gibi_fly | bauzas: I don't know which net is better, plane or starbucks | |
| 15:38:05 | efried | sean-k-mooney: okay, that's what I thought, but dtroyer said he merged it so I wasn't sure if I just didn't know how to read the thing (which would be totally likely) | |
| 15:38:39 | luyao | stephenfin: could you review rest of the patch series about vpmems? | |
| 15:38:42 | sean-k-mooney | he approved it and i guess that normllay allows a ci to merge it but in this case it might need to be manually submitedd | |
| 15:38:48 | stephenfin | luyao: Already on it :) | |
| 15:39:52 | luyao | stephenfin: Thanks! alex_xu will address the numa stuff. :) | |
| 15:41:00 | bauzas | gibi_fly: I'm just surprised you getting internet connection while on flight | |
| 15:41:17 | bauzas | gibi_fly: I never saw this with BA or Swiss or whatever else | |
| 15:41:37 | alex_xu | luyao: go home, it is too late to be office right now | |
| 15:41:46 | gibi_fly | bauzas: Lufthansa has some paidd service | |
| 15:42:32 | gibi_fly | bauzas: 7 EUR to push patches and read reviews sounded like a bargain | |
| 15:43:24 | luyao | alex_xu: I prepared to fight overnight. :D | |
| 15:43:52 | efried | lufthansa is a lovely airline | |
| 15:43:58 | bauzas | oh gosh it's not | |
| 15:44:07 | efried | but maybe only compared to the shit that's standard in the US | |
| 15:44:07 | bauzas | but that's a sidetrack | |
| 15:45:27 | dtroyer | efried, sean-k-mooney: it is probably my github fail… I hit approve… but I've also been multi-tasking two meetings this whole time... | |
| 15:45:31 | stephenfin | when you've flown Ryanair, anything is a lovely airline | |
| 15:45:49 | melwitt | I find lufthansa's gate boarding "system" a total mess (in that there seems to be no system) but other than that, I've been happy enough with it | |
| 15:46:13 | sean-k-mooney | dtroyer: no worries. you have too also manually submit it if there is not a ci set up to do it | |
| 15:48:33 | efried | aspiers: fyi you can knock a patch out of the gate by rebasing it. I can re+W if you want to do that rather than waiting for the gate to fail. | |
| 15:49:18 | aspiers | efried: ah, nice idea thanks! | |
| 15:50:59 | bauzas | well, what I can just say about Lufthansa is how they're aggresive with carry-on luggage | |
| 15:51:05 | openstackgerrit | sean mooney proposed openstack/nova master: multi numa nfv testing job https://review.opendev.org/679656 | |
| 15:51:06 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] testing with new lable. https://review.opendev.org/680738 | |
| 15:51:35 | bauzas | every time I flew with them, my carry-on was taken on checked luggage, just because they felt they were running out of space in the cabin | |
| 15:51:59 | dtroyer | efried, sean-k-mooney: ok, for the record, just talked with Dolpher and his final change is ready to be merged into the Intel-OpenStack-CI-jobs repo, I should get that done after this meeting | |
| 15:52:07 | bauzas | I admit this was in Geneva, but come on, even on August when all people wear light stuff ? | |
| 15:52:12 | efried | dtroyer: nice | |
| 15:52:40 | sean-k-mooney | dtroyer: ok ill close my PR then | |
| 15:52:50 | sean-k-mooney | lets just go with the proper one | |
| 15:53:20 | sean-k-mooney | also i forgot to chnage the rul in the numa tests new patch incoming | |
| 15:54:56 | efried | melwitt: I wonder if you're hitting https://github.com/openstack/keystoneauth/blob/38cd5fc6c39c38a51c11683884caf9696ce5f367/keystoneauth1/exceptions/http.py#L411 which afaict will result in your exception containing bugger-all. | |
| 15:56:08 | openstackgerrit | sean mooney proposed openstack/nova master: multi numa nfv testing job https://review.opendev.org/679656 | |
| 15:56:09 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] testing with new lable. https://review.opendev.org/680738 | |
| 15:56:44 | efried | melwitt: ...which may be possible if the encoding is off or something. | |
| 15:59:04 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test migration with pinning https://review.opendev.org/679754 | |
| 15:59:38 | melwitt | efried: thanks for the idea | |
| 16:01:12 | artom | Stupid sphinx errors line numbers that have nothing to do with anything | |
| 16:01:19 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test non overlaping vcpu pin sets. https://review.opendev.org/679805 | |
| 16:02:06 | sean-k-mooney | artom: ok all the numa jobs should be using the old nodepool lable now and running agaist the patch before the functionl tests | |
| 16:05:50 | sean-k-mooney | maybe i missed one | |
| 16:06:29 | mriedem | artom: if it was reno, i use an online sphinx editor to validate those rather than wait to build them locally | |
| 16:07:08 | aspiers | efried: hrm, bit confused - now the failure has disappeared from the gate but there's another job for it still running in the gate's "integrated" queue which has no failures | |
| 16:07:30 | artom | mriedem, thanks for the suggestion, I think I'll do that | |
| 16:07:38 | artom | sean-k-mooney, cheers, thanks! | |
| 16:07:46 | aspiers | efried: might it have failed in a different queue in the gate? which gate queue(s) actually count for the V+2? | |
| 16:07:48 | sean-k-mooney | fixing the last one | |
| 16:08:02 | efried | aspiers: is it possible you were looking at your patch as it existed as part of some other patch's chain? | |
| 16:08:13 | efried | what's the change #? | |
| 16:08:21 | aspiers | efried: 681254 | |
| 16:08:34 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test non overlaping vcpu pin sets. https://review.opendev.org/679805 | |
| 16:08:35 | aspiers | efried: the failure I saw was at the tip of a chain | |
| 16:09:34 | aspiers | weird, can't see it | |
| 16:09:41 | aspiers | the only failure was from the first time around | |
| 16:09:58 | aspiers | i.e. 2019-09-11T05:56:59 which was before my recheck | |
| 16:10:04 | aspiers | maybe I was hallucinating | |
| 16:10:12 | efried | yeah, it looks okay to me | |
| 16:10:35 | sean-k-mooney | donnyd: fyi i have 4 jobs queue 3 using the old lable and 1 using the new one so ill let you know how it goes. hopeful your router tweaks resolved the issue | |
| 16:14:30 | sean-k-mooney | stephenfin: efried is there anything i can do to help with either the PCPU in plamcenet seriese or pmem? | |
| 16:15:40 | efried | sean-k-mooney: I'm at a point where I'm willing to proxy your vote on https://review.opendev.org/#/c/678455/ -- once we've agreed whether it needs to be changed or not. | |
| 16:15:56 | efried | I'm not clear whether y'all came to an agreement above? | |
| 16:16:02 | stephenfin | sean-k-mooney: I'm on the vpmem one, so if you fancy looking at either https://review.opendev.org/#/c/674895/ | |
| 16:16:08 | stephenfin | efried: alex_xu is on it | |
| 16:16:18 | mriedem | TestInstanceNotificationSampleWithMultipleCompute.test_multiple_compute_actions just reset the gate again, | |
| 16:16:22 | mriedem | i think we have a regression | |
| 16:16:47 | stephenfin | We agreed it was better to keep the one guest NUMA node to one host NUMA node model, even if we don't do proper affinity for the VPMEM devices | |
| 16:17:12 | efried | and that's because qemu has some checks in place that don't allow us to do the slightly better thing? | |
| 16:17:13 | sean-k-mooney | stephenfin: so like pci passthoug with affinity disabled | |
| 16:17:25 | stephenfin | efried: Yup | |
| 16:17:30 | efried | okay cool. | |
| 16:17:42 | stephenfin | sean-k-mooney: Yeah, good analogy | |
| 16:18:01 | stephenfin | efried: I did have a comment on https://review.opendev.org/#/c/678448/20 though | |
| 16:18:36 | stephenfin | I'm not sure if it's worth respinning for but if we don't, we're stuck with that object name forever | |
| 16:19:32 | sean-k-mooney | whats SlugField | |
| 16:20:15 | sean-k-mooney | oh somting that is alpha numeric or has _ | |