Thursday, 2015-01-29

*** jdurgin has quit IRC00:00
*** ericpeterson has quit IRC00:18
*** malini has quit IRC00:18
*** malini has joined #openstack-glance00:21
*** openstackgerrit has quit IRC00:21
*** openstackgerrit has joined #openstack-glance00:21
ativelkovsigmavirus24: do you have time to speak about the semver stuff now?00:26
*** david-lyle is now known as david-lyle_afk00:28
*** annashen has joined #openstack-glance00:41
*** malini has quit IRC00:47
annashenall, can you please take a look at this bug? thanks for your input00:54
annashenhttps://bugs.launchpad.net/glance/+bug/141569100:54
*** mfedosin_ has quit IRC01:00
*** annashen has quit IRC01:03
*** openstackgerrit has quit IRC01:05
*** openstackgerrit has joined #openstack-glance01:05
sigmavirus24ativelkov: just got back from dinner01:07
sigmavirus24also I should probably re-review your spec before we talk so how does tomorrow sound?01:07
ativelkovsigmavirus24: yes, sure01:08
ativelkovHowever I now think about abandoning the spec and going with the existing semantic_version library, by just adding some utility to convert the version to DB-compatible values01:09
ativelkovprobably that is really better then reinventing the wheel01:09
kragnizativelkov: ++01:12
kragnizthe less code we have, the better!01:12
*** buzztroll has quit IRC01:12
*** jaypipes has quit IRC01:15
openstackgerritYvonne Stachowski proposed openstack/python-glanceclient: Fixed CLI help for bash-completion  https://review.openstack.org/15106501:15
ativelkovYup. So, let's revisit this tomorrow.01:15
* kragniz goes to bed01:16
ativelkovgood night :)01:16
kragnizativelkov: you too!01:16
sigmavirus24good night y'all01:40
*** sigmavirus24 is now known as sigmavirus24_awa01:42
*** TravT has quit IRC01:51
*** TravT has joined #openstack-glance01:57
*** TravT has quit IRC02:00
*** wayne___ has quit IRC02:03
*** junhongl has quit IRC02:06
*** rwsu is now known as rwsu-afk02:13
*** haomaiwang has joined #openstack-glance02:14
*** junhongl has joined #openstack-glance02:20
*** openstackgerrit has quit IRC02:20
*** openstackgerrit has joined #openstack-glance02:20
*** buzztroll has joined #openstack-glance03:01
*** jlk has joined #openstack-glance03:02
*** bkopilov has quit IRC03:05
*** buzztroll has quit IRC03:06
*** annashen has joined #openstack-glance03:08
*** openstackgerrit has quit IRC03:20
*** openstackgerrit has joined #openstack-glance03:20
*** junhongl has quit IRC03:43
*** junhongl has joined #openstack-glance04:00
*** junhongl has quit IRC04:09
*** boris-42 has quit IRC04:13
*** annashen has quit IRC04:17
*** annashen has joined #openstack-glance04:18
*** bkopilov has joined #openstack-glance04:19
*** junhongl has joined #openstack-glance04:22
*** annashen has quit IRC04:28
*** annashen has joined #openstack-glance04:31
*** annashen has joined #openstack-glance04:34
*** sileht has quit IRC04:40
*** annashen has quit IRC04:44
*** buzztroll has joined #openstack-glance04:46
*** harlowja is now known as harlowja_away04:48
*** f13o has quit IRC05:00
*** annashen has joined #openstack-glance05:10
*** f13o has joined #openstack-glance05:13
*** spzala has quit IRC05:16
*** julim_ has joined #openstack-glance05:19
*** julim has quit IRC05:22
*** echevemaster has quit IRC05:31
*** Longgeek has joined #openstack-glance05:43
*** annashen has quit IRC06:02
*** Longgeek has quit IRC06:03
*** Longgeek has joined #openstack-glance06:18
*** Longgeek has quit IRC06:22
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Imported Translations from Transifex  https://review.openstack.org/14676506:22
openstackgerrithzrandd proposed openstack/glance: Allow users to configure blacklist of URI schemes  https://review.openstack.org/15073606:25
*** sileht has joined #openstack-glance06:29
*** JoshNang has quit IRC06:52
*** annashen has joined #openstack-glance06:58
*** JoshNang has joined #openstack-glance07:01
*** groen692 has joined #openstack-glance07:03
*** annashen has quit IRC07:07
openstackgerrithzrandd proposed openstack/glance: Modify annotation information.  https://review.openstack.org/15112907:09
*** nlevinki has joined #openstack-glance07:10
openstackgerrithzrandd proposed openstack/glance: Modify annotation information.  https://review.openstack.org/15112907:11
*** jaypipes has joined #openstack-glance07:23
*** tshefi has joined #openstack-glance07:25
*** jaypipes has quit IRC07:33
*** belmoreira has joined #openstack-glance07:40
*** hemanth__ has quit IRC07:46
*** hemanth has joined #openstack-glance07:46
*** f13o has quit IRC07:58
*** f13o has joined #openstack-glance08:16
*** eglynn-pto has quit IRC08:21
*** markus_z has joined #openstack-glance08:26
flaper87zhiyan https://review.openstack.org/#/c/85211/ again, pls :)08:29
*** nellysmitt has joined #openstack-glance08:35
*** chlong has quit IRC08:36
*** buzztroll has quit IRC08:42
*** eglynn-pto has joined #openstack-glance08:46
openstackgerritFlavio Percoco proposed openstack/python-glanceclient: Add a `--limit` parameter to list operations  https://review.openstack.org/15041608:56
flaper87zhiyan: and https://review.openstack.org/#/c/150416/ :)08:58
*** eglynn-pto has quit IRC09:01
*** MattMan has joined #openstack-glance09:15
*** jistr has joined #openstack-glance09:16
*** Longgeek has joined #openstack-glance09:23
*** Longgeek has quit IRC09:27
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/14668909:58
openstackgerritFlavio Percoco proposed openstack/python-glanceclient: Add a `--limit` parameter to list operations  https://review.openstack.org/15041609:58
*** changbl has quit IRC10:12
*** chlong has joined #openstack-glance10:13
*** chlong has quit IRC10:27
*** buzztroll has joined #openstack-glance10:31
*** buzztroll has quit IRC10:31
*** buzztroll has joined #openstack-glance10:31
*** haomaiwang has quit IRC10:34
*** buzztroll has quit IRC10:36
*** harlowja_away has quit IRC10:36
openstackgerritFlavio Percoco proposed openstack/python-glanceclient: Add a `--limit` parameter to list operations  https://review.openstack.org/15041610:37
openstackgerritFlavio Percoco proposed openstack/python-glanceclient: Add a `--limit` parameter to list operations  https://review.openstack.org/15041610:37
*** chlong has joined #openstack-glance10:44
*** chlong has quit IRC11:39
*** aix has joined #openstack-glance11:40
openstackgerritPawel Koniszewski proposed openstack/glance-specs: Use oslo-versioned-objects to deal with upgrades  https://review.openstack.org/15119411:51
*** olc has quit IRC11:56
*** takedakn has joined #openstack-glance11:57
openstackgerritRakesh H S proposed openstack/python-glanceclient: return 130 for keyboard interrupt  https://review.openstack.org/12393411:59
*** eglynn-pto has joined #openstack-glance12:01
*** olc has joined #openstack-glance12:02
*** chlong has joined #openstack-glance12:05
*** flwang has joined #openstack-glance12:09
flwangnikhil_k: ping12:09
flwangnikhil_k: could you pls revisit this https://review.openstack.org/#/c/80178/ ?12:09
*** pkoniszewski has joined #openstack-glance12:13
*** buzztroll has joined #openstack-glance12:20
*** eglynn-pto has quit IRC12:20
*** buzztroll has quit IRC12:25
*** julim has joined #openstack-glance12:43
*** belmoreira has quit IRC12:46
*** julim_ has quit IRC12:46
openstackgerritMerged openstack/python-glanceclient: Add a `--limit` parameter to list operations  https://review.openstack.org/15041612:47
openstackgerritboden proposed openstack/glance-specs: Spec for glance multi-store support  https://review.openstack.org/15096712:49
*** ankit_ag has joined #openstack-glance12:51
openstackgerritboden proposed openstack/glance-specs: Spec for glance async image replication support  https://review.openstack.org/15097112:56
*** adam_g is now known as adam_g_out12:57
openstackgerritboden proposed openstack/glance-specs: Spec for glance async image deletion support  https://review.openstack.org/15098613:02
*** hemanth is now known as Guest3546713:07
*** pkoniszewski has quit IRC13:07
*** bkopilov has quit IRC13:14
*** EmilienM|afk is now known as EmilienM13:14
*** boris-42 has joined #openstack-glance13:32
*** eglynn-pto has joined #openstack-glance13:54
*** changbl has joined #openstack-glance13:54
*** sigmavirus24_awa is now known as sigmavirus2413:55
sigmavirus24o/ meeting in 4 minutes13:56
kragnizsigmavirus24: you're chairing, right?13:57
sigmavirus24yep13:57
*** jistr has quit IRC13:58
*** jistr has joined #openstack-glance14:00
sigmavirus24flaper87: zhiyan #openstack-meeting-4 if you're available14:02
*** eglynn-pto has quit IRC14:03
*** spzala has joined #openstack-glance14:04
*** buzztroll has joined #openstack-glance14:09
*** buzztroll has quit IRC14:13
*** eddie__ has joined #openstack-glance14:18
*** eglynn-pto has joined #openstack-glance14:19
*** peristeri has joined #openstack-glance14:22
openstackgerritFlavio Percoco proposed openstack/glance_store: Remove old dependencies  https://review.openstack.org/15126114:24
*** takedakn has quit IRC14:29
openstackgerritJesse J. Cook proposed openstack/glance-specs: Spec to replace snet config with endpoint config  https://review.openstack.org/14697214:31
*** jasondot_ has joined #openstack-glance14:34
openstackgerritInessa Vasilevskaya proposed openstack/glance: Artifacts API  https://review.openstack.org/13662914:35
*** sigmavirus24 is now known as sigmavirus24_awa14:43
openstackgerritAbhijeet Malawade proposed openstack/glance_store: Validate metadata JSON file  https://review.openstack.org/14131114:43
*** jasondot_ has quit IRC14:45
*** r-daneel has joined #openstack-glance14:51
*** ankit_ag has quit IRC14:56
*** eglynn-pto is now known as eglynn14:59
*** ericpeterson has joined #openstack-glance15:01
openstackgerritFlavio Percoco proposed openstack/glance: Add Support for a Taskflow Executor  https://review.openstack.org/8521115:02
*** jistr has quit IRC15:10
*** jistr has joined #openstack-glance15:12
*** sigmavirus24_awa is now known as sigmavirus2415:13
*** vijendar has joined #openstack-glance15:22
openstackgerritMerged openstack/glance: Make digest algorithm configurable  https://review.openstack.org/8017815:25
sigmavirus24ativelkov: I have a database structure for your semver spec that I think will work. The (major, minor, patch) sets each sacrifice 2 bits a piece and we can accommodate more accurate/complex pre-release versions15:27
*** thangp has joined #openstack-glance15:28
kragnizsigmavirus24: how are you storing the pre-release versions?15:29
openstackgerritKamil Rykowski proposed openstack/glance: Notifications for metadefinition resources  https://review.openstack.org/14854615:29
sigmavirus24kragniz: so my thought is this: you probably don't need more than 5 bits for the pre-release version15:32
sigmavirus24and for the release type there are 4 types: a, b, rc, final15:32
sigmavirus24so 00, 01, 10, 1115:32
sigmavirus24If we split 62 4 ways we still have half a bit unused15:33
sigmavirus24so we could do (major 15, minor 15, patch 16, release type version 15)15:33
sigmavirus24but I have to wonder how many pre-release versions there will actually be15:33
sigmavirus24so 19, 19, 19, 2, 5 seems acceptable to me15:33
kragnizsounds neato15:34
sigmavirus24yeah working backwards from known facts as constraints helps15:34
sigmavirus24really I don't care if we assign 15 bits to the release type version, I just wonder if that's wasteful15:34
*** spzala has quit IRC15:48
*** annashen has joined #openstack-glance15:57
*** buzztroll has joined #openstack-glance15:57
*** eddie__ has quit IRC16:00
*** pkoniszewski has joined #openstack-glance16:02
*** bkopilov has joined #openstack-glance16:02
*** buzztroll has quit IRC16:02
*** zz_jgrimm is now known as jgrimm16:10
*** annashen has quit IRC16:13
*** eddie__ has joined #openstack-glance16:13
*** pkoniszewski has quit IRC16:14
*** nlevinki has quit IRC16:17
*** spzala_ has joined #openstack-glance16:19
*** spzala_ is now known as spzala16:21
*** buzztroll has joined #openstack-glance16:25
*** markus_z has quit IRC16:25
*** tshefi has quit IRC16:27
*** groen692 has quit IRC16:28
*** eddie__ has quit IRC16:36
*** eddie__ has joined #openstack-glance16:42
*** jaypipes has joined #openstack-glance16:48
*** david-lyle_afk is now known as david-lyle16:49
sigmavirus24flaper87: kragniz fwiw, dhellmann just tagged oslo.middleware 0.4.0 with the healthcheck middleware in it16:50
kragniz\o/16:50
flaper87sigmavirus24: are you stalking dhellmann ?16:52
* flaper87 read the email16:52
sigmavirus24flaper87: no. dhellmann pinged me in IRC to let me know16:52
*** rwsu-afk is now known as rwsu16:55
flaper87sigmavirus24: yeaaaaaaaaaaah right16:56
flaper87sigmavirus24: i know you >.>16:56
* sigmavirus24 has eyes16:57
kragnizthat's what they all say, sigmavirus2416:58
sigmavirus24Don't make me get the logs16:59
ativelkovsigmavirus24: so, you are actually suggesting to have a subset of the actual semver, with limited set of available values for pre-release tag16:59
sigmavirus24so I think what is most commonly used is the best to support, and if we're limiting ourselves to unsigned longs, perhaps it's best to limit to the most often used subset of semver17:00
*** eddie__ has quit IRC17:01
ativelkovIn my proposal I wanted to store only the combination of minor-major-patch as a long, and the prerelease label as a string. And do the comparison based on the combination of these fields17:05
ativelkovbecause - according to the spec - pre-release labels are compared as alphanumeric values17:06
ativelkovi.e. "alpha" < "beta" because 'a' < 'b', and any others may be compared the same way, e.g. 'bar' < 'foo'17:07
ativelkovThe only difference between simple string comparisons and the complete semver is that in semver 'alpha.5.blah' < 'alpha.10.foo', but if we compare string, it is the opposite17:11
*** nellysmitt has quit IRC17:11
*** eddie__ has joined #openstack-glance17:19
sigmavirus24ativelkov: so a few things, if we use a proper representation of the version it will handle properly comparing the entire version string but that will not happen in the database17:21
sigmavirus24I haven't looked at the libraries in depth yet, but as for storing these in the database so that we can sort on that column, we should definitely be allowing people to specify alpha.1, alpha.217:22
*** cpallares has joined #openstack-glance17:23
*** EmilienM is now known as EmilienM|afk17:27
*** jistr has quit IRC17:31
*** pdb has quit IRC17:35
*** nellysmitt has joined #openstack-glance17:40
ativelkovsigmavirus24: I've got an idea. We may limit the length of the numeric components of the pre-release tags to some meaningful value (say, 4 digits) and - while doing the pre-processing - add leading zeros.17:49
ativelkovso, alpha.1 will be stored as "alpha.0001"17:49
ativelkovand alpha.10 - as "alpha.0010".17:49
ativelkovthus "alpha.0001" < "alpha.0010"17:50
ativelkovso, we may leave string-based comparison for prerelease tags, thus we will not have to limit the number of components in that tags and the number of literal label only to "alpha-beta-rc"17:51
ativelkovmeanwhile, major-minor-patch triad will remain convertable to long17:52
*** annashen has joined #openstack-glance17:52
*** f13o has quit IRC18:06
*** f13o has joined #openstack-glance18:08
*** buzztroll has quit IRC18:10
*** ff has joined #openstack-glance18:10
*** eglynn has quit IRC18:25
*** pkoniszewski has joined #openstack-glance18:34
*** EmilienM|afk is now known as EmilienM18:36
*** harlowja has joined #openstack-glance18:38
*** harlowja_ has joined #openstack-glance18:42
*** harlowja has quit IRC18:46
*** eglynn has joined #openstack-glance18:50
*** openstackgerrit has quit IRC18:50
*** openstackgerrit has joined #openstack-glance18:51
*** TravT has joined #openstack-glance18:51
openstackgerritMerged openstack/glance_store: Remove old dependencies  https://review.openstack.org/15126118:52
*** flwang has quit IRC18:56
sigmavirus24ativelkov: I think we're talking past each other on this19:00
ativelkovsigmavirus24: what do you mean?19:03
*** spzala has quit IRC19:06
sigmavirus24So let me put together some examples of what I'm thinking of, and I'll show you. I could also just be misunderstanding you and we're in complete agreement19:06
*** jdurgin has joined #openstack-glance19:07
*** annashen has quit IRC19:07
ativelkovsigmavirus24: yes, sure, please do19:14
*** TravT has quit IRC19:14
*** TravT has joined #openstack-glance19:17
*** buzztroll has joined #openstack-glance19:19
*** annashen has joined #openstack-glance19:20
*** buzztroll has quit IRC19:21
*** TravT has quit IRC19:21
*** jaypipes has quit IRC19:31
*** spzala has joined #openstack-glance19:34
*** aix has quit IRC19:35
*** dutsmoc has quit IRC19:38
*** buzztroll has joined #openstack-glance19:38
*** comstud has joined #openstack-glance19:38
*** jwang__ has joined #openstack-glance19:51
*** perister1 has joined #openstack-glance19:54
*** rodrigod` has joined #openstack-glance19:56
*** rodrigods has quit IRC19:56
*** annashen has quit IRC19:56
*** peristeri has quit IRC19:56
*** jwang_ has quit IRC19:56
*** flwang has joined #openstack-glance19:56
*** thangp has quit IRC19:57
sigmavirus24ativelkov: does this help https://gist.github.com/sigmavirus24/25fc7a6ed56e410ee119 ?20:00
*** annashen has joined #openstack-glance20:01
ativelkovsigmavirus24: thanks, I now understand what you propose20:02
ativelkovBut that's quite a small subset of actual sember20:02
ativelkovI will not be able to store something like "1.0.0-alpha.1.29.my-customer-name"20:03
ativelkovwhich is quite a possible version according to semver notation20:03
*** openstackgerrit has quit IRC20:04
*** openstackgerrit has joined #openstack-glance20:04
sigmavirus24ativelkov: agreed but your original proposal didn't allow for that either20:06
sigmavirus24Specifically if you look at the Proposed Change section of https://review.openstack.org/#/c/139595/1/specs/kilo/semver.rst20:07
ativelkovsigmavirus24: I am now trying to improve that20:07
*** nellysmitt has quit IRC20:07
ativelkovI'll share the etherpad with the updated proposal in several minutes20:07
sigmavirus24Thanks that'll help significantly20:07
*** nellysmitt has joined #openstack-glance20:07
ativelkovsigmavirus24: something like this: https://etherpad.openstack.org/p/LF2SY1Za6Y20:12
*** nellysmitt has quit IRC20:12
*** bkopilov has quit IRC20:13
*** annashen has quit IRC20:20
sigmavirus24So here's a potentially stupid question: Why can't some of this information be stored as metadata (e.g., my-customer-name)?20:21
*** fifieldt has quit IRC20:24
*** fifieldt has joined #openstack-glance20:25
*** buzztroll has quit IRC20:28
ativelkovsigmavirus24: well, it can be. But it is up to the user to decide where should it be stored20:31
*** wayne___ has joined #openstack-glance20:32
ativelkovI mean, the semver spec does not limit the user only to a, b and rc20:32
sigmavirus24ativelkov: understood. There's also a difference between SemVer's spec and SemVer usage in reality20:32
ativelkovright. I believe that a reasonable use case is putting the build date into the pre-release label, so one can sort objects by this dat20:33
ativelkove.g. 1.0.1-beta.1.2920:33
ativelkovThat is a real-life example, not a hypothetical attempt for purism20:34
ativelkovin this case we should probably provide a way to sort objects so 1.0.1-beta.4.29 is less then 1.0.1-beta.12.120:35
ativelkovmy proposal with leading zeros should cover that20:35
sigmavirus24I agree20:35
*** buzztroll has joined #openstack-glance20:45
*** pkoniszewski has quit IRC20:51
*** annashen has joined #openstack-glance20:53
*** markus_z has joined #openstack-glance20:58
*** jdurgin has quit IRC21:12
*** wborkowski has joined #openstack-glance21:23
*** wborkowski has left #openstack-glance21:23
*** buzztroll has quit IRC21:24
*** buzztroll has joined #openstack-glance21:29
*** rodrigod` is now known as rodrigods21:31
*** rodrigods has quit IRC21:31
*** rodrigods has joined #openstack-glance21:31
*** flwang has quit IRC21:41
*** flwang has joined #openstack-glance21:41
*** markus_z has quit IRC21:58
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/14668922:00
*** jamielennox|away is now known as jamielennox22:03
*** buzztroll has quit IRC22:05
*** chlong has quit IRC22:08
*** nellysmitt has joined #openstack-glance22:08
openstackgerritOpenStack Proposal Bot proposed openstack/python-glanceclient: Updated from global requirements  https://review.openstack.org/14997722:08
*** nellysmitt has quit IRC22:13
*** ff has quit IRC22:15
*** buzztroll has joined #openstack-glance22:20
*** r-daneel has quit IRC22:22
*** buzztroll has quit IRC22:31
*** perister1 has quit IRC22:34
*** vijendar has quit IRC22:41
*** eddie__ has quit IRC22:44
*** openstackgerrit has quit IRC22:51
*** openstackgerrit has joined #openstack-glance22:51
*** TravT has joined #openstack-glance22:54
openstackgerritIan Cordasco proposed openstack/glance: Pass a real image target to the policy enforcer  https://review.openstack.org/14665122:59
*** sgotliv has quit IRC23:15
*** eglynn has quit IRC23:16
*** cpallares has quit IRC23:18
ativelkovFolks, as agreed on the summit I've submitted an update to global_requirements to add semantic_version there: https://review.openstack.org/#/c/151466/23:29
*** echevemaster has joined #openstack-glance23:30
*** buzztroll has joined #openstack-glance23:40
*** buzztroll has quit IRC23:45
*** chlong has joined #openstack-glance23:53
*** buzztroll has joined #openstack-glance23:56
*** buzztroll has quit IRC23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!