Posted	Nick	Remark
#openstack-sdks - 2017-07-07
15:26:45	GreenBlood	Yeah but the service is the same wether I use the glanceclient or shade/openstackclient nah ?
15:27:40	GreenBlood	Oh wait I think I noticed something
15:27:54	GreenBlood	I think our glanceclient uses v1 and shade uses v2
15:28:08	GreenBlood	that would make sense
15:29:02	mordred	ah - that'll definitely do it - however, it's possible also that we're not sending somehting we need to to get shared images returned
15:29:58	GreenBlood	Judging by the HTTP debug, shade sends "GET /v2/images " and our code using glanceclient sends "GET /v1/images/detail?limit=20&is_public=None"
15:31:30	mordred	kk. so - you can force shade to use v1 by setting image_api_version=1 in your clouds.yaml file - also - for v2, there is this note:
15:31:44	mordred	"If you omit this parameter (visibility ), the response shows public, private, and those shared images with a member status of accepted."
15:32:11	mordred	so images in v2 default to "shared" if you don't set any other status, and 'shared' images will only show up for a user in v2 by default if that user has "accepted" the image
15:33:21	mordred	GreenBlood: so - list_images in v2 with no other parameters omitting shared by not accepted images is 'correct' behavior - although it might be surprising
15:33:35	GreenBlood	Yeah well the user has accepted, I can see it in Horizon and stuff.
15:33:47	GreenBlood	I'll try using v1 api but yeah its a bit wierd imho
15:33:50	mordred	ah! well, that seems like a whole different bug
15:34:11	mordred	in fact, that is a glance behavior that does not match the docs
15:34:21	GreenBlood	Oh
15:35:34	mordred	so - fwiw, the way to get shade to show you shared-but-not-accepted images is to do search_images(filters=dict(member_status='all')) ... that's not super friendly - so I think we should add a flag to the api to make it easy to opt-in to seeing _ALL_ images including unaccepted shared images
15:41:36	GreenBlood	Good to know
15:42:01	GreenBlood	But it's really bugging me this listing issue thingy
15:42:32	GreenBlood	that would be really wierd to have shared images on v1 and not v2 no ?
15:48:41	GreenBlood	Using "image_api_version: 1" in the cloud config file solves the issue
15:53:22	GreenBlood	btw this tool, Shade, is really awesome
15:53:39	GreenBlood	I discovered it the other day and I'm amazed
15:56:15	mordred	GreenBlood: yay! I'm glad you like it!
15:56:39	mordred	GreenBlood: yah - it was a behavior change in the v2 api - v1 doesn't understand shared at all, so it doesn't filter anything
15:57:06	GreenBlood	Yeah but search with member_status='all' doesnt change the result I think
15:57:14	GreenBlood	or maybe i'm just blind
16:07:14	GreenBlood	which would be sad, really
16:21:00	GreenBlood	Even with the CLI, doing "glance --os-image-api-version 1 image list" and without specifying the version, the shared images appears or disappears. I'll send a mail to my cloud provider :)
16:21:07	GreenBlood	Thank you -again- for your help
16:38:29	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
16:38:49	fried_rice	mordred ^
16:59:03	openstackgerrit	Merged openstack/keystoneauth master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479191
17:12:16	mordred	GreenBlood: https://review.openstack.org/481702 made a patch to add a show_all flag
17:14:21	GreenBlood	oooh
17:14:22	GreenBlood	nice
17:21:48	GreenBlood	ok i tested it and the behaviour does not changes, so it must be an issue on the provider's side
17:22:34	GreenBlood	wait no i'm just dumb
17:22:36	GreenBlood	Gah
17:22:44	GreenBlood	I'll test it monday
18:04:39	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Stop publishing yaml file https://review.openstack.org/481305
18:05:20	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add api_reference_project field https://review.openstack.org/480721
18:17:45	openstackgerrit	Eric Fried proposed openstack/service-types-authority master: add types for freezer and monasca-log-api https://review.openstack.org/480718
20:23:12	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
20:25:07	fried_rice	mordred ^ after much frustration with docstring formats. Where can I find out wtf it's expecting? Cause it doesn't seem to be standard RST.
20:39:58	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Fix _run_discovery caching; misc cleanup https://review.openstack.org/481754
20:43:29	fried_rice	mordred ^ needs to go in before we cut the next release, else it's bogus (changes sig of at least one public method, but that method hasn't been released yet AFAICT).
20:44:58	fried_rice	mordred Also, forget what I said yesterday about Discover.versioned_data_for() returning an EndpointData. The contents seem to be quite different - though I'm still trying to understand why and where each applies.
#openstack-sdks - 2017-07-08
00:44:43	openstackgerrit	OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/480277
08:03:07	openstackgerrit	Sean McGinnis proposed openstack/python-openstackclient master: Explicitly use volume API v2 https://review.openstack.org/481799
21:18:07	openstackgerrit	Akihiro Motoki proposed openstack/osc-lib master: Emit warnings to encourage cliff FormattableColumn https://review.openstack.org/462150
#openstack-sdks - 2017-07-10
10:42:44	openstackgerrit	M V P Nitesh proposed openstack/python-openstackclient master: List images with --tags for 'openstack image list' https://review.openstack.org/481615
12:05:27	openstackgerrit	NidhiMittalHada proposed openstack/python-openstackclient master: image-list should support filters 'name','status' https://review.openstack.org/479670
12:21:01	openstackgerrit	NidhiMittalHada proposed openstack/python-openstackclient master: image-list should support filters 'name','status' https://review.openstack.org/479670
12:57:06	openstackgerrit	Merged openstack/service-types-authority master: Stop publishing yaml file https://review.openstack.org/481305
13:01:27	openstackgerrit	Chris Dent proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
13:01:52	openstackgerrit	Merged openstack/service-types-authority master: Add api_reference_project field https://review.openstack.org/480721
13:01:58	openstackgerrit	Merged openstack/service-types-authority master: add types for freezer and monasca-log-api https://review.openstack.org/480718
13:15:16	efried	cdent Are you rebasing https://review.openstack.org/462140 ?
13:15:31	cdent	efried: yeah, i’ll get it
13:15:38	efried	cool
13:28:45	openstackgerrit	Chris Dent proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
14:28:18	mordred	look it's a cdent
14:28:29	mordred	cdent: service-types.openstack.org is LIVE!
14:33:59	cdent	mordred: \o/ but I think my dns is not up to date yet
14:37:07	mordred	cdent: oh! actually - that's not loading right for me right now - I will track down the issue
14:37:22	cmurphy	cdent: mordred static.o.o is having hard times
14:37:26	cmurphy	is why it's broken
14:37:33	mordred	ah- so it is
15:32:49	efried	mordred Is there a plan to expire old service-types.json. files from https://service-types.openstack.org/ ?
15:33:08	efried	Or do we keep those forever?
15:54:32	mordred	efried: there is not a plan - my current thinking is keep-them-forever - it shouldn't be many of them - hopefully we almost never update that data :)
15:56:19	efried	mordred Dig. Perhaps delete 'em all just before sta is considered truly primetime (maybe that's right now) to keep it pretty clean.
15:56:43	efried	mordred I mean, up to this point, we've mainly just been fleshing out entries, adding aliases, etc. There shouldn't be anyone relying on the older revs.
16:22:23	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Expand some discover.py docstrings https://review.openstack.org/482207
16:38:40	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Expand some discover.py docstrings https://review.openstack.org/482207
17:37:51	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
19:06:35	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Make Discover.version_data accept null max_version https://review.openstack.org/482250
19:21:31	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Expand some discover.py docstrings https://review.openstack.org/482207
19:29:17	openstackgerrit	Merged openstack/service-types-authority master: Remove validate.py main() https://review.openstack.org/478562
19:45:09	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Nix EndpointData.get_versioned_data(authenticated) https://review.openstack.org/482260
19:56:21	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Fix _run_discovery caching https://review.openstack.org/481754
19:57:09	efried	mordred ^^ is the split-out version specifically for https://bugs.launchpad.net/keystoneauth/+bug/1703447
19:57:10	openstack	Launchpad bug 1703447 in keystoneauth "URL caching in EndpointData._run_discovery is busted" [Undecided,In progress] - Assigned to Eric Fried (efried)
19:59:53	efried	mordred Actually what I meant to call attention to was the previous one: Nix EndpointData.get_versioned_data(authenticated) https://review.openstack.org/482260
20:00:09	mordred	efried: cool. will read in just a sec
20:00:10	efried	mordred That's the one that needs to get into ksa before the next release.
20:00:36	mordred	efried: btw - I'm pushing up a initial shell for the os-service-types repo (to consume the service-types-authority data) I made you an initial core on it - enjoy :)
20:01:18	efried	mordred Oh? Is there a charter or mission statement of some kind therein that I can read?
20:16:17	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: Miscellaneous cleanup in discover.py https://review.openstack.org/482271
20:17:47	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Add cookiecutter boilerplate https://review.openstack.org/482274
20:17:58	mordred	efried: there's one in the readme ^^
20:38:15	openstackgerrit	Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
20:54:54	efried	mordred Where does this boilerplate come from?
20:56:24	mordred	efried: the cookiecutter tool
20:56:38	efried	?
20:56:39	mordred	efried: http://git.openstack.org/cgit/openstack-dev/cookiecutter
20:56:52	mordred	those are templates for https://github.com/audreyr/cookiecutter
20:57:42	efried	mordred I gotta ask - why wouldn't we just put this function into sta?
20:58:33	mordred	efried: because - we want to keep sta a canonical locatoin for the data and not provide structural advantage to python over non-python
20:58:58	efried	okey.

Earlier   Later