Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-03-19
13:14:47 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Use define version parameter replace service.version https://review.openstack.org/554223
13:50:46 openstackgerrit Ed Leafe proposed openstack/api-wg master: Break up the HTTP guideline into smaller documents https://review.openstack.org/554234
14:00:28 elmiko cdent: missed your ping on friday, where was that meesage?
14:01:08 cdent elmiko: sigs list and dev list
15:23:23 EmilienM dtroyer: hey, could I get a review on https://review.openstack.org/#/c/553374/ please?
16:03:18 elmiko cdent: weird... somehow it dropped through my filter
16:03:46 cdent elmiko: gilles origina message was cc'd to all of us, which might have thrown things off
17:22:23 openstackgerrit Ed Leafe proposed openstack/api-wg master: Break up the HTTP guideline into smaller documents https://review.openstack.org/554234
20:29:41 dhellmann dtroyer : if we're going to do https://review.openstack.org/#/c/538335/ we probably want to do it early in the cycle so we can deal with anything it breaks
20:31:00 dtroyer dhellmann: thanks for the reminder, agreed
20:31:25 elmiko cdent: yup, caught up now
20:31:35 cdent elmiko: cool, thanks
20:31:47 elmiko i would have replied as i started looking at mugsie's stuff and hope to be up to speed with it by the meeting this week
20:31:50 dhellmann dtroyer : I'm not even sure who else has +2 there any more to ask for for reviews.
20:32:07 elmiko but, i don't think that changes any of the stuff you said, which btw i ++
20:33:31 cdent elmiko: I'm relieved. I was trying to be balanced but honest
20:42:28 openstackgerrit Mohammed Naser proposed openstack/os-client-config master: Add image_format for VEXXHOST profile https://review.openstack.org/554351
20:45:22 mnaser could a very kind sdk-core have a quick eye on this simple change ^
20:51:07 elmiko cdent: yeah, i'm not really sure what more we can do. i think providing the venue and helping keep the ball rolling is a big part of our goal, but like you suggested, we can't really assign work to people. the api schema topic is not new
20:57:28 elmiko cdent edleafe dtantsur|afk re: meeting times and apac friendliness. i wonder, if instead of changing the meeting time, we could offer to hold an office hours time slot that would be more appropriate?
20:57:57 elmiko like, i'm just wondering what can we do to help Gilles out and also not set ourselves up for failure in the meeting dept.
20:58:15 cdent elmiko: I think an office hour is likely a good idea anway, but I think the core issue is that things need to happen asynch if they want to happen
20:58:25 elmiko ++
20:59:01 elmiko i feel like we have a great potential contributor/participant in Gilles, but for lack of hours in the day we can't make something happen. i'm just spitballing about how we help
20:59:26 elmiko the other part too is what does Gilles want as output?
21:00:53 cdent yes, that's what my final question to him was sort of about
21:08:13 elmiko hopefully it all works out, i'd like to see the api-schema stuff pick up some momentum
21:13:30 cdent elmiko: you should probalby tell Gilles that, otherwise it might seem he's standing sort of solo, which is not the case
21:23:28 elmiko cdent: ack, i will reply. any suggestions for replying from a digest copy of the email?
21:24:39 elmiko ah, nvm. i'll reply the individual email
21:24:47 cdent you have both?
21:26:45 elmiko yeah, i think my email filter separated them out somehow
21:26:56 elmiko (i just switched to our new hosted email, it's driving me nuts)
21:28:08 elmiko well, we'll see if you actually get it lol. bbl, dinner
#openstack-sdks - 2018-03-20
00:21:58 EmilienM hi, any chance to have a review on https://review.openstack.org/#/c/553374/ please
02:50:08 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Fix 'block_store' aliases define error https://review.openstack.org/554403
07:09:02 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Fix 'block_store' aliases define error https://review.openstack.org/554403
07:25:15 openstackgerrit Peter Balogh proposed openstack/python-openstacksdk master: Fix TypeError in case of FloatingIP add and remove https://review.openstack.org/552112
07:26:56 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Fix 'block_store' aliases define error https://review.openstack.org/554403
10:28:06 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Use defined version instead of service.version parameter https://review.openstack.org/554223
12:36:21 mugsie elmiko: if you have any questions, please give me a ping :)
12:38:52 elmiko mugsie: ack, will do!
12:45:18 cdent elmiko, edleafe, anything blocking merging those microversion-parse changes? You guys +1d, but you should have the power to +2 if you want. I'd like to release a new version, even if its not yet perfect (as long as it doesn't break stuf, that's good enough)
12:49:28 elmiko cdent: nothing blocking from me
12:49:52 elmiko gimme like an hour and i can start adding the +2s
12:50:01 elmiko unless edleafe gets to it first ;)
12:50:16 cdent cool, thanks, no huge rush, just got reminded of it by my to do list
12:59:17 elmiko ack
13:08:45 edleafe cdent: elmiko: I had one little comment, but added a +2. elmiko, if you agree, wanna push it through?
13:09:50 elmiko edleafe: sounds good!
13:53:25 elmiko cdent: i +w to the 2 that had other support
13:53:38 elmiko i'd prefer more than just my lone review on these:
13:53:40 elmiko https://review.openstack.org/#/c/550251/3
13:53:51 elmiko https://review.openstack.org/#/c/550263/3
13:54:01 elmiko https://review.openstack.org/#/c/550264/3
13:54:27 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Do not use redundant attr check for keys in fold_headers https://review.openstack.org/554547
13:55:11 cdent elmiko: yeah. edleafe I added another patch to the end fixing the bug.
13:55:29 elmiko cool
13:55:36 elmiko is that the one you just posted?
13:55:39 cdent yeah
13:55:44 elmiko looked sensible
13:59:29 edleafe cdent: cool. I would have just fixed it in place, but... meh
13:59:43 openstackgerrit Merged openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356
13:59:49 cdent edleafe: for a change that didn't check that method at all? dirty!
14:00:31 openstackgerrit Merged openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212
14:01:17 edleafe Guess I'm a dirty guy
14:02:18 cdent Newflash: edleafe states well known fact, wins t-shirt
14:02:46 edleafe \o/
14:02:56 edleafe My wardrobe needs sprucing up
14:04:12 openstackgerrit Jens Harbott (frickler) proposed openstack/keystoneauth master: Be more verbose when version discovery fails https://review.openstack.org/554044
14:05:29 frickler kmalloc: ^^ just for you ;)
14:10:25 elmiko cdent: LOL
14:20:45 cdent edleafe: use your dirty old guy wisdom to fix that attribute problem on Version, I responded
14:28:20 elmiko that sounds like the best super power ever
14:36:58 openstackgerrit Merged openstack/os-client-config master: Add image_format for VEXXHOST profile https://review.openstack.org/554351
14:47:23 mnaser mordred: did you ever get around this? https://specs.openstack.org/openstack/openstack-specs/specs/clouds-yaml-support.html
14:47:50 mnaser i have a user who likes using python-openstackclient because clouds.yaml, but image member list is missing from it right now, so they have to use glanceclient
14:48:49 mordred mnaser: we should just fix python-openstackclient - I've given up on that spec in favor of thinking we should delete python-*client
14:49:21 mnaser mordred: ok cool, i'll try to push a patch to add that support i guess
14:49:30 mnaser mordred: accepting and rejecting memberships is there, just listing is missing
14:49:53 edleafe cdent: who said I'm old??
14:49:59 mordred edleafe: who didn't?
14:50:30 edleafe cdent: answered
14:56:31 kmalloc frickler: fantastic!
14:56:36 kmalloc frickler: thanks for the test!
14:57:06 kmalloc mordred: can we please delete python-*-client?
14:57:10 kmalloc mordred: please? :)
14:57:22 mordred kmalloc: ++
15:02:04 dtantsur elmiko: office hours is not a bad idea, but I also think that we should encourage people to talk async and on public ML
15:02:28 dtantsur e.g. I won't be able to attend apac-friendly times, I suspect
15:02:36 dtantsur or rather: either me or the US
15:03:39 edleafe dtantsur: early APAC is late for me
15:04:37 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Improve Version.matches() method https://review.openstack.org/550251
15:04:38 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Add MicroversionMiddleware https://review.openstack.org/550252
15:04:38 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Allow passing a json_error_formatter to the middleware https://review.openstack.org/550263
15:04:39 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Clean up package version and python version support https://review.openstack.org/550264
15:04:39 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Do not use redundant attr check for keys in fold_headers https://review.openstack.org/554547
15:04:50 cdent there ya go edleafe, I think. and thanks.
15:07:39 mnaser i'm working on adding support for glance image list
15:07:53 mnaser i'm getting this when trying to run commands in tox venv (tox -evenv -- openstack image list)

Earlier   Later