*** TravT has quit IRC | 00:00 | |
*** buzztroll has joined #openstack-glance | 00:03 | |
*** buzztroll has quit IRC | 00:03 | |
*** jgrimm is now known as zz_jgrimm | 00:09 | |
*** nellysmitt has joined #openstack-glance | 00:09 | |
*** ericpeterson has quit IRC | 00:12 | |
*** nellysmitt has quit IRC | 00:14 | |
*** jaypipes has joined #openstack-glance | 00:17 | |
*** buzztroll has joined #openstack-glance | 00:23 | |
*** buzztroll has quit IRC | 00:24 | |
*** david-lyle is now known as david-lyle_afk | 00:31 | |
*** jaypipes has quit IRC | 00:34 | |
*** fifieldt has quit IRC | 01:00 | |
*** annashen has quit IRC | 01:21 | |
*** nellysmitt has joined #openstack-glance | 02:10 | |
*** haomaiwa_ has joined #openstack-glance | 02:10 | |
*** haomaiwa_ has quit IRC | 02:10 | |
*** haomaiwang has joined #openstack-glance | 02:11 | |
*** nellysmitt has quit IRC | 02:14 | |
openstackgerrit | hzrandd proposed openstack/glance: Modify annotation information. https://review.openstack.org/151129 | 02:27 |
---|---|---|
*** bkopilov has joined #openstack-glance | 02:30 | |
openstackgerrit | hzrandd proposed openstack/glance: Modify annotation information. https://review.openstack.org/151129 | 02:32 |
*** rwsu is now known as rwsu-afk | 02:48 | |
*** harlowja_ is now known as harlowja_away | 02:58 | |
*** wayne___ has quit IRC | 03:27 | |
*** adam_g_out is now known as adam_g | 03:29 | |
*** cpallares has joined #openstack-glance | 03:31 | |
*** buzztroll has joined #openstack-glance | 03:35 | |
*** flwang has quit IRC | 03:37 | |
*** buzztroll has quit IRC | 03:46 | |
*** mtreinish_ has joined #openstack-glance | 03:47 | |
*** annashen has joined #openstack-glance | 03:52 | |
*** flwang has joined #openstack-glance | 03:52 | |
*** takedakn has joined #openstack-glance | 03:54 | |
*** mtreinish_ is now known as mtreinish | 04:04 | |
*** annashen has quit IRC | 04:05 | |
openstackgerrit | hzrandd proposed openstack/glance: Allow users to configure blacklist of URI schemes https://review.openstack.org/150736 | 04:09 |
*** nellysmitt has joined #openstack-glance | 04:11 | |
*** spzala has quit IRC | 04:12 | |
*** flwang has quit IRC | 04:12 | |
*** nellysmitt has quit IRC | 04:15 | |
*** buzztroll has joined #openstack-glance | 04:31 | |
*** TravT has joined #openstack-glance | 04:43 | |
*** annashen has joined #openstack-glance | 05:08 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 05:11 | |
*** TravT has quit IRC | 05:11 | |
*** ozialien has joined #openstack-glance | 05:12 | |
*** annashen has quit IRC | 05:23 | |
*** annashen has joined #openstack-glance | 05:24 | |
*** buzztroll has quit IRC | 05:35 | |
*** buzztroll has joined #openstack-glance | 05:38 | |
*** buzztroll has quit IRC | 05:40 | |
*** TravT has joined #openstack-glance | 05:40 | |
*** buzztroll has joined #openstack-glance | 05:44 | |
*** TravT has quit IRC | 05:46 | |
*** echevemaster has quit IRC | 06:09 | |
*** nellysmitt has joined #openstack-glance | 06:12 | |
*** nellysmitt has quit IRC | 06:16 | |
*** annashen has quit IRC | 06:21 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Imported Translations from Transifex https://review.openstack.org/146765 | 06:22 |
*** annashen has joined #openstack-glance | 06:25 | |
*** EmilienM is now known as EmilienM|afk | 06:29 | |
*** buzztroll has quit IRC | 06:36 | |
*** buzztroll has joined #openstack-glance | 06:39 | |
*** annashen has quit IRC | 06:57 | |
*** pkoniszewski has joined #openstack-glance | 07:08 | |
openstackgerrit | Abhishek Kekane proposed openstack/glance: Eventlet green threads not released back to pool https://review.openstack.org/130839 | 07:14 |
*** belmoreira has joined #openstack-glance | 07:46 | |
*** chlong has quit IRC | 08:08 | |
*** groen692 has joined #openstack-glance | 08:08 | |
*** nellysmitt has joined #openstack-glance | 08:13 | |
*** nellysmitt has quit IRC | 08:14 | |
*** nellysmitt has joined #openstack-glance | 08:14 | |
*** takedakn has quit IRC | 08:17 | |
*** markus_z has joined #openstack-glance | 08:41 | |
*** buzztroll has quit IRC | 09:14 | |
*** MattMan has quit IRC | 09:15 | |
*** eglynn has joined #openstack-glance | 09:17 | |
*** MattMan has joined #openstack-glance | 09:24 | |
*** pdb has joined #openstack-glance | 09:31 | |
*** jistr has joined #openstack-glance | 09:31 | |
*** changbl has quit IRC | 10:01 | |
*** sgotliv has joined #openstack-glance | 10:31 | |
*** jamielennox is now known as jamielennox|away | 10:36 | |
*** chlong has joined #openstack-glance | 10:43 | |
*** jamielennox|away is now known as jamielennox | 10:44 | |
*** pkoniszewski has quit IRC | 10:46 | |
*** pkoniszewski has joined #openstack-glance | 10:46 | |
*** krykowski has joined #openstack-glance | 10:46 | |
*** nellysmitt has quit IRC | 10:50 | |
*** krykowski has quit IRC | 10:55 | |
*** jamielennox is now known as jamielennox|away | 10:56 | |
*** buzztroll has joined #openstack-glance | 11:03 | |
*** krykowski has joined #openstack-glance | 11:04 | |
*** buzztroll has quit IRC | 11:08 | |
*** krykowski has quit IRC | 11:25 | |
*** krykowski has joined #openstack-glance | 11:26 | |
*** aix has joined #openstack-glance | 11:34 | |
*** diegows has joined #openstack-glance | 11:55 | |
*** diegows has quit IRC | 11:55 | |
*** diegows has joined #openstack-glance | 11:56 | |
*** krykowski has quit IRC | 12:08 | |
*** nellysmitt has joined #openstack-glance | 12:10 | |
*** jasondotstar has quit IRC | 12:12 | |
*** krykowski has joined #openstack-glance | 12:14 | |
*** ozialien has quit IRC | 12:21 | |
*** aix has quit IRC | 12:22 | |
*** nellysmitt has quit IRC | 12:31 | |
*** buzztroll has joined #openstack-glance | 12:52 | |
*** krykowski has quit IRC | 12:56 | |
*** buzztroll has quit IRC | 12:56 | |
*** aix has joined #openstack-glance | 13:03 | |
*** haomaiwang has quit IRC | 13:15 | |
*** nellysmitt has joined #openstack-glance | 13:18 | |
*** krykowski has joined #openstack-glance | 13:23 | |
*** EmilienM|afk is now known as EmilienM | 13:28 | |
*** jyoti-ranjan has joined #openstack-glance | 13:31 | |
jyoti-ranjan | I do have one qestion related to usage of swift for Glance data store | 13:31 |
jyoti-ranjan | can I use swift which is not part of my cloud to configure it as a data store in my cloud? | 13:31 |
jyoti-ranjan | As per configuration: | 13:32 |
jyoti-ranjan | default_store = swift | 13:32 |
jyoti-ranjan | swift_store_auth_address = $KEYSTONE_PROTOCOL://$KEYSTONE_HOST:$KEYSTONE_PORT/v2.0/ | 13:32 |
jyoti-ranjan | swift_store_user = $SERVICE_TENANT_NAME:glance | 13:32 |
jyoti-ranjan | swift_store_key = $SERVICE_PASSWORD | 13:32 |
jyoti-ranjan | swift_store_create_container_on_put = True | 13:32 |
jyoti-ranjan | What is significance of swift_store_auth_address? | 13:32 |
kragniz | jyoti-ranjan: that is the keystone server the swift user auths against | 13:32 |
kragniz | ie the keystone running on the cloud the swift cluster you want to use as a data store is part of | 13:33 |
jyoti-ranjan | So, if a user is part of cloud A which implicitly means keystone A. But swift is deployed in cloud B which has keystone B. | 13:33 |
jyoti-ranjan | Can cloud A use keystone B (not of keystone A)? | 13:34 |
jyoti-ranjan | Glance is expected to run in cloud A. | 13:34 |
kragniz | this is assuming glance is part of cloud A, you'll need a swift user on cloud B and point glance at cloud B's keystone address | 13:35 |
kragniz | jyoti-ranjan: as far as I'm aware, that should work | 13:35 |
jyoti-ranjan | yes | 13:36 |
*** tnurlygayanov has quit IRC | 13:41 | |
*** changbl has joined #openstack-glance | 13:41 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:50 | |
*** jyoti-ranjan2 has joined #openstack-glance | 13:58 | |
*** J_ has joined #openstack-glance | 14:04 | |
*** krykowski has quit IRC | 14:08 | |
*** eglynn is now known as eglynn-pto | 14:09 | |
*** krykowski has joined #openstack-glance | 14:14 | |
*** spzala has joined #openstack-glance | 14:15 | |
*** jasondotstar has joined #openstack-glance | 14:15 | |
*** sgotliv has quit IRC | 14:28 | |
*** peristeri has joined #openstack-glance | 14:32 | |
*** changbl has quit IRC | 14:33 | |
*** EmilienM is now known as EmilienM|afl | 14:35 | |
*** EmilienM|afl is now known as EmilienM|afk | 14:35 | |
*** buzztroll has joined #openstack-glance | 14:40 | |
*** buzztroll has quit IRC | 14:45 | |
*** diegows has quit IRC | 14:53 | |
*** zz_jgrimm is now known as jgrimm | 14:59 | |
*** hemanth has joined #openstack-glance | 15:02 | |
*** thangp has joined #openstack-glance | 15:10 | |
*** sgotliv has joined #openstack-glance | 15:13 | |
*** nellysmitt has quit IRC | 15:18 | |
*** r-daneel has joined #openstack-glance | 15:22 | |
*** vijendar has joined #openstack-glance | 15:35 | |
*** pkoniszewski has quit IRC | 15:36 | |
*** krykowski has quit IRC | 15:41 | |
*** kragniz is now known as zingark | 15:53 | |
openstackgerrit | Ian Cordasco proposed openstack/glance: Pass a real image target to the policy enforcer https://review.openstack.org/146651 | 16:03 |
nikhil_k | flwang1: hi | 16:04 |
sigmavirus24 | morning everyone | 16:04 |
nikhil_k | morning! | 16:05 |
nikhil_k | Please forgive my slow start today | 16:05 |
nikhil_k | Back from a very long travel, close to 19hours yesterday | 16:05 |
sigmavirus24 | nikhil_k: i'll never forgive it =P | 16:05 |
sigmavirus24 | ativelkov: are you around by any chance? | 16:05 |
nikhil_k | and around 16hrs on Monday :/ | 16:05 |
ativelkov | sigmavirus24: yup, I am, but heading to office in about 15 minutes | 16:06 |
sigmavirus24 | ativelkov: ping me when you get there =) | 16:06 |
zingark | "I was a small mouse now" I do not understand | 16:06 |
zingark | sigmavirus24: do you know what hzrandd means in that review? | 16:06 |
sigmavirus24 | zingark: I assume it was a google translate error. | 16:06 |
ativelkov | sigmavirus24: ok | 16:06 |
zingark | ah, I didn't think of that | 16:07 |
*** EmilienM|afk is now known as EmilienM | 16:07 | |
sigmavirus24 | zingark: yeah I'm not bothered by it. I was confused at first but I'm pretty sure it isn't a big deal. | 16:07 |
sigmavirus24 | Also hzrandd seems to be the only person representing their company which I think is admirable | 16:07 |
* sigmavirus24 didn't realize that zingark is kragniz backwards at first | 16:08 | |
zingark | I think there are a few other qq folk on other projects | 16:08 |
sigmavirus24 | "qq"? | 16:08 |
zingark | I may be mistaken, though | 16:08 |
zingark | like twitter but chinese | 16:09 |
sigmavirus24 | hzrandd was the only person who associated with them according to Stackalytics (which is obviously not a single source of truth) | 16:09 |
zingark | okay, that's cool then! | 16:09 |
sigmavirus24 | I think hzrandd is with NetEase, not qq | 16:09 |
zingark | I am a fool | 16:09 |
zingark | qq does an email service, too | 16:09 |
sigmavirus24 | No you just don't track people's associations down as methodically as I do | 16:10 |
sigmavirus24 | =P | 16:10 |
zingark | and he has his qq mail associated with gerrit | 16:10 |
zingark | :P | 16:10 |
* sigmavirus24 could make joke about all the people with @gmail addresses but won't | 16:10 | |
nikhil_k | sigmavirus24: gunna go through the glance meeting logs, though do you know anything important for me to look into? | 16:10 |
*** belmoreira has quit IRC | 16:11 | |
sigmavirus24 | nikhil_k: we didn't really talk about much because only 3 of us showed up | 16:12 |
sigmavirus24 | flaper87: wanted to discuss changes-since but no one else showed up so we didn't discuss it | 16:12 |
sigmavirus24 | We just looked at reviews and tricked flaper87 into approving things that had +2s ;) | 16:12 |
zingark | he didn't fall for mine, though | 16:16 |
sigmavirus24 | He did catch on eventually | 16:17 |
zingark | aha, so he did! | 16:17 |
* zingark forgives flaper87 for ever doubting him | 16:17 | |
sigmavirus24 | Oh and for people who missed the channel notice, zuul stopped working around 10:30 UTC and it seems fungi has given a proper kicking and is re-enqueueing jobs according to the logs that they captured | 16:19 |
*** david-lyle_afk is now known as david-lyle | 16:19 | |
sigmavirus24 | tl;dr don't recheck unless you check Zuul to make sure your job is not in fact re-enqueued | 16:20 |
*** cpallares is now known as serallapc | 16:21 | |
*** nellysmitt has joined #openstack-glance | 16:25 | |
serallapc | sigmavirus24: say your joke, say it! | 16:27 |
sigmavirus24 | why is everyone doing nick[::-1] today? | 16:29 |
*** buzztroll has joined #openstack-glance | 16:29 | |
serallapc | sigmavirus24: It's a trend zingark started in the raqaz channel. | 16:30 |
*** TravT has joined #openstack-glance | 16:30 | |
* sigmavirus24 sighs | 16:33 | |
zingark | sigmavirus24: ti htiw og tsuj | 16:34 |
*** buzztroll has quit IRC | 16:34 | |
sigmavirus24 | on | 16:35 |
*** TravT has quit IRC | 16:35 | |
serallapc | zingark: yllej tsuj si 42surivamgis | 16:35 |
zingark | serallapc: setot | 16:36 |
*** jasondotstar has quit IRC | 16:36 | |
*** buzztroll has joined #openstack-glance | 16:39 | |
sigmavirus24 | setaogcm | 16:39 |
sigmavirus24 | setaogcm setot | 16:40 |
ativelkov | sigmavirus24: o/ | 16:42 |
sigmavirus24 | \o | 16:42 |
sigmavirus24 | So I'm wondering if we really need something as complex as SemVer when we have other standards we can use for version numbers | 16:42 |
*** jaypipes has joined #openstack-glance | 16:42 | |
sigmavirus24 | (Especially ones with nearly identical sorting practices but simpler version strings) | 16:42 |
ativelkov | well, one of the reasons is because when I was designing Artifacts I was mostly thinking about Application Distribution | 16:44 |
sigmavirus24 | ativelkov: specifically, https://www.python.org/dev/peps/pep-0440/ is a standard around versioning of applications and distribution of them | 16:44 |
*** jyoti-ranjan has quit IRC | 16:45 | |
ativelkov | as far as I know, pep-440 is even more complicated then semver | 16:45 |
sigmavirus24 | It's one that has also heavily borrowed from other languages (like Ruby) and is rooted in real world usage (where as SemVer as I've seen it is less often used as fully as you are wishing to support it) | 16:45 |
*** jyoti-ranjan has joined #openstack-glance | 16:45 | |
ativelkov | semver being its subset, I believe | 16:45 |
sigmavirus24 | ativelkov: you have it backwards | 16:45 |
sigmavirus24 | The only thing not in SemVer is epochs and those are still rather simple to support | 16:45 |
*** TravT has joined #openstack-glance | 16:46 | |
sigmavirus24 | PEP-440 is far more restrictive | 16:46 |
ativelkov | as far as I know, pbr uses a semver notation | 16:47 |
sigmavirus24 | also better defined (in my opinion) whereas SemVer is meant to encourage a greater deal of freedom that can lead to issues | 16:47 |
*** TravT has quit IRC | 16:47 | |
sigmavirus24 | ativelkov: pbr is using PEP 440 because it has to be compatible with PEP 440 | 16:47 |
sigmavirus24 | We may follow semver's release number prescriptions but pbr doesn't use the full spec of semver because it's specifically incompatible with PEP 440 | 16:48 |
sigmavirus24 | Trust me, this was an issue when pip 6.0 was first released (because pbr was generating invalid version identifiers) | 16:48 |
*** TravT has joined #openstack-glance | 16:49 | |
ativelkov | https://github.com/openstack-dev/pbr/blob/master/doc/source/semver.rst | 16:49 |
*** rwsu-afk is now known as rwsu | 16:49 | |
ativelkov | i.e. their are adding git metadata to the pre-release labels | 16:50 |
sigmavirus24 | ativelkov: they *were* they don't | 16:51 |
sigmavirus24 | ativelkov: if they did that still every gate would be horribly broken | 16:51 |
ativelkov | there is some work in progress here: https://review.openstack.org/#/c/124651/ | 16:52 |
ativelkov | anyway, we don't have to follow them for the artifacts, but still, I tried to give the developers the most possible flexibility | 16:52 |
sigmavirus24 | So here's the thing, there's still a lot of flexibility in PEP440 and it would be simple enough for us to handle efficiently | 16:54 |
sigmavirus24 | There's also https://pypi.python.org/pypi/packaging | 16:54 |
sigmavirus24 | Which will parse those versions for us easily | 16:54 |
sigmavirus24 | It's the library pip uses | 16:54 |
ativelkov | well, it is fine, but PEP4404 is pythonic standard | 16:54 |
ativelkov | 440* | 16:55 |
ativelkov | and semver is generic one | 16:55 |
sigmavirus24 | ativelkov: I'll reiterate, it's influenced by SemVer, real usage on PyPI, and the standards defined in other languages | 16:55 |
sigmavirus24 | It's a standard in Python land, but it's a standard that wasn't developed in a bubble and was made pragmatically | 16:55 |
ativelkov | well, I don't argue. | 16:56 |
ativelkov | But do we want, say, Java developers to learn it? | 16:56 |
sigmavirus24 | Java developers already learn new things while using OpenStack | 16:57 |
sigmavirus24 | Pretty much every consumer of openstack needs to learn new conventions to use it properly because so much of OpenStack already suffers from not-invented-here syndrome | 16:58 |
sigmavirus24 | I'd also argue it's easier to learn a narrowly defined specification than one so broadly defined with undefined behaviour | 16:58 |
ativelkov | Sure, but why increasing the amount of these new things? I mean, are there any reasons while we should use pep440 instead of semver? | 16:59 |
ativelkov | And what is undefined in semver? I'd rather say it is extremely simple | 16:59 |
sigmavirus24 | ativelkov: I'm going to triple check but I'm rather certain that build information's affect on sorting isn't well defined for our purported usecase | 17:00 |
sigmavirus24 | ativelkov: SemVer is something that isn't a universal standard either | 17:01 |
sigmavirus24 | Most people use a fraction of it at best and that fraction that I've seen used is wholly compatible with PEP440 | 17:01 |
ativelkov | According to semver build metadata does NOT affect the precedence | 17:01 |
sigmavirus24 | I don't think people will be learning much of a new thing | 17:01 |
sigmavirus24 | ativelkov: and anyone familiar with operating system packages will think that's ridiculous | 17:02 |
sigmavirus24 | RHEL/CentOS/Debian/Arch/etc. all rely on build metadata for finding the most recently built version of the package | 17:02 |
ativelkov | "Semantic versioning is a popular version identification scheme that is more prescriptive than this PEP regarding the significance of different elements of a release number." | 17:04 |
ativelkov | (quote from the prp440 spec) | 17:05 |
ativelkov | Anyway, I am fine with either of them | 17:05 |
ativelkov | but this still does not solve the main concern: how do we follow the right precedence in database | 17:06 |
nikhil_k | sigmavirus24: thanks, for the input! (Was in a meeting, just back) | 17:06 |
ativelkov | My solution with long+string works for semver. Didn't check if it works for pep440 | 17:06 |
*** zingark is now known as krangiz | 17:07 | |
sigmavirus24 | ativelkov: PEP440 simplifies the potential structure of a version string. The database sorting can be solved similarly, we might just need to further restrict the max value for major/minor/patch versions | 17:07 |
*** krangiz is now known as kragniz | 17:07 | |
ativelkov | it seems it is a bit more complicated | 17:08 |
ativelkov | 1.0.dev4 should be less then 1.0.dev10 | 17:08 |
ativelkov | but if we use plain ascii comparison it will not work | 17:08 |
sigmavirus24 | ativelkov: I'm not proposing we use plain ascii | 17:08 |
sigmavirus24 | In fact I think we can represent everything with the long-type | 17:08 |
*** ericpeterson has joined #openstack-glance | 17:10 | |
ativelkov | Ok, I probably have to look deeper there | 17:11 |
ativelkov | but it really looks much more complicated then semver | 17:11 |
ativelkov | too many entities | 17:12 |
ativelkov | they have this public version vs local version | 17:12 |
ativelkov | final, pre and post-releases | 17:12 |
ativelkov | dev releases | 17:12 |
ativelkov | and epochs | 17:13 |
*** annashen has joined #openstack-glance | 17:13 | |
ativelkov | 5 components of semver (where only 4 do really matter) seems much easier for me | 17:13 |
sigmavirus24 | So here's the problem I think we'll run into with semver though | 17:14 |
sigmavirus24 | We have version 1.10.11+companyA_2, 1.10.11+companyA_03, 1.10.11+companyB_3 that build metadata could really be pretty much anything and every user who sees it has to know exactly what it means to be able to understand what's being presented to them | 17:16 |
sigmavirus24 | there's a context that every user who sees that has to be able to have | 17:16 |
ativelkov | right | 17:16 |
sigmavirus24 | The visual equivalent to that's effect on 440 versions though is defined | 17:17 |
ativelkov | in 440 they have "local version" for that, right? | 17:17 |
sigmavirus24 | yeah and I'm just double checking my memory with the packaging package right now | 17:18 |
*** spzala has quit IRC | 17:18 | |
*** spzala_ has joined #openstack-glance | 17:18 | |
ativelkov | "Comparison and ordering of local versions considers each segment of the local version (divided by a . ) separately. If a segment consists entirely of ASCII digits then that section should be considered an integer for comparison purposes and if a segment contains any ASCII letters than that segment is compared lexicographically with case insensitivity" | 17:18 |
*** spzala_ is now known as spzala | 17:18 | |
ativelkov | that is exactly the same situation which semver has for their "pre-release labels" - and why I had to add string-based fields | 17:19 |
sigmavirus24 | https://gist.github.com/sigmavirus24/af5f6fb36c2d740a296a | 17:20 |
sigmavirus24 | except that it has no effect on sorting in SemVer which means we have no defined way of displaying it to the user | 17:21 |
sigmavirus24 | But yeah, that will make us have to add string fields which is what I was hoping to avoid | 17:21 |
ativelkov | http://paste.openstack.org/show/164247/ | 17:25 |
ativelkov | absolutely same result for semver | 17:25 |
ativelkov | just a bit different partitioning | 17:25 |
ativelkov | in semver 10a10 becomes pre-release tag, openstack.1 becomes build meta | 17:25 |
ativelkov | but I see more serious problem in 440 | 17:26 |
ativelkov | (which is not a real problem - it just indicates it is more powerfull - but it makes it much more complicated to store as a single long | 17:26 |
*** pdb has quit IRC | 17:26 | |
ativelkov | it has an arbitrary number of release components | 17:27 |
ativelkov | semver has just 3 of them | 17:27 |
ativelkov | major-minor-patchs | 17:27 |
ativelkov | 440 is defined as N(.N)* | 17:27 |
ativelkov | which means 1.2.3.4.5.6.7.8 | 17:28 |
ativelkov | is perfectly fine | 17:28 |
sigmavirus24 | True | 17:28 |
sigmavirus24 | hm | 17:28 |
ativelkov | so, if we want to have it stored as a single field we will have to assume some meaningful max value for number of segments and max value of each segment | 17:28 |
sigmavirus24 | version numbers are way too hard | 17:28 |
*** nellysmitt has quit IRC | 17:35 | |
*** changbl has joined #openstack-glance | 17:39 | |
*** spzala has quit IRC | 17:41 | |
*** pennerc has joined #openstack-glance | 17:42 | |
*** sgotliv has quit IRC | 17:45 | |
*** pennerc_ has joined #openstack-glance | 17:45 | |
*** jyoti_ranjan has joined #openstack-glance | 17:46 | |
*** jyoti-ranjan has quit IRC | 17:46 | |
*** jyoti-ranjan2 has quit IRC | 17:46 | |
ativelkov | sigmavirus24: just had a conversation with dims and jaypipes on that | 17:47 |
ativelkov | dims confirms that their are using semver in pbr | 17:47 |
*** pennerc has quit IRC | 17:47 | |
*** pennerc_ has quit IRC | 17:48 | |
*** pennerc has joined #openstack-glance | 17:49 | |
*** harlowja_away is now known as harlowja_ | 17:50 | |
*** jyoti_ranjan has quit IRC | 17:52 | |
*** TravT_ has joined #openstack-glance | 17:52 | |
*** TravT_ has quit IRC | 17:52 | |
*** TravT has quit IRC | 17:52 | |
*** alex_xu has quit IRC | 17:52 | |
sigmavirus24 | ativelkov: where did you have that discussion? I am certain they're incorrect. It's the reason setuptools 8 and pip 6 were blocked for quite a while | 17:53 |
sigmavirus24 | They're using at best the subset of semver that is compatible with pep440 | 17:53 |
ativelkov | Just here at the office :) | 17:54 |
sigmavirus24 | ativelkov: I know we're also using the roles defined for major/minor/patch but the version strings generated are PEP 440 strings | 17:54 |
ativelkov | we may ask dims to join us in this conversation | 17:55 |
sigmavirus24 | Previously the strings included the git SHA in a 440 incompatible way | 17:55 |
sigmavirus24 | dhellmann: is here too | 17:55 |
*** alex_xu has joined #openstack-glance | 17:55 | |
sigmavirus24 | They could be including that in the local identifier though | 17:55 |
sigmavirus24 | hm | 17:55 |
* sigmavirus24 checks published packages | 17:55 | |
ativelkov | That may be, right | 17:55 |
sigmavirus24 | There previously doing things like 0.15.0.<sha> | 17:55 |
sigmavirus24 | which broke with pip 6/setuptools 8 | 17:56 |
sigmavirus24 | They could be doing 0.15.0+<gitsha> | 17:56 |
ativelkov | Anyway, the more important thing is that artifacts should be necessary be pbr-compatible: this is thing which is quite external for openstack | 17:56 |
sigmavirus24 | Why do they need to be pbr compatible? | 17:56 |
ativelkov | and even for python | 17:56 |
*** pennerc has quit IRC | 17:56 | |
ativelkov | sorry, should NOT be | 17:56 |
sigmavirus24 | ativelkov: so here's the thing, users are defining these strings for version identifiers right? | 17:57 |
ativelkov | yes | 17:57 |
ativelkov | so this is actually part of the API | 17:57 |
sigmavirus24 | Right, but (unlike pbr) these versions aren't machine generated/internal | 17:58 |
ativelkov | some of them may be | 17:58 |
sigmavirus24 | I think I'm wondering why we're restricting users at all | 17:58 |
ativelkov | i.e. there may be a CI-CD system which publishes its artifacts in Glance | 17:58 |
*** hemanth has quit IRC | 17:58 | |
sigmavirus24 | ativelkov: sure | 17:59 |
sigmavirus24 | but these are still arbitrary version identifiers chosen by the user and eventually principally consumed by the user | 17:59 |
ativelkov | because we need to have a precedence defined | 17:59 |
*** J_ has quit IRC | 17:59 | |
ativelkov | say, there should be a qury "give me the latest version of artifact with the name=X and type=Z" | 18:00 |
ativelkov | also, there will be a "clone-and-bump-the-version" operation | 18:00 |
ativelkov | so, we have to define the logic for that unambiguously | 18:01 |
ativelkov | if the version is just some meaningless string, then we won't be able to do so | 18:01 |
sigmavirus24 | you realize then that the most practical way of doing this is by using dates, right? | 18:01 |
ativelkov | dates is just one of the way of doing that | 18:02 |
*** eglynn-pto has quit IRC | 18:02 | |
ativelkov | that's why there exist an obvious mapping of the date to semver | 18:02 |
sigmavirus24 | ativelkov: so let me ask you this | 18:02 |
*** spzala has joined #openstack-glance | 18:03 | |
sigmavirus24 | let's say there are versions 1.2.3, and 1.2.4 in the database already | 18:03 |
sigmavirus24 | the user says "clone 1.2.3 and bump the patch versioN" | 18:03 |
sigmavirus24 | what should they get? | 18:03 |
ativelkov | an exception | 18:03 |
sigmavirus24 | or will that only ever be "clone and bump the latest" | 18:03 |
sigmavirus24 | So why can't they clone 1.2.3 and fork it? | 18:04 |
ativelkov | they may, if they specify the name explicitly | 18:04 |
ativelkov | say, clone 1.2.3 and for it to 1.2.3-my-revision | 18:04 |
ativelkov | and fork* it | 18:04 |
*** harlowja_ has quit IRC | 18:04 | |
ativelkov | then it works | 18:04 |
*** julim has quit IRC | 18:05 | |
sigmavirus24 | ativelkov: some services I can think of would bump the version as 1.2.4+1 | 18:05 |
*** julim has joined #openstack-glance | 18:06 | |
*** eglynn-pto has joined #openstack-glance | 18:06 | |
sigmavirus24 | seems like something that's going to happen, raising an exception becomes user-hostile at that point | 18:06 |
sigmavirus24 | they then have to learn how glance is storing the version and providing this other functionality for them | 18:06 |
*** harlowja has joined #openstack-glance | 18:07 | |
*** annashen has quit IRC | 18:08 | |
sigmavirus24 | I'm just trying to wrap my head around all of this artifacts stuff and I totally understand the desire to sort all of this in the database. I 100% agree but I don't see the use case for sorting it in terms of latest where created_at wouldn't be as accurate if not more accurate | 18:08 |
*** mfedosin has quit IRC | 18:08 | |
*** Guest35467 has quit IRC | 18:09 | |
*** alex_xu has quit IRC | 18:09 | |
*** hemanth has joined #openstack-glance | 18:10 | |
*** alex_xu has joined #openstack-glance | 18:11 | |
*** spzala has quit IRC | 18:12 | |
ativelkov | sigmavirus24: imaging you ar eimporting arttifacts from remote source | 18:12 |
ativelkov | i.e. just uploading files with all the metadata embedded | 18:13 |
ativelkov | created_at will actually mean the timestamp of the import operation | 18:13 |
ativelkov | while version denotes the version (and so most probably the creaion order of the artifact itself) | 18:14 |
*** pennerc has joined #openstack-glance | 18:14 | |
ativelkov | not to speak that you may have 1.0 and 2.0 and then maintain both branches, so you will add 2.1 and then 1.1, so created_at order will give you 1.0, 2.0, 2.1, 1.1, which is definetely different from the version order | 18:15 |
*** markus_z has quit IRC | 18:17 | |
*** annashen has joined #openstack-glance | 18:17 | |
*** mfedosin has joined #openstack-glance | 18:20 | |
openstackgerrit | Inessa Vasilevskaya proposed openstack/glance: Artifacts API https://review.openstack.org/136629 | 18:25 |
*** plaurin_ has joined #openstack-glance | 18:28 | |
*** serallapc is now known as cpallares | 18:29 | |
ativelkov | sigmavirus24: checked that with dims: they call it "semantic version" but that is indeed pep440 | 18:29 |
plaurin_ | hello everyone. I have a question. Is there a way to use glance outside of the controller node (a different server than keystone)? And also, is the api-paste config file still in use in the juno release? | 18:30 |
nikhil_k | Yes and yes, plaurin_ | 18:32 |
plaurin_ | (I understand that it's possible configuring the endpoints for remote servers, but I failed to do so, having connection refused issues) | 18:32 |
plaurin_ | nikhil_k: Thanks. In the juno docs (for Ubuntu) there's nothing about the api-paste config file like there use to be in the previous releases. | 18:33 |
plaurin_ | http://docs.openstack.org/juno/install-guide/install/apt/content/glance-install.html | 18:34 |
plaurin_ | nothing about api-paste files like in the havana release | 18:35 |
plaurin_ | Maybe that's why I can't connect to keystone, I did not fill the api paste files. | 18:38 |
*** jaypipes has quit IRC | 18:45 | |
*** jaypipes has joined #openstack-glance | 18:49 | |
*** jaypipes is now known as leakypipes | 18:50 | |
*** MattMan has left #openstack-glance | 18:52 | |
sigmavirus24 | ativelkov: good point on the created_at bit. It's been a lot longer since I read the original Artifacts spec so I'm re-reading that now to see if that helps me figure out better what needs to be supported | 18:53 |
sigmavirus24 | ativelkov: also, yeah, you can munge 440 and SemVer together if you only use major/minor/patch and use the semantics around their meanings | 18:54 |
nikhil_k | plaurin_: hmm, thanks for bringing that up! Mind opening a bug for juno, icehouse, master. This kind of falls under glance and documentation shared category so, we may have to check with annegentle on this. | 18:57 |
ativelkov | sigmavirus24: yes, they definetely have a common subset. | 18:58 |
sigmavirus24 | nikhil_k: btw, you told me to remind you about being added to the drivers team on LaunchPad (same for flaper87 I suppose) | 18:59 |
sigmavirus24 | be back shorlty | 18:59 |
nikhil_k | sigmavirus24: ah yes, still waiting to hear back from Mark | 18:59 |
nikhil_k | thanks for reminder! | 18:59 |
sigmavirus24 | My calendar deserves all the praise =P | 18:59 |
nikhil_k | ;) | 19:00 |
ativelkov | nikhil_k: btw, what about the expansion of the core team? You wanted to postpone it till the mini-summit, but it turned out we didn't discuss it then (or did I miss something?) | 19:02 |
plaurin_ | nikhil_k: Sure, I'll open a bug later today | 19:04 |
*** jistr has quit IRC | 19:08 | |
*** jasondotstar has joined #openstack-glance | 19:10 | |
nikhil_k | plaurin_: thanks! | 19:14 |
nikhil_k | ativelkov: yeah, the mini-summit turned out to be longer than expected and we did not get to discuss a few items like | 19:14 |
nikhil_k | review guidelines, core memberships and changes-since filters | 19:15 |
nikhil_k | tbh, I am a little worried about adding more cores to that program where the existing merges are being an issue for the gate, release management, stability etc | 19:16 |
nikhil_k | Hope that helps! | 19:17 |
*** pennerc has quit IRC | 19:20 | |
*** flwang has joined #openstack-glance | 19:23 | |
stevelle | anyone here able to tell me they work on glance in a devstack environment? | 19:23 |
*** zul has quit IRC | 19:24 | |
*** zul has joined #openstack-glance | 19:27 | |
*** pennerc has joined #openstack-glance | 19:29 | |
kragniz | stevelle: I work on glance in a devstack enviroment | 19:29 |
kragniz | I'm not quite sure if that's what you were asking, though :P | 19:29 |
stevelle | kragniz: I think it is | 19:29 |
stevelle | I am struggling badly with keeping a development environment for glance happy | 19:30 |
stevelle | kragniz: do you just rebuild your devstack from scratch regularly? | 19:30 |
kragniz | stevelle: not very often | 19:31 |
kragniz | I'm not sure when I last rebuilt, but it was a few weeks ago | 19:32 |
*** nellysmitt has joined #openstack-glance | 19:35 | |
nikhil_k | stevelle: what's broken again? | 19:36 |
nikhil_k | :/ | 19:36 |
stevelle | the Registry doesn't start when running tests | 19:36 |
nikhil_k | oh that | 19:37 |
nikhil_k | still? | 19:37 |
sigmavirus24 | stevelle: that's odd | 19:37 |
stevelle | I had stuff working in a devenv for a little while but now it's broken again after this week's changes to master | 19:37 |
stevelle | the only thing that ever worked for me to get the registry to start up in tests was to not use a virtualenv | 19:38 |
sigmavirus24 | stevelle: I saw that earlier this week but rebased my patchset and now it seems to be working okay | 19:38 |
sigmavirus24 | let me rebase again | 19:38 |
stevelle | I'm trying to test my rebase of https://review.openstack.org/#/c/148400/ before I send into review again | 19:39 |
*** nellysmitt has quit IRC | 19:40 | |
* sigmavirus24 is running the tests | 19:40 | |
*** spzala has joined #openstack-glance | 19:45 | |
*** EmilienM is now known as EmilienM|afk | 19:47 | |
*** spzala has quit IRC | 19:49 | |
nikhil_k | well a 12.09pm (EST) jenkins run shows green -> https://review.openstack.org/#/c/149759/ | 19:50 |
* nikhil_k not worried about the gate anymore | 19:50 | |
sigmavirus24 | stevelle: no worries forme | 19:53 |
sigmavirus24 | woo backports | 19:53 |
stevelle | It would help a lot if I knew how to diagnose why registry isn't working but I'm boarding a plane in a few minutes. | 19:55 |
stevelle | will poke at this more later | 19:55 |
kragniz | stevelle: safe flight | 19:55 |
nikhil_k | stevelle: cool, have a good one. Do you not work out of bay area? | 19:56 |
nikhil_k | stevelle: run | 19:56 |
nikhil_k | run_tests.log might be your best friend btw | 19:56 |
stevelle | nikhil_k: I live north of bay area | 19:57 |
nikhil_k | ah | 19:58 |
stevelle | also, run_tests.log only says that registry was expected to start. no errors from registry itself because it is forked. | 19:58 |
sigmavirus24 | stevelle: are you getting a mismatch in return code? | 19:58 |
stevelle | sigmavirus24: no, the only clue is that "expected registry started True was False | 19:58 |
sigmavirus24 | you might also want to make sure there isn't a zombie registry still bound to that port | 19:58 |
sigmavirus24 | lsof -i | 19:58 |
stevelle | no zombies, already checked that | 19:59 |
nikhil_k | yeah, that test does not seem to touch registry | 19:59 |
*** aix has quit IRC | 19:59 | |
nikhil_k | kragniz: while I have you here, currently gunna check your spec https://blueprints.launchpad.net/glance/+spec/swift-retry-wait | 20:01 |
nikhil_k | kragniz: do you have any pending feedback (from anyone) that I may not be aware of | 20:01 |
nikhil_k | heard a few comments in hallway conversations during the mini-summit about it | 20:02 |
kragniz | nikhil_k: waiting to talk to stuart about it | 20:02 |
kragniz | nikhil_k: I haven't spoken with him since the midcycle | 20:02 |
kragniz | anything interesting in the hallway conversations? | 20:03 |
nikhil_k | people not being happy with the defaults being increased | 20:04 |
kragniz | okay | 20:04 |
*** openstackgerrit has quit IRC | 20:06 | |
*** openstackgerrit has joined #openstack-glance | 20:06 | |
nikhil_k | kragniz: if it gets a bit more delayed then we need to merge it (soon) after k2 | 20:07 |
nikhil_k | as it has a some of the reviews against Glance | 20:07 |
kragniz | of course | 20:07 |
kragniz | I'm feeling using the current defaults is okay | 20:08 |
kragniz | I'd like to hear what stuart thinks, though | 20:08 |
kragniz | you guys have mostly persuaded me :P | 20:09 |
nikhil_k | heh | 20:10 |
nikhil_k | I'm neutral on this as we've our own setup of the defaults | 20:11 |
nikhil_k | however, it would be good to have some feedback from the people who use the Glance defaults | 20:11 |
*** spzala has joined #openstack-glance | 20:11 | |
kragniz | indeed | 20:11 |
nikhil_k | btw, glad to see your oslo namespace changes merged https://review.openstack.org/#/q/status:merged+project:openstack/glance+branch:master+topic:bp/drop-namespace-packages,n,z | 20:11 |
kragniz | nikhil_k: https://review.openstack.org/#/q/project:openstack/python-glanceclient+branch:master+topic:bp/drop-namespace-packages,n,z | 20:12 |
kragniz | :P | 20:12 |
nikhil_k | yeah | 20:12 |
nikhil_k | I was looking at why zhiyan did not +A https://review.openstack.org/#/c/145799/ | 20:12 |
nikhil_k | They are _relatively_ smaller than glance related ones | 20:13 |
kragniz | nikhil_k: I think because the patch it depends on wasn't merged | 20:13 |
nikhil_k | and now it's outdated ? | 20:13 |
kragniz | yeah | 20:13 |
kragniz | need to rebase that | 20:14 |
nikhil_k | cool | 20:14 |
nikhil_k | anyone +A that should start auto merge after the dependency merged | 20:14 |
nikhil_k | merges* | 20:14 |
*** groen692 has quit IRC | 20:14 | |
kragniz | I'm working on porting to oslo_log in glance, too | 20:15 |
nikhil_k | ok nice | 20:15 |
nikhil_k | would suggest waiting until after k2 | 20:15 |
kragniz | on that matter, any thoughts on https://github.com/openstack/glance/blob/master/glance/opts.py? | 20:15 |
kragniz | yeah, it's going to take a bit of time | 20:15 |
kragniz | glance.openstack.common.log.common_cli_opts isn't public anymore | 20:16 |
kragniz | and this file was only added for use with oslo-config-generator | 20:16 |
nikhil_k | eh | 20:17 |
nikhil_k | do you have the commit which removed the module from public ns | 20:18 |
nikhil_k | ? | 20:18 |
*** pennerc has quit IRC | 20:18 | |
kragniz | nikhil_k: that happened a bit before the module moved out of the oslo incubator | 20:18 |
kragniz | I'll have a dig | 20:19 |
nikhil_k | ok, no worries | 20:19 |
kragniz | nova and cinder both pick up those extra options from a config file | 20:21 |
kragniz | https://github.com/openstack/nova/blob/master/tools/config/oslo.config.generator.rc | 20:21 |
*** flwang has quit IRC | 20:24 | |
nikhil_k | kragniz: guessing even the logging opts are not global anymore? | 20:24 |
kragniz | nikhil_k: yeah | 20:24 |
*** david-lyle has quit IRC | 20:25 | |
nikhil_k | that mechanism looks fine, though is kinda fragile adding dependencies in a remote file in a list! | 20:25 |
sigmavirus24 | nikhil_k: I'm thinking we should abandon https://review.openstack.org/#/c/145796/ (moving stuff around in config files on stable/* seems like a bad idea) | 20:26 |
nikhil_k | kragniz: as long as we can have tests around this | 20:26 |
kragniz | nikhil_k: I know! | 20:26 |
nikhil_k | sigmavirus24: yeah may be, though it would be great to have stuff consistent :P | 20:28 |
dhellmann | nikhil_k, kragniz : the new oslo config generator in oslo.config lets you include options from libraries, too: http://docs.openstack.org/developer/oslo.config/generator.html | 20:28 |
kragniz | sigmavirus24: +1 to that | 20:28 |
sigmavirus24 | kragniz: +1 to abandonment? | 20:29 |
*** harlowja is now known as harlowja_away | 20:29 | |
kragniz | dhellmann: cool! | 20:30 |
kragniz | (should probably have read the documentation...) | 20:30 |
kragniz | dhellmann saves the day again | 20:30 |
kragniz | sigmavirus24: yes | 20:31 |
openstackgerrit | Sarvesh Ranjan proposed openstack/glance: Typos fixed in the comments https://review.openstack.org/151764 | 20:31 |
kragniz | nikhil_k: in that case, we really don't need glance/opts.py | 20:33 |
sigmavirus24 | kragniz: git rm glance/opts.py ; ./run_tests.sh && git commit -m 'FIXED IT' && git review | 20:34 |
*** harlowja_away is now known as harlowja | 20:34 | |
openstackgerrit | Alexander Tivelkov proposed openstack/glance-specs: Semver Utility for DB storage https://review.openstack.org/139595 | 20:34 |
nikhil_k | kragniz: dhellmann : that looks awesome! | 20:34 |
nikhil_k | yeah, we do not need it | 20:34 |
kragniz | sigmavirus24: sounds good! | 20:35 |
dhellmann | nikhil_k: oslo is full of awesome ;-) | 20:35 |
nikhil_k | heh :) | 20:35 |
kragniz | dhellmann: the magic just keeps coming | 20:35 |
sigmavirus24 | kragniz: project idea for this weekend: oslo-magic-generator ;) | 20:36 |
kragniz | sigmavirus24: does that generate oslo code? | 20:37 |
openstackgerrit | Ian Cordasco proposed openstack/glance: Pass a real image target to the policy enforcer https://review.openstack.org/146651 | 20:37 |
*** eglynn-pto has quit IRC | 20:39 | |
openstackgerrit | Alexander Tivelkov proposed openstack/glance-specs: Semver Utility for DB storage https://review.openstack.org/139595 | 20:40 |
nikhil_k | sigmavirus24: what was the consensus about the SemVer stuff? | 20:40 |
nikhil_k | seems like we need a wrapper around the python library | 20:41 |
nikhil_k | ? | 20:41 |
ativelkov | nikhil_k: I've just updated the spec to address that | 20:41 |
nikhil_k | ativelkov: awesome, thanks! | 20:41 |
ativelkov | and answer most of the other question sigmavirus24 and kragniz had | 20:41 |
nikhil_k | wanted to get first level feedback around the major concerns | 20:41 |
ativelkov | As far as I can see, the main issue is semver vs PEP440 topic | 20:42 |
kragniz | sigmavirus24: probably a silly question, but is there a pep440 library like python-semanticversion? | 20:45 |
ativelkov | kragniz: yes. It is called pbr :) | 20:46 |
ativelkov | just "import pbr.version" | 20:46 |
kragniz | hmm | 20:47 |
kragniz | okay | 20:47 |
ativelkov | that is not perfect solution, as it more targets the package version info, and so has some specifics, but it still may be used to check the differences between semver and pep440 | 20:48 |
sigmavirus24 | kragniz: no the actual one is packaging | 20:48 |
sigmavirus24 | PBR is just already in the global-requirements | 20:48 |
kragniz | right, that makes sense | 20:48 |
sigmavirus24 | packaging carries the official implementation of pep440 | 20:49 |
sigmavirus24 | it's what pip uses too | 20:49 |
ativelkov | ah, yes, thanks sigmavirus24 | 20:49 |
sigmavirus24 | no worries | 20:49 |
sigmavirus24 | being a pypa member makes me more aware of these things sadly | 20:49 |
kragniz | sigmavirus24: I should have been watching my github feed more closely, since you folked that a few hours ago :P | 20:49 |
sigmavirus24 | oh yeah that was because the description.rst on PyPI is rendering funny | 20:50 |
sigmavirus24 | and I know how to fix those all too damned well | 20:50 |
kragniz | heh | 20:51 |
kragniz | sigmavirus24: btw | 20:52 |
kragniz | sigmavirus24: there's still time for you to come to fosdem | 20:52 |
sigmavirus24 | when is it? | 20:52 |
kragniz | tomorrow | 20:52 |
kragniz | totally enough time to fly over | 20:52 |
sigmavirus24 | no thanks | 20:52 |
sigmavirus24 | =P | 20:52 |
kragniz | go on | 20:52 |
sigmavirus24 | if you see Tarék though say hi | 20:52 |
sigmavirus24 | tell him you love all the work he still puts into Flake8 =P | 20:53 |
kragniz | I will do :P | 20:53 |
kragniz | do you know if he's giving a talk this year? | 20:53 |
sigmavirus24 | then ask him why he uses a go board for a gravatar =P | 20:53 |
sigmavirus24 | Not the slightest. I haven't spoken to him in over a year | 20:54 |
kragniz | heh | 20:54 |
kragniz | okay | 20:54 |
kragniz | sigmavirus24: why do you have a three in a row board as your avatar? | 20:55 |
sigmavirus24 | you mean bingo card? | 20:55 |
sigmavirus24 | also I can' tell you how depressing it is when programmers don't know what conway's game of life is | 20:56 |
stevelle | what's that? is it on PS4? :D | 20:56 |
kragniz | stevelle: it's a great game, but the graphics are a little dated | 20:57 |
sigmavirus24 | stevelle: <3 | 20:57 |
*** eglynn-pto has joined #openstack-glance | 20:59 | |
*** pennerc has joined #openstack-glance | 21:00 | |
*** pennerc has quit IRC | 21:05 | |
kragniz | nikhil_k: do we want to do glance client release mid next week? | 21:12 |
*** sigmavirus24 is now known as sigmavirus24_awa | 21:13 | |
nikhil_k | kragniz: after the release, yes | 21:13 |
* nikhil_k needs to check the update on capabilities spec | 21:13 | |
kragniz | okay | 21:13 |
kragniz | we should start merging some patchsets, there's quite a backlog of simple fixes | 21:15 |
kragniz | https://review.openstack.org/#/q/status:open+project:openstack/python-glanceclient,n,z | 21:15 |
*** sigmavirus24_awa is now known as sigmavirus24 | 21:18 | |
*** leakypipes has quit IRC | 21:20 | |
*** mjturek has quit IRC | 21:21 | |
nikhil_k | let's focus on k2 until mid next week | 21:24 |
nikhil_k | https://launchpad.net/glance/+milestone/kilo-2 | 21:24 |
kragniz | didn't we say the same around k1? :P | 21:26 |
sigmavirus24 | nikhil_k: thoughts on https://review.openstack.org/#/c/148387/ ? | 21:27 |
nikhil_k | if you care about some specific ones, feel free to bring them up | 21:27 |
nikhil_k | the whole point of release management is to ensure that we help reviewers keep focus | 21:27 |
*** thangp has quit IRC | 21:28 | |
nikhil_k | sigmavirus24: looks like a good addition | 21:30 |
*** nellysmitt has joined #openstack-glance | 21:36 | |
*** nellysmitt has quit IRC | 21:41 | |
sigmavirus24 | nikhil_k: https://review.openstack.org/#/c/148574/ seems ready for approval =P | 21:47 |
nikhil_k | sg | 21:49 |
*** jgrimm is now known as zz_jgrimm | 21:52 | |
sigmavirus24 | nikhil_k: or review from you more accurately | 21:54 |
nikhil_k | sigmavirus24: it's on my agenda | 21:55 |
nikhil_k | just want to focus on specs first as they can be blockers for the code | 21:55 |
sigmavirus24 | yeah just realized it sounded like I was trying to get you to do a +W without reviewing =P | 21:55 |
*** jasondotstar has quit IRC | 21:56 | |
*** pkoniszewski has joined #openstack-glance | 21:56 | |
nikhil_k | heh, ok | 21:58 |
nikhil_k | I need to put on the reviewing gear on | 21:58 |
nikhil_k | s/put on/put/ | 21:58 |
*** cpallares has quit IRC | 22:03 | |
*** pkoniszewski has quit IRC | 22:03 | |
*** ericpeterson has quit IRC | 22:07 | |
*** peristeri has quit IRC | 22:11 | |
*** TravT has joined #openstack-glance | 22:31 | |
*** leakypipes has joined #openstack-glance | 22:40 | |
nikhil_k | sigmavirus24: see a ton of -1s on this https://review.openstack.org/#/c/139726/ | 22:41 |
nikhil_k | and on the spec, not so much | 22:41 |
sigmavirus24 | Yeah there was confusion | 22:41 |
nikhil_k | although, I'm just catching up and have not read the whole thing including all comments | 22:42 |
sigmavirus24 | zhiyan: has a good point that there needs to be a corresponding change to glance for etc/ and doc/. Erno was worried about backwards compat | 22:42 |
sigmavirus24 | s/Erno/jokke_/ | 22:42 |
openstackgerrit | Merged openstack/glance-specs: Spec to replace snet config with endpoint config https://review.openstack.org/146972 | 22:59 |
*** tellesnobrega has quit IRC | 23:03 | |
*** harlowja is now known as harlowja_away | 23:05 | |
*** jasondotstar has joined #openstack-glance | 23:05 | |
*** jasondotstar has quit IRC | 23:05 | |
nikhil_k | sigmavirus24: heh, should have read your message before reading the whole conversation ;) | 23:06 |
nikhil_k | thanks though! | 23:06 |
sigmavirus24 | nikhil_k: I probably should have pinged you while explaining it =P | 23:06 |
*** leakypipes has quit IRC | 23:10 | |
*** harlowja_away is now known as harlowja | 23:27 | |
*** harlowja has quit IRC | 23:29 | |
*** harlowja has joined #openstack-glance | 23:29 | |
*** leakypipes has joined #openstack-glance | 23:32 | |
*** nellysmitt has joined #openstack-glance | 23:37 | |
*** vijendar has quit IRC | 23:40 | |
*** nellysmitt has quit IRC | 23:42 | |
*** annashen has quit IRC | 23:57 | |
*** changbl has quit IRC | 23:58 | |
openstackgerrit | Ian Cordasco proposed openstack/glance: Pass a real image target to the policy enforcer https://review.openstack.org/146651 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!