Monday, 2015-09-21

« Sunday, 2015-09-20 Index Tuesday, 2015-09-22 »
*** Qiming has quit IRC00:00
openstackgerritMonty Taylor proposed openstack/os-client-config: Handle ksa opt with no deprecated field  https://review.openstack.org/22551900:10
mordredShrews: if you're around ^^ I need to land that because it's actually broken for ansible00:13
openstackgerritMerged openstack/os-client-config: Handle ksa opt with no deprecated field  https://review.openstack.org/22551900:20
*** gildub has joined #openstack-sdks00:41
*** chlong has joined #openstack-sdks00:42
*** Qiming has joined #openstack-sdks00:54
*** stevemar has joined #openstack-sdks01:16
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Format an images properties and tags  https://review.openstack.org/22550501:30
*** stevemar has quit IRC06:05
*** ParsectiX has joined #openstack-sdks06:08
openstackgerritOpenStack Proposal Bot proposed openstack/python-openstackclient: Imported Translations from Zanata  https://review.openstack.org/22539306:09
*** e0ne has joined #openstack-sdks06:10
*** e0ne has quit IRC06:14
*** gildub has quit IRC06:36
openstackgerritQiming Teng proposed stackforge/python-openstacksdk: Add support to stack update  https://review.openstack.org/22556007:00
*** ParsectiX has quit IRC07:00
*** e0ne has joined #openstack-sdks07:06
*** jamielennox is now known as jamielennox|away07:09
*** chlong has quit IRC07:17
*** ParsectiX has joined #openstack-sdks07:21
*** ParsectiX has quit IRC07:30
*** ParsectiX has joined #openstack-sdks07:31
*** ParsectiX has quit IRC08:09
*** ParsectiX has joined #openstack-sdks08:19
*** chlong has joined #openstack-sdks08:43
*** e0ne has quit IRC08:46
*** boris-42 has quit IRC08:53
*** ParsectiX has quit IRC09:15
*** boris-42 has joined #openstack-sdks09:24
*** ParsectiX has joined #openstack-sdks09:34
*** e0ne has joined #openstack-sdks09:58
*** ParsectiX has quit IRC10:32
*** Qiming has quit IRC10:48
*** ParsectiX has joined #openstack-sdks10:49
*** Shrews has quit IRC11:17
*** Shrews has joined #openstack-sdks11:17
*** gildub has joined #openstack-sdks11:20
*** terrylhowe has joined #openstack-sdks11:26
openstackgerritMerged openstack/os-client-config: Test kwargs passing not just argparse  https://review.openstack.org/22434611:30
*** Qiming has joined #openstack-sdks11:37
*** trown|outttypeww is now known as trown11:49
*** gildub has quit IRC12:14
*** e0ne has quit IRC12:44
*** pratikmallya has joined #openstack-sdks13:06
*** pratikma_ has joined #openstack-sdks13:08
*** pratikmallya has quit IRC13:12
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Add image create support for image v2  https://review.openstack.org/22544313:33
*** woodster_ has joined #openstack-sdks13:33
dhellmanndtroyer, terrylhowe: where do things stand with OSC, do you want a release or should I cut a stable branch from the most current release?13:44
terrylhoweI’m not sure dhellmann13:45
*** e0ne has joined #openstack-sdks13:47
dhellmannterrylhowe: ok. today's the day, so we need to figure that out. is dtroyer around this week? I haven't seen him online in a while.13:50
terrylhoweit looks like all the important patches were merged dhellmann but there is one new bug just openned that looks disturbing to me13:51
dhellmannterrylhowe: that sounds like a good candidate for a backport, at this point13:53
*** Qiming has quit IRC13:55
terrylhoweah, dhellmann that bug was not a problem actually, just give dtroyer a bit more time I guess13:56
*** Qiming has joined #openstack-sdks13:57
*** david-ly_ is now known as david-lyle14:01
mordredalso - I'd like to get the glance v2 patch landed14:01
mordredit's got a +2 from stevemar - and it helps remove a workaround in devstack14:01
mordreddhellmann, terrylhowe: https://review.openstack.org/#/c/225443/ and https://review.openstack.org/#/c/225505/ - they should be landable today14:02
*** jlvillal has joined #openstack-sdks14:05
*** ParsectiX has quit IRC14:09
*** jose4183 has joined #openstack-sdks14:09
*** e0ne has quit IRC14:10
*** ParsectiX has joined #openstack-sdks14:10
*** ParsectiX has quit IRC14:11
*** e0ne has joined #openstack-sdks14:14
*** zz_steveortiz is now known as steveortiz14:22
*** david-lyle has quit IRC14:22
dhellmanndtroyer, mordred: I don't have permissions on the os-client-config lp page https://launchpad.net/os-client-config14:23
dhellmanndtroyer, mordred : can one of you fix that up according to http://docs.openstack.org/infra/manual/creators.html#set-up-launchpad14:23
mordredthere's a launchpad page?14:23
mordreddhellmann: I cannot - I also do not have permissions14:24
dhellmannmordred: ack14:24
*** david-lyle has joined #openstack-sdks14:26
*** Qiming has quit IRC14:34
*** Qiming has joined #openstack-sdks14:35
openstackgerritMerged openstack/python-openstackclient: Set default auth plugin back to 'password'  https://review.openstack.org/22528414:40
openstackgerritMonty Taylor proposed openstack/os-client-config: Change ignore-errors to ignore_errors  https://review.openstack.org/22578214:41
openstackgerritOpenStack Proposal Bot proposed stackforge/python-openstacksdk: Updated from global requirements  https://review.openstack.org/22469914:44
*** pratikma_ has quit IRC14:46
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Change ignore-errors to ignore_errors  https://review.openstack.org/22582314:54
dtroyerdhellmann, mordred: os-client-config LP should be good now, openstack-release invited to drivers team15:10
dhellmanndtroyer: ty15:10
openstackgerritMerged openstack/os-client-config: Change ignore-errors to ignore_errors  https://review.openstack.org/22578215:13
*** pratikmallya has joined #openstack-sdks15:17
*** stevemar has joined #openstack-sdks15:22
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Format an images properties and tags  https://review.openstack.org/22550515:26
openstackgerritMonty Taylor proposed stackforge/python-openstacksdk: Change ignore-errors to ignore_errors  https://review.openstack.org/22589515:27
*** pratikma_ has joined #openstack-sdks15:30
*** pratikmallya has quit IRC15:33
mordredstevemar: zomg. we have to pass the string 'True' ?15:40
mordredShrews: ^^ you know how you love interacting with glance api?15:40
dtroyerImage v2 FTW!15:40
dtroyerI have a feeling that'll be the first convert to the SDK when that time comes…15:41
Shrewsmordred:  oh, let me count the ways...15:42
Shrewsfwiw, i did not get past zero15:43
dtroyerShrews: that could also be interpreted as not going negative…15:44
stevemarmordred: this has definitely been a learning experience for me15:45
*** Qiming has quit IRC15:46
mordredstevemar: check me here - from reading the glanceclient schema15:49
mordredthere is no such thing as is_public15:49
mordredthere is "visibility" which is an enum that can be public or private15:50
mordredstevemar: https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/image_schema.py#L173-L18015:51
stevemarmordred: yep, thats what stuart wrote to you :)15:51
*** stevemar has quit IRC15:51
mordredoh - hrm. I'm bad at reading15:51
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Add image create support for image v2  https://review.openstack.org/22544315:57
*** stevemar has joined #openstack-sdks15:57
*** stevemar has quit IRC16:02
openstackgerritMerged stackforge/python-openstacksdk: Updated from global requirements  https://review.openstack.org/22469916:04
*** steveortiz is now known as zz_steveortiz16:07
*** zz_steveortiz is now known as steveortiz16:20
*** stevemar has joined #openstack-sdks16:37
*** e0ne has quit IRC16:41
openstackgerritMerged stackforge/python-openstacksdk: Change ignore-errors to ignore_errors  https://review.openstack.org/22589516:42
*** chlong has quit IRC16:50
*** chlong has joined #openstack-sdks17:02
*** jamespd has quit IRC17:03
*** jamespd has joined #openstack-sdks17:04
openstackgerritMerged openstack/keystoneauth: Change ignore-errors to ignore_errors  https://review.openstack.org/22574017:16
*** e0ne has joined #openstack-sdks17:24
*** hitalia has joined #openstack-sdks17:32
*** subscope has joined #openstack-sdks17:37
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Add image create support for image v2  https://review.openstack.org/22544317:49
mordredstevemar: ^^ sorry to bother you - the patch is now updated per reviews (and the latest versions passes tests too)17:51
stevemarmordred: oh nice, i was just going to pull it down and fix it myself :P17:53
stevemarmordred: yeah, i wasn't sure about ignoring the options, but wanted to throw that out there as an option17:53
stevemardtroyer: ^^^17:53
mordredstevemar: I think also, like the ones with missing functionality, we could certainly do that as a follow on patch non-destructively17:54
mordredyou know, if we don't put the hidden optoins in now, and we hate it, we can always add them17:54
mordredbut as soon as we add them even hidden, we can't remove them17:54
stevemarmordred: minor doc stuff dude, sorry17:57
dtroyermy worry without the compat options is that changing the default from v1 to v2 will break folks immediately.  maybe we need to take that hit eventually anyway, is this the right time?17:57
* mordred cries17:57
mordredno worries17:57
mordreddtroyer: I think it's a real break because v2 just doesn't do several of those things - I personally worry more about accepting theoption and silently ignoring it17:58
mordreddtroyer: maybe a half-way thing - like - put in th esilent options but then test for them and throw an informative error message if someone tries to use them?17:59
mordreddtroyer: like "you tried to use --store, but you are talking ot glance v2 which no longer supports that function"17:59
mordredsince they cannot avoid being broken - but at least this way they'll know what's up18:00
dtroyerhmmm…. I like that idea.  it's more work but like handling a deprecation thats out of our control18:00
mordredyeah18:00
mordredI can code that up and put it in the next rev18:00
dtroyersounds good.  I should be in a place to go over your first review soon...18:00
mordredcool18:00
dtroyerit's see-the-surgeon day ;)18:01
mordreddtroyer: also - ignore most of my devstack patches for now - if we can land this change to openstackclient most of them are not needed18:01
mordredooh! fun.surgeons18:01
stevemardtroyer: ouch18:01
mordreddtroyer: I hear surgery is fun and everyone should do it whenever they can ...18:01
dtroyerthis is good though…all good news, I'm pain-free and the fusion is going well18:01
mordredwoot!18:01
stevemarfusion eh18:02
dtroyeryup, L5-S118:02
stevemardtroyer: you gonna be afk a while? or just an afternoon thing?18:02
dtroyeractually, I'll be back at kbd after I grab some lunch, it was this morning18:03
mordreddtroyer, stevemar: what's the right way to raise that error?18:10
mordredexceptions.CommandError ?18:10
openstackgerritMerged openstack/python-openstackclient: Change ignore-errors to ignore_errors  https://review.openstack.org/22582318:11
*** trown is now known as trown|lunch18:15
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Add image create support for image v2  https://review.openstack.org/22544318:19
mordredstevemar, dtroyer: there ya go18:19
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Add image create support for image v2  https://review.openstack.org/22544318:51
stevemarmordred: minor tweaks ^ review if you can18:51
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Format an images properties and tags  https://review.openstack.org/22550518:53
*** steveortiz is now known as zz_steveortiz18:58
mordredstevemar: looks great - now we just need the dtroyer stamp of approval19:07
mordredstevemar: and we may have accomplished v2 image support19:08
mordred'support'19:08
stevemar\o/19:08
stevemarwell for create anyway...19:08
stevemarsigh, --long doesn't work for list19:09
*** trown|lunch is now known as trown19:10
mordredwith your properties patch?19:11
stevemarmordred: pretty sure with master19:12
stevemarsince my patch didn't touch the list function19:12
stevemarjust create and show19:12
stevemarokay, relatively easy fix, just someone used format_dict instead of format_list for the tags19:13
openstackgerritSteve Martinelli proposed openstack/python-openstackclient: Use format_list instead of format_dict when listing images  https://review.openstack.org/22599419:18
stevemarmordred: terrylhowe dtroyer ^19:21
mordredstevemar: +119:22
dtroyerso at long last image create v219:28
dtroyerthanks guys19:29
*** hitalia has quit IRC19:36
stevemardtroyer: oh shittttt you +A'ed19:40
stevemarmordred: is the glance team going to make v3 of the API any time soon?19:41
stevemari thought i read that somewhere19:41
dtroyerso, y'all say its good, eh?19:41
dtroyerI've seen rumblings to the effect that any fixing to v2 will be a v319:42
dtroyerbut nothing confirmable19:42
mordredstevemar: yes19:42
mordred:)19:42
mordredno - I have no clue19:42
stevemardtroyer: yes, its bueno19:42
dtroyerand here I was sooooo haoy to finally have that ;)19:43
stevemarmordred: so since you have super TC power, can we draw a line in the sand and say no more CLI stuff into glancelcient :)19:43
stevemaror i guess that has to come from glance team :(19:43
dtroyerhow about we focus on the SDK and dropping our dependence on glanceclient?19:43
stevemardtroyer: that's still paramount19:44
dtroyerthat'll clean up some dependencies I'd love to be free of19:44
stevemarbut it would be nice to not have duplicated work19:44
dtroyerthat's a pipe dream19:44
stevemardtroyer: is it?19:44
stevemardtroyer: we're like 90% parity with cinder19:44
stevemarand not far off with glance19:44
mordredI think flaper would totally be on board with ditching the glanceclient shell thing19:45
stevemarmordred: true that, flaper already used OSC for zaqar19:45
mordredI don't know I can force anything with magic TC powers - but I think they'd be happy to join voluntarily19:45
stevemareven better!19:45
dtroyerstevemar: so I read that as glanceclient vs SDK duplication, not CLI duplication which now I see you might have meant19:46
stevemardtroyer: ++19:46
stevemardtroyer: thanks for reading my mind19:46
dtroyermy brain may now be narc-free but it's still a wee bit slowish19:46
stevemardtroyer: while i have you around... https://review.openstack.org/#/c/225956/19:47
stevemarany idea why it's failing jenkins? apparently saharaclient can't be installed properly now19:48
dtroyerlooks like at some point it's being installed from pip, which in a saharaclient functional test I'd not expect to see19:51
stevemargah19:54
*** hitalia has joined #openstack-sdks19:59
*** e0ne has quit IRC19:59
*** e0ne has joined #openstack-sdks20:00
openstackgerritSteve Martinelli proposed openstack/keystoneauth: add openid connect plugins  https://review.openstack.org/22499320:11
stevemarglance makes me so sad20:20
stevemarmordred: does glance not support delete image by name?20:21
stevemardtroyer: ^ ?20:21
dtroyerIIRC I never tried, always used find_resource() to get the ID20:21
mordredstevemar: I do the same thing in shade20:21
mordredstevemar: get id, then delete by ID20:22
stevemarmordred: how do you "get the id"?20:22
stevemari was thinking it would fail if i called something that had name conflicts20:23
mordredgiven a name_or_id param, we fetch the list of images then return one if name_or_id matches image['name'] or image['id']20:23
stevemarblah20:23
mordredalthough, if you're doing get20:23
mordredwe'll throw an error if you get two matches20:23
stevemarimage delete <name> is failing, which probably means <show> fails too20:23
mordredstevemar: https://github.com/openstack-infra/shade/blob/master/shade/_utils.py#L42-L118 are the pattern functoins20:24
mordredstevemar: (we do this for all openstack resources)20:24
stevemarmordred: we have a similar gizmo: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/common/utils.py#L52-L12120:25
mordredyup20:28
mordredlooks similar to me :)20:28
*** e0ne has quit IRC20:29
*** etoews has joined #openstack-sdks21:03
*** trown is now known as trown|outttypeww21:05
*** andrewbogott has joined #openstack-sdks21:07
*** stevemar has quit IRC21:19
*** andrewbogott has left #openstack-sdks21:20
*** stevemar has joined #openstack-sdks21:20
*** stevemar has quit IRC21:24
openstackgerritXav Paice proposed openstack/os-client-config: Add support for Catalyst as vendor  https://review.openstack.org/22605621:25
*** bnemec has quit IRC21:27
mordredoh! neat!21:30
openstackgerritMerged openstack/python-openstackclient: Add image create support for image v2  https://review.openstack.org/22544321:35
*** bnemec has joined #openstack-sdks21:41
mordredWOOT!21:46
openstackgerritMerged openstack/python-openstackclient: Format an images properties and tags  https://review.openstack.org/22550521:54
openstackgerritMerged openstack/python-openstackclient: Use format_list instead of format_dict when listing images  https://review.openstack.org/22599421:54
*** thrash is now known as thrash|g0ne22:14
*** jose4183 has quit IRC22:16
*** stevemar has joined #openstack-sdks22:16
*** stevemar has quit IRC22:20
*** gildub has joined #openstack-sdks22:22
*** pratikma_ has quit IRC22:24
openstackgerritMerged openstack/python-openstackclient: Imported Translations from Zanata  https://review.openstack.org/22539322:39
*** pratikmallya has joined #openstack-sdks22:57
*** jamielennox|away is now known as jamielennox23:23
*** gildub has quit IRC23:23
*** hitalia has quit IRC23:26
*** Qiming has joined #openstack-sdks23:36
*** pratikmallya has quit IRC23:55
*** gildub has joined #openstack-sdks23:56
« Sunday, 2015-09-20 Index Tuesday, 2015-09-22 »

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