| *** rwsu has quit IRC | 00:10 | |
| openstackgerrit | Sabari proposed openstack/glance_store: Fix sorting query string keys for arbitrary url schemes https://review.openstack.org/148767 | 00:12 |
|---|---|---|
| *** rwsu has joined #openstack-glance | 00:21 | |
| openstackgerrit | Sabari proposed openstack/glance_store: VMware: Support Multiple Datastores https://review.openstack.org/148426 | 00:26 |
| *** haomaiwa_ has joined #openstack-glance | 00:26 | |
| *** sgotliv has quit IRC | 00:44 | |
| *** cpallares has quit IRC | 00:46 | |
| *** changbl has joined #openstack-glance | 00:56 | |
| *** esheffield has quit IRC | 00:59 | |
| *** sabari is now known as zz_sabari | 01:05 | |
| *** jasondotstar has joined #openstack-glance | 01:07 | |
| openstackgerrit | Steve Lewis proposed openstack/glance: Eliminate shell param from subprocesses in tests https://review.openstack.org/149111 | 01:33 |
| *** peristeri has quit IRC | 01:34 | |
| openstackgerrit | Fei Long Wang proposed openstack/glance: Make digest algorithm configurable https://review.openstack.org/80178 | 01:35 |
| openstackgerrit | Ian Cordasco proposed openstack/glance-specs: Pass Targets to Glance's Policy Enforcer https://review.openstack.org/149112 | 01:36 |
| openstackgerrit | Ian Cordasco proposed openstack/glance: Pass a real image target to the policy enforcer https://review.openstack.org/146651 | 01:40 |
| openstackgerrit | Fei Long Wang proposed openstack/glance: Make digest algorithm configurable https://review.openstack.org/80178 | 01:41 |
| *** takedakn has joined #openstack-glance | 01:48 | |
| *** d0ugal has quit IRC | 01:55 | |
| *** d0ugal has joined #openstack-glance | 01:55 | |
| *** d0ugal is now known as Guest85104 | 01:56 | |
| *** leseb- has joined #openstack-glance | 01:57 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 01:57 | |
| *** tristanC_ has joined #openstack-glance | 01:58 | |
| *** kragniz_ has joined #openstack-glance | 01:59 | |
| *** julim_ has joined #openstack-glance | 01:59 | |
| *** TravT_ has quit IRC | 02:00 | |
| openstackgerrit | Ian Cordasco proposed openstack/glance-specs: Pass Targets to Glance's Policy Enforcer https://review.openstack.org/149112 | 02:04 |
| openstackgerrit | Ian Cordasco proposed openstack/glance-specs: Pass Targets to Glance's Policy Enforcer https://review.openstack.org/149112 | 02:05 |
| *** leseb has quit IRC | 02:06 | |
| *** tristanC has quit IRC | 02:06 | |
| *** kragniz has quit IRC | 02:06 | |
| *** julim has quit IRC | 02:06 | |
| *** takedakn has quit IRC | 02:06 | |
| *** lifeless has quit IRC | 02:41 | |
| *** lifeless has joined #openstack-glance | 02:42 | |
| openstackgerrit | Steve Lewis proposed openstack/glance: Rewrite SSL tests https://review.openstack.org/148400 | 03:02 |
| *** zz_sabari is now known as sabari | 03:05 | |
| *** rwsu has quit IRC | 03:10 | |
| *** bkopilov has quit IRC | 03:32 | |
| *** sabari is now known as zz_sabari | 03:33 | |
| *** buzztroll has quit IRC | 03:36 | |
| *** zz_sabari is now known as sabari | 03:38 | |
| *** Longgeek has joined #openstack-glance | 03:41 | |
| openstackgerrit | Merged openstack/glance: Move from oslo.config to oslo_config https://review.openstack.org/146476 | 03:43 |
| openstackgerrit | Merged openstack/glance: Move from oslo.db to oslo_db https://review.openstack.org/146612 | 03:43 |
| openstackgerrit | Merged openstack/glance: Fix reference to "stores" from deprecated name https://review.openstack.org/145994 | 03:43 |
| *** jasondotstar has quit IRC | 03:47 | |
| *** zhiyan has quit IRC | 03:48 | |
| *** zhiyan has joined #openstack-glance | 03:49 | |
| *** TravT has joined #openstack-glance | 04:13 | |
| *** rajesht has joined #openstack-glance | 04:16 | |
| *** rajesht has quit IRC | 04:16 | |
| *** TravT has quit IRC | 04:17 | |
| *** ankit_ag has joined #openstack-glance | 04:22 | |
| *** TravT has joined #openstack-glance | 04:24 | |
| *** sabari is now known as zz_sabari | 04:29 | |
| *** TravT has quit IRC | 04:30 | |
| *** zz_sabari is now known as sabari | 04:31 | |
| *** Longgeek has quit IRC | 04:34 | |
| *** Longgeek has joined #openstack-glance | 04:34 | |
| *** bkopilov has joined #openstack-glance | 04:34 | |
| *** buzztroll has joined #openstack-glance | 04:38 | |
| *** bkopilov has quit IRC | 04:39 | |
| *** TravT has joined #openstack-glance | 04:41 | |
| *** TravT has quit IRC | 04:41 | |
| *** TravT has joined #openstack-glance | 04:42 | |
| *** TravT has quit IRC | 04:42 | |
| *** TravT has joined #openstack-glance | 04:43 | |
| *** changbl has quit IRC | 04:44 | |
| *** harlowja is now known as harlowja_away | 04:51 | |
| *** Longgeek_ has joined #openstack-glance | 04:53 | |
| *** Longgeek_ has quit IRC | 04:56 | |
| *** Longgeek has quit IRC | 04:56 | |
| *** Longgeek has joined #openstack-glance | 04:56 | |
| *** Longgeek_ has joined #openstack-glance | 04:59 | |
| *** Longgeek has quit IRC | 04:59 | |
| *** Longgeek has joined #openstack-glance | 05:00 | |
| *** Longgeek has quit IRC | 05:01 | |
| *** Longgeek has joined #openstack-glance | 05:02 | |
| *** Longgeek_ has quit IRC | 05:04 | |
| *** sabari is now known as zz_sabari | 05:10 | |
| *** zz_sabari is now known as sabari | 05:11 | |
| *** Longgeek has quit IRC | 05:12 | |
| *** Longgeek has joined #openstack-glance | 05:12 | |
| openstackgerrit | Merged openstack/glance-specs: Taskflow Integration https://review.openstack.org/130076 | 05:13 |
| *** Longgeek has quit IRC | 05:16 | |
| *** buzztroll has quit IRC | 05:24 | |
| *** buzztroll has joined #openstack-glance | 05:29 | |
| *** sgotliv has joined #openstack-glance | 05:32 | |
| *** Longgeek has joined #openstack-glance | 05:35 | |
| *** Longgeek has quit IRC | 05:40 | |
| *** Longgeek has joined #openstack-glance | 05:50 | |
| *** sgotliv has quit IRC | 05:51 | |
| *** Longgeek has quit IRC | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Imported Translations from Transifex https://review.openstack.org/146765 | 06:08 |
| *** Longgeek has joined #openstack-glance | 06:09 | |
| *** Longgeek has quit IRC | 06:28 | |
| *** Longgeek has joined #openstack-glance | 06:28 | |
| *** sabari is now known as zz_sabari | 06:34 | |
| *** Longgeek has quit IRC | 06:38 | |
| *** groen692 has joined #openstack-glance | 06:48 | |
| *** nlevinki has joined #openstack-glance | 06:53 | |
| *** tshefi has joined #openstack-glance | 06:55 | |
| *** tshefi has quit IRC | 07:29 | |
| *** Longgeek has joined #openstack-glance | 07:37 | |
| *** chlong has quit IRC | 07:38 | |
| *** mflobo has joined #openstack-glance | 07:41 | |
| *** Guest85104 is now known as d0ugal | 07:42 | |
| *** d0ugal is now known as Guest79195 | 07:42 | |
| *** tshefi has joined #openstack-glance | 07:44 | |
| *** Guest79195 is now known as d0ugal | 07:48 | |
| *** d0ugal has joined #openstack-glance | 07:48 | |
| *** sgotliv has joined #openstack-glance | 07:49 | |
| *** jamielennox is now known as jamielennox|away | 08:01 | |
| *** Longgeek has quit IRC | 08:04 | |
| *** takedakn has joined #openstack-glance | 08:18 | |
| *** takedakn has quit IRC | 08:19 | |
| *** TravT_ has joined #openstack-glance | 08:24 | |
| *** TravT has quit IRC | 08:24 | |
| *** Longgeek has joined #openstack-glance | 08:28 | |
| *** markus_z has joined #openstack-glance | 08:34 | |
| *** zz_sabari is now known as sabari | 08:40 | |
| *** Longgeek has quit IRC | 08:42 | |
| *** Longgeek_ has joined #openstack-glance | 08:44 | |
| *** pkoniszewski has joined #openstack-glance | 08:45 | |
| *** ameade_ has quit IRC | 08:52 | |
| *** zhiyan has quit IRC | 08:55 | |
| flaper87 | kragniz_: nikhil_k sigmavirus24_awa jokke_ (wherever you are) zhiyan Question: now that we have locations, would it make sense to just get rid of `direct_url` and make users rely on `locations` to get the direct addresses to the images ? | 08:57 |
| flaper87 | having both things is kind of inconsistent and a PITA | 08:57 |
| *** krykowski has joined #openstack-glance | 09:01 | |
| *** zhiyan has joined #openstack-glance | 09:01 | |
| *** ameade_ has joined #openstack-glance | 09:01 | |
| *** buzztroll has quit IRC | 09:02 | |
| *** pkoniszewski has quit IRC | 09:15 | |
| *** pkonisze has joined #openstack-glance | 09:15 | |
| *** belmoreira has joined #openstack-glance | 09:16 | |
| *** sabari is now known as zz_sabari | 09:16 | |
| *** pkonisze has quit IRC | 09:19 | |
| *** pdb has joined #openstack-glance | 09:21 | |
| *** zz_sabari is now known as sabari | 09:22 | |
| *** jistr has joined #openstack-glance | 09:24 | |
| *** pkoniszewski has joined #openstack-glance | 09:27 | |
| *** krykowski has quit IRC | 09:31 | |
| *** krykowski has joined #openstack-glance | 09:32 | |
| *** flwang1 has joined #openstack-glance | 09:36 | |
| *** pkoniszewski has quit IRC | 09:38 | |
| *** pkoniszewski has joined #openstack-glance | 09:40 | |
| *** pkoniszewski has quit IRC | 09:45 | |
| *** krykowski has quit IRC | 09:49 | |
| *** nellysmitt has joined #openstack-glance | 09:50 | |
| *** bkopilov has joined #openstack-glance | 09:57 | |
| *** krykowski has joined #openstack-glance | 09:57 | |
| *** haomaiwa_ has quit IRC | 10:01 | |
| *** sabari is now known as zz_sabari | 10:04 | |
| *** chlong has joined #openstack-glance | 10:15 | |
| *** aix has joined #openstack-glance | 10:20 | |
| *** chlong has quit IRC | 10:20 | |
| *** krykowski has quit IRC | 10:24 | |
| *** krykowski has joined #openstack-glance | 10:29 | |
| *** jasondotstar has joined #openstack-glance | 10:30 | |
| *** jasondotstar has quit IRC | 10:30 | |
| *** Longgeek_ has quit IRC | 10:32 | |
| openstackgerrit | Nikolay Starodubtsev proposed openstack/glance: Remove optparse from glance-replicator https://review.openstack.org/129864 | 10:32 |
| *** chlong has joined #openstack-glance | 10:33 | |
| *** krykowski has quit IRC | 10:34 | |
| *** chlong has quit IRC | 10:37 | |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Taskflow Executor https://review.openstack.org/85211 | 10:40 |
| openstackgerrit | Julien Danjou proposed openstack/glance: Switch to mox3 https://review.openstack.org/147460 | 10:42 |
| *** flwang1 has quit IRC | 10:46 | |
| *** chlong has joined #openstack-glance | 10:50 | |
| *** buzztroll has joined #openstack-glance | 10:51 | |
| *** buzztroll has quit IRC | 10:56 | |
| *** krykowski has joined #openstack-glance | 10:59 | |
| *** krykowski has quit IRC | 11:26 | |
| *** aix has quit IRC | 11:38 | |
| kragniz_ | flaper87: I don't see why not | 11:41 |
| kragniz_ | having both just makes things more confusing | 11:41 |
| *** kragniz_ is now known as kragniz | 11:45 | |
| stevelle | flaper87 kragniz agreed, inconsistent and confusing | 11:49 |
| *** takedakn has joined #openstack-glance | 11:52 | |
| *** takedakn has quit IRC | 11:52 | |
| *** buzztroll has joined #openstack-glance | 12:00 | |
| flaper87 | kragniz: stevelle +1, I'll submit a patch so we can discuss over it | 12:00 |
| flaper87 | we bumped the api version anyway :) | 12:00 |
| kragniz | flaper87: cool | 12:00 |
| *** wolsen_ has quit IRC | 12:00 | |
| kragniz | flaper87: stuart's probably going to not like you :P | 12:01 |
| *** buzztroll has quit IRC | 12:01 | |
| flaper87 | kragniz: he doesn't like me anyway | 12:02 |
| flaper87 | :P | 12:02 |
| flaper87 | (obviously a joke) | 12:02 |
| *** buzztroll has joined #openstack-glance | 12:02 | |
| *** dosaboy has quit IRC | 12:02 | |
| kragniz | :P | 12:02 |
| *** xianghui has quit IRC | 12:02 | |
| flaper87 | I guess we should mark it as deprecated first? No idea how to mark as deprecated something in the API. I guess we will need a patch in both, the client and the API to print the right warnings | 12:03 |
| *** buzztroll has quit IRC | 12:03 | |
| flaper87 | does jsonschema support deprecated properties? | 12:03 |
| flaper87 | I'll have to check | 12:03 |
| kragniz | flaper87: no idea | 12:03 |
| *** buzztroll has joined #openstack-glance | 12:04 | |
| kragniz | but +1 on marking it as deprecated first | 12:04 |
| *** xianghui has joined #openstack-glance | 12:04 | |
| *** dosaboy has joined #openstack-glance | 12:04 | |
| *** buzztroll has quit IRC | 12:05 | |
| *** wolsen has joined #openstack-glance | 12:05 | |
| flaper87 | kragniz: if you're going to agree with me at least give me an idea >.> | 12:05 |
| flaper87 | :P | 12:05 |
| *** buzztroll has joined #openstack-glance | 12:05 | |
| *** buzztroll has quit IRC | 12:07 | |
| *** buzztroll has joined #openstack-glance | 12:07 | |
| *** buzztroll has quit IRC | 12:09 | |
| *** buzztroll has joined #openstack-glance | 12:09 | |
| *** buzztroll has quit IRC | 12:10 | |
| *** buzztroll has joined #openstack-glance | 12:11 | |
| *** buzztrol_ has joined #openstack-glance | 12:13 | |
| *** buzztroll has quit IRC | 12:13 | |
| *** echevemaster has quit IRC | 12:14 | |
| *** buzztrol_ has quit IRC | 12:14 | |
| *** buzztroll has joined #openstack-glance | 12:14 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Fix for CooperativeReader to properly process read length https://review.openstack.org/148574 | 12:16 |
| *** buzztroll has quit IRC | 12:16 | |
| *** buzztroll has joined #openstack-glance | 12:16 | |
| *** buzztroll has quit IRC | 12:18 | |
| *** buzztroll has joined #openstack-glance | 12:18 | |
| flaper87 | kragniz: where are my colored logs? | 12:18 |
| flaper87 | ah ? ah ? ah? | 12:18 |
| *** buzztroll has quit IRC | 12:19 | |
| *** buzztroll has joined #openstack-glance | 12:20 | |
| *** buzztroll has quit IRC | 12:21 | |
| *** buzztroll has joined #openstack-glance | 12:22 | |
| *** buzztroll has quit IRC | 12:23 | |
| *** buzztroll has joined #openstack-glance | 12:23 | |
| *** buzztrol_ has joined #openstack-glance | 12:25 | |
| *** buzztroll has quit IRC | 12:25 | |
| *** buzztroll has joined #openstack-glance | 12:27 | |
| *** buzztrol_ has quit IRC | 12:27 | |
| *** haomaiwang has joined #openstack-glance | 12:28 | |
| *** buzztroll has quit IRC | 12:28 | |
| *** buzztroll has joined #openstack-glance | 12:29 | |
| *** buzztroll has quit IRC | 12:30 | |
| *** buzztroll has joined #openstack-glance | 12:31 | |
| *** haomai___ has joined #openstack-glance | 12:32 | |
| *** buzztroll has quit IRC | 12:32 | |
| *** buzztroll has joined #openstack-glance | 12:32 | |
| *** Longgeek has joined #openstack-glance | 12:33 | |
| *** buzztroll has quit IRC | 12:34 | |
| *** buzztroll has joined #openstack-glance | 12:34 | |
| *** haomaiwang has quit IRC | 12:35 | |
| *** buzztroll has quit IRC | 12:36 | |
| *** buzztroll has joined #openstack-glance | 12:36 | |
| *** buzztrol_ has joined #openstack-glance | 12:38 | |
| *** buzztroll has quit IRC | 12:38 | |
| *** buzztroll has joined #openstack-glance | 12:40 | |
| *** buzztrol_ has quit IRC | 12:40 | |
| *** buzztroll has quit IRC | 12:41 | |
| *** buzztroll has joined #openstack-glance | 12:41 | |
| stevelle | Marking as deprecated should ideally include metadata pointing at the preferred alternative. | 12:42 |
| *** buzztroll has quit IRC | 12:43 | |
| *** buzztroll has joined #openstack-glance | 12:43 | |
| *** chlong has quit IRC | 12:44 | |
| *** buzztrol_ has joined #openstack-glance | 12:45 | |
| *** buzztro__ has joined #openstack-glance | 12:47 | |
| *** buzztrol_ has quit IRC | 12:47 | |
| *** buzztroll has quit IRC | 12:48 | |
| *** buzztro__ has quit IRC | 12:48 | |
| *** buzztroll has joined #openstack-glance | 12:49 | |
| *** buzztrol_ has joined #openstack-glance | 12:50 | |
| *** buzztroll has quit IRC | 12:50 | |
| *** buzztrol_ has quit IRC | 12:52 | |
| *** buzztroll has joined #openstack-glance | 12:52 | |
| *** buzztroll has quit IRC | 12:54 | |
| *** buzztroll has joined #openstack-glance | 12:54 | |
| kragniz | flaper87: ./stack.sh finished before I got very far with those colored logs :P | 12:55 |
| *** buzztroll has quit IRC | 12:55 | |
| *** buzztroll has joined #openstack-glance | 12:56 | |
| *** buzztroll has quit IRC | 12:57 | |
| *** buzztroll has joined #openstack-glance | 12:58 | |
| *** buzztroll has quit IRC | 12:59 | |
| *** takedakn has joined #openstack-glance | 12:59 | |
| *** buzztroll has joined #openstack-glance | 12:59 | |
| *** buzztroll has quit IRC | 13:01 | |
| *** buzztroll has joined #openstack-glance | 13:01 | |
| flaper87 | now, I've another problem. I think our restriction on custom properties is way too strict | 13:03 |
| *** buzztroll has quit IRC | 13:03 | |
| flaper87 | :/ | 13:03 |
| *** buzztroll has joined #openstack-glance | 13:03 | |
| *** takedakn has quit IRC | 13:03 | |
| flaper87 | I'm finding some custom properties used by other services that may not make sense to have in `image-schema.json` but they do make sense in some cases | 13:03 |
| flaper87 | for example, nova uses `image_state` for images coming from s3 | 13:04 |
| kragniz | flaper87: does that matter if allow_additional_image_properties is enabled? | 13:04 |
| kragniz | (defualts to true) | 13:05 |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Extend images cli v2 with new sorting syntax https://review.openstack.org/148981 | 13:06 |
| *** buzztroll has quit IRC | 13:07 | |
| flaper87 | kragniz: mmh, I guess it doesn't but I'm affraid there's something fishy here | 13:09 |
| flaper87 | afraid* | 13:09 |
| flaper87 | because, if that's the default, I don't know why I've been hitting issues with the properties | 13:09 |
| flaper87 | OHHHH WAIT! | 13:09 |
| flaper87 | I think I know the answer | 13:09 |
| *** jyoti-ranjan2 has joined #openstack-glance | 13:14 | |
| *** tshefi has quit IRC | 13:14 | |
| *** tshefi has joined #openstack-glance | 13:14 | |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: A mixin for jsonpatch requests validation https://review.openstack.org/148588 | 13:20 |
| *** pkoniszewski has joined #openstack-glance | 13:27 | |
| kragniz | rosmaita: ping | 13:27 |
| kragniz | flaper87: any luck? | 13:27 |
| flaper87 | kragniz: still testing. So, glance seems to work as expected | 13:28 |
| flaper87 | I think there's something wrong in the nova wrapper | 13:28 |
| kragniz | hmm, that's strange | 13:29 |
| *** marcusvrn_ has joined #openstack-glance | 13:33 | |
| *** flwang has quit IRC | 13:35 | |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: Artifacts API https://review.openstack.org/136629 | 13:37 |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: Artifacts Domain https://review.openstack.org/132898 | 13:37 |
| *** pkoniszewski has quit IRC | 13:37 | |
| *** bkopilov has quit IRC | 13:39 | |
| *** aix has joined #openstack-glance | 13:40 | |
| rosmaita | kragniz: pong | 13:40 |
| kragniz | rosmaita: re the swift store config changes | 13:41 |
| kragniz | rosmaita: do you think it would be okay if retries were enabled by default, but not the delay? | 13:42 |
| rosmaita | i don't have an opinion, really | 13:42 |
| rosmaita | just thought it was odd that stuart, who is usually very conservative about changes, was all for changing the default of this one | 13:42 |
| kragniz | rosmaita: okay | 13:42 |
| kragniz | rosmaita: I know! | 13:43 |
| rosmaita | does my deployer-config-management scenario make sense? | 13:43 |
| kragniz | yup, it does | 13:43 |
| rosmaita | ok | 13:43 |
| kragniz | I think a small number of retries by default makes most sense | 13:44 |
| kragniz | the other stores do the same | 13:44 |
| rosmaita | kragniz: my feeling is if we change the default, might as well go all the way, i.e., retries + exponential spacing | 13:44 |
| rosmaita | but just want to make sure we're agreed that changing the default makes sense | 13:45 |
| rosmaita | which i'm not convinced of | 13:45 |
| kragniz | rosmaita: okay | 13:45 |
| rosmaita | kragniz: i just need an argument that changing the default is a good idea | 13:46 |
| *** adam_g is now known as adam_g_out | 13:46 | |
| kragniz | rosmaita: okay, makes sense | 13:47 |
| rosmaita | just don't want any unnecessary surprises for deployers of small clouds | 13:47 |
| kragniz | I'll try to post something coherent to the review | 13:47 |
| *** tristanC_ is now known as tristanC | 13:47 | |
| rosmaita | if the cloud is small enough, guess you wouldn't have retries happening anyway | 13:47 |
| rosmaita | i really don't know | 13:47 |
| rosmaita | kragniz: thanks! i will look forward to reading it | 13:48 |
| stevelle | https://www.reddit.com/r/Eve/comments/2t8fl0/oh_god_the_sleepers_are_attacking_the_pos/ | 13:48 |
| kragniz | rosmaita: heh, don't get your hopes up :P | 13:48 |
| *** flwang has joined #openstack-glance | 13:48 | |
| kragniz | stevelle: may have been wrong channel :P | 13:49 |
| stevelle | sorry | 13:50 |
| stevelle | too many tabs :D | 13:50 |
| *** ivasilevskaya has joined #openstack-glance | 13:53 | |
| *** groen693 has joined #openstack-glance | 13:57 | |
| *** groen692 has quit IRC | 13:59 | |
| *** lakshmiS has joined #openstack-glance | 14:04 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Add the ability to specify the sort dir for each key https://review.openstack.org/148688 | 14:04 |
| *** lakshmiS has quit IRC | 14:19 | |
| *** jyoti-ranjan2 has quit IRC | 14:19 | |
| nikhil_k | flaper87: Like the idea of consolidating the locations changes starting with direct_url access | 14:22 |
| flaper87 | nikhil_k: great | 14:23 |
| * flaper87 got one more vote | 14:23 | |
| flaper87 | :P | 14:23 |
| nikhil_k | flaper87: However, at the same time, it would be nice to keep this well documented and explanatory of what's moving where. Now that we've the code shared between glance and the glance_store lib, it might make it more difficult to deal with the 'secret creds stored in the location string' issue. | 14:24 |
| nikhil_k | Although, this seems like a better outcome to the codebase. | 14:24 |
| nikhil_k | the transition would be pain | 14:24 |
| *** krykowski has joined #openstack-glance | 14:25 | |
| flaper87 | nikhil_k: agreed on both, having it well documented and it being painful. | 14:25 |
| flaper87 | I wanted to work on a draft patch but I haven't done so because I'm unsure what the best approach is here | 14:26 |
| flaper87 | :/ | 14:26 |
| *** peristeri has joined #openstack-glance | 14:27 | |
| nikhil_k | flaper87: if you don't get to it until after k2 :P, I can pair with you on that ;) | 14:28 |
| nikhil_k | basically, wish to keep different business use cases consistent. This might be one of the area easily prone to break things different way. | 14:29 |
| nikhil_k | areas* | 14:29 |
| *** esheffield has joined #openstack-glance | 14:29 | |
| stevelle | nikhil_k: that credentials issue seems like a pretty major thorn | 14:29 |
| flaper87 | nikhil_k: yeah. one more thing that concerns me is that I think we should do it for K because of the API version bump | 14:30 |
| flaper87 | I'd like to avoid another version bump on L | 14:30 |
| flaper87 | :/ | 14:30 |
| nikhil_k | flaper87: that sounds good (doing in K) as long as we get involvement from different groups using this. | 14:31 |
| nikhil_k | a spec or email loop might help to move things faster! | 14:31 |
| nikhil_k | s/or/and | 14:31 |
| flaper87 | nikhil_k: totally | 14:31 |
| flaper87 | I know for sure nova is using this | 14:32 |
| flaper87 | but it's hidden behind locations | 14:32 |
| nikhil_k | ironic most likely | 14:32 |
| nikhil_k | yeah | 14:32 |
| flaper87 | as in, they used to use direct_url before | 14:32 |
| flaper87 | now it's converted into a location | 14:32 |
| *** haomai___ has quit IRC | 14:36 | |
| *** takedakn has joined #openstack-glance | 14:38 | |
| *** takedakn has quit IRC | 14:38 | |
| openstackgerrit | Kamil Rykowski proposed openstack/glance: Notifications for metadefinition resources https://review.openstack.org/148546 | 14:45 |
| *** thangp has joined #openstack-glance | 14:46 | |
| openstackgerrit | Kamil Rykowski proposed openstack/glance: Notifications for metadefinition resources https://review.openstack.org/148546 | 14:48 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:50 | |
| kragniz | patches welcome, y'alls: https://github.com/kragniz/openstack-glance-logo | 14:51 |
| sigmavirus24 | flaper87: I'm +1 as well provided a deprecation path and spec | 14:53 |
| sigmavirus24 | Also flaper87 as far as I know, the only way to mark an attr as deprecated is in the description | 14:53 |
| sigmavirus24 | kragniz that's awesome | 14:53 |
| sigmavirus24 | Meeting in 7, yes? | 14:53 |
| kragniz | sigmavirus24: thanks! | 14:53 |
| kragniz | sigmavirus24: yeah | 14:53 |
| flaper87 | sigmavirus24: that's my concern. no one reads the freaking description | 14:54 |
| flaper87 | :( | 14:54 |
| sigmavirus24 | flaper87: yeah. I don't know another way to do it to be honest. json-schema isn't really designed for evolving APIs just for describing the current state of the API | 14:55 |
| *** krykowski has quit IRC | 14:55 | |
| *** rwsu has joined #openstack-glance | 14:56 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Extend images cli v2 with new sorting syntax https://review.openstack.org/148981 | 14:58 |
| *** krykowski has joined #openstack-glance | 15:00 | |
| *** lakshmiS has joined #openstack-glance | 15:00 | |
| *** nellysmitt has quit IRC | 15:05 | |
| *** ericpeterson has joined #openstack-glance | 15:06 | |
| *** zz_sabari is now known as sabari | 15:08 | |
| *** hemanthm has quit IRC | 15:08 | |
| *** hemanth_ has joined #openstack-glance | 15:08 | |
| *** ankit_ag has quit IRC | 15:09 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Extend images cli v2 with new sorting syntax https://review.openstack.org/148981 | 15:09 |
| openstackgerrit | Michal Dulko proposed openstack/python-glanceclient: Add unit tests for log_curl_request https://review.openstack.org/105464 | 15:11 |
| openstackgerrit | Michal Dulko proposed openstack/python-glanceclient: Unit tests covering missing username or password https://review.openstack.org/116894 | 15:12 |
| *** bkopilov has joined #openstack-glance | 15:23 | |
| *** pkoniszewski has joined #openstack-glance | 15:25 | |
| *** vijendar has joined #openstack-glance | 15:28 | |
| *** sabari is now known as zz_sabari | 15:34 | |
| *** MattMan has joined #openstack-glance | 15:36 | |
| *** zz_jgrimm is now known as jgrimm | 15:38 | |
| *** pkoniszewski has quit IRC | 15:40 | |
| sigmavirus24 | ativelkov: bytearrays are mutable, why are you consistently creating new ones? It seems contrary to the point | 15:43 |
| ativelkov | I use bytearray there mostly for result.extend(chunk) | 15:44 |
| ativelkov | at other locations it is actually insignificant from performance point of view | 15:45 |
| *** pkoniszewski has joined #openstack-glance | 15:46 | |
| sigmavirus24 | read will probably be called a lot though. Creating the bytearrays so often could lead to performance drag with garbage collection, especially given how frequently it seems to be discarded | 15:46 |
| ativelkov | sigmavirus24: which of them do you mean particularly? self.buffer? | 15:47 |
| sigmavirus24 | Yeah | 15:48 |
| ativelkov | Well, it is replaced only when a new chunk is returned from the underlying iterator | 15:49 |
| ativelkov | In our case the size of this chunk is usually 16Mb, while the read is being called with 8K length paramter | 15:49 |
| ativelkov | So, it will be 1 bytearray recreation per 2048 calls | 15:50 |
| *** pkoniszewski has quit IRC | 15:50 | |
| openstackgerrit | Chris St. Pierre proposed openstack/glance-specs: JSONify glance-scrubber queue files https://review.openstack.org/145223 | 15:54 |
| ativelkov | But if you think that this is important, I may replace 'self.buffer = self.iterator.next()' with 'self.buffer[:] = self.iterator.next()' and similar to the clearing of it at appropriate locations | 15:55 |
| *** lakshmiS has quit IRC | 15:58 | |
| ativelkov | However, I just feel that it will be faster to replace self.buffer with a regular bytestring instead of bytearrays: we actually not modify it, so there is no sense in mutable datastructure for it | 15:59 |
| *** nlevinki has quit IRC | 16:06 | |
| sigmavirus24 | ativelkov: yeah it will be faster to replace it | 16:07 |
| ativelkov | sigmavirus24: will do it right now | 16:08 |
| sigmavirus24 | ativelkov: That does alleviate my cocners | 16:08 |
| sigmavirus24 | ativelkov: no sorry, don't do the [:] | 16:08 |
| ativelkov | I mean, will replace with regular string | 16:08 |
| sigmavirus24 | that's terrible performance on lists at least so I suspect the same is true for bytearrays. ativelkov you've convinced me the current state of affairs is fine | 16:08 |
| ativelkov | [:] is much slower, as it does the copy | 16:08 |
| sigmavirus24 | once every 2048 calls seems fine | 16:08 |
| *** tshefi has quit IRC | 16:09 | |
| sigmavirus24 | (sorry for not being clearer) | 16:09 |
| sigmavirus24 | you don't need to update that part of your patchset. I haven't finished reviewing the rest of it though (currently in meeting hell) | 16:09 |
| *** krykowski has quit IRC | 16:09 | |
| ativelkov | sigmavirus24: ok, thanks for reviewing :) | 16:10 |
| sigmavirus24 | My pleasure | 16:10 |
| *** ericpeterson has quit IRC | 16:12 | |
| *** ericpeterson has joined #openstack-glance | 16:13 | |
| *** krykowski has joined #openstack-glance | 16:14 | |
| *** ericpeterson has quit IRC | 16:17 | |
| *** julim_ has quit IRC | 16:19 | |
| *** jaypipes has joined #openstack-glance | 16:21 | |
| *** ericpeterson has joined #openstack-glance | 16:22 | |
| flaper87 | The changes-since filter is no longer available on v2. | 16:25 |
| flaper87 | sight | 16:25 |
| *** groen693 has quit IRC | 16:27 | |
| *** zz_sabari is now known as sabari | 16:32 | |
| *** sabari is now known as zz_sabari | 16:34 | |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: Artifacts Domain https://review.openstack.org/132898 | 16:42 |
| sigmavirus24 | flaper87: should it be? | 16:45 |
| openstackgerrit | Nikolay Starodubtsev proposed openstack/glance: Remove optparse from glance-replicator https://review.openstack.org/129864 | 16:47 |
| flaper87 | sigmavirus24: nope, I'm just unsure what's the v2 way to do that, TBH! | 16:48 |
| flaper87 | nikhil_k: kragniz zhiyan ^ | 16:48 |
| flaper87 | ideas ? | 16:48 |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: Artifacts API https://review.openstack.org/136629 | 16:48 |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: Artifacts Domain https://review.openstack.org/132898 | 16:48 |
| nikhil_k | flaper87: What's the nova status on that filter do you know? | 16:49 |
| flaper87 | nikhil_k: tbh, it's a tempest test | 16:49 |
| flaper87 | :P | 16:49 |
| flaper87 | nova just proxies it | 16:49 |
| flaper87 | and it doesn't use it | 16:49 |
| flaper87 | but before proposing a patch to remove that test | 16:49 |
| flaper87 | I'm curious to know if there's a new way to do it | 16:50 |
| nikhil_k | flaper87: I meant what does Nova DB-api do with that filter? | 16:50 |
| flaper87 | nikhil_k: mmh, nothing. | 16:50 |
| flaper87 | not sure I follow the question, tbh | 16:50 |
| flaper87 | :?/ | 16:50 |
| *** ozialien_ has joined #openstack-glance | 16:50 | |
| nikhil_k | flaper87: the changes-since filter is a way for the user to look at the inventory that has been most actively used | 16:51 |
| flaper87 | nikhil_k: yup, that I know | 16:51 |
| nikhil_k | that inventory item can be an image or a VM | 16:51 |
| flaper87 | and we don't have support for it in v2 | 16:51 |
| nikhil_k | so, for images we've deprecated this | 16:51 |
| nikhil_k | what about the VM? | 16:51 |
| flaper87 | ah, no idea | 16:51 |
| flaper87 | :P | 16:51 |
| flaper87 | that's nova's problem | 16:51 |
| flaper87 | hahahaha | 16:51 |
| * flaper87 jokes | 16:51 | |
| nikhil_k | Because, that might be one parity check for v1->v2 upgrade | 16:52 |
| flaper87 | I'm not even sure I want to know the answer to that | 16:52 |
| flaper87 | :P | 16:52 |
| flaper87 | so, let the git grep enlighten us | 16:52 |
| flaper87 | they're still using it for vms (at least it exists in v1,2,3,4,5,65,6,1K) | 16:53 |
| flaper87 | this is interesting | 16:53 |
| flaper87 | damnit | 16:53 |
| *** marcusvrn_ is now known as marcusvrn | 16:57 | |
| *** groen692 has joined #openstack-glance | 16:59 | |
| *** buzztroll has joined #openstack-glance | 16:59 | |
| *** krykowski has quit IRC | 17:01 | |
| *** groen692 has quit IRC | 17:01 | |
| * nikhil_k afk for 2 hours | 17:03 | |
| flaper87 | nikhil_k: how dare you going afk for 2 hours? :0 | 17:09 |
| flaper87 | >.> | 17:09 |
| openstackgerrit | Jesse J. Cook proposed openstack/glance-specs: Spec to replace snet config with endpoint config https://review.openstack.org/146972 | 17:20 |
| *** zz_sabari is now known as sabari | 17:27 | |
| *** aix has quit IRC | 17:27 | |
| *** julim has joined #openstack-glance | 17:36 | |
| *** belmoreira has quit IRC | 17:41 | |
| *** Longgeek has quit IRC | 17:43 | |
| openstackgerrit | Louis Taylor proposed openstack/glance: Move to graduated oslo.log module https://review.openstack.org/149344 | 17:48 |
| *** david-lyle has joined #openstack-glance | 17:50 | |
| *** pdb has quit IRC | 17:56 | |
| *** belmoreira has joined #openstack-glance | 17:58 | |
| *** nellysmitt has joined #openstack-glance | 18:00 | |
| *** sabari is now known as zz_sabari | 18:02 | |
| *** nellysmitt has quit IRC | 18:04 | |
| *** marcusvrn has quit IRC | 18:12 | |
| *** nellysmitt has joined #openstack-glance | 18:17 | |
| *** harlowja_away is now known as harlowja | 18:24 | |
| openstackgerrit | Mike Fedosin proposed openstack/glance: Artifacts API https://review.openstack.org/136629 | 18:36 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Declarative definitions of type-specific Artifact properties https://review.openstack.org/119174 | 18:44 |
| *** nellysmitt has quit IRC | 18:47 | |
| *** ericpeterson has quit IRC | 18:51 | |
| *** markus_z has quit IRC | 18:53 | |
| *** zz_sabari is now known as sabari | 18:56 | |
| *** ericpeterson has joined #openstack-glance | 18:57 | |
| *** jistr has quit IRC | 18:58 | |
| *** nellysmitt has joined #openstack-glance | 19:42 | |
| *** flwang1 has joined #openstack-glance | 19:50 | |
| *** flwang1 has quit IRC | 19:51 | |
| *** flwang2 has joined #openstack-glance | 19:51 | |
| *** ericpeterson has quit IRC | 20:14 | |
| *** sabari is now known as zz_sabari | 20:23 | |
| *** hemanth has joined #openstack-glance | 20:28 | |
| *** hemanth has quit IRC | 20:28 | |
| sigmavirus24 | nikhil_k: https://review.openstack.org/#/c/149387/ done =P | 20:30 |
| *** junhongl has quit IRC | 20:35 | |
| *** junhongl has joined #openstack-glance | 20:37 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 20:40 | |
| nikhil_k | sigmavirus24_awa: cool, thanks. Are you working on icehouce too? | 20:41 |
| *** lifeless has quit IRC | 20:43 | |
| *** lifeless has joined #openstack-glance | 20:51 | |
| *** belmoreira has quit IRC | 20:54 | |
| *** belmoreira has joined #openstack-glance | 20:54 | |
| *** belmoreira has quit IRC | 20:59 | |
| *** Longgeek has joined #openstack-glance | 21:02 | |
| *** zz_sabari is now known as sabari | 21:05 | |
| *** Longgeek has quit IRC | 21:09 | |
| *** TravT_ has quit IRC | 21:09 | |
| nikhil_k | sigmavirus24_awa: ativelkov : ok, icehouse one -? https://review.openstack.org/#/c/149397/ | 21:27 |
| *** ozialien_ has quit IRC | 21:31 | |
| *** TravT has joined #openstack-glance | 21:34 | |
| *** fifieldt__ has quit IRC | 21:37 | |
| *** fifieldt__ has joined #openstack-glance | 21:38 | |
| *** flwang2 has quit IRC | 21:51 | |
| openstackgerrit | Travis Tripp proposed openstack/glance-specs: Software Metadata Definitions https://review.openstack.org/148004 | 21:56 |
| *** jaypipes has quit IRC | 21:59 | |
| *** peristeri has quit IRC | 21:59 | |
| *** flwang1 has joined #openstack-glance | 22:14 | |
| nikhil_k | glance.tests.unit.v1.test_api.TestGlanceAPI.test_delete_during_image_upload_by_normal_user | 22:16 |
| nikhil_k | and | 22:16 |
| nikhil_k | glance.tests.unit.v1.test_api.TestGlanceAPI.test_delete_during_image_upload_by_admin | 22:16 |
| nikhil_k | fail intermittenly on the patch, hmm :/ | 22:16 |
| *** thangp has quit IRC | 22:25 | |
| *** julim has quit IRC | 22:41 | |
| *** Longgeek has joined #openstack-glance | 22:55 | |
| *** jamielennox|away is now known as jamielennox | 23:01 | |
| *** Longgeek has quit IRC | 23:01 | |
| *** jaypipes has joined #openstack-glance | 23:06 | |
| *** ozialien has joined #openstack-glance | 23:10 | |
| *** jaypipes has quit IRC | 23:10 | |
| *** vijendar has quit IRC | 23:11 | |
| *** TravT has quit IRC | 23:14 | |
| *** TravT has joined #openstack-glance | 23:15 | |
| *** jaypipes has joined #openstack-glance | 23:23 | |
| *** jasondotstar has joined #openstack-glance | 23:34 | |
| *** echevemaster has joined #openstack-glance | 23:46 | |
| *** jeblair has joined #openstack-glance | 23:55 | |
| *** david-lyle is now known as david-lyle_afk | 23:56 | |
| *** ChanServ changes topic to "Discussion of OpenStack Glance | logs at http://eavesdrop.openstack.org/irclogs/%23openstack-glance/" | 23:56 | |
| *** chlong has joined #openstack-glance | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!