19:02:36 <Alex_Gaynor> #startmeeting python-openstacksdk
19:02:36 <openstack> Meeting started Tue Jun  3 19:02:36 2014 UTC and is due to finish in 60 minutes.  The chair is Alex_Gaynor. Information about MeetBot at http://wiki.debian.org/MeetBot.
19:02:37 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
19:02:39 <openstack> The meeting name has been set to 'python_openstacksdk'
19:03:01 <Alex_Gaynor> Ok, I guess we'll do roll call to get started :-)
19:03:08 <Alex_Gaynor> Alex Gaynor, Rackspace (filling in for briancurtin today)
19:03:17 <terrylhowe> Terry Howe HP
19:04:26 <Alex_Gaynor> Anyone else? Bueller?
19:04:31 <jamielennox> Jamie Lennox - Red Hat
19:05:01 <Alex_Gaynor> Ok, we'll give it a minute to see if anyone else tickles in
19:07:18 <Alex_Gaynor> Okey doke, I guess we'll just go short handed.
19:07:38 <Alex_Gaynor> I dont' have a particular agenda for today, we got a few PRs, including teh auth one, landed last week, so that's pretty great.
19:07:40 <edleafe> Ed Leafe, Rackspace (running late)
19:08:07 <Alex_Gaynor> terrylhowe: Do you want to walk us through the remaining outstanding CRs?
19:09:15 <terrylhowe> https://review.openstack.org/#/c/94887/
19:09:26 <Alex_Gaynor> #link https://review.openstack.org/#/c/94887/
19:09:48 <terrylhowe> the most important outstanding one I guess, waiting on Dean, but I think he is AFK most of this week
19:11:12 <terrylhowe> I agree with jamielennox comment, but waiting on Dean
19:11:29 <Alex_Gaynor> Does that make us totally blocked until Dean returns?
19:11:51 <jamielennox> terrylhowe: i think if you make my change, then we pass it without him
19:11:57 <jamielennox> he can come back through and fix it later
19:12:35 <terrylhowe> that would unblock some others
19:12:44 <jamielennox> ++
19:12:56 <Alex_Gaynor> Unblocking sounds good :-)
19:13:18 <terrylhowe> I wanted to throw exceptions on 4xxs etc though, not sure how everyone felt about that
19:14:00 <Alex_Gaynor> I'm +1 on defaulting to exceptions on non-{2xx,3xx} responses
19:14:31 <Alex_Gaynor> A nickel for every ValueError: malformed JSON because I tried to parse an HTML 500 :-)
19:14:43 <jamielennox> terrylhowe: i agree, though in my case i had to add a raise_exc=True param by default to make it optional
19:15:08 <jamielennox> hopefully we shouldn't need it here
19:15:12 <terrylhowe> okay
19:16:14 <terrylhowe> #link https://review.openstack.org/#/c/96436/
19:16:25 <terrylhowe> I don’t think anyone disagrees
19:16:41 <Alex_Gaynor> Looks like it needs to be rebased, because of a merge conflict
19:17:38 <Alex_Gaynor> I'm happy to +A it once it is
19:18:16 <terrylhowe> #link https://review.openstack.org/#/c/96437/
19:18:29 <terrylhowe> is next
19:18:59 <Alex_Gaynor> How does this affect the "root" identity service?
19:19:09 <terrylhowe> also needs rebasing
19:19:40 <terrylhowe> I’ve had the auth stuff working without using base_url
19:19:57 <jamielennox> Alex_Gaynor: once there is an auth plugin you just get the url from there
19:19:59 <terrylhowe> I was going to check in some example code later today or tomorrow
19:20:06 <Alex_Gaynor> Ok, cool
19:20:18 <jamielennox> keystone is in the service catalog as well
19:21:50 <terrylhowe> #link https://review.openstack.org/#/c/96604/
19:22:33 <jamielennox> that was failing because of the setuptools bug
19:22:40 <Alex_Gaynor> Straightforward +2. I only didn't because all  tests were failing, will +A now
19:22:44 <jamielennox> should be able to be rechecked now
19:22:54 <Alex_Gaynor> All of zuul is having a bad time now, but that's a different issue
19:23:15 <jamielennox> Alex_Gaynor: right, this is the most backed up i think i've seen it
19:24:30 <terrylhowe> #link https://review.openstack.org/#/c/96513/
19:24:47 <terrylhowe> Session docs, needs probably more careful consideration edleafe maybe?
19:26:27 <edleafe> terrylhowe: I've been pulled in other directions the last few weeks, so haven't had time to do many reviews
19:27:03 <terrylhowe> yeh, I figued so
19:27:37 <terrylhowe> anyway, session docs aren’t so important
19:30:02 <terrylhowe> If we are all done with that for now
19:30:10 <terrylhowe> #link https://review.openstack.org/#/c/96516/
19:30:21 <terrylhowe> That was a test authenticator
19:31:04 <terrylhowe> I have some reswizzling I wanted to do in the example code that is going to follow this
19:31:23 <terrylhowe> where each component would have their own test file to avoid conflicts
19:31:37 <terrylhowe> transport, session, authenticate, …
19:32:02 <terrylhowe> that is next though
19:33:30 <jamielennox> any reason to have the test authenticator run via env variables?
19:33:45 <jamielennox> isn't it easier to just have a basic one that takes those params on __init__
19:34:47 <terrylhowe> yeh, that might be better jamielennox the only reason I added was inspiration from the session docs
19:35:53 <jamielennox> just thinking if you want to use that from elsewhere (other examples) you then need to mess with envs
19:36:21 <Alex_Gaynor> seems reasonable to me
19:39:14 <terrylhowe> well, feel free to -1 that one and I’ll come up with something better
19:40:00 <jamielennox> done
19:40:18 <terrylhowe> Alex_Gaynor: that is it for me
19:40:25 <Alex_Gaynor> Thank you.
19:40:51 <jamielennox> #link https://review.openstack.org/#/c/96253
19:40:55 <jamielennox> is Alex's
19:41:20 <terrylhowe> Petr is technically correct, not sure if I care though
19:41:22 <jamielennox> kind of agree you shouldn't put that requirements change in there - but not fussed
19:41:31 <jamielennox> :)
19:41:33 <Alex_Gaynor> I can move that to a seperate PR.
19:41:41 <Alex_Gaynor> Anyone know where the babel thing came from?
19:41:51 <Alex_Gaynor> I know what babel is, but we don't use it anywhere as far as I know
19:41:52 <jamielennox> it's in everything
19:41:59 <terrylhowe> no idea here
19:42:05 <jamielennox> i assume we probably got it from boilerplate
19:42:15 <Alex_Gaynor> ah, from cookiecutter
19:43:13 <terrylhowe> just approve it and be done I think
19:43:47 <jamielennox> yea, i'm ok
19:44:01 <jamielennox> actually you can edit commit messages from gerrit now
19:44:23 <jamielennox> so tag on 'Remove babel - wtf' and we'll just do it
19:45:55 <Alex_Gaynor> done
19:46:26 <jamielennox> and done
19:49:27 <Alex_Gaynor> Ok, I don't have any other outstanding business. ANyone else have stuff they want to bring up?
19:49:35 <jamielennox> in case it's useful to others i added this to bookmarks toolbar, so hopefully keep on top of reviews a bit better now
19:49:37 <jamielennox> https://review.openstack.org/#/q/project:stackforge/python-openstacksdk+label:Code-Review%253D0%252Cuser%253Dself+is:open+NOT+owner:self,n,z
19:51:45 <jamielennox> but otherwise, nice work terrylhowe
19:52:09 <terrylhowe> thanks
19:52:45 <terrylhowe> I have nothing else
19:52:52 <Alex_Gaynor> Cheers then, I guess we're done a few minutes early.
19:52:55 <Alex_Gaynor> Thanks for coming everyone!
19:53:52 <edleafe> Alex_Gaynor: don't forget #endmeeting
19:53:59 <Alex_Gaynor> edleafe: thanks
19:54:00 <Alex_Gaynor> #endmeeting