Thursday, 2015-07-16

« Wednesday, 2015-07-15 Index Friday, 2015-07-17 »
*** ameybhide has quit IRC00:05
*** britthouser has joined #openstack-sdks00:28
jamielennoxdtroyer_zz, terrylhowe : any reason the volume type create command doesn't exist in cinder v2?00:28
dtroyer_zzjamielennox: no idea, it was a recent add00:35
dtroyer_zzv2 volume type commands00:35
jamielennoxdtroyer_zz: ok, well added a bug https://bugs.launchpad.net/python-openstackclient/+bug/1475060 and related https://bugs.launchpad.net/devstack/+bug/147506200:38
openstackLaunchpad bug 1475060 in python-openstackclient "Volume type create and list commands missing in v2" [Undecided,New]00:38
openstackLaunchpad bug 1475062 in devstack "Volume type fails with V2 API" [Undecided,New]00:38
*** yonglihe has joined #openstack-sdks00:43
*** Qiming has joined #openstack-sdks00:49
*** Qiming has quit IRC01:11
*** openstack has joined #openstack-sdks01:25
*** Yanyanhu has joined #openstack-sdks01:26
*** Qiming has joined #openstack-sdks01:53
openstackgerritTerry Howe proposed stackforge/python-openstacksdk: Add configuration function using os-client-config  https://review.openstack.org/19153502:03
*** hitalia has quit IRC02:49
*** britthou_ has joined #openstack-sdks03:30
*** britthouser has quit IRC03:33
*** briancurtin has quit IRC03:45
*** LinuxHorn has quit IRC03:45
*** boris-42 has quit IRC03:46
*** stevemar has joined #openstack-sdks03:48
*** Klumben has quit IRC03:50
*** LinuxHorn has joined #openstack-sdks03:56
*** briancurtin has joined #openstack-sdks03:57
*** boris-42 has joined #openstack-sdks04:07
*** britthouser has joined #openstack-sdks04:15
*** britthou_ has quit IRC04:16
*** Klumben has joined #openstack-sdks04:47
*** ameybhide has joined #openstack-sdks05:22
*** hitalia has joined #openstack-sdks05:26
*** Caleb-- has quit IRC05:51
*** terrylhowe has quit IRC05:55
*** Caleb-- has joined #openstack-sdks06:00
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Fix the way we call find_resource when only using ID  https://review.openstack.org/20239506:00
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Fix the way we call find_resource when only using ID  https://review.openstack.org/20239506:01
*** ParsectiX has joined #openstack-sdks06:06
*** ameybhide has quit IRC06:08
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Fix the way we call find_resource when only using ID  https://review.openstack.org/20239506:22
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Make trustee/trustor/project searchable by ID  https://review.openstack.org/20064606:31
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Fix the way we call find_resource when only using ID  https://review.openstack.org/20239506:31
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Make trustee/trustor/project searchable by ID  https://review.openstack.org/20064606:31
*** aufi has joined #openstack-sdks06:38
*** aufi has quit IRC06:38
*** aufi has joined #openstack-sdks06:38
*** stevemar has quit IRC06:42
openstackgerritGuojian Shao proposed openstack/python-openstackclient: only return endpoints that have url  https://review.openstack.org/20191107:25
*** ParsectiX has quit IRC07:29
*** stevemar has joined #openstack-sdks07:42
*** stevemar has quit IRC07:47
*** ParsectiX has joined #openstack-sdks07:51
*** boris-42 has quit IRC08:32
*** ParsectiX has quit IRC08:53
*** yonglihe has quit IRC09:06
*** ParsectiX has joined #openstack-sdks09:19
*** stevemar has joined #openstack-sdks09:24
*** stevemar has quit IRC09:28
openstackgerritchengkunye proposed openstack/python-openstackclient: add image share for client  https://review.openstack.org/20246409:33
*** jaosorior has joined #openstack-sdks09:41
*** Yanyanhu has quit IRC10:06
*** Qiming has quit IRC10:27
*** hitalia has quit IRC10:47
openstackgerritchengkunye proposed openstack/python-openstackclient: add image share for client  https://review.openstack.org/20246411:01
*** thrash|g0ne is now known as thrash11:18
*** Qiming has joined #openstack-sdks11:30
*** Guest___ has joined #openstack-sdks11:34
*** stevemar has joined #openstack-sdks12:24
*** stevemar has quit IRC12:29
*** stevemar has joined #openstack-sdks12:30
*** stevemar has quit IRC12:35
*** ParsectiX has quit IRC12:38
*** terrylhowe has joined #openstack-sdks12:41
*** trown|outttypeww is now known as trown12:42
*** ParsectiX has joined #openstack-sdks12:46
*** Guest___ is now known as smashwilson12:49
*** smashwilson has quit IRC12:50
*** stevemar has joined #openstack-sdks12:50
*** smashwilson has joined #openstack-sdks12:50
*** stevemar_ has joined #openstack-sdks12:51
*** stevemar has quit IRC12:54
*** stevemar_ has quit IRC12:55
*** stevemar has joined #openstack-sdks12:56
*** stevemar_ has joined #openstack-sdks12:57
*** stevemar has quit IRC13:01
openstackgerritMonty Taylor proposed openstack/os-client-config: Remove region list from single cloud  https://review.openstack.org/20254613:10
*** ParsectiX has quit IRC13:10
*** ParsectiX has joined #openstack-sdks13:11
*** bnemec has joined #openstack-sdks13:14
*** guojian has joined #openstack-sdks13:17
*** ParsectiX has quit IRC13:18
*** petertr7_away is now known as petertr713:21
*** bknudson has joined #openstack-sdks13:25
*** pm90_ has joined #openstack-sdks13:26
*** petertr7 is now known as petertr7_away13:34
*** petertr7_away is now known as petertr713:35
*** ParsectiX has joined #openstack-sdks13:40
*** pm90_ has quit IRC13:55
*** pm90_ has joined #openstack-sdks14:01
*** sigmavirus24_awa is now known as sigmavirus2414:02
*** ParsectiX has quit IRC14:05
*** zz_steveortiz is now known as steveortiz14:13
*** fifieldt_ has quit IRC14:19
stevemar_dtroyer_zz: ping14:39
stevemar_dtroyer_zz: can you 2x check bknudson's comment here: https://review.openstack.org/#/c/196297/3/keystoneclient/auth/cli.py14:40
dtroyer_zzwe actually don't use that…14:49
*** notmars has joined #openstack-sdks14:59
*** karimb has joined #openstack-sdks15:02
*** notmars has quit IRC15:09
*** trown is now known as trown|brb15:19
*** trown|brb is now known as trown15:24
openstackgerritGuojian Shao proposed openstack/python-openstackclient: add --project-domain option for user v3  https://review.openstack.org/20264515:27
*** stevelle_ is now known as stevelle15:27
*** petertr7 is now known as petertr7_away15:34
*** petertr7_away is now known as petertr715:35
*** guojian has quit IRC15:37
*** terrylhowe has left #openstack-sdks15:40
*** ig0r_ has quit IRC15:40
*** smashwilson has quit IRC15:44
*** pm90_ has quit IRC15:44
*** smashwilson has joined #openstack-sdks15:44
*** pm90_ has joined #openstack-sdks15:44
*** trown is now known as trown|lunch15:58
*** pm90__ has joined #openstack-sdks16:01
*** terrylhowe has joined #openstack-sdks16:01
*** pm90_ has quit IRC16:04
*** hitalia has joined #openstack-sdks16:05
*** petertr7 is now known as petertr7_away16:18
*** smashwilson has quit IRC16:22
*** pm90__ has quit IRC16:25
*** pm90_ has joined #openstack-sdks16:26
*** pm90_ has quit IRC16:26
*** pm90_ has joined #openstack-sdks16:27
*** smashwilson has joined #openstack-sdks16:27
*** notmars has joined #openstack-sdks16:30
*** stevemar_ has quit IRC16:32
openstackgerritGuojian Shao proposed openstack/python-openstackclient: enhance tests for user v3  https://review.openstack.org/20268716:35
*** Qiming has quit IRC16:44
*** pm90_ has quit IRC16:52
*** notmars has quit IRC17:06
*** pm90_ has joined #openstack-sdks17:07
*** pm90_ has quit IRC17:08
*** pm90_ has joined #openstack-sdks17:08
*** petertr7_away is now known as petertr717:09
*** petertr7 is now known as petertr7_away17:19
*** pm90_ has quit IRC17:29
*** pm90_ has joined #openstack-sdks17:29
*** karimb has quit IRC17:30
*** hitalia has quit IRC17:38
*** aufi has quit IRC17:43
*** bknudson has quit IRC17:45
*** trown|lunch is now known as trown17:46
*** stevemar has joined #openstack-sdks18:08
*** petertr7_away is now known as petertr718:09
openstackgerritDoug Hellmann proposed openstack/cliff: Set demo app up with deferred help  https://review.openstack.org/20273018:13
shalehquestion for changes to OSC. Is it preferred to catch things like OSError in the take_action methods or to let them bubble up to the higher level handler? I do not see much error checking in many of the classes.18:22
*** pm90_ has quit IRC18:24
terrylhowemostly errors are fatal, so they should just bubble upt shaleh18:27
shalehterrylhowe: ok. thanks.18:29
*** hitalia has joined #openstack-sdks18:38
*** pm90_ has joined #openstack-sdks18:43
shalehwhat about a case where there is a partial failure?18:44
shalehI noticed that OSC does not support image create for v2 glance api, so I added it with a little guidance from dtroyer.18:45
shaleha significant change in the API is that now the image create just sets up a queued image. The upload is needed to complete it.18:46
*** jaosorior has quit IRC18:46
shalehBut obviously the upload can fail. So I would like to show the queued image ID so it can either be passed in or deleted via a call to image delete.18:46
shalehI usually use existing behavior as a clue but I am not finding many existing classes with rich error handling.18:47
shalehIdeally OSC could say "image created with id <uuid>, pass that in the next time"18:48
dtroyer_zzshaleh: in most cases we just pass up the returned errors from the API as they are 'good enough'.  in this case I agree it needs to be more graceful, especially as it is doing more than one operation per command18:49
dtroyer_zzthe actual formatting of the output isn't obvious since I don't think we've needed to provide both returned info and error info18:50
shalehdtroyer_zz: exactly18:50
*** petertr7 is now known as petertr7_away18:50
dtroyer_zzbut the first step could be to return the image ID (and whatever else) as usal with the error on stdout18:51
shalehdtroyer_zz: one option is making it part of the exception text18:51
shalehI will start there and let the review suggest improvements18:51
dtroyer_zzI thought image create v1 already had an option to pass in an image ID to handle the reservations business18:52
shalehdtroyer_zz: I have it working, now for the pretty18:52
shalehdtroyer_zz: yes, --id exists in both API18:52
*** petertr7_away is now known as petertr718:52
dtroyer_zzdoes that over the 'handle previouus attempt' case?18:52
shalehpoint for me is OSC image create --file foo.img called twice, first is a failure leads to a queued image and another complete image18:52
shalehdtroyer_zz: in APIv1, the upload is part of the create so it is folded into the API error return18:53
shalehI suppose passing --id the next time does the right thing18:53
shalehhere the client is responsible for informing the user or deleting the queued image18:53
shalehboth are valid options18:53
shalehif the id came back for the create I could just invoke a delete on it18:54
dtroyer_zzright.  maybe deleting as part of the error handler is the right thing?18:54
shalehif the id did not come back then we can assume bad args or busted network18:54
dtroyer_zzdo we have any idea what glance users actually want to happen here?18:54
dtroyer_zzwould be a good thing to see what mordred and shade folk are doing…18:55
terrylhoweI guess that is the usecase for creating empty images.  create and upload later so you delete if it fails18:55
shalehif the id was not provided by the user, and one comes back then I will delete it on error. Otherwise I will include the id in the error output.18:55
dtroyer_zzthey may still be using v1 though18:55
shalehothers can comment from there18:55
mordreduhoh18:55
mordredwhat did I do?18:55
dtroyer_zzshaleh: I think that's a good place to start18:55
dtroyer_zzmordred: image uploads… in v2 it is a two step process, if step 2 fails (the actual image upload) it leaves a queued image ID behind18:56
mordredyes.18:56
* shaleh is adding support to OSC for image create/upload18:56
dtroyer_zzI'm thinking we should delete the queued image and start over rather than return the image ID and tell the user to try again but specify the ID18:56
dtroyer_zzbut first, is shade using image v2 yet?18:56
shalehand pointing out that error handling in OSC leaves something to be desired18:56
mordreddtroyer_zz: yes - for PUT I agree18:56
mordreddtroyer_zz: yes - we support ALL of the image upload combinations18:57
mordredit's a LOT of logic to do so, fyi18:57
mordredbecause there are two ways to upload images in v218:57
mordredboth of them are 2 steps18:57
dtroyer_zzright, one reason I'm opinionated about this18:57
mordredand you kind of have to know which one you want18:57
mordredif you do the task version18:57
mordredyou actually don't want to delete, you just want to resubmit the task18:57
mordreddtroyer_zz: oh - so - I left out a thign we do18:58
openstackgerritBrant Knudson proposed openstack/python-openstackclient: Query args for get-only  https://review.openstack.org/20274818:58
mordredwhich is we do an md5 and sha1 calculation so that we can skip unnecessary parts of re-uploading18:58
shalehmordred: what is the "we" here?18:58
mordredoh -sorry - shade18:58
dtroyer_zzI don't see the CLI using task version much, we're stateless between commands, so it would be in a singe command invocation18:58
terrylhoweosc meeting in 2 minutes?18:58
mordreddtroyer_zz: ++18:58
mordreddtroyer_zz: if I were you, I'd implemet the task logic but hide it18:58
dtroyer_zzyes, thanks for the reminder terrylhowe18:58
*** bknudson has joined #openstack-sdks18:59
mordreddtroyer_zz: so that OSC shows "image upload" and does what it needed18:59
shalehterrylhowe: in -meeting?18:59
mordredbut that might just be me18:59
dtroyer_zzshaleh: make sense?  and yes, -meeting18:59
terrylhoweyeh shaleh18:59
dtroyer_zzmordred: you have much more actual user experience here than most of us ;)18:59
dtroyer_zzthanks18:59
mordreddtroyer_zz: :)18:59
shalehthank you morded18:59
shalehlots and lots to learn still19:00
mordredshaleh: I recommend (highly) at least reading through teh image_create codepath in shade19:00
shalehmordred: shall do. Thanks for the pointer.19:00
shalehwhere do I find shade?19:00
mordredshaleh: http://git.openstack.org/cgit/openstack-infra/shade/tree/shade/__init__.py#n139119:00
shalehah, infra19:00
*** pm90_ has quit IRC19:01
mordredit runs from line 1391 to 155219:01
*** pm90_ has joined #openstack-sdks19:01
*** rbrooker has joined #openstack-sdks19:01
shaleheewww, lots of code in __init__19:03
*** trown is now known as trown|outttypeww19:03
mordredshaleh: yeah - ignore that part - we've got some reorg to do later19:03
*** pm90__ has joined #openstack-sdks19:08
openstackgerritMerged openstack/cliff: Allow subcommands to accept --help when using 'deferred_help'.  https://review.openstack.org/20125819:08
*** pm90_ has quit IRC19:10
*** bknudson has quit IRC19:19
*** petertr7 is now known as petertr7_away19:23
*** petertr7_away is now known as petertr719:23
*** dguerri` is now known as dguerri19:26
*** petertr7 is now known as petertr7_away19:31
openstackgerritBrant Knudson proposed openstack/python-openstackclient: Same exception handling for gets() in find_resource  https://review.openstack.org/20276019:50
openstackgerritBrant Knudson proposed openstack/python-openstackclient: Same exception handling for gets() in find_resource  https://review.openstack.org/20276019:52
shalehdtroyer_zz: re: api-list. If you want to talk about implementing it drop me an email or whatever communication you prefer.19:55
dhellmanndtroyer_zz, stevemar, terrylhowe : instead of the prefix based match, what do you think about using levenshtein distance, like git does?20:05
dhellmannthat's regarding https://review.openstack.org/#/c/202053/1320:05
terrylhoweI guess it should at least share the same algorithm with help.py dhellmann20:08
*** smashwilson has quit IRC20:10
dhellmannterrylhowe: heh, true. Both could share the implementation20:10
openstackgerritBrant Knudson proposed openstack/python-openstackclient: Same exception handling for gets() in find_resource  https://review.openstack.org/20276020:11
*** pm90__ has quit IRC20:26
openstackgerritTerry Howe proposed openstack/os-client-config: Rename 'endpoint_type' to 'interface'  https://review.openstack.org/20170420:41
openstackgerritTerry Howe proposed openstack/os-client-config: Have service name default to None  https://review.openstack.org/20131120:42
openstackgerritTerry Howe proposed openstack/os-client-config: Rename 'endpoint_type' to 'interface'  https://review.openstack.org/20170420:47
openstackgerritTerry Howe proposed openstack/os-client-config: Remove py26 and py33 from tox.ini  https://review.openstack.org/20278420:52
*** hitalia has quit IRC21:09
*** stevemar has quit IRC21:09
*** stevemar has joined #openstack-sdks21:12
*** stevemar has quit IRC21:17
openstackgerritTerry Howe proposed stackforge/python-openstacksdk: Add vendor to service filter  https://review.openstack.org/19774121:21
openstackgerritTerry Howe proposed stackforge/python-openstacksdk: Add provider to service filter  https://review.openstack.org/19774121:21
*** notmars has joined #openstack-sdks21:28
*** pm90_ has joined #openstack-sdks21:38
*** pm90__ has joined #openstack-sdks21:40
*** hitalia has joined #openstack-sdks21:41
*** boris-42 has joined #openstack-sdks21:42
*** pm90_ has quit IRC21:43
*** bnemec has quit IRC21:46
openstackgerritTerry Howe proposed stackforge/python-openstacksdk: Dynamically load services  https://review.openstack.org/15536221:56
*** pm90__ has quit IRC21:56
*** notmars has quit IRC21:56
openstackgerritTerry Howe proposed stackforge/python-openstacksdk: Dynamically load services  https://review.openstack.org/15536221:57
openstackgerritTerry Howe proposed stackforge/python-openstacksdk: Add provider to service filter  https://review.openstack.org/19774121:58
*** pm90_ has joined #openstack-sdks22:02
*** pm90__ has joined #openstack-sdks22:03
*** pm90_ has quit IRC22:06
shalehback to ask about UI some more22:31
shalehglance image-create has a --progress to show as the upload of the image occurs22:31
shalehideas on what that should look like in OSC?22:31
dtroyer_zzI had not planned on implementing that, any idea how popular it really is?22:32
shalehdtroyer_zz: not really22:32
shalehI was considering punting on it for now to be honest22:32
dtroyer_zzthat's one of the things we can ask in the UX study22:32
dtroyer_zzit's easier to add later than to take it away when it becomes a problem22:33
shalehdtroyer_zz: ++22:33
terrylhoweprogress bar on object store might be nice too22:34
shalehterrylhowe: that was my concern too. It needed to be added and shared.22:34
shalehIn theory we could re-use the one in python-glanceclient BTW.22:41
shalehit is right there to import22:41
terrylhowewell, using glanceclient for object store would be awkward if that is what you mean22:45
shalehterrylhowe: :-) no.22:49
shalehI mean the progress meter that glance uses is in its common/ dir and we already use other code from that module.22:49
shalehSo we could borrow it for now and port it to either a common lib or OSC directly22:50
*** openstackstatus has joined #openstack-sdks23:34
*** ChanServ sets mode: +v openstackstatus23:34
openstackgerritTerry Howe proposed stackforge/python-openstacksdk: Add wait for delete method  https://review.openstack.org/19379323:38
*** steveortiz is now known as zz_steveortiz23:40
*** pm90__ has quit IRC23:44
*** britthouser has quit IRC23:47
*** britthouser has joined #openstack-sdks23:50
*** dguerri is now known as dguerri`23:54
*** barra204 has joined #openstack-sdks23:54
*** pm90_ has joined #openstack-sdks23:56
« Wednesday, 2015-07-15 Index Friday, 2015-07-17 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!