Posted	Nick	Remark
#openstack-sdks - 2019-06-22
19:26:08	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Use cliff formattable columns in volume v2 commands https://review.opendev.org/658863
19:53:36	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Use cliff formattable columns in volume v1 commands https://review.opendev.org/658856
19:56:19	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Use cliff formattable columns in volume v2 commands https://review.opendev.org/658863
21:39:32	openstackgerrit	Merged openstack/python-openstackclient master: Change default security group protocol to 'any' https://review.opendev.org/518954
21:48:30	openstackgerrit	LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.opendev.org/650062
#openstack-sdks - 2019-06-23
00:45:36	openstackgerrit	Merged openstack/python-openstackclient master: Use cliff formattable columns in volume v1 commands https://review.opendev.org/658856
00:45:38	openstackgerrit	Merged openstack/python-openstackclient master: Use cliff formattable columns in volume v2 commands https://review.opendev.org/658863
02:31:16	openstackgerrit	Merged openstack/python-openstackclient master: Use cliff formattable columns in object storage commands https://review.opendev.org/658871
04:30:52	openstackgerrit	Merged openstack/python-openstackclient master: Use cliff formattable columns in image commands https://review.opendev.org/657911
04:49:34	openstackgerrit	Merged openstack/python-openstackclient master: Use cliff formattable columns in identity commands https://review.opendev.org/657912
#openstack-sdks - 2019-06-24
13:14:25	openstackgerrit	Corey Bryant proposed openstack/os-api-ref master: Add Python 3 Train unit tests https://review.opendev.org/667082
14:30:30	openstackgerrit	Bogdan Dobrelya (bogdando) proposed openstack/openstacksdk master: DNM - Start building pdfs of docs https://review.opendev.org/601659
14:30:31	openstackgerrit	Bogdan Dobrelya (bogdando) proposed openstack/openstacksdk master: Allow deeper levels of nesting for pdf builds https://review.opendev.org/667114
14:52:13	openstackgerrit	Bogdan Dobrelya (bogdando) proposed openstack/openstacksdk master: DNM - Start building pdfs of docs https://review.opendev.org/601659
15:09:11	openstackgerrit	Stephen Finucane proposed openstack/os-api-ref master: Add Python 3 Train unit tests https://review.opendev.org/667082
15:09:13	openstackgerrit	Stephen Finucane proposed openstack/os-api-ref master: Add support for Sphinx v2.1 https://review.opendev.org/667125
22:10:00	openstackgerrit	Merged openstack/os-api-ref master: Add support for Sphinx v2.1 https://review.opendev.org/667125
22:10:01	openstackgerrit	Merged openstack/os-api-ref master: Add Python 3 Train unit tests https://review.opendev.org/667082
#openstack-sdks - 2019-06-25
03:48:50	openstackgerrit	ZhaoBo proposed openstack/openstacksdk master: [WIP] force specify project_id during create Sg_rule may cause issue https://review.opendev.org/667261
04:58:14	openstackgerrit	jayashri bidwe proposed openstack/python-openstackclient master: Compute 'force_service_down' API returns 400 with latest version https://review.opendev.org/667264
05:38:26	openstackgerrit	jayashri bidwe proposed openstack/python-openstackclient master: WIP - Compute 'force_service_down' API returns 400 with latest version https://review.opendev.org/667264
06:28:36	openstackgerrit	ZhaoBo proposed openstack/openstacksdk master: force specify project_id during create Sg_rule may cause issue https://review.opendev.org/667261
07:00:59	Bidwe_jay	Hi All
07:01:37	Bidwe_jay	Working on bug at [1] Want to know the approach for the same uploaded patch WIP at [2]
10:16:39	openstackgerrit	Bogdan Dobrelya (bogdando) proposed openstack/openstacksdk master: DNM - Start building pdfs of docs https://review.opendev.org/601659
13:09:44	goldyfruit	Hi guys
13:10:58	goldyfruit	I would like to know if it's possible with the openstack client to have to required option but if one of them is used then the other one is not required anymore
13:11:02	goldyfruit	I read this: https://docs.openstack.org/python-openstackclient/latest/contributor/command-options.html
13:11:46	goldyfruit	I see the metavar/required but if I configure the two options with these two parameters then they are both required
13:26:36	efried	goldyfruit: Pretty sure that kind of conditional requirement can't be handled at the argparse level - you would have to do a manual check.
13:27:19	efried	but no idea if some kind of framework has been built into osc itself -- dtroyer?
13:28:40	goldyfruit	efried, thanks :)
15:06:08	openstackgerrit	Bogdan Dobrelya (bogdando) proposed openstack/openstacksdk master: DNM - Start building pdfs of docs https://review.opendev.org/601659
#openstack-sdks - 2019-06-26
02:18:41	openstackgerrit	ZhaoBo proposed openstack/openstacksdk master: force specify project_id during create Sg_rule may cause issue https://review.opendev.org/667261
02:34:51	openstackgerrit	pengyuesheng proposed openstack/openstacksdk master: Add Python 3 Train unit tests https://review.opendev.org/667494
14:59:07	edleafe	Bonus! Today the API-SIG Office Hour will be an extra minute long, starting now!!
15:02:39	edleafe	Doh! My calendar tricked me! It's not until tomorrow!
16:07:06	elmiko	tsk tsk edleafe ...
16:07:17	elmiko	XD
22:44:25	openstackgerrit	Corey Bryant proposed openstack/keystoneauth master: Add Python 3 Train unit tests https://review.opendev.org/667747
#openstack-sdks - 2019-06-27
16:00:04	edleafe	The API-SIG Office Hour is happening NOW!
16:04:40	elmiko	\o/
16:50:03	edleafe	Hey elmiko - don't know if you'll be around on the 4th, but I'll be on PTO all next week
16:51:09	elmiko	edleafe: ack, yeah, i will be out thurs/fri. should we send an email about no hours next week?
16:51:24	elmiko	i /think/ dmitry will be out as well
16:51:24	edleafe	Sure - thanks for volunteering! :)
16:51:27	elmiko	++
16:53:18	elmiko	done
17:00:08	elmiko	have a nice pto next week edleafe o/
17:00:29	edleafe	I sure will!
17:00:44	edleafe	Don't misbehave while I'm not around
17:04:25	elmiko	who... me? <.< >.>
17:18:59	openstackgerrit	James E. Blair proposed openstack/openstacksdk master: Replace nodepool func jobs https://review.opendev.org/667992
18:28:57	mgariepy	hello, i wonder if we can backport https://review.opendev.org/#/c/655889 to older branches (possibly down to queens) ?
#openstack-sdks - 2019-06-28
21:53:56	openstackgerrit	melanie witt proposed openstack/python-openstackclient master: Fix BFV server list handling with --name-lookup-one-by-one https://review.opendev.org/668255
22:06:45	openstackgerrit	melanie witt proposed openstack/python-openstackclient master: Fix BFV server list handling with --name-lookup-one-by-one https://review.opendev.org/668255
#openstack-sdks - 2019-06-29
13:29:07	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: Volume.Backup restore fixes https://review.opendev.org/665110
13:33:48	openstackgerrit	Artem Goncharov proposed openstack/openstacksdk master: WIP Use resource layer for compute.server cloud methods https://review.opendev.org/664593
#openstack-sdks - 2019-07-01
03:03:06	openstackgerrit	LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.opendev.org/650062
03:16:09	openstackgerrit	LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.opendev.org/650062
07:08:25	openstackgerrit	pengyuesheng proposed openstack/python-openstackclient master: Add Python 3 Train unit tests https://review.opendev.org/668364
07:13:35	openstackgerrit	pengyuesheng proposed openstack/python-openstackclient master: Update the constraints url https://review.opendev.org/668366
11:23:21	openstackgerrit	inspurericzhang proposed openstack/shade master: Mailing lists change openstack-dev to openstack-discuss https://review.opendev.org/668438
11:33:08	openstackgerrit	inspurericzhang proposed openstack/js-openstack-lib master: Mailing lists change openstack-dev to openstack-discuss https://review.opendev.org/668444
13:22:19	openstackgerrit	Merged openstack/openstacksdk master: Replace nodepool func jobs https://review.opendev.org/667992
19:09:10	openstackgerrit	Matt Riedemann proposed openstack/python-openstackclient master: docs: clarify compute service --service option https://review.opendev.org/668550
20:12:21	openstackgerrit	Merged openstack/keystoneauth master: Add Python 3 Train unit tests https://review.opendev.org/667747
20:23:54	corvus	dtantsur, mordred: would you mind reviewing https://review.opendev.org/667993 (it's a backport which is identical to 667992 which you just approved/merged -- thanks!)
20:24:46	mordred	corvus: looks great
20:25:18	mordred	corvus: the new nodepool job we've approved - does it use openstacksdk from source in the nodepool installation?
20:25:49	corvus	mordred: yes (the -src variants do)
20:26:11	mordred	awesome
20:40:36	corvus	don't think it's urgent?)
20:40:36	corvus	mordred: i did cut corners on the dogpile cache -- i had a lot of trouble getting it set up for some reason (even though i was copying what was in the devstack plugin), so it's not currently enabled (though it is still installed from source). that was pretty early on though, it's possible things are better enough we can just turn it back on. that's worth another look now that you're back (but i
20:42:50	mordred	corvus: yeah - I also don't think it's urgent - I think that whole layer needs a good amount of work
#openstack-sdks - 2019-07-02
15:08:43	openstackgerrit	LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.opendev.org/650062
#openstack-sdks - 2019-07-03
00:09:05	openstackgerrit	Merged openstack/python-openstackclient master: Update the constraints url https://review.opendev.org/668366
04:03:31	openstackgerrit	Merged openstack/python-openstackclient master: Add Python 3 Train unit tests https://review.opendev.org/668364
06:50:07	openstackgerrit	zhufl proposed openstack/openstacksdk master: Fix invalid assert state https://review.opendev.org/668794
12:21:06	brtknr	priteau: mordred gtema mnaser
12:22:23	priteau	Hi folks. I submitted a patch to Ansible about a month ago to fix an error when no volume service is available. Would anyone be able to review it? https://github.com/ansible/ansible/pull/57364
12:46:00	mordred	priteau: lgtm
12:50:47	priteau	Thanks mordred
13:31:47	cdent	dtroyer: if you have a moment to gaze on an osc-placement naming discussion, that would be helpful: https://review.opendev.org/666542
13:31:49	cdent	also: hi
13:41:59	dtroyer	cdent: oh that looks like a fun one, thanks ;)
13:42:13	cdent	dtroyer: I'm all about fun for you
13:42:44	cdent	I suspect placement's nature as "mostly admin" gives it a bit off leeway
13:43:09	dtroyer	(just thinking out loud here) I am not much of a fan of complex queries build from CLI options, if you need a complex query, build a DSL for that?
13:43:21	dtroyer	in OSC so far we have mostly gotten away with that
13:46:28	dtroyer	using the options to detect/attempt specific API microversions in itself is fine with me, sometimes it can be obvious, but that in: magic doesn't sit well with me. I don't like options with values that contain keywords but that turned out to be unavoidable in some places, so there is a bit of precedent for how those should look
13:49:05	dtroyer	so if I read that right the review is basically a straight port of the REST API semantics to the cli options?
14:13:55	mordred	CLI arguments like "--member-of in:<uuid1>:<uuid>" give me hives
14:14:55	mordred	passing complex need-to-be-parsed data structures is hard to deal with on the CLI (although makes sense in the REST API itself) - and invariably gets really brittle (see anything related to block_device_mapping)
14:15:14	mordred	(which I think is me saying I agree with dtroyer :) )
14:17:11	mordred	I'd rather see multiple options - like --member-of <uuid1> --member-of <uuid2> to make the && case like mentioned, but then maybe like --member-in <uuid> --member-in <uuid2> to construct member_of="in:uuid1:uuid2" ... but that's just off the top of my head, and I'm not sure how weird that woudl be to actually use
14:17:50	mordred	or, heck - a --member-in <uuid1>,<uuid2> would bother me less than the in: prefix
14:18:28	mordred	maybe I should say some of that in the review ...
14:19:05	cdent	dtroyer: sorry, stepped away. the suggestion from gibi is to do a straight port of the new microversin
14:19:16	cdent	and for the most part that's what's been doing in several places, but...not always

Earlier   Later