*** haomaiwang has quit IRC | 00:01 | |
*** haomaiwang has joined #openstack-glance | 00:01 | |
*** TravT has joined #openstack-glance | 00:24 | |
*** harshs has quit IRC | 00:27 | |
*** harshs has joined #openstack-glance | 00:36 | |
*** dims has joined #openstack-glance | 00:39 | |
*** haomaiwang has quit IRC | 00:46 | |
*** dims has quit IRC | 01:01 | |
*** davideagnello has joined #openstack-glance | 01:06 | |
*** harshs has quit IRC | 01:11 | |
*** davideagnello has quit IRC | 01:12 | |
*** TravT has quit IRC | 01:25 | |
*** harshs has joined #openstack-glance | 01:26 | |
*** TravT has joined #openstack-glance | 01:30 | |
*** TravT has quit IRC | 01:39 | |
*** haomaiwang has joined #openstack-glance | 01:41 | |
*** haomaiwang has quit IRC | 02:01 | |
*** haomaiwa_ has joined #openstack-glance | 02:01 | |
*** kebray has joined #openstack-glance | 02:07 | |
*** kebray has quit IRC | 02:12 | |
*** kebray has joined #openstack-glance | 02:13 | |
*** jamielennox is now known as jamielennox|away | 02:14 | |
*** kebray has quit IRC | 02:15 | |
*** kebray has joined #openstack-glance | 02:16 | |
*** mingdang1 has joined #openstack-glance | 02:17 | |
*** houming has joined #openstack-glance | 02:26 | |
*** TravT has joined #openstack-glance | 02:48 | |
*** dims has joined #openstack-glance | 02:52 | |
*** davideagnello has joined #openstack-glance | 02:54 | |
*** davideagnello has quit IRC | 02:54 | |
*** TravT has quit IRC | 02:56 | |
*** mdnadeem has joined #openstack-glance | 02:57 | |
*** TravT has joined #openstack-glance | 02:57 | |
*** mingdang1 has quit IRC | 03:00 | |
*** mingdang1 has joined #openstack-glance | 03:00 | |
*** haomaiwa_ has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-glance | 03:01 | |
*** nikhil has quit IRC | 03:04 | |
*** nikhil has joined #openstack-glance | 03:05 | |
*** TravT has quit IRC | 03:07 | |
*** TravT has joined #openstack-glance | 03:11 | |
*** GB21 has joined #openstack-glance | 03:21 | |
*** TravT has quit IRC | 03:25 | |
*** links has joined #openstack-glance | 03:26 | |
*** GB21 has quit IRC | 03:27 | |
*** TravT has joined #openstack-glance | 03:29 | |
*** harshs has quit IRC | 03:39 | |
*** TravT has quit IRC | 03:43 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-glance | 04:01 | |
*** mingdang1 has quit IRC | 04:23 | |
*** mingdang1 has joined #openstack-glance | 04:24 | |
*** kebray_ has joined #openstack-glance | 04:37 | |
*** kebray has quit IRC | 04:37 | |
*** dims has quit IRC | 04:58 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwa_ has joined #openstack-glance | 05:01 | |
openstackgerrit | Jake Yip proposed openstack/glance: Add location_strategy.metadata_rank https://review.openstack.org/268865 | 05:04 |
---|---|---|
*** TravT has joined #openstack-glance | 05:25 | |
*** TravT_ has joined #openstack-glance | 05:30 | |
*** TravT has quit IRC | 05:30 | |
*** sgotliv has joined #openstack-glance | 05:35 | |
*** TravT_ has quit IRC | 05:39 | |
*** TravT has joined #openstack-glance | 05:44 | |
*** TravT has quit IRC | 05:53 | |
*** TravT has joined #openstack-glance | 05:57 | |
*** haomaiwa_ has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-glance | 06:01 | |
*** TravT has quit IRC | 06:11 | |
openstackgerrit | zwei proposed openstack/glance: v2 - "readOnly" key should be used in schemas https://review.openstack.org/267972 | 06:13 |
*** TravT has joined #openstack-glance | 06:16 | |
*** TravT has quit IRC | 06:24 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Imported Translations from Zanata https://review.openstack.org/268592 | 06:25 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/glance_store: Imported Translations from Zanata https://review.openstack.org/268601 | 06:28 |
*** TravT has joined #openstack-glance | 06:29 | |
*** groen692 has joined #openstack-glance | 06:30 | |
*** TravT has quit IRC | 06:34 | |
*** kebray_ has quit IRC | 06:35 | |
*** TravT has joined #openstack-glance | 06:38 | |
*** TravT has quit IRC | 06:43 | |
*** harshs has joined #openstack-glance | 06:45 | |
*** TravT has joined #openstack-glance | 06:47 | |
*** abhishekk has joined #openstack-glance | 06:54 | |
abhishekk | jokke_, mclaren: hi, when you get time, could you please review https://review.openstack.org/#/c/261288 patch | 06:55 |
abhishekk | jokke_, mclaren: this is regarding glanceclient hook solution for request-ids, thank you in advance | 06:56 |
*** chlong has quit IRC | 07:00 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-glance | 07:01 | |
*** TravT has quit IRC | 07:01 | |
*** houming has quit IRC | 07:12 | |
*** houming has joined #openstack-glance | 07:16 | |
*** harshs has quit IRC | 07:24 | |
*** TravT has joined #openstack-glance | 07:25 | |
*** e0ne has joined #openstack-glance | 07:25 | |
*** TravT has quit IRC | 07:33 | |
*** TravT has joined #openstack-glance | 07:38 | |
*** rcernin has joined #openstack-glance | 07:41 | |
*** TravT has quit IRC | 07:47 | |
*** mdnadeem has quit IRC | 07:49 | |
*** TravT has joined #openstack-glance | 07:52 | |
openstackgerrit | Sergey Gotliv proposed openstack/glance_store: Add random read capabilities to the rbd store https://review.openstack.org/268908 | 07:57 |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-glance | 08:01 | |
*** TravT has quit IRC | 08:10 | |
*** mdnadeem has joined #openstack-glance | 08:10 | |
*** belmoreira has joined #openstack-glance | 08:11 | |
*** markus_z has joined #openstack-glance | 08:14 | |
*** sgotliv has quit IRC | 08:17 | |
*** e0ne has quit IRC | 08:20 | |
openstackgerrit | zwei proposed openstack/python-glanceclient: v2 - "readOnly" key should be used in schemas https://review.openstack.org/268923 | 08:26 |
*** TravT has joined #openstack-glance | 08:33 | |
*** comstud has joined #openstack-glance | 08:40 | |
*** TravT has quit IRC | 08:46 | |
*** TravT has joined #openstack-glance | 08:47 | |
*** TravT has quit IRC | 08:51 | |
*** TravT has joined #openstack-glance | 08:56 | |
*** haomaiwang has quit IRC | 09:01 | |
*** 21WAATL0G has joined #openstack-glance | 09:01 | |
*** TravT has quit IRC | 09:05 | |
*** TravT has joined #openstack-glance | 09:05 | |
*** MarcF has joined #openstack-glance | 09:08 | |
*** TravT has quit IRC | 09:10 | |
*** jistr has joined #openstack-glance | 09:14 | |
*** TravT has joined #openstack-glance | 09:14 | |
*** TravT has quit IRC | 09:19 | |
*** TravT has joined #openstack-glance | 09:23 | |
*** aix has joined #openstack-glance | 09:28 | |
*** TravT has quit IRC | 09:28 | |
*** sgotliv has joined #openstack-glance | 09:32 | |
*** TravT has joined #openstack-glance | 09:32 | |
openstackgerrit | Kairat Kushaev proposed openstack/python-glanceclient: Enhance description of instance-uuid option for image-create https://review.openstack.org/268013 | 09:41 |
*** TravT has quit IRC | 09:41 | |
*** 21WAATL0G has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-glance | 10:01 | |
*** davideagnello has joined #openstack-glance | 10:05 | |
*** davideagnello has quit IRC | 10:10 | |
*** e0ne has joined #openstack-glance | 10:11 | |
*** marcusvrn_ has joined #openstack-glance | 10:13 | |
*** mdnadeem has quit IRC | 10:25 | |
openstackgerrit | Rabi Mishra proposed openstack/python-glanceclient: Add find interface to glance v2 client https://review.openstack.org/268969 | 10:29 |
*** MarcF has quit IRC | 10:31 | |
*** hemanthm is now known as lone-rager|afk | 10:32 | |
*** lone-rager|afk is now known as hemanthm | 10:33 | |
*** haomaiwang has quit IRC | 10:36 | |
*** haomaiwang has joined #openstack-glance | 10:37 | |
*** mingdang1 has quit IRC | 10:40 | |
*** MattMan has quit IRC | 10:43 | |
*** MattMan has joined #openstack-glance | 10:43 | |
*** mdnadeem has joined #openstack-glance | 10:51 | |
*** TravT has joined #openstack-glance | 10:53 | |
*** dims has joined #openstack-glance | 10:54 | |
openstackgerrit | Niall Bunting proposed openstack/glance: Cache documentation about differences in files. https://review.openstack.org/267017 | 10:54 |
*** TravT has quit IRC | 11:01 | |
*** TravT has joined #openstack-glance | 11:06 | |
*** flaper87 has quit IRC | 11:11 | |
*** flaper87 has joined #openstack-glance | 11:11 | |
flaper87 | morning, everyone! | 11:24 |
openstackgerrit | Sergey Gotliv proposed openstack/glance_store: Add random read capabilities to the rbd store https://review.openstack.org/268908 | 11:24 |
flaper87 | Remember, today is REVIEW DAY! | 11:24 |
flaper87 | :D | 11:24 |
flaper87 | Hope you all can join | 11:24 |
kairat | morning flaper87 | 11:24 |
flaper87 | kairat: hey hey! :D | 11:24 |
*** TravT has quit IRC | 11:25 | |
flaper87 | kairat: any patches you've review that should get core attention? | 11:25 |
jokke_ | :) | 11:25 |
flaper87 | reviewed* | 11:25 |
flaper87 | jokke_: morning :D | 11:25 |
jokke_ | Morning flaper87 | 11:26 |
jokke_ | how is your week starting? | 11:26 |
flaper87 | So far so good but I haven't read a single email yet. It's about to get worse | 11:26 |
flaper87 | yours ? | 11:26 |
jokke_ | still caughing like crazy, but I think the battle agains cold is mostly won | 11:27 |
flaper87 | jokke_: awesome! Glad to hear that | 11:29 |
*** TravT has joined #openstack-glance | 11:29 | |
flaper87 | brb | 11:30 |
flaper87 | if there are reviews that require attention, please, point them out | 11:30 |
flaper87 | also, I think the glance dashboard is super useful for review days | 11:30 |
flaper87 | brb for realz | 11:30 |
kairat | flaper87, yep, there is some that satisfy the criteria we are established on the meeting | 11:31 |
kairat | https://review.openstack.org/#/c/241363/4 | 11:31 |
kairat | https://review.openstack.org/#/c/226295/ | 11:31 |
kairat | https://review.openstack.org/#/c/254712/ | 11:32 |
kairat | But most of them +2ed by you =) | 11:32 |
*** TravT has quit IRC | 11:39 | |
jokke_ | I just started the workflow for the Bandit change | 11:43 |
*** TravT has joined #openstack-glance | 11:43 | |
openstackgerrit | wangxiyuan proposed openstack/glance: Fix help command in cache manange and replicator https://review.openstack.org/262365 | 11:45 |
jokke_ | flaper87: started the workflows for translations | 11:51 |
flaper87 | jokke_: ++ | 11:52 |
jokke_ | flaper87: left the requirements for you ... there was one actual requirement change and one extras addition, rest is just license comments | 11:56 |
*** mingdang1 has joined #openstack-glance | 11:57 | |
flaper87 | will review now | 11:57 |
flaper87 | When doing reviews, favor patches that are part of our priorities | 11:59 |
flaper87 | There are some tagged as MitakaPriority | 11:59 |
flaper87 | Also, check if landing a "trivial" patch may conflict with "more important" patches | 11:59 |
flaper87 | Also, if a simple rebase would help a patch move along, feel free to do it yourself | 12:01 |
*** TravT has quit IRC | 12:01 | |
openstackgerrit | Niall Bunting proposed openstack/glance: Make cache config options clearer https://review.openstack.org/269011 | 12:04 |
*** smatzek has joined #openstack-glance | 12:05 | |
*** houming has quit IRC | 12:05 | |
*** TravT has joined #openstack-glance | 12:06 | |
*** dims has quit IRC | 12:10 | |
*** houming has joined #openstack-glance | 12:10 | |
*** TravT has quit IRC | 12:10 | |
*** haomaiwang has quit IRC | 12:14 | |
openstackgerrit | Kairat Kushaev proposed openstack/glance: Enhance description of instance-uuid option for image-create https://review.openstack.org/269016 | 12:14 |
*** TravT has joined #openstack-glance | 12:15 | |
*** haomaiwa_ has joined #openstack-glance | 12:16 | |
*** houming has quit IRC | 12:16 | |
*** houming has joined #openstack-glance | 12:17 | |
*** mdnadeem has quit IRC | 12:20 | |
*** TravT has quit IRC | 12:23 | |
*** TravT has joined #openstack-glance | 12:25 | |
*** TravT has quit IRC | 12:29 | |
*** TravT has joined #openstack-glance | 12:34 | |
*** chlong has joined #openstack-glance | 12:36 | |
*** TravT has quit IRC | 12:42 | |
*** TravT has joined #openstack-glance | 12:43 | |
flaper87 | mclaren: around? | 12:45 |
flaper87 | mclaren: re https://review.openstack.org/#/c/170564/ | 12:45 |
flaper87 | I've one question: How does this work interact with the trust's patch ? https://review.openstack.org/248681 | 12:46 |
flaper87 | mclaren: nevermind, I've asked this question already | 12:47 |
flaper87 | and you replied on one of the comments | 12:47 |
flaper87 | nikhil: rosmaita I know you mentioned you're ok with it on IRC last Friday but it'd be cool to have it in the spec as well: https://review.openstack.org/#/c/170564/ | 12:48 |
*** TravT has quit IRC | 12:52 | |
flaper87 | nikhil: rosmaita mclaren https://review.openstack.org/#/c/254710/ | 12:53 |
flaper87 | I remember nikhil saying the spec could be broken down further | 12:53 |
flaper87 | ativelkov ^ | 12:53 |
flaper87 | not sure what parts he was thinking of, though | 12:54 |
flaper87 | kairat: any idea where mfedosin is? :D | 12:54 |
flaper87 | mclaren: https://review.openstack.org/#/c/248681/ <- any final thoughts? comments? Do you think we can move this forward? | 12:55 |
openstackgerrit | Merged openstack/python-glanceclient: Enhance description of instance-uuid option for image-create https://review.openstack.org/268013 | 12:55 |
kairat | flaper87, just a second | 12:55 |
*** mfedosin has joined #openstack-glance | 12:56 | |
*** dshakhray has joined #openstack-glance | 12:56 | |
mfedosin | flaper87, sup :) | 12:56 |
flaper87 | mfedosin: BUDDYyyyYYYYYYYYYYYYYYYYYYYYYYYYYYYYY!!!! | 12:57 |
flaper87 | how's it going? | 12:57 |
*** TravT has joined #openstack-glance | 12:57 | |
flaper87 | Just wanted to sync on the v1->v2 work | 12:57 |
flaper87 | any chance you reviewed johnthetubaguy's comments? | 12:57 |
flaper87 | How's the xenplugin going? What can we help with? | 12:57 |
mfedosin | I rebased xenplugin work | 12:57 |
mfedosin | I wish all test will pass after that | 12:58 |
flaper87 | mclaren: mfedosin jokke_ https://review.openstack.org/#/c/266419/ | 12:58 |
flaper87 | I commented there, it'd be cool to get your feedback | 12:58 |
flaper87 | ( jokke_ commented as well) | 12:58 |
flaper87 | mfedosin: roger | 12:58 |
mfedosin | Also I read his comments, but had no chance to answer, because I was on business trip | 12:58 |
flaper87 | mfedosin: no problem. Lemme know if there's something I or johnthetubaguy can help with | 12:59 |
mfedosin | I'll answer today after finis my work with xwnplugin | 12:59 |
mfedosin | sure man | 12:59 |
mfedosin | btw, I congratulate you with your Principal status ;) | 12:59 |
flaper87 | mfedosin: thank you | 12:59 |
flaper87 | :) | 12:59 |
mfedosin | xenplugin fails :( http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/33/266933/4/3244/logs/var/log/xensource.log | 13:01 |
*** houming has quit IRC | 13:01 | |
mfedosin | but anyway we are close to the victory | 13:01 |
mfedosin | Server_helpers.exec exception_handler: Got exception XENAPI_PLUGIN_FAILURE: [ download_vhd2; TypeError; int argument required ] | 13:02 |
flaper87 | mfedosin: if you need support from folks from the xen team, let me know | 13:04 |
flaper87 | here to help chasing ppl down | 13:04 |
mfedosin | I'm going to ping Sean Dague about it, it's his code | 13:07 |
flaper87 | mfedosin: ++ | 13:07 |
flaper87 | johnthetubaguy: ^ | 13:07 |
openstackgerrit | Rabi Mishra proposed openstack/python-glanceclient: Add find interface to glance v2 client https://review.openstack.org/268969 | 13:08 |
*** TravT has quit IRC | 13:10 | |
*** dims has joined #openstack-glance | 13:11 | |
*** haomaiwa_ has quit IRC | 13:11 | |
openstackgerrit | Merged openstack/glance_store: Imported Translations from Zanata https://review.openstack.org/268601 | 13:11 |
*** edmondsw has joined #openstack-glance | 13:12 | |
*** TravT has joined #openstack-glance | 13:15 | |
openstackgerrit | Flavio Percoco proposed openstack/glance_store: Set documented default directory for filesystem https://review.openstack.org/233378 | 13:15 |
*** rcernin has quit IRC | 13:19 | |
*** aix has quit IRC | 13:21 | |
*** TravT has quit IRC | 13:24 | |
*** TravT has joined #openstack-glance | 13:24 | |
openstackgerrit | Merged openstack/glance_store: Updated from global requirements https://review.openstack.org/268443 | 13:25 |
*** links has quit IRC | 13:33 | |
openstackgerrit | Merged openstack/python-glanceclient: Updated from global requirements https://review.openstack.org/268509 | 13:36 |
*** ducttape_ has joined #openstack-glance | 13:36 | |
*** TravT has quit IRC | 13:42 | |
*** haomaiwang has joined #openstack-glance | 13:46 | |
*** TravT has joined #openstack-glance | 13:47 | |
*** aix has joined #openstack-glance | 13:47 | |
*** TravT has quit IRC | 14:00 | |
*** edmondsw has quit IRC | 14:02 | |
*** abhishekk has quit IRC | 14:10 | |
*** ducttape_ has quit IRC | 14:12 | |
*** jaypipes has joined #openstack-glance | 14:25 | |
*** julim has joined #openstack-glance | 14:26 | |
*** ducttape_ has joined #openstack-glance | 14:32 | |
*** mingdang1 has quit IRC | 14:39 | |
*** bpoulos has joined #openstack-glance | 14:44 | |
*** TravT has joined #openstack-glance | 14:47 | |
*** keekz has joined #openstack-glance | 14:47 | |
*** TravT has quit IRC | 14:51 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:55 | |
*** TravT has joined #openstack-glance | 14:56 | |
*** haypo has joined #openstack-glance | 14:59 | |
haypo | flaper87: hey, i need https://review.openstack.org/257202 indirectly to port projects to python 3 | 14:59 |
haypo | flaper87: it's a backport, the change was already merged into glance master | 15:00 |
haypo | ian had complains, but i hope that i replied to most or all of them in https://review.openstack.org/#/c/250731/ | 15:00 |
haypo | is ian cordasco on IRC? | 15:00 |
sigmavirus24 | haypo: I am Ian | 15:00 |
*** burgerk has joined #openstack-glance | 15:00 | |
haypo | sigmavirus24: hey. i'm coming back for my "old" backport" :-) would you mind reviewing it please? | 15:01 |
* flaper87 clicks | 15:01 | |
*** ducttape_ has quit IRC | 15:03 | |
sigmavirus24 | ah that one haypo | 15:03 |
haypo | the patch only impacts python 3 | 15:05 |
haypo | it's required to fix a bug in oslo.context | 15:05 |
haypo | sigmavirus24: i do understand your complains :) they were discussed in the oslo.context change | 15:07 |
* sigmavirus24 is re-reviewing it | 15:08 | |
* flaper87 reviewed and dropped comments | 15:10 | |
*** TravT has quit IRC | 15:15 | |
jokke_ | haypo, flaper87 & sigmavirus24: I really don't understand how glance tests affects oslo.context? Is glance context middleware returning bytes? | 15:25 |
haypo | jokke_: it looks like glance has an unit test on the oslo.context API | 15:27 |
haypo | jokke_: IMHO the test should be removed, but it don't really care :) | 15:27 |
haypo | jokke_: i want to break oslo.context API on Python 3, which would break the glance gate | 15:27 |
jokke_ | haypo: it tests glance.api.middleware.context not oslo.context | 15:28 |
haypo | jokke_: ah yes. sorry, i don't recall the whole context | 15:29 |
jokke_ | so we really shouldn't be affected what ever you do with oslo.context | 15:29 |
sigmavirus24 | jokke_: that isn't really a valid backport anyway, right? | 15:30 |
haypo | jokke_: glance *is* affected by the oslo.context change, but only on python 3 | 15:30 |
*** chlong has quit IRC | 15:31 | |
haypo | sigmavirus24: do you really think that Glance Liberty is deployed on top of Python 3? | 15:32 |
haypo | i'm not even sure that it will work with Mitaka :-/ i just checked, not all glance unit tests pass on python 3 right now | 15:32 |
jokke_ | haypo: ah glance.context is just subclass of oslo_context | 15:32 |
sigmavirus24 | haypo: I'm talking about the specific review on stable/liberty | 15:32 |
jokke_ | makes sense now | 15:32 |
sigmavirus24 | haypo: I'm not talkign about the change to oslo.context | 15:33 |
haypo | sigmavirus24: both are linked "jokke_> haypo: ah glance.context is just subclass of oslo_context" | 15:33 |
sigmavirus24 | haypo: the change you are making/have made is backwards incompatible and that's irrefutable. I'm talking about the specific backport change | 15:33 |
flaper87 | fwiw, my take is that if this affects oslo.context and the py3 work, I think it's fine to land it as it only affects glance's tests and, more importantly, doesn't break them | 15:33 |
flaper87 | Of course, understanding the need is important and I'm all for that | 15:33 |
jokke_ | haypo: glance stable/liberty uses oslo stable/liberty so the backport should not be valid question here | 15:34 |
flaper87 | just my $0.02 | 15:34 |
haypo | jokke_: "glance stable/liberty uses oslo stable/liberty so the backport should not be valid question here" it's no more true | 15:34 |
haypo | jokke_: there are no more "oslo liberty" | 15:34 |
jokke_ | haypo: is it grenade or what is blowing if that does not land to liberty? | 15:34 |
sigmavirus24 | haypo: https://github.com/openstack/oslo.context/commits/stable/liberty | 15:34 |
haypo | jokke_: it becomes "oslo", no more oslo version per openstack release | 15:34 |
jokke_ | haypo: are you saying that oslo is removing stable releases in middle of the support cycle? | 15:35 |
*** ninag has joined #openstack-glance | 15:36 | |
haypo | jokke_: if oslo.context changes, python3 gate of glance on liberty can start failing | 15:36 |
sigmavirus24 | So I'll reiterate what I said before, haypo. This new version of oslo.context with that fix should be 2.0, not a 1.x | 15:37 |
haypo | jokke_: the idea is more than the old versions of openstack services must work with the latest oslo releases | 15:37 |
sigmavirus24 | That or if we are so very confident that no one is using glance on Python 3, we should make the gate non-voting so oslo can violate OpenStack semver in minor versions | 15:38 |
dims | jokke_ : please see this https://review.openstack.org/#/c/226157/ | 15:38 |
haypo | sigmavirus24: please see requirements.txt of glance liberty. oslo.context is not capped to <=x.y. it's just "oslo.context>=0.2" | 15:38 |
sigmavirus24 | haypo: right, I'm saying we should cap it to <2.0 so oslo.context can release 2.0 with this backwards incompatible change | 15:39 |
dims | sigmavirus24 : lifeless, mriedem and dhellman are working a plan to come up with a release-constraints file that live in stable/liberty branches | 15:39 |
haypo | sigmavirus24: lifeless doesn't want to cap anything, see https://review.openstack.org/#/c/226157/ | 15:40 |
*** mtanino has joined #openstack-glance | 15:40 | |
haypo | sorry, i'm just repeating what i heard. and all these things are going against my specific oslo.context change :) | 15:40 |
sigmavirus24 | haypo: I understand. And fundamentally I agree with your change as necessary | 15:41 |
haypo | sigmavirus24: we can talk forever about the best way to maintain compatiblity, cap versions, etc. but seriously, i'm only changing a minor unit test of glance, i don't understand why it requires so much explanation | 15:41 |
haypo | the change is already merged in glance/master, oslo.middleware/master+liberty and neutron/master+liberty | 15:41 |
haypo | it just makes my life easier, my work on porting openstack to python 3 | 15:42 |
*** mfedosin has quit IRC | 15:42 | |
sigmavirus24 | haypo: I am just currently under the impression that backports of purely test fixes are not actually valid backports | 15:42 |
jokke_ | so if I undertand correctly, to allow libd dropping stable branches we need to bring back upper caps on stable services which was the original cause of huge amounts of issues where solution was to create stable libs so that the libs such as oslo coud move forward | 15:43 |
sigmavirus24 | (See also backporting changes to accommodate test failures for newer versions of mock on stable/kilo) | 15:43 |
jokke_ | libs | 15:43 |
haypo | sigmavirus24: for mock, it's was a great changes | 15:43 |
haypo | sigmavirus24: mock became more strict, it helps to fix a big number of bugs in tests | 15:43 |
sigmavirus24 | haypo: I understand that it was a good change in mock (just like this is a good change in oslo.context) | 15:44 |
sigmavirus24 | I'm not criticizing mock for changing that. I'm pointing out that there is a stable backport policy and that we're now insisting on inconsistency within that policy | 15:44 |
haypo | sigmavirus24: please come back to the beginning. the change only impacts python 3. it's just impossible to run glance on top of python 3 | 15:44 |
jokke_ | sigmavirus24: ++ | 15:44 |
haypo | sigmavirus24: so i just don't understand why you care so much | 15:44 |
jokke_ | haypo: so instead of masking that those tests passes we should remove them from stable/liberty | 15:45 |
haypo | jokke_: i don't really care how the issue is handled, i just want to prevent breaking the world with a new version of oslo.context | 15:46 |
flaper87 | jokke_: why would you remove them? I don't follow | 15:46 |
flaper87 | So, I understand jokke_ and sigmavirus24 concerns and I think they both raise good points re process and policies | 15:46 |
haypo | maybe the liberty gate of glance already use an old version of oslo.context and it's just fine | 15:46 |
flaper87 | however, as I said before, I don't think we should block this patch | 15:46 |
flaper87 | It does help another team, it has a limited impact on the *tests* and it does not break backwards compatibility | 15:47 |
jokke_ | flaper87: why do we have py3 tests on stable/libery (specifically why we have py3 tests passing and giving wrong indication of the status) as glance stable/liberty does not work with py3 | 15:47 |
haypo | jokke_: i made the py3 gate voting to help me to ensure that no regression is introduced in glance | 15:47 |
haypo | jokke_: before, it was common to reintroduce py2 only code | 15:48 |
haypo | which prevent to make any real progress on the long term | 15:48 |
jokke_ | haypo: don't get me wrong I seriously appreciate your work towards py3 and your concerns of breaking stable ... my concerns/annoyance has nothing to do about those nor the actual patch in hand but the overall situation we are driving ourselves to atm. | 15:49 |
flaper87 | jokke_: my understanding is that those tests don't just test py3 | 15:49 |
*** mfedosin has joined #openstack-glance | 15:50 | |
*** ducttape_ has joined #openstack-glance | 15:50 | |
flaper87 | so, that's why I'm asking why we should remove them instead of just landing this small change that allows for running them under py3 for oslo.context | 15:50 |
haypo | flaper87, jokke_ : another way to handle the issue is to simply remove the py3 gate from liberty since it's completly useless on liberty | 15:50 |
haypo | (just to confirm that the change will not impact anyone) | 15:50 |
jokke_ | haypo: my point, thank you | 15:50 |
flaper87 | haypo: if that doesn't break you, then fine | 15:51 |
flaper87 | I thought, for some reason, that was a requirement for you | 15:51 |
haypo | flaper87: it's only a requirements for the master branch | 15:51 |
haypo | requirement* | 15:51 |
flaper87 | in that case, we have a solution | 15:52 |
flaper87 | :D | 15:52 |
flaper87 | we can abandon the backport and disable the py3 gate | 15:52 |
flaper87 | that saves some cycles | 15:52 |
jokke_ | as we have py3 gate job in master and backports needs to land master first the possibilities of regression are miniamal and even that would happen on liberty it won't break anyone as no-one can't be running liberty glance on py3 | 15:52 |
flaper87 | and we're all happy | 15:52 |
flaper87 | pls, update the bug and patch | 15:52 |
haypo | flaper87: i prefer to modify the test | 15:52 |
flaper87 | jokke_: sigmavirus24 haypo thanks for following up on this | 15:52 |
flaper87 | haypo: we can modify it in master | 15:52 |
haypo | flaper87: my change only accepts a different type, it's really minor | 15:53 |
flaper87 | and ignore stable/liberty | 15:53 |
flaper87 | I'm fine either way but, if we have a solution that follows the policy we have in place, I prefer that one | 15:53 |
haypo | come one guys, 3 months to change a single line. this story is just crazy. https://review.openstack.org/#/c/250731/6/oslo_context/context.py | 15:54 |
haypo | opening a new change for project-config will just take forever | 15:55 |
haypo | another round of crazy comments | 15:56 |
haypo | why can't you just take my patch for glance? | 15:56 |
haypo | it's so sad to have to spend 3 months to change a single line in openstack | 15:57 |
haypo | this project make me crazy | 15:57 |
flaper87 | I've +2'd that patch and I'm happy with letting it in but I'd still ike the py3 job for stable/liberty to be dissabled at this point | 15:57 |
flaper87 | haypo: that's unfair. I'm sorry. The whole team has been tracking py3 patches like crazy and making sure those are not blocked | 15:58 |
flaper87 | it's really unfair to have folks coming into IRC complaining just because there are some GLOBAL policies in place | 15:58 |
flaper87 | and I'm not happy with that | 15:58 |
flaper87 | We can track all the py3 related ptaches that have been reviewed and landed during Mitaka | 15:58 |
flaper87 | I +2'd like 3 just today | 15:58 |
flaper87 | I've been under the same situation and I've never come to a project saying it makes me crazy and sad and whatnot | 15:59 |
*** vijendar has joined #openstack-glance | 15:59 | |
flaper87 | so, I'm sorry if I'm going ballistic here but I'm getting to the point where I've had enough of ppl just pointing fingers like crazy at glance w/o noticing the project is coming out of a shitload | 15:59 |
*** ducttape_ has quit IRC | 15:59 | |
* flaper87 breaths | 16:00 | |
*** links has joined #openstack-glance | 16:00 | |
*** ducttape_ has joined #openstack-glance | 16:02 | |
*** haomaiwang has quit IRC | 16:04 | |
*** haomaiwa_ has joined #openstack-glance | 16:05 | |
*** haomaiwa_ has quit IRC | 16:05 | |
*** harshs has joined #openstack-glance | 16:05 | |
*** edmondsw has joined #openstack-glance | 16:16 | |
*** 17WABKWLD has joined #openstack-glance | 16:19 | |
*** TravT has joined #openstack-glance | 16:19 | |
*** belmoreira has quit IRC | 16:22 | |
*** links has quit IRC | 16:23 | |
*** Steap has joined #openstack-glance | 16:24 | |
*** 17WABKWLD has quit IRC | 16:24 | |
*** TravT has quit IRC | 16:28 | |
*** vijendar has quit IRC | 16:32 | |
haypo | sigmavirus24, flaper87, jokke_ : i proposed https://review.openstack.org/#/c/269178/ "Remove glance python34 jobs on Liberty" | 16:35 |
*** haypo has left #openstack-glance | 16:38 | |
-openstackstatus- NOTICE: Gerrit is restarting quickly as a workaround for performance degradation | 16:48 | |
*** vijendar has joined #openstack-glance | 16:49 | |
*** kebray has joined #openstack-glance | 16:49 | |
*** kebray has quit IRC | 16:51 | |
*** mfedosin has quit IRC | 16:51 | |
*** haomaiwang has joined #openstack-glance | 16:51 | |
*** TravT has joined #openstack-glance | 16:52 | |
*** julim has quit IRC | 16:54 | |
*** TravT has quit IRC | 16:56 | |
*** julim has joined #openstack-glance | 16:57 | |
openstackgerrit | Julien Danjou proposed openstack/glance: utils: remove PrettyTable custom class in favor of the eponym libary https://review.openstack.org/234340 | 16:59 |
ativelkov | Courtesy meeting reminder: nikhil_k, ativelkov, mfedosin, docaedo - artifacts subteam meeting at #openstack-meeting-alt | 17:00 |
*** TravT has joined #openstack-glance | 17:01 | |
*** burgerk has quit IRC | 17:02 | |
*** haomaiwang has quit IRC | 17:02 | |
*** dims_ has joined #openstack-glance | 17:05 | |
*** dims has quit IRC | 17:08 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/glance: Add metadefs for Cinder volume type configuration https://review.openstack.org/268357 | 17:09 |
*** davideagnello has joined #openstack-glance | 17:10 | |
*** Steap has left #openstack-glance | 17:10 | |
*** e0ne has quit IRC | 17:19 | |
*** TravT has quit IRC | 17:19 | |
*** TravT has joined #openstack-glance | 17:24 | |
*** ducttape_ has quit IRC | 17:24 | |
*** achanda has joined #openstack-glance | 17:30 | |
*** markus_z has quit IRC | 17:32 | |
*** TravT has quit IRC | 17:32 | |
*** TravT has joined #openstack-glance | 17:33 | |
*** TravT has quit IRC | 17:37 | |
*** dshakhray has quit IRC | 17:38 | |
*** groen692 has quit IRC | 17:38 | |
*** groen692 has joined #openstack-glance | 17:39 | |
*** ducttape_ has joined #openstack-glance | 17:39 | |
*** sb- is now known as strictlyb | 17:42 | |
*** TravT has joined #openstack-glance | 17:42 | |
*** avarner__ is now known as avarner | 17:47 | |
*** groen692 has quit IRC | 17:48 | |
*** vijendar has quit IRC | 17:50 | |
openstackgerrit | Alexander Tivelkov proposed openstack/glance: Move Glance Artifact Repository API to separate endpoint https://review.openstack.org/255274 | 17:51 |
*** ducttape_ has quit IRC | 17:53 | |
*** TravT has quit IRC | 17:55 | |
*** TravT has joined #openstack-glance | 17:56 | |
*** TravT has quit IRC | 18:00 | |
*** vijendar has joined #openstack-glance | 18:03 | |
*** TravT has joined #openstack-glance | 18:05 | |
*** sgotliv has quit IRC | 18:07 | |
openstackgerrit | Olena Logvinova proposed openstack/glance: [docs] Adds Domain API sub-section https://review.openstack.org/259527 | 18:09 |
*** TravT has quit IRC | 18:13 | |
*** TravT has joined #openstack-glance | 18:14 | |
*** jistr has quit IRC | 18:17 | |
*** david-lyle has quit IRC | 18:23 | |
*** TravT has quit IRC | 18:23 | |
openstackgerrit | Nanke_Liu (lnk) proposed openstack/glance: Using LOG.warning replace LOG.warn https://review.openstack.org/262237 | 18:27 |
*** TravT has joined #openstack-glance | 18:28 | |
*** kebray has joined #openstack-glance | 18:30 | |
*** harshs has quit IRC | 18:37 | |
*** TravT has quit IRC | 18:37 | |
*** bpoulos has quit IRC | 18:38 | |
*** david-lyle has joined #openstack-glance | 18:39 | |
*** TravT has joined #openstack-glance | 18:42 | |
*** aix has quit IRC | 18:42 | |
*** david-lyle has quit IRC | 18:46 | |
*** kebray has quit IRC | 18:50 | |
*** TravT has quit IRC | 18:51 | |
*** TravT has joined #openstack-glance | 18:55 | |
*** burgerk has joined #openstack-glance | 19:02 | |
*** haomaiwang has joined #openstack-glance | 19:03 | |
*** TravT has quit IRC | 19:04 | |
*** TravT has joined #openstack-glance | 19:05 | |
*** avarner has quit IRC | 19:06 | |
*** haomaiwang has quit IRC | 19:08 | |
*** ducttape_ has joined #openstack-glance | 19:09 | |
*** david-lyle has joined #openstack-glance | 19:10 | |
*** TravT has quit IRC | 19:13 | |
*** achanda has quit IRC | 19:14 | |
*** ducttape_ has quit IRC | 19:15 | |
*** TravT has joined #openstack-glance | 19:18 | |
*** TravT has quit IRC | 19:27 | |
*** david-lyle_ has joined #openstack-glance | 19:27 | |
*** dslev has joined #openstack-glance | 19:28 | |
*** david-lyle has quit IRC | 19:31 | |
*** TravT has joined #openstack-glance | 19:32 | |
*** vijendar has quit IRC | 19:32 | |
*** vijendar has joined #openstack-glance | 19:32 | |
*** vijendar has joined #openstack-glance | 19:33 | |
*** vijendar has quit IRC | 19:33 | |
*** david-lyle_ has quit IRC | 19:37 | |
*** sgotliv has joined #openstack-glance | 19:39 | |
*** david-lyle has joined #openstack-glance | 19:39 | |
*** achanda has joined #openstack-glance | 19:40 | |
*** TravT has quit IRC | 19:40 | |
*** TravT has joined #openstack-glance | 19:41 | |
*** vijendar has joined #openstack-glance | 19:42 | |
*** david-lyle has quit IRC | 19:46 | |
flaper87 | who is around to help with the review day? | 19:51 |
*** achanda has quit IRC | 19:53 | |
openstackgerrit | Flavio Percoco proposed openstack/glance: Make the task's API admin only by default https://review.openstack.org/265448 | 19:54 |
*** TravT has quit IRC | 19:55 | |
flaper87 | sabari: rosmaita nikhil sigmavirus24 around? | 19:55 |
flaper87 | hemanthm: ^ | 19:55 |
nikhil | in 15 | 19:55 |
* flaper87 sets the countdown | 19:55 | |
flaper87 | :P | 19:55 |
sigmavirus24 | flaper87: I am | 19:56 |
flaper87 | sigmavirus24: w000h000! | 19:56 |
flaper87 | :D | 19:56 |
sigmavirus24 | nikhil: 15 seconds, minutes, hours, days, weeks, months, years, epochs? | 19:56 |
flaper87 | thanks | 19:56 |
flaper87 | just want to know who I can poke for reviews | 19:56 |
*** e0ne has joined #openstack-glance | 19:56 | |
flaper87 | sigmavirus24: any patch you want me to review? | 19:56 |
sigmavirus24 | flaper87: I need to work on two patches. They're tricky though because I want them to be backport-friendly | 19:57 |
sigmavirus24 | I just haven't had time. (I was at the OSSP midcycle last week hacking on bandit) | 19:57 |
flaper87 | sigmavirus24: oh ok! no worries. Any chance you can dedicate time to more reviews today? | 19:57 |
sigmavirus24 | flaper87: certainly | 20:00 |
*** kebray has joined #openstack-glance | 20:08 | |
*** kebray has quit IRC | 20:10 | |
nikhil | sigmavirus24: haha, mins but I guess too late to answer | 20:15 |
sigmavirus24 | nikhil: never too late to answer :P | 20:15 |
nikhil | flaper87: please do | 20:16 |
nikhil | sigmavirus24: :) | 20:16 |
sigmavirus24 | nikhil: I just thought I'd make a teensy joke about the importance of units in measurements | 20:16 |
nikhil | haha, indeed | 20:16 |
openstackgerrit | Flavio Percoco proposed openstack/glance_store: Remove the gridfs driver https://review.openstack.org/255583 | 20:16 |
flaper87 | sigmavirus24: for example, ^ (I just rebased it and there was a conflict in setup.cfg) | 20:17 |
flaper87 | sabari: ^ mind restoring your +2 ? | 20:18 |
flaper87 | :D | 20:18 |
flaper87 | or nikhil, mind adding one ^ ? | 20:18 |
flaper87 | :P | 20:18 |
*** kebray has joined #openstack-glance | 20:19 | |
flaper87 | We're in M-2 release week and we still have a couple of patches we could land before the cut | 20:19 |
openstackgerrit | Flavio Percoco proposed openstack/glance_store: Deprecate the S3 driver https://review.openstack.org/266077 | 20:24 |
openstackgerrit | Flavio Percoco proposed openstack/glance_store: Document supported drivers and maintainers https://review.openstack.org/258237 | 20:24 |
nikhil | ack | 20:24 |
*** vijendar has quit IRC | 20:29 | |
*** vijendar has joined #openstack-glance | 20:36 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 20:37 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 20:37 | |
flaper87 | nikhil: https://review.openstack.org/#/c/260687/ <- stable liberty patch | 20:39 |
nikhil | flaper87: just +Aed https://review.openstack.org/#/c/255583/6 | 20:40 |
flaper87 | nikhil: <3 | 20:40 |
flaper87 | thanks | 20:40 |
*** TravT has joined #openstack-glance | 20:41 | |
nikhil | flaper87: reviewing this https://review.openstack.org/#/c/266077/ now and will review the documentation one later today | 20:41 |
nikhil | flaper87: nit on https://review.openstack.org/#/c/266077/2 | 20:44 |
*** TravT has quit IRC | 20:46 | |
openstackgerrit | Flavio Percoco proposed openstack/glance_store: Deprecate the S3 driver https://review.openstack.org/266077 | 20:46 |
flaper87 | nikhil: thanks, fixed | 20:46 |
nikhil | flaper87: +2ed | 20:47 |
*** dslevin has joined #openstack-glance | 20:48 | |
* flaper87 hugs nikhil | 20:49 | |
flaper87 | nikhil: has the glance dashboard been of any help for you? | 20:49 |
nikhil | flaper87: yeah, works well for me | 20:49 |
nikhil | from time to time | 20:50 |
flaper87 | glad to hear. I don't use it every day myself. But for this kind of review sprints, it's turned out to be very useful | 20:52 |
nikhil | agreed | 20:54 |
*** haomaiwang has joined #openstack-glance | 21:03 | |
flaper87 | sigmavirus24: https://review.openstack.org/#/c/259963/ | 21:05 |
flaper87 | it'd be great to have you review that patch | 21:05 |
flaper87 | it's API related | 21:05 |
flaper87 | nikhil: btw, if you get a chance, could you comment on ativelkov's spec for glare? | 21:05 |
flaper87 | I know you had some comments | 21:05 |
nikhil | hmm surely | 21:07 |
nikhil | flaper87: in a few mins | 21:07 |
flaper87 | nikhil: no rush, just trying to highlight things that come to mind | 21:07 |
flaper87 | sigmavirus24: https://review.openstack.org/#/c/266419 <- here as well :D | 21:07 |
sigmavirus24 | flaper87: also running tests on something else | 21:07 |
flaper87 | sigmavirus24: ++ | 21:08 |
*** haomaiwang has quit IRC | 21:08 | |
*** ducttape_ has joined #openstack-glance | 21:11 | |
*** achanda has joined #openstack-glance | 21:12 | |
*** ducttape_ has quit IRC | 21:15 | |
*** achanda has quit IRC | 21:17 | |
openstackgerrit | Ian Cordasco proposed openstack/glance: Add __setitem__ to ImageTarget policy wrapper https://review.openstack.org/269277 | 21:18 |
sigmavirus24 | flaper87: ^ | 21:18 |
*** haomaiwang has joined #openstack-glance | 21:19 | |
* flaper87 clicks | 21:22 | |
*** avarner has joined #openstack-glance | 21:23 | |
*** haomaiwang has quit IRC | 21:24 | |
*** kebray has quit IRC | 21:28 | |
*** mdnadeem has joined #openstack-glance | 21:29 | |
*** gberginc has quit IRC | 21:46 | |
*** TravT has joined #openstack-glance | 21:46 | |
*** kebray has joined #openstack-glance | 21:49 | |
*** burgerk has quit IRC | 21:53 | |
*** TravT has quit IRC | 21:55 | |
*** TravT has joined #openstack-glance | 22:00 | |
*** edmondsw has quit IRC | 22:02 | |
*** mdnadeem has quit IRC | 22:04 | |
*** TravT has quit IRC | 22:04 | |
*** kebray has quit IRC | 22:05 | |
*** TravT has joined #openstack-glance | 22:09 | |
*** jamielennox|away is now known as jamielennox | 22:09 | |
*** chlong has joined #openstack-glance | 22:12 | |
*** dims has joined #openstack-glance | 22:17 | |
*** dims_ has quit IRC | 22:18 | |
*** TravT has quit IRC | 22:18 | |
*** julim has quit IRC | 22:19 | |
*** haomaiwang has joined #openstack-glance | 22:20 | |
*** TravT has joined #openstack-glance | 22:23 | |
*** haomaiwang has quit IRC | 22:25 | |
*** TravT has quit IRC | 22:31 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Updated from global requirements https://review.openstack.org/266996 | 22:40 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/glance_store: Updated from global requirements https://review.openstack.org/269318 | 22:40 |
*** smatzek has quit IRC | 22:42 | |
*** dims has quit IRC | 22:44 | |
*** dslevin has quit IRC | 22:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-glanceclient: Updated from global requirements https://review.openstack.org/269345 | 22:45 |
*** dims has joined #openstack-glance | 22:47 | |
*** smatzek has joined #openstack-glance | 22:48 | |
*** achanda has joined #openstack-glance | 22:48 | |
*** sgotliv has quit IRC | 22:49 | |
*** achanda has quit IRC | 22:50 | |
*** achanda has joined #openstack-glance | 22:52 | |
*** TravT has joined #openstack-glance | 22:55 | |
*** TravT has quit IRC | 22:59 | |
*** smatzek has quit IRC | 23:00 | |
*** TravT has joined #openstack-glance | 23:04 | |
*** dims has quit IRC | 23:05 | |
*** ducttape_ has joined #openstack-glance | 23:06 | |
*** dims has joined #openstack-glance | 23:07 | |
*** vijendar has quit IRC | 23:12 | |
*** TravT has quit IRC | 23:13 | |
*** achanda has quit IRC | 23:14 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 23:15 | |
*** TravT has joined #openstack-glance | 23:17 | |
*** haomaiwang has joined #openstack-glance | 23:21 | |
*** TravT has quit IRC | 23:22 | |
openstackgerrit | Merged openstack/python-glanceclient: Add help the ability to sort images with multiple keys https://review.openstack.org/263277 | 23:23 |
*** achanda has joined #openstack-glance | 23:23 | |
*** john5223 has quit IRC | 23:26 | |
*** haomaiwang has quit IRC | 23:26 | |
*** john5223 has joined #openstack-glance | 23:28 | |
*** ninag has quit IRC | 23:30 | |
*** ativelkov has quit IRC | 23:31 | |
*** ducttape_ has quit IRC | 23:32 | |
*** tsufiev has quit IRC | 23:32 | |
*** ativelkov has joined #openstack-glance | 23:35 | |
*** dims has quit IRC | 23:35 | |
*** tsufiev has joined #openstack-glance | 23:39 | |
*** dims has joined #openstack-glance | 23:39 | |
*** dslev has quit IRC | 23:40 | |
*** achanda has quit IRC | 23:50 | |
*** dslevin has joined #openstack-glance | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!