| *** xmltok has quit IRC | 00:40 | |
| *** rgbkrk has quit IRC | 01:24 | |
| *** rgbkrk has joined #openstack-sdks | 01:25 | |
| openstackgerrit | Shaunak Kashyap proposed a change to stackforge/openstack-sdk-php: Changes per reviews. https://review.openstack.org/92280 | 02:11 |
|---|---|---|
| *** _shweta_ has joined #openstack-sdks | 04:52 | |
| *** rgbkrk_ has joined #openstack-sdks | 05:25 | |
| *** rgbkrk has quit IRC | 05:25 | |
| *** rgbkrk_ has quit IRC | 05:51 | |
| *** samchoi has quit IRC | 10:43 | |
| *** _shweta_ has quit IRC | 10:49 | |
| *** jamie_h has joined #openstack-sdks | 11:52 | |
| *** bknudson has quit IRC | 13:11 | |
| *** mfer has joined #openstack-sdks | 13:32 | |
| *** bknudson has joined #openstack-sdks | 13:34 | |
| mfer | jamie_h sorry it's taken me a little bit to get to your email and reviews. we had a holiday weekend and then i was sick | 13:50 |
| mfer | i'm back now and I'll get to them starting today | 13:51 |
| jamie_h | mfer no worries - hope you're feeling better! | 13:51 |
| mfer | thanks. i'm 75% better. i'm interested in things other than napping and laying on the couch | 13:51 |
| mfer | food poisioning is no fun | 13:51 |
| openstackgerrit | A change was merged to stackforge/golang-client: Adding contributions guide. https://review.openstack.org/93774 | 13:52 |
| *** rgbkrk has joined #openstack-sdks | 14:05 | |
| openstackgerrit | Henrique Truta proposed a change to openstack/python-openstackclient: Add role assignments list support to identity v3 https://review.openstack.org/91634 | 14:17 |
| jamie_h | mfer what is the purpose of the swiftfs:// wrapper? | 14:18 |
| mfer | jamie_h acting like a filesystem. swift doesn't handle things like directories unless you tell it how to use a seperator. the swift:// stream didn't handle directories. the swiftfs:// one handles them (though likely it's a poor way of handling them). | 14:20 |
| jamie_h | mfer ah okay. if swift doesn't support some filesystem operations (like mkdir etc.) do we need to support it? would anybody use swiftfs:// ? | 14:22 |
| mfer | jamie_h ideally there would be one stream to rule them all. swiftfs was a take at handling directories. then the team was re-orged before we took fixing the problem further | 14:22 |
| jamie_h | yeah i agree about the 1 stream idea. i was never really sure whether we needed a local stream wrapper like swiftfs | 14:24 |
| mfer | swiftfs isn't local. it calls a remote endpoint. it just fakes directories at the remote endpoint to work with existing tools that expect a filesystem | 14:30 |
| mfer | at the time we'd found a lot of PHP apps used streams but simply expected them to always be a filesystem | 14:30 |
| mfer | it was bad development on their part but we wanted to make something that worked | 14:30 |
| jamie_h | I think we could support mkdir, rmdir and url_stat by referencing containers and pseudo dirs | 14:35 |
| jamie_h | ...instead of objects. these functions could be added to StreamWrapper and we could delete the FS version - what do you think? | 14:36 |
| mfer | jamie_h i agree. we just need to make the directory seperator configurable since it is in swift | 14:46 |
| mfer | and make it opt in or opt out. i'm not sure which | 14:46 |
| openstackgerrit | Henrique Truta proposed a change to openstack/python-openstackclient: Add role assignments list support to identity v3 https://review.openstack.org/91634 | 15:00 |
| mfer | jamie_h before our meeting today did you want to add anything to https://wiki.openstack.org/wiki/Meetings/OpenStack-SDK-PHP | 15:14 |
| mfer | glenc ^^ | 15:14 |
| jamie_h | just looking at it now | 15:14 |
| mfer | OpenStack SDK for PHP meeting in 5 minutes in #openstack-meeting-3 in 5 minutes | 15:27 |
| *** ycombinator has joined #openstack-sdks | 15:27 | |
| *** etoews has joined #openstack-sdks | 15:28 | |
| *** samchoi has joined #openstack-sdks | 15:29 | |
| *** etoews has quit IRC | 15:34 | |
| *** etoews has joined #openstack-sdks | 15:35 | |
| openstackgerrit | Terry Howe proposed a change to stackforge/python-openstacksdk: Add command structure to example code https://review.openstack.org/94707 | 15:43 |
| openstackgerrit | Terry Howe proposed a change to stackforge/python-openstacksdk: Add command structure to example code https://review.openstack.org/94707 | 15:44 |
| mfer | notmyname ping | 16:05 |
| notmyname | mfer: ack | 16:05 |
| *** xmltok has joined #openstack-sdks | 16:06 | |
| mfer | notmyname i was poking around the swift end user docs and finding a number of things not listed. Or, I think not listed. like working with object versioning. or, detecting what features and add ons are available. I was looking at http://docs.openstack.org/api/openstack-object-storage/1.0/content/ and http://developer.openstack.org/api-ref-objectstorage-v1.html | 16:06 |
| mfer | do you know of any better pointers so I can find more details? | 16:06 |
| notmyname | mfer: it may be that there aren't complete docs for those. beyond what you linked, you should look over the docs at http://swift.openstack.org | 16:10 |
| mfer | notmyname is there a way to detect what features are enabled? | 16:11 |
| notmyname | mfer: you should look at the "overview and concepts" section and the "middleware" section | 16:11 |
| notmyname | mfer: yes. make a request to /info | 16:11 |
| notmyname | mfer: http://docs.openstack.org/developer/swift/middleware.html#discoverability | 16:12 |
| mfer | thanks. do you have a person who keeps the api docs up to date? | 16:12 |
| notmyname | mfer: eg https://gist.github.com/notmyname/65d3b888f12810c5aaaf | 16:13 |
| mfer | notmyname thanks | 16:13 |
| notmyname | mfer: no. we don't have a designated "docs person" AFAIK | 16:14 |
| mfer | thanks | 16:16 |
| openstackgerrit | Henrique Truta proposed a change to openstack/python-openstackclient: Add role assignments list support to identity v3 https://review.openstack.org/91634 | 16:17 |
| *** etoews has quit IRC | 16:27 | |
| *** etoews has joined #openstack-sdks | 16:29 | |
| jamie_h | if we ended up using Guzzle's URL class, the only way you could swap it out would be to ask end-users to extend GuzzleHttp\Url | 16:31 |
| ycombinator | I guess what I'm thinking is that if we will /always/ pull in Guzzle as a dependency, why couldn't we also always use Guzzle's URL class | 16:31 |
| jamie_h | there is no URL interface | 16:31 |
| jamie_h | ycombinator that's my feeling too. There's a lot of potential, why not use it | 16:31 |
| ycombinator | the only time I see this being an issue is if we decide to one day drop Guzzle as a dependency | 16:32 |
| mfer | what if someone wants to use Guzzle 5 with our library but we don't support it yet? and, guzzle 5 has URL in a different location or with a different api? | 16:32 |
| ycombinator | or that ^ :) | 16:32 |
| jamie_h | yeah, that's a major pitfall too | 16:32 |
| mfer | if dependencies move at a different speed than our project and there are API changes we will run into problems. | 16:32 |
| jamie_h | which is why I thought an independent class is the lesser of two evils | 16:33 |
| ycombinator | right right | 16:33 |
| ycombinator | okay, I guess there's a tradeoff | 16:33 |
| jamie_h | I hate rolling our own, but there's not much choice with such a low-level component | 16:33 |
| mfer | i was recently helping someone using something older and it has issues. because of tight coupling they are in trouble and can't update to something newer | 16:33 |
| ycombinator | alright, I think I'm convinced now | 16:33 |
| mfer | i've run into dependency problems in far too many real world situations | 16:33 |
| ycombinator | tighter coupling is the greater evil here | 16:33 |
| jamie_h | with our own Url class, we can keep as succinct and focused as possible | 16:34 |
| ycombinator | so I guess we'll write our own | 16:34 |
| glenc | FYI, I'm on another call if it appears that I'm not paying attention much | 16:34 |
| mfer | my concern is supporting people with pain in the but problems due to dependency issues when their codebases age | 16:34 |
| ycombinator | okay, I think we have a decision | 16:36 |
| ycombinator | thanks for discussing my concerns | 16:36 |
| mfer | no problem. they are valid. i think this is just a time for prioritizing | 16:36 |
| mfer | and i hope we agree on priorities. if not, please speak up | 16:37 |
| ycombinator | yes, I'm satisfied with how we arrived at the decision and agree that moving forward is important | 16:37 |
| *** paybackman has joined #openstack-sdks | 16:43 | |
| *** jamie_h has quit IRC | 17:45 | |
| *** etoews has quit IRC | 17:50 | |
| *** etoews has joined #openstack-sdks | 17:51 | |
| *** etoews has quit IRC | 17:56 | |
| *** etoews has joined #openstack-sdks | 18:07 | |
| *** jamielennox is now known as jamielennox|away | 18:16 | |
| openstackgerrit | Henrique Truta proposed a change to openstack/python-openstackclient: Add role assignments list support to identity v3 https://review.openstack.org/91634 | 18:50 |
| Alex_Gaynor | Anyone around for a quick review on https://review.openstack.org/#/c/95876/ /cc briancurtin | 18:51 |
| dtroyer | Alex_Gaynor: are you tired of cleaning up after me yet? ;) | 19:13 |
| dtroyer | +A | 19:13 |
| openstackgerrit | A change was merged to stackforge/python-openstacksdk: Wrap lines at the appropriate length and use native sphinx constructs https://review.openstack.org/95876 | 19:18 |
| openstackgerrit | Terry Howe proposed a change to stackforge/python-openstacksdk: Authentication from keystoneclient https://review.openstack.org/91889 | 19:20 |
| openstackgerrit | Terry Howe proposed a change to stackforge/python-openstacksdk: Authentication from keystoneclient https://review.openstack.org/91889 | 19:21 |
| terrylhowe | factories are finally gone, that should keep Jamie happy for the time being | 19:23 |
| *** terrylhowe has quit IRC | 19:44 | |
| mfer | dtroyer did you get a look at my examples/acceptance changes for golang? | 19:46 |
| mfer | and the goroot comment? | 19:46 |
| dtroyer | my GOROOT comment was in regard to the suggested makefile includes that seem to no longer be present | 19:50 |
| dtroyer | include $(GOROOT)/src/Make.$(GOARCH) | 19:50 |
| dtroyer | include $(GOROOT)/src/Make.pkg | 19:50 |
| mfer | ah | 19:50 |
| mfer | i think that would be a different review. | 19:51 |
| * mfer punts discussing that. delegating to future self | 19:51 | |
| dtroyer | yeah…makefile stuff for later | 19:54 |
| mfer | other than that, what do you think? | 19:54 |
| dtroyer | it's running on my Mac! ;) so I got something fixed | 19:54 |
| mfer | lol | 19:54 |
| dtroyer | I thought I'd replied to that already, damn | 19:55 |
| mfer | not to my last comments. but, i know how that goes | 19:55 |
| dtroyer | +1 | 19:55 |
| mfer | thanks | 19:55 |
| dtroyer | I read them and think I wrote a reply and didn't hit send | 19:55 |
| mfer | i'm going to put in like 5% of my time on it. so, it'll take a little time but it will get there | 19:55 |
| *** rgbkrk has quit IRC | 19:56 | |
| dtroyer | one of the things I want to play with there is how to do CI testing…should be fun | 19:56 |
| dtroyer | btw, I'm taking off for a week here in an hour or two…first actual vacation in over a year... | 19:57 |
| mfer | yay! | 19:57 |
| mfer | i talked with mordred and he said we should be able to do testing for things like go on the infra now. i need to look into that for other things | 19:58 |
| dtroyer | I'm excited to test with something that isn't DevStack for once! although we'll need that here eventually too | 19:59 |
| *** rgbkrk has joined #openstack-sdks | 20:01 | |
| *** terrylhowe has joined #openstack-sdks | 20:15 | |
| openstackgerrit | A change was merged to stackforge/golang-client: Adding examples that can optionally be used as acceptance tests. https://review.openstack.org/94709 | 20:21 |
| openstackgerrit | Alex Gaynor proposed a change to stackforge/python-openstacksdk: Flesh out the README a bit more https://review.openstack.org/96253 | 20:35 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/96269 | 21:00 |
| openstackgerrit | Alex Gaynor proposed a change to stackforge/python-openstacksdk: Update sphinx from global-requirements https://review.openstack.org/96278 | 21:05 |
| *** mfer has quit IRC | 21:09 | |
| Alex_Gaynor | terrylhowe: Fastest review in the west! | 21:21 |
| terrylhowe | I was just goofing with that for OSC :) | 21:22 |
| *** TravT has joined #openstack-sdks | 21:43 | |
| *** rgbkrk has quit IRC | 21:44 | |
| *** rgbkrk has joined #openstack-sdks | 21:45 | |
| *** dhellmann is now known as dhellmann_ | 21:58 | |
| *** dhellmann_ is now known as dhellmann | 21:58 | |
| *** dhellmann is now known as dhellmann_ | 21:58 | |
| openstackgerrit | A change was merged to openstack/python-openstackclient: Fix server image create https://review.openstack.org/92864 | 22:05 |
| *** bknudson has quit IRC | 22:16 | |
| *** rgbkrk has quit IRC | 22:42 | |
| Alex_Gaynor | r? https://review.openstack.org/#/c/96253/ and https://review.openstack.org/#/c/96278/ | 22:45 |
| *** etoews has quit IRC | 22:58 | |
| openstackgerrit | A change was merged to openstack/python-openstackclient: Fixed several typos throughout the codebase https://review.openstack.org/94636 | 23:05 |
| *** rgbkrk has joined #openstack-sdks | 23:10 | |
| *** etoews has joined #openstack-sdks | 23:25 | |
| *** etoews has quit IRC | 23:29 | |
| *** etoews has joined #openstack-sdks | 23:49 | |
| *** xmltok has quit IRC | 23:51 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!