Posted	Nick	Remark
#openstack-sdks - 2017-04-17
21:44:14	jamielennox	mordred: commented
21:44:21	mordred	jamielennox: thanks!
21:44:34	jamielennox	mordred: i would drop the client_name bit, i think it's confusing when you have to pass it to the config object
21:44:44	jamielennox	the additional part will take care of that anyway
21:44:59	jamielennox	just stick with app_name and app_version
21:46:13	mordred	jamielennox: replied - it was actually to expose it to users if they wanted to set it to something else themselves
21:46:24	mordred	but I'm happy to remove it and just leave it as app_name
21:47:11	jamielennox	mordred: well once you remove all the *clients os-c-c is the client, additional just gives you somewhere to put it until then
21:47:29	jamielennox	in the adapter i don't consider client_X something that a user would set
21:47:38	mordred	cool
22:13:34	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Split network tests https://review.openstack.org/457385
#openstack-sdks - 2017-04-18
00:11:07	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Low-level Compute v2 API: network https://review.openstack.org/457402
00:49:01	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Low-level Compute v2 API: network https://review.openstack.org/457402
01:03:16	openstackgerrit	yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
01:06:29	openstackgerrit	Merged openstack/python-openstackclient master: Split network tests https://review.openstack.org/457385
02:31:35	openstackgerrit	yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
03:33:52	openstackgerrit	Rui Chen proposed openstack/osc-lib master: Add formattable column base classes https://review.openstack.org/444760
03:34:20	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794
04:30:01	openstackgerrit	Reedip proposed openstack/python-openstackclient master: [WIP] Add Network Tag support to OSC https://review.openstack.org/457458
06:29:45	openstackgerrit	Reedip proposed openstack/python-openstacksdk master: [WIP] Network Tag https://review.openstack.org/443944
06:32:06	openstackgerrit	Reedip proposed openstack/python-openstackclient master: [WIP] Add Network Tag support to OSC https://review.openstack.org/457458
06:38:19	openstackgerrit	Reedip proposed openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
07:06:30	RuiChen	hi reedip, I add some comments in https://review.openstack.org/#/c/437367/
07:11:44	RuiChen	reedip_afk: a question, what's the meaning default_quota of subnet pool? I thought it's part of quota show command, but actually not :)
08:51:28	openstackgerrit	Monty Taylor proposed openstack/os-client-config master: Add ability to pass in user_agent https://review.openstack.org/452550
09:01:16	mordred	jamielennox: ok. I think I got it for real this time ^^
12:05:36	mordred	jamielennox: (also, replied re: the mocks- mostly wanted to leave reworking those tests for later)
12:05:59	jamielennox	mordred: yea, whatever on the mocks - the rest looks good
12:06:40	mordred	awesome. thanks!
12:06:52	mordred	I highly appreciate your help on these
12:07:15	mordred	jamielennox: also, fwiw, I started doing clients: https://review.openstack.org/#/c/456638/ :)
12:07:27	mordred	because I hate myself, most likely
12:13:34	jamielennox	mordred: left a comment on https://review.openstack.org/#/c/456638/
12:13:41	jamielennox	mordred: it's not wrong but there are easier ways
12:16:27	mordred	jamielennox: so just setting it on the class should work for LegacyJSONAdapter then?
12:17:07	jamielennox	mordred: yea, it's subclassed all the way down
12:17:20	jamielennox	(unless novaclient did something stupid)
12:17:26	mordred	jamielennox: cool. I just updated it to take that approach
12:55:37	RuiChen	dtroyer_zz: hi, I add a nova-network gate job for osc functional test, please take a look when you have time https://review.openstack.org/457622
13:34:52	openstackgerrit	Merged openstack/python-openstackclient master: Low-level Compute v2 API: network https://review.openstack.org/457402
13:35:53	openstackgerrit	Merged openstack/python-openstackclient master: Clean up password prompt work-arounds https://review.openstack.org/453832
13:52:47	dtroyer_zz	RuiChen: thanks, I think there are two things to verify yet in that job config
13:56:24	reedip_	RuiChen: you pinged?
13:59:31	openstack	bug 1683787 in OpenStack SDK "some network proxy router functions don't support id as argument" [Undecided,In progress] https://launchpad.net/bugs/1683787 - Assigned to Alon Bar Tzlil (aivrit)
13:59:31	openstackgerrit	Alon Bar Tzlil proposed openstack/python-openstacksdk master: fixes bug #1683787 https://review.openstack.org/457647
14:01:07	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Low-level Compute v2 API: floating ip pool https://review.openstack.org/457368
14:26:29	openstackgerrit	Stephen Finucane proposed openstack/python-openstackclient master: WIP! Start using 'cliff.sphinxext' https://review.openstack.org/452861
14:54:56	openstackgerrit	Brian Curtin proposed openstack/python-openstacksdk master: Allow router related functions to receive an ID https://review.openstack.org/457647
15:51:52	openstackgerrit	Dean Troyer proposed openstack/osc-lib master: Add more API support https://review.openstack.org/457703
17:19:01	openstackgerrit	Merged openstack/python-openstackclient master: Low-level Compute v2 API: floating ip pool https://review.openstack.org/457368
21:41:05	openstackgerrit	Merged openstack/os-client-config master: Add ability to pass in user_agent https://review.openstack.org/452550
22:04:22	dtroyer_zz	reedip: are you around?
22:05:00	dtroyer_zz	I'm looking at https://review.openstack.org/#/c/425515 and see some problems; I'd like to clarify the intent to know what the proper fix should be
#openstack-sdks - 2017-04-19
00:06:06	openstackgerrit	Dean Troyer proposed openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
00:09:02	dtroyer_zz	reedip: I went ahead and made some changes to beef up the tests ^^^^^
01:31:52	reedip	dtroyer_zz : just woke up
02:46:44	openstackgerrit	Dean Troyer proposed openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
02:46:51	dtroyer_zz	reedip: fixed pep8 ^^^
03:29:57	openstackgerrit	yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
03:35:19	openstackgerrit	yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
04:54:49	openstackgerrit	Qiming Teng proposed openstack/python-openstacksdk master: Allow router related functions to receive an ID https://review.openstack.org/457647
06:03:02	reedip	dtroyer_zz : ok, got it, thanks
06:37:36	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet-pools https://review.openstack.org/437367
06:38:32	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet-pools https://review.openstack.org/437367
06:59:38	openstackgerrit	yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
09:30:30	openstackgerrit	Stephen Finucane proposed openstack/python-openstackclient master: Explicitly set 'builders' option https://review.openstack.org/457960
10:01:40	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/python-openstacksdk master: Add direction field to QoS bandwidth limit. https://review.openstack.org/356512
12:42:13	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/python-openstacksdk master: Add direction field to QoS bandwidth limit. https://review.openstack.org/356512
12:44:13	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add direction field to QoS bandwidth limit. https://review.openstack.org/458056
12:53:14	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add direction field to QoS bandwidth limit. https://review.openstack.org/458056
12:56:21	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add direction field to QoS bandwidth limit. https://review.openstack.org/458056
12:58:59	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/python-openstacksdk master: Add direction field to QoS bandwidth limit. https://review.openstack.org/356512
13:38:43	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add direction field to QoS bandwidth limit. https://review.openstack.org/458056
13:50:56	openstackgerrit	Huanxuan Ao proposed openstack/python-openstackclient master: Fix network list functional test https://review.openstack.org/458080
14:29:27	openstackgerrit	Merged openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
14:29:57	openstackgerrit	Merged openstack/osc-lib master: Add formattable column base classes https://review.openstack.org/444760
14:30:02	openstackgerrit	Merged openstack/osc-lib master: Avoid to authenticate twice https://review.openstack.org/452711
14:39:02	openstackgerrit	Merged openstack/osc-lib master: Add more API support https://review.openstack.org/457703
15:06:09	rukh	ping openstack-sdks: trying to use the new openstack sdk andmade some good progres.. however there is some information I am having a hard time getting out .. eg a previous piece of code was getting this piece of info from the connection object member
15:06:49	rukh	How would I get the HorizonPublic endpoint out of any members ?
15:09:26	rukh	ping: Looking up self.conn.identity.endpoints() doesn't have the that endpoint as part of the structure
15:27:57	rukh	ping: stack = self.connection.orchestration.get_stack('e2b0ce7a-760e-42fd-918a-192a8da3f4fb')
15:28:20	rukh	is this a valid usage of the trying to get the stack object ? the id passed is the id of a stack from heat stack-list
15:40:01	dtroyer_zz	sfinucan: I just saw your release request for cliff, want to include a couple more things in that first? I just approved two more reviews
15:42:13	rukh	ping: any idea on my Q ? Sorry for bothering
15:44:09	sfinucan	dtroyer_zz: Yup, I don't see why not
15:45:14	dtroyer_zz	not sure if you're aware of this, but if those create a merge commit you need to use that…
15:47:27	sfinucan	I was not, but I am now
15:47:41	sfinucan	I think I just used the commit ref of HEAD last time
15:48:35	dtroyer_zz	which works fine, but since these are not merged yet we don't have the sha of the merge commit… if I had done them one at a time I don't think that would happen
16:03:23	rukh	ignore; I figured it out : thanks
16:08:18	sfinucan	dtroyer_zz: Ah, I thought they'd merged. I've just put that patch in WIP state until they do so
16:12:58	openstackgerrit	Merged openstack/python-openstackclient master: Fix network list functional test https://review.openstack.org/458080
16:40:57	openstackgerrit	Merged openstack/cliff master: Add support for epilogs https://review.openstack.org/454132
16:48:18	openstackgerrit	Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/455906
16:56:53	dtroyer_zz	sfinucan: there you go, b5593752e5502d9d4561bc91213f2285345f1cf2 is the sha you want to release on
17:25:22	rukh	ping openstack-sdks: is something seriously wrong with get_stack_environment() call ?
17:25:26	rukh	stack = self.conn.orchestration.find_stack(self.configs.plan)
17:25:32	rukh	environment = self.conn.orchestration.get_stack_environment(stack)
17:25:36	rukh	returns nothing

Earlier   Later