Earlier  
Posted Nick Remark
#openstack-sdks - 2018-11-30
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?
14:25:51 Shrews so much magic
15:10:56 openstackgerrit Merged openstack/openstacksdk master: Slightly refactor vendor profile loading https://review.openstack.org/616227
15:13:35 openstackgerrit Merged openstack/osc-lib master: Fix formatter handling for python 3.7 https://review.openstack.org/618137
15:25:27 openstackgerrit Merged openstack/python-openstackclient master: Add Python 3.6 classifier to setup.cfg https://review.openstack.org/621096
15:36:36 tis_at_teuto_net Hi there, i had a problem to perform an create-request for openstack.block_storage.v2.type.Type. The volume-type was created, but the attribute 'os-volume-type-access:is_public' was not set. Further i was unable to query non-public voume-types, because the required flag is not set in the get-request. I've changed two classes to get things working and i wanne give back the two small changes. I saw that this is not
15:36:36 tis_at_teuto_net possible via github-pull-request. Can you tell me what is the easiest way to do this? Regards, Tino.
15:39:23 frickler tis_at_teuto_net: for starters see https://docs.openstack.org/infra/manual/developers.html , feel free to ask again here when you get stuck
15:40:59 tis_at_teuto_net Thanks.
15:59:01 openstackgerrit Tino Schmeier proposed openstack/openstacksdk master: - added query-parameters to filter for non-public volume-types. Needs ?is_public=None https://review.openstack.org/621617
16:18:47 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Support non-public volume types https://review.openstack.org/621617
16:19:28 mordred frickler: ^^ there's that patch from tis_at_teuto_net - I updated the commit message to fix the formatting, but otherwise the patch lgtm
16:24:01 mordred Shrews: feel like reviewing https://review.openstack.org/#/c/616228/ ? I even added a test!
16:29:44 Shrews mordred: yeah, in a bit. knee deep in debugging some new code amt
16:29:46 Shrews atm
16:30:19 mordred Shrews: kk. do the important things
16:44:32 Shrews mordred: ok, found my bug(s). gonna change locations and will review your change
16:46:28 mordred woot!
16:52:54 frickler mordred: I need to look at that patch in more detail tomorrow. it does seem to me like it could use some solid testing, though
18:10:05 Shrews mordred: that's a neat change
18:10:14 mordred Shrews: \o/
18:17:39 mordred mrhillsman: heya - just responded to your comment on 616228 - but it felt like I might be rambling - let me know if that makes sense or sounds like a raving lunatic :)
18:18:34 mrhillsman ah ok
18:18:54 mordred mrhillsman: also - I'm sad I missed that session! too many good things going simultaneously
18:19:27 mrhillsman i did not catch wind of the openstacksdk to publiccloud-wg discussion
18:20:03 mrhillsman so i thought it was initiated by the publiccloud-wg (could still have been) and only saw the rfc5785 proposal
18:20:15 mordred lemme see if I can dig up the thread
18:21:40 mordred mrhillsman: http://lists.openstack.org/pipermail/openstack-dev/2018-November/136260.html
18:21:52 mrhillsman i was just wondering what other sdk stakeholders thought of it
18:22:15 mrhillsman if they already had some implementation in place
18:22:28 mordred mrhillsman: it's a great question - I'd definitely love any feedback people have
18:24:26 mrhillsman ++
18:26:12 mordred but these profiles are already a sort of de facto thing since they're what's consumed by both SDK and OSC - and I'm not aware of any alternate collections of cloud vendor profiles in the other sdks ... hopefully this will get us away from http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/config/vendors and instead to things like https://vexxhost.com/.well-known/openstack/api
18:29:12 mrhillsman yeah, i believe gophercloud supports this, not sure of the others unfortunately
18:45:07 mordred yeah. I'm hoping if we can get them better defined we can get them supported more broadly
19:13:57 openstackgerrit Merged openstack/openstacksdk master: Move server munch transformation into normalize https://review.openstack.org/621585
20:34:28 openstackgerrit Yuval Shalev proposed openstack/openstacksdk master: Add host aggregate missing functions https://review.openstack.org/621695
20:36:05 openstackgerrit Yuval Shalev proposed openstack/openstacksdk master: Add host aggregate missing functions https://review.openstack.org/621695
21:32:42 umbSublime Hi, Is this the right channel to ask about openstack-client features ?
21:41:58 slaweq mordred: hi
21:42:24 mordred hey slaweq !
21:42:25 slaweq mordred: I'm trying to get osprofiler reports for neutron API calls when using openstackclient
21:42:44 mordred slaweq: that sounds like a worthy thing to want to do :)
21:42:50 slaweq mordred: and it looks for me that our SDK don't send X-Trace-Info header like it should
21:42:59 slaweq do You know something about it? is it a bug/
21:43:02 slaweq bug?
21:44:54 mordred I'm gonna bet it's a bug - I don't think we've ever implemented support for x-trace-info
21:46:14 slaweq it works for e.g. nova or glance commands in OSC but it's probably because nova/glance client bindings are used there instead of SDK
21:46:30 slaweq and for neutron SDK is used and that don't work
21:46:42 slaweq ok, I will try to find out how to fix it in SDK then
21:46:44 slaweq thx
22:10:14 mordred slaweq: sweet. I'd hope it's a thing we can do once at a low level in the proxy/adapter layer or something
22:10:20 mordred slaweq: but I look forward to learning more
#openstack-sdks - 2018-12-04
03:13:00 openstackgerrit Jason SUN proposed openstack/python-openstackclient master: Change openstack-dev to openstack-discuss https://review.openstack.org/621914
09:06:12 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
09:20:55 openstackgerrit LIU Yulong proposed openstack/python-openstackclient master: Supports router gateway IP QoS https://review.openstack.org/622130
10:47:51 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
11:08:20 openstackgerrit Andriy Shevchenko proposed openstack/service-types-authority master: Change openstack-dev to openstack-discuss https://review.openstack.org/622218
11:37:41 openstackgerrit Merged openstack/service-types-authority master: Change openstack-dev to openstack-discuss https://review.openstack.org/622218
12:18:19 openstackgerrit Merged openstack-infra/shade master: change default python 3 env in tox to 3.5 https://review.openstack.org/617151
12:19:38 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154

Earlier   Later