Posted	Nick	Remark
#openstack-sdks - 2020-06-08
15:39:57	gtema	feel free to set +A yourself
15:40:26	mordred	gtema: thansk!
15:41:38	mordred	gtema: so - with the cliff patch, the idea is that in osc we'll have a file that has a ton of "commandmanager.add_command" calls pointing to the classes in-tree rather than registering our own plugins as entrypoints
15:42:00	mordred	then, since cliff will first attempt to resolve those from what it knows before scanning entrypoints
15:42:23	mordred	for the common case we can avoid the entrypoint cost- and only take that cost if someone asks for a command tha we don't know
15:42:36	gtema	yupp, makes sense
16:20:38	openstackgerrit	Monty Taylor proposed openstack/cliff master: Add load_plugins option https://review.opendev.org/733961
16:40:14	smcginnis	mordred: Hey there. Doing a search on requirements, I can't find any repos that have shade declaring it as a dependency in their requirements files. Does that seem right?
16:40:43	mordred	smcginnis: yeah - I don't think anything should be depending on shade at this point
16:40:58	mordred	smcginnis: everything that used to should be depending on openstacksdk at this point
16:41:01	mordred	smcginnis: same with os-client-config
16:41:09	smcginnis	OK, great. Another one I can clean out of global reqs then.
16:41:28	mordred	smcginnis: we're probably in a good place to change the release model for them now too
16:41:39	mordred	smcginnis: maybe just to independent?
16:42:11	smcginnis	mordred: If you think it will only need occasional releases from time to time, that may be better.
16:44:09	mordred	smcginnis: yeah- the're mostly on life-support for backwards compat
16:44:17	mordred	they definitely don't need cycle based releases
16:46:51	smcginnis	mordred: cinder
16:46:57	smcginnis	Darn clipboard.
16:47:01	smcginnis	mordred: https://review.opendev.org/734149
17:18:40	openstackgerrit	Merged openstack/cliff master: Remove python3.5 https://review.opendev.org/733968
17:18:40	openstackgerrit	Merged openstack/cliff master: Migrate to stestr https://review.opendev.org/733969
17:18:41	openstackgerrit	Merged openstack/cliff master: Remove unneeded tests https://review.opendev.org/733970
18:44:55	mordred	mnaser, dtantsur|afk, smcginnis: if you get a sec, mind reviewing https://review.opendev.org/#/c/733964 ? It's mostly just moving code from osc-lib to cliff so that we can remove it from osc-lib and then do a followup which will let us optimize osc by skipping entrypoints loading for common cases
19:04:59	dmsimard	hey o/ maybe I'm blind but I can't find an openstacksdk method that maps to keystone's /v3/users/{user_id}/projects ? https://docs.openstack.org/api-ref/identity/v3/?expanded=list-users-detail,list-projects-for-user-detail#list-projects-for-user
19:05:06	dmsimard	looking at the openstacksdk docs from https://docs.openstack.org/openstacksdk/latest/user/proxies/identity_v3.html#user-operations
19:11:34	dmsimard	I guess I found https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/identity/v3/_proxy.py#L507-L520 but not seeing in the docs :p
19:33:39	mordred	dmsimard: we might just not have it linked in the docs :(:
19:36:52	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add user_projects method to docs index https://review.opendev.org/734183
19:37:08	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add user_projects method to docs index https://review.opendev.org/734183
19:37:13	mordred	dmsimard: ^^ thanks!
19:38:22	dmsimard	\o/
21:18:06	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Add cliff to libs-from-git for devstack functional tips https://review.opendev.org/734095
21:20:45	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Remove os-client-config references https://review.opendev.org/734097
21:34:35	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: identity: autogenerate docs https://review.opendev.org/692931
21:34:35	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: image: autogenerate docs https://review.opendev.org/693025
21:49:24	mordred	amotoki: if you get some time, would you mind reviewing https://review.opendev.org/#/c/733964/ ? the more important bits are the followup change which won't start working until https://review.opendev.org/#/c/734000/ is landed, but that won't start working until we release osc-lib with the first change :)
21:50:05	mordred	then once all of that is in and happy, we can make an osc patch to register the in-tree plugins with add_command and skip the entrypoint scan cost for most of the commands
22:06:35	openstackgerrit	Hervé Beraud proposed openstack/cliff master: drop mock from lower-constraints and requirements https://review.opendev.org/734326
22:58:27	openstackgerrit	Merged openstack/osc-lib master: Add get_osc_show_columns_for_sdk_resource function https://review.opendev.org/698874
23:17:53	openstackgerrit	Merged openstack/python-openstackclient master: Add OpenStack Client for Manila docs https://review.opendev.org/706252
23:18:22	openstackgerrit	Hervé Beraud proposed openstack/keystoneauth master: drop mock from lower-constraints https://review.opendev.org/734359
23:40:27	openstackgerrit	Merged openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097
#openstack-sdks - 2020-06-09
00:21:52	openstackgerrit	Merged openstack/python-openstackclient stable/train: Resolve PEP8 https://review.opendev.org/732437
00:21:53	openstackgerrit	Merged openstack/python-openstackclient master: Make container list --all work https://review.opendev.org/728366
01:05:03	openstackgerrit	Gabriel Ramirez proposed openstack/python-openstackclient master: Allow openstack flavor set to update flavor description using name https://review.opendev.org/733688
01:17:27	openstackgerrit	Gabriel Ramirez proposed openstack/python-openstackclient master: Allow openstack flavor set to update flavor description using name https://review.opendev.org/733688
02:10:28	openstackgerrit	Merged openstack/python-openstackclient master: Add cliff to libs-from-git for devstack functional tips https://review.opendev.org/734095
02:10:30	openstackgerrit	Merged openstack/python-openstackclient master: identity: autogenerate docs https://review.opendev.org/692931
02:10:31	openstackgerrit	Merged openstack/python-openstackclient master: image: autogenerate docs https://review.opendev.org/693025
05:50:43	openstackgerrit	Merged openstack/openstacksdk master: Add user_projects method to docs index https://review.opendev.org/734183
08:50:10	openstackgerrit	Daniel Bengtsson proposed openstack/osc-lib master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694510
08:52:11	openstackgerrit	Hervé Beraud proposed openstack/openstacksdk master: Use unittest.mock instead of mock https://review.opendev.org/734455
09:43:10	openstackgerrit	Artem Goncharov proposed openstack/python-openstackclient master: [POC] project cleanup https://review.opendev.org/734485
10:11:40	openstackgerrit	Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
10:12:27	openstackgerrit	Hervé Beraud proposed openstack/shade master: Use unittest.mock instead of mock https://review.opendev.org/734552
10:13:57	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add notes to README about deps and versions https://review.opendev.org/734557
10:14:22	openstackgerrit	Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
10:16:00	openstackgerrit	wu.chunyang proposed openstack/cliff master: remove mock from requirements https://review.opendev.org/734566
10:17:53	openstackgerrit	Vishakha Agarwal proposed openstack/openstacksdk master: Add access rule CRUD support https://review.opendev.org/734549
11:10:52	openstackgerrit	subham rai proposed openstack/openstacksdk master: baremetal: support for volume connectors API https://review.opendev.org/718746
12:09:13	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add notes to README about deps and versions https://review.opendev.org/734557
12:41:26	iurygregory	mordred, dtantsur https://review.opendev.org/#/c/718746/ finally the functional tests for volume connectors are passing https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_96f/718746/14/check/openstacksdk-functional-devstack-ironic/96fb7a5/testr_results.html , so when you have a chance please review the patch =)
12:42:28	dtantsur	I will, not promising about today
12:42:50	iurygregory	sure =) don't need to be today
13:31:46	openstackgerrit	Merged openstack/cliff master: drop mock from lower-constraints and requirements https://review.opendev.org/734326
13:32:50	openstackgerrit	Artem Goncharov proposed openstack/python-openstackclient master: [POC] project cleanup https://review.opendev.org/734485
13:58:40	mordred	iurygregory: woot!
13:59:31	iurygregory	yup! sad that CI failed in openstack-tox-py37 D:
14:02:39	mordred	iurygregory: well, we can't have everything :)
14:03:34	iurygregory	mordred, it's CI we know the jobs have a chat to decide which one should fail =)
14:13:16	dtantsur	so true
14:23:57	mordred	the jobs have come alive
14:32:43	gtema_	mordred, dtantsur: first thoughts on https://review.opendev.org/#/c/734485/ are welcome (it's a mess with dependent changes on SDK, therefore currently not set at all)
14:33:33	gtema_	it's only about starting the "interface" in the OSC, and not about SDK implementation
14:33:44	dtantsur	the first thought: I have no clue what the difference between "cleanup" and "purge" is
14:33:46	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add notes to README about deps and versions https://review.opendev.org/734557
14:34:00	gtema_	purge will be dropped
14:35:33	gtema_	it's a old implementation of the project_cleanup, which will not be as "powerful" as my cleanup :D
14:36:32	dtantsur	why not reuse the old command?
14:37:17	iurygregory	they were always alive, know they can talk (this is the problem)
14:37:19	iurygregory	XD
14:48:57	gtema_	dtantsur - we can reuse the command, but I would suggest to do this when cleanup is ready. Currently I don't want to break somebody who use master (or if we need to do interim release)
14:54:55	mordred	gtema_: I'd think auth-project should default to true if project isn't given, no?
14:55:16	gtema_	agree. I just for the moment took the I/F from the purge
14:55:47	mordred	++
14:56:13	gtema_	on the other hand - if you run accidentally with admin - having auth_project will perhaps help avoid
14:57:46	mordred	gtema_: that is a good point
14:58:36	mordred	openstack cleanup --yes-I-mean-it<accidental enter>
14:59:03	mordred	gtema: if only you had access to a public cloud, you could run a bouncer in a vm ;)
14:59:06	gtema	yupp, something like that
14:59:13	gtema	sure
15:00:18	gtema	for the --yes-I-mean-it we can do a dry run, print results and have a countdown to allow aborting (giggle)
15:01:16	mordred	gtema: make it like 2fa - you have to respond to a text to proceed
15:01:33	gtema	hmm
15:01:36	gtema	nice idea
15:01:38	mordred	"we have sent a message to your mother, please call her to find out the code"
15:02:06	mordred	gtema: but overall I think that's looking good
15:02:34	gtema	sorry, I'm laying on the floor laughing out loud
15:02:41	mordred	:)

Earlier   Later