Posted	Nick	Remark
#openstack-sdks - 2018-05-18
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: Rename service_key to service_type https://review.openstack.org/558049
13:34:18	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935
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: Decode additional heat files https://review.openstack.org/567648
00:41:30	openstackgerrit	Merged openstack/openstacksdk master: Added few image properties to Image class https://review.openstack.org/544049
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?
12:01:19	gtema	dtansur: yes, you are right. Sometimes even much more low-level. They are basically the "basic rest calls" wrappers mostly
12:03:40	dtantsur	I see, it makes sense
12:03:49	dtantsur	so, this is intended, right?
12:04:33	gtema	well, yes. The connection methods are then giving a higher level of abstraction. So to say "function"
12:09:34	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
12:29:34	gtema	hey SDK/shade/ansible cores: I've found out, that May 25. is the community freeze for ansible 2.6 (https://docs.ansible.com/ansible/devel/roadmap/ROADMAP_2_6.html). If we want to land shade retirement (see https://github.com/ansible/ansible/pull/40532) - it's really time to review the PR. Otherwise we might be late for 2.6
15:38:47	mordred	gtema: awesome! I just started looking at it last night - looks great - I should be ale to actually look today
15:46:07	gtema	mordred: great. I just found, that in the meanwhile new module was added: os_server_metadata. I am fighting with rebasing. If it doesn't work we can still merge PR and then additional PR to switch new module (is not a conflicting change)
15:53:07	mordred	gtema: yah - I think merge one and then do an additional is fine if the conflicts suck too much
15:53:53	dtantsur	cdent: hey, is there a meeting today?
15:54:54	cdent	dtantsur: I reckon not. I also reckon we forgot to announce that there's not. If you're inclined to show up in case any one else does, that would probably be useful
15:55:08	mordred	gtema: there's one more thing we should do that's an issue - I think someone maybe started looking at it already ... (also, I can hopefully help a bit -today isn't as busy)
15:55:24	dtantsur	cdent: I can be here, but if everyone else is on the Forum...
15:57:30	mordred	gtema: which is any place using cloud.nova_client we need to replace those calls since sdk doesn't have novaclient connnections
15:57:55	mordred	it's mostly going to be lib/ansible/modules/cloud/openstack/os_server_action.py
15:59:21	mordred	I think we need to add a few calls to sdk - but in the mean time the action calls are pretty easy rest calls, so we can just use the rest passthrough and then in 2.6 replace the rest calls with proper sdk method calls
15:59:35	gtema	mordred: ooh, yes
15:59:52	mordred	oh - nevermind: https://github.com/ansible/ansible/pull/40462
15:59:53	gtema	so let's decide what to do now
16:00:45	gtema	ok, hopefully would be no merge conflict
16:01:48	mordred	ah - that was just one of the files - still need to do the actions one - lemme see if I can crank out a patch real quick
16:02:00	gtema	ok
16:06:38	mordred	gtema: http://paste.openstack.org/show/721908 it probably needs to go into your patch, since we need the code to use the sdk and not just shade for it to work properly
16:07:17	mordred	gtema: that patch is the first one - but it should e pretty easy to do the rest - the url call is the same, https://developer.openstack.org/api-ref/compute/#servers-run-an-action-servers-action has the payloads needed for the others
16:07:40	mordred	gtema: (I can make the rest if you want - whichever you think it easier)
16:09:35	gtema	mordred: please make them, since it's an evening here in Germany and my wife will shortly start blaming ;-)
16:11:55	mordred	:)
16:12:36	mordred	ok. I think maybe let's just land your PR now - and we can do the nova_client cleanup as a followup as a bugfix
16:12:37	gtema	I assume for servers.start it's json={'os-start'
16:12:55	mordred	http://paste.openstack.org/show/721909
16:13:03	mordred	gtema: I went ahead and did the whole set ^^
16:13:21	gtema	great, PR will be updated in couple of minutes
16:13:33	mordred	sweet. thanks - and thanks for doing the PR - it looks great!
16:13:44	gtema	thanks
16:17:14	gtema	mordred: it's there
16:19:36	gtema	mordred: thanks for helping
16:31:24	mordred	gtema: found one more if you're still around: http://paste.openstack.org/show/721911
16:31:58	gtema	yes, still here
16:37:31	gtema	mordred: ok, rebase completed and new module os_server_metadata is also part of the change
16:37:57	mordred	sweet. thanks!
16:41:26	gtema	mordred: another nova_client usage: os_server: L543 - remove_floating_ip
16:42:32	mordred	gtema: yah - there's a pr up for that one
16:42:46	mordred	gtema: I'm gonna see if I can get all of these in today
16:43:18	gtema	ok
16:43:53	gtema	right, missed that already. Code-search
16:46:14	gtema	mordred: do you know if one can "recheck" in github ansible - 3 last jobs were cancelled due to the following changes and now it stucked
17:06:34	gtema	mordred: I am off for today. If further changes are necessary - just leave a note. I will cover them tomorrow morning ASAP.
#openstack-sdks - 2018-05-25
00:09:03	harlowja	did shade ever release with the fix to http://paste.openstack.org/show/721925/ ?
00:10:09	harlowja	https://review.openstack.org/#/c/564861/ i think was the fix
00:10:14	harlowja	but doesn't ever seem to have gotten released?
01:29:00	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
02:45:51	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
06:18:22	openstackgerrit	tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
08:55:39	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Several improvements to resource.wait_for_status https://review.openstack.org/570538
09:16:48	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Several improvements to resource.wait_for_status https://review.openstack.org/570538
12:00:54	openstackgerrit	Dmitry Tantsur proposed openstack/openstacksdk master: Several improvements to resource.wait_for_status https://review.openstack.org/570538
14:34:07	gtema	mordred: any progress on https://github.com/ansible/ansible/pull/40532? ansible-dev is already buzzing around todays dev-freeze
16:10:43	openstackgerrit	Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974

Earlier   Later