Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-11-29
20:25:08 mriedem just wanted to come by and say https://docs.openstack.org/keystoneauth/latest/using-sessions.html is nice
20:25:09 mriedem thanks
20:28:32 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: DNM Strawman stats abstraction https://review.openstack.org/620990
20:32:30 mordred mriedem: \o/
21:13:25 openstackgerrit Merged openstack/openstacksdk master: Check result of server metadata operation https://review.openstack.org/619478
#openstack-sdks - 2018-11-30
06:52:08 openstackgerrit Rabi Mishra proposed openstack/osc-lib master: Fix formatter handling for python 3.7 https://review.openstack.org/618137
07:13:53 openstackgerrit Filippo Inzaghi proposed openstack/python-openstackclient master: Add Python 3.6 classifier to setup.cfg https://review.openstack.org/621096
07:36:08 openstackgerrit Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139
08:26:15 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993
08:27:48 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993
09:23:03 gtema dtantsur|msg, are you here?
09:24:34 dtantsur|mtg gtema: I am
09:25:09 gtema dtantsur|msg: as a continue to removal of xxxDetails I have found similar issue in orchestrate
09:25:19 gtema but the difference is that it is not list, but create/update
09:25:30 dtantsur|mtg I think all Details are broken more or less
09:25:36 gtema basically StackPreview
09:26:14 gtema I feel it would be good to apply the same approach as for Details here, but this would mean extension for also other request types
09:26:30 gtema and extending basic proxy/resource/test_proxy
09:26:43 gtema this is however becoming relatively big
09:26:55 gtema I mean everywhere adding optional base_path
09:27:08 gtema wanted to ask, what is your opinion here?
09:28:04 dtantsur|mtg I need to think about it, I'm quite sleepy right now :) and I'm not familiar with orchestration enough
09:28:54 gtema it's not about orchestration itself. I am asking more about adding base_path optional parameter not only for "list", what you did for Details, but also for other request types
09:30:00 gtema use case: upon some parameter to proxy (i.e. preview) the endpoint for "create" is different
09:44:28 dtantsur|mtg gtema: this can be done in _prepare_request (used by everything except for list)
09:45:12 gtema sure, this is what I have done. But you would first need to also extend Proxy._create (and _update) and pass this optional parameter into Resource
09:45:46 gtema and this mean that test_proxy_base should be also extended with additional check of base_path for such calls
09:46:35 gtema that's what I ment "is getting big"
09:47:29 gtema and wanted to check before I land a patch not to face resistence
09:47:54 gtema s/resistence/resistance/
09:58:21 dtantsur|mtg the approach seems good to me at first glance, but I really need to see the code
09:58:37 gtema ok, than I will work further
09:58:46 gtema thanks
13:02:50 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
13:04:43 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
14:11:36 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
14:46:49 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
16:39:11 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
16:41:06 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
16:48:43 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
16:57:41 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154
18:40:11 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: WIP Fix version discovery for rackspace public cloud https://review.openstack.org/621257
18:49:27 kmalloc mordred: ^ that makes me sad.
18:49:44 Shrews kmalloc: yah, he's working on a better thilng
18:49:51 kmalloc i figured
18:49:53 kmalloc but man.
18:50:00 kmalloc gross.
18:50:54 kmalloc cmurphy: ^ cc
18:58:50 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fix version discovery for clouds with int project_ids https://review.openstack.org/621257
19:09:35 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fix version discovery for clouds with int project_ids https://review.openstack.org/621257
19:35:36 openstackgerrit Nathan Kinder proposed openstack/osc-lib master: Default to using public interface for all clients https://review.openstack.org/621276
20:00:23 mordred kmalloc: ^^ care to weigh in on that one?
20:01:32 kmalloc the most recent one?
20:01:50 kmalloc or the nkinder change?
20:02:11 kmalloc cause YES PUBLIC INTERFACES
20:03:05 kmalloc mordred: +1 (since i don't have +2 on that repo)
20:03:11 kmalloc but i'd +2 that if i could
20:18:38 mordred kmalloc: sweet. perhaps dhellmann or dtroyer will do the +3
20:21:24 dhellmann lgtm
21:14:40 larsks mordred: https://review.openstack.org/#/c/618519/ has two +2s and +verified, but it hasn't merged yet. Is something stuck?
21:15:04 mordred larsks: it's marked cannot merge
21:15:09 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: syntax errors and undefined exceptions in service_description.py https://review.openstack.org/618519
21:15:20 mordred larsks: I rebased it - let's see if that fixes the issue
21:16:09 larsks mordred: I must not have been looking at the right spot. Thanks!
21:16:31 mordred larsks: fingers crossed gerrit likes us this time
21:40:48 openstackgerrit Monty Taylor proposed openstack/promstat master: Add initial code https://review.openstack.org/621295
23:29:54 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Transform server with munch before normalizing https://review.openstack.org/621316
23:30:47 mordred Shrews: ^^
#openstack-sdks - 2018-12-01
06:08:39 openstackgerrit Merged openstack/openstacksdk master: syntax errors and undefined exceptions in service_description.py https://review.openstack.org/618519
07:16:25 openstackgerrit Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193
14:03:06 openstackgerrit Merged openstack/promstat master: Add initial code https://review.openstack.org/621295
16:25:42 openstackgerrit Merged openstack/keystoneauth master: Fix version discovery for clouds with int project_ids https://review.openstack.org/621257
16:37:11 openstackgerrit Merged openstack/openstacksdk master: Transform server with munch before normalizing https://review.openstack.org/621316
21:23:39 openstackgerrit Merged openstack/osc-lib master: Default to using public interface for all clients https://review.openstack.org/621276
22:02:44 openstackgerrit Gonéri Le Bouder proposed openstack/openstacksdk master: object_store: exposes the prefix parameter https://review.openstack.org/621381
#openstack-sdks - 2018-12-02
03:55:31 openstackgerrit Merged openstack/openstacksdk master: Fix some spelling in documentation https://review.openstack.org/618198
#openstack-sdks - 2018-12-03
09:21:31 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
10:25:53 openstackgerrit Merged openstack/openstacksdk master: Add missing properties for role https://review.openstack.org/614705
13:33:18 mordred dtantsur: does https://review.openstack.org/#/c/616227/ look ok to you now?
13:33:30 dtantsur lemme see
13:35:19 dtantsur +2
13:44:24 mordred \o/
14:02:24 Shrews mordred: for that ping from late friday for 621316, seems like _normalize_server() should handle that, no?
14:03:24 Shrews seems silly to have to Munch a thing before we call a thing that returns a Munch
14:03:32 mordred Shrews: well - normalize_server handles making a new munch - but previously the rest call itself had returned a munch rather than a dict ...
14:03:48 mordred but yeah - maybe the right answer there is to just munch the flavor?
14:04:09 Shrews mordred: what you have should work, it's just confusing to me
14:04:33 Shrews and is sort of a broken interface
14:05:47 Shrews we call _normalize_server() in multiple places, but only pre-munch in the one
14:06:11 Shrews mordred: so maybe it's still broken for certain paths
14:06:32 mordred oh - yeah. good point
14:07:00 mordred lemme try something different
14:08:37 Shrews okie
14:15:17 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Move server munch transformation into normalize https://review.openstack.org/621585
14:15:21 mordred Shrews: how's that?
14:16:30 Shrews mordred: looks like that would work
14:16:54 mordred woot
14:17:08 Shrews though, without a test, how do we know? ;)
14:24:49 mordred hahaha
14:24:53 mordred Shrews: MAGIC?

Earlier   Later