Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-01-31
21:30:57 mordred adriant: awesome
21:31:35 adriant https://github.com/openstack/python-openstacksdk/blob/master/openstack/proxy.py#L43
21:31:36 adriant https://github.com/openstack/python-openstacksdk/blob/master/openstack/proxy2.py#L18
21:32:33 mordred adriant: hah. yes. that is, indeed, broken
21:32:49 mordred although amusingly the patch to rename BaseProxy to Proxy would fix that
21:32:50 adriant and after fixing that i get: AttributeError: 'module' object has no attribute 'Body'
21:33:26 mordred adriant: what's trying to call in through Proxy2?
21:33:47 adriant lemme see
21:34:39 mordred adriant: on master for me, openstack network list --external gets through to where it's supposed to fail
21:35:20 adriant oh ffs
21:35:23 adriant I think i see my issue
21:35:57 adriant ugh, the random fucking pyc files for 'meter' didn't get cleaned up
21:36:13 adriant i'm very confused by that...
21:36:20 adriant I nuked my venv
21:36:48 mordred I hate leftover .pyc files
21:38:59 adriant *sigh* and other random cruft leftover from using "python setup.py install" ages ago with an old version that included meter code :/
21:39:04 adriant works now...
21:44:40 adriant mordred: for that query mapping fix
21:45:17 adriant right now, if I just add the values to my checking, and nothing else, the 'values' data takes precedence over the real expected key if both are supplied
21:45:42 adriant are you ok with my making it so that if you supply both, we take the one we prefer them to sue?
21:45:43 adriant use*
21:45:59 adriant e.g.is_router_external > router:external
21:46:44 mordred adriant: yah - I think that seems right - if you supply both, we should have the 'official' thing wins
21:47:01 adriant mordred: currently would do it the other way https://github.com/openstack/python-openstacksdk/blob/master/openstack/resource.py#L234
21:47:19 adriant so I'll update transpose to check if the key is already present
21:47:26 mordred ++
21:49:55 adriant oh wait no I'm being silly. transpose just drops them. I need to make it actually try and use both.
21:50:38 mordred adriant: I must afk for a couple of hours - will check back in when I'm back - thanks for diving in on this!
22:39:04 openstackgerrit Merged openstack/python-openstackclient master: Make functional-tips job voting https://review.openstack.org/525283
22:41:17 openstackgerrit OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/539718
22:44:38 openstackgerrit Merged openstack/python-openstacksdk master: Let enforcer.py work under both py2 and py3 https://review.openstack.org/539685
22:53:27 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Add server-side names to query param checking https://review.openstack.org/539722
22:56:32 adriant mordred: fixed ^
22:57:13 adriant and the fix actually adds some nicer logic overall
#openstack-sdks - 2018-02-02
00:51:26 mordred rods_: I thought you'd like that!
01:21:22 openstackgerrit Nguyen Hung Phuong proposed openstack/service-types-authority master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/539810
11:07:40 openstackgerrit Andreas Jaeger proposed openstack/os-client-config master: Add osc tips jobs back https://review.openstack.org/526706
11:10:13 openstackgerrit Andreas Jaeger proposed openstack/keystoneauth master: Remove PYTHONHASHSEED setting https://review.openstack.org/533798
11:26:10 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/540345
12:27:17 openstackgerrit Merged openstack-infra/shade master: Plumb use-direct-get through factory functions https://review.openstack.org/539586
12:27:19 openstackgerrit Merged openstack-infra/shade master: Fix get_server to work with use_direct_get https://review.openstack.org/539663
13:14:49 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Update base test case to use base from oslotest https://review.openstack.org/540372
13:14:49 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Update all test base classes to use base.TestCase https://review.openstack.org/540373
13:14:50 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Run examples tests with functional tests https://review.openstack.org/540374
13:16:16 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770
13:28:39 openstackgerrit Merged openstack/os-client-config master: Zuul: Remove project name https://review.openstack.org/538901
14:04:26 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770
14:04:26 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fix get_server to work with use_direct_get https://review.openstack.org/540389
14:04:27 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add all_projects as a preferred alias for all_tenants https://review.openstack.org/540390
14:09:14 rabel_ any opinions on https://bugs.launchpad.net/python-openstackclient/+bug/1744349 ? i'd like to implement this, shouldn't be a big deal. just want to hear some opinions before
14:09:14 openstack Launchpad bug 1744349 in python-openstackclient "Interface of "quota set" and "quota show" is missleading" [Undecided,New]
14:52:33 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove support for arbitrary attributes from CloudRegion https://review.openstack.org/540406
14:59:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Update all test base classes to use base.TestCase https://review.openstack.org/540373
14:59:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Run examples tests with functional tests https://review.openstack.org/540374
14:59:19 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Re-enable orchestration functional tests https://review.openstack.org/540412
15:32:01 rods_ mordred amazing work on https://review.openstack.org/#/c/540169/1, left two very minor comments
15:32:15 mordred rods_: awesome - thanks for reading through that!
15:38:07 mordred rods_: those are great feedback- thanks
16:10:44 openstackgerrit James E. Blair proposed openstack/openstackclient master: Zuul: Remove project name https://review.openstack.org/540433
17:37:34 Shrews mordred: your proxy change is next on my todo list after lunch
18:02:27 mordred Shrews: awesome! rods liked it. oh- I fixed it from his reviews ...
18:06:10 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Generate proxy methods from resource objects https://review.openstack.org/540169
18:10:18 mordred rods_: ^^ updated per your comments
18:10:32 rods_ mordred looking
18:31:06 openstackgerrit Merged openstack/openstackclient master: Zuul: Remove project name https://review.openstack.org/540433
18:33:22 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Use instance.get_interface instead of instance.interface https://review.openstack.org/540481
18:33:56 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove support for arbitrary attributes from CloudRegion https://review.openstack.org/540406
18:42:08 rods_ mordred https://review.openstack.org/#/c/540169/2 LGTM
18:42:24 mordred rods_: woot
18:43:53 rods_ mordred if we decide that ^^ is a viable solution, I'd be happy to help updating the other Services
20:19:01 Shrews mordred: comment left for you on 540169
20:20:22 Shrews mordred: oh! i'm stupid. the value *is* the return of that. duh
20:20:25 Shrews nm
20:21:15 Shrews mordred: but perhaps we should set the default to False?
20:21:59 Shrews or would you rather an exception be raised if one of the allow_* attrs are missing?
20:32:49 Shrews I'm going to assume that's the behavior you want. LGTM otherwise. +3
20:58:49 mordred Shrews: well - it is defaulted in the base Resource class: http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/resource.py#n298
20:59:40 mordred Shrews: but yes - if something is missing there it's a pretty fundamental internal programming error - and everything should come crashing to a halt because ZOMG everything is terrible
21:07:42 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: [WIP] Add option '--mtu' to network create/update https://review.openstack.org/540530
21:28:34 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Add option '--mtu' to network create/set https://review.openstack.org/540530
21:40:06 Shrews mordred: so it is! apologies
21:55:47 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fix issue with url parameter strictness and OSC https://review.openstack.org/540537
22:00:25 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Disallow setting default on internal network https://review.openstack.org/539303
22:06:55 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Stop sending tenant_id to sdk network list https://review.openstack.org/540539
22:15:12 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make Profile fallback go bye-bye https://review.openstack.org/540541
23:16:27 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Stop sending tenant_id to sdk network list https://review.openstack.org/540539
23:26:04 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fix issue with missing url parameters https://review.openstack.org/540537
23:28:35 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Don't look for ResourceNotFound string in test https://review.openstack.org/540539
23:37:36 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Don't look for ResourceNotFound string in test https://review.openstack.org/540539
23:37:37 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make Profile fallback go bye-bye https://review.openstack.org/540541
23:37:37 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make osc-functional-devstack-tips actually use tips https://review.openstack.org/540554
23:38:05 mordred dtroyer: guess what? The osc-functional-devstack-tips job? that I thought was showing us that we were ok on the devstack/sdk interactions?
23:38:39 mordred dtroyer: ^^ 540554 fixes it to ACTUALLY test against tips - it was very happily testing against the previously released verisons of things
23:38:41 mordred *headdesk*
23:40:44 mordred dtroyer: in any case - the carnage is surprisingly small considering the number of changes involved in this last release - I think with the one patch i've got up to sdk that's pretty small and one up for osc (which is just updating a test that was looking for a string that changed)
23:40:53 mordred and we'll be good to go
23:41:27 mordred dtroyer: I was really starting to lose my mind ... why are we seeing osc tips jobs be red NOW but they were green when we landed the changes
23:43:51 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Gate on osc-functional-tips https://review.openstack.org/540556
23:44:20 mordred dtroyer: ^^ and that will turn that job to gating on sdk - not that we've landed any breaking changes due to ignoring it non-voting ... but let's be extra sure
#openstack-sdks - 2018-02-03
03:07:22 openstackgerrit Huang Cheng proposed openstack/python-openstackclient master: Fix subnet host_routes error https://review.openstack.org/540568

Earlier   Later