| *** buzztroll has joined #openstack-glance | 00:26 | |
| *** buzztroll has quit IRC | 00:37 | |
| *** smc7 has joined #openstack-glance | 00:41 | |
| *** smc7 has quit IRC | 00:47 | |
| openstackgerrit | Sabari proposed openstack/glance: Update config and docs for multiple datastores support https://review.openstack.org/161970 | 00:54 |
|---|---|---|
| *** mikedillion has joined #openstack-glance | 01:03 | |
| *** mikedillion has quit IRC | 01:05 | |
| *** mikedillion has joined #openstack-glance | 01:06 | |
| *** aix has joined #openstack-glance | 01:16 | |
| *** rwsu is now known as rwsu-afk | 01:17 | |
| *** r-daneel has quit IRC | 01:31 | |
| *** mikedillion has quit IRC | 02:02 | |
| *** annashen has joined #openstack-glance | 02:07 | |
| *** flwang has quit IRC | 02:10 | |
| *** annashen has quit IRC | 02:12 | |
| *** haomaiwang has joined #openstack-glance | 02:13 | |
| *** flwang1 has joined #openstack-glance | 02:15 | |
| *** vilobhmm2 has quit IRC | 02:23 | |
| *** vilobhmm1 has joined #openstack-glance | 02:23 | |
| *** ericpeterson has joined #openstack-glance | 02:42 | |
| *** ericpeterson has quit IRC | 02:48 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/glance: Remove use of contextlib.nested https://review.openstack.org/161649 | 02:48 |
| openstackgerrit | Wayne Okuma proposed openstack/python-glanceclient: Support for Metadata Definition Catalog for Tags https://review.openstack.org/138612 | 02:52 |
| *** vilobhmm1 has quit IRC | 02:55 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/glance: Remove use of contextlib.nested https://review.openstack.org/161649 | 02:56 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/glance: Remove use of contextlib.nested https://review.openstack.org/161649 | 03:37 |
| *** david-lyle is now known as david-lyle_afk | 03:44 | |
| *** harlowja is now known as harlowja_away | 03:57 | |
| *** julim has quit IRC | 03:57 | |
| *** vilobhmm has joined #openstack-glance | 03:58 | |
| *** takedakn has joined #openstack-glance | 04:04 | |
| *** Longgeek has joined #openstack-glance | 04:08 | |
| *** takedakn has quit IRC | 04:08 | |
| *** mriedem_away has quit IRC | 04:10 | |
| *** wayne__ has left #openstack-glance | 04:11 | |
| *** mriedem has joined #openstack-glance | 04:11 | |
| *** smc7 has joined #openstack-glance | 04:19 | |
| *** madorn has quit IRC | 04:19 | |
| *** smc7 has quit IRC | 04:24 | |
| *** vilobhmm1 has joined #openstack-glance | 05:01 | |
| *** vilobhmm has quit IRC | 05:04 | |
| *** DCWilliams_VA has joined #openstack-glance | 05:09 | |
| *** DCWilliams_VA has quit IRC | 05:13 | |
| *** vilobhmm1 has quit IRC | 05:27 | |
| openstackgerrit | Lakshmi N Sampath proposed openstack/glance-specs: Catalog Index Service https://review.openstack.org/138051 | 05:30 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Imported Translations from Transifex https://review.openstack.org/158570 | 06:02 |
| *** anik80 has joined #openstack-glance | 06:14 | |
| anik80 | goodmorning all :) | 06:14 |
| *** Longgeek has quit IRC | 06:39 | |
| *** groen692 has joined #openstack-glance | 06:50 | |
| *** vilobhmm has joined #openstack-glance | 06:54 | |
| *** jamielennox is now known as jamielennox|away | 07:16 | |
| *** pkoniszewski has joined #openstack-glance | 07:16 | |
| *** Longgeek has joined #openstack-glance | 07:19 | |
| *** vilobhmm has quit IRC | 07:39 | |
| *** Longgeek has quit IRC | 07:43 | |
| *** mriedem has quit IRC | 07:47 | |
| *** mriedem has joined #openstack-glance | 07:47 | |
| *** ParsectiX has joined #openstack-glance | 07:49 | |
| *** smc7 has joined #openstack-glance | 07:57 | |
| *** zul has quit IRC | 08:01 | |
| *** markus_z has joined #openstack-glance | 08:01 | |
| *** chlong has quit IRC | 08:11 | |
| *** zul has joined #openstack-glance | 08:14 | |
| *** buzztroll has joined #openstack-glance | 08:17 | |
| *** krtaylor has quit IRC | 08:22 | |
| *** krtaylor has joined #openstack-glance | 08:35 | |
| *** jgrimm is now known as zz_jgrimm | 08:49 | |
| ivasilevskaya | anik80, morning) | 08:54 |
| *** krtaylor has quit IRC | 08:54 | |
| *** jistr has joined #openstack-glance | 09:02 | |
| *** krtaylor has joined #openstack-glance | 09:10 | |
| *** pdb has joined #openstack-glance | 09:18 | |
| ParsectiX | καλημέρα = good morning :) | 09:26 |
| *** Longgeek has joined #openstack-glance | 09:32 | |
| *** jistr is now known as jistr|biab | 09:36 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: SemVer utility to store object versions in DB https://review.openstack.org/107676 | 09:50 |
| *** MattMan has quit IRC | 10:02 | |
| *** Longgeek has quit IRC | 10:06 | |
| *** MattMan has joined #openstack-glance | 10:14 | |
| *** Longgeek has joined #openstack-glance | 10:14 | |
| openstackgerrit | Kamil Rykowski proposed openstack/glance: Notifications for metadefinition resources https://review.openstack.org/148546 | 10:14 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Declarative definitions of Artifact Types https://review.openstack.org/119174 | 10:16 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Declarative definitions of Artifact Types https://review.openstack.org/119174 | 10:19 |
| *** TravT has quit IRC | 10:30 | |
| *** smc7 has quit IRC | 10:32 | |
| *** jistr|biab is now known as jistr | 10:39 | |
| *** TravT has joined #openstack-glance | 10:57 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Declarative definitions of Artifact Types https://review.openstack.org/119174 | 11:04 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/glance: Remove use of contextlib.nested https://review.openstack.org/161649 | 11:08 |
| *** haomaiwang has quit IRC | 11:09 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Artifacts Repository - DB https://review.openstack.org/115998 | 11:10 |
| *** chlong has joined #openstack-glance | 11:22 | |
| *** nellysmitt has joined #openstack-glance | 11:25 | |
| *** buzztroll has quit IRC | 11:33 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/glance: Remove use of contextlib.nested https://review.openstack.org/161649 | 11:41 |
| openstackgerrit | Pawel Koniszewski proposed openstack/glance: Provide a way to upgrade metadata definitions https://review.openstack.org/159532 | 11:44 |
| *** flaper87 has quit IRC | 11:47 | |
| *** flaper87 has joined #openstack-glance | 11:47 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Database layer for Artifact Repository https://review.openstack.org/115998 | 11:57 |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: Artifacts plugin loader https://review.openstack.org/134300 | 11:59 |
| *** buzztroll has joined #openstack-glance | 12:00 | |
| *** buzztroll has quit IRC | 12:02 | |
| *** buzztroll has joined #openstack-glance | 12:02 | |
| *** buzztrol_ has joined #openstack-glance | 12:04 | |
| *** buzztroll has quit IRC | 12:04 | |
| *** buzztroll has joined #openstack-glance | 12:05 | |
| *** buzztrol_ has quit IRC | 12:05 | |
| *** buzztroll has quit IRC | 12:07 | |
| *** buzztroll has joined #openstack-glance | 12:07 | |
| *** buzztrol_ has joined #openstack-glance | 12:09 | |
| *** buzztroll has quit IRC | 12:09 | |
| *** buzztrol_ 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 | |
| *** buzztrol_ has quit IRC | 12:14 | |
| *** buzztroll has joined #openstack-glance | 12:15 | |
| *** buzztrol_ has joined #openstack-glance | 12:16 | |
| *** buzztroll has quit IRC | 12:16 | |
| *** buzztrol_ has quit IRC | 12:18 | |
| *** buzztroll has joined #openstack-glance | 12:18 | |
| *** smc7 has joined #openstack-glance | 12:18 | |
| *** buzztroll has quit IRC | 12:20 | |
| *** buzztroll has joined #openstack-glance | 12:20 | |
| *** buzztrol_ has joined #openstack-glance | 12:22 | |
| *** buzztroll has quit IRC | 12:22 | |
| *** smc7 has quit IRC | 12:23 | |
| *** buzztrol_ has quit IRC | 12:23 | |
| *** buzztroll has joined #openstack-glance | 12:24 | |
| *** buzztroll has quit IRC | 12:25 | |
| anik80 | <ivasilevskaya> hi..u there? | 12:25 |
| *** buzztroll has joined #openstack-glance | 12:26 | |
| *** buzztrol_ has joined #openstack-glance | 12:27 | |
| *** buzztrol_ has quit IRC | 12:29 | |
| *** buzztro__ has joined #openstack-glance | 12:29 | |
| *** buzztroll has quit IRC | 12:29 | |
| *** buzztro__ has quit IRC | 12:31 | |
| *** buzztroll has joined #openstack-glance | 12:31 | |
| *** buzztroll has quit IRC | 12:32 | |
| *** buzztroll has joined #openstack-glance | 12:33 | |
| *** buzztroll has quit IRC | 12:34 | |
| *** buzztroll has joined #openstack-glance | 12:35 | |
| *** buzztroll has quit IRC | 12:36 | |
| *** buzztroll has joined #openstack-glance | 12:36 | |
| *** aix has quit IRC | 12:38 | |
| *** buzztroll has quit IRC | 12:38 | |
| *** buzztroll has joined #openstack-glance | 12:38 | |
| *** buzztroll has quit IRC | 12:40 | |
| *** buzztroll has joined #openstack-glance | 12:40 | |
| *** buzztrol_ has joined #openstack-glance | 12:42 | |
| *** buzztroll has quit IRC | 12:42 | |
| *** buzztrol_ has quit IRC | 12:43 | |
| *** buzztroll has joined #openstack-glance | 12:44 | |
| *** buzztrol_ has joined #openstack-glance | 12:45 | |
| *** buzztroll has quit IRC | 12:45 | |
| *** buzztrol_ has quit IRC | 12:47 | |
| *** buzztroll has joined #openstack-glance | 12:47 | |
| *** buzztroll has quit IRC | 12:49 | |
| *** buzztrol_ has joined #openstack-glance | 12:49 | |
| *** buzztrol_ has quit IRC | 12:51 | |
| *** buzztroll has joined #openstack-glance | 12:51 | |
| pkoniszewski | nikhil_k: any thoughts on my comment? | 12:52 |
| *** buzztroll has quit IRC | 12:52 | |
| *** buzztroll has joined #openstack-glance | 12:53 | |
| *** buzztroll has quit IRC | 12:54 | |
| *** buzztroll has joined #openstack-glance | 12:55 | |
| *** buzztroll has quit IRC | 12:56 | |
| *** buzztroll has joined #openstack-glance | 12:56 | |
| *** buzztroll has quit IRC | 12:58 | |
| *** buzztroll has joined #openstack-glance | 12:58 | |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Remove deprecated option db_enforce_mysql_charset https://review.openstack.org/159611 | 12:59 |
| *** buzztroll has quit IRC | 13:00 | |
| *** buzztroll has joined #openstack-glance | 13:00 | |
| *** buzztroll has quit IRC | 13:01 | |
| *** buzztroll has joined #openstack-glance | 13:02 | |
| *** buzztroll has quit IRC | 13:03 | |
| *** buzztroll has joined #openstack-glance | 13:04 | |
| *** DCWilliams_VA has joined #openstack-glance | 13:04 | |
| *** buzztroll has quit IRC | 13:05 | |
| *** buzztroll has joined #openstack-glance | 13:05 | |
| *** humble_ has joined #openstack-glance | 13:06 | |
| *** buzztroll has quit IRC | 13:07 | |
| *** buzztroll has joined #openstack-glance | 13:07 | |
| *** buzztroll has quit IRC | 13:09 | |
| *** buzztroll has joined #openstack-glance | 13:09 | |
| *** buzztroll has quit IRC | 13:10 | |
| *** buzztroll has joined #openstack-glance | 13:11 | |
| *** buzztroll has quit IRC | 13:12 | |
| *** buzztroll has joined #openstack-glance | 13:13 | |
| *** buzztroll has quit IRC | 13:14 | |
| *** buzztroll has joined #openstack-glance | 13:15 | |
| *** buzztroll has quit IRC | 13:16 | |
| *** buzztroll has joined #openstack-glance | 13:16 | |
| *** buzztroll has quit IRC | 13:18 | |
| *** buzztroll has joined #openstack-glance | 13:18 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Artifact Plugins Loader https://review.openstack.org/134300 | 13:19 |
| *** buzztroll has quit IRC | 13:20 | |
| *** buzztroll has joined #openstack-glance | 13:20 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Artifact Plugins Loader https://review.openstack.org/134300 | 13:21 |
| *** buzztrol_ has joined #openstack-glance | 13:22 | |
| *** buzztroll has quit IRC | 13:22 | |
| *** buzztroll has joined #openstack-glance | 13:24 | |
| *** buzztrol_ has quit IRC | 13:24 | |
| *** buzztroll has quit IRC | 13:25 | |
| *** buzztroll has joined #openstack-glance | 13:25 | |
| *** buzztroll has quit IRC | 13:27 | |
| *** buzztroll has joined #openstack-glance | 13:27 | |
| *** buzztroll has quit IRC | 13:29 | |
| *** buzztroll has joined #openstack-glance | 13:29 | |
| *** aix has joined #openstack-glance | 13:30 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: A mixin for jsonpatch requests validation https://review.openstack.org/148588 | 13:30 |
| *** buzztroll has quit IRC | 13:31 | |
| *** buzztroll has joined #openstack-glance | 13:31 | |
| *** buzztrol_ has joined #openstack-glance | 13:33 | |
| *** buzztroll has quit IRC | 13:33 | |
| *** stpierre has joined #openstack-glance | 13:35 | |
| *** buzztrol_ has quit IRC | 13:35 | |
| *** buzztroll has joined #openstack-glance | 13:35 | |
| *** buzztrol_ has joined #openstack-glance | 13:36 | |
| *** buzztroll has quit IRC | 13:36 | |
| *** buzztroll has joined #openstack-glance | 13:38 | |
| *** buzztrol_ has quit IRC | 13:38 | |
| *** buzztroll has quit IRC | 13:40 | |
| *** buzztrol_ has joined #openstack-glance | 13:40 | |
| *** buzztrol_ has quit IRC | 13:41 | |
| *** buzztroll has joined #openstack-glance | 13:42 | |
| *** buzztrol_ has joined #openstack-glance | 13:44 | |
| *** buzztroll has quit IRC | 13:44 | |
| *** buzztrol_ has quit IRC | 13:45 | |
| *** buzztroll has joined #openstack-glance | 13:45 | |
| *** buzztroll has quit IRC | 13:47 | |
| *** buzztroll has joined #openstack-glance | 13:47 | |
| *** buzztroll has quit IRC | 13:49 | |
| *** buzztroll has joined #openstack-glance | 13:49 | |
| *** buzztrol_ has joined #openstack-glance | 13:51 | |
| *** buzztroll has quit IRC | 13:51 | |
| *** julim has joined #openstack-glance | 13:52 | |
| *** buzztrol_ has quit IRC | 13:52 | |
| *** buzztroll has joined #openstack-glance | 13:53 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 13:54 | |
| *** buzztroll has quit IRC | 13:54 | |
| *** buzztroll has joined #openstack-glance | 13:55 | |
| *** buzztroll has quit IRC | 13:56 | |
| *** buzztroll has joined #openstack-glance | 13:56 | |
| *** DCWilliams_VA has quit IRC | 13:57 | |
| *** buzztroll has quit IRC | 13:58 | |
| *** buzztroll has joined #openstack-glance | 13:58 | |
| *** annegentle has joined #openstack-glance | 13:59 | |
| *** buzztroll has quit IRC | 14:00 | |
| *** buzztroll has joined #openstack-glance | 14:00 | |
| *** annegentle has quit IRC | 14:00 | |
| *** annegentle has joined #openstack-glance | 14:01 | |
| *** buzztroll has quit IRC | 14:01 | |
| *** buzztroll has joined #openstack-glance | 14:02 | |
| openstackgerrit | Louis Taylor proposed openstack/python-glanceclient: v2: read limit for list from --limit in shell https://review.openstack.org/162161 | 14:03 |
| *** buzztroll has quit IRC | 14:03 | |
| *** buzztroll has joined #openstack-glance | 14:04 | |
| nikhil_k | pkoniszewski: link? | 14:04 |
| *** Longgeek has quit IRC | 14:05 | |
| *** fifieldt_ has quit IRC | 14:05 | |
| *** buzztroll has quit IRC | 14:05 | |
| *** buzztroll has joined #openstack-glance | 14:05 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Domain layer for Artifact Repository https://review.openstack.org/132898 | 14:06 |
| *** zz_jgrimm is now known as jgrimm | 14:07 | |
| *** buzztroll has quit IRC | 14:07 | |
| *** smc7 has joined #openstack-glance | 14:07 | |
| *** buzztroll has joined #openstack-glance | 14:07 | |
| *** buzztrol_ has joined #openstack-glance | 14:09 | |
| *** buzztroll has quit IRC | 14:09 | |
| openstackgerrit | Kamil Rykowski proposed openstack/glance: Notifications for metadefinition resources https://review.openstack.org/148546 | 14:11 |
| *** buzztrol_ has quit IRC | 14:11 | |
| *** buzztroll has joined #openstack-glance | 14:11 | |
| *** smc7 has quit IRC | 14:11 | |
| *** buzztroll has quit IRC | 14:12 | |
| *** buzztroll has joined #openstack-glance | 14:13 | |
| *** buzztroll has quit IRC | 14:14 | |
| *** buzztroll has joined #openstack-glance | 14:15 | |
| *** buzztroll has quit IRC | 14:16 | |
| *** buzztroll has joined #openstack-glance | 14:16 | |
| *** markus_z has quit IRC | 14:17 | |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Glance Image Introspection https://review.openstack.org/156249 | 14:18 |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Basic support for image conversion https://review.openstack.org/159129 | 14:18 |
| *** buzztroll has quit IRC | 14:18 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: A mixin for jsonpatch requests validation https://review.openstack.org/148588 | 14:18 |
| *** buzztroll has joined #openstack-glance | 14:18 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Domain layer for Artifact Repository https://review.openstack.org/132898 | 14:19 |
| *** buzztroll has quit IRC | 14:20 | |
| *** buzztroll has joined #openstack-glance | 14:20 | |
| *** Longgeek has joined #openstack-glance | 14:20 | |
| *** buzztroll has quit IRC | 14:22 | |
| *** buzztroll has joined #openstack-glance | 14:22 | |
| *** buzztroll has quit IRC | 14:23 | |
| *** buzztroll has joined #openstack-glance | 14:24 | |
| nikhil_k | ativelkov: https://review.openstack.org/#/c/107676/ | 14:24 |
| nikhil_k | ativelkov: mfedosin : would be good to get that merged today | 14:24 |
| ativelkov | nikhil_k: yup, working on that | 14:24 |
| mfedosin | yep | 14:25 |
| nikhil_k | ativelkov: added some comments | 14:25 |
| ativelkov | I am now traversing the whole commit chain | 14:25 |
| nikhil_k | cool | 14:25 |
| ativelkov | It turns out that working for a month without zuul checks really leads to tons of trouble :) | 14:25 |
| *** buzztroll has quit IRC | 14:25 | |
| *** buzztrol_ has joined #openstack-glance | 14:26 | |
| *** buzztrol_ has quit IRC | 14:27 | |
| *** buzztroll has joined #openstack-glance | 14:27 | |
| nikhil_k | heh, | 14:27 |
| kragniz | ativelkov: heh, good old zuul | 14:28 |
| nikhil_k | ativelkov: not a big deal, I was just suggesting that it seems a small enough change to be merged today. if it doesn't make it, that's cool | 14:28 |
| ativelkov | nikhil_k: I'll fix them, sure | 14:28 |
| *** peristeri has joined #openstack-glance | 14:29 | |
| *** buzztroll has quit IRC | 14:29 | |
| *** buzztroll has joined #openstack-glance | 14:29 | |
| ativelkov | nikhil_k: just some thoughts on your comments: semver commit is a generic one, it is needed by Artifacts but there are no direct artifact stuff there. So, InvalidVersion is just a generic exception which may be thrown by semver code - it is not mandatory artifact-related, anyone who uses semver_db may get it | 14:31 |
| *** buzztroll has quit IRC | 14:31 | |
| ativelkov | So, I don't really think we should rename it | 14:31 |
| *** buzztroll has joined #openstack-glance | 14:31 | |
| *** annegentle is now known as superanne | 14:31 | |
| nikhil_k | ativelkov: hm, does it apply to API versions? | 14:31 |
| nikhil_k | ativelkov: may be you can rename it to "DataEntityVersion.." | 14:32 |
| ativelkov | nikhil_k: well, currently it does not, as nobody else is using semver_db except artifacts. But in fact, semver_db is just a generic utility, so its exception should be generic as well, imho | 14:32 |
| nikhil_k | ativelkov: not a big deal, I'm okay with it | 14:33 |
| *** thangp has joined #openstack-glance | 14:33 | |
| nikhil_k | if someone gets confused they can change it | 14:33 |
| ativelkov | nikhil_k: and I didn't get your comment about str() conversion for longs - why do we need it there? | 14:33 |
| nikhil_k | ativelkov: not needed but good to have, long() is printed with a trailing L (in python) | 14:34 |
| *** chlong has quit IRC | 14:34 | |
| ativelkov | ah, got it - you mean, str() the items of the output tuple? | 14:34 |
| nikhil_k | http://stackoverflow.com/questions/799434/print-long-integers-in-python | 14:35 |
| nikhil_k | ativelkov: yes, (sorry should have be clearer) | 14:35 |
| ativelkov | got it. Will fix now | 14:36 |
| *** buzztroll has quit IRC | 14:36 | |
| ativelkov | BTW, its not a big deal to them, but didn't we agree that we do not stop for non-critical issues but just log such comments to address after the whole thing has landed? | 14:37 |
| nikhil_k | ativelkov: sure, once you state that you've noted them and proposed to fix in the later PS will +2 | 14:38 |
| nikhil_k | ativelkov: that's for bookkeeping. I've no objections when to fix these :) | 14:38 |
| nikhil_k | good to have all of this out there on the review | 14:38 |
| ativelkov | We don't have any reviews for dependent patches yet, so I'll fix them now. I just don't want to do another round of rebases after we get some comments there, as we loose them as soon as rebase happens | 14:40 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/glance: Switch to latest oslo-incubator https://review.openstack.org/161907 | 14:40 |
| ativelkov | BTW, I've created an etherpad to track all such comments at a single page. It just has two first commits now, but I'll add others tonight: https://etherpad.openstack.org/p/Artifacts_Reviews | 14:42 |
| nikhil_k | ativelkov: thanks! updated https://etherpad.openstack.org/p/kilo-glance-k3 with it | 14:46 |
| *** thangp_ has joined #openstack-glance | 14:48 | |
| ativelkov | nikhil_k: regarding your comment about imports: I thought that third-party should go before local imports, shouldn't they? | 14:49 |
| ativelkov | https://www.python.org/dev/peps/pep-0008/#id17 | 14:49 |
| ativelkov | https://www.python.org/dev/peps/pep-0008/#imports | 14:49 |
| nikhil_k | ativelkov: oh yes, I forgot sorry. first are the standard ones not in-tree ones.. my bad | 14:50 |
| *** vijendar has quit IRC | 14:50 | |
| *** thangp has quit IRC | 14:50 | |
| *** boris-42 has quit IRC | 14:52 | |
| *** ericpeterson has joined #openstack-glance | 14:58 | |
| *** ParsectiX has quit IRC | 14:58 | |
| *** sgotliv has quit IRC | 15:00 | |
| *** sgotliv has joined #openstack-glance | 15:00 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: SemVer utility to store object versions in DB https://review.openstack.org/107676 | 15:04 |
| dhellmann | hi, folks, who manages releases for glance-store? | 15:04 |
| ativelkov | nikhil_k: addressed your comments https://review.openstack.org/107676 | 15:05 |
| dhellmann | I'm looking at this requirements bump that's asking for a new patch release of glance-store because it includes new features, and I wanted to find out if there was a reason for having a patch release there instead of a feature release. https://review.openstack.org/#/c/161079/1 | 15:05 |
| *** hemanth has joined #openstack-glance | 15:06 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Declarative definitions of Artifact Types https://review.openstack.org/119174 | 15:08 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: A mixin for jsonpatch requests validation https://review.openstack.org/148588 | 15:08 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Database layer for Artifact Repository https://review.openstack.org/115998 | 15:08 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Domain layer for Artifact Repository https://review.openstack.org/132898 | 15:08 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Artifact Plugins Loader https://review.openstack.org/134300 | 15:08 |
| *** aarondelp has joined #openstack-glance | 15:10 | |
| *** pkoniszewski has quit IRC | 15:12 | |
| *** ericpeterson has quit IRC | 15:15 | |
| *** smc7 has joined #openstack-glance | 15:17 | |
| *** changbl has quit IRC | 15:19 | |
| *** r-daneel has joined #openstack-glance | 15:20 | |
| openstackgerrit | Qin Zhao proposed openstack/python-glanceclient: Encode x-auth-token in http header https://review.openstack.org/131991 | 15:23 |
| nikhil_k | dhellmann: it's for a feature | 15:23 |
| *** vijendar has joined #openstack-glance | 15:24 | |
| nikhil_k | dhellmann: https://blueprints.launchpad.net/glance/+spec/store-capabilities has some info | 15:24 |
| nikhil_k | zhiyan: ^ | 15:24 |
| sigmavirus24 | nikhil_k: dhellmann is asking about the version number choice | 15:24 |
| sigmavirus24 | nikhil_k: the new features should have made it 0.2 instead of 0.1.{x + 1} | 15:24 |
| sigmavirus24 | /cc kragniz | 15:24 |
| nikhil_k | sigmavirus24: hmm, that was release | 15:25 |
| sigmavirus24 | It was released recently, yes | 15:26 |
| zhiyan | nikhil_k: do we ready to bump a mine version? I think for store cap one , a minor bump is ok | 15:26 |
| nikhil_k | it's older than the most recent | 15:26 |
| sigmavirus24 | zhiyan: it's already released, no point in doing a new release just to bump the version number imo | 15:26 |
| sigmavirus24 | I think dhellmann mostly wants to make sure everyone in openstack is on the same page | 15:27 |
| nikhil_k | too late | 15:27 |
| zhiyan | Yep | 15:27 |
| sigmavirus24 | Because this is how we break stable branches | 15:27 |
| zhiyan | So 0.1.11 is enough/ok | 15:27 |
| sigmavirus24 | yeah but it /should/ have been 0.2.0 ;) | 15:27 |
| zhiyan | Hmm | 15:27 |
| nikhil_k | dhellmann: sorry about that, much needs to be fixed about release structure | 15:27 |
| kragniz | yes, this should have been 0.2.0 | 15:28 |
| zhiyan | We can do it when l open | 15:28 |
| nikhil_k | dhellmann: we'r exploring choices for stable braches, release version/mgmt, etc. | 15:28 |
| nikhil_k | i | 15:28 |
| zhiyan | Store could go first, but glance release need to be unblocked | 15:29 |
| nikhil_k | it's a sorry state until we've figured a few things out | 15:29 |
| nikhil_k | kragniz: actually, by that logic we'r easily at 0.4+ | 15:30 |
| nikhil_k | sigmavirus24: ^ | 15:30 |
| kragniz | nikhil_k: yes | 15:30 |
| nikhil_k | multiple containers, endpoint config change etc | 15:30 |
| sigmavirus24 | nikhil_k: right well given we were at 0.1 previously, 0.2 would have been the logical successor | 15:31 |
| sigmavirus24 | Also for everyone interested: https://review.openstack.org/#/c/155072/ | 15:31 |
| nikhil_k | nice | 15:31 |
| *** ericpeterson has joined #openstack-glance | 15:31 | |
| sigmavirus24 | nikhil_k: added you to that review and the nova PTL | 15:31 |
| *** changbl has joined #openstack-glance | 15:32 | |
| nikhil_k | thanks! | 15:33 |
| mfedosin | nikhil_k, there is a first documentation part https://review.openstack.org/#/c/160441/ that describes Glance Architecture | 15:33 |
| sigmavirus24 | nikhil_k: have you reached out to the inactive cores yet? | 15:33 |
| mfedosin | is it possible to merge it before 12? | 15:33 |
| sigmavirus24 | mfedosin: all things are possible ;) | 15:34 |
| sigmavirus24 | I think we have a limited number of cores who are all focusing on the artifacts patches though | 15:34 |
| openstackgerrit | Steve McLellan proposed openstack/glance: Catalog Index Service https://review.openstack.org/138269 | 15:34 |
| kragniz | sigmavirus24: that's not artifacts specific, though | 15:35 |
| sigmavirus24 | kragniz: I realize that | 15:35 |
| mfedosin | sigmavirus24, eey :) review it, please, if you have some time | 15:35 |
| sigmavirus24 | I'm saying, it might be hard to find core-reviewers with bandwidth to approve it | 15:35 |
| kragniz | right, I just read your message again :P | 15:35 |
| sigmavirus24 | mfedosin: already on it | 15:35 |
| kragniz | mfedosin: it looks good to me | 15:36 |
| *** freerunner has quit IRC | 15:36 | |
| kragniz | smallest of nits: should it be "the glance_store library" rather than "a glance_store library"? | 15:37 |
| nikhil_k | not all new cores seem eager is all i've to say, will reply to that thread today | 15:37 |
| mfedosin | dotted line between nova and store means that "<jokke_> mfedosin: the lib is out there, Nova can use it if they want to" | 15:37 |
| kragniz | it makes it sound like there's more than one glance_store library | 15:37 |
| mfedosin | sigmavirus24, kragniz thanks - the second part about DB will be on Tuesday | 15:37 |
| * nikhil_k hides | 15:37 | |
| * kragniz goes nikhil_k hunting | 15:38 | |
| sigmavirus24 | mfedosin: Tuesday may be cutting it close. | 15:38 |
| sigmavirus24 | Perhaps just postpone that to L? | 15:38 |
| mfedosin | sigmavirus24, mb, but anyway we're going to publish it on Tuesday | 15:40 |
| mfedosin | the problem is we're not good at sphinx syntax | 15:40 |
| sigmavirus24 | mfedosin: yeah just left a comment about that | 15:40 |
| sigmavirus24 | mfedosin: you all realize you can build it locally to see what's happening, right? | 15:41 |
| sigmavirus24 | simply tox -e docs; pushd docs/build/html ; python -m SimpleHTTPServer <port> ; popd | 15:41 |
| mfedosin | sigmavirus24, of course | 15:41 |
| sigmavirus24 | mfedosin: just making sure | 15:41 |
| sigmavirus24 | Some people don't do that when reviewing docs related changes and it baffles me why they wouldn' | 15:41 |
| mfedosin | I use just sphinx-build | 15:42 |
| superanne | mfedosin: I found this super helpful: http://rst.ninjs.org/ | 15:42 |
| superanne | sigmavirus24: ^^ way easier even then running autorefresh on your SimpleHTTPServer | 15:42 |
| mfedosin | btw, without your advices it would be worse | 15:42 |
| sigmavirus24 | superanne: all you need to auto-refresh is to rebuild the docs | 15:42 |
| sigmavirus24 | i.e., open new terminal, tox -e docs and refresh | 15:42 |
| superanne | sigmavirus24: yeah I know but side by side for the win | 15:43 |
| kragniz | sigmavirus24: I just use the output of gate-glance-docs when reviewing docs :v | 15:43 |
| sigmavirus24 | Or at least that always works for me with my sphinx stuff | 15:43 |
| mfedosin | sigmavirus24, I mean the sorting spec :) thanks for that | 15:43 |
| sigmavirus24 | superanne: ah I see what you mean | 15:43 |
| sigmavirus24 | mfedosin: oh right lol | 15:43 |
| * sigmavirus24 has been writing reStructuredText for Sphinx for 3 years | 15:43 | |
| sigmavirus24 | If I can't visually lint it by now, I don't know what I'm good for | 15:43 |
| superanne | plus, inline error descriptions | 15:43 |
| sigmavirus24 | superanne: btw, Matt's been keeping me in the loop about doc8 | 15:43 |
| sigmavirus24 | It's unfortunate that the upstream rst-lint project is inactive or whatever | 15:44 |
| sigmavirus24 | I'm also a bit surprised sphinx doesn't have some sort of linting built in | 15:44 |
| superanne | sigmavirus24: care to reply to my email yesterday to the -docs mailing list? | 15:44 |
| superanne | sigmavirus24: oh they're active and responsive | 15:45 |
| sigmavirus24 | oh | 15:45 |
| superanne | sigmavirus24: it's just not meeting our current need for speed and "easy button | 15:45 |
| sigmavirus24 | superanne: Matt keeps trying to get me to write a new linter for it and I already have enough projects =P | 15:45 |
| sigmavirus24 | But I'll reply to it | 15:45 |
| superanne | sigmavirus24: hee | 15:46 |
| superanne | sigmavirus24: cool thanks | 15:46 |
| sigmavirus24 | superanne: if you look at my GitHub that's only a fraction of what I do | 15:47 |
| kragniz | sigmavirus24: look at you showing off your fancy github profile | 15:47 |
| sigmavirus24 | kragniz: it's not fancy | 15:48 |
| kragniz | :P | 15:48 |
| sigmavirus24 | It's a sign that I have a serious problem | 15:48 |
| kragniz | so no github = no problem? | 15:49 |
| sigmavirus24 | kragniz: no | 15:51 |
| sigmavirus24 | there's still bitbucket and gitlab | 15:51 |
| sigmavirus24 | no more gitorious though | 15:51 |
| kragniz | rip | 15:51 |
| sigmavirus24 | lolno | 15:51 |
| sigmavirus24 | gitorious was terribad for so long | 15:51 |
| sigmavirus24 | It took them /forever/ to upgrade to rails *2* | 15:52 |
| sigmavirus24 | And they did that right before rails 4 was released | 15:52 |
| sigmavirus24 | so much headdesk | 15:52 |
| kragniz | heh | 15:52 |
| sigmavirus24 | gitlab's been much better at it | 15:52 |
| sigmavirus24 | both are f/oss projects but gitlab seems better architected and managed | 15:52 |
| dhellmann | nikhil_k: sorry, missed your reply. sigmavirus24 is right, I'm trying to make sure we're all using version numbers consistently, so if there's a new feature the minor version should be incremented | 15:55 |
| *** superanne has quit IRC | 15:59 | |
| *** superanne has joined #openstack-glance | 16:00 | |
| *** ndonegan has joined #openstack-glance | 16:02 | |
| *** Miouge has joined #openstack-glance | 16:02 | |
| nikhil_k | dhellmann: do you need one now or can we let this request happen and then make the bump in next release? | 16:03 |
| *** humble_ has quit IRC | 16:03 | |
| nikhil_k | dhellmann: not sure what you intended for us to do atm | 16:03 |
| *** superanne has quit IRC | 16:07 | |
| *** superanne has joined #openstack-glance | 16:08 | |
| sigmavirus24 | superanne: replied and added a suggestion | 16:10 |
| dhellmann | nikhil_k: it would be better to go ahead and signal the feature addition by releasing with a good version number | 16:14 |
| nikhil_k | zhiyan: ^ | 16:15 |
| nikhil_k | dhellmann: thanks, however today is friday :) | 16:15 |
| sigmavirus24 | nikhil_k: no bad things ever happen when people release software on fridays | 16:15 |
| nikhil_k | dhellmann: I can try but it could be trixy | 16:15 |
| dhellmann | nikhil_k: if you re-tag the same version you won't cause any issues | 16:15 |
| * sigmavirus24 hides from fungi | 16:15 | |
| *** cpallares has joined #openstack-glance | 16:20 | |
| *** sabari has quit IRC | 16:20 | |
| *** sabari has joined #openstack-glance | 16:22 | |
| *** krtaylor has quit IRC | 16:23 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Artifacts API https://review.openstack.org/136629 | 16:29 |
| *** freerunner has joined #openstack-glance | 16:31 | |
| *** ParsectiX has joined #openstack-glance | 16:32 | |
| *** chlong has joined #openstack-glance | 16:33 | |
| *** krtaylor has joined #openstack-glance | 16:35 | |
| *** cpallares has quit IRC | 16:36 | |
| *** pkoniszewski has joined #openstack-glance | 16:44 | |
| *** stpierre has quit IRC | 16:44 | |
| *** stpierre has joined #openstack-glance | 16:44 | |
| *** pkoniszewski has quit IRC | 16:49 | |
| *** cpallares has joined #openstack-glance | 16:52 | |
| *** boris-42 has joined #openstack-glance | 16:53 | |
| *** buzztroll has joined #openstack-glance | 16:53 | |
| *** Miouge has quit IRC | 16:58 | |
| *** changbl has quit IRC | 17:00 | |
| *** superanne has quit IRC | 17:08 | |
| *** vilobhmm has joined #openstack-glance | 17:09 | |
| *** vilobhmm1 has joined #openstack-glance | 17:09 | |
| *** vilobhmm has quit IRC | 17:13 | |
| *** changbl has joined #openstack-glance | 17:20 | |
| *** annashen has joined #openstack-glance | 17:22 | |
| *** pkoniszewski has joined #openstack-glance | 17:22 | |
| *** vilobhmm1 has quit IRC | 17:25 | |
| *** vilobhmm has joined #openstack-glance | 17:25 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Updated from global requirements https://review.openstack.org/161386 | 17:29 |
| *** rwsu-afk is now known as rwsu | 17:33 | |
| *** jistr has quit IRC | 17:37 | |
| *** david-lyle_afk is now known as david-lyle | 17:43 | |
| *** pdb has quit IRC | 18:03 | |
| *** superanne has joined #openstack-glance | 18:04 | |
| *** annashen has quit IRC | 18:16 | |
| *** browne has joined #openstack-glance | 18:17 | |
| browne | hi folks, anyone up for an easy review? https://review.openstack.org/#/c/158576 Lots of +1s, no +2s | 18:18 |
| kragniz | browne: already +1ed! | 18:19 |
| browne | kragniz: thx! | 18:20 |
| *** superanne has quit IRC | 18:27 | |
| *** spzala has joined #openstack-glance | 18:44 | |
| *** groen692 has quit IRC | 19:00 | |
| *** superanne has joined #openstack-glance | 19:04 | |
| *** annashen has joined #openstack-glance | 19:11 | |
| *** superanne has quit IRC | 19:24 | |
| *** aix has quit IRC | 19:24 | |
| *** MattMan has quit IRC | 19:37 | |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: REST API layer for Artifact Repository https://review.openstack.org/136629 | 19:39 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: Domain layer for Artifact Repository https://review.openstack.org/132898 | 19:39 |
| openstackgerrit | Alexander Tivelkov proposed openstack/glance: REST API layer for Artifact Repository https://review.openstack.org/136629 | 19:41 |
| openstackgerrit | Merged openstack/python-glanceclient: Fix v2 image create --file documentation https://review.openstack.org/145270 | 19:49 |
| ativelkov | folks, is zuul dead or something? | 19:49 |
| *** superanne has joined #openstack-glance | 19:50 | |
| *** superanne has quit IRC | 19:51 | |
| *** superanne has joined #openstack-glance | 19:52 | |
| openstackgerrit | Merged openstack/python-glanceclient: v2: Allow upload from stdin on image-create https://review.openstack.org/145272 | 19:55 |
| *** superanne has quit IRC | 19:56 | |
| *** superanne has joined #openstack-glance | 19:57 | |
| sigmavirus24 | ativelkov: status.openstack.org/zuul, I'm not sure though, why? | 19:59 |
| nikhil_k | all: heads up, we'r having glance_store 0.2.0 release to unblock https://review.openstack.org/#/c/161079/ | 20:00 |
| sigmavirus24 | :+1: | 20:01 |
| kragniz | nikhil_k: sounds good | 20:02 |
| kragniz | nikhil_k: is that a retag of the latest release? | 20:03 |
| kragniz | or a new release | 20:03 |
| *** mriedem has quit IRC | 20:05 | |
| *** Longgeek has quit IRC | 20:06 | |
| openstackgerrit | Sabari proposed openstack/glance_store: VMware: Support Multiple Datastores https://review.openstack.org/148426 | 20:07 |
| nikhil_k | kragniz: a new one | 20:09 |
| openstackgerrit | Sabari proposed openstack/glance_store: VMware: Support Multiple Datastores https://review.openstack.org/148426 | 20:11 |
| *** mriedem has joined #openstack-glance | 20:13 | |
| *** nellysmitt has quit IRC | 20:15 | |
| openstackgerrit | Steve McLellan proposed openstack/glance: WIP Catalog indexing service documentation https://review.openstack.org/160445 | 20:25 |
| *** ericpete_ has joined #openstack-glance | 20:30 | |
| *** ericpeterson has quit IRC | 20:34 | |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Glance Image Introspection https://review.openstack.org/156249 | 20:41 |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Basic support for image conversion https://review.openstack.org/159129 | 20:41 |
| *** pkoniszewski has quit IRC | 20:41 | |
| *** smc7 has quit IRC | 20:50 | |
| nikhil_k | sigmavirus24: kragniz flaper87 : please resond to the email or better yet please send me suggestions on IRC if that will help move things faster | 21:06 |
| kragniz | nikhil_k: which email? | 21:06 |
| nikhil_k | kragniz: you'd get it soon :) | 21:07 |
| kragniz | hehe | 21:07 |
| nikhil_k | (I got it) | 21:07 |
| * sigmavirus24 sees only glance_store release | 21:07 | |
| nikhil_k | about | 21:07 |
| kragniz | irc travels faster than email | 21:07 |
| nikhil_k | [openstack-dev] [Glance] Core nominations. | 21:07 |
| sigmavirus24 | oh there it is | 21:07 |
| sigmavirus24 | ;) | 21:07 |
| kragniz | aha! | 21:07 |
| kragniz | got it now | 21:07 |
| nikhil_k | jokke_ might be watching us on the logs | 21:08 |
| nikhil_k | so you too jokke_ | 21:08 |
| * kragniz waves to the all-seeing jokke_ | 21:08 | |
| nikhil_k | that almost sounds like all-seeing self | 21:09 |
| nikhil_k | and logically makes jokke_ a prophet | 21:09 |
| kragniz | :o | 21:09 |
| *** arosen has left #openstack-glance | 21:09 | |
| sigmavirus24 | jokke_ tell unto us the fruits of our future labor? are they worth it? | 21:10 |
| * sigmavirus24 waits for the all seeing oracle, jokke_, to answer | 21:10 | |
| sigmavirus24 | Wait.. oracle... is jokke_ just a bot written in Java?! | 21:10 |
| * sigmavirus24 shouldn't have had that second pot of espresso | 21:10 | |
| kragniz | this all makes more sense now | 21:10 |
| sigmavirus24 | kragniz: all of it? | 21:11 |
| sigmavirus24 | or most of it? | 21:11 |
| kragniz | I'll do a test and see what the sun does to him | 21:11 |
| sigmavirus24 | Wait, Sun doesn't exist here. Do you have Sun? I want to be where Sun is | 21:11 |
| nikhil_k | sigmavirus24: you need to move south | 21:12 |
| sigmavirus24 | nikhil_k: there's a second meaning in there too | 21:12 |
| nikhil_k | :P | 21:13 |
| sigmavirus24 | =P | 21:13 |
| sigmavirus24 | (just in case it was too subtle) | 21:13 |
| nikhil_k | nope | 21:13 |
| *** annashen has quit IRC | 21:14 | |
| nikhil_k | oh yes, mfedosin too^ | 21:14 |
| *** annashen has joined #openstack-glance | 21:14 | |
| nikhil_k | and sigmavirus24 get brownie points! | 21:15 |
| nikhil_k | gets* | 21:15 |
| *** smc7 has joined #openstack-glance | 21:15 | |
| sigmavirus24 | nikhil_k: for what? | 21:16 |
| kragniz | nikhil_k: I'm not sure where you're going with "core-member status", but the rest sounds good | 21:16 |
| kragniz | +1 on removing old and adding new | 21:16 |
| nikhil_k | kragniz: me neither :P | 21:16 |
| nikhil_k | at least yet | 21:17 |
| kragniz | aha :P | 21:17 |
| nikhil_k | but have an idea, let's see if that can be formalized | 21:17 |
| * sigmavirus24 wonders who that Ian Cordasco fellow is that's being nominated. I don't like the sound of his name | 21:18 | |
| kragniz | me neither | 21:19 |
| kragniz | nikhil_k: I take back the +1 on adding new | 21:19 |
| kragniz | let' | 21:19 |
| kragniz | s have sigmavirus24 instead of this "Ian Cordasco" guy | 21:19 |
| kragniz | never heard of him | 21:19 |
| sigmavirus24 | kragniz: I don't need your handouts. I'm an adult | 21:20 |
| sigmavirus24 | =P | 21:20 |
| stevelle | sigmavirus24: where do we keep the spare age jokes? I used up all of mine. | 21:21 |
| nikhil_k | heh | 21:21 |
| sigmavirus24 | stevelle: the expiration date is close for those | 21:22 |
| stevelle | so they're getting old? | 21:23 |
| sigmavirus24 | bingo | 21:23 |
| *** stpierre has quit IRC | 21:27 | |
| *** julim has quit IRC | 21:27 | |
| openstackgerrit | Sabari proposed openstack/glance_store: Deprecate VMware store single datastore options https://review.openstack.org/162308 | 21:28 |
| kragniz | nikhil_k: do you have any spare cycles for a quick glanceclient review? | 21:30 |
| kragniz | (or any other core, but no one else seems to be around) | 21:30 |
| kragniz | https://review.openstack.org/#/c/161839/ | 21:31 |
| kragniz | (it's got a stuart +2 already) | 21:32 |
| openstackgerrit | Sabari proposed openstack/glance: Update config and docs for multiple datastores support https://review.openstack.org/161970 | 21:33 |
| nikhil_k | kragniz: sure | 21:33 |
| kragniz | thanks! | 21:34 |
| *** harlowja_away has quit IRC | 21:34 | |
| nikhil_k | flaper87: hey, Rackspace public cloud uses tasks on production systems | 21:40 |
| nikhil_k | so based on that I can say that the operations can be really expensive if on the API node | 21:41 |
| nikhil_k | if the tasks run targz then even a single process can be heavy handed let along 50! | 21:41 |
| nikhil_k | he does not seem to be around | 21:42 |
| nikhil_k | time to copy paste to the review ;) | 21:42 |
| * flaper87 here | 21:42 | |
| flaper87 | nikhil_k: ^ | 21:42 |
| flaper87 | mmmh, interesting | 21:42 |
| flaper87 | I didn't know you guys were using this in production. That's good to know. | 21:43 |
| flaper87 | Also, note that these are 50 greenthreads that we're talking about | 21:43 |
| nikhil_k | flaper87: yeah | 21:43 |
| nikhil_k | flaper87: basically, we've one sema for 2 types of tasks | 21:43 |
| nikhil_k | and max threads are allowed are 10 | 21:44 |
| *** superanne has quit IRC | 21:44 | |
| nikhil_k | gah, too many typos today :(( | 21:44 |
| flaper87 | mmh, in that case, we should probably take that number back to 2, or pump it to 4 | 21:44 |
| nikhil_k | flaper87: so, you can queue up 10 or 100 tasks or something but one runs at a time | 21:45 |
| nikhil_k | but run one at a time* | 21:45 |
| flaper87 | You answered my exact comment. I mean, the fact that w/o actual data, we can't get that number right | 21:45 |
| nikhil_k | cool | 21:45 |
| nikhil_k | flaper87: glad to be of help | 21:45 |
| nikhil_k | flaper87: do you need me to say more there? | 21:45 |
| flaper87 | nikhil_k: yeah, but I guess we could actually get to 10, the scheduler should know what to do. What do you think? | 21:46 |
| nikhil_k | flaper87: sure, 10 seems decent for now | 21:46 |
| flaper87 | nikhil_k: it'd be cool if you could mention this there just as reference. | 21:46 |
| nikhil_k | we can always change defaults right? | 21:46 |
| nikhil_k | sure | 21:46 |
| flaper87 | yeah | 21:46 |
| kragniz | "we can always change defaults right?" - nikhil_k, 2015 | 21:48 |
| sigmavirus24 | kragniz: 2014-2015. RIP | 21:48 |
| sigmavirus24 | =P | 21:48 |
| mtreinish | flaper87: qq would you want to give me a hand and migrate the glanceclient cli tests out of tempest? | 21:50 |
| mtreinish | it should be pretty straightforward, jogo pushed up some instructions to the ML on how to do it: http://lists.openstack.org/pipermail/openstack-dev/2015-February/057458.html | 21:50 |
| sigmavirus24 | flaper87: it's a trap ;) | 21:51 |
| flaper87 | kragniz: LOL | 21:52 |
| sigmavirus24 | lol | 21:52 |
| flaper87 | mtreinish: mmmh, mmh. I'll take a look and let you know. I need to make sure I've enough time to commit to this | 21:52 |
| *** superanne has joined #openstack-glance | 21:52 | |
| nikhil_k | flaper87: commented | 21:52 |
| mtreinish | heh, well I was going to bug sigmavirus24, but he already volunteered to help try and fix some skip bugs in tempest. | 21:53 |
| sigmavirus24 | mtreinish: so I'm overcommitted as it is | 21:54 |
| sigmavirus24 | /but/ I'll see what I can finagle next week | 21:54 |
| kragniz | mtreinish: I might be able to pick it up if neither sigmavirus24 or flaper87 want to do it | 21:54 |
| sigmavirus24 | kragniz: go hog wild =P | 21:54 |
| flaper87 | kragniz: it's yours | 21:54 |
| kragniz | lol | 21:54 |
| flaper87 | sigmavirus24: LOL | 21:54 |
| sigmavirus24 | ^5 flaper87 | 21:54 |
| sigmavirus24 | flaper87: we two overcommit everything | 21:54 |
| * kragniz fell for that one | 21:54 | |
| mtreinish | kragniz: cool thanks :) | 21:54 |
| sigmavirus24 | Also it's not technically my day job to work on openstack | 21:55 |
| sigmavirus24 | Or at least not on openstack/* projects | 21:55 |
| sigmavirus24 | So I can't technically do code during the day | 21:55 |
| flaper87 | sigmavirus24: if you're really able to do that, I'll admire you forever | 21:55 |
| flaper87 | I mean, saying "this is not my day to code on X and not doing it" | 21:56 |
| flaper87 | that's impossible for me | 21:56 |
| nikhil_k | hemanth: what your # suggestion on the default workers https://review.openstack.org/#/c/159078/? options we poised are 2, 4, 10 | 21:56 |
| sigmavirus24 | flaper87: I do it at night | 21:56 |
| flaper87 | I'm a butterfly coder | 21:56 |
| sigmavirus24 | flaper87: technical 0 of my days are days to code on glance (or anything in openstack upstream) | 21:56 |
| flaper87 | Jeeeeeez, did I really say that? | 21:56 |
| flaper87 | :P | 21:56 |
| flaper87 | nikhil_k: ^^^^ | 21:57 |
| sigmavirus24 | flaper87: so you use emacs? | 21:57 |
| flaper87 | DO NOT ADD HIM TO CORE! | 21:57 |
| flaper87 | sigmavirus24: yeah, I've been using it lately | 21:57 |
| sigmavirus24 | Yeah seriously. I don't know what everyone's thinking | 21:57 |
| flaper87 | I used to be a vimer | 21:57 |
| sigmavirus24 | flaper87: https://xkcd.com/378/ | 21:57 |
| kragniz | flaper87: how are you finding it? | 21:58 |
| kragniz | I'm still a vimmer, but emacs looks shiny | 21:58 |
| sigmavirus24 | to be fair, evil mode is the best vim replacement of any editor/IDE I've heard | 21:58 |
| sigmavirus24 | So emacs has that going for it | 21:59 |
| hemanth | nikhil_k, I'd say 4 from from what I have seen | 21:59 |
| nikhil_k | use sublime and be happy with all ;) | 21:59 |
| nikhil_k | flaper87: that's the magic number "4" ^ | 21:59 |
| nikhil_k | thanks hemanth | 21:59 |
| flaper87 | kragniz: I'm liking it, TBH, I switched to emacs because I was having lots of trouble with vim | 21:59 |
| flaper87 | and I gotta admit Rust-lang support for emacs is better | 22:00 |
| flaper87 | :P | 22:00 |
| flaper87 | and it's compile mode and tramp-mode | 22:00 |
| flaper87 | I mean, tramp-mode is really fantastic | 22:00 |
| *** ericpete_ has quit IRC | 22:00 | |
| nikhil_k | http://www.sublimetext.com/ | 22:01 |
| sigmavirus24 | nikhil_k: nice try =P | 22:01 |
| sigmavirus24 | flaper87: I mostly use emacs for clojure =P. nrepl is amazing | 22:01 |
| nikhil_k | :P | 22:02 |
| flaper87 | nikhil_k: fixing the default | 22:02 |
| flaper87 | don't go away | 22:02 |
| *** ericpeterson has joined #openstack-glance | 22:02 | |
| nikhil_k | go. go. and away. | 22:02 |
| nikhil_k | s/go/up/gc | 22:02 |
| nikhil_k | We'd add new cores on Mon in any case :P | 22:03 |
| nikhil_k | (I'm serious) | 22:04 |
| sigmavirus24 | Wait is there a minimum age requirement for cores? =P kragniz might be in trouble | 22:04 |
| nikhil_k | oh right, how could we forget | 22:04 |
| flaper87 | sigmavirus24: it's 60 | 22:05 |
| nikhil_k | ! | 22:05 |
| sigmavirus24 | flaper87: thank god I maintain requests and flake8. At least I look the age | 22:05 |
| *** jgrimm is now known as zz_jgrimm | 22:06 | |
| nikhil_k | jokke_ is really monitoring the logs | 22:06 |
| nikhil_k | or flaper87 has telepathic abilities | 22:06 |
| *** ericpeterson has quit IRC | 22:07 | |
| flaper87 | Or I have been breaking his !#@$#@!$#@ asking for reviews | 22:07 |
| flaper87 | :P | 22:07 |
| nikhil_k | flaper87: we're disappointed that you love us less than him :P | 22:08 |
| flaper87 | nikhil_k: last time I gave you guys priority, sigmavirus24 left 21 comments on my patch | 22:09 |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Glance Image Introspection https://review.openstack.org/156249 | 22:09 |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Remove the eventlet executor https://review.openstack.org/159078 | 22:09 |
| openstackgerrit | Flavio Percoco proposed openstack/glance: Basic support for image conversion https://review.openstack.org/159129 | 22:09 |
| flaper87 | nikhil_k: ^ | 22:09 |
| flaper87 | :P | 22:09 |
| nikhil_k | just 21 | 22:09 |
| nikhil_k | huh, why are we addiing him to core now? | 22:09 |
| nikhil_k | sigmavirus24: needs to buckle up | 22:09 |
| sigmavirus24 | damn sorry | 22:10 |
| sigmavirus24 | 42 is clearly the minimum number of comments to leave on one of flaper87's changes | 22:10 |
| sigmavirus24 | flaper87: incoming =P | 22:10 |
| nikhil_k | flaper87: we need to wait jenkins :P | 22:11 |
| flaper87 | sigmavirus24: f#@$@# u | 22:11 |
| flaper87 | :) | 22:11 |
| flaper87 | nikhil_k: no no no, +2+A, short circuit | 22:11 |
| sigmavirus24 | nikhil_k: does jenkins make the zuul go round or do the zuuls on the jenkins go round and round? | 22:12 |
| flaper87 | FWIW, the gate will run the check anyway | 22:12 |
| sigmavirus24 | flaper87: lolol no | 22:12 |
| flaper87 | if there's something wrong, it won't go through | 22:12 |
| sigmavirus24 | flaper87: must wait for monday when I can -2 =P | 22:12 |
| flaper87 | there's no point on waiting for jenkins if the patch will be approved anyway | 22:12 |
| flaper87 | nikhil_k: +2 NOW!! Haven't you read what sigmavirus24 just said? | 22:12 |
| flaper87 | jeeeeeez :P | 22:12 |
| sigmavirus24 | lmao | 22:12 |
| nikhil_k | flaper87: based on the default value hemanth might -2 too :P | 22:13 |
| sigmavirus24 | flaper87: should I bring a bullet proof vest to PyCon? | 22:13 |
| sigmavirus24 | nikhil_k: does hemanth do glance deploys for public cloud by hand? | 22:14 |
| nikhil_k | may be | 22:14 |
| nikhil_k | flaper87: sigmavirus24 got brownie points for replying to the email on ML so, I can't really +2 against his intentions now :P | 22:15 |
| flaper87 | sigmavirus24: that won't help, I'm black belt :) | 22:15 |
| * nikhil_k goes for chai time | 22:17 | |
| flaper87 | nikhil_k: you really didn't +2 my patch :( | 22:18 |
| flaper87 | really, jenkins runs checks if they haven't be run | 22:18 |
| sigmavirus24 | flaper87: give the chai time to work | 22:18 |
| flaper87 | just short circuit it, PLEASE! | 22:18 |
| sigmavirus24 | nikhil_k: will warm up to it | 22:18 |
| * flaper87 has no freaking idea what chai time is | 22:18 | |
| flaper87 | but if that's going to get me a +2, then fine | 22:19 |
| sigmavirus24 | tea | 22:19 |
| nikhil_k | ok ok | 22:20 |
| nikhil_k | fine | 22:20 |
| nikhil_k | fine | 22:20 |
| nikhil_k | <fine> | 22:20 |
| flaper87 | fine in italian means "end" | 22:20 |
| sigmavirus24 | flaper87: fine in english, when repeated so often means nikhil_k is at the *end* of his rope | 22:21 |
| nikhil_k | lol | 22:21 |
| nikhil_k | I'm just kidding | 22:21 |
| sigmavirus24 | nikhil_k: sure you were | 22:21 |
| sigmavirus24 | ;) | 22:21 |
| nikhil_k | buono | 22:22 |
| nikhil_k | buono | 22:22 |
| nikhil_k | <buono> | 22:22 |
| flaper87 | :P | 22:22 |
| flaper87 | w000t | 22:22 |
| *** r-daneel has quit IRC | 22:23 | |
| *** chlong has quit IRC | 22:24 | |
| sigmavirus24 | hold on | 22:24 |
| sigmavirus24 | I have to -1 this | 22:24 |
| sigmavirus24 | =P | 22:24 |
| * flaper87 takes his flamethrower | 22:25 | |
| *** thangp_ has quit IRC | 22:27 | |
| kragniz | sigmavirus24: low blow on the minimum age requirement for glance cores :P | 22:28 |
| flaper87 | kk, gtg.... you guys better merge that patch >.> | 22:28 |
| sigmavirus24 | kragniz: I'd talk about a minimum height requirement but I doubt anyone's shorter than 183cm | 22:28 |
| flaper87 | have a great w/e everyone | 22:28 |
| sigmavirus24 | flaper87: did you see my CR? | 22:28 |
| sigmavirus24 | flaper87: you too | 22:28 |
| kragniz | flaper87: have a good weekend too! | 22:29 |
| openstackgerrit | Merged openstack/python-glanceclient: Fix leaking sockets after v2 list operation https://review.openstack.org/161839 | 22:37 |
| openstackgerrit | Merged openstack/python-glanceclient: v2: read limit for list from --limit in shell https://review.openstack.org/162161 | 22:38 |
| kragniz | two for the price of one! | 22:38 |
| nikhil_k | kragniz: u following world cup? | 22:39 |
| * nikhil_k thinks kragniz is from uk | 22:40 | |
| kragniz | nope, sorry | 22:40 |
| * kragniz isn't a football person | 22:40 | |
| nikhil_k | cricket :P | 22:40 |
| kragniz | lol | 22:40 |
| sabari | nikhil_k: I added a mention for it. And you are the first to trigger it. | 22:40 |
| nikhil_k | football was last year | 22:41 |
| kragniz | nikhil_k: that's how much I know about it | 22:41 |
| nikhil_k | :) | 22:41 |
| *** julim has joined #openstack-glance | 22:41 | |
| nikhil_k | sabari: not sure, what u meant | 22:41 |
| kragniz | nikhil_k: I'm guessing you are? How is it? | 22:41 |
| nikhil_k | ok for now | 22:41 |
| *** aarondelp has quit IRC | 22:42 | |
| nikhil_k | ireland is doing gr8 | 22:42 |
| nikhil_k | england isn't | 22:42 |
| sabari | nikhil_k: I meant I added an IRC mention keyword for 'world cup' and no one ever spoke about it in any of my channels. | 22:43 |
| sabari | you were the first! | 22:43 |
| kragniz | sabari: haha | 22:43 |
| nikhil_k | ah really! niicee | 22:43 |
| kragniz | I have a mention for "global variable" | 22:44 |
| sabari | yeah, I just wanted to see if anyone follows around here :) | 22:44 |
| nikhil_k | kragniz: are u from ireland then? | 22:44 |
| nikhil_k | stuart mentioned u go to uk uni | 22:44 |
| kragniz | nikhil_k: from england, but living in ireland at the moment | 22:44 |
| sabari | kragniz: lol oslo.config must be a busy channel for you then :) | 22:45 |
| kragniz | nikhil_k: I go to university in wales | 22:45 |
| sabari | kragniz: There is a match today for Ireland. | 22:45 |
| kragniz | sabari: it is, I should turn it off | 22:46 |
| kragniz | or mute #openstack-oslo | 22:46 |
| nikhil_k | kragniz: ah | 22:46 |
| kragniz | nikhil_k: too many countries! | 22:46 |
| nikhil_k | heh, | 22:47 |
| nikhil_k | uk is confusing that way | 22:47 |
| kragniz | it is | 22:47 |
| *** mriedem has quit IRC | 22:47 | |
| nikhil_k | sabari: 2vd fav team? | 22:50 |
| sabari | kragniz: the funny thing is the Ireland match starts probably 3:30 AM your local time. | 22:50 |
| sabari | I have no idea why one would schedule a match like that. | 22:50 |
| * nikhil_k is typing while having chai | 22:50 | |
| sabari | nikhil_k: I am hoping for NZ to win this time. | 22:51 |
| nikhil_k | :) | 22:51 |
| kragniz | sabari: haha | 22:51 |
| kragniz | sabari: where is the match played? | 22:52 |
| sabari | just to be fair and it wouldn't hurt to lose to someone who never have won before ;) | 22:52 |
| nikhil_k | aus, nz | 22:52 |
| sigmavirus24 | :3 | 22:53 |
| kragniz | ah, okay. If it had been over here, it would be rather odd to start at 3:30 am! | 22:53 |
| kragniz | sigmavirus24: are you sending regular cat faces over a public irc channel? | 22:54 |
| *** peristeri has quit IRC | 22:54 | |
| kragniz | sigmavirus24: would you do such a thing? | 22:54 |
| sigmavirus24 | kragniz: you lied about "global variable" didn't you? | 22:54 |
| kragniz | I won't see it if it's in a channel number > 65 | 22:55 |
| kragniz | I have too many channels for my irssi config :( | 22:55 |
| sigmavirus24 | LMAO | 22:58 |
| sigmavirus24 | kragniz: well I tested it in #openstack-api | 22:58 |
| sigmavirus24 | So | 22:58 |
| kragniz | yeah, I saw when I looked in my status pane | 22:58 |
| sigmavirus24 | So instead of pinging kragniz I'll just say global variable from now on | 22:59 |
| *** superanne has quit IRC | 22:59 | |
| kragniz | ;_; | 23:00 |
| *** ParsectiX has quit IRC | 23:00 | |
| *** ParsectiX has joined #openstack-glance | 23:00 | |
| sigmavirus24 | If I say "global variable" 3 times while looking into IRC, a kragniz will appear | 23:01 |
| nikhil_k | global variable | 23:01 |
| sigmavirus24 | global variable | 23:01 |
| kragniz | :3 | 23:01 |
| sigmavirus24 | heh | 23:01 |
| *** r-daneel has joined #openstack-glance | 23:02 | |
| kragniz | sigmavirus24: you prompted me to expand my view of my channels | 23:02 |
| *** ericpeterson has joined #openstack-glance | 23:03 | |
| nikhil_k | flaper87: replied to the ML, the universe has unfolded now! | 23:05 |
| *** r-daneel_ has joined #openstack-glance | 23:06 | |
| *** r-daneel has quit IRC | 23:07 | |
| sigmavirus24 | I like flaper87's "Oh dear" | 23:07 |
| *** ericpeterson has quit IRC | 23:07 | |
| kragniz | is the oh dear to the joke, or to the fact sigmavirus24 will be able to -2 flaper87's patches? | 23:08 |
| * kragniz ponders | 23:08 | |
| *** r-daneel_ has quit IRC | 23:10 | |
| *** r-daneel_ has joined #openstack-glance | 23:10 | |
| sigmavirus24 | kragniz: yes | 23:12 |
| kragniz | sigmavirus24: thanks for clearing that up | 23:13 |
| sigmavirus24 | kragniz: always a pleasure | 23:13 |
| sigmavirus24 | nikhil_k: are there docs for adding a driver to glance_store? | 23:16 |
| *** ericpeterson has joined #openstack-glance | 23:17 | |
| nikhil_k | sigmavirus24: none | 23:18 |
| nikhil_k | spec should do it | 23:18 |
| sigmavirus24 | There's currently debate over whether a driver module should be importable to retrieve options from it or not | 23:19 |
| *** vijendar has quit IRC | 23:19 | |
| nikhil_k | oh | 23:19 |
| sigmavirus24 | to which I'm debating whether or not we should rethink how options are loaded in glance_store or just formalize whether modules should be importable | 23:19 |
| sigmavirus24 | there are tradeoffs to each | 23:19 |
| nikhil_k | right | 23:19 |
| sigmavirus24 | And I don't particularly care but I'd like to see it standardized | 23:19 |
| nikhil_k | I think the main concern was to not install deps unless you needed the driver | 23:20 |
| sigmavirus24 | So this argument can stop since mreidem and I figured out two problems due to this already and now we're trying to apply our experiences to a third change | 23:20 |
| sigmavirus24 | nikhil_k: exactly | 23:20 |
| sigmavirus24 | Without it installed though (if the imports aren't conditional) then you can't load the opts for that store | 23:20 |
| sigmavirus24 | Because you can't import the module | 23:20 |
| sigmavirus24 | And stevedore will ignore it | 23:20 |
| nikhil_k | yeah | 23:20 |
| sigmavirus24 | So | 23:20 |
| sigmavirus24 | Either the modules need to be mandated to be importable with just requirements.txt | 23:21 |
| sigmavirus24 | Or we need to separate opts from drivers | 23:21 |
| sigmavirus24 | I think the latter is sub-optimal | 23:21 |
| sigmavirus24 | The former means raising a BadStoreConfiguration when the store is actually created (or rather when we attempt to create it) | 23:21 |
| kragniz | sigmavirus24: the way options are handled should be changed, anyway | 23:21 |
| kragniz | this is ugly: https://github.com/openstack/glance/blob/master/glance/cmd/api.py#L75-77 | 23:21 |
| sigmavirus24 | kragniz: agreed | 23:22 |
| sigmavirus24 | Ugh | 23:22 |
| sigmavirus24 | I feel strongly about this to write the spec but unless junhongl helps me with the replicator HTTPS issue, I don't think I'll have the ability to commit to implementing another spec =P | 23:22 |
| nikhil_k | heh | 23:23 |
| *** ericpeterson has quit IRC | 23:23 | |
| sigmavirus24 | oh it's after 5 | 23:23 |
| sigmavirus24 | time to work on policy | 23:23 |
| nikhil_k | policy? | 23:23 |
| nikhil_k | oh | 23:23 |
| *** annashen has quit IRC | 23:23 | |
| nikhil_k | work on as in keep up to it? | 23:23 |
| sigmavirus24 | so two things with policy | 23:24 |
| sigmavirus24 | 1. my spec | 23:24 |
| sigmavirus24 | 2. graduating to oslo.policy | 23:24 |
| nikhil_k | btw, I'm not against the gloabl reqs for deps (for drivers) | 23:24 |
| nikhil_k | ah | 23:24 |
| kragniz | sigmavirus24: hasn't it been graduated already? :P | 23:24 |
| sigmavirus24 | nikhil_k: that would only aggravate our downstream redistributors | 23:24 |
| sigmavirus24 | kragniz: =_= | 23:24 |
| nikhil_k | the issue with that though is there would be unnecessary pair for those not using it | 23:25 |
| nikhil_k | so, changing configs seems better | 23:25 |
| nikhil_k | style for using configs | 23:25 |
| sabari | sigmavirus24: I am thinking that while creating the store we should check with stevedore's extension manager. | 23:27 |
| sabari | If the store is not loaded we could right away throw the BadStoreConfiguration. This way we do not instantiate the Store class and run into import error. | 23:27 |
| nikhil_k | ok, I think we can do this outside of kilo | 23:28 |
| nikhil_k | so, may be I will look away for now and see if we can get some of those reviews done later tonight/tomorrow | 23:28 |
| sigmavirus24 | sabari: I think that's more work than is necessary | 23:30 |
| sabari | sigmavirus24: wouldn't it be similar to what backend.register_opts does today ? | 23:35 |
| sabari | I am ok with both catching import errors in driver or checking generically. | 23:35 |
| sabari | Just trying to understand the expected behavior :). | 23:36 |
| sigmavirus24 | sabari: so if I understand correctly, other drivers are doing the conditional import and raising the "expected" exceptions when the store is created. | 23:39 |
| sigmavirus24 | _load_store handles a RuntimeError while _load_stores handles BadStoreConfiguration | 23:40 |
| sigmavirus24 | (or at least that's what mreidem and I figured out the other night) | 23:40 |
| sigmavirus24 | So it makes sense for store creation to raise a BadStoreConfiguration exception with details for the user to fix it (e.g., please install oslo.vmware) | 23:40 |
| sigmavirus24 | sabari: and to be clear, I don't blame you for any of this. There should be a spec for adding a driver/maintaining it, etc. | 23:40 |
| sabari | sigmavirus24: haha, I didn't take it that way either, I was only trying to understand what is the ideal behavior. | 23:45 |
| sigmavirus24 | Yeah | 23:45 |
| sabari | I agree with you that a BadStoreConfiguration should be thrown in such cases. | 23:45 |
| sigmavirus24 | I get it | 23:45 |
| sigmavirus24 | I think the ideal behaviour is to be able to load all options for all drivers | 23:45 |
| sigmavirus24 | I think there are 1 or two drivers that still need help with that though | 23:46 |
| sabari | sigmavirus24 Ok got it. So you do want the plugin to be loaded by stevedore so that the opts are loaded. | 23:47 |
| sabari | I totally got it wrong the problem you were trying to solve :) | 23:47 |
| sigmavirus24 | Yes | 23:47 |
| sigmavirus24 | Sorry. I've been really tired lately | 23:47 |
| sigmavirus24 | I've put in at least 3, 10+ hour days | 23:47 |
| sabari | haha :) that's what the weekend is for. | 23:48 |
| sigmavirus24 | I would unbrain this weekend but kilo freeze is coming up | 23:48 |
| sigmavirus24 | heh, psychic | 23:48 |
| sabari | does that mean I can ask for CR's over the weekend ;) | 23:49 |
| sabari | jk. have a good weekend. | 23:49 |
| sigmavirus24 | I'll probably try to focus on writing code this weekend | 23:50 |
| sigmavirus24 | I'll probably be on IRC though | 23:50 |
| *** cpallares has quit IRC | 23:51 | |
| sabari | and I will fix my patch following what we discussed now :). | 23:51 |
| sabari | I can actually do it for other stores too. | 23:52 |
| *** annashen has joined #openstack-glance | 23:52 | |
| sabari | So far, I see swift store still doesn't do conditional import | 23:53 |
| sigmavirus24 | :D sabari | 23:54 |
| sigmavirus24 | yeah there's one for swift that I did | 23:54 |
| sigmavirus24 | sabari: so review that ;) | 23:54 |
| sigmavirus24 | project:openstack/glance_store status:open owner:"Ian Cordasco" | 23:54 |
| sabari | ah, I should have probably guessed after seeing your S3 patch :) | 23:54 |
| sigmavirus24 | it's sad that I can type out gerrit queries now without needing help | 23:54 |
| sabari | ^^ I think I should get rid of bookmarks to be able to do that | 23:55 |
| sabari | sigmavirus24: I only see https://review.openstack.org/#/c/161428/ | 23:57 |
| sigmavirus24 | d'oh | 23:58 |
| sigmavirus24 | See how bad my brain is on this little sleep/ | 23:58 |
| * sigmavirus24 will be back shortly | 23:59 | |
| sabari | sigmavirus24 haha, np | 23:59 |
| sabari | * stepping out too * | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!