Posted	Nick	Remark
#openstack-sdks - 2017-05-25
11:42:19	sdague	having the 2 ids does end up being useful, so you can distinguish the fact that there are a lot of cross calls some times
11:42:53	sdague	the oslo side changes are all landed and released, so now it's just plumbing clients and callers
11:44:12	jamielennox	sdague: commented, code feels odd, i don't mind which way it's fixed
11:44:44	jamielennox	sdague: it does feel weird to be setting this on the Adapter, adapters are created per client so you're not going to share that request_id across a lot of connections that way
11:45:51	jamielennox	though with sessions lasting a long time there's not really a good spot to put a request level middleware in currently
11:47:38	sdague	jamielennox: ok, I'm not sure I quite understand the concerns there
11:48:15	sdague	because we don't want to be sharing request_id more broadly, the important thing is that it's relevant to the flow right now
11:49:06	jamielennox	note the way i always hoped this would work is that auth_token middleware currently passes down an auth plugin. that auth plugin when used to contact another service does all the correct things like uses a service token and presents the right catalog and stuff
11:49:30	jamielennox	and would be able to pass through a request-id that it received to the auth plugin and auto end up in other requests
11:49:48	jamielennox	but it was super slow work getting people to adopt that and it didn't get very far
11:51:00	jamielennox	sdague: yea, if it's useful there then that's fine. Re code, i just mean to store the variable in one place. I don't mind if it's a property around additional_headers or if you manually add it to headers later like we do for user_agent, just not stored twice
11:52:32	sdague	ok
12:09:22	sdague	jamielennox: for reference - https://review.openstack.org/#/c/467242/ is what the nova change looks like with some passing on to clients
12:10:06	sdague	https://review.openstack.org/#/c/467417/ is it all glued together, you'll see some global_ids logged in cinder / neutron with that
12:10:10	jamielennox	sdague: yea, you can get a fair way with additional_headers now, but i'm happy to make it a real thing
12:12:00	sdague	jamielennox: yeh, I've been just direct plumbing into neutron/cinder/glance clients, but the neutron folks wanted it in keystonauth. I'm kind of fine either way, because it's pretty much just a single field to pop out on the constructor if it's totally at the top level
12:12:51	sdague	even though we're setting global_request_id in constructors quite often in time, in code, it's typically consolidated to a single location in the caller
12:27:39	openstackgerrit	Matthew Booth proposed openstack/os-client-config master: Don't pop from os.environ https://review.openstack.org/468020
12:47:13	openstackgerrit	Sean Dague proposed openstack/keystoneauth master: Add global_request_id param to Adapter constructor https://review.openstack.org/467991
12:53:01	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: Rework server migrate to support cold and live https://review.openstack.org/460059
13:31:55	openstackgerrit	Monty Taylor proposed openstack/os-client-config master: Don't pop from os.environ https://review.openstack.org/468020
13:32:01	mordred	Shrews, dtroyer: if you get time, I'd really like to get the top five patches from https://review.openstack.org/#/q/project:openstack/os-client-config+status:open+label:Workflow=0 landed and a new occ out - they're all small/easy
13:32:16	mordred	Shrews, dtroyer: but there are a few behavior bugs that are biting people
13:38:10	Shrews	mordred: k
13:42:07	Shrews	mordred: is https://review.openstack.org/465195 still relevant with https://review.openstack.org/468020 ?
13:42:54	mordred	Shrews: the reason for writing it isn't - but it's still a good idea I think - the OpenStackConfig object is really intended to be a singleton
15:19:49	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311
15:34:29	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
15:36:57	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery https://review.openstack.org/459710
15:36:58	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Add document describing consuming version discovery https://review.openstack.org/459405
15:36:58	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Describe the publication of service-types-authority data https://review.openstack.org/462815
15:37:32	mordred	cdent, edleafe, elmiko: I believe I've gotten all current reviews taken care of
15:37:43	cdent	yay!
15:38:18	edleafe	mordred: on my list
15:46:52	elmiko	mordred: sweet, i'm rounding the corner on my last lap for the reviews you posted
15:47:59	mordred	elmiko: woot!
15:48:10	elmiko	also kudos on consuming-version-discovery.rst, this thing is a beast!
15:52:17	mordred	elmiko: it really is, isn't it?
15:52:34	mordred	elmiko: the scary part is that somehow I actually have all of that stored in my head
15:53:09	elmiko	mordred: that is scary...
15:53:31	elmiko	mordred: i really like your enumeration of the algorithms though, i think it will help readers understand this better
15:54:57	mordred	elmiko: thanks! ... I certainly hope so
15:55:13	mordred	the fun part is that next I need to write a spec for keystone about implementing this in keystoneauth :)
15:56:35	elmiko	good luck sir
15:58:48	openstackgerrit	Yuval Shalev proposed openstack/python-openstacksdk master: Added server console output method https://review.openstack.org/468078
17:32:48	elmiko	edleafe: draft up, https://etherpad.openstack.org/p/api-wg-newsletter
17:32:57	elmiko	i realize you may be out, i'll wait 15-20 then post it
18:08:38	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Add flavor value options https://review.openstack.org/302795
19:30:17	openstackgerrit	Merged openstack/python-openstackclient master: Convert volume functional tests into JSON format https://review.openstack.org/465208
19:31:34	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Add support for historical service type aliases https://review.openstack.org/460654
19:31:34	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery https://review.openstack.org/459710
19:31:35	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Add document describing consuming version discovery https://review.openstack.org/459405
19:31:35	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Describe the publication of service-types-authority data https://review.openstack.org/462815
19:31:36	openstackgerrit	Monty Taylor proposed openstack/api-wg master: Add guideline about consuming endpoints from catalog https://review.openstack.org/462814
19:53:51	openstackgerrit	Merged openstack/python-openstackclient master: Add functional test for volume service https://review.openstack.org/432282
20:20:10	dhellmann	dtroyer : in ATL we talked about a daemon mode for cliff/OSC. My notes say "add an option to emit a null value after the formatter output". Is that really all we need there?
20:50:05	dtroyer	dhellmann: that is the minimum needed to delimit return data for each command. I didn't want to invent a protocol for this so hopefully that is all we would need
20:50:24	dhellmann	dtroyer : ok
20:50:49	dhellmann	do we want a --daemon-mode that implies --interactive and also adds the null? or just a --null-delimited-output option?
20:52:08	dtroyer	I was only thinking about the —null option as interactive is already implied when no command is given
20:52:17	dhellmann	makes sense
20:52:28	dhellmann	just --null then?
20:52:44	dhellmann	not --null-delimited-output?
20:53:27	dtroyer	hmmm, it may need to be more specific, to be honest I have been thinking about cribbing the xargs -0 option, not sure if we need to be able to specify the delimiter char
20:53:38	dhellmann	oh, -0 works
20:53:56	dhellmann	ok, the other thing we talked about was "command extensions"
20:54:00	dhellmann	I'm looking at that right now
20:54:04	dtroyer	the semantics are not identical to xargs though so maybe that connection is not quite as useful
20:54:17	dhellmann	I can either load them from a separate namespace for each use (args, pre, & post hooks)
20:54:34	dhellmann	or I can load them from one namespace and expect a class with a specific API for each of those cases
20:54:39	dhellmann	any preference?
20:55:01	dhellmann	I'll make --null-delimited-output and -0 aliases
20:55:26	dtroyer	I've imagined it as a single class with essentially hook methods to override, but I don't have a huge preference yet
20:55:33	dtroyer	the alias sounds good
20:55:55	dhellmann	ok, I was leaning toward a class, too
20:56:04	dtroyer	having a single namespace does feel cleaner
20:56:27	dhellmann	I was going to make the namespace the application namespace + cmd_name.replace(' ', '_')
20:56:39	dhellmann	then plugins can have their own names
20:56:56	dtroyer	sounds good
20:56:58	dhellmann	k
21:42:20	dtroyer	dhellmann: if you're still around… with the major rev of cliff on the near horizon, do you want to wait for that release for this stuff to land ^^^^
22:55:10	openstackgerrit	Merged openstack/os-client-config master: Add helper method to fetch service catalog https://review.openstack.org/464238
22:55:20	openstackgerrit	Merged openstack/os-client-config master: Add ironicclient to constructors list https://review.openstack.org/464239
22:55:25	openstackgerrit	Merged openstack/os-client-config master: Keep a singleton to support multiple get_config calls https://review.openstack.org/465195
22:55:30	openstackgerrit	Merged openstack/os-client-config master: Don't pop from os.environ https://review.openstack.org/468020
23:05:18	openstackgerrit	Doug Hellmann proposed openstack/cliff master: add cmd_name argument to CompleteCommand https://review.openstack.org/468195
23:05:19	openstackgerrit	Doug Hellmann proposed openstack/cliff master: pass the command name from HelpCommand https://review.openstack.org/468196
23:05:19	openstackgerrit	Doug Hellmann proposed openstack/cliff master: add hook for manipulating the argument parser https://review.openstack.org/468197
23:05:20	openstackgerrit	Doug Hellmann proposed openstack/cliff master: add hook for get_epilog https://review.openstack.org/468198
23:05:20	openstackgerrit	Doug Hellmann proposed openstack/cliff master: add before and after hooks https://review.openstack.org/468199
23:05:35	dhellmann	dtroyer : let's see how closely this matches up with what you were thinking ^^
23:10:13	openstackgerrit	Doug Hellmann proposed openstack/cliff master: add hook for get_epilog https://review.openstack.org/468198
23:10:14	openstackgerrit	Doug Hellmann proposed openstack/cliff master: add before and after hooks https://review.openstack.org/468199
23:10:41	openstackgerrit	Merged openstack/os-client-config master: Revert "Revert "Use interface not endpoint_type for keystoneclient"" https://review.openstack.org/464299
#openstack-sdks - 2017-05-26
00:27:59	openstackgerrit	Takashi NATSUME proposed openstack/python-openstackclient master: List/show all server migration types https://review.openstack.org/450119
02:47:28	openstackgerrit	ZhangHongtao proposed openstack/python-openstackclient master: Add optional parameter "user_id" and "type" to list credentials https://review.openstack.org/468256
03:39:38	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: Replace "Display Name" by "Name" in volume list https://review.openstack.org/423081
10:59:25	RuiChen	osc folks, gate-osc-dsvm-functional-tips-ubuntu-xenial-nv no voting jobs failed, seems it's a devstack bug, but not sure, when USE_PYTHON3=True, keystonemiddleware is not installed in py27 lib, that cause uploading image failed because swift (run in py27) can't import keystonemiddleware, I had pushed a bug to trace it
10:59:27	RuiChen	https://bugs.launchpad.net/devstack/+bug/1693665
10:59:27	openstack	Launchpad bug 1693665 in python-openstackclient "[Gate] gate-osc-dsvm-functional-tips-ubuntu-xenial-nv failed because of uploading image error" [Undecided,New]

Earlier   Later