Earlier  
Posted Nick Remark
#openstack-sdks - 2019-04-17
15:10:39 mordred json=self.fake_search_return),
15:10:43 mordred but that's just me :)
15:11:18 mordred in any case - I am very supportive of the problem you want to solve
15:11:30 mordred quibbling about impl details notwithstanding
15:11:39 efried cool. Finding someone to do the work any time in the near future, that's going to be the tough part.
15:11:49 mordred yeah. but maybe we can!
15:11:58 efried I guess I should file a story or something.
15:13:14 mordred efried: mildly related - where's the nova ptg schedule? I believe there's an sdk related topic and I want to make sure it's on my calendar so I don't forget about it
15:13:45 efried mordred: Other than slots for three or so of the xproj sessions, I haven't put it together yet.
15:14:59 mordred ok. cool. I'll keep my eyes out :)
16:06:49 efried mordred, dustinc_away: https://storyboard.openstack.org/#!/story/2005475
17:54:56 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
18:17:52 elmiko edleafe, cdent, wanted to give you a heads up, etoews is giving a talk about the work he's doing in about 45 minutes, url for video conference https://bluejeans.com/8933049334
18:18:16 elmiko the general topic has been relayed as gitops
18:18:42 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Prevent setUpClass failures in volume v1 tests https://review.openstack.org/648760
18:18:42 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
18:18:45 cdent blast from the past
18:18:49 elmiko yeah totally!
18:19:00 elmiko when i saw his name popup on my email i thought i had to check it out
18:19:15 elmiko i guess the openshift commons group invited him to give a talk
18:19:23 cdent In about 45 minutes I hope to be buried in my dinner
18:19:50 edleafe That's a novel way of disposing of a corpse
18:20:14 cdent need something to eat for the long journey
18:21:08 elmiko lol
18:21:30 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
18:21:41 elmiko cdent: as a followup to your request from earlier, i think the sharding story makes sense at the level presented in the doc
18:22:02 elmiko i think there will be plenty of technical challenges, but in general i like the /idea/
18:22:43 cdent elmiko: does a header seems like the sensible route?
18:23:08 elmiko i think so, for me it has the lowest impact on things that are already installed
18:23:23 elmiko i can't imagine modifying the body of requests, well i can but it's not pretty
18:23:51 elmiko the other option would be something in the uri, but i /really/ don't like that
18:24:28 elmiko imo, unless someone comes with a better idea, the header seems like a good first approach
18:25:31 cdent cool, thanks
18:25:39 elmiko np =)
18:26:17 cdent i'm hoping that mordred is going to have something to say about a universal "this is openstack X" identifier that is already in the works
18:28:15 elmiko wouldn't that be lovely
19:00:53 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
19:52:29 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
21:23:23 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
22:53:46 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
22:54:52 efried mordred, dustinc_away: FYI, filed a blueprint for openstacksdk-in-nova (https://blueprints.launchpad.net/nova/+spec/openstacksdk-in-nova) and updated topics and commit messages in the nova and sdk patches to refer to it https://review.openstack.org/#/q/topic:bp/openstacksdk-in-nova+(status:open+OR+status:merged)
22:57:57 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
#openstack-sdks - 2019-04-18
08:08:52 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
08:11:09 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Update baremetal to use proxy logger https://review.openstack.org/652083
09:41:56 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register https://review.openstack.org/637238
10:30:43 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
13:03:55 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Remove now unused task_manager file https://review.openstack.org/653731
13:43:15 gtema mordred: do you have an opinion on https://review.openstack.org/#/c/651534/12/openstack/resource.py@425?
13:43:41 gtema do we want to support this logic everywhere?
13:46:55 gtema dtantsur: https://review.openstack.org/#/c/651534/12/openstack/image/v1/image.py@103 - an existing logic to give possibility to the requestor to self iterate over the result
13:47:16 mordred gtema: yes - I replied
13:47:42 gtema thanks
13:48:22 gtema I can also move "output" logic under the Image.download method if we want it now
13:48:53 mordred yeah - I think that's a good idea
13:49:24 gtema ok. Will also add few tests on the proxy.download_image (were missing so far)
13:50:10 dtantsur gtema: then why not change the second place to just 'return resp'?
13:50:34 gtema hmm
13:50:48 gtema maybe
13:54:24 gtema what do we want for chunk_size?
13:54:44 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Support for the baremetal introspection service https://review.openstack.org/650044
13:54:51 dtantsur mordred: a new service ^^^ \o/
13:55:09 dtantsur gtema: I'd put at least 4k, but I don't have any reliable reasoning behind it
13:55:15 dtantsur ideally, we should benchmark it :)
13:56:07 gtema :D Benchmark will actually depend on the chosen setup/platform/image/etc
14:01:44 gtema dtantsur: returning resp instead of content breaks current behavior. This is not something I wanted to get with this change
14:17:04 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
14:21:47 dtantsur gtema: ah, I see
14:49:09 openstackgerrit Merged openstack/openstacksdk master: Move Orchestration methods into Proxy https://review.openstack.org/649273
15:02:38 openstackgerrit Merged openstack/openstacksdk master: Allow replacing service implementation https://review.openstack.org/648704
16:00:10 elmiko API SIG office hour is now open!
16:00:30 edleafe Yay!!!!
16:00:34 elmiko \o/
16:00:51 elmiko ooh la la
16:01:41 edleafe I did want to go over some outstanding api-sig patches, if that's ok with you
16:02:10 elmiko sure, i'm game!
16:02:15 edleafe I think the DNS guideline is ready for freeze https://review.openstack.org/#/c/651222/
16:02:30 elmiko ++
16:02:47 edleafe So is the version discovery doc, but it has a merge conflict https://review.openstack.org/#/c/459405/
16:03:00 elmiko i guess we should just send custom freeze emails to the ml since we don't have the newsletter?
16:03:22 elmiko oh wow, so exciting to see the version discovery finally getting there =)
16:03:38 edleafe Generally that's what the tools do: send an email to the liaisons, and add them to the reviewer list for the patch
16:04:04 elmiko ok, sweet
16:04:06 edleafe But sure, we could also send out an email
16:04:29 elmiko i suppose we don't have to, i was just curious and apparently the cob-webs have gotten the better of that part of my memory XD
16:04:47 edleafe dtantsur had a suggestion on the liaisons file patch https://review.openstack.org/#/c/645223/
16:04:59 edleafe Once I fix that, I think that that should be a quick approve
16:05:21 edleafe I.e., it's not changing a guideline or anything
16:05:35 elmiko ok, cool. i will remove my +2 with a note just for clarity sake
16:05:42 elmiko i agree with dtantsur's idea
16:05:42 edleafe schweet
16:06:28 edleafe I thought that the tags update was ready, but after thinking a bit more about Akihiro's comment, I want to revisit it https://review.openstack.org/#/c/642497/
16:07:05 edleafe And I was going to fast-approve the emai list name change https://review.openstack.org/#/c/632255/1
16:07:25 edleafe but it was stated that it was a dupe without referencing what it is a dupe of
16:07:31 edleafe Do you have any idea?
16:07:58 elmiko edleafe: i agree about Akihiro's comment, thanks for taking that on
16:08:05 dtantsur edleafe: I don't see a dup, let's approve it..
16:08:46 edleafe I just posted a reply to the dupe patch a few minutes ago. There's no rush, so if there is no good answer by next week, let's approve it then
16:09:49 elmiko edleafe: agreed
16:10:57 edleafe That's all I had on my plate to discuss
16:11:26 elmiko edleafe ++

Earlier   Later