| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-30 | |||
| 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? | |
| 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 | possible via github-pull-request. Can you tell me what is the easiest way to do this? Regards, Tino. | |
| 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: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 | |