Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-01-25
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
22:14:58 dtroyer I think we should do it, it's been half-done since I wrote it originally. My concern was in breaking things and I think I've convinced myself that this doesn't do that
22:15:24 dtroyer my worry was mostly around how the —block-device-mapping arguemtns are formatted, and that isn't part of the change
22:15:50 dtroyer just fixing our assumption of 'vda' as the boot device name. boot_index is absolutely the right thing
22:16:24 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient: Adds domain specification for SetUser https://review.openstack.org/425276
22:16:38 stevemar dtroyer: ^ should be ready now
22:16:54 stevemar dtroyer: ++
22:17:17 stevemar dtroyer: right, that was my concern (the --block-device-mapping syntax) but that's not really our realm
22:17:44 stevemar so does https://review.openstack.org/#/c/408654/3
22:19:05 stevemar dtroyer: punted allowed_address through
22:19:21 dtroyer kk
22:19:42 dtroyer I wasn't stuck on it hard enough to block
22:26:38 openstackgerrit Merged openstack/python-openstackclient: Remove the fixed set of choices for network_type https://review.openstack.org/421598
22:27:31 openstackgerrit Merged openstack/python-openstackclient: Fix 'mapping set' return value https://review.openstack.org/425293
22:27:39 openstackgerrit Merged openstack/python-openstackclient: Add --project and --project-domain option to "volume snapshot list" https://review.openstack.org/417410
22:32:29 dtroyer stevemar: another for the good-to-have list when you get a chance: https://review.openstack.org/391331
22:32:39 dtroyer gimmie a network!
22:45:33 stevemar ohhh
22:45:38 stevemar i had been avoiding tht one
22:51:18 openstackgerrit Sindhu Devale proposed openstack/python-openstackclient: OSC Quota List https://review.openstack.org/379813
22:53:09 dtroyer stevemar: if you dont have time thats fine… it needs some understanding, and I wouldnt mind huanxuan's blessing too
22:54:00 stevemar dtroyer: switching gears to network meter: https://review.openstack.org/#/c/378058/15/doc/source/command-objects/network-meter-rule.rst
22:54:12 stevemar is --remote-ip-prefix required? why is it not in [] ?
22:55:03 dtroyer ya, its required
22:56:35 stevemar dtroyer: no set command?
22:57:41 dtroyer I guess not…honestly I had not noticed
22:58:15 stevemar dtroyer: no biggie
22:58:18 stevemar probably no set command
23:00:43 stevemar dtroyer: i really hope the SDK requirements patch doesn't get -2'ed from zuul
23:02:16 dtroyer the failures I've seen today laready feel 'odd', but that may still be residue from whatever it was overnight that gummed up the works
23:02:38 sindhu dtroyer: ping?
23:02:44 dtroyer sindhu: hey
23:02:47 stevemar dtroyer: well the integrated queue is 70 deep... http://status.openstack.org/zuul/
23:03:05 sindhu dtroyer: any conclusion on this one: https://review.openstack.org/#/c/383025/ ??
23:03:19 dtroyer stevemar: but things like this puzzle me: http://logs.openstack.org/19/356219/20/gate/gate-osc-dsvm-functional-ubuntu-xenial/6d01f4f/
23:04:33 stevemar dtroyer: i'll look at https://review.openstack.org/#/c/391331/ after #dadops is done
23:07:18 dtroyer sindhu: we are not just "moving over neutronclient commands" without evaluating their structure and making them fit OSC. I will not have time to dig into this particular issue until next week sometime to answer those questions myself. That is my suggestion for structuring the commands being implemented.
23:10:04 sindhu dtroyer: Ok cool ... thanks for taking a look. Will ping you again next week.
23:10:27 dtroyer it would be helpful (speed things up) to have answers to those questions
23:13:36 openstackgerrit Merged openstack/python-openstackclient: change assert_show_fields to not fail on new fields https://review.openstack.org/424433
23:33:09 openstackgerrit Dean Troyer proposed openstack/python-openstackclient: Support "--no-property" option in volume snapshot set https://review.openstack.org/416182
#openstack-sdks - 2017-01-26
00:34:42 openstackgerrit Merged openstack/python-openstackclient: Add 'allowed address pairs' option to 'port create/set/unset' https://review.openstack.org/356219
01:19:10 openstackgerrit Merged openstack/python-openstackclient: Add meter rule to OSC https://review.openstack.org/378058
01:19:17 openstackgerrit Merged openstack/python-openstackclient: Fix Identity functional tests to not require OS_AUTH_URL https://review.openstack.org/425421
01:36:11 openstackgerrit Merged openstack/python-openstackclient: Add server test for image and flavor lookups https://review.openstack.org/424901
01:40:15 openstackgerrit Merged openstack/python-openstackclient: Add server_boot_from_volume() test https://review.openstack.org/425312
01:51:19 openstackgerrit Merged openstack/python-openstackclient: To support '--project' and '--project-domain' options https://review.openstack.org/408654
02:02:30 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient: Switch server create to block_device_mapping_v2 https://review.openstack.org/407111
02:27:11 stevemar dtroyer: "network auto allocated topology" is a terrible command name
02:36:12 stevemar dtroyer: commented on https://review.openstack.org/#/c/391331/19 -- i'm OK with kicking it through
03:01:10 openstackgerrit Merged openstack/python-openstackclient: Adds domain specification for SetUser https://review.openstack.org/425276

Earlier   Later