Sunday, 2015-09-20

*** stevemar has joined #openstack-sdks00:05
*** stevemar has quit IRC00:18
*** stevemar has joined #openstack-sdks00:18
*** stevemar has quit IRC00:22
*** stevemar has joined #openstack-sdks02:42
*** pratikmallya has joined #openstack-sdks03:17
*** pratikmallya has quit IRC03:22
*** boris-42 has joined #openstack-sdks03:48
*** pratikmallya has joined #openstack-sdks04:14
*** pratikmallya has quit IRC04:15
*** pratikma_ has joined #openstack-sdks04:15
*** woodster_ has quit IRC04:49
*** Qiming has joined #openstack-sdks05:01
*** stevemar has quit IRC05:04
*** pratikma_ has quit IRC05:12
*** pratikmallya has joined #openstack-sdks05:26
*** pratikmallya has quit IRC05:30
*** pratikmallya has joined #openstack-sdks05:55
*** pratikmallya has quit IRC06:42
*** pratikmallya has joined #openstack-sdks06:51
*** pratikmallya has quit IRC06:54
*** pratikmallya has joined #openstack-sdks06:57
*** pratikmallya has quit IRC07:01
*** stevemar has joined #openstack-sdks07:04
*** stevemar has quit IRC07:09
*** stevemar has joined #openstack-sdks08:06
*** stevemar has quit IRC08:10
*** jamielennox|away is now known as jamielennox08:59
*** stevemar has joined #openstack-sdks10:06
*** stevemar has quit IRC10:11
*** boris-42 has quit IRC10:20
*** jamielennox is now known as jamielennox|away11:26
*** pratikmallya has joined #openstack-sdks11:46
*** stevemar has joined #openstack-sdks12:07
*** stevemar has quit IRC12:12
*** stevemar has joined #openstack-sdks14:08
*** stevemar has quit IRC14:12
*** pratikmallya has quit IRC14:55
*** pratikmallya has joined #openstack-sdks15:00
*** pratikmallya has quit IRC15:19
*** Qiming has quit IRC15:24
*** stevemar has joined #openstack-sdks15:42
stevemarmordred: i think your image create patch is closer than the one thats been in the works15:45
mordredstevemar: k. I'll go pull the good bits from theirs15:45
*** pratikmallya has joined #openstack-sdks15:46
stevemarmordred: are you ever not available on irc :P15:46
mordredstevemar: also, you mentioned store and volume, and I don't see them in the v2 api15:46
mordredstevemar: nope :)15:46
mordredstevemar: irc bouncer + phone app == win15:47
stevemarmordred: hmm, those were in the v1 image create command, its cool if they are not in v2, let me look at the apis15:47
mordredI could very easily be wrong15:48
stevemarmordred: ah that's an osc-ism (to include volume and force)15:55
stevemarlooks like we call cinder to find the volume specified, then call cinderclient.upload_to_image()15:55
stevemarhttps://github.com/openstack/python-openstackclient/blob/master/openstackclient/image/v1/image.py#L218-L23015:55
stevemarwe can ignore that one for now15:55
stevemaris also looks like they removed the 'store' option in v2?15:57
mordrednod15:57
stevemarso the delta between v1 and v2 is -store and +tags16:01
stevemari have a massive TODO for the summit to get glance and cinder buy-in for osc - we're so close on those two that it's silly to iterate on 2 things16:02
*** pratikmallya has quit IRC16:05
*** pratikmallya has joined #openstack-sdks16:10
*** pratikmallya has quit IRC16:15
mordredstevemar: so, do you think we should do tags like properties?16:43
stevemarmordred: we could do that in another patch tbh16:52
stevemarbut yes, like properties, but without k=v pairs, just keys16:52
mordred++16:55
mordredstevemar: there is some good code in the other change - I'm pulling their good stuff into mine17:02
mordredstevemar: I'll add them both to co-authored-by17:02
stevemarmordred: cool cool17:02
stevemarmordred: just don't change the args, or face the wrath of dtroyer17:02
mordredstevemar: oh dear god no17:07
mordredstevemar: https://review.openstack.org/#/c/225443/17:32
stevemarmordred: ewww properties are flattened into kwargs?17:36
stevemarwhy ...17:36
mordredstevemar: I have stopped asking such questions so long ago17:49
stevemarmordred: sillly q about the patch17:50
stevemari had a comment about calling update17:50
stevemaryou replied with why its needed17:50
mordredstevemar: the answer is "because bunny rabbits hate you"17:50
stevemarthen changed update to get17:50
mordredstevemar: I did- then I learned about upload17:50
* stevemar goes to learn about upload17:51
mordredglanceclient/v2/images.py17:51
mordredline 20317:51
stevemarhmm, there now17:52
mordredstevemar: oh! and in fact, if I'd cargo-culted code from shade properly17:52
mordredstevemar: I would have seen that the v2 version of image_upload_put does, indeed, use upload17:53
mordredat least this is code to support put and not task-create17:55
stevemarmordred: hmm so .create() doesn't actually upload it yet, thats interesting17:59
stevemarand v1 doesn't have upload18:00
stevemarneat18:00
stevemari like that separation18:00
mordredyeah. honestly, I think it makes it clearer what's going on18:03
stevemarmordred: oh thats weird as balls - the image returned from glanceclient contains a schema with it, and you can't change the returned object if it violates the schema18:45
mordredstevemar: yes. the glancev2 client is ...18:52
mordredstevemar: well, it is very different than the rest of anything in openstack18:53
mordredbecause warlock18:53
mordredstevemar: https://github.com/openstack-infra/shade/blob/master/shade/meta.py#L333-L34818:54
mordredstevemar: we had to special case it18:54
stevemaryep, warlock seems to be the cause19:05
stevemari wanted to format stuff and things, but i can do that in another patch19:05
mordredstevemar: I'm hitting recheck on the "remove the workaround" patch19:06
stevemarmordred: i will have another patch to format the properties and tags into how we normally show them19:06
mordredstevemar: oh, awesome119:07
stevemarmordred: maybe 3 minor changes to your patch, then +219:08
mordredwoot19:08
mordredstevemar: you're teasing me. I see no changes listed?19:09
mordredstevemar: ah I understand now19:14
stevemarmordred: now we wait for dtroyer's final stamp of approval19:17
stevemarhmm, what happened to the bot, not notification for osc changes19:45
stevemarmordred: https://review.openstack.org/#/c/225505/19:46
mordredstevemar: the bot is having a sad today19:54
mordredstevemar: looks great19:55
*** e0ne has joined #openstack-sdks20:08
*** karimb has joined #openstack-sdks20:12
*** e0ne has quit IRC20:16
*** e0ne has joined #openstack-sdks20:19
*** e0ne has quit IRC20:24
mordredstevemar: your update had a testing sad20:25
stevemarmordred: yeah, apparently fakereources don't like iterkeys20:25
mordredstevemar: many times I feel like I spend more time coding to the fakes than actually solving the problem20:26
stevemaryup20:26
stevemarthats why i like funtional tests for this sort of stuff20:26
*** karimb has quit IRC20:27
mordredyup20:27
*** e0ne has joined #openstack-sdks20:29
*** karimb has joined #openstack-sdks20:29
*** e0ne has quit IRC20:36
*** e0ne has joined #openstack-sdks20:40
*** e0ne has quit IRC20:42
*** e0ne has joined #openstack-sdks20:46
*** karimb has quit IRC20:46
*** e0ne has quit IRC20:50
*** pratikmallya has joined #openstack-sdks20:57
*** e0ne has joined #openstack-sdks21:02
*** e0ne has quit IRC21:03
*** subscope has quit IRC21:11
*** boris-42 has joined #openstack-sdks21:53
*** stevemar has quit IRC22:43
*** chlong has quit IRC22:52
*** pratikmallya has quit IRC22:53
mordredaw. I just missed stevemar22:59
*** Shrews has quit IRC23:01
*** jamielennox|away is now known as jamielennox23:19
*** Shrews has joined #openstack-sdks23:19
*** Shrews has quit IRC23:22
*** Shrews has joined #openstack-sdks23:22
*** openstackgerrit has joined #openstack-sdks23:26
*** openstackgerrit has quit IRC23:31
*** stevemar has joined #openstack-sdks23:38
*** stevemar has quit IRC23:43
*** Qiming has joined #openstack-sdks23:46
*** openstackgerrit has joined #openstack-sdks23:47
*** openstackgerrit has quit IRC23:48
*** openstackgerrit has joined #openstack-sdks23:49

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