14:01:24 <sigmavirus24> #startmeeting Glance
14:01:25 <openstack> Meeting started Thu Jan 29 14:01:24 2015 UTC and is due to finish in 60 minutes.  The chair is sigmavirus24. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:01:26 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:28 <openstack> The meeting name has been set to 'glance'
14:01:58 <sigmavirus24> So it looks like today is going to be a very light day on participation and topics to discuss
14:02:16 <kragniz> I don't think many are here
14:02:34 <sigmavirus24> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:02:45 <sigmavirus24> #topic TODOs/Updates
14:02:54 <sigmavirus24> Anyone have anything to discuss for this?
14:02:58 <flaper87> o/
14:03:06 * flaper87 thought there was no meeting today
14:03:20 <sigmavirus24> No worries flaper87
14:03:28 <sigmavirus24> Any TODOs/Updates from you flaper87 ?
14:03:51 <flaper87> the client patch landed (with the `--limit` fix)
14:03:58 <flaper87> I need to add it to the to-release list of things
14:04:10 <sigmavirus24> Good
14:04:17 <flaper87> other than that, I don't have many updates. I'd like us to talk a bit about what we'll do with changes-since
14:04:28 <flaper87> since we need to make a final decision there
14:04:31 <flaper87> and
14:04:36 <kragniz> flaper87: probably wait until more people are around for that
14:04:40 <sigmavirus24> Come back around to that?
14:04:50 <flaper87> just in case some folks missed this: We decieded to push the glance v1->v2 changes to L
14:04:51 <sigmavirus24> (It won't be long I promise)
14:04:59 <sigmavirus24> flaper87: in Nova you mean :)
14:05:06 <flaper87> yup
14:05:18 <flaper87> kragniz: sigmavirus24 yup, we definitely need more ppl for that
14:05:32 * flaper87 proposed the push back himself
14:05:40 * flaper87 sometimes doesn't understand himself
14:05:43 <flaper87> but oh well.
14:05:54 <sigmavirus24> On my end the policy spec implementation is going slower than I expected but I think I'm picking up momentum. Still not sure it'll be ready for k-2 though
14:05:57 <flaper87> other than that, I think I'm ok
14:06:06 <sigmavirus24> Also I still have feedback from stevelle and zhiyan to address on it
14:06:17 <sigmavirus24> kragniz: any updates you'd like ot include?
14:06:33 <kragniz> for swift retry spec, I'm going to wait for feedback from stuart before pushing any new changes
14:06:59 <sigmavirus24> Seems fair kragniz
14:07:05 <kragniz> that's about it
14:07:15 <sigmavirus24> #topic Reviews/Bugs/Releases
14:07:24 <sigmavirus24> #link https://review.openstack.org/#/c/148575
14:07:53 <flaper87> I think we should just use oslo.middleware
14:07:59 <flaper87> I've been meaning to get back to that review
14:08:02 <kragniz> I feel quite strongly we should not create more code duplication
14:08:09 <sigmavirus24> So that healthcheck middleware that jokke_ implemented (I pinged him but he's still not here) was cribbed mostly from swift but oslo.middleware now has a heftier implementation available
14:08:36 * flaper87 has no idea what heftier means
14:08:47 <sigmavirus24> I think jokke_ is thoroughly set against that, so if it is the decision of the glance team to use oslo.middleware, someone else may need to implement it
14:09:04 <flaper87> #link https://review.openstack.org/#/c/137416/ <- these babies need some love
14:09:05 <sigmavirus24> flaper87: the oslo.middleware version does a lot more (database checks, and other checks that I'm forgetting)
14:09:08 <sigmavirus24> It's a lot more code
14:09:11 <kragniz> flaper87: bigger
14:09:16 <flaper87> ah ok, nice
14:09:22 * flaper87 learnt something today
14:09:25 <flaper87> (as every other day)
14:09:29 <flaper87> yeah, I agree
14:09:39 <sigmavirus24> flaper87: I agree. That review is wonderful
14:10:00 <flaper87> I'm with y'all re using oslo.middleware right away
14:10:22 <sigmavirus24> #link https://review.openstack.org/#/c/150992/
14:10:31 <sigmavirus24> If anyone was having issues with grenade yesterday, you can recheck now I think
14:10:44 <sigmavirus24> That review was merged and pinned oslo.vmware to <0.9.0 because of another requirements clash
14:10:50 <kragniz> awesome
14:11:03 * sigmavirus24 is thinking the oslo libraries should all be capped in stable/juno at this point maybe
14:11:08 <flaper87> nice, the taskflow patch got stuck on that
14:11:16 <sigmavirus24> That's the second release this week that broke the gate
14:11:22 <sigmavirus24> (With the exact same problem too)
14:11:42 <kragniz> sigmavirus24: wasn't there talk of stable oslo branches?
14:11:43 <sigmavirus24> Oh one other review I want people to look at while I have you all gathered here
14:11:48 * kragniz may be making things up
14:12:08 <sigmavirus24> #link https://review.openstack.org/#/c/80178/
14:12:21 <sigmavirus24> kragniz: I don't think so. The main discussion around oslo has been py26 compat
14:12:43 <kragniz> okay
14:12:56 <flaper87> Btw, there are some glance_store patches that also need discussion
14:13:09 <flaper87> For example: remove cinder? remove py26 support? remove gridfs ?
14:13:16 <sigmavirus24> #link https://review.openstack.org/#/c/137310/
14:13:20 <sigmavirus24> ^ Remove py26 support
14:13:25 <sigmavirus24> (I was just looking for that one flaper87 :))
14:13:37 <flaper87> For the latest two, I think we pretty much agreed on doing that
14:13:45 <flaper87> the first one, however, needs some extra discussion
14:13:53 <kragniz> we can't remove py26 support yet, right?
14:14:08 <flaper87> kragniz: not sure, we did it in glance and all server projects
14:14:10 <sigmavirus24> kragniz: that was my impression, yes
14:14:17 <sigmavirus24> flaper87: so the server projects are fine
14:14:21 <sigmavirus24> The libraries are not iirc
14:14:23 <kragniz> since juno supports py26 and depends on py26
14:14:29 <flaper87> oslo libraris not, yes.
14:14:32 <sigmavirus24> The libraries are used at least in juno
14:14:34 <kragniz> glance_store, rather
14:14:50 <sigmavirus24> flaper87: unless we're capping stable/juno's requirement on glance_store (or have already done so)
14:14:58 <flaper87> that was the goal, AFAICT
14:15:06 <flaper87> to cap stable branches
14:15:10 <dhellmann> sigmavirus24, flaper87 : there is work going on now to cap all of the juno requirements
14:15:10 <flaper87> lemme dig more on that
14:15:19 <dhellmann> jogo and dtroyer are working on it
14:15:21 <flaper87> awesome
14:15:26 * flaper87 attended that session
14:15:28 <kragniz> cool
14:15:32 <flaper87> but my memory is terrible
14:15:40 <sigmavirus24> flaper87: https://github.com/openstack/glance/blob/stable/juno/requirements.txt#L60
14:15:46 <dhellmann> and I'll look into an oslo.middleware release for you today or monday
14:15:56 <sigmavirus24> dhellmann: good to know
14:15:58 <sigmavirus24> Thanks
14:16:16 <kragniz> so if glance_store is capped on stable, we can remove py26?
14:16:22 <flaper87> sigmavirus24: I think we can cap glance_sotre for juno after the next release
14:16:26 <flaper87> kragniz: ^
14:16:28 <sigmavirus24> kragniz: that seems the reasonable way to do it
14:16:32 <flaper87> the next glance_store release, that is
14:16:35 <kragniz> okay
14:16:37 <sigmavirus24> flaper87: right
14:16:51 <sigmavirus24> And currently https://github.com/openstack/glance_store/blob/master/requirements.txt#L11 won't bite us :)
14:17:07 <sigmavirus24> (That's been the requirement causing problems for oslo.{vmware,messaging})
14:17:37 <flaper87> sigmavirus24: we can actually remove that requirement
14:17:39 <flaper87> I'll do that
14:17:52 <flaper87> we needed it for a different oslo-inc lib
14:17:58 <flaper87> we now just have context
14:18:03 <sigmavirus24> #link https://review.openstack.org/#/c/144631/
14:18:07 <flaper87> (which we can remove later since oslo.context exists)
14:18:10 <sigmavirus24> ^ That should be an easy review to workflow
14:18:19 <flaper87> or was that in utils? ARGH, my memory sucks
14:18:37 <sigmavirus24> flaper87: you do too much :)
14:18:45 <flaper87> sigmavirus24: are you abusing my +2 powers right now?
14:18:49 * flaper87 has to be careful
14:18:53 <flaper87> approved the last one
14:19:02 <flaper87> sigmavirus24: and yes, I should realllyy do fewer things
14:19:04 <flaper87> :(
14:19:32 <sigmavirus24> flaper87: I may be
14:19:44 <flaper87> lol
14:20:00 <sigmavirus24> Any other reviews to point out specifically?
14:20:05 <kragniz> if we're approving namespace changes, this would be nice to have in: https://review.openstack.org/#/c/145238/
14:20:19 <kragniz> since a few others depend on it
14:20:23 <sigmavirus24> flaper87: while we have you trapped here ;)
14:21:34 <flaper87> LOL
14:22:06 <flaper87> kragniz: I'll take a look at that one in a bit
14:22:12 <kragniz> flaper87: cool
14:22:20 <kragniz> no rush
14:22:23 <sigmavirus24> That whole topic needs some review love
14:22:25 <sigmavirus24> :)
14:22:33 <kragniz> yup
14:22:49 * kragniz isn't having fun with porting glance to oslo_log
14:23:13 <flaper87> it's never fun but needed
14:23:18 <sigmavirus24> As for bugs
14:23:20 <sigmavirus24> #link https://bugs.launchpad.net/glance-store/+bug/1415679
14:23:35 <flaper87> I'd really like to understand why s3 depends on eventlet
14:23:37 <flaper87> oh lord
14:23:38 <sigmavirus24> That's one I didn't try to replicate last night but I added glance-store to it because it seemed most likely to be there
14:25:31 <sigmavirus24> Seemed like one we might also want to debug before kilo is finally released
14:25:42 <flaper87> mmh, yeah
14:25:42 <sigmavirus24> Anyone have other bugs/blueprints/reviews?
14:25:46 <flaper87> nope
14:25:48 <kragniz> https://bugs.launchpad.net/glance/+bug/1412798
14:25:55 <kragniz> can anyone reproduce this?
14:25:57 <sigmavirus24> #link https://bugs.launchpad.net/glance-store/+bug/1415679
14:26:04 <flaper87> #link https://review.openstack.org/151261
14:26:06 <flaper87> fresh ^
14:26:07 <sigmavirus24> kragniz: I haven't tried yet, but I will
14:26:12 <kragniz> sigmavirus24: thanks
14:27:41 <sigmavirus24> Anything else?
14:27:53 <kragniz> not from me
14:27:56 <kragniz> I think
14:28:24 <sigmavirus24> #topic Open Discussion
14:28:55 <sigmavirus24> Doesn't seem like many more people showed up flaper87 but if you still want us to discuss that topic, we might as well
14:29:13 <flaper87> nah, it can wait
14:29:16 <sigmavirus24> (changes-since == that topic)
14:29:25 <flaper87> I'd like to go through git-log and read the logs
14:29:36 <flaper87> probably there's some more info in those reviews
14:29:44 <flaper87> that will give us more context to make a final decision
14:30:53 <kragniz> oh, one quick thing
14:31:06 <kragniz> do we want hacking checks for the new oslo imports?
14:31:19 <kragniz> I see nova and neutron are adding them
14:31:23 <flaper87> sure
14:31:26 <sigmavirus24> kragniz: crib those from nova?
14:31:26 <flaper87> why not
14:31:28 <flaper87> :D
14:31:35 <kragniz> sigmavirus24: that's my thought
14:31:37 <kragniz> :P
14:31:38 <sigmavirus24> Before we do that
14:31:56 <sigmavirus24> dhellmann: won't the old namespaces give deprecation warnings on newer versions of the oslo.* libraries?
14:32:12 <kragniz> sigmavirus24: yes, they do
14:32:59 <sigmavirus24> I wonder if it'd be easier to catch those warnings in tests and fail if they're raised as an alternative method
14:33:21 <kragniz> hmm
14:33:37 <dhellmann> you'd have to configure that particular warning to be handled as an error
14:34:22 <dhellmann> once for each library, which isn't a lot
14:35:32 <kragniz> I think a simple regex hacking rule would be fine
14:35:46 <sigmavirus24> I'm also wondering if there's value in having tests fail vs having tox -epep8 fail =P
14:35:48 <kragniz> (nova's proposed new rule: https://review.openstack.org/#/c/146294/20/nova/hacking/checks.py)
14:36:17 <sigmavirus24> (gerrit is really slow again today for me)
14:37:07 <sigmavirus24> kragniz: yeah looks like it should work
14:37:15 <kragniz> it's short and simple
14:37:47 <kragniz> but yeah, that's not a pressing concern
14:38:57 <sigmavirus24> Should we close the meeting early?
14:39:00 <kragniz> just checking no one morally opposed it
14:39:01 <kragniz> sure
14:39:14 <kragniz> I don't think we have much more to talk about
14:39:46 <sigmavirus24> flaper87: objections?
14:39:49 <flaper87> nope
14:39:57 * flaper87 reads the backlog after saying nope
14:40:13 <flaper87> so yeah, no objections
14:40:14 <flaper87> :P
14:40:22 <sigmavirus24> #endmeeting