Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-01-11
18:36:54 elmiko sorry cdent, was otherwise occupied :|
18:37:05 cdent no problem
19:53:17 adriant mordred: as long as we make the defaults sensible, and give the user control over them, I'm game. My problem is that if I give it incorrect params, or even correct params thatthe API accepts but we haven't coded in yet, and the code gobbles them up and doesn't tell me...
19:54:21 adriant I'll be screwed when i realise the query I got back includes a list of everything, and none of the filters were actually passed along.
19:55:00 adriant throwing an error at least tells me to submit a patch to add missing filters to the SDK, or that I made a typo and threw in the wrong filter.
19:55:14 adriant silent failure = pain
19:57:09 adriant like I was using 0.9.19 which switched identity to resource2 but didn't yet have the identity filters merged, so when trying to filter project list by name I got back a full list, which... confused me since resource1 just passed the whole dict through.
19:59:42 adriant mordred: as new services add filters, people will try using them via the SDK sometimes before we add them there, so we either need to make it clear we don't support them yet (error), or a way to explicitly bypass which filters we configure and just pass the filters through raw. The latter of which really could just be a case of: "do a raw API cal
19:59:42 adriant l" if we don't want to add too much complexity.
20:07:00 openstackgerrit Merged openstack/osc-lib master: Fix find() interface when attr is not specified https://review.openstack.org/529934
21:03:19 openstackgerrit Merged openstack/python-openstackclient master: Add floating IP qos_policy actions https://review.openstack.org/517845
21:14:49 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Re-enable octavia functional tests https://review.openstack.org/531514
#openstack-sdks - 2018-01-12
02:41:20 openstackgerrit Jake Yip proposed openstack/python-openstackclient master: Fix error with image show when image name is None https://review.openstack.org/529464
04:21:31 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Raise error when supplying invalid query params https://review.openstack.org/532723
04:39:04 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Raise error when supplying invalid query params https://review.openstack.org/532723
04:55:12 adriant mordred, briancurtin: I think this now mostly makes sense: https://review.openstack.org/#/c/532723
05:39:06 adriant also, has anyone yet started migrating object_store to resource2 ? I can potentially do that if needed.
13:48:56 briancurtin adriant: go for it
15:42:40 jeremyfreudberg mordred, I've put up a preliminary patch for Sahara APIv2 stuff in python-saharaclient https://review.openstack.org/#/c/533066/
15:43:10 jeremyfreudberg in my part time -ness i've only had time to do things the hacky way
15:43:17 jeremyfreudberg but that should make it easier for your to swoop in
15:44:07 mordred jeremyfreudberg: awesome! I'll open it up and look in just a sec!
15:45:06 jeremyfreudberg mordred: appreciate it
16:50:30 mordred adriant: awesome, thanks! patch looks great - and I think it leaves the door open to later adding the fancier thing I was suggesting - your patch makes things safe at least :)
16:51:01 mordred adriant: also, yes, what briancurtin said - would love help migrating object_store - although if you don't get to it in the next day or two it's high up there on my list
17:20:57 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Add factory helper function which returns a singleton https://review.openstack.org/533250
18:41:43 openstackgerrit Merged openstack/python-openstacksdk master: Prefer links dicts for pagination https://review.openstack.org/530769
18:44:58 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Merge user and users sections of the docs https://review.openstack.org/532633
18:44:58 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rationalize logging helpers and docs https://review.openstack.org/532634
18:44:59 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add ability to register non-official services https://review.openstack.org/533263
18:44:59 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove meter service https://review.openstack.org/533264
18:45:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove metric service https://review.openstack.org/533265
21:42:00 openstackgerrit Merged openstack/python-openstacksdk master: Add reno for tag support on heat stacks https://review.openstack.org/527381
23:29:21 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Merge user and users sections of the docs https://review.openstack.org/532633
23:29:22 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rationalize logging helpers and docs https://review.openstack.org/532634
23:29:22 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add ability to register non-official services https://review.openstack.org/533263
23:29:23 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove meter service https://review.openstack.org/533264
23:29:23 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove metric service https://review.openstack.org/533265
23:29:24 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove message v1 support https://review.openstack.org/533322
23:29:24 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Migrate object_store to resource2/proxy2 https://review.openstack.org/533323
23:30:34 mordred adriant: I started on object_store ^^
23:31:44 mordred adriant: I didn't get a chance to touch tests at all - but I figured I'd push it up in case you felt like hacking - I'm about out for the evening, feel free to take that patch over
23:31:54 mordred adriant: or to ignore it
23:38:21 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Base devstack sanity check on os_service_types https://review.openstack.org/533325
23:38:54 mordred johnsom: ^^ did a followup to your patch taking a stab at using ost for that devstack test - no clue if that'll work :)
23:39:48 johnsom Cool!
#openstack-sdks - 2018-01-13
01:14:05 openstackgerrit Merged openstack/python-openstacksdk master: Re-enable octavia functional tests https://review.openstack.org/531514
05:36:03 openstackgerrit Merged openstack/python-openstacksdk master: Remove legacy client factory functions https://review.openstack.org/532347
06:15:43 adriant mordred: will take a look at it monday my time when I'm back at work. I can try it out, potentially fix some tests, or at least leave some review.
06:15:56 adriant I am so very very much looking forward to 1.0 of the SDK
17:33:38 openstackgerrit Merged openstack/osc-lib master: Relocate utils tests to match the main code https://review.openstack.org/529956
#openstack-sdks - 2018-01-14
06:12:04 openstackgerrit Bar Elharar proposed openstack/osc-lib master: Suppress subTest() expected errors https://review.openstack.org/531025
14:09:14 openstackgerrit Merged openstack/cliff master: Remove empty files https://review.openstack.org/530674
#openstack-sdks - 2018-01-15
00:32:15 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Raise error when supplying invalid query params https://review.openstack.org/532723
01:10:19 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: change update/create method options https://review.openstack.org/533438
01:13:06 adriant mordred: ^ this one here makes the swift stuff a little nicer
01:13:20 adriant since we can remove the custom update method and just use the resource2 one properly
01:14:30 adriant am working on the object_store patch. Some tests are passing, but there is more actual logic to fix first as well (hence the above patch)
02:21:49 openstackgerrit lei zhang proposed openstack/js-openstack-lib master: Remove the deprecated "giturl" option https://review.openstack.org/533444
02:25:13 openstackgerrit lei zhang proposed openstack/python-openstacksdk master: Remove the deprecated "giturl" option https://review.openstack.org/533449
03:14:21 adriant mordred: ugh that's not nice.... objects in swift have last-modified coming from body and headers
03:14:49 adriant and in resource 2 we pop that so it doesn't turn up in the headers one.
03:14:51 adriant hmmmm
03:34:00 adriant although... no they are different keys
03:34:44 adriant mordred: the one in body is 'last_modified' and headers is 'last-modified', you just entered the wrong key into the body when you defined it :P
03:35:41 adriant but that is confusing as hell.
03:35:57 adriant if you GET the object, the last_modified value is last_modified_at
03:36:08 adriant but a list of objects will have it at last_modified
03:36:19 adriant so we really need an alias or something between them
03:44:48 openstackgerrit lei zhang proposed openstack/js-openstack-lib master: Remove the deprecated "giturl" option https://review.openstack.org/533444
03:50:27 openstackgerrit lei zhang proposed openstack/python-openstacksdk master: Remove the deprecated "giturl" option https://review.openstack.org/533449
06:24:06 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: change update/create method options https://review.openstack.org/533438
06:24:07 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Merge user and users sections of the docs https://review.openstack.org/532633
06:24:07 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Rationalize logging helpers and docs https://review.openstack.org/532634
06:24:08 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Add ability to register non-official services https://review.openstack.org/533263
06:24:08 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Remove meter service https://review.openstack.org/533264
06:24:09 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Remove metric service https://review.openstack.org/533265
06:24:09 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Remove message v1 support https://review.openstack.org/533322
06:24:10 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Migrate object_store to resource2/proxy2 https://review.openstack.org/533323
06:24:28 adriant sorry about the spam... I had to rebase a thing.
06:25:51 adriant mordred: I did some wory on the object_store patch. Can't continue right away, but hopefully that at least gets you there part of the way.
06:26:09 adriant the test_proxy tests aren't passing, but everything else is...
06:26:25 adriant so I go part of the way there.
06:28:24 adriant Also, I changed allow_get=False for objects, since well... it doesn't make sense unless we write a variant of _translate_response that stores the returned data in the data field since it would otherwise act the same way as HEAD, and you then added download which kind of makes calling get directly fairly silly.
08:21:01 openstackgerrit Dinesh Bhor proposed openstack/keystoneauth master: Split request logging into four different loggers https://review.openstack.org/505764
10:36:35 openstackgerrit Dmitry Tantsur proposed openstack/api-wg master: Add guideline on exposing microversions in SDKs https://review.openstack.org/532814
10:46:51 openstackgerrit Dmitry Tantsur proposed openstack/api-wg master: Add guideline on exposing microversions in SDKs https://review.openstack.org/532814
10:54:01 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
10:54:54 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
11:00:38 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/533610
14:04:28 cdent edleafe, dtantsur : distant early warning that I will be travelling during the next api-sig meeting
14:04:41 dtantsur ack
14:08:01 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
15:09:55 openstackgerrit Andreas Jaeger proposed openstack/os-api-ref master: Fix wrong links https://review.openstack.org/501079
15:10:06 openstackgerrit Andreas Jaeger proposed openstack/os-api-ref master: Fix wrong links https://review.openstack.org/501079
15:13:23 openstackgerrit Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/533610
15:30:09 openstackgerrit Merged openstack/os-service-types master: Add factory helper function which returns a singleton https://review.openstack.org/533250
21:12:59 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Split request logging into four different loggers https://review.openstack.org/505764
21:13:00 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Remove PYTHONHASHSEED setting https://review.openstack.org/533798
22:23:32 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: [WIP] Support filtering Floating IP by device_id https://review.openstack.org/533809

Earlier   Later