Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-11
15:18:26 dansmith mriedem: I don't think that's whathe's saying
15:18:29 mriedem it probably won't be because if you have old computes,
15:18:42 mriedem rpc is going to be pinned and we'll backlevel the new numa migrate_data object stuff
15:18:56 dansmith this ^ is what he's saying
15:19:14 dansmith so he just doesn't want to specifically say that the src or dst is too old.. right?
15:19:23 mriedem we never asked him to say that
15:19:35 mriedem we just said "say something about computes not being upgraded yet"
15:19:38 dansmith I don't think he claimed we did
15:19:46 mriedem who's on first?
15:19:55 dansmith I think he just said he's trying to wordsmith a suitably generic message right?
15:20:03 mriedem i provided one
15:20:16 artom Well there's the logging on L196 as well - it all fits together
15:20:32 artom Or I'm just overcomplicating everything?
15:20:50 dansmith I don't see any logging on L 196
15:21:14 artom You asked for it :)
15:21:40 dansmith you mean the logging that would be on L196 if you updated?
15:21:40 mriedem artom: i think dan just meant adjusting the msg object
15:21:53 mriedem if the config is enabled, we log a warning
15:21:58 mriedem else we raise, but with the same message
15:22:21 dansmith mriedem: I was suggesting one more log message inside the service version check
15:23:10 mriedem ok i'm going to move onto reviewing gibi's stuff now
15:23:14 artom Haha
15:23:45 dansmith how about artom just fixes the test issue, adds logs like he thinks he needs, and then we can sort out the log minutia in a later patch if necessary?
15:24:20 bauzas stephenfin: good news, I now start looking at your reshaper change :)
15:24:22 mriedem that's fine
15:24:30 stephenfin huzzah!
15:26:15 gibi_fly mriedem: hit me with any questions
15:27:14 artom dansmith, btw, the compute RPC version is per-cell, right? So, if we're in a cell with mixed stein/train computes, none of the will be able to send 5.3?
15:27:46 artom There's no backporting like what's done for objects - if a node can send 5.3, then all others must be able to receive it?
15:27:48 dansmith artom: not really, but I definitely do not think that level of detail needs to be in there
15:27:49 efried http://52.27.155.124/74/681474/2/check/pmem-tempest-plugin-filtered/134b243/job-output.txt.gz @ 2019-09-11 14:17:14.914120 is still applying PS13
15:27:49 efried sean-k-mooney: https://review.opendev.org/#/c/681474/2 appears not to have worked --
15:28:05 artom dansmith, yeah, maybe I'm overthinking it
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 bauzas but that's a sidetrack
15:44:07 efried but maybe only compared to the shit that's standard in the US
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

Earlier   Later