| *** dims has quit IRC | 00:04 | |
| *** shakamunyi has joined #openstack-glance | 00:10 | |
| *** chlong has quit IRC | 00:12 | |
| *** ducttape_ has quit IRC | 00:15 | |
| *** ninag has joined #openstack-glance | 00:26 | |
| *** RichardRaseley has joined #openstack-glance | 00:31 | |
| *** dims has joined #openstack-glance | 00:33 | |
| RichardRaseley | Does this (http://paste.openstack.org/show/480714/) seem like the same issue reported here (https://bugs.launchpad.net/glance/+bug/1476770) ? | 00:36 |
|---|---|---|
| openstack | Launchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [High,In progress] - Assigned to Flavio Percoco (flaper87) | 00:36 |
| RichardRaseley | I get that first error in the nova-api logs whenever I try to launch an instance with an image from glance or try to access the image details. | 00:37 |
| *** rcernin has quit IRC | 00:37 | |
| RichardRaseley | Though, I can successfully do an `openstack image list` and `openstack image show <image_id>` from the CLI and the later returns the ID property of the image without issue. | 00:38 |
| *** e0ne has joined #openstack-glance | 00:41 | |
| *** kebray has quit IRC | 00:42 | |
| *** ninag has quit IRC | 00:44 | |
| *** ninag has joined #openstack-glance | 00:44 | |
| *** e0ne has quit IRC | 00:45 | |
| *** ninag has quit IRC | 00:49 | |
| *** mingdang1 has joined #openstack-glance | 00:50 | |
| *** mingdang1 has quit IRC | 00:50 | |
| *** mingdang1 has joined #openstack-glance | 00:50 | |
| *** RichardRaseley has quit IRC | 01:01 | |
| *** shakamunyi has quit IRC | 01:02 | |
| *** shakamunyi has joined #openstack-glance | 01:03 | |
| *** flaper87 has left #openstack-glance | 01:15 | |
| *** flaper87 has joined #openstack-glance | 01:15 | |
| *** sgotliv__ has quit IRC | 01:22 | |
| openstackgerrit | Merged openstack/glance: Move store config opt to glance_store section https://review.openstack.org/252551 | 01:22 |
| *** dims has quit IRC | 01:30 | |
| *** mancdaz has quit IRC | 01:39 | |
| *** mingdang1 has quit IRC | 01:39 | |
| *** mancdaz has joined #openstack-glance | 01:40 | |
| *** mtanino has quit IRC | 01:47 | |
| *** chlong has joined #openstack-glance | 01:48 | |
| *** chlong has quit IRC | 01:51 | |
| *** dims has joined #openstack-glance | 02:00 | |
| *** chlong has joined #openstack-glance | 02:00 | |
| *** chlong has quit IRC | 02:06 | |
| *** ninag has joined #openstack-glance | 02:16 | |
| *** chlong has joined #openstack-glance | 02:20 | |
| *** ninag has quit IRC | 02:27 | |
| *** ninag has joined #openstack-glance | 02:28 | |
| *** ninag has quit IRC | 02:28 | |
| *** ninag has joined #openstack-glance | 02:29 | |
| *** ninag has quit IRC | 02:33 | |
| *** chlong has quit IRC | 02:35 | |
| *** mingdang1 has joined #openstack-glance | 02:40 | |
| *** mingdang1 has quit IRC | 02:45 | |
| *** chlong has joined #openstack-glance | 02:49 | |
| *** houming has joined #openstack-glance | 02:59 | |
| *** ayoung has joined #openstack-glance | 03:01 | |
| *** ayoung has quit IRC | 03:01 | |
| *** ayoung has joined #openstack-glance | 03:03 | |
| *** e0ne has joined #openstack-glance | 03:06 | |
| *** jerrygb has quit IRC | 03:19 | |
| *** chlong has quit IRC | 03:21 | |
| *** ducttape_ has joined #openstack-glance | 03:29 | |
| *** azbiswas has quit IRC | 03:35 | |
| *** e0ne has quit IRC | 03:35 | |
| *** azbiswas has joined #openstack-glance | 03:42 | |
| *** ducttape_ has quit IRC | 04:12 | |
| *** jerrygb has joined #openstack-glance | 04:20 | |
| *** spzala has quit IRC | 04:23 | |
| *** jerrygb has quit IRC | 04:26 | |
| *** mdnadeem has left #openstack-glance | 04:38 | |
| *** dims has quit IRC | 04:52 | |
| *** GB21 has joined #openstack-glance | 04:52 | |
| *** links has joined #openstack-glance | 05:06 | |
| *** e0ne has joined #openstack-glance | 05:08 | |
| openstackgerrit | Merged openstack/glance: Add the Docker container format https://review.openstack.org/249282 | 05:10 |
| *** changbl has quit IRC | 05:13 | |
| *** e0ne has quit IRC | 05:14 | |
| *** ianbrown has joined #openstack-glance | 05:14 | |
| openstackgerrit | Merged openstack/glance: Remove version from setup.cfg https://review.openstack.org/252021 | 05:15 |
| *** e0ne has joined #openstack-glance | 05:16 | |
| *** haomaiwang has joined #openstack-glance | 05:16 | |
| *** ianbrown has quit IRC | 05:21 | |
| *** GB21 has quit IRC | 05:29 | |
| *** changbl has joined #openstack-glance | 05:30 | |
| *** azbiswas has quit IRC | 05:40 | |
| *** TravT has joined #openstack-glance | 05:42 | |
| *** e0ne has quit IRC | 06:11 | |
| *** melwitt has quit IRC | 06:19 | |
| *** melwitt has joined #openstack-glance | 06:22 | |
| *** jerrygb has joined #openstack-glance | 06:22 | |
| *** e0ne has joined #openstack-glance | 06:24 | |
| *** e0ne has quit IRC | 06:24 | |
| *** jerrygb has quit IRC | 06:26 | |
| *** e0ne has joined #openstack-glance | 06:29 | |
| *** e0ne has quit IRC | 06:35 | |
| *** mingdang1 has joined #openstack-glance | 06:36 | |
| *** e0ne has joined #openstack-glance | 06:43 | |
| *** e0ne has quit IRC | 06:45 | |
| *** GB21 has joined #openstack-glance | 06:53 | |
| *** azbiswas has joined #openstack-glance | 06:57 | |
| *** e0ne has joined #openstack-glance | 07:02 | |
| *** e0ne has quit IRC | 07:06 | |
| *** teju has joined #openstack-glance | 07:08 | |
| *** bpoulos has quit IRC | 07:09 | |
| *** sgotliv__ has joined #openstack-glance | 07:16 | |
| *** dshakhray has joined #openstack-glance | 07:17 | |
| *** rcernin has joined #openstack-glance | 07:19 | |
| *** rcernin_ has joined #openstack-glance | 07:24 | |
| *** e0ne has joined #openstack-glance | 07:35 | |
| *** e0ne has quit IRC | 07:37 | |
| openstackgerrit | Brian Rosmaita proposed openstack/glance-specs: Image Import Refactor https://review.openstack.org/232371 | 07:38 |
| *** flaper87 has quit IRC | 07:44 | |
| *** flaper87 has joined #openstack-glance | 07:49 | |
| *** chlong has joined #openstack-glance | 07:55 | |
| openstackgerrit | Atsushi SAKAI proposed openstack/python-glanceclient: Add docker to image_schema on glance v2 cli https://review.openstack.org/252806 | 07:55 |
| *** wxy_ has joined #openstack-glance | 07:59 | |
| *** wxy__ has quit IRC | 08:02 | |
| *** e0ne has joined #openstack-glance | 08:03 | |
| *** belmoreira has joined #openstack-glance | 08:04 | |
| *** markus_z has joined #openstack-glance | 08:16 | |
| *** e0ne has quit IRC | 08:24 | |
| openstackgerrit | Merged openstack/python-glanceclient: Fix help for image-create https://review.openstack.org/251436 | 08:24 |
| *** MattMan has quit IRC | 08:24 | |
| *** MattMan has joined #openstack-glance | 08:25 | |
| *** Guest66073 has joined #openstack-glance | 08:26 | |
| *** chlong has quit IRC | 08:29 | |
| *** teju has quit IRC | 08:34 | |
| *** azbiswas has quit IRC | 08:34 | |
| *** GB21 has quit IRC | 08:43 | |
| *** rcernin has quit IRC | 08:50 | |
| *** rcernin_ has quit IRC | 08:51 | |
| *** GB21 has joined #openstack-glance | 09:02 | |
| *** belmoreira has quit IRC | 09:04 | |
| *** aix has joined #openstack-glance | 09:29 | |
| openstackgerrit | Kairat Kushaev proposed openstack/python-glanceclient: Support docker images in client https://review.openstack.org/252843 | 09:39 |
| *** houming has quit IRC | 09:49 | |
| *** rcernin has joined #openstack-glance | 09:59 | |
| openstackgerrit | Kairat Kushaev proposed openstack/python-glanceclient: Run py34 env first when launching tests https://review.openstack.org/252857 | 10:01 |
| *** osho_ has joined #openstack-glance | 10:02 | |
| openstackgerrit | Kairat Kushaev proposed openstack/glance: Run py34 env first when launching tests https://review.openstack.org/252858 | 10:04 |
| *** marekd has quit IRC | 10:05 | |
| *** jistr has joined #openstack-glance | 10:09 | |
| *** marekd has joined #openstack-glance | 10:13 | |
| *** marekd has quit IRC | 10:15 | |
| *** marekd has joined #openstack-glance | 10:16 | |
| openstackgerrit | zhangsong proposed openstack/glance_store: Sheepdog: Change storelocation format https://review.openstack.org/251211 | 10:20 |
| *** tsufiev has quit IRC | 10:22 | |
| *** osho_ has quit IRC | 10:23 | |
| *** tsufiev has joined #openstack-glance | 10:29 | |
| *** ativelkov has quit IRC | 10:29 | |
| openstackgerrit | Darja Shakhray proposed openstack/glance: Fix Glance doesn't catches UnicodeDecodeError exception https://review.openstack.org/252867 | 10:31 |
| *** ativelkov has joined #openstack-glance | 10:32 | |
| *** ninag has joined #openstack-glance | 10:33 | |
| *** e0ne has joined #openstack-glance | 10:33 | |
| *** e0ne has quit IRC | 10:36 | |
| *** ninag has quit IRC | 10:37 | |
| *** belmoreira has joined #openstack-glance | 10:38 | |
| *** haomaiwang has quit IRC | 10:38 | |
| *** Guest66073 has quit IRC | 10:39 | |
| *** venkatesh_ has joined #openstack-glance | 11:01 | |
| *** esheffield_ has joined #openstack-glance | 11:01 | |
| *** chlong has joined #openstack-glance | 11:02 | |
| *** wxy_ has quit IRC | 11:02 | |
| *** rosmaita_ has joined #openstack-glance | 11:03 | |
| *** wxy has joined #openstack-glance | 11:03 | |
| *** dims_ has joined #openstack-glance | 11:03 | |
| *** venkatesh has quit IRC | 11:03 | |
| *** esheffield has quit IRC | 11:03 | |
| *** rosmaita has quit IRC | 11:03 | |
| *** aix has quit IRC | 11:18 | |
| *** mclaren has quit IRC | 11:34 | |
| kairat | flaper87, ping | 11:40 |
| kairat | flaper87, I have a question. If I assigned bug to myself but someone applied the patch for this bug before me (without referencing to the bug) | 11:41 |
| kairat | flaper87, that's more proper, abandon my patch I assign the bug to the person who applied the patch | 11:42 |
| kairat | *what is | 11:42 |
| bunting | Thats what i would probably do | 11:43 |
| *** houming has joined #openstack-glance | 11:43 | |
| kairat | I think the same also | 11:43 |
| kairat | Just would like to know because I may be in the same situation if I apply the patch before other person=) | 11:45 |
| bunting | Yeah probably comment mentioning the bug number | 11:46 |
| *** houming has quit IRC | 11:48 | |
| *** houming has joined #openstack-glance | 11:51 | |
| bunting | kairat: ping | 11:57 |
| kairat | bunting, pong | 11:58 |
| bunting | I'm just trying to get my head around your 4th comment on https://review.openstack.org/#/c/238074/5 | 11:59 |
| kairat | yep | 11:59 |
| bunting | just a bit confused what you mean | 11:59 |
| kairat | I am wondering why you need to parse keystone auth address | 12:00 |
| kairat | If you have auth_version in configs | 12:00 |
| kairat | I am not sure that it is right decision | 12:00 |
| bunting | https://bugs.launchpad.net/glance-store/+bug/1507610 | 12:00 |
| openstack | Launchpad bug 1507610 in glance_store "Keystone v3 incompatable with keystone v2" [Undecided,In progress] - Assigned to Niall Bunting (niall-bunting) | 12:00 |
| bunting | If they have the old type and change it to keystone v3 | 12:00 |
| bunting | it won't work | 12:01 |
| bunting | by old type i mean the url stored in the database | 12:01 |
| kairat | have you updated swift_store_auth_version? | 12:02 |
| kairat | to 3? | 12:02 |
| kairat | although it won't work | 12:02 |
| kairat | Because it is hard-coded:) | 12:02 |
| bunting | in the configs? | 12:02 |
| kairat | But why do we need to parse keystone auth address, why can't we initialize the version with swift_store_auth_version? | 12:03 |
| kairat | yep | 12:03 |
| kairat | in glance-api.conf or conig reference file in case of swift+config | 12:03 |
| bunting | Because if the database contains a url that is something like keystone:5000/v2 if you try to use auth version 3 it wont work | 12:04 |
| bunting | due to keystone url changing from /v2.0/tokens to /v3/auth/tokens | 12:05 |
| *** houming has quit IRC | 12:08 | |
| *** houming has joined #openstack-glance | 12:08 | |
| *** houming has quit IRC | 12:09 | |
| *** houming has joined #openstack-glance | 12:09 | |
| kairat | Just a second | 12:09 |
| kairat | bunting, I have a meeting, will be available in 1 hour | 12:10 |
| kairat | I have several questions about your case then:) | 12:10 |
| kairat | First question, should we update swift_url address if config and version in keystone url is different | 12:11 |
| kairat | instead of parsing version from keystone url | 12:12 |
| kairat | bunting, ^ | 12:12 |
| bunting | Hmmn but what happens if this uses a diffrent keystone? from the one in the configs? | 12:16 |
| *** flaper87 has quit IRC | 12:17 | |
| *** flaper87 has joined #openstack-glance | 12:17 | |
| kairat | So if you updated config then glance_store need to use the new version | 12:18 |
| kairat | IMO | 12:18 |
| flaper87 | kairat: yeah, that's what I would do | 12:19 |
| kairat | flaper87, thanks | 12:19 |
| bunting | So we should just ignore the url in the database? | 12:20 |
| kairat | That's the question | 12:20 |
| kairat | I don't mean url but keystone version in url | 12:20 |
| bunting | Because we currently use it, that leads to this problem | 12:20 |
| kairat | it sounds like a question for meeting | 12:21 |
| kairat | or letter to openstack-dev=) | 12:21 |
| kairat | I am in favor of this | 12:21 |
| kairat | But there could be other opinions | 12:21 |
| bunting | Yeah! I'll put it on the agenda for now | 12:22 |
| *** jaypipes has joined #openstack-glance | 12:25 | |
| *** GB21 has quit IRC | 12:29 | |
| *** openstackgerrit has quit IRC | 12:32 | |
| *** openstackgerrit has joined #openstack-glance | 12:32 | |
| *** aix has joined #openstack-glance | 12:39 | |
| *** haomaiwa_ has joined #openstack-glance | 12:43 | |
| *** ducttape_ has joined #openstack-glance | 12:47 | |
| *** ninag has joined #openstack-glance | 12:59 | |
| *** ninag has quit IRC | 12:59 | |
| *** ninag has joined #openstack-glance | 13:00 | |
| *** ninag has quit IRC | 13:04 | |
| *** ducttape_ has quit IRC | 13:14 | |
| *** openstackgerrit has quit IRC | 13:17 | |
| *** openstackgerrit has joined #openstack-glance | 13:17 | |
| *** smatzek has joined #openstack-glance | 13:18 | |
| *** jerrygb has joined #openstack-glance | 13:21 | |
| *** ninag has joined #openstack-glance | 13:21 | |
| *** ninag_ has joined #openstack-glance | 13:22 | |
| *** spzala has joined #openstack-glance | 13:23 | |
| *** bdossant has joined #openstack-glance | 13:25 | |
| *** ninag has quit IRC | 13:26 | |
| *** ctina_ has joined #openstack-glance | 13:32 | |
| openstackgerrit | Merged openstack/python-glanceclient: Fix tests for image-create https://review.openstack.org/251183 | 13:34 |
| openstackgerrit | Niall Bunting proposed openstack/glance_store: Make glance_store use the correct keystone version https://review.openstack.org/238074 | 13:39 |
| *** haomaiwa_ has quit IRC | 13:40 | |
| *** bpoulos has joined #openstack-glance | 13:45 | |
| *** links has quit IRC | 13:51 | |
| *** e0ne has joined #openstack-glance | 14:04 | |
| *** e0ne has quit IRC | 14:05 | |
| *** julim has joined #openstack-glance | 14:06 | |
| *** e0ne has joined #openstack-glance | 14:06 | |
| *** rosmaita_ is now known as rosmaita | 14:07 | |
| *** RichardRaseley has joined #openstack-glance | 14:09 | |
| *** edmondsw has joined #openstack-glance | 14:11 | |
| *** takedakn has joined #openstack-glance | 14:15 | |
| *** takedakn has quit IRC | 14:20 | |
| *** mingdang1 has quit IRC | 14:20 | |
| kairat | krotscheck, can I use wildcards or reg-exp for list of headers exposed/allowed? | 14:21 |
| RichardRaseley | Hello Glance team. Quick question to help me in orienting my troubleshooting. Does this ( http://paste.openstack.org/show/480714/ ) error I am experiencing seem like a manifestation of this ( https://bugs.launchpad.net/glance/+bug/1476770 ) already reported bug? Running OpenStack Kilo from RDO packaging. Looking at PIP I have requests (2.8.1) and urllib3 (1.12) installed. | 14:23 |
| openstack | Launchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [High,In progress] - Assigned to Flavio Percoco (flaper87) | 14:23 |
| *** bdossant_ has joined #openstack-glance | 14:30 | |
| *** bdossant has quit IRC | 14:31 | |
| nikhil | RichardRaseley: apparently so (to me) | 14:33 |
| *** e0ne_ has joined #openstack-glance | 14:34 | |
| *** houming has quit IRC | 14:34 | |
| RichardRaseley | nikhil : Thank you. | 14:34 |
| *** smatzek has quit IRC | 14:35 | |
| *** e0ne has quit IRC | 14:36 | |
| *** wxy__ has joined #openstack-glance | 14:45 | |
| *** wxy has quit IRC | 14:48 | |
| *** bdossant_ has quit IRC | 14:53 | |
| *** bdossant has joined #openstack-glance | 14:54 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:57 | |
| *** ducttape_ has joined #openstack-glance | 15:00 | |
| *** mclaren has joined #openstack-glance | 15:00 | |
| *** smatzek has joined #openstack-glance | 15:02 | |
| *** julim has quit IRC | 15:03 | |
| kairat | bunting, I think you can write a letter to openstack-dev | 15:03 |
| kairat | IMO, it would be more effective than waiting a week for next meeting | 15:03 |
| *** houming has joined #openstack-glance | 15:04 | |
| bunting | kairat: Yeah, i agree with you | 15:05 |
| *** julim has joined #openstack-glance | 15:06 | |
| flaper87 | bunting: kairat ++ for using the mailing list | 15:07 |
| *** bdossant has quit IRC | 15:16 | |
| *** bdossant has joined #openstack-glance | 15:18 | |
| *** houming has quit IRC | 15:20 | |
| openstackgerrit | Kairat Kushaev proposed openstack/glance_store: Change approach to request storage url for multi-tenant store https://review.openstack.org/252370 | 15:22 |
| bunting | kairat: | 15:24 |
| kairat | bunting, yes | 15:25 |
| *** houming has joined #openstack-glance | 15:25 | |
| bunting | Just as i write this email do you mean that we would take the version off the end of the url? | 15:25 |
| bunting | and use the one in the config? | 15:25 |
| kairat | nope | 15:25 |
| kairat | I mean we should use the value from config | 15:25 |
| kairat | AFAIK, it should be defined by deployer | 15:25 |
| kairat | By default the value is 2 | 15:26 |
| kairat | The problem is that the version in config and url could contradict | 15:26 |
| kairat | So what should we do in this case | 15:26 |
| kairat | Stick to config or | 15:26 |
| kairat | use url | 15:26 |
| kairat | I think we need to stick to config | 15:27 |
| bunting | well if we stick to config we can't use the full url | 15:27 |
| kairat | so we need to update it | 15:27 |
| bunting | if it is something like swift:redacted@10.0.0.8:5000/v2.0/ | 15:27 |
| kairat | that's one of option | 15:27 |
| bunting | then we would need to change that to v3 | 15:27 |
| kairat | the second option is yours | 15:27 |
| bunting | or else it would not work if there is a 3 in the config | 15:27 |
| kairat | but why do we need config at all then | 15:27 |
| kairat | I think we need to choose the single approach | 15:28 |
| bunting | because the 'new' style is swift:ref/glance | 15:28 |
| bunting | it replaces the ref | 15:28 |
| *** bpoulos has quit IRC | 15:28 | |
| kairat | when ref is parsed the attribute value would be keystone auth address | 15:28 |
| kairat | AFAIK | 15:29 |
| kairat | so there should not be a problem | 15:29 |
| bunting | Yeah in that case it uses all the values in the config | 15:29 |
| *** dshakhray has quit IRC | 15:30 | |
| bunting | kairat: I just don't have a clear picture in my head what values you propose to use | 15:31 |
| *** mancdaz has quit IRC | 15:31 | |
| *** julim has quit IRC | 15:31 | |
| *** mancdaz has joined #openstack-glance | 15:31 | |
| kairat | ok, we have swift_store_auth_version or auth_version in reference config file | 15:32 |
| *** julim has joined #openstack-glance | 15:32 | |
| bunting | yup | 15:33 |
| kairat | That option defines what version need to be used as auth_version in swift | 15:33 |
| bunting | well with that example url above | 15:33 |
| bunting | the auth_url would be 10.0.0.8:5000/v2.0/ | 15:33 |
| bunting | so we could not use that url if both the configs are set to 3 | 15:34 |
| kairat | So you can change this also | 15:34 |
| kairat | AFAIK these options don't need to specified | 15:34 |
| kairat | swift_store_auth_version is deprecated at least for single-tenant store | 15:34 |
| kairat | because it is expose creds to users | 15:35 |
| kairat | Soo | 15:35 |
| bunting | Yeah it is deprecated, but some users may still have full locations in the database | 15:35 |
| kairat | yep, that's the problem | 15:35 |
| kairat | My question is, should we support authenticating with old keystone versions if config changes | 15:36 |
| kairat | or we can use the same version for all images | 15:37 |
| bunting | So you think if we change the config, we could potentially mean users can't use there images any more? | 15:37 |
| kairat | if old keystone api will be dropped | 15:37 |
| kairat | then yes | 15:37 |
| kairat | and there would be no way except db update to fix that | 15:38 |
| kairat | I am not sure if it is appropriate | 15:38 |
| kairat | I would like to have the single source of auth_version information | 15:39 |
| kairat | But of course there could be some problems | 15:39 |
| bunting | okay i'll try to get your stance across in the email as the other option | 15:40 |
| kairat | thanks bunting | 15:41 |
| *** kebray has joined #openstack-glance | 15:41 | |
| openstackgerrit | Kairat Kushaev proposed openstack/glance: Import i18 functions directly https://review.openstack.org/228859 | 15:42 |
| * flaper87 dives into the import process spec | 15:43 | |
| kairat | bunting, if you stick with your implementation | 15:44 |
| kairat | then you don't need to consider config at all | 15:44 |
| kairat | It becomes useless | 15:44 |
| mclaren | flaper87: I'm around for a bit if you want to ping me on anything | 15:44 |
| kairat | so anyway the patch might need to be updated | 15:44 |
| flaper87 | mclaren: I do, I'll need you to elaborate a bit more on your comment | 15:45 |
| flaper87 | mclaren: what are you proposing there? | 15:45 |
| mclaren | Well I'm just throwing it out there to see what folks think... | 15:45 |
| mclaren | we could separate the bikeshed from the image | 15:46 |
| flaper87 | mclaren: yup, is that a separate endpoint from images ? | 15:46 |
| flaper87 | ok | 15:46 |
| mclaren | yeah, so basically the swift case works pretty nicely | 15:46 |
| mclaren | so I was thinking what could we do to make the glance case more like that | 15:46 |
| mclaren | totally separate endpoint for bikesheds | 15:47 |
| mclaren | a fairly dumb thing to support PUT, list, DELETE bikesheds independently of images | 15:48 |
| mclaren | It feels like we may need much of that anyway (although I'm prepared to accept I'm wrong on that) | 15:49 |
| mclaren | and I wonder about spaghettification if we try to put it into the existing code paths | 15:49 |
| flaper87 | mclaren: mmh, I think you're wrong on that. I don't see the difference between the `/imaves/$id/bikeshed` and `/bokesheds` other than I don't see the need of list/deletes in the former | 15:50 |
| mclaren | one example I can think of is v1 delete: it would need to become bikeshed aware if a bikeshed is part of an image | 15:50 |
| flaper87 | mclaren: that's a good point, though | 15:50 |
| mclaren | imaging two users Alice and Bob | 15:51 |
| mclaren | there's 100 GB of storage, each gets a quota of 50GB. | 15:51 |
| mclaren | Bob shouldn't be able to use Alice's quota | 15:51 |
| mclaren | Bob should be able to see what's using his quota | 15:51 |
| *** haomaiwa_ has joined #openstack-glance | 15:52 | |
| mclaren | Are those fair assumptions? | 15:52 |
| mclaren | imaging/imagine | 15:52 |
| flaper87 | sure, but that can be done with `/images/$id/bikeshed` without adding lists/deletes | 15:53 |
| flaper87 | I think the same rules for `/file` apply to `/bikeshed` | 15:53 |
| flaper87 | with the difference that `/bikeshed` will have to be processed | 15:53 |
| flaper87 | same rules with regard to quota, I mean. | 15:53 |
| mclaren | The bikeshed needs a checksum/size and to co-exist with /file -- agreed? | 15:54 |
| flaper87 | mmh, I think so... yes. | 15:55 |
| mclaren | and you need to be able to list them? | 15:55 |
| flaper87 | nope, that's what the image is for | 15:55 |
| flaper87 | we should have that info in the image | 15:56 |
| mclaren | So that't the kind of thing that worries me slightly | 15:56 |
| krotscheck | kairat: No, they have to be explicitly defined. | 15:57 |
| flaper87 | mclaren: anything that changes the state of things worries you <half-joke> :D | 15:57 |
| mclaren | a bikeshed isn't necessarily a natural thing that an image has | 15:57 |
| kairat | krotscheck, ok, thanks, we have some metadata in headers for v1 | 15:58 |
| flaper87 | sure | 15:58 |
| mclaren | if v1 show should show me my image, should it show me the bikeshed? | 15:58 |
| krotscheck | kairat: Gotcha. Annotate my patch and I'll update it after my meeting this morning :) | 15:58 |
| kairat | krotscheck, these headers can be user-defined | 15:59 |
| *** RichardRaseley has quit IRC | 15:59 | |
| krotscheck | kairat: Huhn. | 15:59 |
| krotscheck | kairat: Looked at the meeting logs - I'm guessing v2 support only is acceptable. | 15:59 |
| kairat | krotscheck, so it would not be good idea to use ity with v1) | 15:59 |
| kairat | yep | 15:59 |
| mclaren | we could say that image show is just different in v1 and v2, but separating /bikeshed avoids that | 15:59 |
| flaper87 | fwiw, image-show is already different in v1 and v2 | 16:00 |
| *** haomaiwa_ has quit IRC | 16:00 | |
| *** julim has quit IRC | 16:01 | |
| openstackgerrit | Kairat Kushaev proposed openstack/glance: Validate empty location value for v1 api https://review.openstack.org/226295 | 16:01 |
| flaper87 | TBH, I'm not that worried about this | 16:01 |
| *** 18VAAEQWZ has joined #openstack-glance | 16:01 | |
| mclaren | Do you see and advantage in keeping the /bikeshed as part of an image? | 16:01 |
| *** aix has quit IRC | 16:01 | |
| flaper87 | Separating bikeshed will likely endup being like the tasks endpoint we have | 16:01 |
| *** 18VAAEQWZ has quit IRC | 16:02 | |
| mclaren | in what sense? | 16:02 |
| flaper87 | yes, it sends a clear message that it's related to the image and that, whenever the image is changed/deleted/whatever that'll likely affect the bikeshed as we;; | 16:02 |
| *** haomaiwa_ has joined #openstack-glance | 16:02 | |
| *** haomaiwa_ has quit IRC | 16:03 | |
| flaper87 | s/we;;/well/ | 16:03 |
| *** haomaiwa_ has joined #openstack-glance | 16:03 | |
| *** julim has joined #openstack-glance | 16:03 | |
| flaper87 | The close relation between the place the image is upload to (when using direct-upload) and the image resource is important | 16:03 |
| *** haomaiwa_ has quit IRC | 16:04 | |
| mclaren | on the other hand it's more consitent with the import from swift approach | 16:04 |
| flaper87 | I see bikeshed the same way I see /file | 16:04 |
| *** haomaiwang has joined #openstack-glance | 16:04 | |
| *** haomaiwang has quit IRC | 16:05 | |
| flaper87 | but swift is an external resource, it's like the old copy-from | 16:05 |
| *** haomaiwa_ has joined #openstack-glance | 16:05 | |
| *** haomaiwa_ has quit IRC | 16:06 | |
| *** haomaiwang has joined #openstack-glance | 16:06 | |
| mclaren | I see the bikeshed as a staging area for unvalidated bytes | 16:06 |
| mclaren | like the swift case | 16:06 |
| flaper87 | and like the /file endpoint | 16:06 |
| *** haomaiwang has quit IRC | 16:07 | |
| flaper87 | the bytes there are unvalidated | 16:07 |
| mclaren | do you not see any potential for the implementation to be cleaner? | 16:07 |
| flaper87 | I guess the difference is that I see the /bikeshed endpoint as something internal and strictly related to the image process | 16:07 |
| *** haomaiwang has joined #openstack-glance | 16:07 | |
| *** haomaiwang has quit IRC | 16:08 | |
| flaper87 | I think the latest PS made it smaller by removing the swift part and a set of resources that aren't needed | 16:08 |
| flaper87 | the implementation can still re-use existing code and be cleaner | 16:08 |
| *** haomaiwa_ has joined #openstack-glance | 16:08 | |
| *** haomaiwa_ has quit IRC | 16:09 | |
| mclaren | which PS? | 16:09 |
| flaper87 | what worries me more is the process itself as I don't think bikshed being under /images is a problem | 16:09 |
| flaper87 | PS7 | 16:09 |
| flaper87 | it made it smaller compared to PS5 | 16:09 |
| *** haomaiwa_ has joined #openstack-glance | 16:09 | |
| *** haomaiwa_ has quit IRC | 16:10 | |
| *** 5EXAADMHT has joined #openstack-glance | 16:10 | |
| mclaren | I think bikeshed under /images has the potential to make the code more complex/entangled than it needs to be. You're going to have to handle more dependencies. | 16:10 |
| *** 5EXAADMHT has quit IRC | 16:11 | |
| flaper87 | why? | 16:11 |
| flaper87 | Having it separted is moving the dependencies to the user, basically | 16:11 |
| *** haomaiwang has joined #openstack-glance | 16:11 | |
| *** haomaiwang has quit IRC | 16:12 | |
| mclaren | because the number of potential internal states an image can be in will increase. And failure for each operation on each of those states will need to be handled. For v1 and v2. | 16:12 |
| *** 5EXAADMKT has joined #openstack-glance | 16:12 | |
| flaper87 | I think you still need them. The separate /bikeshed resource is just moving them somewhere else | 16:12 |
| mclaren | I diagree. | 16:12 |
| *** rcernin has quit IRC | 16:12 | |
| flaper87 | we have to agree to disagree :D | 16:12 |
| *** 5EXAADMKT has quit IRC | 16:13 | |
| flaper87 | Until now, the user just cared about uploading stuff to the glance upload path | 16:13 |
| flaper87 | now the user needs to worry about /bikeshed | 16:13 |
| *** 5EXAADML5 has joined #openstack-glance | 16:13 | |
| flaper87 | list it, figure out if it's using space and whatnot | 16:13 |
| mclaren | You have two things, each of which have a small set of states. Rather than one thing which has a large set. The number of permutations is much larger in the latter case. | 16:13 |
| *** 5EXAADML5 has quit IRC | 16:13 | |
| flaper87 | Right, but, as I said, you still need them. You just moved those states and responsibility somewhere else | 16:14 |
| flaper87 | I don't think this is something we should offload to the user, fwiw. | 16:14 |
| mclaren | it's like the difference between two decks of cards with 10 cards each or one deck of cards with 20 cards. The number of states is much bigger with one larger deck. | 16:14 |
| mclaren | mathematically and shit :-) | 16:15 |
| flaper87 | I understand that and I'm not really worried about that, tbh. | 16:15 |
| flaper87 | but that's me | 16:15 |
| flaper87 | I'm more worried about what this means for the user | 16:15 |
| flaper87 | and how this impacts the process | 16:15 |
| flaper87 | which is exactly what we're trying to solve here | 16:15 |
| mclaren | sure | 16:16 |
| mclaren | From a user's point of view the process becomes the same for swift or glance: upload the bits to some store (provided by glance or swift) and then import the bits | 16:17 |
| mclaren | you can reuse your object if you want in both cases | 16:17 |
| mclaren | you can choose to delete it or not in both cases | 16:17 |
| *** bdossant has quit IRC | 16:18 | |
| mclaren | you have a command to list your objects in both cases | 16:18 |
| flaper87 | no no, it's not the same. Until now, the user didn't have to worry about that and I don't think the user should worry about that. Not when using the direct-upload to glance | 16:18 |
| flaper87 | The bikeshed endpoint as you described it will be just an http server | 16:19 |
| mclaren | Hmm, so I'm not sure that the need for any of that goes away if the bikeshed goes into the image | 16:19 |
| tsufiev | nikhil, speaking of multipart/form-data... | 16:19 |
| mclaren | user's will still need to 'manage' their bikesheds | 16:19 |
| nikhil | tsufiev: hi | 16:20 |
| flaper87 | it does. The user just uploads the image and the used doesn't have to worrye about managing the bikeshed | 16:20 |
| tsufiev | nikhil, hello :) | 16:20 |
| nikhil | tsufiev: let's chat somewhere else | 16:20 |
| flaper87 | no, because the bikeshed will be strictly related to the image | 16:20 |
| mclaren | users will need to know where their bikesheds are right? They'll need a call (whatever it looks like) "show me my bikesheds" | 16:21 |
| flaper87 | if re-using image bits is a requirement, we can very well implement image-copy in the future | 16:21 |
| flaper87 | no, why do users need that? | 16:21 |
| flaper87 | Do they need a "show me my files" call now ? | 16:21 |
| mclaren | if you have loads of images and a few bikesheds how you do find them? | 16:22 |
| mclaren | (that's probably a fairly common scenario) | 16:22 |
| flaper87 | uh crap, meeting in 8mins. | 16:22 |
| flaper87 | the info is in the image metadata | 16:22 |
| flaper87 | I'll comment on the spec with my thoughts | 16:23 |
| mclaren | sure, but you've got to search a large number of images for a small number of bikesheds | 16:23 |
| flaper87 | just filter images with size != 0 ? | 16:23 |
| flaper87 | or something like that | 16:23 |
| flaper87 | whatever filters there are that can be used | 16:24 |
| mclaren | you mean an 'images with bikesheds' filter? | 16:24 |
| flaper87 | I guess | 16:24 |
| * flaper87 shrugs | 16:24 | |
| mclaren | because that's kind of listing bikesheds :-) | 16:24 |
| flaper87 | I knew you were going to say that but I still believe it's not the same | 16:24 |
| flaper87 | as the image is the resource | 16:25 |
| mclaren | lol | 16:25 |
| flaper87 | the bikeshed is just some data associated with that image | 16:25 |
| flaper87 | Once the image is processed, the bikeshed is going away as it'll become the processed, imag data | 16:26 |
| flaper87 | ok, gtg now. | 16:26 |
| flaper87 | I'll comment on the spec | 16:26 |
| mclaren | ok, thanks | 16:26 |
| flaper87 | np, thank you for throwing more ideas out. It's better to know we're exploring as many ideas as we can | 16:27 |
| mclaren | sure | 16:27 |
| *** kebray has quit IRC | 16:37 | |
| *** davemc_afk has quit IRC | 16:37 | |
| *** dshakhray has joined #openstack-glance | 16:39 | |
| *** belmoreira has quit IRC | 16:42 | |
| *** rcernin has joined #openstack-glance | 16:54 | |
| *** jaypipes has quit IRC | 16:59 | |
| *** ninag_ has quit IRC | 17:00 | |
| *** ninag_ has joined #openstack-glance | 17:04 | |
| *** ninag_ has quit IRC | 17:09 | |
| *** houming has quit IRC | 17:09 | |
| openstackgerrit | Merged openstack/glance_store: Put py34 first in the env order of tox https://review.openstack.org/252442 | 17:12 |
| *** RichardRaseley has joined #openstack-glance | 17:19 | |
| *** jistr has quit IRC | 17:21 | |
| *** bpoulos has joined #openstack-glance | 17:25 | |
| hemanthm | flaper87: https://review.openstack.org/#/c/253088/ | 17:27 |
| *** mtanino has joined #openstack-glance | 17:28 | |
| bpoulos | hemanthm: you up for discussing the sha-2 stuff further? | 17:32 |
| bpoulos | the signature_utils only supports SHA-2 hashes, but there isn't a default in place -- the user has to specify which hash method to use (this is the hash for the signature verification, not for the glance checksum) | 17:34 |
| bpoulos | and this uses the cryptography library, which appears to work on the latest version of openssl 0.9.8: http://cryptography.readthedocs.org/en/latest/installation/ | 17:35 |
| bpoulos | and no hashes are done unless all of the signature properties are provided during upload | 17:35 |
| *** ninag has joined #openstack-glance | 17:36 | |
| *** ninag has quit IRC | 17:36 | |
| hemanthm | bpoulos: o/ | 17:36 |
| bpoulos | hemanthm: hello | 17:37 |
| hemanthm | let me get the error I see with openssl 0.9.8 | 17:37 |
| bpoulos | ok, great | 17:37 |
| hemanthm | http://paste.openstack.org/show/480800/ | 17:39 |
| hemanthm | bpoulos ^ | 17:39 |
| bpoulos | hemanthm: thanks | 17:39 |
| hemanthm | essentially, openssl 0.9.8 doesn't support SHA-2 out of the box | 17:39 |
| bpoulos | I hesitate to allow support for SHA-1 (adding it to the list of hash algorithm options supported) since it's considered out-of-date and insecure. It sounds like the signature_utils code should catch the unsupported algorithm exception, and log that signature verification isn't supported on platforms that don't support sha-2 | 17:45 |
| *** aix has joined #openstack-glance | 17:49 | |
| bpoulos | hemathm: thank you for bringing this up -- I think it deserves a separate bug report, where how to handle it can be discussed (i.e., support SHA-1 or add log messages/don't support signature verification on older platforms) | 17:51 |
| bpoulos | hemanthm: I'll put together the bug report, unless you'd like to | 17:51 |
| hemanthm | bpoulos: I'm not necessarily saying we should support SHA-1 | 17:52 |
| hemanthm | just that glance shouldn't break it can't use SHA-2 | 17:52 |
| hemanthm | *when it can't use SHA-2 | 17:52 |
| hemanthm | bpoulos: feel free to do the bug report, you can probably add more detail to it than I can | 17:54 |
| bpoulos | hemanthm: ok, will do, thanks again! | 17:54 |
| hemanthm | thanks for looking into it, bpoulos! | 17:54 |
| *** ninag has joined #openstack-glance | 17:56 | |
| *** aix has quit IRC | 18:09 | |
| bpoulos | hemanthm: I added the bug report (https://bugs.launchpad.net/glance/+bug/1522525) -- feel free to add any further details if you see something missing | 18:10 |
| openstack | Launchpad bug 1522525 in Glance "test_signature_utils tests fail when openssl is older than 1.0.1" [Undecided,New] - Assigned to Brianna Poulos (brianna-poulos) | 18:10 |
| *** bpoulos has quit IRC | 18:11 | |
| *** aix has joined #openstack-glance | 18:13 | |
| hemanthm | thank you, bpoulos | 18:14 |
| *** markus_z has quit IRC | 18:16 | |
| openstackgerrit | Mike Fedosin proposed openstack/glance: Implement trust support for api v2 https://review.openstack.org/241986 | 18:21 |
| *** azbiswas has joined #openstack-glance | 18:22 | |
| *** davideagnello has quit IRC | 18:26 | |
| *** davideagnello has joined #openstack-glance | 18:26 | |
| openstackgerrit | Hemanth Makkapati proposed openstack/glance: Rename glance-store to glance_store https://review.openstack.org/253126 | 18:36 |
| hemanthm | nikhil ^ | 18:36 |
| *** ctina_ has quit IRC | 18:53 | |
| *** RichardRaseley has quit IRC | 19:03 | |
| *** edmondsw has quit IRC | 19:06 | |
| *** ninag has quit IRC | 19:06 | |
| *** ninag has joined #openstack-glance | 19:10 | |
| *** ninag_ has joined #openstack-glance | 19:11 | |
| *** ninag has quit IRC | 19:14 | |
| *** ninag_ has quit IRC | 19:16 | |
| *** ninag has joined #openstack-glance | 19:19 | |
| *** ninag has quit IRC | 19:25 | |
| *** ninag has joined #openstack-glance | 19:26 | |
| *** ninag has quit IRC | 19:27 | |
| *** ninag has joined #openstack-glance | 19:28 | |
| *** ninag has quit IRC | 19:30 | |
| *** ninag has joined #openstack-glance | 19:31 | |
| *** azbiswas has quit IRC | 19:49 | |
| *** davideagnello has quit IRC | 19:50 | |
| *** ninag has quit IRC | 19:50 | |
| *** davideagnello has joined #openstack-glance | 19:50 | |
| *** ninag has joined #openstack-glance | 19:52 | |
| *** azbiswas_ has joined #openstack-glance | 19:53 | |
| *** chlong has quit IRC | 19:57 | |
| *** edmondsw has joined #openstack-glance | 19:58 | |
| *** chlong has joined #openstack-glance | 20:16 | |
| *** RichardRaseley has joined #openstack-glance | 20:25 | |
| *** changbl has quit IRC | 20:26 | |
| *** ninag has quit IRC | 20:31 | |
| *** avarner has quit IRC | 20:32 | |
| openstackgerrit | lifeless proposed openstack/glance: Stop using tearDown in v1/test_api.py. https://review.openstack.org/252091 | 20:38 |
| flaper87 | hemanthm: https://review.openstack.org/#/c/208851/ pls | 20:40 |
| openstackgerrit | lifeless proposed openstack/glance: Stop using tearDown in v1/test_api.py. https://review.openstack.org/252091 | 20:40 |
| *** changbl has joined #openstack-glance | 20:44 | |
| *** openstackgerrit has quit IRC | 20:47 | |
| *** openstackgerrit has joined #openstack-glance | 20:47 | |
| *** zul has quit IRC | 20:48 | |
| *** azbiswas_ has quit IRC | 20:50 | |
| *** azbiswas has joined #openstack-glance | 20:54 | |
| *** azbiswas has joined #openstack-glance | 20:54 | |
| *** mancdaz has quit IRC | 20:58 | |
| *** mancdaz has joined #openstack-glance | 21:01 | |
| *** rcernin has quit IRC | 21:05 | |
| flaper87 | sabari: https://review.openstack.org/#/c/250736/ pls | 21:06 |
| *** delatte has quit IRC | 21:07 | |
| sabari | flaper87 in a meeting :( but will do asap | 21:08 |
| sabari | flaper87 that was an easy one - sorry I even gave an excuse for it :D | 21:15 |
| sabari | *nods in the meeting room though has no clue what's going on* | 21:17 |
| *** smatzek has quit IRC | 21:23 | |
| *** julim has quit IRC | 21:24 | |
| flaper87 | sabari: lol, that's the story of my life | 21:25 |
| flaper87 | sabari: actually, I lied. #lifeofaremotee | 21:25 |
| *** e0ne_ has quit IRC | 21:26 | |
| *** spzala has quit IRC | 21:26 | |
| *** changbl has quit IRC | 21:34 | |
| *** cdelatte has joined #openstack-glance | 21:38 | |
| *** diegows has joined #openstack-glance | 21:46 | |
| *** changbl has joined #openstack-glance | 21:47 | |
| openstackgerrit | Drew Varner proposed openstack/glance_store: Add reno for release notes management in glance_store https://review.openstack.org/251561 | 21:56 |
| *** edmondsw has quit IRC | 22:05 | |
| *** agireud has quit IRC | 22:47 | |
| *** burgerk has quit IRC | 22:48 | |
| *** maliniB has joined #openstack-glance | 22:53 | |
| *** chlong has quit IRC | 23:08 | |
| *** spzala has joined #openstack-glance | 23:09 | |
| *** jerrygb has quit IRC | 23:12 | |
| *** diegows has quit IRC | 23:22 | |
| *** azbiswas has quit IRC | 23:28 | |
| *** azbiswas has joined #openstack-glance | 23:28 | |
| *** azbiswas has quit IRC | 23:32 | |
| *** spzala has quit IRC | 23:43 | |
| *** agireud has joined #openstack-glance | 23:52 | |
| *** ducttape_ has quit IRC | 23:53 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!