*** btully has quit IRC | 00:07 | |
*** btully has joined #openstack-glance | 00:10 | |
*** sdake_ has quit IRC | 00:13 | |
*** ducttape_ has quit IRC | 00:14 | |
*** jamielennox is now known as jamielennox|away | 00:30 | |
*** mtanino has quit IRC | 00:34 | |
*** markvoelker has joined #openstack-glance | 00:36 | |
openstackgerrit | Alexander Bashmakov proposed openstack/glance: Added overwrite warning for db_export_metadefs. https://review.openstack.org/368231 | 00:39 |
---|---|---|
*** markvoelker has quit IRC | 00:41 | |
*** arahal_ has joined #openstack-glance | 00:46 | |
openstackgerrit | Jamie Lennox proposed openstack/glance_store: Refactor get_manager_for_store in an OO manner https://review.openstack.org/384749 | 00:46 |
*** sdake has joined #openstack-glance | 00:47 | |
*** btully has quit IRC | 00:47 | |
openstackgerrit | Jamie Lennox proposed openstack/glance_store: Refactor get_manager_for_store in an OO manner https://review.openstack.org/384749 | 00:48 |
*** jamielennox|away is now known as jamielennox | 00:48 | |
*** arahal_ has quit IRC | 00:50 | |
*** btully has joined #openstack-glance | 00:58 | |
*** ducttape_ has joined #openstack-glance | 01:02 | |
*** ducttape_ has quit IRC | 01:07 | |
*** ducttape_ has joined #openstack-glance | 01:08 | |
*** markvoelker has joined #openstack-glance | 01:12 | |
*** sdake has quit IRC | 01:31 | |
*** ducttape_ has quit IRC | 01:32 | |
*** atod has joined #openstack-glance | 01:43 | |
*** tsymanczyk has quit IRC | 01:44 | |
*** btully has quit IRC | 01:56 | |
*** btully has joined #openstack-glance | 02:00 | |
*** sdake has joined #openstack-glance | 02:07 | |
*** sdake has quit IRC | 02:07 | |
*** sdake has joined #openstack-glance | 02:07 | |
*** jcook has quit IRC | 02:08 | |
*** sdake has quit IRC | 02:09 | |
*** markvoelker has quit IRC | 02:10 | |
*** markvoelker has joined #openstack-glance | 02:10 | |
*** sdake has joined #openstack-glance | 02:12 | |
*** jcook has joined #openstack-glance | 02:13 | |
*** btully has quit IRC | 02:14 | |
*** rosmaita_ has quit IRC | 02:17 | |
*** rosmaita has quit IRC | 02:17 | |
*** rosmaita has joined #openstack-glance | 02:19 | |
*** btully has joined #openstack-glance | 02: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 IRC | 02:52 | |
*** MattMan has joined #openstack-glance | 02:53 | |
*** sdake_ has joined #openstack-glance | 02:56 | |
*** sdake has quit IRC | 02:59 | |
*** ducttape_ has joined #openstack-glance | 03:02 | |
*** ducttape_ has quit IRC | 03:15 | |
*** btully has quit IRC | 03:25 | |
*** btully has joined #openstack-glance | 03:26 | |
*** bpokorny has quit IRC | 03:40 | |
*** TravT has joined #openstack-glance | 03:49 | |
*** links has joined #openstack-glance | 04:01 | |
*** btully has quit IRC | 04:03 | |
*** btully has joined #openstack-glance | 04:03 | |
*** TravT has quit IRC | 04:11 | |
*** ducttape_ has joined #openstack-glance | 04:15 | |
*** ducttape_ has quit IRC | 04:20 | |
*** robcresswell has quit IRC | 04:23 | |
*** robcresswell has joined #openstack-glance | 04:25 | |
*** agireud has quit IRC | 04:36 | |
*** btully has quit IRC | 04:39 | |
*** btully has joined #openstack-glance | 04:42 | |
*** agireud has joined #openstack-glance | 04:43 | |
*** btully has quit IRC | 04:55 | |
*** e0ne has joined #openstack-glance | 05:00 | |
*** btully has joined #openstack-glance | 05:03 | |
*** e0ne has quit IRC | 05:04 | |
*** markvoelker has quit IRC | 05:13 | |
*** markvoelker has joined #openstack-glance | 05:15 | |
*** markvoelker has quit IRC | 05:16 | |
*** ducttape_ has joined #openstack-glance | 05:17 | |
*** markvoelker has joined #openstack-glance | 05:18 | |
*** markvoelker has quit IRC | 05:18 | |
*** markvoelker has joined #openstack-glance | 05:19 | |
*** markvoelker has quit IRC | 05:20 | |
*** ducttape_ has quit IRC | 05:22 | |
*** markvoelker_ has joined #openstack-glance | 05:25 | |
*** markvoelker has joined #openstack-glance | 05:27 | |
*** markvoelker has quit IRC | 05:28 | |
*** markvoelker has joined #openstack-glance | 05:28 | |
*** TravT has joined #openstack-glance | 05:30 | |
*** korzen has joined #openstack-glance | 05:33 | |
*** TravT_ has joined #openstack-glance | 05:35 | |
*** TravT has quit IRC | 05:35 | |
*** TravT_ has quit IRC | 05:37 | |
*** TravT has joined #openstack-glance | 05:38 | |
*** d0ugal has quit IRC | 05:40 | |
*** TravT_ has joined #openstack-glance | 05:40 | |
*** d0ugal has joined #openstack-glance | 05:41 | |
*** TravT has quit IRC | 05:42 | |
*** markvoelker has quit IRC | 05:44 | |
*** TravT_ has quit IRC | 05:44 | |
*** markvoelker has joined #openstack-glance | 05:45 | |
*** TravT has joined #openstack-glance | 05:45 | |
*** e0ne has joined #openstack-glance | 05:47 | |
*** TravT has quit IRC | 05:49 | |
*** TravT has joined #openstack-glance | 05:50 | |
*** TravT has quit IRC | 05:54 | |
*** TravT has joined #openstack-glance | 05:54 | |
*** e0ne has quit IRC | 05:56 | |
*** TravT has quit IRC | 05:59 | |
*** TravT has joined #openstack-glance | 05:59 | |
*** groen692 has joined #openstack-glance | 06:02 | |
*** TravT has quit IRC | 06:03 | |
*** TravT has joined #openstack-glance | 06:04 | |
*** rcernin has joined #openstack-glance | 06:07 | |
*** TravT_ has joined #openstack-glance | 06:08 | |
*** TravT has quit IRC | 06:08 | |
*** TravT_ has quit IRC | 06:13 | |
*** TravT has joined #openstack-glance | 06:13 | |
*** TravT has quit IRC | 06:14 | |
*** TravT has joined #openstack-glance | 06:14 | |
*** TravT has quit IRC | 06:15 | |
*** TravT has joined #openstack-glance | 06:16 | |
*** ducttape_ has joined #openstack-glance | 06:18 | |
*** TravT has quit IRC | 06:19 | |
*** TravT has joined #openstack-glance | 06:19 | |
*** ducttape_ has quit IRC | 06:22 | |
*** TravT has quit IRC | 06:24 | |
*** TravT has joined #openstack-glance | 06:24 | |
*** TravT has quit IRC | 06:29 | |
*** btully has quit IRC | 06:29 | |
*** pcaruana has joined #openstack-glance | 06:36 | |
*** diegows has quit IRC | 06:39 | |
*** diegows has joined #openstack-glance | 06:40 | |
*** btully has joined #openstack-glance | 06:40 | |
*** atod has quit IRC | 06:58 | |
openstackgerrit | liwei proposed openstack/glance: Add more resource url in readme.rst https://review.openstack.org/365525 | 07:03 |
openstackgerrit | liwei proposed openstack/glance: Use OS::Nova::HostAggregate instead of OS::Nova::Aggregate https://review.openstack.org/359742 | 07:05 |
*** btully has quit IRC | 07:12 | |
*** btully has joined #openstack-glance | 07:13 | |
openstackgerrit | Andrey Pavlov proposed openstack/glance_store: Add cinder_volume_type to cinder store configuration https://review.openstack.org/357827 | 07:14 |
*** flaper87 has quit IRC | 07:24 | |
*** btully has quit IRC | 07:47 | |
*** hogepodge has quit IRC | 07:51 | |
*** btully has joined #openstack-glance | 07:53 | |
*** aarefiev has quit IRC | 07:55 | |
*** aarefiev has joined #openstack-glance | 07:56 | |
*** markvoelker has quit IRC | 07:57 | |
*** hogepodge has joined #openstack-glance | 07:58 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-glance | 08:00 | |
*** e0ne has joined #openstack-glance | 08:00 | |
*** sdake_ has quit IRC | 08:21 | |
*** dshakhray_ has joined #openstack-glance | 08:31 | |
*** burgerk_ has joined #openstack-glance | 08:32 | |
*** cjmarti has joined #openstack-glance | 08:35 | |
*** rosmaita_ has joined #openstack-glance | 08:36 | |
*** dosaboy_ has joined #openstack-glance | 08:37 | |
*** jraju has joined #openstack-glance | 08:41 | |
*** links has quit IRC | 08:41 | |
*** rosmaita has quit IRC | 08:41 | |
*** hoonetorg has quit IRC | 08:41 | |
*** burgerk has quit IRC | 08:41 | |
*** castulo has quit IRC | 08:41 | |
*** ediardo has quit IRC | 08:41 | |
*** fungi has quit IRC | 08:41 | |
*** mgagne has quit IRC | 08:41 | |
*** odyssey4me has quit IRC | 08:41 | |
*** toan has quit IRC | 08:41 | |
*** hyakuhei has quit IRC | 08:41 | |
*** tsufiev has quit IRC | 08:41 | |
*** dosaboy has quit IRC | 08:41 | |
*** onovy has quit IRC | 08:41 | |
*** cjmarti is now known as castulo | 08:41 | |
*** atod has joined #openstack-glance | 08:43 | |
*** tsufiev has joined #openstack-glance | 08:44 | |
*** pumaranikar has joined #openstack-glance | 08:45 | |
*** hoonetorg has joined #openstack-glance | 08:46 | |
*** atod has quit IRC | 08:48 | |
*** btully has quit IRC | 08:49 | |
*** ediardo has joined #openstack-glance | 08:49 | |
*** fungi has joined #openstack-glance | 08:49 | |
*** mgagne has joined #openstack-glance | 08:49 | |
*** toan has joined #openstack-glance | 08:49 | |
*** odyssey4me has joined #openstack-glance | 08:49 | |
*** hyakuhei has joined #openstack-glance | 08:49 | |
*** onovy has joined #openstack-glance | 08:49 | |
*** dharinic has joined #openstack-glance | 08:50 | |
*** btully has joined #openstack-glance | 08:51 | |
*** anush has joined #openstack-glance | 08:52 | |
*** dshakhray_ has quit IRC | 08:56 | |
*** markvoelker has joined #openstack-glance | 08:57 | |
*** markvoelker has quit IRC | 09:03 | |
*** mfedosin has joined #openstack-glance | 09:06 | |
*** openstack has joined #openstack-glance | 09:29 | |
*** onovy has quit IRC | 09:30 | |
*** onovy has joined #openstack-glance | 09:38 | |
*** dshakhray_ has joined #openstack-glance | 09:41 | |
*** markvoelker has joined #openstack-glance | 09:59 | |
*** btully has quit IRC | 10:00 | |
*** markvoelker has quit IRC | 10:04 | |
*** TravT has joined #openstack-glance | 10:04 | |
*** TravT has quit IRC | 10:08 | |
*** dshakhray_ has quit IRC | 10:09 | |
*** dshakhray has joined #openstack-glance | 10:11 | |
*** btully has joined #openstack-glance | 10:12 | |
*** TravT has joined #openstack-glance | 10:14 | |
*** jraju has quit IRC | 10:15 | |
*** TravT has quit IRC | 10:18 | |
*** TravT has joined #openstack-glance | 10:18 | |
*** sdake has joined #openstack-glance | 10:23 | |
*** ducttape_ has joined #openstack-glance | 10:23 | |
*** TravT_ has joined #openstack-glance | 10:23 | |
*** TravT has quit IRC | 10:23 | |
*** TravT_ has quit IRC | 10:27 | |
*** ducttape_ has quit IRC | 10:27 | |
*** TravT has joined #openstack-glance | 10:28 | |
*** mvk has quit IRC | 10:30 | |
*** TravT has quit IRC | 10:32 | |
*** TravT has joined #openstack-glance | 10:32 | |
*** TravT has quit IRC | 10:37 | |
*** TravT_ has joined #openstack-glance | 10:37 | |
*** akorzeni__ has joined #openstack-glance | 10:40 | |
*** korzen has quit IRC | 10:42 | |
*** TravT has joined #openstack-glance | 10:42 | |
*** TravT_ has quit IRC | 10:42 | |
*** btully has quit IRC | 10:45 | |
*** TravT has quit IRC | 10:46 | |
*** TravT has joined #openstack-glance | 10:46 | |
*** btully has joined #openstack-glance | 10:47 | |
*** korzen has joined #openstack-glance | 10:48 | |
*** akorzeni__ has quit IRC | 10:49 | |
*** TravT has quit IRC | 10:51 | |
*** TravT_ has joined #openstack-glance | 10:51 | |
*** TravT_ has quit IRC | 10:56 | |
*** TravT has joined #openstack-glance | 10:56 | |
*** markvoelker has joined #openstack-glance | 10:59 | |
*** TravT_ has joined #openstack-glance | 11:00 | |
*** TravT has quit IRC | 11:01 | |
*** mvk has joined #openstack-glance | 11:01 | |
*** markvoelker has quit IRC | 11:04 | |
*** TravT_ has quit IRC | 11:05 | |
*** TravT has joined #openstack-glance | 11:05 | |
*** links has joined #openstack-glance | 11:09 | |
*** TravT has quit IRC | 11:10 | |
*** sdake_ has joined #openstack-glance | 11:14 | |
*** TravT has joined #openstack-glance | 11:15 | |
*** sdake has quit IRC | 11:15 | |
*** TravT has quit IRC | 11:19 | |
*** TravT has joined #openstack-glance | 11:19 | |
*** TravT has quit IRC | 11:24 | |
*** ducttape_ has joined #openstack-glance | 11:24 | |
*** TravT has joined #openstack-glance | 11:24 | |
*** ducttape_ has quit IRC | 11:28 | |
*** TravT has quit IRC | 11:28 | |
*** TravT has joined #openstack-glance | 11:29 | |
*** TravT_ has joined #openstack-glance | 11:33 | |
*** TravT has quit IRC | 11:34 | |
*** jamielennox is now known as jamielennox|away | 11:36 | |
*** TravT_ has quit IRC | 11:38 | |
*** TravT has joined #openstack-glance | 11:38 | |
*** TravT has quit IRC | 11:42 | |
*** TravT has joined #openstack-glance | 11:43 | |
*** TravT has quit IRC | 11:47 | |
*** TravT has joined #openstack-glance | 11:48 | |
*** openstackgerrit has quit IRC | 11:48 | |
*** openstackgerrit has joined #openstack-glance | 11:48 | |
*** TravT has quit IRC | 11:52 | |
*** TravT has joined #openstack-glance | 11:52 | |
*** TravT_ has joined #openstack-glance | 11:57 | |
*** TravT has quit IRC | 11:57 | |
*** flaper87 has joined #openstack-glance | 11:57 | |
*** flaper87 is now known as Guest9738 | 11:58 | |
*** markvoelker has joined #openstack-glance | 12:00 | |
*** Guest9738 has quit IRC | 12:01 | |
*** TravT_ has quit IRC | 12:01 | |
*** TravT has joined #openstack-glance | 12:01 | |
*** markvoelker has quit IRC | 12:05 | |
*** TravT has quit IRC | 12:08 | |
*** TravT has joined #openstack-glance | 12:09 | |
*** TravT has quit IRC | 12:13 | |
*** TravT_ has joined #openstack-glance | 12:13 | |
*** edmondsw has joined #openstack-glance | 12:14 | |
*** ducttape_ has joined #openstack-glance | 12:17 | |
*** openstackgerrit has quit IRC | 12:18 | |
*** TravT_ has quit IRC | 12:18 | |
*** openstackgerrit has joined #openstack-glance | 12:18 | |
*** TravT has joined #openstack-glance | 12:18 | |
*** TravT has quit IRC | 12:22 | |
*** TravT has joined #openstack-glance | 12:23 | |
*** darrenswhite has quit IRC | 12:24 | |
*** darrenswhite has joined #openstack-glance | 12:24 | |
*** TravT_ has joined #openstack-glance | 12:28 | |
*** TravT has quit IRC | 12:28 | |
*** ducttape_ has quit IRC | 12:30 | |
*** ducttape_ has joined #openstack-glance | 12:30 | |
*** TravT_ has quit IRC | 12:32 | |
*** ducttape_ has quit IRC | 12:37 | |
*** korzen has quit IRC | 12:41 | |
*** markvoelker has joined #openstack-glance | 12:43 | |
*** flaper87 has joined #openstack-glance | 12:51 | |
*** flaper87 has quit IRC | 12:51 | |
*** flaper87 has joined #openstack-glance | 12:51 | |
openstackgerrit | Darren White proposed openstack/glance: Glance retired images https://review.openstack.org/382981 | 13:02 |
openstackgerrit | Darren White proposed openstack/glance: Glance retired images https://review.openstack.org/382981 | 13:17 |
*** ctrath has joined #openstack-glance | 13:17 | |
*** rosmaita_ is now known as rosmaita | 13:19 | |
rosmaita | sigmavirus: 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_cache | 13:27 |
*** TravT has joined #openstack-glance | 13:40 | |
*** ctrath has quit IRC | 13:57 | |
*** ctrath has joined #openstack-glance | 14:00 | |
sigmavirus | rosmaita: which patch? | 14:00 |
rosmaita | sigmavirus: yeah, that would help | 14:00 |
*** croelandt has joined #openstack-glance | 14:00 | |
*** anush_ has joined #openstack-glance | 14:00 | |
sigmavirus | rosmaita: yeah, but once we have the inter-conscious link set-up, it won't be necessary :P | 14:01 |
rosmaita | sigmavirus: https://review.openstack.org/#/c/384655/ | 14:01 |
sigmavirus | oh yeah | 14:01 |
*** ducttape_ has joined #openstack-glance | 14:01 | |
sigmavirus | that's a good bug | 14:01 |
rosmaita | sigmavirus: not sure that would help, would be too noisy, at least on my end | 14:01 |
rosmaita | and an old bug | 14:01 |
Jokke_ | o/ | 14:01 |
rosmaita | hi erno | 14:01 |
sigmavirus | rosmaita: yeah, maybe we should wait for a "do not disturb" feature in that link :P | 14:01 |
croelandt | \o | 14:01 |
croelandt | hi | 14:02 |
* sigmavirus hides from Jokke_ | 14:02 | |
*** mfedosin has quit IRC | 14:02 | |
rosmaita | hello cyril | 14:02 |
rosmaita | gimme a sec to find the etherpad, want to make sure i start the meeting correctly | 14:02 |
croelandt | yep | 14:02 |
*** e0ne has quit IRC | 14:03 | |
rosmaita | #startmeeting image import sync | 14:03 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:03 |
openstack | The meeting name has been set to 'image_import_sync' | 14:03 |
rosmaita | #link https://etherpad.openstack.org/p/glance-ocata-image-import-sync | 14:03 |
rosmaita | ok, so we were going to refresh ourselves about taskflow | 14:04 |
rosmaita | (which i personally did not have time to do) | 14:04 |
Jokke_ | So the base_imprt flow is good starting point | 14:05 |
rosmaita | ok | 14:05 |
Jokke_ | due to it's architecture we cannot use it as is, but it's pretty good skeleton for what we want to do | 14:05 |
croelandt | sound optimistic | 14:06 |
rosmaita | for the people following along: | 14:06 |
*** e0ne has joined #openstack-glance | 14:06 | |
rosmaita | #link http://git.openstack.org/cgit/openstack/glance/tree/glance/async/flows/base_import.py | 14:06 |
rosmaita | i'm looking at all of flavio's notes about glance_store | 14:07 |
*** e0ne has quit IRC | 14:07 | |
rosmaita | that's making me think a design session on glance_store refactoring is a good idea for barcelona | 14: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 IRC | 14:07 | |
rosmaita | yea, flavio does not write boring code | 14:07 |
Jokke_ | rosmaita: that's very nice and polite of you :P | 14:08 |
rosmaita | i try | 14: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 there | 14:09 |
croelandt | I see | 14:10 |
croelandt | what endpoints would that cover exactly? | 14:10 |
*** anush_ has quit IRC | 14: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 store | 14:10 |
Jokke_ | croelandt: none | 14:10 |
Jokke_ | that would be just the taskflow to be used under the hood when the image import is done | 14:11 |
Jokke_ | with the new workflow | 14:11 |
croelandt | yeah | 14:12 |
croelandt | So the only issue with the inside-out approach is that I have no idea what we're trying to code | 14:12 |
rosmaita | so we'll need: user -> scratchpad, (validation while in scratchpad), scratchpad -> backend | 14:12 |
Jokke_ | croelandt: so the execution of that taskflow would be triggered by the last call of the image import process | 14:12 |
rosmaita | Jokke_ is talkind about scratchpad -> backend | 14:13 |
croelandt | ok | 14:13 |
rosmaita | croelandt: it might be worth thinking about how the validation should happen | 14: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 |
croelandt | rosmaita: isn't that part of the spec? | 14:14 |
rosmaita | croelandt: yes, but i think the spec says what more than how | 14:15 |
rosmaita | so the swift stuff doesn't need to happen in the MVP | 14:17 |
*** mfedosin has joined #openstack-glance | 14:17 | |
Jokke_ | rosmaita: but the design needs to be able to facilitate it | 14:17 |
croelandt | rosmaita: can't see the word "scratchpad" in either the spec nor the current code :p | 14:17 |
rosmaita | Jokke_: very true | 14:17 |
rosmaita | croelandt: 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 |
croelandt | oh right | 14:17 |
croelandt | not 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 |
rosmaita | croelandt: sorry about that | 14:18 |
rosmaita | croelandt: in the first few drafts, we called it "the bikeshed" because we couldn't agree on what to call it | 14:18 |
Jokke_ | croelandt: same here, so I constantly use wrong ones. Sorry | 14: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 :P | 14:19 |
rosmaita | well, i think i got overruled on having PUT v2/bikeshed in the api | 14:19 |
rosmaita | (that's why we call it the "staging area" -- PUT v2/stage) | 14:20 |
Jokke_ | unfortunately yes. Perhaps we should revive that | 14:20 |
rosmaita | it woudl be good for openstack to have a bikeshedding api | 14:20 |
rosmaita | would keep things more organized | 14: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/hardening | 14:23 |
Jokke_ | == profit | 14:23 |
rosmaita | so, 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 on | 14:24 |
rosmaita | croelandt: what parts interest you most? | 14:25 |
croelandt | not sure exactly where to start, but why not | 14:25 |
Jokke_ | croelandt: we can sync on that offline | 14:25 |
croelandt | rosmaita: 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 |
croelandt | I could give the user -> bikeshed a try | 14:26 |
rosmaita | Jokke_: can you use help on the bikeshed -> backend part? | 14:26 |
croelandt | if Jokke_ does the bikeshed -> store part, then we have the full stack \o/ | 14:26 |
rosmaita | yeah, 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 level | 14:26 |
Jokke_ | maybe that helps out to understand what we need there and what bits we are working | 14:27 |
rosmaita | i 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 backend | 14:28 |
Jokke_ | correct | 14:28 |
rosmaita | so for collaboration, | 14:28 |
croelandt | yep | 14:28 |
rosmaita | croelandt, you could either work on the same with Erno (if that would make sense) | 14:29 |
rosmaita | or work on another part that already assumes the bits are in the bikeshed | 14: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 profit | 14:29 |
rosmaita | not sure if it's too early for that, though | 14:29 |
croelandt | validation might be fun | 14:29 |
rosmaita | (i meant about validation stuff, not Jokke_ 's workflow) | 14:30 |
croelandt | rosmaita: is there anything in particular you'd like to work on? | 14:30 |
* abashmak pipes up | 14:30 | |
rosmaita | hello alex | 14:30 |
abashmak | I have bandwidth to help | 14:30 |
Jokke_ | there's actually good question about the validation | 14:30 |
rosmaita | croelandt: i want to work on the discovery part, i need to revise the schema | 14:30 |
*** anush_ has joined #openstack-glance | 14:31 | |
Jokke_ | do we want that just being tasks on the import flow or separate taskflow that then triggers the import flow | 14:31 |
rosmaita | Jokke_: not sure i understand your question | 14:31 |
rosmaita | abashmak: have you had a chance to work with taskflow yet? | 14:32 |
abashmak | sadly no | 14:32 |
rosmaita | that's ok | 14:32 |
rosmaita | my suggestion would be to look at taskflow and the stuff in the glance/async directory to see how it's being used | 14:32 |
croelandt | rosmaita: oh I have a patch for that | 14:32 |
rosmaita | croelandt: yes, i know | 14:33 |
rosmaita | croelandt: 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 discussions | 14:33 |
croelandt | rosmaita: feel free to ping me about that | 14:33 |
rosmaita | croelandt: i do have a question about whether our current json-schema code will be able to handle the proposed schema | 14:34 |
Jokke_ | sorry, had to dial in to the next meeting | 14:34 |
rosmaita | Jokke_: ok | 14:34 |
Jokke_ | so as the validation is precondition to the import | 14: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 call | 14:36 |
rosmaita | the key thing is that we'll want operators to be able to insert custom validation code | 14:37 |
rosmaita | i'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 it | 14:37 |
rosmaita | maybe (1) is because the validation part is more isolated? | 14:37 |
rosmaita | ok, so we are thinking the same thing | 14:38 |
Jokke_ | it would keep the import code away from the people who want to implement their own validations | 14:38 |
rosmaita | yeah, that would be good | 14: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 finished | 14:39 |
rosmaita | i like that | 14:39 |
rosmaita | it keeps the database interaction stuff away from the operators | 14:40 |
rosmaita | we handle all the stuff that modifies the image record | 14:40 |
* croelandt has a headache now | 14:40 | |
rosmaita | we are way over time | 14:40 |
Jokke_ | no worries | 14:40 |
abashmak | another 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 that | 14:40 |
Jokke_ | croelandt: I'll do the flowchart and walk you through it | 14:41 |
rosmaita | abashmak: that's a good idea, the less we leave to operators, the more friendly the import will be | 14:41 |
*** sdake_ is now known as sdake | 14:42 | |
Jokke_ | abashmak: fell free to look into how to do that wrapper with the glance async/ taskflow code :P | 14:42 |
croelandt | ok | 14:42 |
croelandt | might be nice to sum this up in the etherpad | 14:42 |
*** ctrath has quit IRC | 14:42 | |
rosmaita | ok, got some notes in the etherpad | 14:45 |
rosmaita | ok what's next? | 14:46 |
*** ctrath has joined #openstack-glance | 14:46 | |
rosmaita | i guess we have enough for a week? | 14:46 |
Jokke_ | yeah, maybe more | 14:46 |
rosmaita | ok, let's plan to meet again next week | 14:47 |
rosmaita | and we can find each other here if we need to | 14:47 |
rosmaita | thanks, and sorry about this going way over 20 min | 14:48 |
rosmaita | #endmeeting image import sync | 14:48 |
croelandt | np | 14:48 |
openstack | Meeting ended Tue Oct 11 14:48:11 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:48 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/image_import_sync/2016/image_import_sync.2016-10-11-14.03.html | 14:48 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/image_import_sync/2016/image_import_sync.2016-10-11-14.03.txt | 14:48 |
openstack | Log: http://eavesdrop.openstack.org/meetings/image_import_sync/2016/image_import_sync.2016-10-11-14.03.log.html | 14:48 |
croelandt | Same time next week? | 14:48 |
rosmaita | croelandt: yes ... does that work for you? | 14:48 |
croelandt | ok | 14:50 |
*** mfedosin has quit IRC | 14:52 | |
rosmaita | croelandt: ok, great, i'd like us to keep 14:00-14:20 utc open on tuesdays throughout the cycle | 14:54 |
croelandt | works for me | 14:54 |
rosmaita | ok, cool | 14:54 |
Jokke_ | thanks rosmaita | 14:55 |
rosmaita | Jokke_: thank you! | 14:56 |
*** mfedosin has joined #openstack-glance | 15:04 | |
*** btully has quit IRC | 15:09 | |
*** btully has joined #openstack-glance | 15:10 | |
*** ccesario has quit IRC | 15:17 | |
*** jistr is now known as jistr|biab | 15:25 | |
*** _ducttape_ has joined #openstack-glance | 15:25 | |
*** ducttape_ has quit IRC | 15:29 | |
*** ccesario has joined #openstack-glance | 15:31 | |
*** pcaruana has quit IRC | 15:31 | |
openstackgerrit | Dharini Chandrasekar proposed openstack/glance: Handling scrubber's exit in non-daemon mode. https://review.openstack.org/382711 | 15:35 |
*** ctrath has quit IRC | 15:38 | |
dharinic | rosmaita, stevelle: regarding this- https://review.openstack.org/#/c/375123/ | 15:39 |
*** ctrath has joined #openstack-glance | 15:39 | |
dharinic | stevelle suggested deprecation header for the use of Content-Range. I wasnt sure how deprecations worked when it involved APIs | 15:42 |
rosmaita | looking | 15:42 |
stevelle | I don't think anyone will see the header if it was added | 15:43 |
stevelle | I also don't think anyone will notice the log message | 15:43 |
stevelle | and I have doubts that operators will read the release notes, but I'm not an optimist :) | 15:43 |
*** atod has joined #openstack-glance | 15:43 | |
rosmaita | hmmm ... we pretty much assume that operators read the release notes | 15:44 |
rosmaita | may need to revise that assumption! | 15:44 |
stevelle | it's a best-effort thing | 15:44 |
stevelle | also operators are not necessarily the audience you need to reach for API changes | 15:44 |
stevelle | users of the cloud may need to know, though I'm not sure why they are reading segments of images from glance in that case | 15:45 |
stevelle | at the least I would want to log use of the deprecated header in requests | 15:46 |
dharinic | I am not sure how in this case we can retain support for both Content-range and range | 15:46 |
dharinic | while deprecating content-range | 15:46 |
dharinic | it might make things a little messy on wsgi.py | 15:46 |
dharinic | This 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 directly | 15:47 |
Jokke_ | ;) | 15:47 |
dharinic | can you guys tell me how we can go about this? | 15:48 |
*** korzen has joined #openstack-glance | 15:48 | |
dharinic | The related patch: https://review.openstack.org/#/c/367528/ | 15:48 |
stevelle | I 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 period | 15:49 |
dharinic | cool. | 15:50 |
stevelle | Jokke_: you're probably right that nobody really uses range requests -- particularly because we don't recognize the right header :P | 15:50 |
dharinic | Just wanted to clarify if that was okay especially cos it was an API related change | 15:50 |
stevelle | but it's a hard breaking change | 15:50 |
rosmaita | dharinic: what w3c doc are you relying on to say content-range is illegal on requests? | 15:50 |
dharinic | Let me pull it up | 15:51 |
rosmaita | ty | 15:51 |
rosmaita | i am looking at one now, but it's pretty ambiguous | 15:51 |
dharinic | https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html | 15: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 behavior | 15:51 |
dharinic | Jokke_, True | 15:51 |
*** e0ne has joined #openstack-glance | 15:51 | |
stevelle | RFC 7233 is the right one to look at btw, as it updated 2616 | 15:52 |
dharinic | rosmaita: https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html | 15:52 |
rosmaita | dharinic: ty | 15:52 |
stevelle | https://tools.ietf.org/html/rfc7233 | 15:52 |
rosmaita | stevelle: ty | 15:54 |
*** links has quit IRC | 15:54 | |
openstackgerrit | Dharini Chandrasekar proposed openstack/glance: Handling scrubber's exit in non-daemon mode. https://review.openstack.org/382711 | 15: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 it | 15:58 |
*** tsymanczyk has joined #openstack-glance | 15:58 | |
stevelle | so parsing either but not just range, right Jokke_? | 15:59 |
dharinic | rosmaita: https://tools.ietf.org/html/rfc7233#section-4.2 | 15: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 rfcs | 15:59 |
Jokke_ | stevelle: correct | 15:59 |
dharinic | This was what nikhil referred to, when he commented on the patch suggesting we be using Range | 16:00 |
rosmaita | dharinic: you need to add a ref to rfc 7233 in your spec-lite | 16:00 |
rosmaita | also, looks like the blanket statement "content-range not appropriate for requests" is incorrect | 16:01 |
dharinic | Okay. Sure. | 16:01 |
rosmaita | incorrect for GET | 16:01 |
rosmaita | but may be legal for PUT from what i can tell | 16:01 |
stevelle | but makes no sense | 16:01 |
rosmaita | which doesnt' matter | 16:01 |
dharinic | Yes. Thats right. | 16:01 |
rosmaita | no, you could replace the middle bytes of some object | 16:01 |
stevelle | that would better be as a PATCH though | 16:02 |
Jokke_ | rosmaita: we definitely do not support partial uploads ;) | 16:02 |
dharinic | :D | 16:02 |
rosmaita | Jokke_: agreed | 16:02 |
stevelle | I'm splitting hairs though | 16:02 |
rosmaita | me too | 16:02 |
rosmaita | key thing is just to describe clearly the scope of this change | 16:03 |
Jokke_ | ++ | 16:03 |
rosmaita | becuase these docs are a real pain to wade through | 16:03 |
Jokke_ | and not break backwards compatibility | 16:03 |
rosmaita | i've only been looking for 8 min and already have a headache | 16:03 |
stevelle | XD | 16:03 |
dharinic | :D | 16:03 |
Jokke_ | regardless if it happens to be against rfc but work or not | 16:03 |
rosmaita | yes, Jokke_ is correct, we must preserve backwards compat | 16:03 |
rosmaita | i am thinking that we accept content-range as a pseudonym of range on GETs | 16:04 |
rosmaita | but don't tell anyone | 16:04 |
rosmaita | we only doc range | 16:04 |
stevelle | +1 leave Content-Range undocumented | 16:04 |
dharinic | So, is it alright that we dont fix partial downloads cos we are waiting on the spec for using "Range" | 16:04 |
rosmaita | how big a maintenance problem is that? | 16:04 |
stevelle | state deprecation though | 16:04 |
Jokke_ | ++ | 16:04 |
Jokke_ | stevelle: we don't need to even state it deprecated as we will never be removing it anyways | 16:04 |
dharinic | or do you guys siuggest that I fix it (latest patch on https://review.openstack.org/#/c/367528/6) with usage "Content-range" | 16:05 |
stevelle | Jokke_: never? ok then | 16:05 |
stevelle | I'm fine with that | 16:05 |
Jokke_ | stevelle: that's what the community wants. You can deprecate, you cannot remove | 16:05 |
Jokke_ | so in this point why bother even deprecate | 16:05 |
rosmaita | yeah, we are just being a bit liberal in what we accept | 16:06 |
stevelle | Jokke_: I'm not entirely sure about the microversion thing, and not a fan in general, but what you said makes perfect sense | 16:06 |
Jokke_ | stevelle: Glance does not use microversions either | 16:06 |
Jokke_ | stevelle: and even with microversions you still need to maintain/provide the previous ones so that won't help functionality wise | 16:07 |
stevelle | dharinic: 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-Range | 16:07 |
*** jistr|biab is now known as jistr | 16:07 | |
stevelle | Jokke_: that's where I was unclear, because the whole idea of minor api version #s bothers me I never investigated | 16:08 |
dharinic | Thanks stevelle | 16:08 |
*** atod has quit IRC | 16:08 | |
dharinic | Thanks rosmaita and Jokke_ :) | 16:09 |
stevelle | dharinic: your endurance through all the indecision on this is appreciated :) | 16:09 |
*** _ducttape_ has quit IRC | 16:09 | |
rosmaita | ++ to what stevelle said | 16: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 API | 16:09 |
dharinic | Will edit the lite-spec and resubmit. After which I will go ahead and add Range functionality to 367528 | 16:09 |
*** ducttape_ has joined #openstack-glance | 16:10 | |
dharinic | My pleasure :) | 16:10 |
*** TravT has quit IRC | 16:10 | |
Jokke_ | thanks dharinic | 16:10 |
rosmaita | dharinic: sounds good, thank you! | 16:10 |
dharinic | One last thing: This patch paved way for https://review.openstack.org/#/c/380674/ | 16:10 |
*** mfedosin has quit IRC | 16:10 | |
dharinic | Reviews welcome. Will add dependency as abashmak suggested. :) | 16:11 |
*** TravT has joined #openstack-glance | 16:11 | |
*** TravT has quit IRC | 16:14 | |
*** TravT has joined #openstack-glance | 16:16 | |
*** hemanth|away is now known as hemanth | 16:16 | |
*** _ducttape_ has joined #openstack-glance | 16:21 | |
openstackgerrit | Timothy Symanczyk proposed openstack/glance: WIP Utilize new visibility values for images https://review.openstack.org/379852 | 16:23 |
tsymanczyk | all py34 tests should now pass. | 16:24 |
*** ducttape_ has quit IRC | 16:24 | |
*** groen692 has quit IRC | 16:24 | |
*** hoonetorg has quit IRC | 16:35 | |
*** groen692 has joined #openstack-glance | 16:46 | |
*** hoonetorg has joined #openstack-glance | 16:48 | |
openstackgerrit | Dharini Chandrasekar proposed openstack/glance: Handling scrubber's exit in non-daemon mode. https://review.openstack.org/382711 | 16:54 |
*** groen692 has quit IRC | 17:01 | |
*** mtanino has joined #openstack-glance | 17:16 | |
stevelle | would 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 IRC | 17:49 | |
*** _ducttape_ has quit IRC | 17:51 | |
*** ducttape_ has joined #openstack-glance | 17:51 | |
*** dshakhray has quit IRC | 17:52 | |
*** ducttape_ has quit IRC | 17:52 | |
*** ducttape_ has joined #openstack-glance | 17:52 | |
*** atod has joined #openstack-glance | 17:53 | |
*** atod has quit IRC | 17:58 | |
*** mvk has quit IRC | 18:04 | |
*** korzen has quit IRC | 18:07 | |
*** jamielennox|away is now known as jamielennox | 18:10 | |
*** ducttape_ has quit IRC | 18:21 | |
*** ducttape_ has joined #openstack-glance | 18:21 | |
*** ducttape_ has quit IRC | 18:25 | |
*** dshakhray has joined #openstack-glance | 18:26 | |
*** ctrath has quit IRC | 18:28 | |
*** ctrath has joined #openstack-glance | 18:29 | |
*** hoonetorg has quit IRC | 18:35 | |
*** ducttape_ has joined #openstack-glance | 18:43 | |
*** hoonetorg has joined #openstack-glance | 18:51 | |
*** TravT has joined #openstack-glance | 18:58 | |
*** TravT has quit IRC | 19:01 | |
rosmaita | stevelle: agree about release note for https://review.openstack.org/#/c/384655/ , please put a comment on the patch | 19:02 |
*** TravT has joined #openstack-glance | 19:02 | |
stevelle | will do | 19:02 |
*** ducttape_ has quit IRC | 19:13 | |
*** ctrath has quit IRC | 19:30 | |
openstackgerrit | Dharini Chandrasekar proposed openstack/glance-specs: Use ``Range`` header in HTTP requests for partial downloads. https://review.openstack.org/375123 | 19:30 |
*** ctrath has joined #openstack-glance | 19:34 | |
*** e0ne has quit IRC | 19:37 | |
*** e0ne has joined #openstack-glance | 19:41 | |
*** atod has joined #openstack-glance | 19:42 | |
openstackgerrit | boden proposed openstack/glance: Replace retrying with tenacity https://review.openstack.org/367614 | 19:44 |
*** atod has quit IRC | 19:46 | |
*** e0ne has quit IRC | 19:50 | |
*** e0ne has joined #openstack-glance | 19:55 | |
*** e0ne has quit IRC | 19:58 | |
*** e0ne has joined #openstack-glance | 20:02 | |
*** exploreshaifali has joined #openstack-glance | 20:02 | |
*** burgerk__ has joined #openstack-glance | 20:30 | |
*** burgerk_ has quit IRC | 20:34 | |
*** dshakhray has quit IRC | 20:37 | |
*** dshakhray has joined #openstack-glance | 20:38 | |
*** hemanth is now known as hemanth|bbl | 20:38 | |
*** sdake_ has joined #openstack-glance | 20:39 | |
*** sdake has quit IRC | 20:41 | |
*** burgerk__ has quit IRC | 20:48 | |
*** sdake has joined #openstack-glance | 20:51 | |
*** sdake_ has quit IRC | 20:53 | |
*** ducttape_ has joined #openstack-glance | 20:54 | |
*** dshakhray has quit IRC | 20:55 | |
*** anush_ has quit IRC | 21:08 | |
*** exploreshaifali has quit IRC | 21:09 | |
*** ducttape_ has quit IRC | 21:16 | |
*** btully has quit IRC | 21:19 | |
*** bpokorny has joined #openstack-glance | 21:20 | |
*** mvk has joined #openstack-glance | 21:22 | |
*** ducttape_ has joined #openstack-glance | 21:32 | |
*** dshakhray has joined #openstack-glance | 21:36 | |
*** dshakhray has quit IRC | 21:41 | |
*** tsymanczyk has quit IRC | 21:44 | |
*** atod has joined #openstack-glance | 21:45 | |
*** e0ne has quit IRC | 21:48 | |
*** e0ne has joined #openstack-glance | 21:49 | |
*** e0ne has quit IRC | 21:50 | |
*** e0ne has joined #openstack-glance | 21:50 | |
*** e0ne has quit IRC | 21:50 | |
*** e0ne has joined #openstack-glance | 21:51 | |
*** e0ne has quit IRC | 21:51 | |
*** e0ne has joined #openstack-glance | 21:52 | |
*** e0ne has quit IRC | 21:52 | |
*** e0ne has joined #openstack-glance | 21:52 | |
*** e0ne has quit IRC | 21:53 | |
*** e0ne has joined #openstack-glance | 21:53 | |
*** e0ne has quit IRC | 21:54 | |
*** e0ne has joined #openstack-glance | 21:54 | |
*** e0ne has quit IRC | 21:54 | |
*** zigo has quit IRC | 22:02 | |
*** zigo has joined #openstack-glance | 22:05 | |
*** zigo is now known as Guest50630 | 22:05 | |
*** ctrath has quit IRC | 22:28 | |
*** ducttape_ has quit IRC | 22:36 | |
*** TravT_ has joined #openstack-glance | 22:52 | |
*** TravT has quit IRC | 22:54 | |
*** TravT_ has quit IRC | 22:57 | |
*** btully has joined #openstack-glance | 23:04 | |
openstackgerrit | Dharini Chandrasekar proposed openstack/glance-specs: Use ``Range`` header in HTTP requests for partial downloads. https://review.openstack.org/375123 | 23:10 |
*** sdake has quit IRC | 23:16 | |
*** btully has quit IRC | 23:17 | |
*** btully has joined #openstack-glance | 23:18 | |
*** edmondsw has quit IRC | 23:30 | |
*** david-lyle has joined #openstack-glance | 23:31 | |
*** btully has quit IRC | 23:31 | |
*** david-lyle has quit IRC | 23:34 | |
*** david-lyle has joined #openstack-glance | 23:35 | |
*** dshakhray has joined #openstack-glance | 23:35 | |
*** david-lyle has quit IRC | 23:38 | |
*** david-lyle has joined #openstack-glance | 23:39 | |
*** david-lyle has quit IRC | 23:49 | |
*** sdake has joined #openstack-glance | 23:50 | |
*** sdake_ has joined #openstack-glance | 23:51 | |
*** sdake has quit IRC | 23:55 | |
*** ducttape_ has joined #openstack-glance | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!