21:00:08 #startmeeting nova_cells 21:00:09 Meeting started Wed Jun 14 21:00:08 2017 UTC and is due to finish in 60 minutes. The chair is dansmith. Information about MeetBot at http://wiki.debian.org/MeetBot. 21:00:10 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 21:00:12 The meeting name has been set to 'nova_cells' 21:00:15 o/ 21:00:17 who dat 21:00:20 HI 21:00:31 dtp: inside voice please 21:00:46 _hi_ 21:00:52 better 21:00:55 * melwitt casts summon mriedem 21:00:56 o/ 21:00:57 no mriedem 21:00:58 heh 21:01:05 cool! 21:01:07 saving throw failed 21:01:10 #topic bugs/testing 21:01:13 haha 21:01:29 mriedem: this is your section of the meeting 21:01:41 oh god 21:01:48 I have this fix up: https://review.openstack.org/#/c/473931/ 21:01:58 that's the only recent one that comes to mind 21:02:01 which fixes ironic's devstack plugin when we're in multicell 21:02:06 woot, that's progress 21:02:24 I shall review it 21:02:37 I shall be appreciative 21:02:49 so that's not a bug though? there's no bug reference in the commit message 21:02:52 nothing else on fire? 21:03:01 melwitt: it is a bug but we found it pre-merge so nobody filed anything 21:03:08 oh k 21:03:10 melwitt: and it's really just a straggler from the bp 21:03:47 right we aren't going to backport this 21:03:50 so don't really need a bug 21:03:50 no other bugs.. going once ... 21:03:53 yeah 21:04:10 cool 21:04:29 #topic open wounds 21:04:50 quota battle damage 21:04:53 melwitt's stuff is still up, 21:04:56 I +Wd the bottom change recently and the next one has a -1 on it 21:05:44 the next one is updated already, mriedem said he'd +2 it :) 21:05:50 oh cool 21:05:56 i will after +2ing this stats thing 21:06:11 yeah, he's burning through these and it's awesome 21:06:21 melwitt: we call that "having given up" 21:06:32 >:| hrmph! 21:06:43 i -1 each one :) 21:06:50 so i haven't given up yet 21:06:52 well, it's still resulting in you getting patches merged, so .. 21:06:55 yeah, I mean it's great for one's stats 21:06:56 i'll give up when it comes to reviewing placement changes 21:07:05 heh you and me both 21:07:15 nested resource providers anyone? anyone? 21:07:15 anything else up for review? 21:07:33 bueller? 21:07:39 the placement project/user stuff is up and actively reviewed 21:07:45 oh right 21:07:46 but I think we all knew that 21:07:49 i still have this host api one 21:08:03 https://review.openstack.org/#/c/461532/ 21:08:04 yeah I need to review the host api ones 21:08:31 https://review.openstack.org/#/c/464280/ is also basically ready for review; the -2 is because i have to get the os-hypervisors API changes done and then we're going to lock-step those i think 21:08:37 I just starrred it 21:08:37 like we did for 2.36 in newton 21:08:57 so we -2 all other microversions until those get in, at some point 21:09:05 the os-hypervisors changes are turning out to be a major pain in my ass 21:09:49 yeah 21:10:09 anything else up? 21:10:10 if it weren't a total api rewrite, 21:10:14 then it woudn't be so bad 21:10:18 how come dtp gets away with not having open reviews? 21:10:26 he's an "operator" 21:10:30 oh 21:10:41 i dunno about that.. 21:11:30 okay well, anyway... anything else up we should mention here? 21:12:16 okay then 21:12:21 #topic open discussion 21:12:26 discuss, openly 21:12:44 I wanted to openly mention I resurrected the old console objects patches, 21:13:10 I openly hear what you're saying 21:13:29 and they mostly worked, but the noVNC console isn't working as the code expects. so I'm digging into that more. but AFAICT so far, it's not something that's going to end up working (putting server uuid in the access url), 21:14:24 and the effort might have to be redesigned in the worst case. I'm not 100% sure what's going on yet bc I don't know much about noVNC and how our websockets proxy works 21:14:52 okay I remember discussing this but it has all fallen out of my head 21:15:10 the idea was that we could put server uuid in the access url we give out to make things work with cells v2 21:15:19 this is the thing where the websocket proxy needs to know what cell to search for a token in? 21:15:31 and that it would be backward compatible with old proxies, but what I see in tempest is that for noVNC it is not backward compatible 21:15:35 yeah 21:15:48 melwitt: thorst and markus_z might be able to help 21:15:51 and this is because we assume a global auth service thing? 21:15:56 i think thorst's team added the vnc tests to tempest 21:16:02 and markus has done a bunch of console crap 21:16:05 mriedem: yeah 21:16:19 mriedem: cool, thanks. I think I'll soon be needing to ask some questions 21:16:38 the problem is we thought old proxies would ignore the server uuid added to the url 21:16:52 but noVNC isn't ignoring it and it's blowing up when the server uuid is added to the url 21:17:00 we should also consider if there's a reasonable way to shard this on cells too, like make each cell run its own proxy and just make the thing that returns the user point directly at the right one 21:17:04 because of the way it forwards the token as a header cookie 21:18:03 hm, okay. will think on that 21:18:30 it sounds like that could work. would eliminate the need for instance uuid in url 21:18:35 I would think sharding that service per cell would probably be good anyway, 21:18:52 since that could get overwhelmed by tons of people with open consoles 21:18:58 * melwitt nods 21:19:00 but yeah, think on it and feel free to ping me if you want to dive deep on it 21:19:07 cool, thank you 21:19:20 okay what else? 21:19:22 +2 on https://review.openstack.org/#/c/446241/ now so it's on dansmith 21:19:24 oh wait, 21:19:27 author, can't +W 21:19:34 yeah 21:19:42 https://review.openstack.org/#/c/473931/ +2 now it's on melwitt :) 21:19:46 jay'll do it 21:20:13 mriedem: looking at it now 21:20:20 nothing else from me 21:20:24 aight cool 21:20:32 * dansmith waits a few more seconds 21:21:06 #endmeeting