Posted	Nick	Remark
#openstack-sdks - 2018-01-30
15:03:04	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232
15:03:05	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234
15:03:22	mordred	adriant: ok. that ^^ should take care of all of your questions and use cases (I hope)
15:36:16	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Update build to put content into sphinx location https://review.openstack.org/539239
15:37:26	mordred	dtroyer: jeez. it seems like devstack on pike is just flat broken for us there does't it :(
16:23:53	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Update build to put content into sphinx location https://review.openstack.org/539239
16:30:16	mordred	dtroyer: oh - those errors may have just been this morining's image issue
16:30:42	dtroyer	mordred: I'm hoping so
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

Earlier   Later