Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-02-02
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 ...
16:15:41 openstackgerrit Merged openstack/service-types-authority master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/539810
16:36:27 mordred Qiming: https://review.openstack.org/532641 <-- pushed that up on top of your patches
16:36:37 mordred Qiming: if it works, we can squash
18:06:17 openstackgerrit Merged openstack/python-openstacksdk master: Fix issue with missing url parameters https://review.openstack.org/540537
20:58:04 openstackgerrit James E. Blair proposed openstack/python-openstackclient master: Zuul: Remove project name https://review.openstack.org/541004
22:53:30 mordred Qiming: ok. I thnk this latest patch is going to do it: https://review.openstack.org/532641 Update sdk connection, tests and isoformat
22:53:41 mordred Qiming: I squashed the stack together and removed the debugging lines
22:54:44 mordred Qiming: I left one of the log line additions - since senlin wraps the underlying exception from sdk before passing it up, the log line there was the thing that was needed to find what was actually wrong
22:57:58 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Calculate name in CloudRegion https://review.openstack.org/541059
22:57:59 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Use get_session_client in Connection https://review.openstack.org/541060
23:00:24 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make osc-functional-devstack-tips actually use tips https://review.openstack.org/540554
23:00:30 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Gate on osc-functional-tips https://review.openstack.org/540556
23:07:58 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add some comments explaining split_loggers flag logic https://review.openstack.org/541066
#openstack-sdks - 2018-02-06
00:16:00 mordred Qiming: http://logs.openstack.org/41/532641/5/check/senlin-dsvm-tempest-py27-functional/0fa2ee4/job-output.txt.gz#_2018-02-05_23_19_18_244439 <-- that failure doesn't seem to be related to the sdk issue - otherwise that ran green
00:19:35 openstackgerrit James E. Blair proposed openstack/keystoneauth master: Zuul: Remove project name https://review.openstack.org/541080
02:30:55 openstackgerrit pangliye proposed openstack/python-openstacksdk master: fix misspelling of 'volume' https://review.openstack.org/541095
07:48:50 openstackgerrit Hiroaki Kobayashi proposed openstack/service-types-authority master: Add a reservation service type (blazar project) https://review.openstack.org/541164
07:59:17 openstackgerrit Hiroaki Kobayashi proposed openstack/os-service-types master: Add blazar project https://review.openstack.org/541168
08:53:06 openstackgerrit Hiroaki Kobayashi proposed openstack/os-service-types master: Add blazar project https://review.openstack.org/541168
11:28:57 openstackgerrit Thomas Herve proposed openstack-infra/shade master: Allow not resolving outputs on get stacks https://review.openstack.org/541244
14:48:35 rabel do positional arguments have to be in <> in help messages? e.g. server create --help says <server-name>, but subnet create --help says name (without <>)
14:52:17 tosky rabel: without; if you see that in documentation, it's a usual way to describe the options
14:52:45 openstackgerrit Merged openstack/service-types-authority master: Add a reservation service type (blazar project) https://review.openstack.org/541164
14:53:48 rabel tosky: so the help message of server create is wrong?
14:54:39 tosky rabel: or the other one

Earlier   Later