| terrylhowe | I see what you mean jamielennox I was originally thinking the resource would have a service identifier, but that doesn't cover the region and visibility stuff. I'll have to think about it some more. | 00:07 |
|---|---|---|
| jamielennox | terrylhowe: if the dict is stored on the class it will have the same problem, but in keystoneclient we were generating it | 00:08 |
| terrylhowe | it really seems that visibility and region should be had by the authenticator | 00:09 |
| terrylhowe | normally, at least | 00:09 |
| jamielennox | region yes | 00:10 |
| jamielennox | visibility not always | 00:10 |
| jamielennox | there should be a default on authenticator | 00:11 |
| jamielennox | but there are certain resources that are admin URL only | 00:11 |
| jamielennox | terrylhowe: i think i had the default on session rather than authenticator | 00:11 |
| jamielennox | authenticator might be shared between different objects doing different regions | 00:12 |
| jamielennox | terrylhowe: i have an issue though you might be able to help me with | 00:12 |
| jamielennox | i think we need to abstract the operations part of the resource somewhat | 00:13 |
| jamielennox | say for example i have user has roles, but roles is also global | 00:14 |
| terrylhowe | I have to run, family is waiting on me, but I'll be back in an hour or so, sorry | 00:14 |
| jamielennox | terrylhowe: no worries | 00:14 |
| terrylhowe | write down your thoughts here if you like | 00:14 |
| jamielennox | ok | 00:16 |
| jamielennox | so take tenant has user has roles tenant, user, roles are all top level resource | 00:17 |
| jamielennox | http://docs.openstack.org/api/openstack-identity-service/2.0/content/GET_listRolesForUserOnTenant_v2.0_tenants__tenantId__users__userId__roles_Tenant_Operations.html | 00:18 |
| jamielennox | so it's a Roles.list operation (without filters) but the base_url is dynamic | 00:19 |
| jamielennox | i'm thinking at the moment that we need an operations object | 00:20 |
| jamielennox | so that Role.list() would do: ListOp(Role, cls.base_url).perform(presentation, limit=None, marker=None) | 00:22 |
| jamielennox | and user.roles() could do return ListOp(Role, '/tenants/{id}/users/{id}/roles').perform(presentation) | 00:24 |
| jamielennox | i'm also thinking that there might be a reason for a Resource to override there own ListOp (maybe they have custom filters) | 00:26 |
| jamielennox | i'm just not sure how it all works | 00:27 |
| *** VeggieMeat_ has joined #openstack-sdks | 00:54 | |
| *** VeggieMeat has quit IRC | 00:54 | |
| *** TravT has quit IRC | 00:57 | |
| *** samchoi has quit IRC | 01:16 | |
| *** mfer has joined #openstack-sdks | 01:33 | |
| *** mfer has quit IRC | 01:59 | |
| *** terrylhowe has quit IRC | 02:42 | |
| *** etoews has quit IRC | 02:51 | |
| *** etoews has joined #openstack-sdks | 03:36 | |
| *** etoews has quit IRC | 03:40 | |
| *** mfer has joined #openstack-sdks | 03:51 | |
| *** mfer has quit IRC | 03:56 | |
| *** etoews has joined #openstack-sdks | 05:37 | |
| *** etoews has quit IRC | 05:45 | |
| *** WhatTheFoley has joined #openstack-sdks | 05:46 | |
| *** WhatTheFoley has quit IRC | 05:51 | |
| *** etoews has joined #openstack-sdks | 07:42 | |
| *** etoews has quit IRC | 07:50 | |
| *** jamie_h has joined #openstack-sdks | 08:23 | |
| *** etoews has joined #openstack-sdks | 08:46 | |
| *** etoews has quit IRC | 08:51 | |
| openstackgerrit | Jamie Hannaford proposed a change to stackforge/openstack-sdk-php: Remove 'credits' file from codebase because it's not necessary or standard https://review.openstack.org/91305 | 09:03 |
| *** etoews has joined #openstack-sdks | 09:50 | |
| *** etoews has quit IRC | 09:54 | |
| *** etoews has joined #openstack-sdks | 10:39 | |
| *** etoews has quit IRC | 10:43 | |
| *** openstackgerrit has quit IRC | 10:51 | |
| *** jamie_h has quit IRC | 11:07 | |
| *** jamie_h has joined #openstack-sdks | 11:08 | |
| *** VeggieMeat_ is now known as VeggieMeat | 11:20 | |
| *** mfer has joined #openstack-sdks | 12:51 | |
| *** bknudson has joined #openstack-sdks | 13:22 | |
| *** terrylhowe has joined #openstack-sdks | 13:27 | |
| terrylhowe | It just seems like the easy cop out way to solve the tenant/users/roles problem would be make some ProjectUsers resource jamielennox | 13:52 |
| *** openstackgerrit has joined #openstack-sdks | 13:56 | |
| *** wchrisj has joined #openstack-sdks | 14:21 | |
| jamie_h | mfer I'm looking at our ClientInterface - could we separate out the process of creating a Request from sending it? | 14:57 |
| jamie_h | sometimes users might want to create a Request first, add a custom header, and then send with a separate method | 14:58 |
| jamie_h | right now doRequest creates and sends | 14:58 |
| mfer | jamie_h hey | 15:03 |
| mfer | so, i'd like to keep interfaces as simple as possible while providing everything. | 15:04 |
| mfer | with doRequest you can set headers | 15:04 |
| mfer | can you share a use case to separate it? | 15:04 |
| wchrisj | elight you around today? | 15:06 |
| wchrisj | elight ... or travelling? | 15:07 |
| terrylhowe | briancurtin jamielennox bknudson edleafe wchrisj alright if I approve Dean's example code https://review.openstack.org/#/c/87859 | 15:07 |
| jamie_h | mfer Requests can emit events. You might want to attach a custom subscriber, say for error handling or logging | 15:08 |
| elight | wchrisj: traveling...l | 15:08 |
| terrylhowe | Otherwise, someone is going to have to explain to me how to cherry pick that over to my presentation stuff | 15:08 |
| terrylhowe | I'd like to update the examples to get a better feel for higher level design | 15:08 |
| jamie_h | right now we're *assuming* that users want to immediate send all requests and never modify or edit them beforehand | 15:08 |
| jamie_h | it's 1 extra method that separates concerns - I don't think it overcomplicates the interface | 15:09 |
| jamie_h | but I agree, we need to keep as simple as possible | 15:09 |
| terrylhowe | I should of had dtroyer on that too | 15:09 |
| mfer | jamie_h it's easy to add lots of things that might seem useful someday. i'd like to know some real world use cases for the added feature. i'm not saying we shouldn't do it. | 15:10 |
| mfer | this is an SDK so the goal should be to keep is simple and easy for the long tail of developers to use | 15:10 |
| jamie_h | mfer It isn't adding lots of things, it's taking a way an assumption. I've already mentioned a real-world case: adding a subscriber that attaches itself to a request. One example could be a subscriber that retries the request if the API responds with a 500/401 error | 15:11 |
| jamie_h | separating into 2 methods isn't adding complexity - it's removing assumptions and making things clearer IMO | 15:12 |
| jamie_h | Guzzle already does this, as does ZF2 | 15:12 |
| mfer | jamie_h i'm on a call until the meeting starts. let me look a little more into this | 15:13 |
| *** samchoi has joined #openstack-sdks | 15:18 | |
| mfer | PHP SDK meeting in #openstack-meeting-3 in 5 minutes | 15:25 |
| mfer | #startmeeting openstack-sdk-php | 15:30 |
| openstack | Meeting started Wed Apr 30 15:30:22 2014 UTC and is due to finish in 60 minutes. The chair is mfer. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:30 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:30 |
| openstack | The meeting name has been set to 'openstack_sdk_php' | 15:30 |
| terrylhowe | oops | 15:30 |
| mfer | Can everyone please state your name and any applicable association. | 15:30 |
| mfer | #endmeeting | 15:30 |
| openstack | Meeting ended Wed Apr 30 15:30:55 2014 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:30 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_sdk_php/2014/openstack_sdk_php.2014-04-30-15.30.html | 15:30 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_sdk_php/2014/openstack_sdk_php.2014-04-30-15.30.txt | 15:31 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_sdk_php/2014/openstack_sdk_php.2014-04-30-15.30.log.html | 15:31 |
| mfer | it's going to be one of those days I think | 15:31 |
| openstackgerrit | A change was merged to stackforge/openstack-sdk-php: Remove 'credits' file from codebase because it's not necessary or standard https://review.openstack.org/91305 | 16:12 |
| *** TravT has joined #openstack-sdks | 16:18 | |
| *** WhatTheFoley has joined #openstack-sdks | 16:24 | |
| *** etoews has joined #openstack-sdks | 16:24 | |
| mfer | jamie_h btw, i'm doing the HP legal thing now. If you add a new file by all means put the openstack one in place | 16:35 |
| jamie_h | mfer okay - how about if an existing file is changed? | 16:36 |
| mfer | i'm not sure how this works. this isn't licensing but copyright and those are different. basically, i need to talk with lawyers first to understand | 16:36 |
| mfer | jamie_h got a sec, i'd like to run something past you | 16:40 |
| jamie_h | mfer sure, go for it | 16:40 |
| mfer | jamie_h i'd like to bring in a 3rd party to chair the meeting. i have someone in mind. so we don't debate technical things while I chair. | 16:41 |
| mfer | it feels ackward for me | 16:41 |
| jamie_h | okay - I have no problem with that :) | 16:42 |
| mfer | great | 16:42 |
| jamie_h | can I run by you some of the changes I've thought about today? Are you free right now or shall I send an e-mail to openstack-dev? | 16:44 |
| jamie_h | minor tweaks to ClientInterface, etc. | 16:44 |
| mfer | i'm about to go into another meeting. can you send an email? i'll get to it in a bit but I bet it'll be in the evening for you | 16:45 |
| jamie_h | okay, I'll stick to e-mail - has the added benefit of keeping those not here in the loop too | 16:46 |
| *** etoews has quit IRC | 16:57 | |
| *** wchrisj has quit IRC | 17:00 | |
| *** jamielennox is now known as jamielennox|away | 17:00 | |
| *** WhatTheFoley has quit IRC | 17:24 | |
| *** wchrisj__ has quit IRC | 17:29 | |
| *** WhatTheFoley has joined #openstack-sdks | 17:40 | |
| *** wchrisj has joined #openstack-sdks | 18:25 | |
| *** samchoi has quit IRC | 18:30 | |
| *** jamie_h has quit IRC | 18:31 | |
| *** TravT|2 has joined #openstack-sdks | 19:10 | |
| *** TravT has quit IRC | 19:11 | |
| *** mfer has quit IRC | 19:32 | |
| *** mfer has joined #openstack-sdks | 19:45 | |
| terrylhowe | Alex_Gaynor briancurtin jamielennox bknudson edleafe wchrisj dtroyer https://review.openstack.org/#/c/91448/ | 20:06 |
| terrylhowe | Updated example code with presentation | 20:06 |
| *** TravT|2 has quit IRC | 20:12 | |
| terrylhowe | I'm kind of wondering what people think about putting something like the connection class in the SDK https://review.openstack.org/#/c/91448/1/examples/connection.py | 20:15 |
| -openstackstatus- NOTICE: the gate is backed up due to broken nodepool images, fix in progress (eta 22:00 utc) | 20:26 | |
| *** ChanServ changes topic to "the gate is backed up due to broken nodepool images, fix in progress (eta 22:00 utc)" | 20:26 | |
| *** TravT has joined #openstack-sdks | 20:30 | |
| *** etoews has joined #openstack-sdks | 20:48 | |
| *** etoews has quit IRC | 21:05 | |
| *** etoews has joined #openstack-sdks | 21:05 | |
| *** etoews has quit IRC | 21:19 | |
| *** bknudson has quit IRC | 22:29 | |
| *** mfer has quit IRC | 22:29 | |
| *** bknudson has joined #openstack-sdks | 22:49 | |
| *** bknudson has quit IRC | 22:54 | |
| *** jamielennox|away is now known as jamielennox | 23:01 | |
| *** bknudson has joined #openstack-sdks | 23:09 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!