| *** etoews has joined #openstack-sdks | 00:20 | |
| *** etoews has quit IRC | 00:25 | |
| *** wchrisj has joined #openstack-sdks | 00:56 | |
| *** wchrisj has quit IRC | 01:32 | |
| *** etoews has joined #openstack-sdks | 03:03 | |
| *** etoews has quit IRC | 03:10 | |
| *** al_ex has joined #openstack-sdks | 06:42 | |
| *** jamielennox is now known as jamielennox|away | 07:18 | |
| *** al_ex has quit IRC | 07:48 | |
| *** al_ex has joined #openstack-sdks | 07:50 | |
| *** sahilsinha has joined #openstack-sdks | 10:35 | |
| *** mfer has joined #openstack-sdks | 12:55 | |
| *** bknudson has quit IRC | 13:06 | |
| *** HenryG has quit IRC | 13:21 | |
| *** etoews has joined #openstack-sdks | 13:24 | |
| *** wchrisj has joined #openstack-sdks | 13:58 | |
| *** wchrisj has quit IRC | 14:31 | |
| *** wchrisj has joined #openstack-sdks | 14:32 | |
| *** bknudson has joined #openstack-sdks | 14:41 | |
| *** HenryG has joined #openstack-sdks | 14:48 | |
| *** mhagedorn has joined #openstack-sdks | 15:33 | |
| *** etoews_ has joined #openstack-sdks | 15:38 | |
| *** etoews has quit IRC | 15:38 | |
| *** al_ex has quit IRC | 15:39 | |
| *** mhagedorn has quit IRC | 16:19 | |
| *** mhagedorn has joined #openstack-sdks | 16:24 | |
| *** WhatTheFoley has joined #openstack-sdks | 17:20 | |
| *** mhagedorn_ has joined #openstack-sdks | 17:28 | |
| *** mhagedorn has quit IRC | 17:29 | |
| *** mhagedorn_ is now known as mhagedorn | 17:29 | |
| *** etoews_ has quit IRC | 18:03 | |
| *** etoews has joined #openstack-sdks | 18:03 | |
| *** mhagedorn_ has joined #openstack-sdks | 18:14 | |
| *** mhagedorn has quit IRC | 18:16 | |
| *** mhagedorn_ is now known as mhagedorn | 18:16 | |
| *** mhagedorn_ has joined #openstack-sdks | 18:34 | |
| *** mhagedorn has quit IRC | 18:37 | |
| *** mhagedorn_ is now known as mhagedorn | 18:37 | |
| *** terrylhowe has joined #openstack-sdks | 19:24 | |
| *** satiar has joined #openstack-sdks | 19:29 | |
| elight | mhagedorn wchrisj: did you guys settle on an approach to fog-hp-tng? | 20:07 |
|---|---|---|
| mhagedorn | elight.. the assumption is that the inheritance branch is the go forward | 20:07 |
| wchrisj | elight do you mean composition vs inheritance? | 20:07 |
| elight | I mean that I did this last week: https://github.com/fog-openstack-tng/fog-rackspace-tng/blob/master/lib/fog/rackspacetng/identity.rb | 20:07 |
| elight | So I’m reusing ServiceDiscovery and subclassing IdentityV1 and V2 in mine | 20:08 |
| elight | wchrisj mhagedorn: https://github.com/fog-openstack-tng/fog-rackspace-tng/blob/master/lib/fog/rackspacetng/services/identity_v2.rb | 20:08 |
| wchrisj | elight So you guys dont do any sort of recoping with an existing token? | 20:15 |
| wchrisj | rescoping | 20:15 |
| elight | wchrisj: Nope | 20:15 |
| wchrisj | k | 20:15 |
| elight | One user, one tenant | 20:15 |
| elight | Which is why I was so damned confused when I first saw “tenant” in OpenStack. | 20:15 |
| wchrisj | yeah | 20:16 |
| wchrisj | elight How is "osc_options_from" used? Isn't apparent in the code... | 20:17 |
| elight | Uhhhhh oops? ;-) | 20:17 |
| wchrisj | It isn't used AFAICT | 20:17 |
| wchrisj | ok | 20:18 |
| elight | Bear in mind this is *very* WIP | 20:18 |
| wchrisj | cool - understood | 20:18 |
| wchrisj | like the simplicity | 20:18 |
| elight | I have a sneaking suspicion there’s a commit maybe I didn’t make somewhere. I don’t know how it would’ve ever worked withuot calling it on the args to ServiceDiscovery | 20:18 |
| wchrisj | yeah | 20:18 |
| mhagedorn | elight: yours is very slightly different than mine.. investigating…. | 20:19 |
| mhagedorn | elight 99% same | 20:19 |
| elight | mhagedorn: Hah. Good. | 20:19 |
| elight | mhagedorn: I don’t think I peeked at yours other than how you had your files organized. | 20:20 |
| elight | mhagedorn: I mostly wanted to try to connect the dots we laid out in OSC | 20:20 |
| mhagedorn | yours a bit simpler which I like | 20:20 |
| elight | Mine also doesn’t do much yet. ;-) | 20:20 |
| elight | Well... | 20:20 |
| elight | Once it connects, it will do whatever our cloud supports from OSC Identity. | 20:21 |
| elight | wchrisj: Hah. That is definitely not everything I wrote Friday. I remember having to make osc_options_from a class method. Odd. | 20:23 |
| mhagedorn | elight… as it turns out to be feature compatible with :hp…. (cant speak for rax) it doesnt have to do much (Identitywise) | 20:23 |
| elight | And I sync this code to myself over dropbox. odd. | 20:23 |
| wchrisj | Good to have different eyes looking at this - would like there to be as little code as possible in our extension layer.. | 20:24 |
| *** rgbkrk has joined #openstack-sdks | 20:30 | |
| elight | wchrisj mhagedorn: Ok, updated it. Working again | 20:44 |
| elight | Weird…. | 20:44 |
| elight | I don’t even know how it worked last week. I distinctly remember not having a 3 arg create_token method... | 20:44 |
| wchrisj | what do you mean? with thnant? | 20:45 |
| wchrisj | tenant | 20:45 |
| wchrisj | ? | 20:45 |
| elight | yeah | 20:48 |
| elight | Oh… I see. | 20:52 |
| elight | Yep. I didn’t need osc_options_from at all. | 20:53 |
| elight | I’m adapting the args hash inside the individual Identity service by version | 20:53 |
| elight | I suppose it makes sense. Each version may need to do something different. That is, I have no idea what v1 needs. ;-) | 20:53 |
| *** mfer has quit IRC | 21:01 | |
| *** krames has joined #openstack-sdks | 21:03 | |
| *** sahilsinha has quit IRC | 21:26 | |
| elight | wchrisj: “DEFAULT_VERSION = 2” is going to be odd for services that are still v1. Isn’t that most of them? I’m thinking Storage now but anything other than Compute and Auth? | 21:39 |
| elight | maybe I’m wrong | 21:39 |
| elight | (about the other service versions) | 21:39 |
| elight | wchrisj: Ooooh. So we’re currently forcing the service class names to be versioned under all circumstances. Hmm…. | 21:47 |
| *** WhatTheFoley has quit IRC | 21:51 | |
| *** krames has quit IRC | 22:03 | |
| *** satiar has quit IRC | 22:18 | |
| wchrisj | elight That default version should probably not even be in there | 22:29 |
| wchrisj | in retrospect | 22:30 |
| elight | wchrisj: It should be an argument. Like how Hash#fetch works | 22:30 |
| wchrisj | yeah | 22:31 |
| wchrisj | but, it should prob be svc cat driven for all but Identity... | 22:31 |
| elight | And if it's not provided, and a version cannot be determined, it raises. | 22:31 |
| wchrisj | what do you think of that? | 22:31 |
| elight | Look at Hash#fetch's docs. Almost just like that | 22:31 |
| wchrisj | a bit of a deviation from now | 22:31 |
| elight | Well... Ok, you could hardcore default versions for each service, I suppose. | 22:32 |
| elight | But it means there's something else to change when I add a new service | 22:32 |
| elight | That's a nuisance | 22:32 |
| wchrisj | yeah, I agree with the fetch sentiment | 22:32 |
| elight | Yeah. Fetch handles that by making ages of everything and then just providing the logic to operate on them | 22:33 |
| wchrisj | well, that's my point; for every service except identity, the SC should contain all th detail needed | 22:33 |
| wchrisj | dontcha think? | 22:33 |
| elight | But I have to change SC whenever I add a new service. I don't see the value. | 22:33 |
| wchrisj | for what? version? | 22:34 |
| elight | Why can't Identity say, for instance, tell SC that my default, if you can't determine one, is 2? | 22:34 |
| wchrisj | agreed | 22:34 |
| elight | SC becomes a function. It has no state. | 22:34 |
| wchrisj | ok | 22:34 |
| elight | And that's mostly how we use it now. | 22:34 |
| wchrisj | ok | 22:34 |
| elight | Also allows us to get rid of that internal array of supported services. | 22:35 |
| elight | And hell even that first arg, I think | 22:35 |
| wchrisj | not sure how we are disagreeing... ;-) | 22:35 |
| elight | I think we're agreeing.... | 22:35 |
| wchrisj | except for th initial hit against identity, we have to be explicit | 22:35 |
| wchrisj | for that one | 22:35 |
| wchrisj | then for every other service the SC will drive it | 22:36 |
| elight | ok I'm not tracking. ca we talk about it on hangout tomorrow? have to boogie soon. | 22:36 |
| wchrisj | boogie on, my man... | 22:37 |
| wchrisj | ;-) | 22:37 |
| *** rgbkrk has quit IRC | 23:05 | |
| *** jamielennox|away is now known as jamielennox | 23:06 | |
| *** bknudson has quit IRC | 23:12 | |
| *** wchrisj has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!