Tuesday, 2016-10-11

*** btully has quit IRC00:07
*** btully has joined #openstack-glance00:10
*** sdake_ has quit IRC00:13
*** ducttape_ has quit IRC00:14
*** jamielennox is now known as jamielennox|away00:30
*** mtanino has quit IRC00:34
*** markvoelker has joined #openstack-glance00:36
openstackgerritAlexander Bashmakov proposed openstack/glance: Added overwrite warning for db_export_metadefs.  https://review.openstack.org/36823100:39
*** markvoelker has quit IRC00:41
*** arahal_ has joined #openstack-glance00:46
openstackgerritJamie Lennox proposed openstack/glance_store: Refactor get_manager_for_store in an OO manner  https://review.openstack.org/38474900:46
*** sdake has joined #openstack-glance00:47
*** btully has quit IRC00:47
openstackgerritJamie Lennox proposed openstack/glance_store: Refactor get_manager_for_store in an OO manner  https://review.openstack.org/38474900:48
*** jamielennox|away is now known as jamielennox00:48
*** arahal_ has quit IRC00:50
*** btully has joined #openstack-glance00:58
*** ducttape_ has joined #openstack-glance01:02
*** ducttape_ has quit IRC01:07
*** ducttape_ has joined #openstack-glance01:08
*** markvoelker has joined #openstack-glance01:12
*** sdake has quit IRC01:31
*** ducttape_ has quit IRC01:32
*** atod has joined #openstack-glance01:43
*** tsymanczyk has quit IRC01:44
*** btully has quit IRC01:56
*** btully has joined #openstack-glance02:00
*** sdake has joined #openstack-glance02:07
*** sdake has quit IRC02:07
*** sdake has joined #openstack-glance02:07
*** jcook has quit IRC02:08
*** sdake has quit IRC02:09
*** markvoelker has quit IRC02:10
*** markvoelker has joined #openstack-glance02:10
*** sdake has joined #openstack-glance02:12
*** jcook has joined #openstack-glance02:13
*** btully has quit IRC02:14
*** rosmaita_ has quit IRC02:17
*** rosmaita has quit IRC02:17
*** rosmaita has joined #openstack-glance02:19
*** btully has joined #openstack-glance02:22
-openstackstatus- NOTICE: Jobs running on osic nodes are failing due to network issues with the mirror. We are temporarily disabling the cloud.02:47
*** MattMan has quit IRC02:52
*** MattMan has joined #openstack-glance02:53
*** sdake_ has joined #openstack-glance02:56
*** sdake has quit IRC02:59
*** ducttape_ has joined #openstack-glance03:02
*** ducttape_ has quit IRC03:15
*** btully has quit IRC03:25
*** btully has joined #openstack-glance03:26
*** bpokorny has quit IRC03:40
*** TravT has joined #openstack-glance03:49
*** links has joined #openstack-glance04:01
*** btully has quit IRC04:03
*** btully has joined #openstack-glance04:03
*** TravT has quit IRC04:11
*** ducttape_ has joined #openstack-glance04:15
*** ducttape_ has quit IRC04:20
*** robcresswell has quit IRC04:23
*** robcresswell has joined #openstack-glance04:25
*** agireud has quit IRC04:36
*** btully has quit IRC04:39
*** btully has joined #openstack-glance04:42
*** agireud has joined #openstack-glance04:43
*** btully has quit IRC04:55
*** e0ne has joined #openstack-glance05:00
*** btully has joined #openstack-glance05:03
*** e0ne has quit IRC05:04
*** markvoelker has quit IRC05:13
*** markvoelker has joined #openstack-glance05:15
*** markvoelker has quit IRC05:16
*** ducttape_ has joined #openstack-glance05:17
*** markvoelker has joined #openstack-glance05:18
*** markvoelker has quit IRC05:18
*** markvoelker has joined #openstack-glance05:19
*** markvoelker has quit IRC05:20
*** ducttape_ has quit IRC05:22
*** markvoelker_ has joined #openstack-glance05:25
*** markvoelker has joined #openstack-glance05:27
*** markvoelker has quit IRC05:28
*** markvoelker has joined #openstack-glance05:28
*** TravT has joined #openstack-glance05:30
*** korzen has joined #openstack-glance05:33
*** TravT_ has joined #openstack-glance05:35
*** TravT has quit IRC05:35
*** TravT_ has quit IRC05:37
*** TravT has joined #openstack-glance05:38
*** d0ugal has quit IRC05:40
*** TravT_ has joined #openstack-glance05:40
*** d0ugal has joined #openstack-glance05:41
*** TravT has quit IRC05:42
*** markvoelker has quit IRC05:44
*** TravT_ has quit IRC05:44
*** markvoelker has joined #openstack-glance05:45
*** TravT has joined #openstack-glance05:45
*** e0ne has joined #openstack-glance05:47
*** TravT has quit IRC05:49
*** TravT has joined #openstack-glance05:50
*** TravT has quit IRC05:54
*** TravT has joined #openstack-glance05:54
*** e0ne has quit IRC05:56
*** TravT has quit IRC05:59
*** TravT has joined #openstack-glance05:59
*** groen692 has joined #openstack-glance06:02
*** TravT has quit IRC06:03
*** TravT has joined #openstack-glance06:04
*** rcernin has joined #openstack-glance06:07
*** TravT_ has joined #openstack-glance06:08
*** TravT has quit IRC06:08
*** TravT_ has quit IRC06:13
*** TravT has joined #openstack-glance06:13
*** TravT has quit IRC06:14
*** TravT has joined #openstack-glance06:14
*** TravT has quit IRC06:15
*** TravT has joined #openstack-glance06:16
*** ducttape_ has joined #openstack-glance06:18
*** TravT has quit IRC06:19
*** TravT has joined #openstack-glance06:19
*** ducttape_ has quit IRC06:22
*** TravT has quit IRC06:24
*** TravT has joined #openstack-glance06:24
*** TravT has quit IRC06:29
*** btully has quit IRC06:29
*** pcaruana has joined #openstack-glance06:36
*** diegows has quit IRC06:39
*** diegows has joined #openstack-glance06:40
*** btully has joined #openstack-glance06:40
*** atod has quit IRC06:58
openstackgerritliwei proposed openstack/glance: Add more resource url in readme.rst  https://review.openstack.org/36552507:03
openstackgerritliwei proposed openstack/glance: Use OS::Nova::HostAggregate instead of OS::Nova::Aggregate  https://review.openstack.org/35974207:05
*** btully has quit IRC07:12
*** btully has joined #openstack-glance07:13
openstackgerritAndrey Pavlov proposed openstack/glance_store: Add cinder_volume_type to cinder store configuration  https://review.openstack.org/35782707:14
*** flaper87 has quit IRC07:24
*** btully has quit IRC07:47
*** hogepodge has quit IRC07:51
*** btully has joined #openstack-glance07:53
*** aarefiev has quit IRC07:55
*** aarefiev has joined #openstack-glance07:56
*** markvoelker has quit IRC07:57
*** hogepodge has joined #openstack-glance07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-glance08:00
*** e0ne has joined #openstack-glance08:00
*** sdake_ has quit IRC08:21
*** dshakhray_ has joined #openstack-glance08:31
*** burgerk_ has joined #openstack-glance08:32
*** cjmarti has joined #openstack-glance08:35
*** rosmaita_ has joined #openstack-glance08:36
*** dosaboy_ has joined #openstack-glance08:37
*** jraju has joined #openstack-glance08:41
*** links has quit IRC08:41
*** rosmaita has quit IRC08:41
*** hoonetorg has quit IRC08:41
*** burgerk has quit IRC08:41
*** castulo has quit IRC08:41
*** ediardo has quit IRC08:41
*** fungi has quit IRC08:41
*** mgagne has quit IRC08:41
*** odyssey4me has quit IRC08:41
*** toan has quit IRC08:41
*** hyakuhei has quit IRC08:41
*** tsufiev has quit IRC08:41
*** dosaboy has quit IRC08:41
*** onovy has quit IRC08:41
*** cjmarti is now known as castulo08:41
*** atod has joined #openstack-glance08:43
*** tsufiev has joined #openstack-glance08:44
*** pumaranikar has joined #openstack-glance08:45
*** hoonetorg has joined #openstack-glance08:46
*** atod has quit IRC08:48
*** btully has quit IRC08:49
*** ediardo has joined #openstack-glance08:49
*** fungi has joined #openstack-glance08:49
*** mgagne has joined #openstack-glance08:49
*** toan has joined #openstack-glance08:49
*** odyssey4me has joined #openstack-glance08:49
*** hyakuhei has joined #openstack-glance08:49
*** onovy has joined #openstack-glance08:49
*** dharinic has joined #openstack-glance08:50
*** btully has joined #openstack-glance08:51
*** anush has joined #openstack-glance08:52
*** dshakhray_ has quit IRC08:56
*** markvoelker has joined #openstack-glance08:57
*** markvoelker has quit IRC09:03
*** mfedosin has joined #openstack-glance09:06
*** openstack has joined #openstack-glance09:29
*** onovy has quit IRC09:30
*** onovy has joined #openstack-glance09:38
*** dshakhray_ has joined #openstack-glance09:41
*** markvoelker has joined #openstack-glance09:59
*** btully has quit IRC10:00
*** markvoelker has quit IRC10:04
*** TravT has joined #openstack-glance10:04
*** TravT has quit IRC10:08
*** dshakhray_ has quit IRC10:09
*** dshakhray has joined #openstack-glance10:11
*** btully has joined #openstack-glance10:12
*** TravT has joined #openstack-glance10:14
*** jraju has quit IRC10:15
*** TravT has quit IRC10:18
*** TravT has joined #openstack-glance10:18
*** sdake has joined #openstack-glance10:23
*** ducttape_ has joined #openstack-glance10:23
*** TravT_ has joined #openstack-glance10:23
*** TravT has quit IRC10:23
*** TravT_ has quit IRC10:27
*** ducttape_ has quit IRC10:27
*** TravT has joined #openstack-glance10:28
*** mvk has quit IRC10:30
*** TravT has quit IRC10:32
*** TravT has joined #openstack-glance10:32
*** TravT has quit IRC10:37
*** TravT_ has joined #openstack-glance10:37
*** akorzeni__ has joined #openstack-glance10:40
*** korzen has quit IRC10:42
*** TravT has joined #openstack-glance10:42
*** TravT_ has quit IRC10:42
*** btully has quit IRC10:45
*** TravT has quit IRC10:46
*** TravT has joined #openstack-glance10:46
*** btully has joined #openstack-glance10:47
*** korzen has joined #openstack-glance10:48
*** akorzeni__ has quit IRC10:49
*** TravT has quit IRC10:51
*** TravT_ has joined #openstack-glance10:51
*** TravT_ has quit IRC10:56
*** TravT has joined #openstack-glance10:56
*** markvoelker has joined #openstack-glance10:59
*** TravT_ has joined #openstack-glance11:00
*** TravT has quit IRC11:01
*** mvk has joined #openstack-glance11:01
*** markvoelker has quit IRC11:04
*** TravT_ has quit IRC11:05
*** TravT has joined #openstack-glance11:05
*** links has joined #openstack-glance11:09
*** TravT has quit IRC11:10
*** sdake_ has joined #openstack-glance11:14
*** TravT has joined #openstack-glance11:15
*** sdake has quit IRC11:15
*** TravT has quit IRC11:19
*** TravT has joined #openstack-glance11:19
*** TravT has quit IRC11:24
*** ducttape_ has joined #openstack-glance11:24
*** TravT has joined #openstack-glance11:24
*** ducttape_ has quit IRC11:28
*** TravT has quit IRC11:28
*** TravT has joined #openstack-glance11:29
*** TravT_ has joined #openstack-glance11:33
*** TravT has quit IRC11:34
*** jamielennox is now known as jamielennox|away11:36
*** TravT_ has quit IRC11:38
*** TravT has joined #openstack-glance11:38
*** TravT has quit IRC11:42
*** TravT has joined #openstack-glance11:43
*** TravT has quit IRC11:47
*** TravT has joined #openstack-glance11:48
*** openstackgerrit has quit IRC11:48
*** openstackgerrit has joined #openstack-glance11:48
*** TravT has quit IRC11:52
*** TravT has joined #openstack-glance11:52
*** TravT_ has joined #openstack-glance11:57
*** TravT has quit IRC11:57
*** flaper87 has joined #openstack-glance11:57
*** flaper87 is now known as Guest973811:58
*** markvoelker has joined #openstack-glance12:00
*** Guest9738 has quit IRC12:01
*** TravT_ has quit IRC12:01
*** TravT has joined #openstack-glance12:01
*** markvoelker has quit IRC12:05
*** TravT has quit IRC12:08
*** TravT has joined #openstack-glance12:09
*** TravT has quit IRC12:13
*** TravT_ has joined #openstack-glance12:13
*** edmondsw has joined #openstack-glance12:14
*** ducttape_ has joined #openstack-glance12:17
*** openstackgerrit has quit IRC12:18
*** TravT_ has quit IRC12:18
*** openstackgerrit has joined #openstack-glance12:18
*** TravT has joined #openstack-glance12:18
*** TravT has quit IRC12:22
*** TravT has joined #openstack-glance12:23
*** darrenswhite has quit IRC12:24
*** darrenswhite has joined #openstack-glance12:24
*** TravT_ has joined #openstack-glance12:28
*** TravT has quit IRC12:28
*** ducttape_ has quit IRC12:30
*** ducttape_ has joined #openstack-glance12:30
*** TravT_ has quit IRC12:32
*** ducttape_ has quit IRC12:37
*** korzen has quit IRC12:41
*** markvoelker has joined #openstack-glance12:43
*** flaper87 has joined #openstack-glance12:51
*** flaper87 has quit IRC12:51
*** flaper87 has joined #openstack-glance12:51
openstackgerritDarren White proposed openstack/glance: Glance retired images  https://review.openstack.org/38298113:02
openstackgerritDarren White proposed openstack/glance: Glance retired images  https://review.openstack.org/38298113:17
*** ctrath has joined #openstack-glance13:17
*** rosmaita_ is now known as rosmaita13:19
rosmaitasigmavirus: when you get a chance, could you take a look at this patch? It's proposed for a bug you set as High  ... wondering why the only thing he changed was manage_image_cache13:27
*** TravT has joined #openstack-glance13:40
*** ctrath has quit IRC13:57
*** ctrath has joined #openstack-glance14:00
sigmavirusrosmaita: which patch?14:00
rosmaitasigmavirus: yeah, that would help14:00
*** croelandt has joined #openstack-glance14:00
*** anush_ has joined #openstack-glance14:00
sigmavirusrosmaita: yeah, but once we have the inter-conscious link set-up, it won't be necessary :P14:01
rosmaitasigmavirus: https://review.openstack.org/#/c/384655/14:01
sigmavirusoh yeah14:01
*** ducttape_ has joined #openstack-glance14:01
sigmavirusthat's a good bug14:01
rosmaitasigmavirus: not sure that would help, would be too noisy, at least on my end14:01
rosmaitaand an old bug14:01
Jokke_o/14:01
rosmaitahi erno14:01
sigmavirusrosmaita: yeah, maybe we should wait for a "do not disturb" feature in that link :P14:01
croelandt\o14:01
croelandthi14:02
* sigmavirus hides from Jokke_ 14:02
*** mfedosin has quit IRC14:02
rosmaitahello cyril14:02
rosmaitagimme a sec to find the etherpad, want to make sure i start the meeting correctly14:02
croelandtyep14:02
*** e0ne has quit IRC14:03
rosmaita#startmeeting image import sync14:03
openstackMeeting started Tue Oct 11 14:03:09 2016 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.14:03
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:03
openstackThe meeting name has been set to 'image_import_sync'14:03
rosmaita#link https://etherpad.openstack.org/p/glance-ocata-image-import-sync14:03
rosmaitaok, so we were going to refresh ourselves about taskflow14:04
rosmaita(which i personally did not have time to do)14:04
Jokke_So the base_imprt flow is good starting point14:05
rosmaitaok14:05
Jokke_due to it's architecture we cannot use it as is, but it's pretty good skeleton for what we want to do14:05
croelandtsound optimistic14:06
rosmaitafor the people following along:14:06
*** e0ne has joined #openstack-glance14:06
rosmaita#link http://git.openstack.org/cgit/openstack/glance/tree/glance/async/flows/base_import.py14:06
rosmaitai'm looking at all of flavio's notes about glance_store14:07
*** e0ne has quit IRC14:07
rosmaitathat's making me think a design session on glance_store refactoring is a good idea for barcelona14:07
Jokke_I was just gonna say that most of the lines are Flavio ranting, but we just ignore that part for now ;)14:07
Jokke_;)14:07
* croelandt loves the "def revert(self): pass"14:07
*** rcernin has quit IRC14:07
rosmaitayea, flavio does not write boring code14:07
Jokke_rosmaita: that's very nice and polite of you :P14:08
rosmaitai try14:08
Jokke_so my plan for this week is to steal the bits from the base_import we can utilize and make api_image_import flow there14:09
croelandtI see14:10
croelandtwhat endpoints would that cover exactly?14:10
*** anush_ has quit IRC14:10
Jokke_and get the configuration option to use for the scratchpad ... hopefully I'll be able to import image from it to the back end store14:10
Jokke_croelandt: none14:10
Jokke_that would be just the taskflow to be used under the hood when the image import is done14:11
Jokke_with the new workflow14:11
croelandtyeah14:12
croelandtSo the only issue with the inside-out approach is that I have no idea what we're trying to code14:12
rosmaitaso we'll need: user -> scratchpad, (validation while in scratchpad), scratchpad -> backend14:12
Jokke_croelandt: so the execution of that taskflow would be triggered by the last call of the image import process14:12
rosmaitaJokke_ is talkind about scratchpad -> backend14:13
croelandtok14:13
rosmaitacroelandt: it might be worth thinking about how the validation should happen14:14
Jokke_and that taskflow would be then pluggable to pull the image data either from the FS (code ready in the base_import) or swift (needs to be done) or external uri (already implemented in the base_import) and push it to the glance_store (already implemented in the base_import)14:14
croelandtrosmaita: isn't that part of the spec?14:14
rosmaitacroelandt: yes, but i think the spec says what more than how14:15
rosmaitaso the swift stuff doesn't need to happen in the MVP14:17
*** mfedosin has joined #openstack-glance14:17
Jokke_rosmaita: but the design needs to be able to facilitate it14:17
croelandtrosmaita: can't see the word "scratchpad" in either the spec nor the current code :p14:17
rosmaitaJokke_: very true14:17
rosmaitacroelandt: sorry, "staging area"14:17
Jokke_just like the external uri (that just happens to exist already)14:17
rosmaita(formerly, "the bikeshed")14:17
croelandtoh right14:17
croelandtnot a native speaker, so I sometimes have trouble jumping between the terms :)14:18
Jokke_sorry, yeah, I'll keep using bikeshed again so we are on the same page ;)14:18
rosmaitacroelandt: sorry about that14:18
rosmaitacroelandt: in the first few drafts, we called it "the bikeshed" because we couldn't agree on what to call it14:18
Jokke_croelandt: same here, so I constantly use wrong ones. Sorry14:18
Jokke_rosmaita: actually we called it bikeshed from the very beginning as we _knew_ we wouldn't be able to agree what to call it :P14:19
rosmaitawell, i think i got overruled on having PUT v2/bikeshed in the api14:19
rosmaita(that's why we call it the "staging area" -- PUT v2/stage)14:20
Jokke_unfortunately yes. Perhaps we should revive that14:20
rosmaitait woudl be good for openstack to have a bikeshedding api14:20
rosmaitawould keep things more organized14:20
Jokke_so yes, when we get that taskflow working, we can then implement the staging and the import endpoints and we effectively have MVP ... then the swift part and the discovery and tons of testing/hardening14:23
Jokke_== profit14:23
rosmaitaso, if Jokke_ is working on the bikeshed -> backend bit, is it too early for croelandt to start on the user -> bikeshed part?14:23
Jokke_I think that'd be fine if you croelandt want to take that on14:24
rosmaitacroelandt: what parts interest you most?14:25
croelandtnot sure exactly where to start, but why not14:25
Jokke_croelandt: we can sync on that offline14:25
croelandtrosmaita: I honestly do not really see where we are going, especially because of the inside/out methodology (which is definitely right one in terms of reviews, though)14:25
croelandtI could give the user -> bikeshed a try14:26
rosmaitaJokke_: can you use help on the bikeshed -> backend part?14:26
croelandtif Jokke_ does the bikeshed -> store part, then we have the full stack \o/14:26
rosmaitayeah, we would basically have image upload re-implemented!14:26
Jokke_ok, I revisit my work plan that much that I try to make flowchart of the macro level14:26
Jokke_maybe that helps out to understand what we need there and what bits we are working14:27
rosmaitai think Jokke_ 's idea right now is that we start with the user's bits already in the bikeshed, and not worry about how they got there, and write the flow to get them into the backend14:28
Jokke_correct14:28
rosmaitaso for collaboration,14:28
croelandtyep14:28
rosmaitacroelandt, you could either work on the same with Erno (if that would make sense)14:29
rosmaitaor work on another part that already assumes the bits are in the bikeshed14:29
rosmaita(that woudl be the validation stuff)14:29
Jokke_so as long as I know the path to the image file on the local node I can just make the workflow, drop the image to that path, use the old _deprecated_ tasks api *sigh* to trigger the flow and see profit14:29
rosmaitanot sure if it's too early for that, though14:29
croelandtvalidation might be fun14:29
rosmaita(i meant about validation stuff, not Jokke_ 's workflow)14:30
croelandtrosmaita: is there anything in particular you'd like to work on?14:30
* abashmak pipes up14:30
rosmaitahello alex14:30
abashmakI have bandwidth to help14:30
Jokke_there's actually good question about the validation14:30
rosmaitacroelandt: i want to work on the discovery part, i need to revise the schema14:30
*** anush_ has joined #openstack-glance14:31
Jokke_do we want that just being tasks on the import flow or separate taskflow that then triggers the import flow14:31
rosmaitaJokke_: not sure i understand your question14:31
rosmaitaabashmak: have you had a chance to work with taskflow yet?14:32
abashmaksadly no14:32
rosmaitathat's ok14:32
rosmaitamy suggestion would be to look at taskflow and the stuff in the glance/async directory to see how it's being used14:32
croelandtrosmaita: oh I have a patch for that14:32
rosmaitacroelandt: yes, i know14:33
rosmaitacroelandt: i don't want to take over your patch, but i need to revise the spec to make sure we've captured all the stuff that came up in the defcore discussions14:33
croelandtrosmaita: feel free to ping me about that14:33
rosmaitacroelandt: i do have a question about whether our current json-schema code will be able to handle the proposed schema14:34
Jokke_sorry, had to dial in to the next meeting14:34
rosmaitaJokke_: ok14:34
Jokke_so as the validation is precondition to the import14:34
Jokke_we have two options 1) we have validation taskflow that gets triggered by the api call and when it finishes it triggers the import taskflow or 2) the validation is bunch of tasks as part of the import taskflow and will be one integrated taskflow triggered by the api call14:36
rosmaitathe key thing is that we'll want operators to be able to insert custom validation code14:37
rosmaitai'm not sure if either of those is more conducive to that?14:37
Jokke_yes and that's why I'd prefer having separate taskflow for it14:37
rosmaitamaybe (1) is because the validation part is more isolated?14:37
rosmaitaok, so we are thinking the same thing14:38
Jokke_it would keep the import code away from the people who want to implement their own validations14:38
rosmaitayeah, that would be good14:38
Jokke_so basically we could document "Write taskflow, set it to config being the used validation flow, make sure it calls api_image_import flow when it's finished14:39
rosmaitai like that14:39
rosmaitait keeps the database interaction stuff away from the operators14:40
rosmaitawe handle all the stuff that modifies the image record14:40
* croelandt has a headache now14:40
rosmaitawe are way over time14:40
Jokke_no worries14:40
abashmakanother possibility is for the custom code to plug into a wrapper, that way it doesn't have to call the api_image_import flow, the wrapper would do that14:40
Jokke_croelandt: I'll do the flowchart and walk you through it14:41
rosmaitaabashmak: that's a good idea, the less we leave to operators, the more friendly the import will be14:41
*** sdake_ is now known as sdake14:42
Jokke_abashmak: fell free to look into how to do that wrapper with the glance async/ taskflow code :P14:42
croelandtok14:42
croelandtmight be nice to sum this up in the etherpad14:42
*** ctrath has quit IRC14:42
rosmaitaok, got some notes in the etherpad14:45
rosmaitaok what's next?14:46
*** ctrath has joined #openstack-glance14:46
rosmaitai guess we have enough for a week?14:46
Jokke_yeah, maybe more14:46
rosmaitaok, let's plan to meet again next week14:47
rosmaitaand we can find each other here if we need to14:47
rosmaitathanks, and sorry about this going way over 20 min14:48
rosmaita#endmeeting image import sync14:48
croelandtnp14:48
openstackMeeting ended Tue Oct 11 14:48:11 2016 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:48
openstackMinutes:        http://eavesdrop.openstack.org/meetings/image_import_sync/2016/image_import_sync.2016-10-11-14.03.html14:48
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/image_import_sync/2016/image_import_sync.2016-10-11-14.03.txt14:48
openstackLog:            http://eavesdrop.openstack.org/meetings/image_import_sync/2016/image_import_sync.2016-10-11-14.03.log.html14:48
croelandtSame time next week?14:48
rosmaitacroelandt: yes ... does that work for you?14:48
croelandtok14:50
*** mfedosin has quit IRC14:52
rosmaitacroelandt: ok, great, i'd like us to keep 14:00-14:20 utc open on tuesdays throughout the cycle14:54
croelandtworks for me14:54
rosmaitaok, cool14:54
Jokke_thanks rosmaita14:55
rosmaitaJokke_: thank you!14:56
*** mfedosin has joined #openstack-glance15:04
*** btully has quit IRC15:09
*** btully has joined #openstack-glance15:10
*** ccesario has quit IRC15:17
*** jistr is now known as jistr|biab15:25
*** _ducttape_ has joined #openstack-glance15:25
*** ducttape_ has quit IRC15:29
*** ccesario has joined #openstack-glance15:31
*** pcaruana has quit IRC15:31
openstackgerritDharini Chandrasekar proposed openstack/glance: Handling scrubber's exit in non-daemon mode.  https://review.openstack.org/38271115:35
*** ctrath has quit IRC15:38
dharinicrosmaita, stevelle: regarding this- https://review.openstack.org/#/c/375123/15:39
*** ctrath has joined #openstack-glance15:39
dharinicstevelle suggested deprecation header for the use of Content-Range. I wasnt sure how deprecations worked when it involved APIs15:42
rosmaitalooking15:42
stevelleI don't think anyone will see the header if it was added15:43
stevelleI also don't think anyone will notice the log message15:43
stevelleand I have doubts that operators will read the release notes, but I'm not an optimist :)15:43
*** atod has joined #openstack-glance15:43
rosmaitahmmm ... we pretty much assume that operators read the release notes15:44
rosmaitamay need to revise that assumption!15:44
stevelleit's a best-effort thing15:44
stevellealso operators are not necessarily the audience you need to reach for API changes15:44
stevelleusers of the cloud may need to know, though I'm not sure why they are reading segments of images from glance in that case15:45
stevelleat the least I would want to log use of the deprecated header in requests15:46
dharinicI am not sure how in this case we can retain support for both Content-range and range15:46
dharinicwhile deprecating content-range15:46
dharinicit might make things a little messy on wsgi.py15:46
dharinicThis change/spec is blocking the resolution of the bug on HTTP range requests.15:47
Jokke_stevelle: tbh, I tend to disagree. I don't know too many cloud users (Obviously apart from Monty) who would use the APIs directly15:47
Jokke_;)15:47
dhariniccan you guys tell me how we can go about this?15:48
*** korzen has joined #openstack-glance15:48
dharinicThe related patch: https://review.openstack.org/#/c/367528/15:48
stevelleI would tolerate a bit of extra code in wsgy.py to look for the Range header, and else look for Content-Range, during a deprecation period15:49
dhariniccool.15:50
stevelleJokke_: you're probably right that nobody really uses range requests -- particularly because we don't recognize the right header :P15:50
dharinicJust wanted to clarify if that was okay especially cos it was an API related change15:50
stevellebut it's a hard breaking change15:50
rosmaitadharinic: what w3c doc are you relying on to say content-range is illegal on requests?15:50
dharinicLet me pull it up15:51
rosmaitaty15:51
rosmaitai am looking at one now, but it's pretty ambiguous15:51
dharinichttps://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html15:51
Jokke_so first of all, based on the current deprecation line of OpenStack we can deprecate it and advice not to use it, but we cannot just drop the old behavior15:51
dharinicJokke_, True15:51
*** e0ne has joined #openstack-glance15:51
stevelleRFC 7233 is the right one to look at btw, as it updated 261615:52
dharinicrosmaita: https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html15:52
rosmaitadharinic: ty15:52
stevellehttps://tools.ietf.org/html/rfc723315:52
rosmaitastevelle: ty15:54
*** links has quit IRC15:54
openstackgerritDharini Chandrasekar proposed openstack/glance: Handling scrubber's exit in non-daemon mode.  https://review.openstack.org/38271115:56
Jokke_so fwiw I'm fine us actually implementing the parsing of Range but I'm agains to just replace the Content-Range with it15:58
*** tsymanczyk has joined #openstack-glance15:58
stevelleso parsing either but not just range, right Jokke_?15:59
dharinicrosmaita: https://tools.ietf.org/html/rfc7233#section-4.215:59
Jokke_and reference to that, we have actually quite a few respond code conflicts in our APIs as changing them would be backwards incompatible, which is more important than following the rfcs15:59
Jokke_stevelle: correct15:59
dharinicThis was what nikhil referred to, when he commented on the patch suggesting we be using Range16:00
rosmaitadharinic: you need to add a ref to rfc 7233 in your spec-lite16:00
rosmaitaalso, looks like the blanket statement "content-range not appropriate for requests" is incorrect16:01
dharinicOkay. Sure.16:01
rosmaitaincorrect for GET16:01
rosmaitabut may be legal for PUT from what i can tell16:01
stevellebut makes no sense16:01
rosmaitawhich doesnt' matter16:01
dharinicYes. Thats right.16:01
rosmaitano, you could replace the middle bytes of some object16:01
stevellethat would better be as a PATCH though16:02
Jokke_rosmaita: we definitely do not support partial uploads ;)16:02
dharinic:D16:02
rosmaitaJokke_: agreed16:02
stevelleI'm splitting hairs though16:02
rosmaitame too16:02
rosmaitakey thing is just to describe clearly the scope of this change16:03
Jokke_++16:03
rosmaitabecuase these docs are a real pain to wade through16:03
Jokke_and not break backwards compatibility16:03
rosmaitai've only been looking for 8 min and already have a headache16:03
stevelleXD16:03
dharinic:D16:03
Jokke_regardless if it happens to be against rfc but work or not16:03
rosmaitayes, Jokke_ is correct, we must preserve backwards compat16:03
rosmaitai am thinking that we accept content-range as a pseudonym of range on GETs16:04
rosmaitabut don't tell anyone16:04
rosmaitawe only doc range16:04
stevelle+1 leave Content-Range undocumented16:04
dharinicSo, is it alright that we dont fix partial downloads cos we are waiting on the spec for using "Range"16:04
rosmaitahow big a maintenance problem is that?16:04
stevellestate deprecation though16:04
Jokke_++16:04
Jokke_stevelle: we don't need to even state it deprecated as we will never be removing it anyways16:04
dharinicor do you guys siuggest that I fix it (latest patch on https://review.openstack.org/#/c/367528/6) with usage "Content-range"16:05
stevelleJokke_: never?  ok then16:05
stevelleI'm fine with that16:05
Jokke_stevelle: that's what the community wants. You can deprecate, you cannot remove16:05
Jokke_so in this point why bother even deprecate16:05
rosmaitayeah, we are just being a bit liberal in what we accept16:06
stevelleJokke_: I'm not entirely sure about the microversion thing, and not a fan in general, but what you said makes perfect sense16:06
Jokke_stevelle: Glance does not use microversions either16:06
Jokke_stevelle: and even with microversions you still need to maintain/provide the previous ones so that won't help functionality wise16:07
stevelledharinic: for https://review.openstack.org/#/c/367528 - Retaining use of Content-Range makes perfect sense, use the lite spec to add Range parsing without removing Content-Range16:07
*** jistr|biab is now known as jistr16:07
stevelleJokke_: that's where I was unclear, because the whole idea of minor api version #s bothers me I never investigated16:08
dharinicThanks stevelle16:08
*** atod has quit IRC16:08
dharinicThanks rosmaita and Jokke_ :)16:09
stevelledharinic: your endurance through all the indecision on this is appreciated :)16:09
*** _ducttape_ has quit IRC16:09
rosmaita++ to what stevelle said16:09
Jokke_stevelle: we use just the minor api versions to indicate that something (like new feature or backwards incompatible change due to security fix) has happened to the API16:09
dharinicWill edit the lite-spec and resubmit. After which I will go ahead and add Range functionality to 36752816:09
*** ducttape_ has joined #openstack-glance16:10
dharinicMy pleasure :)16:10
*** TravT has quit IRC16:10
Jokke_thanks dharinic16:10
rosmaitadharinic: sounds good, thank you!16:10
dharinicOne last thing: This patch paved way for https://review.openstack.org/#/c/380674/16:10
*** mfedosin has quit IRC16:10
dharinicReviews welcome. Will add dependency as abashmak suggested. :)16:11
*** TravT has joined #openstack-glance16:11
*** TravT has quit IRC16:14
*** TravT has joined #openstack-glance16:16
*** hemanth|away is now known as hemanth16:16
*** _ducttape_ has joined #openstack-glance16:21
openstackgerritTimothy Symanczyk proposed openstack/glance: WIP Utilize new visibility values for images  https://review.openstack.org/37985216:23
tsymanczykall py34 tests should now pass.16:24
*** ducttape_ has quit IRC16:24
*** groen692 has quit IRC16:24
*** hoonetorg has quit IRC16:35
*** groen692 has joined #openstack-glance16:46
*** hoonetorg has joined #openstack-glance16:48
openstackgerritDharini Chandrasekar proposed openstack/glance: Handling scrubber's exit in non-daemon mode.  https://review.openstack.org/38271116:54
*** groen692 has quit IRC17:01
*** mtanino has joined #openstack-glance17:16
stevellewould a release note be helpful for https://review.openstack.org/#/c/384655 since the default policy is changed? Operators may want to take special note of changes to default configurations, since that may have special impacts depending on how they deploy.17:16
*** TravT has quit IRC17:49
*** _ducttape_ has quit IRC17:51
*** ducttape_ has joined #openstack-glance17:51
*** dshakhray has quit IRC17:52
*** ducttape_ has quit IRC17:52
*** ducttape_ has joined #openstack-glance17:52
*** atod has joined #openstack-glance17:53
*** atod has quit IRC17:58
*** mvk has quit IRC18:04
*** korzen has quit IRC18:07
*** jamielennox|away is now known as jamielennox18:10
*** ducttape_ has quit IRC18:21
*** ducttape_ has joined #openstack-glance18:21
*** ducttape_ has quit IRC18:25
*** dshakhray has joined #openstack-glance18:26
*** ctrath has quit IRC18:28
*** ctrath has joined #openstack-glance18:29
*** hoonetorg has quit IRC18:35
*** ducttape_ has joined #openstack-glance18:43
*** hoonetorg has joined #openstack-glance18:51
*** TravT has joined #openstack-glance18:58
*** TravT has quit IRC19:01
rosmaitastevelle: agree about release note for https://review.openstack.org/#/c/384655/ , please put a comment on the patch19:02
*** TravT has joined #openstack-glance19:02
stevellewill do19:02
*** ducttape_ has quit IRC19:13
*** ctrath has quit IRC19:30
openstackgerritDharini Chandrasekar proposed openstack/glance-specs: Use ``Range`` header in HTTP requests for partial downloads.  https://review.openstack.org/37512319:30
*** ctrath has joined #openstack-glance19:34
*** e0ne has quit IRC19:37
*** e0ne has joined #openstack-glance19:41
*** atod has joined #openstack-glance19:42
openstackgerritboden proposed openstack/glance: Replace retrying with tenacity  https://review.openstack.org/36761419:44
*** atod has quit IRC19:46
*** e0ne has quit IRC19:50
*** e0ne has joined #openstack-glance19:55
*** e0ne has quit IRC19:58
*** e0ne has joined #openstack-glance20:02
*** exploreshaifali has joined #openstack-glance20:02
*** burgerk__ has joined #openstack-glance20:30
*** burgerk_ has quit IRC20:34
*** dshakhray has quit IRC20:37
*** dshakhray has joined #openstack-glance20:38
*** hemanth is now known as hemanth|bbl20:38
*** sdake_ has joined #openstack-glance20:39
*** sdake has quit IRC20:41
*** burgerk__ has quit IRC20:48
*** sdake has joined #openstack-glance20:51
*** sdake_ has quit IRC20:53
*** ducttape_ has joined #openstack-glance20:54
*** dshakhray has quit IRC20:55
*** anush_ has quit IRC21:08
*** exploreshaifali has quit IRC21:09
*** ducttape_ has quit IRC21:16
*** btully has quit IRC21:19
*** bpokorny has joined #openstack-glance21:20
*** mvk has joined #openstack-glance21:22
*** ducttape_ has joined #openstack-glance21:32
*** dshakhray has joined #openstack-glance21:36
*** dshakhray has quit IRC21:41
*** tsymanczyk has quit IRC21:44
*** atod has joined #openstack-glance21:45
*** e0ne has quit IRC21:48
*** e0ne has joined #openstack-glance21:49
*** e0ne has quit IRC21:50
*** e0ne has joined #openstack-glance21:50
*** e0ne has quit IRC21:50
*** e0ne has joined #openstack-glance21:51
*** e0ne has quit IRC21:51
*** e0ne has joined #openstack-glance21:52
*** e0ne has quit IRC21:52
*** e0ne has joined #openstack-glance21:52
*** e0ne has quit IRC21:53
*** e0ne has joined #openstack-glance21:53
*** e0ne has quit IRC21:54
*** e0ne has joined #openstack-glance21:54
*** e0ne has quit IRC21:54
*** zigo has quit IRC22:02
*** zigo has joined #openstack-glance22:05
*** zigo is now known as Guest5063022:05
*** ctrath has quit IRC22:28
*** ducttape_ has quit IRC22:36
*** TravT_ has joined #openstack-glance22:52
*** TravT has quit IRC22:54
*** TravT_ has quit IRC22:57
*** btully has joined #openstack-glance23:04
openstackgerritDharini Chandrasekar proposed openstack/glance-specs: Use ``Range`` header in HTTP requests for partial downloads.  https://review.openstack.org/37512323:10
*** sdake has quit IRC23:16
*** btully has quit IRC23:17
*** btully has joined #openstack-glance23:18
*** edmondsw has quit IRC23:30
*** david-lyle has joined #openstack-glance23:31
*** btully has quit IRC23:31
*** david-lyle has quit IRC23:34
*** david-lyle has joined #openstack-glance23:35
*** dshakhray has joined #openstack-glance23:35
*** david-lyle has quit IRC23:38
*** david-lyle has joined #openstack-glance23:39
*** david-lyle has quit IRC23:49
*** sdake has joined #openstack-glance23:50
*** sdake_ has joined #openstack-glance23:51
*** sdake has quit IRC23:55
*** ducttape_ has joined #openstack-glance23:56

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