Posted	Nick	Remark
#openstack-sdks - 2017-03-31
21:14:30	figleaf	if they request multiple ranges that are all within the resource length, 416 isn't applicable
21:14:45	openstackgerrit	Cedric Brandily proposed openstack/python-openstackclient master: Enable to add/remove port to/from a server https://review.openstack.org/452328
21:14:54	figleaf	But since you don't support multiple ranges, 400 is returned
21:15:03	rosmaita	ok, 400 plus informative error message sounds good ... what about just dumping the entire content with a 200 ?
21:15:23	stevelle	reviewed where we talked about 400 vs 416, and I don't see any reason to dwell on that here. 400 or 200 is where I think we are at
21:15:38	figleaf	rosmaita: wait - are they requesting the whole thing, just in parts?
21:15:54	rosmaita	figleaf: not necessarily
21:16:09	rosmaita	i think we don't even want to try to find out
21:16:29	rosmaita	we are just saying "no multiple range requests are accepted"
21:16:34	figleaf	So no, I wouldn't return anything other than what they requested
21:16:46	figleaf	'zactly
21:17:03	figleaf	brb
21:17:27	rosmaita	i'm leaving, too ... have a good weekend, and thanks for the help
21:24:18	figleaf	no worries - enjoy your weekend!
22:39:47	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient master: start using api.versions https://review.openstack.org/451618
#openstack-sdks - 2017-04-01
02:44:15	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: Make block-device-mapping more stable and clear https://review.openstack.org/438141
02:44:19	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: Replace "Display Name" by "Name" in volume list https://review.openstack.org/423081
05:02:38	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311
05:07:15	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311
05:30:16	reedip	RuiChen: yiur patch https://review.openstack.org/#/c/449879 wont help OSC till the new lib is released, right?
05:34:19	reedip	RuiChen : I have reopened https://review.openstack.org/#/c/447938/9 to allow merge of https://review.openstack.org/#/c/383025/73
06:22:49	RuiChen	sorry, reedip , I'm in a meeting, I will take a look later
06:34:48	RuiChen	reedip: I'm back, yes, need a new openstacksdk version to ship the change
06:35:36	RuiChen	reedip: I prefer to remove the floating ip cache, then use a internal method to support getting floating ip by both uuid and address
06:36:02	RuiChen	reedip: then switch to new openstacksdk version when it's ready
08:05:18	openstackgerrit	Rui Chen proposed openstack/python-openstackclient master: Add help commands withouth auth in functional https://review.openstack.org/452407
08:25:16	reedip	RuiChen : Didnt get u
08:44:44	RuiChen	reedip: I means don't use ip_cache in _find_floating_ip() at all, I can push a patch as reference, if you don't mind
08:45:30	reedip	u mean in the function definition?
08:46:17	RuiChen	all related part in floating_ip.py
09:03:30	reedip	RuiChen: u mean in the function definition?
09:10:14	RuiChen	reedip: I remove all the cache related code now
09:10:27	reedip	oh , I also did the same :D
09:12:13	RuiChen	reedip: you're updating the patch?
09:12:16	reedip	RuiChen : Can you check this oatch ?
09:12:23	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Structure FindFloatingIP() to work without ip_cache https://review.openstack.org/447938
09:12:35	reedip	If you see anything different then please update it
09:12:41	RuiChen	ah, of course
09:12:58	reedip	sorry for the rework...but your idea struck me :)
09:14:19	reedip	Thanks :)
09:16:41	RuiChen	reedip: thank you work hard for this patch, my English is bad, it might make you confused :(
09:16:45	reedip	RuiChen : I think the Functional Test case needs to be changed
09:17:12	reedip	RuiChen : Python is not dependent on English ! :D
09:17:35	RuiChen	reedip: good point :-)
09:19:27	reedip	Let me know if anything needs to be changed
09:20:51	RuiChen	reedip: the current implement is same with I wish, thank you
09:21:15	reedip	RuiChen : Okay great, awaiting your +1 once Jenkins finishes troubling me :D
09:23:27	RuiChen	seems jenkins is recovered now, can't get log in the morning today
09:24:32	reedip	FUnctional test case may be damaged, checking that :)
09:29:52	RuiChen	reedip: hmmm, still can't get jenkins job log, 404 not found
09:30:30	RuiChen	reedip: there are a pep8 error in patch http://status.openstack.org/zuul/
09:30:49	reedip	Checking and fixing :)
09:36:07	RuiChen	+1 for https://review.openstack.org/447938
09:52:27	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Structure FindFloatingIP() to work without ip_cache https://review.openstack.org/447938
09:53:21	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Structure FindFloatingIP() to work without ip_cache https://review.openstack.org/447938
16:15:19	openstackgerrit	Cedric Brandily proposed openstack/python-openstackclient master: Enable to specify which fixed-ip to add to a vm. https://review.openstack.org/452235
21:07:29	openstackgerrit	Merged openstack/python-openstackclient master: doc: Remove local fork of apidoc https://review.openstack.org/447589
21:08:44	openstackgerrit	Merged openstack/python-openstackclient master: doc: Correct Sphinx warnings https://review.openstack.org/447590
21:18:16	openstackgerrit	Merged openstack/python-openstackclient master: Use Sphinx 1.5 warning-is-error https://review.openstack.org/446629
#openstack-sdks - 2017-04-02
14:33:59	reedip	ankur-gupta-f4 :if u r around, https://review.openstack.org/#/c/447938/ :D
14:40:14	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311
16:24:48	openstackgerrit	Monty Taylor proposed openstack/os-client-config master: Add ability to pass in user_agent https://review.openstack.org/452550
#openstack-sdks - 2017-04-03
02:04:24	reedip	o/
02:06:49	ankur-gupta-f4	reedip: dont file bug. Only if there was existing. But its just a refactor so its fine
02:07:19	reedip	ankur-gupta-f4 : were u just waiting for me with that response already typed ???? :D
02:07:31	reedip	I am not filing a bug :) , dont worry :P
02:08:53	ankur-gupta-f4	No just rmred u commented on patch
02:08:58	ankur-gupta-f4	N saw u wave
02:09:13	reedip	:D
02:21:53	openstackgerrit	Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
03:10:51	openstackgerrit	Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
05:06:30	openstackgerrit	Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
06:26:58	openstackgerrit	Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
06:31:07	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
06:48:46	reedip	RuiChen : I updated the Floating IP Patch, awaiting your review :)
09:13:00	openstackgerrit	Samriddhi proposed openstack/keystoneauth master: Updated inconsistent value of scope parameter https://review.openstack.org/452652
10:03:54	ZZelle_	ok
10:31:45	reedip	RuiChen : there?
10:57:14	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
11:06:06	openstackgerrit	Cedric Brandily proposed openstack/osc-lib master: Avoid to authenticate twice https://review.openstack.org/452711
11:26:48	openstackgerrit	Jens Rosenboom proposed openstack/python-openstackclient master: Fix block-device-mapping when volume_size is empty https://review.openstack.org/451432
12:29:19	reedip	ankur-gupta-f4 : woke up ?
12:29:55	reedip	sindhu : there ?
12:53:19	openstackgerrit	Dirk Mueller proposed openstack/cliff master: .gitignore: Ignore eggs https://review.openstack.org/449740
13:02:42	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
14:37:57	sindhu	reedip: hey wass up
14:38:38	reedip	hey hi , small issue with unicode in the functional test of https://review.openstack.org/356263
14:40:41	reedip	sindhu : the unset command fails to find the dict in the list of dicts in https://review.openstack.org/#/c/356263/23/openstackclient/network/v2/port.py L#987
14:41:28	reedip	cant think straight as of now, if you have some time to day, would be great if you can tell me how to ignore the unicodes
14:41:46	reedip	unicode mismatch is the only issue, the items are in the correct value
14:42:01	sindhu	reedip: cool, will look into it today :)
14:42:17	reedip	gr8 thanks :)
14:58:12	ankur-gupta-f4	reedip: good evening
15:05:36	reedip	ankur-gupta-f4 : hello
15:23:21	openstackgerrit	Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
15:23:43	reedip	sindhu : also need to get this merged so that the floating IP merges https://review.openstack.org/447938
15:25:05	sindhu	reedip: yes, the change in SDK won't help us now like u mentioned
15:25:29	reedip	yeah , so thats why want this to be merged so that the floating ip change merges soon
15:27:09	sindhu	yeah, I agree :)
15:44:35	openstackgerrit	Stephen Finucane proposed openstack/cliff master: WIP: Add 'cliff' Sphinx directive https://review.openstack.org/450322

Earlier   Later