Posted	Nick	Remark
#openstack-sdks - 2018-07-25
14:42:48	corvus	mordred: still wants auth_url
14:43:32	corvus	File \"/home/corvus/git/openstack-infra/zuul-jobs/.tox/py35/lib/python3.5/site-packages/keystoneauth1/loading/base.py\", line 162, in load_from_options\n raise exceptions.MissingRequiredOptions(missing_required)\nkeystoneauth1.exceptions.auth_plugins.MissingRequiredOptions: Auth plugin requires parameters which were not given: auth_url\n",
14:44:09	corvus	that's from the get_one line
14:45:18	mordred	poo. lemme poke further 0 sorry about that
14:48:39	mordred	corvus: ok. I jave reproduced that lcaly
14:49:39	dtantsur	folks, how do I deal with "The service catalog is empty" from session.get_endpoint of the service catalog is NOT empty?
14:49:50	dtantsur	in what other cases can this happen?
14:50:02	mordred	dtantsur: I've never seen that
14:50:08	dtantsur	:(
14:50:19	mordred	dtantsur: are you sure you have a token/catalog?
14:50:48	dtantsur	mordred: the session is created with keystoneauth1.identity.v3.Token, the token comes from Mistral, no idea how it creates it
14:51:01	dtantsur	it's tripleo, there is nothing I can be sure about :D
14:51:35	mordred	dtantsur: it's possible the catalog got stripped from the token if it came from middleware
14:52:00	dtantsur	mordred: the token - is it a string or an object? I think I get a string
14:52:01	mordred	people like doing that to reduce size of the payload for reasons I don't really understand
14:52:20	dtantsur	and now a funny thing: if I wrap it in an Adapter, it seems to work Oo
14:52:29	mordred	weird
14:53:54	dtantsur	extremely weird
14:57:02	dtantsur	so, this apparently blows up https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/base.py#L275
15:02:16	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Support passing profile to get_one https://review.openstack.org/585780
15:02:24	mordred	corvus: ^^
15:04:06	dtantsur	I guess I short report it against keystoneauth. it should not report EmptyCatalog if it's not empty in reality.
15:04:24	corvus	mordred: that works, thanks!
15:04:38	dtantsur	s/short/should/ (WUT)
15:05:13	mordred	corvus: yay!
15:13:37	dtantsur	filed https://bugs.launchpad.net/keystoneauth/+bug/1783590
15:13:38	openstack	Launchpad bug 1783590 in keystoneauth "EmptyCatalog raised when the catalog is not empty" [Undecided,New]
16:10:59	dtantsur	mordred: do you know if I can pass the catalog to keystoneauth somehow? given that Token auth is, well, broken apparently in this regard
16:12:52	dtantsur	ok, never mind, it just always treats the provided token as unscoped..
17:04:26	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Add DNS support to floating IP commands https://review.openstack.org/558824
17:06:19	dtroyer	mordred: if you're happy with the conversation, https://review.openstack.org/#/c/582334/ I think is ready
17:07:13	dtroyer	that and https://review.openstack.org/#/c/558824/ are the two I think we want to wait on for the release…I'm doing a last pass through release notes yet
17:49:57	mordred	dtroyer: +A on that one
17:51:37	dtroyer	mordred: Thanks. did you intend https://review.openstack.org/585030 to go in now too? I just rechecked it...
17:52:14	mordred	dtroyer: no - I mean, I wish - but we need a keystoneauth prelease to pick uit up
17:52:26	dtroyer	ah, right. ok
17:52:32	mordred	silly freezes
17:52:35	mordred	;)
17:52:57	mordred	tbh - I've been meaning to write the ksa patch for weeks, so it's totally my fault
17:57:45	openstackgerrit	Merged openstack/python-openstackclient master: Add command to show all service versions https://review.openstack.org/584905
19:20:27	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Release note cleanup for 3.16.0 release https://review.openstack.org/585853
19:31:35	openstackgerrit	Merged openstack/python-openstackclient master: Don't sent disk_over_commit if nova api > 2.24 https://review.openstack.org/582334
19:58:50	dtroyer	mordred: https://review.openstack.org/#/c/585853/ is the relnote cleanup and likely 3.16.0 release commit, ready to go…
20:07:57	mordred	love it
20:36:34	openstackgerrit	Merged openstack/python-openstackclient master: Release note cleanup for 3.16.0 release https://review.openstack.org/585853
20:56:38	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Add DNS support to floating IP commands https://review.openstack.org/558824
21:04:59	mordred	dtroyer: woot. now I guess for next cycle there's no excuse I can give for not figuring out the remaining osc/sdk bits
#openstack-sdks - 2018-07-26
01:51:08	openstackgerrit	Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037
02:58:41	openstackgerrit	Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661
05:02:39	openstackgerrit	Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037
05:03:45	openstackgerrit	Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037
07:11:51	openstackgerrit	yanpuqing proposed openstack/python-openstackclient master: Add error message when network resources use "set" "unset" command https://review.openstack.org/579515
07:24:24	usr2033	hi, does official openstack-sdk support quota operations?
08:58:15	openstackgerrit	OpenStack Release Bot proposed openstack/python-openstackclient master: Update reno for stable/rocky https://review.openstack.org/586006
09:01:02	openstackgerrit	Josephine Seifert proposed openstack/python-openstackclient master: osc-included image signing (using openstacksdk) https://review.openstack.org/580086
09:02:57	openstackgerrit	Josephine Seifert proposed openstack/python-openstackclient master: osc-included image signing (using openstacksdk) https://review.openstack.org/580086
11:54:44	openstackgerrit	Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661
12:00:01	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Update create_object to handled chunked data https://review.openstack.org/585532
12:34:05	openstackgerrit	OpenStack Release Bot proposed openstack/keystoneauth master: Update reno for stable/rocky https://review.openstack.org/586083
12:35:07	openstackgerrit	OpenStack Release Bot proposed openstack/openstacksdk master: Update reno for stable/rocky https://review.openstack.org/586094
12:35:37	openstackgerrit	OpenStack Release Bot proposed openstack/os-client-config master: Update reno for stable/rocky https://review.openstack.org/586100
12:36:32	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Add flag for returning unofficial types https://review.openstack.org/585411
12:36:33	openstackgerrit	OpenStack Release Bot proposed openstack/osc-lib master: Update reno for stable/rocky https://review.openstack.org/586112
12:48:32	odyssey4me	mordred: I thought you might like to know a new and exciting and totally useful error I got the other day when I tried to do a compute thing with the openstack client. :) http://logs.openstack.org/15/582315/9/check/openstack-ansible-functional-ubuntu-xenial/911cdf7/job-output.txt.gz#_2018-07-21_09_14_00_043030
12:49:09	odyssey4me	It turns out, when you do not have an endpoint for the service, you get the rather oscure "'list' object has no attribute 'get'" error.
13:18:10	mordred	odyssey4me: well! that certainly is exciting
13:24:33	mordred	odyssey4me: I'm not 100% sure if that's coming from osc or novaclient- how hard would it be to recreate that with --debug ?
13:25:31	odyssey4me	mordred: hmm, let me do that and get back to you - I might already have one up that I can break
13:25:55	odyssey4me	oh bother, I don't - ok, let me kick off a build
13:36:14	mordred	odyssey4me: woot. thanks!
16:07:40	odyssey4me	mordred: it turns out that it's far more fun than anticipated - to replicate it you have to setup the compute endpoint with the wrong service backing it
16:08:09	odyssey4me	in my test I put the placement service endpoints into the catalog, saying it was the compute service
16:08:11	odyssey4me	https://gist.github.com/odyssey4me/fdfb2474bdd6fde4d1368af2af4044c6
16:08:45	odyssey4me	while of course this is not someone one would expect someone to do, it is perhaps better to handle the error somewhat more elegantly
16:12:18	dtroyer	odyssey4me: this sounds like it might be a good integration test for OSC. Those drop commands in the top and use requests-mock so we let the entire stack have a chance to get it wrong, but still wouldn't require a specially-configured could to test against...
16:27:58	cdent	thanks elmiko, was rather distracted during all that
16:28:36	elmiko	cdent: no worries =)
16:28:51	openstackgerrit	Merged openstack/api-wg master: Add links to errors-example.json https://review.openstack.org/578369
16:28:52	openstackgerrit	Merged openstack/api-wg master: Expand schema for error.codes to reflect reality https://review.openstack.org/580703
16:33:32	edleafe	cdent, elmiko, dtantsur: https://etherpad.openstack.org/p/api-sig-newsletter is ready for your review
16:34:17	edleafe	Oops - hold on a minute
16:34:56	edleafe	ok, it's ready now. Forgot to paste the links
16:35:07	cdent	lgtm
16:35:50	elmiko	edleafe: maybe reference that notmyname suggested we look at the draft?
16:36:12	edleafe	elmiko: sure. You wanna add that?
16:36:18	elmiko	k
16:37:23	elmiko	edleafe: how's that look?
16:37:37	edleafe	lgtm
16:37:39	edleafe	ship it?
16:37:45	elmiko	+1
16:38:03	elmiko	thanks again edleafe !
16:39:03	edleafe	sent
16:39:18	elmiko	\o/
19:10:20	mordred	dtroyer, odyssey4me: yes to integration test!
19:11:20	mordred	odyssey4me: and thanks for making that traceback ... it also looks like the base issue is in novaclient with an assumption on its part on how error payloads look. when we shift stuff over to sdk the error handler there is more permissive because it has to handle all of the services- so it's coded defensively to check and see what exists
19:11:35	mordred	in any case, yes - I do think that error could be handled more elegantly
20:02:55	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Update create_object to handled chunked data https://review.openstack.org/585532
20:02:55	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add method for returning a raw response for an object https://review.openstack.org/586262
20:02:56	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add support for streaming object responses https://review.openstack.org/586250
22:10:35	openstackgerrit	Merged openstack/os-service-types master: Add flag for returning unofficial types https://review.openstack.org/585411

Earlier   Later