Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-01-25
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
03:02:46 reedip_ stevemar : there?
03:17:52 stevemar reedip_: hola
03:20:19 reedip_ How r u ? ( I just realized that my patches have merged just now , so now I a, clueless about asking you for review comments :D )
03:24:21 openstackgerrit Reedip proposed openstack/python-openstackclient: Introduce overwrite/clear functionality in ``osc project set`` https://review.openstack.org/423612
03:29:57 reedip_ stevemar : one quick question, may seem silly
03:46:18 stevemar reedip_: shoot boss
03:46:24 reedip_ stevemar : https://review.openstack.org/#/c/356263/13/openstackclient/network/v2/port.py the __init__ from the called from the base, right ? I
03:47:26 reedip_ the issue is sending out the kwargs to the base class
03:47:33 reedip_ in the __init__ function
03:48:16 openstackgerrit Reedip proposed openstack/python-openstackclient: Introduce overwrite/clear functionality in ``osc project set`` https://review.openstack.org/423612
04:01:41 openstackgerrit Reedip proposed openstack/osc-lib: [WIP]Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
04:09:35 stevemar reedip_: yep, the __init__ from the base class will be called
04:10:08 reedip_ ok g8 :)
04:13:38 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff: Updated from global requirements https://review.openstack.org/425520
04:16:59 openstackgerrit Tin Lam proposed openstack/keystoneauth: Fix ClientException message property not set properly https://review.openstack.org/285757
04:49:10 dtroyer stevemar: re auto-allocated… yup, horrible name, I don't have a better one, we've been around with folk about that already. I'm OK with waiting on this one, looks like you've found more to fix
04:57:21 openstackgerrit Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
05:23:14 stevemar dtroyer: minor stuff anyway
06:17:36 openstackgerrit Merged openstack/cliff: Updated from global requirements https://review.openstack.org/425520
06:38:21 openstackgerrit Merged openstack/python-openstackclient: Switch server create to block_device_mapping_v2 https://review.openstack.org/407111
06:38:27 openstackgerrit Merged openstack/python-openstackclient: Add options to "server list" command https://review.openstack.org/405122
06:38:31 openstackgerrit Merged openstack/python-openstackclient: Support "--no-property" option in volume snapshot set https://review.openstack.org/416182
12:28:21 openstackgerrit David Stanek proposed openstack/keystoneauth: Fix ClientException message property not set properly https://review.openstack.org/285757

Earlier   Later