Posted	Nick	Remark
#openstack-sdks - 2017-01-25
04:26:49	openstackgerrit	Steve Martinelli proposed openstack/python-openstackclient: change assert_show_fields to not fail on new fields https://review.openstack.org/424433
04:55:06	reedip	amotoki : ping, one query
04:55:27	amotoki	reedip: hey
04:56:18	reedip	amotoki : just asking , is it necessary to have the separator for classless route as ',' or can we use ';'? Example : value="169.254.169.254/32;40.40.40.5"
04:56:37	reedip	amotoki : just a query
05:02:04	amotoki	reedip: not sure
05:03:24	reedip	amotoki : hmm, maybe I would ask it once ,but anyways, let me fix the current issue. Actually , we can use ";" as a separator, but needs to be confirmed with openstackclient members
05:04:14	reedip	RuiChen, dtroyer, stevemar, ankur-gupta-f1 : suggestions requested to use ";" as a separator in https://review.openstack.org/#/c/356263/
05:04:34	reedip	in the value of dhcp-options
05:05:27	amotoki	reedip: The specified value is passed down to neutron underlying dhcp server (like dnsmasq) and I think a string with comma like 169.254.169.254/32,20.20.20.1 is what we actually set.
05:05:51	openstack	Launchpad bug 1605421 in python-neutronclient "Unable to add classless-static-route in extra_dhcp_opt extension" [Medium,In progress] - Assigned to Shih-Hao Li (shihli)
05:05:51	amotoki	reedip: so the bug author of https://bugs.launchpad.net/python-neutronclient/+bug/1605421 just wants to pass it as opt-value.
05:06:00	amotoki	reedip: it depends on a situation.
05:06:02	reedip	amotoki : we can change it later after parsing
05:06:28	amotoki	reedip: but it depends on a situation and IMHO OSC should not assume such backend behavior
05:06:50	amotoki	reedip: some other use cases may want to use a semi-colon.
05:06:53	reedip	amotoki : hmm
05:08:20	reedip	amotoki: okay, makes sense . The RFC which handles classless static routes doesnt state any descriptor to separate the routes, that depends on the implementation. Neutron has implemeneted it in a specific way. OSC can handle it, but yes there may be a use case for ;
05:08:31	amotoki	reedip: what we need to address is to allow users to pass (opt_name = classless-static-route, opt_value = 169.254.169.254/32,20.20.20.1)
05:08:53	amotoki	reedip: what kind of formatting is used is up to OSC side.
05:09:41	amotoki	double-quote escaping is okay. special parsing for comma is okay. it is up to OSC.
05:11:16	reedip	amotoki : ok, lets see what others have to say. I will however go forward the fix for MultiKeyValue pair for now.
05:11:34	reedip	amotoki: thanks for your help :)
05:28:12	openstackgerrit	Merged openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/423277
06:24:05	openstackgerrit	Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
06:25:30	openstackgerrit	Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
06:57:57	openstackgerrit	Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
06:58:41	openstackgerrit	Rui Chen proposed openstack/python-openstackclient: Rework port functional tests on json output format https://review.openstack.org/423970
07:00:27	openstackgerrit	Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
07:22:58	openstackgerrit	harishchandra proposed openstack/python-openstackclient: Correctly format the empty fields of "openstack network show" o/p. https://review.openstack.org/424733
07:53:58	openstackgerrit	harishchandra proposed openstack/python-openstackclient: Correctly format the empty fields of "openstack network show" o/p. https://review.openstack.org/424733
08:59:19	reedip	RuiChen : have a great vacation :)
09:11:40	openstackgerrit	Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
10:01:59	openstackgerrit	harishchandra proposed openstack/python-openstackclient: Correctly format the empty fields of "openstack network show" o/p. https://review.openstack.org/424733
10:12:17	openstackgerrit	harishchandra proposed openstack/python-openstackclient: Correctly format the empty fields of "openstack network show" o/p. https://review.openstack.org/424733
10:18:40	openstackgerrit	harishchandra proposed openstack/python-openstackclient: Correctly format the empty fields of "openstack network show" o/p. https://review.openstack.org/424733
12:13:33	stevemar	dtroyer: looks like we're short rui and huanxuan -- chinese new year :)
12:37:06	dtroyer	stevemar: timing!
12:38:34	stevemar	:)
12:45:36	reedip_	:)
12:45:52	reedip_	Yeah, RuiChen seemed pretty happy in the afternoon
12:53:28	stevemar	reedip_: its been a busy week, i don't blame him :P
12:53:38	stevemar	reedip_: i'll be very happy in about 48 hours
12:53:49	reedip_	heheheh :)
12:56:12	openstackgerrit	Steve Martinelli proposed openstack/python-openstackclient: change assert_show_fields to not fail on new fields https://review.openstack.org/424433
13:55:10	Cagelin	ping
13:56:28	Cagelin	Can anyone shine some light on the availability of OpenAPI spec files for (at least) keystone and maybe nova, glance and neutron? I'm wondering if there's an ETA available for when we could use this to generate client API's to use in our company.
13:56:53	Cagelin	(I just stumbled across this: https://www.openstack.org/videos/video/openapi-as-a-standard-a-new-way-forward-for-api-documentation-design-and-tool)
13:58:51	openstackgerrit	Merged openstack/js-openstack-lib: Add flavorList to Openstack wrapper https://review.openstack.org/406612
15:39:17	dtroyer	stevemar: https://review.openstack.org/#/c/424847/ should be ready to go now, andrey has removed his -1 and novalcient seems to be good
15:56:32	dtroyer	stevemar: also, do you want https://review.openstack.org/#/c/424433/ for stable?
16:10:20	openstackgerrit	Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585
16:14:49	openstackgerrit	Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585
16:43:14	openstackgerrit	Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585
16:44:48	openstackgerrit	Samuel Pilla proposed openstack/python-openstackclient: Adds domain specification for SetUser https://review.openstack.org/425276
16:47:17	openstackgerrit	Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3/3.5 into OSC https://review.openstack.org/421585
16:55:12	openstackgerrit	Samuel Pilla proposed openstack/python-openstackclient: Adds domain specification for SetUser https://review.openstack.org/425276
17:12:50	openstackgerrit	Colleen Murphy proposed openstack/python-openstackclient: Fix 'mapping set' return value https://review.openstack.org/425293
17:20:53	stevemar	dtroyer: umm
17:21:05	stevemar	dtroyer: i'm having trouble with https://review.openstack.org/#/c/424433/
17:24:31	stevemar	dtroyer: the "user show" command will have new fields from keystone server
17:24:59	stevemar	i was trying to fix our tests to not choke when the server adds new fields
17:25:14	stevemar	but i'm so scatterbrained right now
17:30:58	stevemar	dtroyer: easy +2/+A https://review.openstack.org/#/c/425293/ -- we had output for a 'set' command
17:35:54	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient: Add server_boot_from_volume() test https://review.openstack.org/425312
17:36:47	dtroyer	stevemar: ^^^ is a preamble to https://review.openstack.org/#/c/407111/ and was the only way I could convince myself that we weren't breaking things
17:37:05	dtroyer	which I am still not quite certain of
17:39:07	dtroyer	stevemar: do you want https://review.openstack.org/#/c/424433/ for stable/ocata?
17:47:02	openstackgerrit	Sindhu Devale proposed openstack/python-openstackclient: OSC Quota List https://review.openstack.org/379813
18:00:47	stevemar	dtroyer: i do, cause we're going to merge stuff into keystone in the next 24 hours that'll break the osc gate without it :)
18:03:22	dtroyer	ah, ok. I'll have a closer look when I finish up the block-device-mapping bits and some lunch
18:04:34	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient: Switch server create to block_device_mapping_v2 https://review.openstack.org/407111
18:57:01	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient: Support "--no-property" option in volume snapshot set https://review.openstack.org/416182
19:56:34	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient: Add server_boot_from_volume() test https://review.openstack.org/425312
19:57:20	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient: Switch server create to block_device_mapping_v2 https://review.openstack.org/407111
19:57:47	stevemar	dtroyer: please send me a list of patches you want me to push/look at :)
19:58:41	dtroyer	stevemar: I've finished my look through the queue, the two at the top are my only blockers: https://review.openstack.org/425312 and https://review.openstack.org/407111
19:59:10	dtroyer	https://review.openstack.org/#/c/408654/ and https://review.openstack.org/#/c/378058/ are probably OK but not blocking
19:59:46	dtroyer	I would love your opinion on my last comment in https://review.openstack.org/356219, I'm may want to change my suggestion
20:00:19	dtroyer	that's it. I'm running to the courthouse for car tags, back in an hour or so
20:03:46	stevemar	see ya
21:09:55	openstackgerrit	Steve Martinelli proposed openstack/python-openstackclient: change assert_show_fields to not fail on new fields https://review.openstack.org/424433
21:16:03	stevemar	dtroyer: this should work now, finally, https://review.openstack.org/#/c/424433/6
21:16:15	stevemar	dtroyer: i will refactor those to -f json format ASAP
21:20:41	dtroyer	Heh, I can't run any identity functional tests against my shiny new Newton DevStack 'cause it doesn't have V2.0 or v3 in the auth URL
21:21:08	dtroyer	wait, that's not it...
21:21:21	dtroyer	ah, it assumes its getting it from OS_AUTH_URL
21:27:49	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient: Fix Identity functional tests to not require OS_AUTH_URL https://review.openstack.org/425421
21:27:55	dtroyer	There, ^^^'s the ticket
21:31:40	dtroyer	stevemar: wow, that got ripe on the vine: https://review.openstack.org/#/c/405122/
21:31:45	openstackgerrit	Dean Troyer proposed openstack/python-openstackclient: Add options to "server list" command https://review.openstack.org/405122
21:37:48	openstackgerrit	Gage Hugo proposed openstack/keystoneauth: Fix ClientException message property not set properly https://review.openstack.org/285757
21:53:45	stevemar	dtroyer: ah the server list one
21:53:52	stevemar	that slipped through somehow
21:58:57	stevemar	dtroyer: gonna punt it through?
21:59:11	dtroyer	I think so
21:59:52	dtroyer	I'm looking for simple-ish things that would just pile up during the freeze
22:00:07	dtroyer	whatever gets merged by the time we need to release will make it :)
22:14:14	stevemar	dtroyer: your call on the block-device-mapper thing
22:14:19	stevemar	i don't know nova like you do :P

Earlier   Later