Posted	Nick	Remark
#openstack-sdks - 2017-03-16
20:07:04	dhellmann	dtroyer : it looks like appdirs comes from virtualenv
20:10:45	elmiko	edleafe: thanks, i'll take a look
21:38:35	openstackgerrit	Ankur proposed openstack/python-openstackclient master: Network L3 Router Commands for OSC https://review.openstack.org/385729
22:00:54	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/446675
#openstack-sdks - 2017-03-17
00:30:40	openstackgerrit	Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/446675
03:03:31	openstackgerrit	Qiming Teng proposed openstack/python-openstacksdk master: Avoid imports in openstack/__init__.py https://review.openstack.org/446823
03:05:07	openstackgerrit	Qiming Teng proposed openstack/python-openstacksdk master: Trivial: fix Template resource in orchestration https://review.openstack.org/446824
03:18:44	reedip	ankur-gupta-f4 , stevemar, RuiChen : Please review https://review.openstack.org/#/c/383025/
03:18:55	reedip	It has been in statis for a long time now
03:24:23	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311
05:32:13	openstackgerrit	Huanxuan Ao proposed openstack/python-openstackclient master: Support to add/remove multi users for "group add/remove user" https://review.openstack.org/446848
05:54:36	openstackgerrit	Qiming Teng proposed openstack/python-openstacksdk master: StackTemplate resource for orchestration https://review.openstack.org/446849
06:04:51	openstackgerrit	Qiming Teng proposed openstack/python-openstacksdk master: Shift some compute attributes within request body https://review.openstack.org/444933
06:41:44	openstackgerrit	Merged openstack/python-openstacksdk master: Shift some compute attributes within request body https://review.openstack.org/444933
07:05:08	RuiChen	yeah reedip , I'm around, long patchsets in https://review.openstack.org/#/c/383025/ , I'm jumping into it now : )
08:15:44	openstackgerrit	Qiming Teng proposed openstack/python-openstacksdk master: Add StackEnvironment resource to orchestration v1 https://review.openstack.org/446880
08:43:23	openstackgerrit	Huanxuan Ao proposed openstack/python-openstackclient master: Add "--sort" option to "volume backup list" in a common way https://review.openstack.org/446892
09:16:34	openstackgerrit	Qiming Teng proposed openstack/python-openstacksdk master: Add StackFiles resource to orchestration v1 https://review.openstack.org/446904
09:29:06	RuiChen	Hi reedip , I add some comments in patch https://review.openstack.org/#/c/383025
09:30:04	RuiChen	reedip only worried about self.ip_cache, other just some small change
10:04:32	openstackgerrit	Ethan Lynn proposed openstack/python-openstacksdk master: Remove type restrict of block_device_mapping https://review.openstack.org/446931
13:16:35	openstackgerrit	Merged openstack/python-openstacksdk master: Trivial: fix Template resource in orchestration https://review.openstack.org/446824
13:22:38	openstackgerrit	Merged openstack/python-openstacksdk master: Remove type restrict of block_device_mapping https://review.openstack.org/446931
15:35:48	openstackgerrit	Bence Romsics proposed openstack/python-openstackclient master: New formatter for trunk_details port property https://review.openstack.org/447035
15:36:47	ankur-gupta-f4	reedip: Will get to it this afternoon
16:11:30	openstackgerrit	Pepijn Oomen proposed openstack/python-openstacksdk master: Expose ha_state property from HA enabled L3 Agents. https://review.openstack.org/447050
16:12:26	pepijn	Qiming,briancurtin ^
16:13:08	pepijn	this was needed because the previous change was corrupted since it was submitted during a DoS from a CI-system, causing duplicate change-ids
16:13:42	pepijn	i thought this was fixed by abandoning the duplicates, but this caused problems for submitting further patch sets
16:16:21	pepijn	anyone else wants to vote on this change?
16:59:02	reedip	\o\
17:01:32	openstackgerrit	Ankur proposed openstack/python-openstackclient master: [WIP] Add support for Octavia Plugin https://review.openstack.org/447068
#openstack-sdks - 2017-03-20
01:27:04	openstackgerrit	Merged openstack/python-openstacksdk master: StackTemplate resource for orchestration https://review.openstack.org/446849
01:27:20	openstackgerrit	Merged openstack/python-openstacksdk master: Add StackEnvironment resource to orchestration v1 https://review.openstack.org/446880
01:41:06	openstackgerrit	Adrian Turjak proposed openstack/python-openstackclient master: Non-Admin can't list own projects https://review.openstack.org/376056
01:42:23	openstackgerrit	Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025
01:42:47	reedip	RuiChen : Fixed ur points ^^
02:32:34	openstackgerrit	Zhenyu Zheng proposed openstack/api-wg master: Mention max length limit information for tags https://review.openstack.org/447344
06:31:02	openstackgerrit	Ha Van Tu proposed openstack/python-openstackclient master: Add tags filter options to os network list command https://review.openstack.org/380012
06:45:33	reedip	RuiChen : we are getting error when changing the @mock.patch
06:48:32	openstackgerrit	Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025
07:56:02	RuiChen	reedip: yeah, looks like naming conflict, the import floating_ip is same with the class attribute floating_ip = network_fakes.FakeFloatingIP.create_one_floating_ip , in line 616
07:56:52	RuiChen	reedip: let me download the patch code to dive deep in code
07:57:55	reedip	:)
08:10:21	openstackgerrit	qtlu proposed openstack/python-openstackclient master: Add Progressbar for uploadimg, downloadimg image with v1 and v2 client https://review.openstack.org/447406
08:57:31	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025
08:59:01	RuiChen	Hi reedip, I commit a new patch ^^ and fix the mock naming conflict, that test cases pass in my local env, you can check the diff detail in patch
08:59:06	reedip	Welcome to the patch RuiCHen :D
08:59:15	reedip	Checked it
08:59:33	reedip	Didnt find anything MUCH different from the one I tried earlier
08:59:53	reedip	Lets see what Jenkins states :)
09:01:19	RuiChen	that is the detail https://review.openstack.org/#/c/383025/67..68/openstackclient/tests/unit/network/v2/test_floating_ip.py
09:01:54	RuiChen	@mock.patch => @mock.patch.object and rename something
09:02:04	RuiChen	just that :-)
09:06:44	reedip	yeah , the patch.object was missing :)
15:40:53	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: Make MAC address of port updatable https://review.openstack.org/442543
15:42:36	sshank	Can I please get reviews on patch: https://review.openstack.org/#/c/387611/ . I think its ready. Please look into it and comment.
15:42:45	sshank	reedip, ^
16:19:28	lbragstad	o/ hey folks - curious if anyone would be able to provide some clarity on a question i have about the guidelines for tags http://specs.openstack.org/openstack/api-wg/guidelines/tags.html
16:20:17	edleafe	lbragstad: We can try
16:20:41	lbragstad	edleafe the modifying tags section eludes to some set limit of tags that can be modified in a single request - http://specs.openstack.org/openstack/api-wg/guidelines/tags.html#modifying-the-tag-list
16:21:22	lbragstad	edleafe my first question is whether or not the API WG provides a baseline as to what that limit should be?
16:21:49	lbragstad	edleafe my second question is if the API WG has any recommendations on the total number of tags a thing can have
16:21:56	edleafe	lbragstad: it's not a design limit of tags; it's the limit of the web server handling the request
16:22:13	edleafe	The number of tags isn't as important as the total size of the tags
16:22:32	lbragstad	edleafe ah - sure, ok that makes sense
16:22:40	lbragstad	edleafe so that suggestion is to protect against that case
16:22:53	edleafe	IOW, if you have 100 10-byte tags, it will be better than 1 50GB tag :)
16:23:50	lbragstad	right
16:24:29	openstackgerrit	Nakul Dahiwade proposed openstack/python-openstackclient master: OSC Network Flavor Profile https://review.openstack.org/396783
16:24:41	lbragstad	edleafe we have a spec proposed to add tags to projects - https://review.openstack.org/#/c/431785
16:24:46	edleafe	The main point of that suggestion is the response code to return
16:25:44	lbragstad	so is the limit referring to the size of the tag, or the total number of tags?
16:26:05	lbragstad	because the example provided makes me think its the total size of the tags
16:26:15	lbragstad	not limiting the actual number of tags
16:27:21	edleafe	My understanding is that while the total size of all the tags is the limiting factory, an API developer might want to limit the number of tags allowed for more practical purposes
16:28:21	lbragstad	ok
16:28:21	edleafe	IOW, it would be nearly meaningless to assign 10K tags to a server
16:28:37	lbragstad	right
16:28:56	lbragstad	that kind of what i was getting at with my second question
16:29:16	lbragstad	does it make sense to limit the total number of tags a thing can have (not just limiting the request)
16:29:40	edleafe	lbragstad: my opinion is that it is dependent on the project, and the resource being tagged
16:30:05	edleafe	I don't know of any API-WG stance on that
16:30:24	lbragstad	edleafe ok - so for keystone, we could choose to limit the total number of tags a project can have, and we can choose to limi the total number of tags that can be modified in a single request
16:30:55	edleafe	lbragstad: well, those should be the same
16:31:00	lbragstad	I would think there would be a pretty good correlation between two things
16:31:03	lbragstad	edleafe ++
16:31:17	edleafe	since a PUT has to contain the complete list
16:31:26	edleafe	(not a PATCH)
16:31:29	lbragstad	so my second question doesn't make sense based on the answer from the first :)
16:31:37	edleafe	heh
16:31:52	lbragstad	but what about the case where single tags are added?
16:32:51	lbragstad	if i limit the total number of tags in a single request to 50, that shouldn't stop me from using PUT /servers/1234567890/tags/<random_tag> 50+ times
16:33:44	edleafe	lbragstad: it wouldn't, but you have to keep in mind that GET /servers/1234567890/tags is a thing that will return the full list
16:34:44	edleafe	So yeah, you can run individual PUTs as many times as you like, but you might have too big a body for the GET
16:35:07	lbragstad	edleafe at which point you'd have to start pruning things based upon that list, right?
16:35:31	lbragstad	because using PUT to modify the entire list would rewrite the whole thing
16:36:10	edleafe	lbragstad: yes, PUT with a body replaces the current tags with the body contents
16:37:18	lbragstad	edleafe so what i'm hearing is that it should be possible to exceed the total number of tags limited in a PUT request with individual tag management
16:38:00	edleafe	lbragstad: unless you do server-side checking, yes

Earlier   Later