Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-01-30
16:32:39 mordred dtroyer: although the volume type create seems less happy
16:33:37 mordred locally, with stable/pike osc and master sdk I get:
16:33:39 mordred mordred@solace:~/src/git.openstack.org/openstack/python-openstackclient$ openstack --os-cloud=vexxhost volume type create monty
16:33:41 mordred Policy doesn't allow volume_extension:types_manage to be performed. (HTTP 403) (Request-ID: req-d9acb541-fa27-4983-9e07-65c5c0e42f6b)
16:33:52 dtroyer In the log I poked through that seemed to be the first osc invocation… I wonder if something got moved around?
16:33:56 mordred (which is what I'd expect, since I can't do that - but it's certainly getting past auth)
16:33:58 dtroyer oh, that's different
16:34:12 mordred so - my local error seems like correct behavior
16:34:29 mordred dtroyer: so - I might need to spin up a devstack and poke
16:36:51 mordred dtroyer: I put in a zuul hold so we can grab the node from that job next time it breaks and look
17:37:55 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
17:37:55 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234
17:37:56 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
18:20:59 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232
18:21:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234
18:21:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233
18:21:01 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
18:21:01 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
19:09:53 openstackgerrit Merged openstack/python-openstacksdk master: Provide compatibility for people passing raw sessions https://review.openstack.org/538968
19:09:55 openstackgerrit Merged openstack/python-openstacksdk master: Update reno for stable/queens https://review.openstack.org/538693
19:19:42 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: [WIP] Disallow setting default on internal network https://review.openstack.org/539303
20:36:56 adriant mordred: looking at those constructor patches now
20:39:57 mordred adriant: awesome! thanks!
20:39:58 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Disallow setting default on internal network https://review.openstack.org/539303
20:50:17 openstackgerrit Colleen Murphy proposed openstack/python-openstackclient master: Add CRUD support for application credentials https://review.openstack.org/536163
20:53:11 adriant mordred: with that code I think I can swap to just reusing the session, which actually makes things much simpler
20:53:33 adriant still reading, found a few typos on the comments/docs, but so far the code looks good
21:17:37 mordred adriant: awesome
22:33:41 openstackgerrit Merged openstack/python-openstacksdk master: orchestration: fix typo in doc https://review.openstack.org/539116
#openstack-sdks - 2018-01-31
00:11:10 mordred dtroyer: ok. I added a ton of debugging - and it stopped failing :(
00:12:22 mordred dtroyer: so I've rebased the real patch again and we'll see if it magically passes this time
01:48:33 adriant mordred: left you a note and a paste on one of those patches
01:49:43 adriant when I try initialising the connection directly with auth vars rather than a session, it still looks at the envvars
01:50:04 adriant but the sessions logic works! :P
01:51:13 mordred adriant: yay for the sessions! boo for the other thing ... thanks for test driving
01:51:22 mordred adriant: I've got another fun patch almost ready ...
01:52:04 adriant mordred: not a problem, and my current use case is a good one for testing this since it's within the context/scope of a running openstack client command, so there are envvars/clouds.yaml things to explicitly avoid :P
01:52:31 adriant it's rare someone will ever hit this same problem, but... better to ensure they don't because their context may be even worse
01:58:13 mordred the complex problems are the best :)
02:12:54 mordred adriant: BWAHAHAHAHA. found it and it's silly
02:13:17 mordred adriant: http://paste.openstack.org/show/658082
02:13:59 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232
02:14:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234
02:14:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233
02:14:01 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
02:14:01 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
02:14:02 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove openstack_cloud factory function https://review.openstack.org/539395
02:14:02 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Merge Connection and OpenStackCloud https://review.openstack.org/539394
02:14:03 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions https://review.openstack.org/539396
02:14:14 adriant mordred: hah!
02:14:30 mordred and there's tonight's patch bomb ... I shall now remove myself from the compute and eat some fish
02:14:48 adriant good plan, will play with those some more and see how it looks
02:14:53 adriant mostly looks great
06:38:45 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: resource: don't early terminate list https://review.openstack.org/539424
08:45:03 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: Implement list projects for user https://review.openstack.org/539460
08:46:40 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: resource: don't early terminate list https://review.openstack.org/539424
10:48:09 openstackgerrit Chen Hanxiao proposed openstack/python-openstackclient master: Add --image-property parameter in 'server create' https://review.openstack.org/535664
11:08:14 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: Fix TypeError for overrided get methods https://review.openstack.org/539494
14:20:44 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Merge Connection and OpenStackCloud https://review.openstack.org/539394
14:20:45 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions https://review.openstack.org/539396
14:20:45 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove openstack_cloud factory function https://review.openstack.org/539395
14:20:46 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
14:20:46 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
14:57:18 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Shift voting flag for ansible-devel job https://review.openstack.org/539563
14:57:37 tosky hi! I noticed a regression when running some tempest tests; while requirements and upper-constraints pins to openstacksdk 0.9.19, tempest install the last version (0.11.0)
14:57:49 tosky and when running a simple `openstack network list --external`, I get
14:57:53 tosky Invalid query params: router:external
14:59:33 tosky I don't seem to see an existing bug, so I'm going to file one, but I'm not sure if openstackclient 3.14.0 is supposed to work with openstacksdk 0.11.0
15:06:12 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Shift voting flag and test_matrix_branch for ansible-devel job https://review.openstack.org/539563
15:10:38 mordred tosky: yes - we do expect openstackclient 3.14.0 to work with 0.11 - so yes please to the bug
15:11:05 tosky mordred: so again openstacksdk only, not against openstackclient?
15:11:25 mordred tosky: yah - I'd consider it a bug in the 0.11 release that we should figure out and get a point-release cut for
15:11:40 tosky mordred: thanks, I was halfway through it, incoming
15:12:29 tosky here you are: https://bugs.launchpad.net/python-openstacksdk/+bug/1746535
15:12:30 openstack Launchpad bug 1746535 in OpenStack SDK "`openstack network list --external` breaks with openstacksdk 0.11.0 + openstackclient 3.14.0" [Undecided,New]
15:33:39 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Split request logging into four different loggers https://review.openstack.org/505764
15:38:10 mnaser mordred: forgive me if i'm misusing shade, but get_volume() seems to want to list volumes and then find it (assumingly trying to filter by name first)
15:38:20 mnaser so trying to do get_volume() in the admin tenant results in no volume found
15:38:43 mnaser is there some switch i'm not turning on?
15:38:59 mordred mnaser: yes - that is the default behavior. there is a constructor flag "use_direct_get" which will cause it to use get instead of list for gets
15:39:15 mnaser mordred: wonderful, that's exactly what i needed, thank you
15:40:01 mnaser hmm
15:40:36 mordred mnaser: I believe we'll flip the value of that default in the openstacksdk version of shade - we filter list calls to support nodepool, which I promise you do not want doing 1000s of get calls ... but for humans who are not nodepool it can be unexpected behavior :)
15:41:06 mnaser mordred: i think that flag might have been lost somehow
15:41:17 openstackgerrit Merged openstack-infra/shade master: Add devel branches and override-checkout for ansible-devel job https://review.openstack.org/536159
15:41:22 mnaser it seems like it is not a constructor value but a thing to set as an attribute to cloud
15:41:28 mnaser (aka cloud.use_direct_get = True)
15:41:38 mordred mnaser: I see it on the constructor?
15:41:54 mnaser note to self don't trust github search
15:42:16 mordred mnaser: http://git.openstack.org/cgit/openstack-infra/shade/tree/shade/openstackcloud.py#n137
15:42:21 mordred mnaser: never trust github
15:42:37 mnaser it doesn't seem to be in the release shipped with rdo pike, maybe i'll ask for a bump sometime soon
15:42:51 mordred oh - yah - it's a new flag
15:43:00 mordred got implemented this cycle
15:43:15 mnaser shade==1.22.2 and this merged in 1.24.0
15:43:21 mnaser ok cool venv it is
15:43:27 mordred \o/
15:43:43 mordred mnaser: luckily 1.24.0 is post-client-ectomy, so the venv should be quite small
15:43:53 mnaser oh awesome
15:45:44 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Shift voting flag and test_matrix_branch for ansible-devel job https://review.openstack.org/539563

Earlier   Later