Wednesday, 2017-01-11

*** jamielennox is now known as jamielennox|away00:07
*** harlowja has quit IRC00:09
*** jamielennox|away is now known as jamielennox00:11
*** markvoelker has quit IRC00:12
*** markvoelker has joined #openstack-glance00:12
*** jroll has quit IRC00:14
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/41596100:16
*** jroll has joined #openstack-glance00:21
*** harlowja has joined #openstack-glance00:25
*** dirk has joined #openstack-glance01:33
*** briancurtin has joined #openstack-glance01:35
*** namnh has joined #openstack-glance01:39
*** markvoelker has quit IRC01:43
*** markvoelker has joined #openstack-glance01:44
*** markvoelker has quit IRC01:49
*** cdelatte has quit IRC01:54
*** jose-phillips has quit IRC02:12
*** chlong has quit IRC02:32
*** nicolasbock has quit IRC03:31
*** markvoelker has joined #openstack-glance03:36
*** links has joined #openstack-glance03:47
*** flwang1 has quit IRC03:59
*** gb21 has joined #openstack-glance04:21
*** jose-phillips has joined #openstack-glance04:27
*** jose-phillips has quit IRC04:30
*** ccesario has quit IRC05:02
*** adisky_ has joined #openstack-glance05:11
*** gcb has quit IRC05:13
*** ccesario has joined #openstack-glance05:14
openstackgerritM V P Nitesh proposed openstack/python-glanceclient: Added a check for the length of the tag to be less than 255 Closes-Bug: #1640442  https://review.openstack.org/41868605:30
openstackbug 1640442 in Glance "glance image-tag-update, not updating a tag whose length is more than 255" [Undecided,Confirmed] https://launchpad.net/bugs/1640442 - Assigned to M V P Nitesh (m-nitesh)05:30
*** ratailor has joined #openstack-glance05:45
*** bkopilov has joined #openstack-glance06:06
*** itisha has quit IRC06:12
*** dims has quit IRC06:13
*** pcaruana has joined #openstack-glance06:25
*** ezoszed has joined #openstack-glance06:35
*** groen692 has joined #openstack-glance06:42
*** bhagyashris has joined #openstack-glance06:51
*** mfedosin has joined #openstack-glance06:54
*** groen692 has quit IRC07:02
*** mosulica has joined #openstack-glance07:03
*** mosulica has quit IRC07:08
*** tesseract has joined #openstack-glance07:08
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient: Add request id to returned objects  https://review.openstack.org/35289207:10
*** mosulica has joined #openstack-glance07:15
*** udesale has joined #openstack-glance07:19
*** mosulica has quit IRC07:24
*** tshefi has joined #openstack-glance07:26
*** groen692 has joined #openstack-glance07:30
*** bkopilov has quit IRC07:55
*** hogepodge has quit IRC07:56
*** mosulica has joined #openstack-glance07:56
*** mosulica has quit IRC08:03
*** mosulica has joined #openstack-glance08:07
*** mfedosin has quit IRC08:31
*** bkopilov has joined #openstack-glance08:51
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-glance09:00
*** namnh has quit IRC09:20
*** mvk has quit IRC09:32
*** gb21 has quit IRC09:40
*** gb21 has joined #openstack-glance09:46
*** gb21 has quit IRC09:48
*** gb21 has joined #openstack-glance09:48
*** e0ne has joined #openstack-glance10:02
*** mvk has joined #openstack-glance10:10
*** udesale has quit IRC10:17
ccardI am trying to use python-glanceclient to load an image from a local file using the glance v2 client. With the v1 client I can successfully load an image from a 'location' url, what is the syntax for glance.images.create() to load an image from a local file?10:20
*** ezoszed has quit IRC10:35
*** thiagolib has joined #openstack-glance10:39
*** aarefiev has joined #openstack-glance10:47
*** aarefiev_afk has quit IRC10:50
*** ezoszed has joined #openstack-glance10:51
*** ezoszed has quit IRC11:00
*** smatzek has joined #openstack-glance11:18
*** smatzek has quit IRC11:21
*** smatzek has joined #openstack-glance11:21
*** bhagyashris has quit IRC11:22
*** nicolasbock has joined #openstack-glance11:35
*** dims has joined #openstack-glance11:39
*** ezoszed has joined #openstack-glance11:39
*** ratailor has quit IRC11:57
*** mvk has quit IRC12:03
*** ezoszed has quit IRC12:09
*** mvk has joined #openstack-glance12:15
*** ezoszed has joined #openstack-glance12:42
*** gb21 has quit IRC12:53
*** smatzek has quit IRC13:02
*** ezoszed has quit IRC13:05
*** chlong has joined #openstack-glance13:08
*** ezoszed has joined #openstack-glance13:08
*** smatzek has joined #openstack-glance13:12
*** ducttape_ has quit IRC13:14
*** ducttape_ has joined #openstack-glance13:17
*** mvk has quit IRC13:21
openstackgerritIan Cordasco proposed openstack/python-glanceclient: Handle formatting of subcommand name in error output  https://review.openstack.org/40831813:32
*** mvk has joined #openstack-glance13:33
*** ccesario has quit IRC13:50
*** david-lyle has quit IRC13:56
*** david-lyle has joined #openstack-glance13:59
sigmavirusccard: so image creation and image upload are two separate steps in V2, if I remember correctly14:00
*** links has quit IRC14:02
*** itisha has joined #openstack-glance14:11
*** ducttape_ has quit IRC14:14
*** jamielennox is now known as jamielennox|away14:14
*** david-lyle has quit IRC14:16
*** udesale has joined #openstack-glance14:22
*** mosulica has quit IRC14:35
*** chlong has quit IRC14:37
*** ducttape_ has joined #openstack-glance14:45
jokke_v2 definitely v1 as well iirc14:52
rosmaitaccard: you need to create the image first and then use upload()14:55
*** mosulica has joined #openstack-glance14:55
*** ezoszed has quit IRC14:56
*** ezoszed has joined #openstack-glance15:02
*** david-lyle has joined #openstack-glance15:17
*** TravT has joined #openstack-glance15:20
*** david-lyle has quit IRC15:21
*** ezoszed has quit IRC15:26
*** markvoelker has quit IRC15:28
*** markvoelker has joined #openstack-glance15:29
*** markvoelker has quit IRC15:35
*** udesale has quit IRC15:36
*** mvk has quit IRC15:48
rosmaitajokke_: need your opinion on whether https://review.openstack.org/#/c/418993/ is a good backport candidate15:50
*** wxy| has joined #openstack-glance15:50
*** mosulica has quit IRC15:56
*** markvoelker has joined #openstack-glance15:57
*** chlong has joined #openstack-glance16:11
*** ccesario has joined #openstack-glance16:13
jokke_looking16:16
*** bkopilov has quit IRC16:16
jokke_rosmaita: I'm not convinced. What do you think16:17
jokke_I'd challenge that being a bug.16:17
rosmaitajokke_: well, that fix is already in Ocata, so I'm ok with it being a bug16:20
rosmaitajust wondering whether it makes sense to backport16:20
rosmaitathough i guess maybe if the rolling upgrade stuff happens, the ocata script may not be compatible with newton ?16:21
jokke_rosmaita: I did not mean that. I noticed that it had been merged to master against the bug, which is fineish ... My point was I'm not convinced that something backportable16:21
rosmaitaso maybe we do need to have a newton version available16:21
rosmaitaok, sorry, i misunderstood16:22
rosmaitai'm thinking under normal circumstances, it would not be16:22
rosmaitabut i guess we need hemanthm and alex_bash to weigh in on whether the glance-manage script database operations in ocata will make the script not work in newton16:23
jokke_but that's just to change the behavior of feature that was already flagged as security risk and should not be used. True it's pretty well contained change, but why should we backport it?16:23
rosmaitawell, that's a good point16:24
rosmaitathough if we allow people to delete everything up to 1 day old, why not everything?16:24
rosmaitathe thing that convinced me that this was ok for ocata is when it's used in testing scenarios16:25
rosmaitaallows you to clean out the db easily16:25
jokke_true but that's how it was originally designed. It's not buggy behavior, it was bad design, which imo is questionable why we should allow backporting it and then we have already adviced to not use the feature in first place due to security issues, I just don't see any reason what so ever why it would be good idea to backport with our current backport policies16:26
*** bkopilov has joined #openstack-glance16:28
jokke_even the logic "Why not allow it" is valid it does not make it valid backport candidate16:32
rosmaitajokke_: thanks ... i asked for your opinion, and i got it!  That's why you are "Mr. Stability" for glance16:33
rosmaitai'll put a comment on cyril's patch16:33
jokke_;)16:34
*** tshefi has quit IRC16:48
jcookhemanthm so with the CacheFilter, upon request, if the image is cached or caching, the middleware returns the response with an iterator to the cached / caching image?16:52
jcookhemanthm where does the code end up if the middleware returns None as there is no cache / caching image?16:52
hemanthmjcook: yes to your first question16:54
hemanthmjcook: let me look into the code16:54
*** mfedosin has joined #openstack-glance16:55
*** groen692 has quit IRC16:55
jcookhemanthm righteous, thanks!16:56
*** groen692 has joined #openstack-glance16:57
* jcook doesn't fully grok the middleware or pipeline in glance16:57
jokke_jcook: anything specific about it?17:00
jokke_it's really quite simple system when you get grasp of it17:00
jcookjokke_ just in general, I haven't looked into it much. Like entry and exit points, how responses from middleware are processed. Is there a doc on it?17:04
*** e0ne has quit IRC17:06
ccardrosmaita: thanks, I did work that out by looking at the code for the glance cli. But trying that I'm getting an error "'dict' object has no attribute 'read'" when I call upload()17:07
hemanthmjcook: https://github.com/openstack/glance/blob/master/etc/glance-api-paste.ini#L22-L2317:08
jcookhemanthm I don't follow, maybe my question was poorly worded17:08
hemanthmif that's the pipeline we are using and there is no cache hit, the request will be handled by the rootapp, which is the glance API17:08
jcookhemanthm I'm curious where downloading occurs17:09
*** wxy| has quit IRC17:09
jokke_jcook: so each middleware just have a hook where it's called and when it returns the output will be sent to the hook of the next one in pipeline17:09
jcookjokke_ so a cache iterator is passed along to the end of the pipeline or some such, and then downloaded and returned with the response? If ti's not set, then the response is created by some other code?17:10
jokke_jcook: and they get called both ways so request comes in to first middleware, unless any middleware does not provide response it lands finally to the wsgi app. and returnes comes back via the middlewares in reverse order17:10
jcookinteresting17:10
*** briancurtin has quit IRC17:11
jcookhemanthm I'm really curious where the download is initiated if it's not already in progress17:11
jokke_but for example the healthcheck middleware returns response so we just reverse the order at that point instead of processing the request through the whole stack17:12
*** groen692 has quit IRC17:12
hemanthmjcook: if it's a cache miss, the rootapp will serve the request17:12
hemanthmhttps://github.com/openstack/glance/blob/master/etc/glance-api-paste.ini#L37-L5017:12
*** pcaruana has quit IRC17:12
hemanthmdepending upon whether it is v1 or v2, it will hit the respective router17:13
jcookoh so a response causes it to immediately about face?17:13
*** jose-phillips has joined #openstack-glance17:13
*** wxy has quit IRC17:14
jokke_jcook: I'm 95% sure it was like that when I dug in it at some point17:14
hemanthmjcook: yes17:14
sigmavirusrosmaita: forgive my terrible memory, for lite specs, is one core's approval enough? re Iac2f8ad018aed19219e3250cb516e660fb348dcd17:14
jcookhemanthm so glance.api.v2.image_data:download for example?17:14
hemanthmany of the middlewares in the pipeline can intercept the request and return the response without letting the request hit the rootapp17:15
hemanthmjcook: aye, eventually17:15
hemanthmbut the entry point into the rootapp will be the router17:16
*** tesseract has quit IRC17:20
rosmaitasigmavirus: needs regular 2 +2 ... the "lightness" comes from the size of the spec17:26
rosmaitasigmavirus: looking at it now17:26
rosmaitabut, typo corrections only require a single +217:27
sigmavirusThanks rosmaita17:30
rosmaitasigmavirus: lgtm17:37
sigmavirusNot my spec, but thanks17:38
sigmavirusThat'll help dharinic since this person is removing mox for us ;)17:38
dharinicsigmavirus :D That reminds me to share this with you guys: https://review.openstack.org/#/c/414361/17:40
dharinicNo failure ever since17:40
dharinicsigmavirus: lite-sepc to remove mox?17:40
dharinicspec*17:40
sigmavirusdharinic: i don't know how to explain it17:41
sigmavirusBut those tests were failing with regularity for a period of time there17:41
openstackgerritMerged openstack/glance-specs: Replace mox with mock  https://review.openstack.org/41104917:41
sigmavirusdharinic: that ^ is what we're talking about17:41
dharinicCool.17:41
dharinicThanks sigmavirus17:43
*** aarefiev is now known as aarefiev_afk17:54
*** mvk has joined #openstack-glance17:59
openstackgerritDirk Mueller proposed openstack/glance: Invoke monkey_patching early enough for eventlet 0.20.1  https://review.openstack.org/41907418:04
*** david-lyle has joined #openstack-glance18:15
*** david-lyle has quit IRC18:16
*** david-lyle has joined #openstack-glance18:16
*** antwash has joined #openstack-glance18:24
antwashstevelle : rosmaita : hemathm : sigmavirus :  Hey there working on adding multi node jobs for glance using grenade in order to test previous & newer version communication compatibility (rolling upgrades)  -- was wondering could I get some feedback about it.18:32
rosmaitaantwash: sure, very glad to hear you are working on it!18:32
rosmaitawhat kind of feedback do you need?18:32
*** dharinic is now known as dharinic|lunch18:36
*** adisky_ has quit IRC18:39
*** chlong has quit IRC18:46
*** jlwhite has joined #openstack-glance18:51
antwashrosmaita : Here's the review link I posted https://review.openstack.org/#/c/414326/3 -- just trying to make sure I'm going about testing it right really.18:53
*** thiagolib has quit IRC18:58
*** chlong has joined #openstack-glance18:59
*** mtanino has joined #openstack-glance19:02
rosmaitaantwash: ack -- will look19:02
*** briancurtin has joined #openstack-glance19:09
*** ccesario has quit IRC19:12
*** wxy has joined #openstack-glance19:12
*** tshefi has joined #openstack-glance19:23
*** dharinic|lunch is now known as dharinic19:26
openstackgerritDharini Chandrasekar proposed openstack/glance_store: Buffered reader: Upload recovery for swift store  https://review.openstack.org/12086619:50
*** tshefi has quit IRC19:52
*** flwang1 has joined #openstack-glance20:13
*** e0ne has joined #openstack-glance20:14
*** e0ne has quit IRC20:43
*** TravT has quit IRC20:53
*** jamielennox|away is now known as jamielennox21:01
*** smatzek has quit IRC21:13
*** mfedosin has quit IRC21:25
*** markvoelker has quit IRC22:29
*** markvoelker has joined #openstack-glance22:32
*** markvoelker has quit IRC22:34
*** markvoelker has joined #openstack-glance22:34
*** markvoelker_ has joined #openstack-glance22:39
*** markvoelker has quit IRC22:39
*** kberger has joined #openstack-glance22:41
*** catintheroof has quit IRC22:50
*** catintheroof has joined #openstack-glance22:51
*** chlong has quit IRC22:55
*** catintheroof has quit IRC22:55
dharinicSwift multi-tenant backport proposals:23:14
dharinichttps://review.openstack.org/#/c/417329/23:14
dharinichttps://review.openstack.org/#/c/417197/23:14
*** markvoelker_ has quit IRC23:32
*** markvoelker has joined #openstack-glance23:40

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