17:00:18 #startmeeting nova_cells 17:00:19 Meeting started Wed Aug 31 17:00:18 2016 UTC and is due to finish in 60 minutes. The chair is alaski. Information about MeetBot at http://wiki.debian.org/MeetBot. 17:00:20 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:00:22 The meeting name has been set to 'nova_cells' 17:00:31 ohai 17:00:34 o/ 17:00:41 o/ 17:00:43 gonna be a quick one today 17:00:49 #topic Open discussion 17:00:52 no real agenda 17:01:06 melwitt: your patch is ready and dependent on the fix? 17:01:30 dansmith: I just now pushed a new version pointing at the devstack change 17:01:55 changed from depending on the grenade hack patch to the devstack patch, that is 17:02:07 melwitt: right, that. so unless I find anything else we can +2 that yes? 17:02:07 which fix? 17:02:47 alaski: we needed to do something to get the rabbit userid variable usable in grenade so the mq switch could login to rabbit 17:02:53 I guess I can look at the depends-on 17:03:05 melwitt: oh yeah, that one 17:03:22 without that, the switch fails auth missing the userid 17:03:25 so your patch is ready other than that? 17:03:28 yes 17:03:31 great 17:03:40 alaski: I will hit that when we're done here 17:03:53 we have a couple of open devstack/grenade patches up that I may need to ping people on 17:04:11 the only other things I'd like to see get in are my two patches 17:04:13 but 17:04:24 I'm having an issue with service_version that I need to resolve 17:04:31 alaski: I was just looking at https://review.openstack.org/#/c/356138/ and it looks like there might be legit tempest failures on the list servers negative tests? 17:04:31 to be clear, that's https://review.openstack.org/#/c/356138/ 17:04:34 ? 17:04:47 yes 17:05:02 right now get_minimum_version or whatever returns 0 17:05:09 even though every service says it's version 15 17:05:21 alaski: is that happening before any service record has been written? 17:05:24 that's a known quirk 17:05:34 no, this is quite late 17:05:40 alaski: basically, if you're the first service you can't figure out what the real minimum is 17:05:41 okay 17:05:42 like halfway through a tempest run 17:05:48 okay 17:05:54 okay, sorry I didn't realize the bug you're working through was on the bottom patch, I assumed it was on the WIP one 17:05:55 alaski: using the wrong binary name? 17:06:14 dansmith: maybe. I just added some more logging to see what's going on 17:06:22 but I'll check the code for that 17:06:41 melwitt: well, the bug is on a merged patch. but isn't exposed until the bottom one 17:06:53 gotcha 17:06:56 but it's very rare so I didn't see it until two days ago 17:07:36 If I can fix that, and add testing for the current WIP one, those are what I'd like to get in 17:07:44 but if not, we're still in a good spot I think 17:07:56 i'm asking sean about FF for devstack/grenade, 17:08:02 not sure if we have some more time there or not 17:08:02 that's all I've got, just wanted to touch base real quick 17:08:04 before they branch it 17:08:08 mriedem: okay 17:08:11 because i see https://review.openstack.org/#/c/362154/ 17:08:15 alaski: they branch super late 17:08:16 and https://review.openstack.org/#/q/status:open+project:openstack-dev/grenade+branch:master+topic:setup_cell0_before_migrations 17:08:20 alaski: so I kinda expect not 17:08:24 er, mriedem ^ 17:08:32 they branch well after the final release normally I think 17:08:35 are any of the nova changes dependent on devstack/grenade cahnges? 17:08:40 also https://review.openstack.org/#/c/362154/ 17:08:57 https://review.openstack.org/#/c/356138/ is 17:09:01 dependant 17:09:47 hmm, ok 17:09:49 my patch depends on https://review.openstack.org/#/c/363295/ in devstack 17:10:05 there's also https://review.openstack.org/#/c/355602/ which should depend on one of the above patches 17:10:48 so that's where things are at 17:10:50 ok, i can rebase those 2 devstack changes to line them up 17:10:58 thanks 17:11:09 anything else to mention? 17:11:25 what about https://review.openstack.org/#/c/362310/1 ? 17:11:30 that's the bottom of the grenade series 17:11:53 there should be something depending on that somewhere, but that's needed too 17:12:05 it's the bottom of hte series and the nova change is dependent on that series 17:12:19 ahh 17:12:34 my point being, 17:12:38 if we really want https://review.openstack.org/#/c/356138/ for FF 17:12:44 then we need those 2 grenade changes to land 17:12:53 yes 17:13:23 ok i'm pinging sdague about that 17:13:28 I want https://review.openstack.org/#/c/356138/ to land. but I don't think it has an upgrade impact, in that if it doesn't make it I think we're okay in O 17:13:47 we just won't hit the goal I had hoped to hit 17:13:47 and even for ^, 17:13:55 we don't have anything going to cell0 yet right? 17:14:08 mriedem: right, the WIP is the first patch to really use it 17:14:15 because https://review.openstack.org/#/c/267827/ 17:14:46 that's actually not necessary any more 17:14:51 my WIP supercedes it 17:14:52 i'm not sure how https://review.openstack.org/#/c/267827/ even passes jenkins w/o depending on these other changes 17:14:55 oh 17:14:58 can we drop https://review.openstack.org/#/c/267827/ then? 17:15:04 yes 17:15:52 abandoned 17:16:15 anything more? 17:16:46 great. thanks all 17:16:49 #endmeeting