Posted	Nick	Remark
#openstack-sdks - 2017-08-11
02:50:29	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794
07:24:40	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794
07:59:08	openstackgerrit	Zhaokun Fu proposed openstack/python-openstackclient master: add --password option for openstack server create Closes-Bug:1708570 https://review.openstack.org/490718
09:30:30	openstackgerrit	yfzhao proposed openstack/os-client-config master: Fix wrong links https://review.openstack.org/492924
09:42:57	openstackgerrit	sudhir agarwal proposed openstack/python-openstackclient master: Replace six.iteritems() with .items() https://review.openstack.org/492929
10:26:37	openstackgerrit	Merged openstack/keystoneauth master: Enable some off-by-default checks https://review.openstack.org/477775
12:05:01	_heena_py__	Hi
12:05:25	_heena_py__	I dont know if this is the right IRC but I needed to know a way to fetch cinder volume attachments in python
13:12:55	openstackgerrit	Sean Handley proposed openstack/os-client-config master: DataCentred supports Keystone V3 and Glance V2. https://review.openstack.org/493002
14:13:40	openstackgerrit	Merged openstack/python-openstackclient master: Update reno for stable/pike https://review.openstack.org/492660
14:36:24	openstackgerrit	Merged openstack/python-openstackclient master: Update release/stable branch table in docs https://review.openstack.org/492699
15:43:56	openstackgerrit	Merged openstack/python-openstackclient master: Add .htaccess for docs migration https://review.openstack.org/491631
17:26:56	openstackgerrit	Merged openstack/osc-lib feature/osc4: Refactor initialize_app() to allow overriding https://review.openstack.org/491846
18:18:14	openstackgerrit	Merged openstack/cliff master: Make openstackdocstheme an optional doc dependency https://review.openstack.org/491378
18:31:15	openstackgerrit	Merged openstack/os-client-config master: Update the documentation link for doc migration https://review.openstack.org/491374
22:22:17	fried_rice	mordred How do you feel about returning a list from ServiceTypes.get_service_data_for_project?
22:50:55	openstackgerrit	Eric Fried proposed openstack/os-service-types master: Non-unique project names: by_project as list https://review.openstack.org/493254
22:50:59	fried_rice	mordred ^
22:54:49	openstackgerrit	Eric Fried proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
22:56:08	fried_rice	Made ^ depend on -^
#openstack-sdks - 2017-08-12
11:41:12	openstackgerrit	OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/493298
11:50:14	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/491294
13:14:53	mordred	fried_rice: what if instead we don't do project: nova for placement? I think by_project's value is that it can answer the question "what is the service-type that people mean when they say {name}?" - placement is NEVER the answer to that question for name==nova - it just happens to live in the nova repo right now
18:31:47	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Support secondary services https://review.openstack.org/493325
18:34:26	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add flag to indicate a service is a secondary service https://review.openstack.org/493326
18:34:26	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
18:34:48	mordred	fried_rice: ok. there's a different stab at the issue
18:36:06	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add flag to indicate a service is a secondary service https://review.openstack.org/493326
18:36:06	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
#openstack-sdks - 2017-08-13
12:55:32	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Convert network segment functional tests to JSON https://review.openstack.org/491612
16:11:35	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
16:11:36	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Ensure git rev-parse output works in python3 https://review.openstack.org/493361
16:11:36	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add flag to indicate a service is a secondary service https://review.openstack.org/493326
16:11:37	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Only run json.dump once https://review.openstack.org/493363
16:11:37	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Use consistent url schemes https://review.openstack.org/493362
16:11:38	openstackgerrit	Monty Taylor proposed openstack/service-types-authority master: Add three more transformations https://review.openstack.org/493364
16:11:39	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Support secondary services https://review.openstack.org/493325
16:11:40	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Update unittests to be able to sync latest data https://review.openstack.org/493365
16:11:41	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Consume new mapping fields from service-type-authority https://review.openstack.org/493366
#openstack-sdks - 2017-08-14
06:38:33	openstackgerrit	Qearl proposed openstack/python-openstacksdk master: fix the bug that cannot create a listener by openstacksdk https://review.openstack.org/493449
07:55:06	openstackgerrit	Qearl proposed openstack/python-openstacksdk master: The patch is fix the bug that cannot create a listener by openstacksdk https://review.openstack.org/493449
08:28:54	openstackgerrit	Colleen Murphy proposed openstack/keystoneauth master: Allow discovery URLs to have trailing slashes https://review.openstack.org/492484
10:33:11	openstackgerrit	Qearl proposed openstack/python-openstacksdk master: fix the bug that cannot create a listener by openstacksdk https://review.openstack.org/493449
13:01:57	efried	mordred Okay. It ain't pretty, but it's better than breaking compatibility I suppose.
13:02:13	efried	Though then you went and did that with https://review.openstack.org/#/c/493366
13:11:03	openstackgerrit	Merged openstack/keystoneauth master: Allow discovery URLs to have trailing slashes https://review.openstack.org/492484
13:19:24	efried	mordred Why is jenkins ignoring https://review.openstack.org/#/c/493365/ ?
13:22:08	mordred	efried: I broke compat in 493366?
13:22:30	mordred	efried: and good question
13:22:45	efried	mordred Yeah, jenkins is ignoring that whole series.
13:23:28	efried	mordred The compat question... If my os-service-types is using a local mirror that's not current?
13:23:39	efried	local mirror of sta data, that is.
13:24:06	efried	I suppose that's not terribly likely.
13:24:52	mordred	efried: yah. but I do believe we could put in a check for that and at least give an "update your local mirror" error - or fallback to builtin data witha warning
13:25:34	mordred	I mean, the only reason to not use builtin data is in case the builtin data is stale - so if the lib discovers that the builtin data is newer than locally mirrored data, it should use builtin
13:25:39	mordred	efried: I'll follow up with that
13:27:40	efried	mordred So how is a guy supposed to use the new method, anyway? In the case of my nova patch to load up the right ksa adapter, I think it's really not terribly useful.
13:28:50	mordred	efried: you have a project and want to get a service-type right?
13:29:16	efried	mordred I have a service type and want to get a conf section name.
13:29:19	efried	in the case of placement.
13:29:36	efried	mordred So I have 'placement' and want to get 'placement' :)
13:29:43	efried	without hardcoding that specific mapping.
13:30:19	mordred	right. well, you're likely going ot need to hardcode that specific mapping, because nova has a naming convention internally that it is breaking with placement
13:30:21	mordred	BUT
13:30:23	mordred	that new method isn't for you
13:30:48	efried	mordred I actually did it without hardcoding: https://review.openstack.org/#/c/492247/5/nova/utils.py
13:31:19	mordred	awesome. that's great logic
13:31:36	efried	...unless nova puts a [nova] section in their conf. Then I'm effed.
13:31:50	mordred	well, that would be nova poking itself in the eye :)
13:32:03	mordred	for the other bit, the new method doesn't help you - you have always been able to get the project name for a service given the service type
13:32:24	mordred	in the other direction though, for documentation publication validation, one starts with project name and needs to go to service-type
13:32:39	mordred	so that you can verify "did this git repo publish docs to the correct url"
13:33:23	efried	mordred ...and enables us to add the placement service to sta at all.
13:33:27	mordred	yah
13:35:29	mordred	exactly. adding placement without adding the secondary flag breaks that question - but by making placement not show up in "what is the service-type for nova" - a user interested in knowing that both compute and placement come from nova has a hard time - that's what the new method is for (honestly not sure if anyone has that use, but the secondary flag removed it whereas your "return a list" patch
13:35:31	mordred	would have provided that info - so it seemed like a compromise)
13:35:46	openstackgerrit	liyi proposed openstack/python-openstacksdk master: Support node-adopt/preview CLI https://review.openstack.org/493542
13:37:24	cdent	why would someone looking up nova want to know about placement?
13:37:42	cdent	presumably someone who needs/wants placement would look up placeent
13:37:45	cdent	placement
13:37:57	mordred	no clue -but we have the data, so adding "get_all_service_data_for_project" was fairly easy
13:37:57	cdent	and if they haven’t heard of placement, there’s no need to look it up
13:38:16	cdent	yes, but I don’t get it. Why is “for project” meaningful?
13:38:28	mordred	meta-validation - things like docs validation jobs
13:38:55	mordred	"I have the nova repo, it made docs for /compute - is that correct?"
13:38:57	cdent	yeah, I’m semi-disappointed that we’ve added all that cruft on top of something that was supposed to be really simple
13:39:13	mordred	well - I mean - the data mapping was already there
13:39:17	cdent	we’ve gone from service types authority
13:39:30	cdent	to authority for several things
13:39:33	cdent	which is fine
13:39:44	mordred	you need to know that 'image' and 'glance' are related for multiple reasons
13:39:49	cdent	but soon it will have emacs, a kitchen sink, and book my flight reservations
13:40:27	mordred	it really won't - and we really haven't added all that much
13:40:42	mordred	the registry has always containe a mapping between service-type and project
13:41:05	mordred	because it's needed in some places for computer consumption (like nova config which uses "glance" for image config rather than "image")
13:41:31	cdent	(I was being sarcky, I’m not too disappointed with the changes, I’m more disappointed with the reality that requires them)
13:41:42	mordred	cdent: ok. sorry - I'm still on first coffee :)
13:41:42	cdent	right, and nova is wrong!
13:41:55	mordred	and yes - I totally agree about with the disappointment in reality
13:42:13	mordred	I need a tattoo "I'm disappointed in the reality that makes this thing necessary"

Earlier   Later