Posted	Nick	Remark
#openstack-sdks - 2018-05-18
12:31:02	jroll	so either you do that in small pieces (and have many major versions in the end), or you have to do it all at once to keep it in a single major version bump
12:31:40	jroll	ok, ignore "bad apis" for now, we can discuss that bit later :)
12:31:47	mordred	I think it should be more about making progress towards a place where the 'default' api people see is nicer than it is about removing things
12:32:26	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375
12:32:38	mordred	the reason I've come to like microversions so much is that I can completely and totally ignore them - and for the most part that's what I do
12:33:23	mordred	the problem I have with bumping a min is that it makes caring about microversions cease to be optional - and now microversions become the new major versions and there are 48 of them
12:34:15	mordred	I'd rather us add things / change things over a few cycles using microversions until we like where we've got ... and then do a legit new major version based on where we got by iterating over time with the microversions
12:34:51	mordred	that way consumers can continue to ignore microversions if they don't care about a specific feature and just program to the current major api version - and when a new major api version comes out then they can start caring about it
12:35:09	mordred	but if we did new majors too often, it would also cause undue burden on people
12:35:31	mordred	so it's certainly a balancing act that would take thought and likely can't be codified in a policy about 'when' a major is cut
12:35:40	jroll	why are you making so much sense, I want to argue
12:35:42	jroll	:)
12:36:02	dtantsur	:D
12:36:05	jroll	lol
12:36:14	jroll	I think I'm good with that proposal, though probably needs more thought
12:37:16	mordred	jroll: yes. much more thought - and discussion and whatnot
12:38:15	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Enable bare metal unit tests https://review.openstack.org/569401
12:38:16	dtantsur	mordred: this is quite urgent IMO ^^^
12:39:19	mordred	dtantsur: I agree - +A
12:40:21	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375
12:51:34	mordred	dtantsur: there's gonna me a small rabbit hole here - we need to land an osc patch that I pushed up this morning to fix the osc gate so that those patches will be green. but I have confidence we can get it done soon
13:00:16	dtantsur	I see
13:08:40	mordred	dtantsur: much text is needed in many places :)
13:09:05	dtantsur	yep :) since I'm doing some work with openstacksdk now, I may be able to write a few paragraphs there
13:09:25	dtantsur	(probably not today though)
13:46:42	openstackgerrit	Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --name-lookup option to server list https://review.openstack.org/568345
14:02:11	gtema	mordred: in my cloud router enable_snat is False by default. But in both openstacksdk and shade it is assumed to be True and therefore can be only switched off.
14:02:23	gtema	mordred: Nice comment in the code "
14:02:24	gtema	Sending it can cause a policy violation error
14:02:26	gtema	# on some clouds.
14:02:54	gtema	mordred: " makes me wonder how to fix the situation
14:14:19	mordred	gtema: it looks to me like it defaults to None... ah
14:16:28	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569422
14:16:34	mordred	gtema: how about that ^^ ?
14:18:01	gtema	great. Was also thinking about that, but was confused by this comment. shade will also need the same, since ansible is having same problem
14:19:38	gtema	mordred: what is the plan to switch ansible from shade to sdk?
14:20:03	mordred	gtema: yes - we'll need the same patch for shade
14:22:09	mordred	gtema: there's two parts to the plane. one is to make shade a thin wrapper around openstacksdk like we just did with os-client-config. the other is to update the ansible modules... I think we could start that transition in the modules before the thin-shim layer is done for ansible 2.6 - as shade for ansible 2.5 already has a dependency on openstacksdk, so people should already have it installed
14:23:34	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569425
14:23:41	mordred	gtema: there's the cherry-pick to shade
14:24:00	gtema	mordred: thanks
14:24:58	gtema	mordred: do you which me to start some work on transition?
14:26:07	gtema	mordred: ah those typos: do you wish, that I start?
14:28:49	mordred	gtema: :) yes - that would be great - although my brain is totally distracted by the summit next week so I may not be able to provide much help until it's over
14:29:16	mordred	gtema: if you wnated to start thinking about the modules though, https://github.com/ansible/ansible/blob/devel/lib/ansible/module_utils/openstack.py#L111 is probably a good place to start
14:30:02	gtema	mordred: no problem. Kurt will ping you if necessary ;-)
14:30:13	mordred	maybe a PR that changes that method to just import openstack and create a connection through sdk - and then update any of the modules that are declaring a minimum shade version or are using the 'shade' return value (usually for exception handling)
14:30:17	mordred	gtema: awesome :)
16:39:50	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569422
18:42:24	openstackgerrit	Joshua Harlow proposed openstack-infra/shade master: Add 'on_chunk_cb' param to list_servers https://review.openstack.org/569491
18:42:26	openstackgerrit	Monty Taylor proposed openstack/os-client-config master: Turn osc-functional-devstack-tips to nonvoting https://review.openstack.org/569492
18:45:31	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Turn osc-functional-devstack-tips to nonvoting https://review.openstack.org/569493
18:46:41	mordred	dhellmann: https://review.openstack.org/#/c/569398/ is green after your recheck
18:46:55	mordred	dhellmann: well, it's green on voting jobs
19:24:45	openstackgerrit	Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
19:25:24	openstackgerrit	Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
20:08:44	openstackgerrit	Slawek Kaplonski proposed openstack/python-openstackclient master: Don't display router's is_ha and is_distributed attributes always https://review.openstack.org/567620
20:17:23	openstackgerrit	Slawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule https://review.openstack.org/567828
20:43:36	openstackgerrit	Merged openstack/python-openstackclient master: Update command test for volume.v3 https://review.openstack.org/569398
21:17:37	openstackgerrit	Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
#openstack-sdks - 2018-05-19
13:34:18	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935
13:34:18	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049
15:48:30	openstackgerrit	Merged openstack/openstacksdk master: Enable bare metal unit tests https://review.openstack.org/569401
#openstack-sdks - 2018-05-20
19:30:33	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Plumb additional_user_agent support throughout https://review.openstack.org/569666
21:30:55	openstackgerrit	Monty Taylor proposed openstack/os-client-config master: Remove app_name config tests https://review.openstack.org/569673
21:31:13	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Plumb additional_user_agent support throughout https://review.openstack.org/569666
21:38:03	openstackgerrit	Monty Taylor proposed openstack/os-client-config master: Add release note for the vendor data regression https://review.openstack.org/569675
22:54:04	openstackgerrit	Merged openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375
22:58:29	openstackgerrit	Merged openstack/os-client-config master: fix saltstack breakage introduced in v1.31.0 https://review.openstack.org/569291
23:04:54	openstackgerrit	Merged openstack/os-client-config master: Add release note for the vendor data regression https://review.openstack.org/569675
#openstack-sdks - 2018-05-21
03:36:03	openstackgerrit	Trent Lloyd proposed openstack/python-openstackclient master: openstack image list: Support for community images https://review.openstack.org/569692
06:25:47	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
07:48:31	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
19:50:33	openstackgerrit	Matt Smith (_alastor_) proposed openstack/openstacksdk master: Adding block_storage v3 API support https://review.openstack.org/566355
20:27:44	openstackgerrit	Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
21:49:25	Guest83262	mordred: you around? I tried to catch you after your sdk talk, but you move too fast :)
#openstack-sdks - 2018-05-22
05:55:29	openstackgerrit	sonu proposed openstack/openstacksdk master: Fix filter style consistency for keystone assignment API https://review.openstack.org/569924
07:45:01	openstackgerrit	Chason Chan proposed openstack/python-openstackclient master: Update the content about Import Format https://review.openstack.org/569931
08:53:37	frickler	mordred: another hit on the "import openstack.py" ansible issue in #openstack. maybe a rename of that script in the ansible repo would be helpful?
08:54:17	gtema	fricker: I work on PR for 2.6 to switch from shade to openstacksdk
08:54:50	gtema	import importlib
08:54:55	gtema	sdk = importlib.import_module('openstack')
08:55:44	gtema	frickler: ^ ^
11:27:53	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
11:31:29	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
13:43:34	openstackgerrit	Artem Goncharov proposed openstack-infra/shade master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569425
14:16:16	openstackgerrit	Daniel Speichert proposed openstack/openstacksdk master: Added few image properties to Image class https://review.openstack.org/544049
16:42:06	openstackgerrit	Slawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule https://review.openstack.org/567828
16:42:17	openstackgerrit	Slawek Kaplonski proposed openstack/python-openstackclient master: Don't display router's is_ha and is_distributed attributes always https://review.openstack.org/567620
#openstack-sdks - 2018-05-23
07:35:18	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
09:20:50	openstackgerrit	lei zhang proposed openstack/python-openstackclient master: Add cliff project link https://review.openstack.org/570150
10:45:09	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
15:37:55	frickler	mordred: seems this bug still affects ocata+pike stable branches, would you have time to look into backporting? https://storyboard.openstack.org/#!/story/1733258
15:38:19	frickler	side note: would be interesting to discuss how we should track backports in storyboard
#openstack-sdks - 2018-05-24
00:41:30	openstackgerrit	Merged openstack/openstacksdk master: Added few image properties to Image class https://review.openstack.org/544049
00:41:30	openstackgerrit	Merged openstack/openstacksdk master: Decode additional heat files https://review.openstack.org/567648
05:54:23	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
11:37:13	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
11:54:24	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
12:00:16	dtantsur	mordred: hey! is my impression right that things exposed through proxies in openstacksdk (connection.compute.* etc) are a bit more low-level than things on connection itself?

Earlier   Later