Tuesday, 2015-12-08

*** avarner has quit IRC00:00
*** jerrygb has quit IRC00:00
*** jerrygb has joined #openstack-glance00:02
*** sgotliv_ has quit IRC00:02
*** jamielennox|away is now known as jamielennox00:05
*** takedakn has quit IRC00:07
*** takedakn has joined #openstack-glance00:07
*** jerrygb has quit IRC00:18
*** jerrygb has joined #openstack-glance00:18
*** ianbrown has quit IRC00:20
*** ianbrown has joined #openstack-glance00:20
*** smatzek has quit IRC00:20
*** jerrygb has quit IRC00:23
*** ianbrown has quit IRC00:25
*** ianbrown_ has joined #openstack-glance00:25
*** mtanino has quit IRC00:33
*** ducttape_ has quit IRC00:40
*** ducttape_ has joined #openstack-glance00:40
*** takedakn has quit IRC00:43
*** alejandrito has quit IRC00:46
*** RichardRaseley has quit IRC00:56
*** changbl has quit IRC01:00
*** changbl has joined #openstack-glance01:13
*** changbl has quit IRC01:14
*** ianbrown_ has quit IRC01:20
*** nikhil_k has joined #openstack-glance01:23
*** nikhil has quit IRC01:25
*** ducttape_ has quit IRC01:25
*** links has joined #openstack-glance01:27
*** ducttape_ has joined #openstack-glance01:33
*** changbl has joined #openstack-glance01:34
*** wxy has quit IRC01:37
*** wxy has joined #openstack-glance01:38
*** yasu has joined #openstack-glance01:39
*** ducttape_ has quit IRC01:42
*** r-daneel has quit IRC01:43
*** changbl has quit IRC01:47
*** ianbrown_ has joined #openstack-glance01:48
*** 7F1AA32U1 has joined #openstack-glance01:58
*** changbl has joined #openstack-glance02:00
*** 7F1AA32U1 has quit IRC02:01
*** haomaiwang has joined #openstack-glance02:01
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/25446902:05
*** jerrygb has joined #openstack-glance02:14
*** jerrygb has quit IRC02:16
*** itisha has quit IRC02:26
*** ayoung has quit IRC02:29
*** ducttape_ has joined #openstack-glance02:41
*** houming has joined #openstack-glance02:42
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/25446902:50
*** ianbrown_ has quit IRC02:57
*** ducttape_ has quit IRC03:00
*** haomaiwang has quit IRC03:01
*** azbiswas has quit IRC03:01
*** haomaiwang has joined #openstack-glance03:01
*** azbiswas has joined #openstack-glance03:01
openstackgerritwangxiyuan proposed openstack/glance: Update the cache middleware flavor guideline.  https://review.openstack.org/25451703:02
*** azbiswas has quit IRC03:06
*** edmondsw has quit IRC03:07
*** haomaiwang has quit IRC03:15
*** haomaiwang has joined #openstack-glance03:15
*** haomaiwang has quit IRC03:16
*** haomaiwang has joined #openstack-glance03:16
*** haomaiwang has quit IRC03:17
*** ayoung has joined #openstack-glance03:18
*** ayoung has quit IRC03:18
*** haomaiwang has joined #openstack-glance03:18
*** haomaiwang has quit IRC03:19
*** 17WAA5E9R has joined #openstack-glance03:19
*** 17WAA5E9R has quit IRC03:20
*** haomaiwang has joined #openstack-glance03:20
*** haomaiwang has quit IRC03:21
*** haomaiwang has joined #openstack-glance03:24
*** haomaiwang has quit IRC03:25
*** julim has quit IRC03:25
*** haomaiwang has joined #openstack-glance03:25
*** haomaiwang has quit IRC03:26
*** haomaiwang has joined #openstack-glance03:26
*** haomaiwang has quit IRC03:27
*** haomaiwang has joined #openstack-glance03:27
*** haomaiwang has quit IRC03:28
*** ducttape_ has joined #openstack-glance03:28
*** gberginc has joined #openstack-glance03:28
*** haomaiwang has joined #openstack-glance03:28
*** haomaiwang has quit IRC03:29
*** haomaiwang has joined #openstack-glance03:29
*** haomaiwang has quit IRC03:30
*** 7F1AA34GJ has joined #openstack-glance03:30
*** 7F1AA34GJ has quit IRC03:31
*** haomaiwang has joined #openstack-glance03:31
*** haomaiwang has quit IRC03:32
*** haomaiwa_ has joined #openstack-glance03:33
*** haomaiwa_ has quit IRC03:34
*** haomaiwang has joined #openstack-glance03:34
*** haomaiwang has quit IRC03:35
*** jerrygb has joined #openstack-glance03:35
*** haomaiwa_ has joined #openstack-glance03:38
*** ducttape_ has quit IRC03:54
*** ducttape_ has joined #openstack-glance03:54
*** ianbrown has joined #openstack-glance03:55
*** haomaiwa_ has quit IRC04:01
*** haomaiwang has joined #openstack-glance04:01
*** azbiswas has joined #openstack-glance04:04
*** darrenc is now known as darrenc_afk04:06
*** azbiswas has quit IRC04:08
*** ducttape_ has quit IRC04:17
*** ianbrown has quit IRC04:35
*** darrenc_afk is now known as darrenc04:42
*** boris-42_ has quit IRC04:43
*** azbiswas has joined #openstack-glance04:51
*** abhishekk has joined #openstack-glance05:00
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-glance05:01
*** ianbrown has joined #openstack-glance05:02
*** gberginc has quit IRC05:22
*** GB21 has joined #openstack-glance05:30
*** jerrygb has quit IRC05:50
*** ianbrown has quit IRC05:57
*** ianbrown has joined #openstack-glance05:57
*** ianbrown_ has joined #openstack-glance05:57
*** ianbrown_ has quit IRC05:57
*** dims has quit IRC05:57
*** ianbrown_ has joined #openstack-glance06:00
*** ianbrown has quit IRC06:00
*** haomaiwang has quit IRC06:01
*** haomaiwang has joined #openstack-glance06:01
*** wolsen has quit IRC06:04
*** wolsen has joined #openstack-glance06:05
*** kebray has quit IRC06:08
*** gberginc has joined #openstack-glance06:13
*** ianbrown_ has quit IRC06:17
*** dims has joined #openstack-glance06:20
*** sgotliv_ has joined #openstack-glance06:22
*** ianbrown has joined #openstack-glance06:23
*** haomaiwang has quit IRC06:32
*** 18WABB396 has joined #openstack-glance06:33
openstackgerritwangxiyuan proposed openstack/glance: Update the cache middleware flavor guideline.  https://review.openstack.org/25451706:34
*** e0ne has joined #openstack-glance06:38
*** azbiswas has quit IRC06:38
*** ianbrown has quit IRC06:40
*** ianbrown has joined #openstack-glance06:40
*** ianbrown has quit IRC06:41
*** ianbrown_ has joined #openstack-glance06:41
*** ianbrown__ has joined #openstack-glance06:43
*** ianbrown_ has quit IRC06:43
*** e0ne has quit IRC06:45
*** e0ne has joined #openstack-glance06:49
*** e0ne has quit IRC06:52
*** groen692 has joined #openstack-glance06:52
openstackgerritJinxing Fang proposed openstack/glance: Remove the qpid message driver from config file  https://review.openstack.org/25458006:53
*** azbiswas has joined #openstack-glance06:54
*** 18WABB396 has quit IRC07:01
*** haomaiwang has joined #openstack-glance07:01
*** e0ne has joined #openstack-glance07:02
*** houming has quit IRC07:08
*** haomaiwang has quit IRC07:09
*** haomaiwang has joined #openstack-glance07:09
*** dims has quit IRC07:11
*** houming has joined #openstack-glance07:11
*** houming has quit IRC07:16
*** e0ne has quit IRC07:20
*** e0ne has joined #openstack-glance07:21
*** haomaiwang has quit IRC07:28
*** haomaiwa_ has joined #openstack-glance07:28
*** dims has joined #openstack-glance07:31
*** houming has joined #openstack-glance07:42
*** dims has quit IRC07:52
*** GB21 has quit IRC07:52
*** e0ne has quit IRC07:56
*** jerrygb has joined #openstack-glance08:06
*** azbiswas has quit IRC08:07
*** azbiswas has joined #openstack-glance08:08
*** jerrygb has quit IRC08:12
*** GB21 has joined #openstack-glance08:12
*** azbiswas has quit IRC08:13
*** MattMan has quit IRC08:26
*** MattMan has joined #openstack-glance08:32
*** markus_z has joined #openstack-glance08:37
*** groen692 has quit IRC08:47
*** groen692 has joined #openstack-glance08:48
*** links has quit IRC08:55
*** e0ne has joined #openstack-glance09:03
*** links has joined #openstack-glance09:09
*** bunting has quit IRC09:11
*** bunting has joined #openstack-glance09:13
*** shakamunyi has joined #openstack-glance09:14
*** shakamunyi has quit IRC09:23
*** gberginc has quit IRC09:48
*** houming has quit IRC09:59
*** aix has joined #openstack-glance10:01
*** jistr has joined #openstack-glance10:14
*** aix has quit IRC10:21
*** gberginc has joined #openstack-glance10:26
openstackgerritStuart McLaren proposed openstack/python-glanceclient: Handle 403 forbidden on download  https://review.openstack.org/25432810:31
*** aix has joined #openstack-glance10:33
*** links has quit IRC10:37
*** links has joined #openstack-glance10:58
*** openstackgerrit has quit IRC11:02
*** openstackgerrit has joined #openstack-glance11:02
*** haomaiwa_ has quit IRC11:06
*** Kennan has quit IRC11:14
*** Kennan has joined #openstack-glance11:17
*** smatzek has joined #openstack-glance11:28
*** Kennan has quit IRC11:29
flaper87sabari: I'm working on creating tasks for that spec. We're good as long as you've time11:31
*** Kennan has joined #openstack-glance11:32
*** azbiswas has joined #openstack-glance11:32
*** azbiswas has quit IRC11:37
*** Kennan has quit IRC11:39
*** Kennan has joined #openstack-glance11:42
*** Kennan has quit IRC11:43
*** abhishekk has quit IRC11:44
*** Kennan has joined #openstack-glance11:46
*** Kennan has quit IRC11:47
*** yasu has quit IRC11:47
*** Kennan has joined #openstack-glance11:50
openstackgerritAlexander Tivelkov proposed openstack/glance-specs: Glare Public (plugin-independent) API  https://review.openstack.org/25471011:53
*** houming has joined #openstack-glance11:59
openstackgerritAlexander Tivelkov proposed openstack/glance-specs: Deprecate v3 API  https://review.openstack.org/25416312:00
*** dshakhray has joined #openstack-glance12:00
*** e0ne has quit IRC12:03
*** jerrygb has joined #openstack-glance12:09
*** e0ne has joined #openstack-glance12:11
*** jerrygb has quit IRC12:13
*** Kennan has quit IRC12:20
*** GB21 has quit IRC12:21
*** Kennan has joined #openstack-glance12:27
*** links has quit IRC12:36
*** Kennan has quit IRC12:48
*** Kennan has joined #openstack-glance12:54
*** ducttape_ has joined #openstack-glance12:58
*** ducttape_ has quit IRC13:02
*** ducttape_ has joined #openstack-glance13:06
*** ducttape_ has quit IRC13:16
*** dims has joined #openstack-glance13:16
flaper87kairat: btw, you abandoned yourfix for: https://bugs.launchpad.net/glance/+bug/151940213:18
openstackLaunchpad bug 1519402 in python-glanceclient "Add docker container format to defaults" [Wishlist,In progress] - Assigned to Atsushi SAKAI (sakaia)13:18
flaper87do you have the other link handy?13:18
flaper87I mean, the link to the patch that superseded yours ?13:19
kairatflaper87, https://review.openstack.org/#/c/252806/13:19
kairatflaper87, I proposed to update the patch with the reference to the bug but it seems that the progress is not so fast as expected =)13:20
flaper87kairat: FWIW, feel free to take over and add yourself as a co-author. If you do a commit, the original author will be preserved13:22
*** ianbrown__ has quit IRC13:22
*** ianbrown_ has joined #openstack-glance13:22
kairatflaper87, ok, will do this if there will be no updates today13:22
*** edmondsw has joined #openstack-glance13:31
*** itisha has joined #openstack-glance13:33
*** mingdang1 has joined #openstack-glance13:34
*** jerrygb has joined #openstack-glance13:44
*** julim has joined #openstack-glance13:55
*** nikhil_k is now known as nikhil14:05
*** ducttape_ has joined #openstack-glance14:08
openstackgerritDina Belova proposed openstack/glance: Do not use api-paste.ini osprofiler options  https://review.openstack.org/24506014:09
*** changbl has quit IRC14:10
*** ducttape_ has quit IRC14:15
*** gberginc has quit IRC14:16
kairatnikhil, see comments here https://review.openstack.org/#/c/246943/ to understand why we can't fail with authrization error in swiftclient14:29
kairatnikhil, unfortunately we can lose data here14:29
*** mclaren_ has joined #openstack-glance14:30
kairatI will refine spec description to explain why we can't use that approach14:30
rosmaitamclaren_: hello14:30
flaper87-1 on starting on theswift case14:30
mclaren_ok, why?14:30
flaper87I'm working on a series of tasks for us to start working on this spec14:30
flaper87but the first scenario we should cover is direct upload14:30
mclaren_why?14:31
flaper87that's what's mostly broken now and that's what's part of the compute group in defcore14:31
mclaren_right, but we're more ready to start on the swift one14:31
rosmaitamclaren_: what flaper87 said, that's the feedback from the rest of the community14:31
flaper87(sorry, as I mentioned I've some lag, my messages go out a bit late)14:31
nikhilkairat: ack, will look. I a curious about overlap with https://review.openstack.org/#/c/12086614:31
flaper87mclaren_: I'm sorry but no. whatever time we will put on coding hte swift part I'd rather put it in figuring out the solution ofr the direct upload one14:32
rosmaitamclaren_: i think the key thing is that we don't do anything that fucks up the swift-local workflow14:32
mclaren_ok, what can we make a start on before finalizing the bikeshed side of things?14:32
mclaren_gotta go, back in ~15mins14:32
flaper87mclaren_: as I said, I'm breaking the spec down into tasks. Started yesterday and I'll finish today14:32
nikhil-1 on starting now14:32
kairatnikhil, i reviewed that, there will be no overlap, will explain this in spec also.14:33
rosmaitamclaren_: ping when you get back14:33
flaper87that should give us several places to start on that don't require bikeshed to be ready14:33
flaper87mclaren_: ^14:33
nikhilthere's been so much time spent discussing this, starting w/o full idea is going to make things worse14:33
nikhilkairat: thanks!14:33
flaper87nikhil: there are things we can start with. There are required policy changes, api version bumps and other cosmetic things that will be needed anyway14:34
rosmaitaflaper87: mclaren_: my feeling is that we start with no end-user exposure of bikeshed, and then if it turns out that it woudl be really useful, we can rethink it14:34
flaper87thaose are the things I'm mentioning that we could get done now14:34
flaper87nothing big that requires discussion14:34
flaper87rosmaita: ++ exactly my thoughts now14:34
flaper87it'd be easier to add the bikeshed resource later than adding it now and having to maintain it forever14:35
*** mingdang1 has quit IRC14:35
flaper87w/ the discoverability in place, it shouldn't be hard to add it later14:35
rosmaitai think all the end-user wants to know is (1) glance has my data, and (2) glance gives me a checksum so i know my data wasn't messed up in transit14:36
rosmaitaother than that, i've handed my data over to glance to do its magic and turn it into image data14:37
rosmaitaso i shouldn't care what happens after that14:37
nikhilrosmaita: I think (1) is probably more like "glance has my data in the format that works for the operator"14:37
nikhiland (2) would change accordingly14:37
rosmaitanikhil: well, for (2) i'm thinking of the checksum of what glance received,  before glance manipulates it as part of import process14:38
*** smatzek has quit IRC14:38
nikhilrosmaita: I see, so we need to add that separately somewhere?14:38
rosmaitanikhil: i think so, maybe as part of the Image "message" element14:39
nikhilgotcha14:39
rosmaitaso for your use case, user (1) creates Image object, (2) does a PUT of an OVA, (3) does GET on Image to see status indicating "ready to import" and the checksum of what is in the bikeshed14:40
*** ayoung has joined #openstack-glance14:40
nikhilmakes sense14:41
rosmaitathen user (4) does an import call, and (5) does GET on Image until status==active, and at that point the 'checksum' in the Image is the checksum of what's actually stored in glance, i.e., the image in the OVA14:41
mclaren_ping14:41
rosmaitahi stuart14:41
flaper87kinda pong14:41
flaper87:D14:41
mclaren_sorry, scrum meeting14:41
mclaren_back now14:42
flaper87Ill have to leave in 10/15 mins14:43
nikhiland I need to be back in 15, sorry14:43
mclaren_I'm not sure I've much to add at this point, will think about the 'invisible' bikeshed :-)14:43
rosmaitamclaren_: i think we can always make it visibile if that turns out to be the right thing14:44
flaper87right14:44
rosmaitai'd like to try seeing how far we get without making it visible14:44
flaper87invisible -> visible is easier than the other way around14:44
rosmaitamclaren_: i think we can keep separation of concerns "behind the scenes"14:45
rosmaitahopefully14:45
mclaren_well if you've a new state, not really14:45
mclaren_and if v1 delete deletes the bikeshed, not really either14:45
mclaren_you're modifying your existing implementation14:46
*** edmondsw has quit IRC14:48
rosmaitamclaren_: ok, you are right14:48
rosmaitaso now i will say separation of concerns is overrated14:48
rosmaitasomething like "too much consistency"14:48
rosmaita:)14:48
rosmaitamclaren_: keep pushing, i really don't want to do something stupid here (as i have apparently done already with tasks)14:49
mclaren_I really don't like being pushy!14:50
rosmaitamclaren_: ok, you can nudge, then14:51
mclaren_and I don't think I have it all figured out either btw14:51
*** dims_ has joined #openstack-glance14:52
rosmaitamclaren_: ok, your v1-delete point is well taken ... but how does making the bikeshed visible help that?14:52
mclaren_But when, for example, I see what AWS have done -- they didn't hide the checksum. That feels a bit weird. I uploaded that thing yesterday. Did the checksum verification work? I can't remember.14:52
mclaren_how does making the bikeshed visible help that? By not having it fused with an image. A la the S3 implementation.14:53
mclaren_I guess my gut feeling here is this14:53
mclaren_we have some code that's a bit like a wobbly Jenga tower14:53
rosmaitaso if we took this to an extreme, you could set up 2 glances -- one for bootable images, and one for bikeshedding14:54
rosmaita(sorry, didn't mean to interrupt)14:54
mclaren_we have the option of either A) swapping some blocks in that tower around and then building more stuff on top of that or B) not touching it and building a separate tower14:55
*** dims has quit IRC14:55
rosmaitanote to self: want to ask sigmavirus24_awa if he knows what a yak pen is called, we could use that term instead of 'bikeshed'14:55
rosmaitamclaren_: i guess i'm not convinced that we can really do B14:55
rosmaitathat's basically what the import task did14:56
rosmaitaand it has been enthusiastically rejected14:56
mclaren_well the import task only worked with external data14:56
mclaren_so it was a task, not data14:56
mclaren_I don't see it as the same at all14:56
mclaren_but I do get what you're saying, the community doesn't necessarily listen to rational arguments14:58
*** ducttape_ has joined #openstack-glance14:59
mclaren_I can try to ping Jay maybe14:59
mclaren_see what he thinks15:00
rosmaitai would be interested15:00
rosmaitaparticularly aobut separation of concerns15:00
mclaren_ok, let me try to reach out to him.15:01
rosmaitacool15:01
mclaren_I'm happy that we've decided on strategy and have moved on to tactics15:01
rosmaitai guess so15:02
rosmaitai don't know what i think anymore!15:02
mclaren_yeah it's been a slog, we'll get there though15:02
*** ducttape_ has quit IRC15:03
*** ducttape_ has joined #openstack-glance15:03
sigmavirus24_awarosmaita: lol15:05
sigmavirus24_awaI've never used a yak pen before15:05
rosmaitait was your "shaving yaks" comment that got me thinking15:05
*** sigmavirus24_awa is now known as sigmavirus2415:06
rosmaitamaybe this will have something: http://www.fao.org/docrep/006/ad347e/ad347e0j.htm15:07
rosmaitaapparently yak pens are sometimes made of feces ... is that appropriate to our use case, or what?15:08
*** azbiswas has joined #openstack-glance15:08
*** TravT has quit IRC15:09
*** jerrygb has quit IRC15:09
*** jerrygb has joined #openstack-glance15:10
jokke_I would really really like to just stamp the bikeshed as bikeshed ... lets just make it official ... "Dump your data to the bikeshed, let us know and we'll take it from there"15:11
*** edmondsw has joined #openstack-glance15:13
*** smatzek has joined #openstack-glance15:14
*** azbiswas has quit IRC15:14
*** jerrygb has quit IRC15:15
sigmavirus24jokke_: cloud providers determine the color of the bikeshed?15:20
sigmavirus24a.k.a., what the image is in the end15:20
sigmavirus24Hey, I have an idea15:20
sigmavirus24What if we make a new standard that handles all the other image formats and adds new goodies on top to unify everything?15:21
* sigmavirus24 hides in a corner15:21
*** _ducttape_ has joined #openstack-glance15:21
*** vijendar has joined #openstack-glance15:22
flaper87rosmaita: commented w/ a list of tasks15:22
rosmaitaflaper87: thanks, i will get to work15:23
* sigmavirus24 cracks whip over rosmaita's back15:23
rosmaitasigmavirus24: glad to have you back! it wasn't the same without you15:24
*** jerrygb has joined #openstack-glance15:24
sigmavirus24Whoops. I keep confusing myself for jcook15:24
sigmavirus24sorry rosmaita15:24
* sigmavirus24 puts whip away15:24
*** ducttape_ has quit IRC15:24
*** mtanino has joined #openstack-glance15:33
*** erhudy has joined #openstack-glance15:36
*** jistr is now known as jistr|mtg15:43
*** gokrokve has joined #openstack-glance15:59
jcook?16:02
jcookwhat about me :)16:02
sigmavirus24Oh never you mind :P16:03
*** avarner has joined #openstack-glance16:03
jcook:)16:06
flaper87nikhil: sanity check, pls, https://bugs.launchpad.net/glance/+bug/151843116:07
openstackLaunchpad bug 1518431 in Glance "Glance failed to upload image to swift storage" [Undecided,Confirmed]16:07
flaper87ops16:07
flaper87nikhil: I meant, https://review.openstack.org/#/c/250774/16:07
*** aix has quit IRC16:08
*** smatzek has quit IRC16:09
*** ninag has joined #openstack-glance16:09
*** ninag has quit IRC16:09
sigmavirus24flaper87: that bug is interesting16:11
flaper87sigmavirus24: it is, indeed. I think we finally found the root cause16:13
flaper87let's see what fix cyril comes up with16:13
flaper87:D16:13
sigmavirus24wooo for "compatible" APIs, amiright?16:13
kairatI noticed that swiftclient seems not send content-length or transfer-encoding to rados16:14
kairatIt seems that apache FastCGI blocks the request because it is not compatible with HTTP specification16:14
kairatWe ar sending our request here: https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/swift/store.py#L55016:15
*** changbl has joined #openstack-glance16:15
kairatThere is no content-length specified: content_length = None16:15
kairatall that we have are container, name and reader16:16
*** ianbrown_ has quit IRC16:17
*** spzala has quit IRC16:18
kairatAfter that swiftclient prepares a request here: https://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L111716:18
kairat*prepares data16:18
*** pumaranikar has joined #openstack-glance16:22
kairatHmm, it looks like swiftclient do not specify transfer encoding nor content-length if image size is not defined16:23
kairatI am wondering who need to specify this then16:24
*** dshakhray has quit IRC16:24
*** changbl has quit IRC16:25
*** changbl has joined #openstack-glance16:26
openstackgerritMerged openstack/glance-specs: Revert "Cap Pillow to fix doc gates"  https://review.openstack.org/23925916:27
*** ianbrown_ has joined #openstack-glance16:29
*** changbl has quit IRC16:30
*** smatzek has joined #openstack-glance16:32
openstackgerritMerged openstack/glance-specs: Fix reviewers list render (db purge)  https://review.openstack.org/25077416:33
kairatmclaren_, I know you also participated in swift development16:37
kairatmclaren_, it looks like that swiftclient do not specify transfer encoding nor content-length if image size is not defined16:38
*** gberginc has joined #openstack-glance16:40
*** groen692 has quit IRC16:42
*** changbl has joined #openstack-glance16:43
*** boris-42_ has joined #openstack-glance16:44
sigmavirus24kairat: link to the code? This might be handled by requests for us16:44
*** openstackgerrit has quit IRC16:47
*** openstackgerrit has joined #openstack-glance16:47
kairatsigmavirus24, https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/swift/store.py#L55016:47
kairatthat is the request to swift if we don't know content-length16:47
sigmavirus24So swiftclient is handling this case then16:48
kairatimage_size is 0 so content-length is None16:48
kairatyep, but if you look at the paste in the bug16:48
kairathttps://bugs.launchpad.net/glance/+bug/151843116:49
openstackLaunchpad bug 1518431 in Glance "Glance failed to upload image to swift storage" [Undecided,Confirmed]16:49
kairatit seems that not length or transfer encoding specified16:49
kairatSo apparently apache do not allow that16:49
kairatI raised this question in openstack-swift16:49
nikhilflaper87: ack16:50
sigmavirus24kairat: in https://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L1547 content_length is passed on16:51
kairatsigmavirus24, we are passing None AFAIK16:51
sigmavirus24kairat: which is the default for that method16:52
kairatyep16:52
kairatbut swift do not handle this correctly then16:52
sigmavirus24kairat: while you're in openstack-swift, ask them how they approved https://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L1096..L110616:52
kairatheh16:52
sigmavirus24retry calls https://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L103416:53
sigmavirus24kairat: I'd guess this is a bug in those lines actually that I linked16:54
kairatWill do as soon as receive an answer about the bug)16:54
sigmavirus24Because the put_object method calls that put_object function16:54
sigmavirus24So, they appear to always be assigning an `int` to the header value which afaik neither urllib3 or requests like16:55
sigmavirus24oh gosh16:55
sigmavirus24nevermind16:55
sigmavirus24I'm confusing content-length and content-type16:55
kairatsigmavirus24, I don't think so16:55
kairatheaders is None16:55
*** ianbrown__ has joined #openstack-glance16:55
*** alejandrito has joined #openstack-glance16:55
kairatso finally we should be here: https://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L111716:56
sigmavirus24So here is what happens https://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L111716:56
sigmavirus24yeah16:56
sigmavirus24Right16:56
kairatAnd I didn't find any code where they specifies transfer-encoding16:56
kairatSo it looks like violation of HTTP spec16:56
sigmavirus24so if that gets passed to requests requests should emit transfer-encoding: chunked16:56
kairatPerhaps16:57
sigmavirus24kairat: did you do a packet trace to show that it's not actually being set by swiftclient?16:57
sigmavirus24The ReadableToIterable is a class that makes requests think its handling a generator16:57
*** ianbrown_ has quit IRC16:57
sigmavirus24Because of that requests handles it as a chunked transfer-encoding16:57
sigmavirus24(Trust me, I know this very well)16:57
sigmavirus24kairat: if you're able to reproduce this, please send me a packet trace showing that we're not setting the t-e header16:58
kairatnope, unfortunately, need some time to configure ceph back-end:(16:58
kairatsigmavirus24, all that I have is this: http://paste.openstack.org/show/479621/16:58
kairatIIUC, there is no transfer-encoding generated in request16:59
kairatalthough, requests can add this header after logging17:00
kairatso you can be right then17:00
sigmavirus24requests has no logging of its own and urllib3 has logging but doesn't log headers because of potentially sensitive information17:00
openstackgerritCyril Roelandt proposed openstack/glance_store: Fix a race condition in the Swift store  https://review.openstack.org/25487317:00
openstackgerritCyril Roelandt proposed openstack/glance_store: Fix a race condition in the Swift store  https://review.openstack.org/25487317:03
*** alex_xu has quit IRC17:04
kairatso yes17:04
sigmavirus24kairat: ?17:05
kairatneed to trace requests to ensure if this is the problem with transfer-encoding: chunked17:05
kairatI asked guys about that17:05
kairatunfortunately i do not have ceph installed17:05
kairatwaiting for answer17:05
*** kebray has joined #openstack-glance17:05
*** azbiswas has joined #openstack-glance17:05
*** Kennan has quit IRC17:06
*** kebray has quit IRC17:06
*** houming has quit IRC17:07
*** kebray has joined #openstack-glance17:07
*** Kennan has joined #openstack-glance17:07
*** ayoung has quit IRC17:10
openstackgerritCyril Roelandt proposed openstack/glance_store: Fix a race condition in the Swift store  https://review.openstack.org/25487317:10
*** alex_xu has joined #openstack-glance17:10
*** azbiswas has quit IRC17:11
*** RichardRaseley has joined #openstack-glance17:17
*** dshakhray has joined #openstack-glance17:19
*** jistr|mtg is now known as jistr17:21
*** ayoung has joined #openstack-glance17:25
*** ayoung has quit IRC17:31
openstackgerritNiall Bunting proposed openstack/python-glanceclient: Disable suggestion of v1 help for v2 commands  https://review.openstack.org/25488317:32
*** smatzek has quit IRC17:39
*** TravT has joined #openstack-glance17:43
*** ayoung has joined #openstack-glance17:44
*** ctina_ has joined #openstack-glance17:46
*** TravT has quit IRC17:48
*** TravT has joined #openstack-glance17:48
*** e0ne has quit IRC17:50
*** ianbrown__ has quit IRC17:52
openstackgerritMerged openstack/glance: Fix default value with postgreSQL  https://review.openstack.org/20885117:53
*** ianbrown__ has joined #openstack-glance17:56
*** markus_z has quit IRC17:57
*** kebray has quit IRC18:00
*** smatzek has joined #openstack-glance18:02
*** kebray has joined #openstack-glance18:03
*** TravT has quit IRC18:03
*** TravT has joined #openstack-glance18:03
*** jistr has quit IRC18:05
*** alejandrito has quit IRC18:19
*** alejandrito has joined #openstack-glance18:19
*** vijendar has left #openstack-glance18:21
*** kebray has quit IRC18:22
*** gokrokve has quit IRC18:24
*** gokrokve has joined #openstack-glance18:25
*** shakamunyi has joined #openstack-glance18:25
*** pumaranikar has quit IRC18:29
*** gokrokve has quit IRC18:30
*** dims_ has quit IRC18:30
*** gokrokve has joined #openstack-glance18:31
*** azbiswas has joined #openstack-glance18:32
*** gokrokve has quit IRC18:33
*** gokrokve has joined #openstack-glance18:34
*** jerrygb has quit IRC18:39
*** e0ne has joined #openstack-glance18:39
*** pumaranikar has joined #openstack-glance18:43
*** Kennan has quit IRC18:43
*** Kennan has joined #openstack-glance18:44
*** etoews has joined #openstack-glance18:44
*** etoews has left #openstack-glance18:45
*** ctina_ has quit IRC18:46
*** jerrygb has joined #openstack-glance18:46
*** david-ly_ has joined #openstack-glance18:50
*** david-ly_ is now known as david-lyle_18:52
*** david-lyle has quit IRC18:54
*** sgotliv_ has quit IRC18:55
*** openstackgerrit has quit IRC19:02
*** openstackgerrit has joined #openstack-glance19:02
*** TravT has quit IRC19:05
*** avarner has quit IRC19:07
*** avarner has joined #openstack-glance19:07
*** gokrokve has quit IRC19:08
*** kebray has joined #openstack-glance19:11
*** david-lyle_ is now known as david-lyle19:14
*** gokrokve has joined #openstack-glance19:16
*** gokrokve has quit IRC19:31
*** sgotliv_ has joined #openstack-glance19:34
*** kebray has quit IRC19:38
*** harlowja has quit IRC19:38
*** harlowja has joined #openstack-glance19:38
*** erhudy has quit IRC19:40
*** TravT has joined #openstack-glance19:54
openstackgerritBertrand Lallau proposed openstack/glance: Remove iso8601 dependency  https://review.openstack.org/24656519:58
*** kebray has joined #openstack-glance20:06
*** kebray_ has joined #openstack-glance20:07
*** kebray has quit IRC20:07
*** kebray_ has quit IRC20:18
*** vijendar has joined #openstack-glance20:23
*** kebray has joined #openstack-glance20:38
*** avarner has quit IRC20:53
*** kebray has quit IRC21:01
*** ianbrown__ has quit IRC21:19
*** ianbrown has joined #openstack-glance21:23
*** cdelatte has quit IRC21:23
*** smatzek has quit IRC21:23
*** TravT has quit IRC21:25
*** ianbrown has quit IRC21:27
*** ianbrown has joined #openstack-glance21:28
*** dims has joined #openstack-glance21:31
*** ayoung has quit IRC21:31
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/25446921:31
*** mingdang1 has joined #openstack-glance21:33
*** ianbrown_ has joined #openstack-glance21:33
*** ianbrown has quit IRC21:34
*** ianbrown has joined #openstack-glance21:41
*** cdelatte has joined #openstack-glance21:49
openstackgerritMerged openstack/glance_store: Updated from global requirements  https://review.openstack.org/25408721:49
*** azbiswas has quit IRC21:51
*** gonzalo2kx has joined #openstack-glance21:58
*** dims has quit IRC21:59
*** sgotliv_ has quit IRC22:00
*** gonzalo2kx has quit IRC22:03
*** gonzalo2kx has joined #openstack-glance22:05
*** gonzalo2kx has quit IRC22:05
*** gokrokve has joined #openstack-glance22:05
*** gokrokve has quit IRC22:07
*** gokrokve has joined #openstack-glance22:07
*** dshakhray has quit IRC22:08
*** david-ly_ has joined #openstack-glance22:10
*** david-lyle has quit IRC22:13
*** david-ly_ is now known as david-lyle22:13
*** belmoreira has joined #openstack-glance22:19
*** julim has quit IRC22:21
*** e0ne has quit IRC22:26
*** kebray has joined #openstack-glance22:26
*** changbl has quit IRC22:31
*** kebray has quit IRC22:31
*** mingdang1 has quit IRC22:33
openstackgerritBrianna Poulos proposed openstack/glance: Add sign-the-hash deprecation warning  https://review.openstack.org/25498722:34
*** pumaranikar has quit IRC22:35
*** _ducttape_ has quit IRC22:38
*** gokrokve has quit IRC22:39
*** ayoung has joined #openstack-glance22:43
*** gokrokve has joined #openstack-glance22:49
*** gokrokve has quit IRC22:50
*** gokrokve has joined #openstack-glance22:53
*** ianbrown_ has quit IRC22:55
*** gokrokve_ has joined #openstack-glance22:55
*** gokrokve has quit IRC22:56
*** sigmavirus24 is now known as sigmavirus24_awa22:56
*** ianbrown_ has joined #openstack-glance22:57
*** burgerk has quit IRC22:58
*** ianbrown has quit IRC23:02
*** gokrokve_ has quit IRC23:03
*** chlong has quit IRC23:10
*** edmondsw has quit IRC23:21
*** burgerk has joined #openstack-glance23:29
*** vijendar has quit IRC23:31
*** alex_xu has quit IRC23:36
*** alex_xu has joined #openstack-glance23:41
*** jerrygb has quit IRC23:55
*** jerrygb has joined #openstack-glance23:56
*** burgerk has quit IRC23:56
*** takedakn has joined #openstack-glance23:56
*** mingdang1 has joined #openstack-glance23:59

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