Wednesday, 2016-02-03

*** jamielennox|away is now known as jamielennox00:02
*** azbiswas_ has quit IRC00:15
*** jamielennox is now known as jamielennox|away00:38
*** jamielennox|away is now known as jamielennox00:39
*** ducttape_ has quit IRC00:42
*** harshs has quit IRC01:02
*** ducttape_ has joined #openstack-glance01:06
*** ninag has quit IRC01:12
*** haomaiwang has joined #openstack-glance01:12
*** haomaiwang has quit IRC01:27
*** jamielennox is now known as jamielennox|away01:30
*** achanda has joined #openstack-glance01:32
*** flwang has quit IRC01:39
*** mtanino has quit IRC01:45
*** flwang has joined #openstack-glance01:53
*** avarner has quit IRC01:54
*** kebray has joined #openstack-glance01:59
*** kebray has quit IRC02:03
*** kebray has joined #openstack-glance02:04
*** haomaiwang has joined #openstack-glance02:23
*** jamielennox|away is now known as jamielennox02:31
openstackgerritRen Qiaowei proposed openstack/glance: Replace exit() by sys.exit()  https://review.openstack.org/27452302:32
*** dims_ has quit IRC02:40
*** azbiswas has joined #openstack-glance02:48
*** harshs has joined #openstack-glance02:57
*** harshs has quit IRC02:59
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-glance03:01
openstackgerritzwei proposed openstack/python-glanceclient: v2 - "readOnly" key should be used in schemas  https://review.openstack.org/26940603:02
*** links has joined #openstack-glance03:06
*** ducttape_ has quit IRC03:17
*** ducttape_ has joined #openstack-glance03:21
*** azbiswas has quit IRC03:24
openstackgerritMerged openstack/glance: Change exception format checks in artifact tests  https://review.openstack.org/27519003:26
*** ducttape_ has quit IRC03:28
*** mingdang1 has joined #openstack-glance03:38
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/27465503:38
*** ducttape_ has joined #openstack-glance03:41
openstackgerritFei Long Wang proposed openstack/glance-specs: Spec for improve image quota  https://review.openstack.org/26666703:43
*** ducttape_ has quit IRC03:44
openstackgerritFei Long Wang proposed openstack/glance-specs: Spec for improve image quota  https://review.openstack.org/26666703:52
openstackgerritFei Long Wang proposed openstack/glance-specs: Spec for improve image quota  https://review.openstack.org/26666703:53
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-glance04:01
*** ducttape_ has joined #openstack-glance04:02
*** ducttape_ has quit IRC04:02
*** ducttape_ has joined #openstack-glance04:03
*** jamielennox is now known as jamielennox|away04:09
openstackgerritzhongshengping proposed openstack/glance: Replace deprecated LOG.warn with warning  https://review.openstack.org/27516504:17
openstackgerritzhongshengping proposed openstack/glance: Replace deprecated LOG.warn with warning  https://review.openstack.org/27516504:18
*** achanda has quit IRC04:28
*** azbiswas has joined #openstack-glance04:33
*** azbiswas has quit IRC04:35
*** harshs has joined #openstack-glance04:39
*** achanda has joined #openstack-glance04:43
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-glance05:01
*** ducttape_ has quit IRC05:03
*** sudipto_ has joined #openstack-glance05:27
*** harshs has quit IRC05:35
*** sgotliv has joined #openstack-glance05:43
*** haomaiwang has quit IRC06:01
*** haomaiwang has joined #openstack-glance06:01
*** harshs has joined #openstack-glance06:06
*** sgotliv has quit IRC06:19
*** groen692 has joined #openstack-glance06:32
*** zul has joined #openstack-glance06:33
*** zul has quit IRC06:51
*** abhishekk has joined #openstack-glance06:57
abhishekkkairat: hi you around?06:57
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-glance07:01
*** harshs has quit IRC07:07
*** AJaeger has joined #openstack-glance07:26
openstackgerritAndreas Jaeger proposed openstack/glance: Fix setup.cfg  https://review.openstack.org/27553907:29
openstackgerritvenkatamahesh proposed openstack/glance: Fix some word spellings  https://review.openstack.org/27554307:43
*** belmoreira has joined #openstack-glance07:47
*** mvk_ has quit IRC07:56
*** haomaiwang has quit IRC08:01
*** haomaiwa_ has joined #openstack-glance08:01
*** dshakhray has joined #openstack-glance08:12
*** jistr has joined #openstack-glance08:14
*** boris-42 has joined #openstack-glance08:18
*** sgotliv has joined #openstack-glance08:21
*** markus_z has joined #openstack-glance08:21
*** jistr is now known as jistr|mtg08:22
*** akanksha_ has joined #openstack-glance08:25
*** sinese has joined #openstack-glance08:25
*** nikhil has joined #openstack-glance08:25
*** mvk_ has joined #openstack-glance08:27
*** nikhil_k has quit IRC08:28
*** sudipto_ has quit IRC08:35
*** zul has joined #openstack-glance08:40
*** mfedosin has joined #openstack-glance08:44
*** sudipto_ has joined #openstack-glance08:52
*** haomaiwa_ has quit IRC09:01
*** haomaiwa_ has joined #openstack-glance09:01
*** sudipto_ has quit IRC09:13
*** achanda has quit IRC09:14
*** achanda has joined #openstack-glance09:18
*** chlong has quit IRC09:20
*** markus_z has quit IRC09:22
*** achanda has quit IRC09:24
*** mfedosin has quit IRC09:24
*** sudipto_ has joined #openstack-glance09:29
*** mfedosin has joined #openstack-glance09:40
*** e0ne has joined #openstack-glance09:45
*** mfedosin has quit IRC09:48
abhishekkkairat: hi you around?09:49
*** AJaeger has quit IRC09:49
kairatabhishekk, Hey!09:49
abhishekkhi, could you please review https://review.openstack.org/26128809:50
abhishekkkairat: ^^ when you get time?09:50
kairatI will try to review tomorrow09:50
kairatI have some urgent work today=(09:51
abhishekksure, thank you and congrats to be a core09:51
kairatthanks09:51
abhishekk:)09:51
openstackgerritvenkatamahesh proposed openstack/python-glanceclient: Fix spellings for two words  https://review.openstack.org/27559709:57
*** gjayavelu has joined #openstack-glance09:59
*** haomaiwa_ has quit IRC10:01
*** haomaiwang has joined #openstack-glance10:01
*** gjayavelu has quit IRC10:02
*** zul has quit IRC10:03
*** mfedosin has joined #openstack-glance10:03
*** e0ne has quit IRC10:05
*** zul has joined #openstack-glance10:07
*** AJaeger has joined #openstack-glance10:11
*** d0ugal has quit IRC10:42
*** d0ugal has joined #openstack-glance10:42
*** e0ne has joined #openstack-glance10:51
*** mingdang1 has quit IRC10:54
*** zul has quit IRC11:00
*** haomaiwang has quit IRC11:01
*** zul has joined #openstack-glance11:01
*** haomaiwang has joined #openstack-glance11:01
*** jistr|mtg has quit IRC11:27
*** mingdang1 has joined #openstack-glance11:39
*** boris-42 has quit IRC11:43
*** baojg has joined #openstack-glance11:44
openstackgerritCyril Roelandt proposed openstack/glance: Python 3: fix glance.tests.functional.db.simple  https://review.openstack.org/27564511:52
*** haomaiwang has quit IRC12:01
*** haomaiwa_ has joined #openstack-glance12:01
*** julim_ has quit IRC12:03
*** smatzek has joined #openstack-glance12:04
*** sudipto_ has quit IRC12:06
*** julim has joined #openstack-glance12:11
*** links has quit IRC12:11
*** zul has quit IRC12:24
*** baojg has quit IRC12:29
*** dims has joined #openstack-glance12:32
*** haomaiw__ has joined #openstack-glance12:32
*** haomaiwa_ has quit IRC12:34
*** abhishekk has quit IRC12:36
-openstackstatus- NOTICE: Infra running with lower capacity now, due to a temporary problem affecting one of our nodepool providers. Please expect some delays in your jobs. Apologies for any inconvenience caused.12:40
kairatflaper87, ping, I am trying to find any useful info how vitual size is generated, perhaps you have some info about tha?12:40
openstackgerritNina Goradia proposed openstack/glance-specs: Glance support for nested quotas  https://review.openstack.org/27567412:51
*** julim has quit IRC12:54
*** zul has joined #openstack-glance12:55
*** sinese has quit IRC12:56
*** julim has joined #openstack-glance12:57
*** sinese has joined #openstack-glance12:57
*** haomaiw__ has quit IRC13:01
*** haomaiwa_ has joined #openstack-glance13:01
*** sinese_ has joined #openstack-glance13:01
*** openstackgerrit has quit IRC13:02
*** openstackgerrit has joined #openstack-glance13:02
*** sinese has quit IRC13:04
*** jistr|mtg has joined #openstack-glance13:12
*** haomaiwa_ has quit IRC13:16
*** ducttape_ has joined #openstack-glance13:17
*** ducttape_ has quit IRC13:28
AJaegerglance cores, could you review https://review.openstack.org/275539 , please? That's a setup.cfg change that broke some assumption we had in infra scripts - and might in the future elsewhere. We fixed the infra scripts but getting this in would be great.13:29
kragnizAJaeger: done13:33
AJaegerthanks, kragniz and Louis Taylor13:33
kairatheh13:33
kairatkragniz is Louis Taylor)13:34
AJaegerThanks kairat for correcting me and for reviewing!13:35
buntingThat must have been the fastest review I have ever seen13:36
*** sinese_ has quit IRC13:36
*** haomaiwang has joined #openstack-glance13:36
*** edmondsw has joined #openstack-glance13:38
AJaegerbunting: perhaps on glance ;)13:38
AJaegerbunting: https://review.openstack.org/#/c/275655/ was faster today ;)13:39
*** cdelatte has quit IRC13:39
AJaegerbunting: still ,very much appreciated!13:39
*** sinese_ has joined #openstack-glance13:39
buntingAJaeger: Yeah that beats it by a fair margin13:47
*** baojg has joined #openstack-glance13:51
*** cdelatte has joined #openstack-glance13:55
*** gjayavelu has joined #openstack-glance13:58
*** baojg has quit IRC14:00
*** haomaiwang has quit IRC14:01
*** haomaiwa_ has joined #openstack-glance14:01
*** julim has quit IRC14:05
*** ninag has joined #openstack-glance14:06
*** baojg has joined #openstack-glance14:07
*** mingdang1 has quit IRC14:08
*** baojg has quit IRC14:08
*** baojg has joined #openstack-glance14:09
*** gberginc has quit IRC14:15
*** jistr|mtg is now known as jistr14:20
*** cdelatte has quit IRC14:22
*** baojg has quit IRC14:23
mfedosinflaper87: maybe we can merge it? https://review.openstack.org/#/c/255274/14:25
jokke_hemanthm: around?14:26
*** baojg has joined #openstack-glance14:32
*** burgerk has quit IRC14:33
*** gjayavelu has quit IRC14:38
*** cdelatte has joined #openstack-glance14:40
*** delattec has joined #openstack-glance14:42
openstackgerritMerged openstack/glance: Fix setup.cfg  https://review.openstack.org/27553914:45
*** cdelatte has quit IRC14:45
*** baojg has quit IRC14:46
*** akanksha_ has quit IRC14:47
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Updated from global requirements  https://review.openstack.org/27465514:47
*** julim has joined #openstack-glance14:48
*** baojg has joined #openstack-glance14:49
*** ducttape_ has joined #openstack-glance14:54
*** ducttape_ has quit IRC14:54
*** ducttape_ has joined #openstack-glance14:54
*** pushkaru has joined #openstack-glance14:54
rosmaitaflwang: please accept my apologies, i feel like i always leave negative comments on your specs! I think it's a good idea, but we need to think through the implications some more14:56
*** avarner has joined #openstack-glance14:59
openstackgerritTristan Cacqueray proposed openstack/glance: Prevent user to remove last location of the image  https://review.openstack.org/27573715:00
*** haomaiwa_ has quit IRC15:01
*** haomaiwang has joined #openstack-glance15:01
*** delatte has joined #openstack-glance15:03
*** delattec has quit IRC15:03
*** delatte has quit IRC15:04
*** AJaeger has left #openstack-glance15:05
rosmaitajokke_: ping15:07
jokke_pong15:08
jokke_rosmaita: ^^15:09
*** vijendar has joined #openstack-glance15:09
rosmaitajokke_: got a quick question for you, viz., what is our attitude toward correcting spelling in comments in DB migration files?15:10
rosmaitahttps://review.openstack.org/#/c/275543/115:10
jokke_I'd say rather not15:10
rosmaitai guess my worry is that someone might think there was a serious change in the file to worry about15:10
rosmaitaok, we are on same page, then15:10
*** cdelatte has joined #openstack-glance15:12
jokke_yeah ... I kind of treat each migration file as read-only15:12
jokke_as they are versioned15:12
rosmaitathat's my feeling, too15:12
*** boris-42 has joined #openstack-glance15:16
openstackgerritvenkatamahesh proposed openstack/glance: Fix some word spellings  https://review.openstack.org/27554315:18
openstackgerritvenkatamahesh proposed openstack/glance: Fix some word spellings  https://review.openstack.org/27554315:18
*** mtanino has joined #openstack-glance15:19
*** delattec has joined #openstack-glance15:19
*** cdelatte has quit IRC15:22
*** sigmavirus24_awa is now known as sigmavirus2415:24
*** delatte has joined #openstack-glance15:27
*** delattec has quit IRC15:28
*** delatte has quit IRC15:29
*** harshs has joined #openstack-glance15:30
*** baojg has quit IRC15:35
*** baojg has joined #openstack-glance15:35
*** cdelatte has joined #openstack-glance15:37
*** delattec has joined #openstack-glance15:40
*** cdelatte has quit IRC15:43
openstackgerritJustin Shepherd proposed openstack/glance: Updating message for conversion_format cfg_opt  https://review.openstack.org/27577115:46
*** zul has quit IRC15:46
*** baojg has quit IRC15:49
*** baojg has joined #openstack-glance15:56
*** burgerk has joined #openstack-glance15:59
*** haomaiwang has quit IRC16:01
*** haomaiwang has joined #openstack-glance16:01
*** avarner__ has joined #openstack-glance16:02
*** avarner has quit IRC16:02
*** avarner__ has quit IRC16:25
*** avarner__ has joined #openstack-glance16:25
*** avarner__ has quit IRC16:27
*** avarner___ has joined #openstack-glance16:27
*** belmoreira has quit IRC16:28
openstackgerritVictor Stinner proposed openstack/glance: Fix _wait_on_task_execution()  https://review.openstack.org/27581516:37
openstackgerritVictor Stinner proposed openstack/glance: Monkey-patch with eventlet when running tests  https://review.openstack.org/27581616:39
*** mgarza_ has joined #openstack-glance16:39
*** vijendar has quit IRC16:39
*** julim has quit IRC16:42
*** haypo has joined #openstack-glance16:43
haypohey, i found a bug which may cause random failures on the integration tests of glance: https://bugs.launchpad.net/glance/+bug/154148716:44
openstackLaunchpad bug 1541487 in Glance "glance.tests.integration.v2.test_tasks_api.TestTasksApi.test_all_task_api hangs when run with testtools" [Undecided,New]16:44
haypoi proposed two different fixes, pick the one that you prefer :)16:44
*** julim has joined #openstack-glance16:44
*** avarner has joined #openstack-glance16:45
nikhilflaper87: hi16:45
*** tonyb has quit IRC16:45
*** avarner___ has quit IRC16:45
*** wxy_ has quit IRC16:45
nikhilflaper87: I want to propose a couple of topics for virtual conf and wanted to see how you wanted to schedule them. Do I put them on Thursday or Friday or do I need to send a request to extend the time on the ML?16:46
*** wxy_ has joined #openstack-glance16:46
*** tonyb has joined #openstack-glance16:46
*** aarefiev has quit IRC16:48
*** vijendar has joined #openstack-glance16:48
*** aarefiev has joined #openstack-glance16:49
*** delattec has quit IRC16:51
*** vijendar has quit IRC16:54
*** belmoreira has joined #openstack-glance16:57
*** jistr has quit IRC16:57
*** e0ne has quit IRC16:59
*** vijendar has joined #openstack-glance17:00
*** avarner has quit IRC17:00
*** avarner has joined #openstack-glance17:00
*** haomaiwang has quit IRC17:01
*** 7F1AA87I3 has joined #openstack-glance17:01
*** dims has quit IRC17:04
*** dims has joined #openstack-glance17:06
*** dims has quit IRC17:12
*** sinese_ has quit IRC17:13
*** sgotliv has quit IRC17:14
*** baojg has quit IRC17:16
*** mvk_ has quit IRC17:16
*** dims has joined #openstack-glance17:18
*** groen692 has quit IRC17:26
*** achanda has joined #openstack-glance17:27
*** julim has quit IRC17:33
*** harshs has quit IRC17:41
*** achanda has quit IRC17:42
*** julim has joined #openstack-glance17:42
*** belmoreira has quit IRC17:45
* sigmavirus24 thinks he may be running two video conferences at the same time tomorrow17:51
sigmavirus24although, most of what I care about is on Friday17:51
sigmavirus24So maybe I'll save my bandwidth and sanity tomorrow morning17:52
kragniznah, just have them both running at the same time17:53
kragnizthere's no way it can get confusing17:53
*** vijendar has quit IRC17:53
*** vijendar has joined #openstack-glance17:55
*** 7F1AA87I3 has quit IRC18:01
*** haomaiwang has joined #openstack-glance18:01
*** sinese has joined #openstack-glance18:04
tristanCflaper87: jokke_: the prevent location removal is failing master gate: http://logs.openstack.org/37/275737/1/check/gate-glance-python34/936845b/console.html18:15
tristanCflaper87: jokke_: the error seems related, it's test test_update_replace_locations_invalid and test_update_delete_image_location_unauthorized18:16
*** baojg has joined #openstack-glance18:17
*** _ducttape_ has joined #openstack-glance18:24
*** e0ne has joined #openstack-glance18:24
*** ducttape_ has quit IRC18:27
*** achanda has joined #openstack-glance18:29
*** baojg has quit IRC18:33
*** mvk_ has joined #openstack-glance18:41
*** dshakhray has quit IRC18:53
*** TravT has quit IRC18:55
*** TravT has joined #openstack-glance18:56
*** gjayavelu has joined #openstack-glance19:00
*** harshs has joined #openstack-glance19:00
*** haomaiwang has quit IRC19:01
*** haomaiwang has joined #openstack-glance19:01
*** julim has quit IRC19:04
*** _ducttape_ has quit IRC19:05
*** ducttape_ has joined #openstack-glance19:06
*** julim has joined #openstack-glance19:06
flaper87nikhil: hey, just read your email re-agenda. Thanks for chiming in. The agenda I sent is not fixed so we should totally improve it19:09
*** kebray has quit IRC19:09
*** kebray has joined #openstack-glance19:15
*** achanda has quit IRC19:17
*** gjayavelu has quit IRC19:21
*** azbiswas has joined #openstack-glance19:31
*** baojg has joined #openstack-glance19:33
*** e0ne has quit IRC19:35
*** achanda has joined #openstack-glance19:35
nikhilflaper87: ack19:37
*** baojg has quit IRC19:38
*** ducttape_ has quit IRC19:41
*** achanda has quit IRC19:43
*** boris-42 has quit IRC19:43
*** mgarza__ has joined #openstack-glance19:50
*** mgarza_ has quit IRC19:53
*** ducttape_ has joined #openstack-glance19:54
*** mfedosin has quit IRC19:55
*** kebray has quit IRC19:57
*** achanda has joined #openstack-glance19:59
*** kebray has joined #openstack-glance19:59
*** haomaiwang has quit IRC20:01
*** 7JTAAES9V has joined #openstack-glance20:01
flwangrosmaita: haha, i accept the apologies :D20:08
rosmaitagood, no hard feelings, then?20:10
rosmaitaalso, looks like you have a hot topic, there's another spec for quotas20:10
rosmaitaflwang: ^^20:10
flwangin Glance?20:10
sigmavirus24flaper87: yep20:11
sigmavirus24* flwang20:11
sigmavirus24also, Hi flwang !!20:11
flwangrosmaita: i saw the cross-project mentioned quota as well and i left a comments for Mike, but i didn't see any response20:11
flwangsigmavirus24: hey man20:11
flwangrosmaita: i totally agree we should be consistent with the other projects and it would be nice if there is consistent/unified solution across openstack20:12
flwangunfortunately, no, and we don't know when20:12
flwangso the question is if we should wait until there is a unified quota solution20:13
flwangoslo.quota hasn't been used in any project20:13
flwangwe can try it, that's not a problem20:13
flwangsince it's just a lib20:14
flwangthe question is if we want to improve the quota system of glance for now or just leave it as it's20:14
flwangflaper87: ^20:14
rosmaitaflwang: can you make it to the virtual midcycle tomorrow?20:16
flwangrosmaita: sure20:17
rosmaitagood, nikhil put it on the agenda20:18
rosmaitahttps://etherpad.openstack.org/p/glance-mitaka-virtual-mid-cycle20:18
flwangrosmaita: haha, i think i know the owner of another quota spec :)20:18
rosmaitaflwang: what's the cross-project spec you mentioned? do you have a link?20:18
flwangwhen i worked for ibm, we're under the same big team20:18
flwangrosmaita: yep, wait a sec20:19
*** mgarza__ has quit IRC20:19
rosmaitai was wondering about that20:19
flwangrosmaita: https://review.openstack.org/#/c/266072/3/doc/source/cross-project.rst20:21
rosmaitathanks20:21
rosmaitaflwang: did you get an answer to that question you posed on the patch?20:23
flwangrosmaita: unfortunately, no20:23
rosmaitabummer20:24
flwangrosmaita: and TBH, I'm not sure if there is anybody can give a clear answer20:24
flwangsince it's big change20:24
rosmaitai agree20:24
flwangnova and cinder's quota implementation are very similar, but very different from Neutron's20:24
flwangi haven't dig into the quota implementation of swfit20:25
flwangbut I assume it's also different20:25
rosmaitai can't decide if local implementations are best, since closer to the subject matter, or whether a global effort to avoid duplication would be best20:25
*** sgotliv has joined #openstack-glance20:25
flwangrosmaita: i don't mind propose the implementation for cross-project20:25
flwangbut i can imagine it will take very long time to get the spec merged20:26
rosmaitayeah, i wonder whether it would be worth it20:26
flwangrosmaita: exactly, that's why i decide to do this20:27
flwangin glance firstly20:27
flwangand follow the most common way, nova and cinder did20:28
flwangand we can use oslo.quota since it's extracted from nova/cinder20:28
dimsflwang : https://review.openstack.org/#/c/132127/1/specs/kilo/review_quota_module.rst20:32
flwangdims: wow, you're around, thanks for the link20:33
dimsflwang : we tried a while ago :)20:33
flwangdims: but you know the problem is not adopting the lib, the problem is the API20:33
dimsflwang : problem as that the database needed to be maintained by a library20:33
flwangdims: yep, i see, that's another issue20:34
flwangdims: will the olso.quota take care the db schema?20:34
flwangdims: does it has any change recently against the nested quota?20:34
dimsflwang : i'd encourage fresh discussion with Nova folks20:34
flwangdims: do you think it's a good idea to propose it in across-project?20:35
dimsflwang : projects don't like delegating responsibility for DB to a library20:35
flwangdims: true20:35
*** sgotliv has quit IRC20:36
*** vijendar has quit IRC20:36
dimsflwang : i'd suggest start with ML discussion and see who is still interested in that topic20:36
*** baojg has joined #openstack-glance20:37
rosmaitadims: was there any discussion about this in vancouver?20:37
rosmaitalooks like maybe no20:37
dimsrosmaita : we can check etherpads, probably not20:37
rosmaitabummer20:37
flwangdims: yep, ML is one thing i'm going to bug20:38
flwangi mentioned that with flaper87 but i'm too lazy :(20:38
flwangi will draft a mail today20:38
dims:) thanks flwang and rosmaita20:39
flwangdims: thank you!20:40
rosmaitadims: ty20:40
rosmaitait's a tough topic20:40
flwangrosmaita: i can imagine we won't see the enhanced quota in Newton, hah20:40
rosmaitawell, you never know ... your old company may be up to the task20:41
*** baojg has quit IRC20:45
*** vijendar has joined #openstack-glance20:48
flwangrosmaita: yep, possibly20:53
flwangrosmaita: and i have another concern, even we can follow the api of cinder/nova, we still can't do that, since we don't have tenant id in URL20:53
flwanglike nova/cinder doing20:54
flwangthat's why i use the way like nova/cinder did in the spec, though i'm not really sure we should to do that20:54
*** dims has quit IRC20:56
tristanCjokke_: flaper87: turns out the stable/kilo fix for bug 1525915 was missing encodeutils... beside failing the gate, does it have a real production impact or is the NameError properly catch and service recovers from it ?20:57
openstackbug 1525915 in Glance liberty "[OSSA 2016-006] Normal user can change image status if show_multiple_locations has been set to true (CVE-2016-0757)" [Undecided,New] https://launchpad.net/bugs/1525915 - Assigned to Erno Kuvaja (jokke)20:57
*** 7JTAAES9V has quit IRC21:01
*** haomaiwa_ has joined #openstack-glance21:01
*** smatzek has quit IRC21:04
rosmaitaflwang: hopefully we can find a way around that and use the tenant id from the context21:04
flwangrosmaita: yep, as for the two specs of quota21:05
*** dims has joined #openstack-glance21:05
flwangpersonally, i would like to split them to focus on normal quota and nested quota if we do them after figuring out the way21:06
flwangrosmaita: any link about this Heirarchial Images support ?21:10
tristanCjokke_: flwang and the master version of the fix is failing gate-glance-python34 because of TypeError: Unexpected keyword: message='Cannot set locations to empty list.'21:10
rosmaitaflwang: just that gerrit review that's in the midcycle etherpad21:11
flwangrosmaita: cool, thanks21:11
rosmaitaflwang: it looks a lot like the cinder spec you linked to in your spec21:13
*** clayton has quit IRC21:14
rosmaitaflwang: sorry, i am confused21:14
rosmaitai know nothing about hierarchial images support21:15
flwangi'm confused too ;)21:15
*** boris-42 has joined #openstack-glance21:15
rosmaitanikhil: can you put a link or some info about "hierarchical images support" on midcycle etherpad?21:15
rosmaitaflwang: that was nikhil 's item21:15
flwangrosmaita: ah, got it21:16
flaper87tristanC: did you get help?21:25
tristanCflaper87: nope, I now have a local py34 setup to try to reproduce the master test failure21:26
tristanCflaper87: and I submitted a new patchset for stable/kilo, could you approve it please: https://review.openstack.org/#/c/275735/221:26
flwangdims: can you let me know where i can find the source code of oslo.quota? thanks21:27
*** julim has quit IRC21:27
dimsflwang : kilo version i think - https://github.com/openstack/oslo-incubator/blob/stable/kilo/openstack/common/quota.py21:28
*** zul has joined #openstack-glance21:29
flwangdims: i can't find it in latest code21:30
flwangdims: does that mean it's dead? :021:30
dimsyep21:30
flwang:-(21:31
*** sgotliv has joined #openstack-glance21:32
*** achanda has quit IRC21:35
*** zul has quit IRC21:40
*** clayton has joined #openstack-glance21:43
*** achanda has joined #openstack-glance21:43
*** jistr has joined #openstack-glance21:46
*** jistr has quit IRC21:47
tristanCjokke_: any particular reason your patch does 'raise webob.exc.HTTPForbidden(message=msg)' while the rest of the code do 'raise webob.exc.HTTPForbidden(msg)' ?21:50
openstackgerritTristan Cacqueray proposed openstack/glance: Prevent user to remove last location of the image  https://review.openstack.org/27573721:52
flaper87tristanC: I don't think there's a specific reason other than overlooked ? I don't recall exactly21:55
flaper87tristanC: if that fixes the py34 issue w/o breaking py2721:56
flaper87then we're good21:56
tristanCflaper87: alright, so I submitted a new patchset for that too21:56
tristanCwe'll see21:56
tristanCthose are the 2 reviews that need core assistance: https://review.openstack.org/#/c/275737/ and https://review.openstack.org/#/c/275735/21:57
flaper87tristanC: core-assisting as we speak21:57
tristanCflaper87: thank you sir :-)21:58
flaper87tristanC: done21:58
flaper87hope that goes through21:58
flaper87if it doesn't, ping flwang he's in NZ and probably awake when we're all asleep21:59
flwangflaper87: i'm around, boss21:59
flaper87he's not a zombie21:59
* flaper87 hugs flwang21:59
flwangflaper87: bear hugs back :)21:59
*** haomaiwa_ has quit IRC22:01
sigmavirus24gah, flaper87 you already approved the one I had open ready to +222:01
*** haomaiwang has joined #openstack-glance22:01
flaper87sigmavirus24: I'm a ninja, I ninja approve things22:02
flwanghttps://review.openstack.org/#/c/275737/ wow, this one is approved22:02
flwangi'm wondering if it will impact the location update command22:02
flaper87flwang: it will but it's fixing a security bug22:02
flaper87read comments in the linked LP22:02
flaper87we went through that scenario22:02
flaper87flwang: by all means, feel free to disagree :P22:03
tristanCflaper87: flwang: your help is very appreciated here, I'm waiting for approve and verify on those patch to send the advisory22:04
flaper87flwang: ^  can you keep an eye on those patches?22:05
flaper87I'm about to go offline22:05
flwangflaper87: no problem22:05
flaper87I was actually mostly offline today but that's a different story22:05
flwangflaper87: but i think i can't approve stable branch patches22:05
flwangtristanC: did you test your patch with location-update command?22:06
tristanCflwang: hum nope, I copied the same webob syntax used in the rest of the module (thus omiting the message keyword)22:09
tristanCflwang: do you think it could have an impact ?22:09
flwangtristanC: not really, i need to test22:10
flwangi will test and let you know22:10
*** ducttape_ has quit IRC22:11
flaper87flwang: what's your concern?22:12
flwangflaper87: the commit message said "This change also prevents doing the same operation via replacing the locations with empty list."22:12
flwangflaper87: IIRC, glance client is patching the locations with empty list and add a new location to do the location update22:13
flwangflaper87: so maybe the patch will totally block the command, need to review the code to double confirm22:14
flaper87flwang: but it does all that in a single operation, I believe22:14
flaper87if the client breaks with this, we'll have to patch it and release it22:14
flaper87flwang: test it and let us know your findings22:14
flaper87:D22:14
flwangflaper87: yep, hopefully we're safe https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/images.py#L36422:15
*** achanda has quit IRC22:16
*** dims_ has joined #openstack-glance22:17
*** sinese has quit IRC22:18
*** sgotliv has quit IRC22:21
*** dims has quit IRC22:21
*** ducttape_ has joined #openstack-glance22:24
*** pushkaru has quit IRC22:26
*** dims_ has quit IRC22:27
*** vijendar has quit IRC22:29
*** burgerk has quit IRC22:59
*** haomaiwang has quit IRC23:01
*** haomaiwa_ has joined #openstack-glance23:01
*** baojg has joined #openstack-glance23:02
*** ninag has quit IRC23:07
*** ninag has joined #openstack-glance23:07
*** baojg has quit IRC23:07
*** achanda has joined #openstack-glance23:09
*** achanda has quit IRC23:12
*** ninag has quit IRC23:12
*** ninag has joined #openstack-glance23:18
*** e0ne has joined #openstack-glance23:24
*** ducttape_ has quit IRC23:24
*** MattMan has quit IRC23:25
*** MattMan has joined #openstack-glance23:25
*** sigmavirus24 is now known as sigmavirus24_awa23:30
*** e0ne has quit IRC23:33
*** azbiswas has quit IRC23:39
*** jamielennox|away is now known as jamielennox23:47
*** edmondsw has quit IRC23:49

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