| *** haomaiwang has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-glance | 00:01 | |
| *** darrenc_afk is now known as darrenc | 00:05 | |
| *** vilobhmm11 has joined #openstack-glance | 00:05 | |
| *** ducttape_ has quit IRC | 00:12 | |
| *** vilobhmm11 has quit IRC | 00:19 | |
| *** vilobhmm11 has joined #openstack-glance | 00:20 | |
| *** vilobhmm11 has quit IRC | 00:21 | |
| *** vilobhmm11 has joined #openstack-glance | 00:21 | |
| *** vilobhmm11 has quit IRC | 00:21 | |
| *** gberginc has joined #openstack-glance | 00:21 | |
| *** vilobhmm11 has joined #openstack-glance | 00:22 | |
| *** kzaitsev_mb has quit IRC | 00:23 | |
| *** gberginc has quit IRC | 00:26 | |
| *** ducttape_ has joined #openstack-glance | 00:59 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-glance | 01:01 | |
| *** kzaitsev_mb has joined #openstack-glance | 01:20 | |
| *** kzaitsev_mb has quit IRC | 01:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Updated from global requirements https://review.openstack.org/282471 | 01:34 |
|---|---|---|
| *** haomaiwang has quit IRC | 01:36 | |
| *** ducttape_ has quit IRC | 01:42 | |
| *** dims has quit IRC | 01:59 | |
| *** vilobhmm11 has quit IRC | 01:59 | |
| *** vilobhmm11 has joined #openstack-glance | 02:03 | |
| *** vilobhmm11 has quit IRC | 02:03 | |
| *** vilobhmm11 has joined #openstack-glance | 02:03 | |
| *** TravT has joined #openstack-glance | 02:07 | |
| *** pcaruana has quit IRC | 02:07 | |
| *** vilobhmm11 has quit IRC | 02:12 | |
| *** haomaiwang has joined #openstack-glance | 02:16 | |
| *** pcaruana has joined #openstack-glance | 02:19 | |
| *** kzaitsev_mb has joined #openstack-glance | 02:22 | |
| *** lakshmiS has quit IRC | 02:25 | |
| *** kzaitsev_mb has quit IRC | 02:27 | |
| *** TravT has quit IRC | 02:28 | |
| *** TravT has joined #openstack-glance | 02:29 | |
| *** alanf_ has quit IRC | 02:33 | |
| *** houming has joined #openstack-glance | 02:38 | |
| *** mingdang1 has joined #openstack-glance | 02:49 | |
| *** vilobhmm11 has joined #openstack-glance | 02:59 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-glance | 03:01 | |
| *** pcaruana has quit IRC | 03:01 | |
| *** TravT has joined #openstack-glance | 03:02 | |
| *** TravT has quit IRC | 03:07 | |
| *** dims has joined #openstack-glance | 03:13 | |
| *** avarner has quit IRC | 03:14 | |
| *** pcaruana has joined #openstack-glance | 03:16 | |
| *** haomaiwang has quit IRC | 03:20 | |
| *** kzaitsev_mb has joined #openstack-glance | 03:23 | |
| *** kzaitsev_mb has quit IRC | 03:28 | |
| *** gberginc has joined #openstack-glance | 03:34 | |
| *** links has joined #openstack-glance | 03:46 | |
| *** boris-42 has quit IRC | 03:54 | |
| *** ducttape_ has joined #openstack-glance | 03:58 | |
| *** pcaruana has quit IRC | 04:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Updated from global requirements https://review.openstack.org/282471 | 04:04 |
| *** vilobhmm11 has quit IRC | 04:04 | |
| *** pcaruana has joined #openstack-glance | 04:15 | |
| *** ducttape_ has quit IRC | 04:15 | |
| *** jamielennox is now known as jamielennox|away | 04:54 | |
| *** haomaiwang has joined #openstack-glance | 05:04 | |
| *** sgotliv has joined #openstack-glance | 05:20 | |
| *** kzaitsev_mb has joined #openstack-glance | 05:24 | |
| *** kzaitsev_mb has quit IRC | 05:29 | |
| *** mfedosin has joined #openstack-glance | 05:33 | |
| *** pcaruana has quit IRC | 05:38 | |
| *** mfedosin has quit IRC | 05:38 | |
| *** dims has quit IRC | 05:48 | |
| *** pcaruana has joined #openstack-glance | 05:53 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-glance | 06:01 | |
| *** dshakhray has joined #openstack-glance | 06:02 | |
| *** vilobhmm11 has joined #openstack-glance | 06:02 | |
| *** openstack has joined #openstack-glance | 13:20 | |
| *** openstackstatus has joined #openstack-glance | 13:20 | |
| *** ChanServ sets mode: +v openstackstatus | 13:20 | |
| *** edmondsw has joined #openstack-glance | 13:27 | |
| *** 32NAAD4T7 has quit IRC | 13:31 | |
| *** mfedosin has joined #openstack-glance | 13:32 | |
| *** ninag has joined #openstack-glance | 13:34 | |
| *** ninag has quit IRC | 13:37 | |
| *** ninag has joined #openstack-glance | 13:38 | |
| *** ninag has quit IRC | 13:39 | |
| *** dims has joined #openstack-glance | 13:40 | |
| *** ninag has joined #openstack-glance | 13:40 | |
| *** ninag has quit IRC | 13:44 | |
| *** ninag has joined #openstack-glance | 13:44 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| openstackgerrit | Wenjun Wang proposed openstack/python-glanceclient: Optimize function 'do_image_show' in v1 client https://review.openstack.org/284140 | 14:01 |
| *** haomaiwang has joined #openstack-glance | 14:01 | |
| *** mingdang1 has joined #openstack-glance | 14:02 | |
| *** ayoung has quit IRC | 14:04 | |
| openstackgerrit | Merged openstack/glance: Python 3: fix glance.tests.functional.db.simple https://review.openstack.org/275645 | 14:06 |
| openstackgerrit | Merged openstack/glance: Python 3: fix a few simple "str vs bytes" issues. https://review.openstack.org/283090 | 14:06 |
| *** mingdang1 has quit IRC | 14:06 | |
| *** mingdang1 has joined #openstack-glance | 14:06 | |
| *** avarner_ has joined #openstack-glance | 14:11 | |
| *** avarner has quit IRC | 14:11 | |
| *** groen692 has quit IRC | 14:15 | |
| *** julim has joined #openstack-glance | 14:15 | |
| openstackgerrit | Andreas Jaeger proposed openstack/glance: Always use constraints https://review.openstack.org/282894 | 14:15 |
| AJaeger | flaper87, nikhil : I had to rebase ^ ;( | 14:16 |
| flaper87 | AJaeger: no worries | 14:16 |
| flaper87 | There, approved again | 14:16 |
| AJaeger | flaper87: wow, you're fast ;) THANKS! | 14:16 |
| rsjethani | kairat please review https://review.openstack.org/#/c/261288/ :) | 14:18 |
| *** itisha has quit IRC | 14:19 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/glance: [WIP] Trying latest oslo.* from master https://review.openstack.org/284152 | 14:19 |
| rsjethani | Hi flaper87 | 14:20 |
| rsjethani | request id related patched for cinder and nova are merged... | 14:21 |
| rsjethani | patches* | 14:21 |
| kairat | Will try to review that soon | 14:22 |
| rsjethani | thanks kairat | 14:22 |
| kairat | rsjethani, Are they implemented as part of resource? | 14:22 |
| kairat | Resource object | 14:22 |
| kairat | in nova and cinder? | 14:22 |
| kairat | So I mean are cinder and nova implementations in accordance with x-proj spec? | 14:23 |
| rsjethani | yes | 14:23 |
| rsjethani | http://52.1.123.151/review/#/c/499/ | 14:23 |
| rsjethani | above link is for cinder | 14:24 |
| kairat | Oh, that's not good =(, but we need to do something anyway | 14:24 |
| kairat | What is review number? | 14:24 |
| kairat | link is broken for me | 14:25 |
| rsjethani | 499 | 14:25 |
| rsjethani | ohh sorry wait | 14:25 |
| rsjethani | here it is: https://review.openstack.org/#/c/257199/ | 14:27 |
| kairat | Ok, thanks | 14:28 |
| rsjethani | :) | 14:28 |
| *** smatzek_ has quit IRC | 14:32 | |
| *** rsjethani has quit IRC | 14:33 | |
| *** rsjethani has joined #openstack-glance | 14:40 | |
| *** rsjethani has quit IRC | 14:41 | |
| *** dane-fichter has joined #openstack-glance | 14:44 | |
| *** ayoung has joined #openstack-glance | 14:46 | |
| openstackgerrit | Chaozhe Chen(ccz) proposed openstack/python-glanceclient: Test: use assert_has_calls() instead https://review.openstack.org/281676 | 14:54 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:58 | |
| *** mtanino has joined #openstack-glance | 14:59 | |
| *** ayoung has quit IRC | 14:59 | |
| *** smatzek_ has joined #openstack-glance | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** ducttape_ has joined #openstack-glance | 15:01 | |
| *** haomaiwang has joined #openstack-glance | 15:01 | |
| flaper87 | kairat: https://review.openstack.org/#/c/168507/ when you get a chance | 15:02 |
| *** jdillaman has quit IRC | 15:02 | |
| *** smatzek__ has joined #openstack-glance | 15:02 | |
| flaper87 | sigmavirus24: re ^ I don't really mind. Your last patch-set is from May 2015 | 15:02 |
| kairat | flaper87, yep, that's my next in review list:=) | 15:02 |
| kairat | reviewing that now | 15:02 |
| flaper87 | kairat: awesome, thanks | 15:02 |
| bjornar | why is v1 still listed in versions when I comment out /v1 from composite:rootapp ? | 15:04 |
| flaper87 | bjornar: because `/versions` is a different endpoint | 15:05 |
| sigmavirus24 | damnit flaper87 you beat me to it :P | 15:05 |
| bjornar | flaper87, ? | 15:05 |
| flaper87 | sigmavirus24: :P | 15:05 |
| *** smatzek_ has quit IRC | 15:05 | |
| flaper87 | bjornar: https://github.com/openstack/glance/blob/master/glance/api/versions.py | 15:05 |
| flaper87 | bjornar: sorry, was getting a link | 15:05 |
| openstackgerrit | Merged openstack/glance-specs: Image Import Refactor Update https://review.openstack.org/278086 | 15:05 |
| flaper87 | bjornar: you need to disable v1 in glance-api.conf | 15:06 |
| sigmavirus24 | bjornar: /versions is an endpoint that returns the versions it knows about. It does not know about the Paste.ini and so it doesn't know you disabled v1 via paste | 15:06 |
| flaper87 | bjornar: https://github.com/openstack/glance/blob/master/glance/api/versions.py#L65 | 15:06 |
| bjornar | ok.. what is the point of paste then? | 15:06 |
| flaper87 | bjornar: that's a good question that I'm willing to rant about over beers | 15:06 |
| flaper87 | :P | 15:07 |
| flaper87 | paste helps building the pipeline | 15:07 |
| bjornar | flaper87, exactly.. in keystone it works exactly as I say | 15:07 |
| *** burgerk has joined #openstack-glance | 15:12 | |
| *** avarner_ has quit IRC | 15:15 | |
| *** avarner_ has joined #openstack-glance | 15:15 | |
| *** avarner__ has joined #openstack-glance | 15:18 | |
| *** avarner_ has quit IRC | 15:18 | |
| bjornar | Why would I get a <html> error message saying 404 when I visit /v2/ | 15:21 |
| bjornar | the error might be correct, since I am playing around, but it should be json | 15:22 |
| *** avarner_ has joined #openstack-glance | 15:23 | |
| *** avarner_ is now known as avarner | 15:23 | |
| *** avarner__ has quit IRC | 15:23 | |
| *** tjcocozz_ has joined #openstack-glance | 15:24 | |
| *** dims has quit IRC | 15:29 | |
| bjornar | anyone? | 15:30 |
| *** dims has joined #openstack-glance | 15:30 | |
| *** mingdang1 has quit IRC | 15:30 | |
| *** annasort has quit IRC | 15:31 | |
| *** ninag has quit IRC | 15:34 | |
| *** dims has quit IRC | 15:34 | |
| *** TravT has quit IRC | 15:35 | |
| sigmavirus24 | bjornar: because the library/framework we use only *just* gave us the ability to return JSON error messages | 15:38 |
| bjornar | ok.. but then its better just just return empty and a status code is my opionion here | 15:39 |
| *** vijendar has joined #openstack-glance | 15:40 | |
| bjornar | sigmavirus24, so are you working on this currently? | 15:41 |
| sigmavirus24 | bjornar: so hot patch your instance of glance to do that? And yes I fixed it in webob for us because no one else from any of the myrid OpenStack projects had picked up that work | 15:42 |
| *** avarner has quit IRC | 15:42 | |
| *** avarner has joined #openstack-glance | 15:42 | |
| sigmavirus24 | bjornar: I think someone else has already fixed glance to use the new version of webob in master (i.e., for mitaka) so we return JSON appropriately | 15:43 |
| *** vijendar1 has joined #openstack-glance | 15:43 | |
| bjornar | im using master, and seems no.. | 15:43 |
| sigmavirus24 | Prior to this, returning an HTML message was the best we could do because no message gives the user no visibility into the why of the error (because there are many more errors than 404) | 15:43 |
| sigmavirus24 | bjornar: what version of webob do you have installed? | 15:43 |
| bjornar | I hope you look at how different project do stuff. keystone is a good example when itt comes to api | 15:44 |
| bjornar | WebOb-1.5.1-py2.py3-none-any.whl | 15:44 |
| *** pushkaru has joined #openstack-glance | 15:46 | |
| *** vijendar has quit IRC | 15:46 | |
| openstackgerrit | Merged openstack/glance_store: Updated from global requirements https://review.openstack.org/282472 | 15:48 |
| dane-fichter | If anyone has time, I'd love some feedback on https://review.openstack.org/#/c/255584/ | 15:50 |
| dane-fichter | Trying to get that feature merged before the feature freeze | 15:51 |
| *** azbiswas has joined #openstack-glance | 15:52 | |
| openstackgerrit | Merged openstack/python-glanceclient: Updated from global requirements https://review.openstack.org/278725 | 15:54 |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-glance | 16:01 | |
| *** dims has joined #openstack-glance | 16:03 | |
| *** annasort has joined #openstack-glance | 16:14 | |
| *** annasort_ has joined #openstack-glance | 16:15 | |
| *** daemontool has joined #openstack-glance | 16:18 | |
| daemontool | Hi all, I have this issue on glanceclient on kilo: http://paste.openstack.org/show/488044/ | 16:18 |
| sigmavirus24 | bjornar: each project does things differently. keystone doesn't use webob and to remove webob from our dependencies now would be silly because the issue is fixed and also because so much of glance relies on it | 16:19 |
| daemontool | not critical but still not optimal | 16:19 |
| sigmavirus24 | bjornar: try webob 1.6.0 | 16:19 |
| sigmavirus24 | (that may be an release candidate now) | 16:19 |
| *** annasort has quit IRC | 16:19 | |
| *** annasort_ is now known as annasort | 16:19 | |
| *** ayoung has joined #openstack-glance | 16:20 | |
| *** alanf_ has joined #openstack-glance | 16:24 | |
| *** vijendar1 has quit IRC | 16:26 | |
| *** alanf_ has quit IRC | 16:28 | |
| *** belmoreira has quit IRC | 16:31 | |
| sigmavirus24 | daemontool: what issue are you having specifically? | 16:31 |
| sigmavirus24 | daemontool: oh missing schema problem | 16:32 |
| sigmavirus24 | flaper87: can speak better to that daemontool | 16:32 |
| daemontool | ah ok sigmavirus24 | 16:32 |
| *** bjornar__ has joined #openstack-glance | 16:32 | |
| daemontool | ty | 16:32 |
| sigmavirus24 | daemontool: I want to say it was fixed but I don't honestly recall | 16:32 |
| bjornar__ | Was it my webob version? | 16:32 |
| daemontool | sigmavirus24, ok, not super urgent tho | 16:32 |
| *** ihrachys has quit IRC | 16:33 | |
| *** vijendar has joined #openstack-glance | 16:33 | |
| sigmavirus24 | bjornar__: I suspect if you upgrade webob you'll get JSON error responses | 16:34 |
| sigmavirus24 | bjornar__: it looks like you're using pip so use `pip install -U --pre webob` | 16:34 |
| bjornar__ | sigmavirus24, so perhaps requirements.txt should be updated? | 16:35 |
| sigmavirus24 | bjornar__: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L241 looks fine to me | 16:42 |
| dane-fichter | tjcocozz_: Can you clarify what you mean by i18n compliant? | 16:42 |
| *** ducttape_ has quit IRC | 16:42 | |
| tjcocozz_ | dane-fichter, you should just need to wrap your error messages in _() | 16:42 |
| *** ducttape_ has joined #openstack-glance | 16:43 | |
| dane-fichter | Ah, ok. Thanks | 16:43 |
| tjcocozz_ | dane-fichter, so they can get translated :-) | 16:43 |
| *** ducttape_ has quit IRC | 16:45 | |
| *** jistr has quit IRC | 16:45 | |
| *** ducttape_ has joined #openstack-glance | 16:45 | |
| *** vilobhmm11 has joined #openstack-glance | 16:46 | |
| openstackgerrit | Merged openstack/glance: Updated from global requirements https://review.openstack.org/282471 | 16:50 |
| openstackgerrit | Merged openstack/glance: Encourage usage of identity API v3 https://review.openstack.org/280887 | 16:50 |
| *** Guest51435 is now known as mgagne | 16:56 | |
| *** mgagne has quit IRC | 16:56 | |
| *** mgagne has joined #openstack-glance | 16:56 | |
| *** lakshmiS has joined #openstack-glance | 16:59 | |
| *** dims has quit IRC | 17:00 | |
| bjornar__ | sigmavirus24, did not realize the fix was in a -alpha | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-glance | 17:01 | |
| *** ninag has joined #openstack-glance | 17:02 | |
| *** dims has joined #openstack-glance | 17:03 | |
| *** kzaitsev_mb has quit IRC | 17:05 | |
| *** belmoreira has joined #openstack-glance | 17:06 | |
| *** avarner has quit IRC | 17:06 | |
| *** avarner has joined #openstack-glance | 17:07 | |
| bjornar__ | /keystone/lib/python3.4/site-packages/oslo_middleware/ssl.py:28: DeprecationWarning: The 'oslo_middleware.ssl' module usage is deprecated, please use oslo_middleware.http_proxy_to_wsgi instead | 17:10 |
| bjornar__ | "oslo_middleware.http_proxy_to_wsgi") | 17:10 |
| bjornar__ | forget about keystone, it is glance. just my hacked mountpoint atm | 17:10 |
| bjornar__ | About glance-registry, that api is not public, but called from glance-api ? | 17:13 |
| *** achanda has joined #openstack-glance | 17:15 | |
| *** achanda has quit IRC | 17:15 | |
| *** achanda has joined #openstack-glance | 17:15 | |
| *** mvk has quit IRC | 17:22 | |
| *** edmondsw has quit IRC | 17:23 | |
| openstackgerrit | Erno Kuvaja proposed openstack/glance-specs: Initial lite-specs.rst and per deliverable folders https://review.openstack.org/282516 | 17:24 |
| *** vijendar has left #openstack-glance | 17:25 | |
| *** ihrachys has joined #openstack-glance | 17:26 | |
| *** kzaitsev_mb has joined #openstack-glance | 17:29 | |
| *** ihrachys has quit IRC | 17:29 | |
| *** kzaitsev_mb has quit IRC | 17:44 | |
| *** edmondsw has joined #openstack-glance | 17:47 | |
| bjornar__ | How can I make glance use v3 keystone? | 17:47 |
| bjornar__ | I see "POST /v2.0/tokens HTTP/1.1" no matter seems.. | 17:48 |
| *** jdillaman has joined #openstack-glance | 17:58 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-glance | 18:01 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 18:01 | |
| *** dshakhray has quit IRC | 18:02 | |
| *** julim has quit IRC | 18:03 | |
| *** mfedosin has quit IRC | 18:04 | |
| *** avarner has quit IRC | 18:05 | |
| *** avarner has joined #openstack-glance | 18:06 | |
| *** permalac has quit IRC | 18:07 | |
| *** julim has joined #openstack-glance | 18:07 | |
| bjornar__ | Merged openstack/glance: Encourage usage of identity API v3 https://review.openstack.org/280887 | 18:10 |
| bjornar__ | This for me seems incorrect | 18:10 |
| bjornar__ | auth_url = http://localhost:35357 does not work here | 18:10 |
| bjornar__ | the port and scheme gets overwritten by defaults seems | 18:11 |
| *** avarner has quit IRC | 18:12 | |
| openstackgerrit | Merged openstack/glance: Always use constraints https://review.openstack.org/282894 | 18:13 |
| flaper87 | AJaeger: ^ | 18:18 |
| AJaeger | thanks, flaper87 | 18:19 |
| openstackgerrit | Erno Kuvaja proposed openstack/glance-specs: Initial lite-specs.rst and per deliverable folders https://review.openstack.org/282516 | 18:20 |
| *** belmoreira has quit IRC | 18:23 | |
| *** sgotliv has quit IRC | 18:25 | |
| *** azbiswas has quit IRC | 18:27 | |
| dims | nikhil : were you looking for me? :) | 18:30 |
| *** vijendar has joined #openstack-glance | 18:30 | |
| openstackgerrit | Stuart McLaren proposed openstack/python-glanceclient: Handle 403 forbidden on download https://review.openstack.org/254328 | 18:34 |
| *** vijendar has left #openstack-glance | 18:41 | |
| openstackgerrit | Andreas Jaeger proposed openstack/glance: Move bandit into pep8 https://review.openstack.org/282885 | 18:45 |
| *** azbiswas has joined #openstack-glance | 18:51 | |
| bjornar__ | Should it be possible to use keystone v3 now? | 18:52 |
| openstackgerrit | dane-fichter proposed openstack/glance: Add sign-the-data signature verification https://review.openstack.org/255584 | 18:54 |
| *** daemontool has quit IRC | 18:55 | |
| sabari | sigmavirus24: flaper87: Thanks for taking care of the http store refactor patch while I was away :) | 18:58 |
| *** ninag has quit IRC | 18:59 | |
| sabari | kairat: thanks for your comments on the vmware store refactor. I will respond shortly. | 18:59 |
| dane-fichter | tjcocozz_: new patchset is up, thanks for the review | 19:00 |
| *** ninag has joined #openstack-glance | 19:00 | |
| tjcocozz_ | dane-fichter, i will check it out shortly | 19:00 |
| *** haomaiwang has quit IRC | 19:01 | |
| dane-fichter | tjcocozz_: Thanks :) | 19:01 |
| *** 16WAAEDST has joined #openstack-glance | 19:01 | |
| flaper87 | sabari: no worries :) | 19:03 |
| openstackgerrit | Erno Kuvaja proposed openstack/glance-specs: Initial lite-specs.rst and per deliverable folders https://review.openstack.org/282516 | 19:05 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 19:11 | |
| *** jbernard_ is now known as jbernard | 19:13 | |
| *** ninag has quit IRC | 19:19 | |
| *** belmoreira has joined #openstack-glance | 19:20 | |
| flaper87 | Anyone can help with this patch: https://review.openstack.org/#/c/252370/ ? | 19:21 |
| *** mfedosin has joined #openstack-glance | 19:22 | |
| nikhil | flaper87: ack | 19:22 |
| nikhil | in a few mins I will review it thoroughly | 19:23 |
| flaper87 | nikhil: thank you | 19:28 |
| *** ducttape_ has quit IRC | 19:38 | |
| nikhil | np! | 19:42 |
| nikhil | oh this review is gunna hurt mclaren | 19:43 |
| nikhil | and by review (Above) I meant the actual code proposal (sent as a review to gerrit) | 19:43 |
| flaper87 | nikhil: yeah but that patch has been around longer nad it's reviews more updates recently. :( | 19:45 |
| nikhil | flaper87: heh, yeah. I was just thinking out loud and possibly trying to exclaim that we may need a bit of time near end of m-3 for that feature .. | 19:46 |
| nikhil | flaper87: I think it makes sense to let the above one go in first, it's small and concise. | 19:47 |
| flaper87 | right | 19:47 |
| flaper87 | the only problem is, this week is the last week for non-client library releases | 19:47 |
| flaper87 | which means, if these patches don't land in this week, they won't be released until the start of Newton | 19:47 |
| *** vijendar has joined #openstack-glance | 19:48 | |
| *** ninag has joined #openstack-glance | 19:48 | |
| *** ducttape_ has joined #openstack-glance | 19:49 | |
| *** sgotliv has joined #openstack-glance | 19:52 | |
| *** ninag has quit IRC | 19:52 | |
| *** vijendar has quit IRC | 19:53 | |
| *** ninag has joined #openstack-glance | 19:53 | |
| *** vijendar has joined #openstack-glance | 19:56 | |
| *** azbiswas has quit IRC | 19:57 | |
| *** 16WAAEDST has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-glance | 20:01 | |
| nikhil | flaper87: oh my, I didn't realize on that one.. | 20:12 |
| nikhil | flaper87: btw, I've +Wed on kairat's patch | 20:12 |
| nikhil | heh, it's +W-ed (not wed) | 20:12 |
| flaper87 | nikhil: hehehe, cool | 20:12 |
| flaper87 | thanks | 20:12 |
| openstackgerrit | Mike Fedosin proposed openstack/glance: Deprecate use_user_token parameter https://review.openstack.org/237742 | 20:14 |
| *** gberginc has joined #openstack-glance | 20:16 | |
| *** kzaitsev_mb has joined #openstack-glance | 20:35 | |
| flaper87 | nikhil: if you have some time for this one too: https://review.openstack.org/#/c/251850/ | 20:41 |
| flaper87 | It'd be super awesome | 20:41 |
| nikhil | flaper87: oh, that one is bigger and I'm still catching up on lunch :/ I can do that one in the evening.. | 20:42 |
| nikhil | it might be morning for you before you see something on it | 20:42 |
| flaper87 | nikhil: sounds good | 20:43 |
| *** ihrachys has joined #openstack-glance | 20:47 | |
| *** azbiswas has joined #openstack-glance | 20:48 | |
| *** boris-42 has joined #openstack-glance | 20:53 | |
| *** belmoreira has quit IRC | 20:53 | |
| flaper87 | rosmaita_: any chance we can get your review here? https://review.openstack.org/#/c/265152/ | 21:00 |
| *** haomaiwang has quit IRC | 21:01 | |
| rosmaita_ | flaper87: sure | 21:01 |
| *** haomaiwang has joined #openstack-glance | 21:01 | |
| *** achanda has quit IRC | 21:01 | |
| *** rosmaita_ is now known as rosmaita | 21:02 | |
| rosmaita | i hate the underscore | 21:02 |
| openstackgerrit | Merged openstack/glance_store: Change approach to request storage url for multi-tenant store https://review.openstack.org/252370 | 21:03 |
| * flaper87 hates the underscore too | 21:03 | |
| rosmaita | flaper87: shoot, that's related to that bug i haven't filed yet about v1/v2 image metadata asymmetry | 21:03 |
| flaper87 | rosmaita: I know... MUAHAHAHAHHAHAHAHAHA | 21:03 |
| flaper87 | OVA/OVF folks, please, https://review.openstack.org/#/c/214810/ | 21:03 |
| rosmaita | flaper87: i'll get something out a bit later today ... for realz | 21:03 |
| *** ducttape_ has quit IRC | 21:04 | |
| flaper87 | sabari: https://review.openstack.org/#/c/214810/ <- mind providing a review for this one? Jenkins is not happy but it's py3 specific | 21:04 |
| *** azbiswas has quit IRC | 21:04 | |
| *** ducttape_ has joined #openstack-glance | 21:04 | |
| *** azbiswas_ has joined #openstack-glance | 21:06 | |
| openstackgerrit | Eric Brown proposed openstack/glance: Remove bandit.yaml in favor of default config https://review.openstack.org/282844 | 21:08 |
| *** annasort has quit IRC | 21:08 | |
| *** azbiswas_ has quit IRC | 21:08 | |
| *** azbiswas_ has joined #openstack-glance | 21:09 | |
| *** pcaruana has quit IRC | 21:10 | |
| *** vijendar has quit IRC | 21:11 | |
| *** vijendar has joined #openstack-glance | 21:12 | |
| *** ihrachys has quit IRC | 21:17 | |
| flwang | flaper87: around? | 21:22 |
| flwang | flaper87: have mins to discuss the locations update issue? | 21:22 |
| flwang | i hate the timezone, earth should be flat :( | 21:24 |
| *** smatzek__ has quit IRC | 21:24 | |
| kragniz | flwang: timezones are the worst | 21:25 |
| flwang | kragniz: yep, i'm always trying to catch up with the team, but unfortunately, we're in different tz | 21:30 |
| flaper87 | flwang: on a call, bbib | 21:34 |
| *** ekarlso- has quit IRC | 21:34 | |
| *** ekarlso- has joined #openstack-glance | 21:34 | |
| flaper87 | flwang: sup ? | 21:36 |
| flwang | flaper87: about the location update issue | 21:37 |
| flwang | flaper87: https://bugs.launchpad.net/glance/+bug/1537626 | 21:37 |
| openstack | Launchpad bug 1537626 in python-glanceclient "`glance location-update` deletes locations and backend images" [High,In progress] - Assigned to Fei Long Wang (flwang) | 21:37 |
| flwang | flaper87: do you remember this fix https://review.openstack.org/#/c/275737/ ? | 21:38 |
| flwang | unfortunately, it's breaking the location update | 21:38 |
| flaper87 | flwang: yeah, I know you have a patch up for that one | 21:41 |
| flaper87 | right? | 21:41 |
| flwang | flaper87: and when I fix the problem i found another issue, when user want to update location's metadata, if one of the location has a restrict scheme, glance will reject to do the update, since https://github.com/openstack/glance/blob/master/glance/location.py#L339 | 21:42 |
| flwang | https://github.com/openstack/glance/blob/master/glance/common/store_utils.py#L117 | 21:42 |
| flwang | flaper87: so i would like to get your opinion about this case | 21:43 |
| flaper87 | flwang: didn't your patch fix those issues ? | 21:43 |
| flaper87 | or just the first one? | 21:43 |
| *** achanda has joined #openstack-glance | 21:43 | |
| flwang | just the first one | 21:43 |
| flaper87 | flwang: anyway we can have a follow-up patch fixing the second issue? | 21:44 |
| flwang | i'm going to submit another patch to fix the 2nd, but seems it's not easy | 21:44 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance_store: Updated from global requirements https://review.openstack.org/284429 | 21:44 |
| flwang | that's why i would like to get your opinion | 21:44 |
| *** achanda has quit IRC | 21:45 | |
| *** MattMan has quit IRC | 21:45 | |
| flwang | if we're sure the user is only updating location's metadata, not the location URL, is it ok just update it instead of doing all the check? | 21:45 |
| *** MattMan has joined #openstack-glance | 21:45 | |
| flwang | IMHO, it's not necessary to check the location if it's just a metadata update, thoughts? | 21:46 |
| flaper87 | As long as the locations are not overwritten and the image status is not changed, I think we're fine | 21:46 |
| flwang | awesome | 21:46 |
| flaper87 | If you read the bug that issue fixed, it involved ppl overwriting the image locations and changing the image back to queued | 21:47 |
| flaper87 | which is bad, really bad | 21:47 |
| flwang | besides, we should document somewhere, the location update is only used for location's metadata update, it can't be used for locations' url update | 21:47 |
| flaper87 | so, as long as the patch doesn't introduce a regression for that, I think we're good | 21:47 |
| flwang | yep, i see | 21:47 |
| flwang | flaper87: but i'm thinking another case | 21:48 |
| flwang | which may still be breaking our promise -- image is immutable | 21:48 |
| flwang | for example, image has two locations, A and B | 21:49 |
| flwang | with fix https://review.openstack.org/#/c/275737/ user can't remove the last location, right? | 21:49 |
| flwang | but if user remove location B and add location C, then remove location A, is that ok? | 21:50 |
| *** _ducttape_ has joined #openstack-glance | 21:50 | |
| flaper87 | mmh, technically, yes but I see where you're coming from | 21:52 |
| *** ducttape_ has quit IRC | 21:54 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/glance: [WIP] Trying latest oslo.* from master https://review.openstack.org/284152 | 21:57 |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-glance | 22:01 | |
| *** ninag has quit IRC | 22:02 | |
| flwang | mfedosin: left a comment on your bug | 22:08 |
| bjornar__ | Why would I get 404 on a call to /v2/ ? | 22:10 |
| *** daemontool has joined #openstack-glance | 22:11 | |
| bjornar__ | Is that normal, perhaps? | 22:13 |
| *** dims has quit IRC | 22:14 | |
| *** _ducttape_ has quit IRC | 22:14 | |
| *** ducttape_ has joined #openstack-glance | 22:15 | |
| mfedosin | flwang: thanks, frankly speaking I don't see any good solution for this | 22:15 |
| mfedosin | in Glare we use 3-stepped workflow: creation, uploading, activation | 22:16 |
| mfedosin | after activation any modifications with data are forbidden | 22:16 |
| mfedosin | unfortunately we can't use it in Glance v2 | 22:17 |
| *** kzaitsev_mb has quit IRC | 22:17 | |
| mfedosin | flaper87: I have to share it http://lists.openstack.org/pipermail/openstack/2016-February/015290.html | 22:18 |
| flaper87 | mfedosin: flwang please, let's follow up on the *bug* | 22:18 |
| flaper87 | mfedosin: I replied to that thread | 22:19 |
| *** vijendar has quit IRC | 22:19 | |
| flwang | flaper87: sure, i will follow up the bug since recently i'm working on the location stuff | 22:19 |
| flaper87 | flwang: mfedosin just don't discuss that bug on IRC | 22:20 |
| flaper87 | thanks :) | 22:20 |
| mfedosin | no one reads this chat anyway :) | 22:20 |
| *** smatzek__ has joined #openstack-glance | 22:21 | |
| flaper87 | Google does | 22:21 |
| flaper87 | hahahaha | 22:21 |
| flaper87 | and I do :P | 22:21 |
| flwang | mfedosin: as for your mail, can you give an image id when you create it? | 22:21 |
| flwang | we support given image id in v1, not sure v2, can't remember | 22:21 |
| mfedosin | can't in v2 | 22:22 |
| flwang | mfedosin: aoh | 22:22 |
| *** daemontool has quit IRC | 22:23 | |
| flaper87 | gtg | 22:24 |
| *** _ducttape_ has joined #openstack-glance | 22:25 | |
| *** vijendar has joined #openstack-glance | 22:27 | |
| *** ducttape_ has quit IRC | 22:28 | |
| *** dane-fichter has quit IRC | 22:30 | |
| *** julim has quit IRC | 22:33 | |
| *** delattec has joined #openstack-glance | 22:34 | |
| openstackgerrit | Deepti Ramakrishna proposed openstack/glance: Support importing OVA/OVF package to Glance https://review.openstack.org/214810 | 22:36 |
| *** cdelatte has quit IRC | 22:37 | |
| *** smatzek__ has quit IRC | 22:37 | |
| *** ninag has joined #openstack-glance | 22:37 | |
| bjornar__ | where is the cunked upload iterator implemented? | 22:40 |
| *** burgerk has quit IRC | 22:45 | |
| *** jdillaman has quit IRC | 22:48 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 22:54 | |
| *** harshs has joined #openstack-glance | 22:57 | |
| *** harshs has quit IRC | 22:58 | |
| *** jdillaman has joined #openstack-glance | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-glance | 23:01 | |
| *** mfedosin has quit IRC | 23:04 | |
| *** sgotliv has quit IRC | 23:06 | |
| *** azbiswas_ has quit IRC | 23:07 | |
| *** dims has joined #openstack-glance | 23:08 | |
| *** achanda has joined #openstack-glance | 23:10 | |
| *** _ducttape_ has quit IRC | 23:15 | |
| *** pushkaru has quit IRC | 23:18 | |
| *** gberginc has quit IRC | 23:18 | |
| *** vijendar has quit IRC | 23:21 | |
| *** vilobhmm11 has quit IRC | 23:25 | |
| *** vilobhmm11 has joined #openstack-glance | 23:25 | |
| *** vijendar has joined #openstack-glance | 23:28 | |
| openstackgerrit | Erno Kuvaja proposed openstack/glance-specs: Initial lite-specs.rst and per deliverable folders https://review.openstack.org/282516 | 23:29 |
| *** kzaitsev_mb has joined #openstack-glance | 23:38 | |
| *** pushkaru has joined #openstack-glance | 23:45 | |
| *** vijendar has quit IRC | 23:51 | |
| *** pushkaru has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!