Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-02-02
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: Fix get_server to work with use_direct_get https://review.openstack.org/540389
14:04:26 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770
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 openstack Launchpad bug 1744349 in python-openstackclient "Interface of "quota set" and "quota show" is missleading" [Undecided,New]
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: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: Run examples tests with functional tests https://review.openstack.org/540374
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: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 osc-functional-devstack-tips actually use tips https://review.openstack.org/540554
23:37:37 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make Profile fallback go bye-bye https://review.openstack.org/540541
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
14:50:26 openstackgerrit Slawek Kaplonski proposed openstack/python-openstacksdk master: Add tenant_id parameter to be mapped to project_id in network object https://review.openstack.org/540591
15:25:06 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fix issue with missing url parameters https://review.openstack.org/540537
15:26:20 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make osc-functional-devstack-tips actually use tips https://review.openstack.org/540554
15:26:21 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make Profile fallback go bye-bye https://review.openstack.org/540541
15:32:52 mordred slaweq: re: 540591 - I've got that in https://review.openstack.org/#/c/540537/ - along with a few other things needed
15:34:28 slaweq mordred: ok, thx for info. I was looking for patch with title which could be related but I didn't found this one
15:34:43 slaweq mordred: I will abandon my patch then
15:36:00 mordred slaweq: now I just hope it passes ...
15:36:42 mordred slaweq: I think you were offline when I was telling dtroyer yesterday - part of the problem is that the osc-functional-devstack-tips job, which we also run on sdk patches - had a bug
15:36:58 mordred and was not actually running with tip of sdk patches, but instead with last released version
15:37:25 mordred so we weren't actually getting tests that our patches were ok with osc :(
15:37:51 slaweq mordred: yes, I was offline most of the day yesterday so I probably missed that
15:38:23 mordred slaweq: I've got a patch upto fix the job too: https://review.openstack.org/#/c/540554/
15:38:28 slaweq but I say patch with fix for that also
15:38:46 slaweq mordred: right, I saw it today :)
15:39:24 slaweq mordred: I'm on FOSDEM today so I don't have too much time to review :)
15:43:38 mordred slaweq: have fun at FOSDEM! I saw your tweet with the alcohol :)
15:44:13 slaweq mordred: thx a lot. Now I'm listening about VPP packet processing and drinks belgium beer ;)
15:45:03 slaweq and this photo was done in marketplace
15:46:58 mordred slaweq: I think you must drink belgium beer to listen about VPP packet processing
15:47:17 slaweq mordred: LOL, it's not that bad :)
#openstack-sdks - 2018-02-04
08:15:03 openstackgerrit Praveen N proposed openstack/python-openstackclient master: displays flavor specific for project https://review.openstack.org/540633
#openstack-sdks - 2018-02-05
01:18:44 openstackgerrit Nguyen Hung Phuong proposed openstack/service-types-authority master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/539810
02:55:49 openstackgerrit Huang Cheng proposed openstack/python-openstackclient master: Fix subnet host_routes error https://review.openstack.org/540568
10:00:47 openstackgerrit Cedric Brandily proposed openstack/python-openstackclient master: Remove insane options for network agent add/remove actions https://review.openstack.org/527984
10:01:15 openstackgerrit Cedric Brandily proposed openstack/python-openstackclient master: Remove insane options from network agent add/remove actions https://review.openstack.org/527984
10:28:27 openstackgerrit chenge proposed openstack/python-openstackclient master: Replace Chinese quotes to English quotes https://review.openstack.org/540814
14:46:13 rabel still no opinions on https://bugs.launchpad.net/python-openstackclient/+bug/1744349 ?
14:46:14 openstack Launchpad bug 1744349 in python-openstackclient "Interface of "quota set" and "quota show" is missleading" [Undecided,New]
15:20:32 mordred Qiming: if you're still up - I thikn your job issue might be sdk related - do you have a link to a failure?
15:23:48 mordred Qiming: oh - nm - the thing I thought it might be doesn't seem likely looking at your devsack plugin
15:38:05 mordred Qiming: ah - I see you're poking at it https://review.openstack.org/#/c/540625 - I'm happy to help iterate on that
15:38:26 mordred Qiming: also - I think we sohuld set up a gate job for sdk on senlin's functional tests
15:54:33 Qiming mordred, thank you so much
15:55:00 Qiming Just confirmed the gate issue is related to SDK
15:55:34 Qiming When creating a connection, current code still tries to support profile and other parameters
15:55:43 Qiming But we don't pass in authenticator parameter
15:55:59 Qiming That leads to authenticator.auth_url test failure
15:56:27 Qiming I am trying to fix it by incorporating your changes, so that we can use 0.11.2 version
15:56:42 Qiming However, local test result is that SDK doesn't like the 'token' parameter
15:57:04 Qiming We need to pass in a token parameter when creating a trust
16:03:40 mordred yah - I've got an update locally I think will help ... lemme push up as soon as this local run finishes
16:08:21 Qiming Speaking of functional tests, we are adding functional tests to python-senlinclient, you know, senlinclient is a thin wrapper of sdk, it is mainly a OSC plugin after the senlin CLI being removed.
16:13:37 mordred ++
16:13:48 mordred Qiming: gah. I made my changes on top of the wrong patch - one sec ...

Earlier   Later