Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-01-05
14:54:13 briancurtin Dinesh_Bhor: yeah, if you do sort_key=[list, of, things] it will be fine
14:55:21 Dinesh_Bhor briancurtin: ok, thanks
14:57:03 Dinesh_Bhor briancurtin: bdw I have one more doubt if you have time then
14:58:02 briancurtin Dinesh_Bhor: sure, what’s up
14:58:15 Dinesh_Bhor briancurtin: If I pass any invalid sort_key to the servers then the nova-api returns the proper 400 but at the openstacksdk side I can just see "HttpException: Bad Request"
14:58:56 Dinesh_Bhor briancurtin: But at nova-api it has returned exception with message: Returning 400 to user: Sort key supplied was not valid.
14:59:35 Dinesh_Bhor briancurtin: conn.compute.servers(sort_key=['nadame', 'created_at'])
14:59:38 openstackgerrit Jens Rosenboom proposed openstack/python-openstackclient: Fix creating a private flavor with ID auto https://review.openstack.org/416940
15:01:20 Dinesh_Bhor briancurtin: It looks like here https://github.com/openstack/python-openstacksdk/blob/master/openstack/session.py#L74 we are not getting anything in the message as well as in details
15:02:48 briancurtin yeah, a lot of projects do a lot of different/bad/wrong things with what they return. it’s probably best if you report a bug to https://bugs.launchpad.net/python-openstacksdk and myself or someone can take a look at it
15:03:25 Dinesh_Bhor briancurtin: ok, Can I take up this task?
15:03:40 briancurtin Dinesh_Bhor: sure, if you’d like to
15:04:08 Dinesh_Bhor briancurtin: ok, thanks I will push my patch soon
15:09:17 briancurtin Dinesh_Bhor: take a look at https://review.openstack.org/#/c/386440/ which did something about this same issue seen in Neutron, but it was reverted due to causing other issues. Qiming may know more about the other issues, i can’t remember what they were
15:13:45 Dinesh_Bhor briancurtin: ok, thanks I will take a look at it
15:15:26 Qiming Dinesh_Bhor, one thing worth a try is to wrap line 77 in that patch into a try...except statement
15:15:56 Qiming not all exceptions are returning things that can be converted to a JSON
15:16:33 Qiming that patch was reverted because it is over simplifying the situation
15:17:15 Dinesh_Bhor Qiming: can we do something like this: https://github.com/openstack/python-glanceclient/blob/master/glanceclient/exc.py#L156
15:17:55 Qiming looks nicer
15:19:09 Qiming just I'm not sure if there will always be a content-type header when an exception is raised
15:22:28 stevemar edleafe: thanks!
17:03:45 openstackgerrit Merged openstack/api-wg: Trivial fix: correct names of capitalization styles https://review.openstack.org/411391
17:15:32 cdent elmiko or etoews or edleafe can you proofread https://etherpad.openstack.org/p/api-wg-newsletter please
17:18:30 edleafe cdent: made on quick change to add the review link. Thinking of lazy people who might not be up to 2 clicks
17:18:58 cdent edleafe: I did that on purpose. I didn't want people to read the review without looking at the etherpad :)
17:19:24 cdent context is everything, after all
17:19:47 cdent but yeah, your change is fine
17:20:03 edleafe and here I thought that looking at the review might spark interest in the issue, causing them to go back and read the whole etherpad discussion :)
17:20:31 cdent I think my bias is showing: I want the reading of the etherpad to predispose the person reading the review...
17:20:42 cdent I'll ship what we've got showing now unless you've got addtional changes
17:21:49 cdent thar she blows
17:25:53 cdent damn, found an old typo after sending it
18:02:23 elmiko cdent: sorry, missed the ping
18:02:40 cdent elmiko: no worries, edleafe and I figured it out.
18:02:58 elmiko awesome =)
18:34:20 openstackgerrit Dean Troyer proposed openstack/python-openstackclient: Functional tests - flavor https://review.openstack.org/417132
18:45:10 stevemar dtroyer: so i guess you're around? :)
19:03:24 dtroyer hi ho!
20:02:05 sshank dtroyer, I have a question regarding adding '--option' (example: https://review.openstack.org/#/c/387611/28/doc/source/command-objects/network-agent.rst@39). Initially we are asked to change to have a --option. Please confirm which way to follow.
20:04:36 sshank dtroyer, ^^
20:07:44 sshank dtroyer, In patch: https://review.openstack.org/#/c/385729/8/doc/source/command-objects/network-agent.rst@85 --router is suggested. But later in patch: https://review.openstack.org/#/c/387611/ it is suggested to not have --option.
20:16:55 dtroyer I don't see a —option anywhere?
20:18:04 openstackgerrit Ian Cordasco proposed openstack/api-wg: Define pagination guidelines https://review.openstack.org/390973
20:18:05 dtroyer you may be getting confused between resources, this review has two different things it is addressing
20:18:41 dtroyer I'm heading out for a while, will be back later today to look at this again
20:18:48 sshank dtroyer, Okay.
20:18:56 sshank dtroyer, Thanks.
20:30:37 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/417175
23:24:55 openstackgerrit Merged openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/417175
23:28:12 openstackgerrit Eric Brown proposed openstack/keystoneauth: Remove references to Python 3.4 https://review.openstack.org/417236
#openstack-sdks - 2017-01-06
01:21:50 openstackgerrit Merged openstack/keystoneauth: Remove references to Python 3.4 https://review.openstack.org/417236
02:56:22 openstackgerrit Merged openstack/python-openstacksdk: Add filter "user_id" for cluster receiver list https://review.openstack.org/416159
04:50:40 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient: Fix "ip availability show" command https://review.openstack.org/417286
05:00:53 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient: Fix "ip availability show" command https://review.openstack.org/417286
05:38:16 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient: Fix network service provider functional test https://review.openstack.org/417289
05:57:36 openstackgerrit Merged openstack/python-openstackclient: Fix creating a private flavor with ID auto https://review.openstack.org/416940
06:23:49 openstackgerrit Zhou Zhihong proposed openstack/python-openstackclient: Add options to allow filtering on agent list https://review.openstack.org/398017
07:48:57 openstackgerrit Abhishek Kekane proposed openstack/keystoneauth: Log request-id for each api call https://review.openstack.org/392442
10:52:34 openstackgerrit jiahui.qiang proposed openstack/python-openstackclient: functional test for volume https://review.openstack.org/417349
12:55:06 openstackgerrit Xinhui Li proposed openstack/python-openstacksdk: Add workflow service (mistral) https://review.openstack.org/414919
13:07:59 openstackgerrit Xinhui Li proposed openstack/python-openstacksdk: Add workflow service (mistral) https://review.openstack.org/414919
14:09:18 openstackgerrit Ed Leafe proposed openstack/api-wg: Clarify the status values in versions https://review.openstack.org/411849
14:34:08 openstackgerrit David Stanek proposed openstack/keystoneauth: Add __ne__ built-in function https://review.openstack.org/398294
15:08:53 openstackgerrit jiahui.qiang proposed openstack/python-openstackclient: Add --project and --project-domain option to "volume snapshot list" https://review.openstack.org/417410
15:13:25 openstackgerrit jiahui.qiang proposed openstack/python-openstackclient: Fix filter error in os volume list https://review.openstack.org/416852
16:36:08 openstackgerrit Ed Leafe proposed openstack/api-wg: Add guideline for invalid query parameters https://review.openstack.org/417441
16:44:00 openstackgerrit Shashank Kumar Shankar proposed openstack/python-openstackclient: Introduce Neutron DHCP agent commands to OSC. https://review.openstack.org/387611
17:40:14 openstackgerrit Shashank Kumar Shankar proposed openstack/python-openstackclient: Introduce Neutron DHCP agent commands to OSC. https://review.openstack.org/387611
17:58:50 openstackgerrit Merged openstack/python-openstackclient: Fix "ip availability show" command https://review.openstack.org/417286
18:12:48 openstackgerrit Merged openstack/keystoneauth: Add __ne__ built-in function https://review.openstack.org/398294
18:53:17 openstackgerrit Sean Dague proposed openstack/service-types-authority: add ideas on naming https://review.openstack.org/286090
18:53:18 openstackgerrit Sean Dague proposed openstack/service-types-authority: add proposal on non official types https://review.openstack.org/286091
18:53:18 openstackgerrit Sean Dague proposed openstack/service-types-authority: add initial stub for service types https://review.openstack.org/286089
18:57:17 openstackgerrit Sean Dague proposed openstack/service-types-authority: add proposal on non official types https://review.openstack.org/286091
18:57:55 openstackgerrit Merged openstack/service-types-authority: add ideas on naming https://review.openstack.org/286090
19:01:23 openstackgerrit Ankur proposed openstack/python-openstacksdk: Network L3 Router Commands https://review.openstack.org/385728
19:01:56 openstackgerrit Ankur proposed openstack/python-openstackclient: Network L3 Router Commands for OSC https://review.openstack.org/385729
19:02:53 openstackgerrit Sean Dague proposed openstack/service-types-authority: add proposal on non official types https://review.openstack.org/286091
19:02:54 openstackgerrit Sean Dague proposed openstack/service-types-authority: add initial stub for service types https://review.openstack.org/286089
19:11:03 sdague cdent et al - I merged https://review.openstack.org/28609 because it had 4 +2s but was just in the wrong place in the patch chain
19:11:32 cdent wrong linke sdague ?
19:12:58 sdague oh, missing 0
19:13:02 sdague https://review.openstack.org/286090
19:13:23 cdent yup, rad, good choice
19:52:52 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient: set chunk size to 65536 when saving objects https://review.openstack.org/417490
20:00:39 openstackgerrit Goutham Pacha Ravi proposed openstack/service-types-authority: Add share as service type https://review.openstack.org/311808
20:16:36 openstackgerrit Dean Troyer proposed openstack/python-openstackclient: Functional tests - flavor https://review.openstack.org/417132
20:39:24 openstackgerrit Dean Troyer proposed openstack/openstackclient: Initial commit for the OSC metapackage repo. https://review.openstack.org/414380
21:12:46 openstackgerrit Merged openstack/osc-lib: Add deprecated_option_warning() method to Command https://review.openstack.org/416665
21:45:46 stevemar dtroyer: you're the best
21:55:10 dtroyer stevemar: WTF happened to the IDP tests? consistently failing most of the afternoon
21:55:59 stevemar dtroyer: grr
21:56:12 stevemar oh fudge
21:57:00 dtroyer it's all the same thing around domain_id somehwere?
21:58:41 stevemar dtroyer: we now tag a domain_id to the idp if you don't specify one
21:58:47 stevemar that commit just merged
21:58:52 stevemar let me fix
22:00:42 openstackgerrit Merged openstack/openstackclient: Initial commit for the OSC metapackage repo. https://review.openstack.org/414380
22:02:33 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient: add domain id to expected IdP fields https://review.openstack.org/417515

Earlier   Later