Thursday, 2015-08-06

« Wednesday, 2015-08-05 Index Friday, 2015-08-07 »
*** r-daneel has quit IRC00:16
*** annegentle has quit IRC00:18
*** boris-42 has quit IRC00:20
*** bapalm has joined #openstack-glance00:37
*** bapalm has quit IRC00:39
*** tsekiyam_ has joined #openstack-glance00:45
*** mtanino has quit IRC00:46
*** tsekiyama has quit IRC00:48
*** tsekiyam_ has quit IRC00:50
*** annegentle has joined #openstack-glance01:14
*** dims_ has joined #openstack-glance01:21
*** dims has quit IRC01:22
*** sdake has joined #openstack-glance01:27
*** sdake_ has joined #openstack-glance01:30
*** sdake has quit IRC01:34
*** harshs has joined #openstack-glance01:40
*** mingdang1 has joined #openstack-glance01:54
*** harshs has quit IRC01:56
*** spzala has quit IRC01:57
*** dims has joined #openstack-glance02:03
*** dims_ has quit IRC02:05
*** dims_ has joined #openstack-glance02:06
*** dims has quit IRC02:09
*** sdake_ has quit IRC02:09
*** dims_ has quit IRC02:22
*** annashen has joined #openstack-glance02:28
*** harshs has joined #openstack-glance02:43
openstackgerritSabari proposed openstack/glance_store: Handle swift store's optional dependecy  https://review.openstack.org/20974403:12
openstackgerritSabari proposed openstack/glance_store: Conditionally import dependencies of swift store  https://review.openstack.org/20974403:13
*** boris-42 has joined #openstack-glance03:27
*** junhongl has joined #openstack-glance03:54
flaper87sabari: thanks for the reviews03:55
flaper87sabari: Did you read my last comment on the spec?03:55
*** ayoung has quit IRC03:56
*** annashen has quit IRC04:17
*** annegentle has quit IRC04:41
openstackgerritMerged openstack/glance_store: Imported Translations from Transifex  https://review.openstack.org/20492005:01
*** junhongl has quit IRC05:15
*** junhongl has joined #openstack-glance05:15
*** harshs has quit IRC05:22
*** takedakn has joined #openstack-glance05:24
*** takedakn has quit IRC05:28
*** junhongl has quit IRC05:30
openstackgerritMerged openstack/glance: Make utf8 the default charset for mysql  https://review.openstack.org/18134705:30
*** tpeoples has quit IRC05:46
*** tpeoples has joined #openstack-glance05:51
*** groen692 has joined #openstack-glance06:03
*** dims has joined #openstack-glance06:03
openstackgerritEdgar Magana proposed openstack/glance: Fix few typos in glance  https://review.openstack.org/20911706:07
openstackgerritEdgar Magana proposed openstack/glance: Fix few typos in glance  https://review.openstack.org/20911706:08
*** dims has quit IRC06:09
openstackgerritAbhijeet Malawade proposed openstack/glance: Fix Request ID has a double 'req-' at the start  https://review.openstack.org/20652106:18
*** boris-42 has quit IRC06:20
*** e0ne has joined #openstack-glance06:21
openstackgerritMerged openstack/glance_store: Add explicit dependencies for store dependencies  https://review.openstack.org/20711606:22
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Imported Translations from Transifex  https://review.openstack.org/20818606:25
openstackgerritJamie Lennox proposed openstack/glance: Turn on debug logging for ValueError  https://review.openstack.org/20977406:26
*** belmoreira has joined #openstack-glance06:51
*** junhongl has joined #openstack-glance06:52
*** e0ne has quit IRC07:00
*** takedakn has joined #openstack-glance07:01
*** ishant has joined #openstack-glance07:01
*** e0ne has joined #openstack-glance07:11
*** tpeoples has quit IRC07:14
*** markus_z has joined #openstack-glance07:24
*** e0ne has quit IRC07:26
*** takedakn has quit IRC07:28
*** pbourke has quit IRC07:33
*** pbourke has joined #openstack-glance07:33
*** sayali has joined #openstack-glance07:50
*** sdake has joined #openstack-glance07:54
*** ishant_ has joined #openstack-glance07:55
*** ishant has quit IRC07:59
*** GB21 has joined #openstack-glance08:02
*** jistr has joined #openstack-glance08:14
*** e0ne has joined #openstack-glance08:22
*** e0ne has quit IRC08:41
*** e0ne has joined #openstack-glance08:58
jokke_Morning09:00
*** haomaiwang has quit IRC09:21
*** haomaiwang has joined #openstack-glance09:21
*** bdossant has joined #openstack-glance09:25
*** MattMan has quit IRC09:42
*** e0ne has quit IRC09:43
*** MattMan has joined #openstack-glance09:44
openstackgerritKentaro Takeda proposed openstack/glance: Incorrect variable name is declared in test_validate_key_cert_key_cant_read() method.  https://review.openstack.org/20984609:46
openstackgerritKentaro Takeda proposed openstack/glance: Incorrect variable name is declared in  test_validate_key_cert_key_cant_read() method.  https://review.openstack.org/20984609:47
openstackgerritKentaro Takeda proposed openstack/glance: Incorrect variable name is declared  https://review.openstack.org/20984609:47
openstackgerritKentaro Takeda proposed openstack/glance: Incorrect variable name is declared  https://review.openstack.org/20984609:48
*** dims has joined #openstack-glance09:49
openstackgerritKentaro Takeda proposed openstack/glance: Incorrect variable name is declared  https://review.openstack.org/20984609:52
*** GB21 has quit IRC09:56
*** GB21 has joined #openstack-glance09:58
*** e0ne has joined #openstack-glance09:59
*** eandersson has joined #openstack-glance10:26
openstackgerritNiall Bunting proposed openstack/glance: Removes *.sql files from git ignore  https://review.openstack.org/20986610:50
*** dims_ has joined #openstack-glance10:51
*** e0ne has quit IRC10:52
*** dims has quit IRC10:53
*** e0ne has joined #openstack-glance10:54
openstackgerritMerged openstack/glance: Include metadefs files in all packages  https://review.openstack.org/20679010:57
*** smatzek has joined #openstack-glance11:00
*** sdake has quit IRC11:07
jamielennoxis there a restriction placed on the glance API used in the gate?11:21
jamielennoxlike an explicit use v11:21
jamielennox111:21
*** mingdang1 has quit IRC11:23
*** spzala has joined #openstack-glance11:24
*** shalq has joined #openstack-glance11:24
jokke_jamielennox: not that I know ... we do test v2 api in the gate11:27
jamielennoxjokke_: hmm, i can't replicate an issue i have in gate locally11:28
jamielennoxthought it might have been a different API as i'm getting different log messages11:28
jokke_jamielennox: interesting, what are you seeing?11:28
jamielennoxjokke_: so i'm trying to fix devstack to use glance -> swift v3 auth api11:28
jamielennoxwith a few addiitonal debug patches i'm seeing http://logs.openstack.org/51/209351/3/check/gate-tempest-dsvm-full/7c182e9/logs/screen-g-api.txt.gz#_2015-08-06_10_27_19_75811:29
jamielennoxwhich is really annoying because it means it gets caught by ValueError at https://github.com/openstack/glance/blob/master/glance/api/v1/upload_utils.py#L27211:30
jamielennoxwhen i do this locally i don't see the debug messages i'm seeing in gate, so i thought maybe locally i was uploading images to glance with v2, but v1 was being used in the gate11:30
jamielennoxi *think* it means that i've got invalid credentials but i can't get a useful message11:31
jokke_jamielennox: ah, one of those things we mask the exception actually raised to us11:31
jamielennoxjokke_: right - catching ValueError so broadly is a bit... dumb11:32
jamielennoxbut then swiftclient not being able to serialize exceptions is also wrong11:32
jokke_so by the gate issue, it looks like swiftclient is trying to do something nasty when it's authentication fails11:33
jokke_yeah11:33
jamielennoxyep - this also passes on my local machine11:33
jokke_jamielennox: on devstack?11:33
jamielennoxjokke_: i had Depends-On: https://review.openstack.org/#/c/209800/11:34
jamielennoxjokke_: which i hoped would at least show the issue but it didn't show up11:34
* jokke_ is just thinking could this be one of those things that issue has been fixed, but not released so devstack looks all good as it pulls the gits but gate does not as it pulls the packages from pypi11:35
*** eandersson has quit IRC11:35
jamielennoxright, as that review is actually failing so it could never have worked11:36
*** eandersson has joined #openstack-glance11:36
jamielennoxjokke_: so i feel the real error is http://logs.openstack.org/51/209351/3/check/gate-tempest-dsvm-full/7c182e9/logs/screen-g-api.txt.gz#_2015-08-06_10_27_17_89311:37
jokke_my thought exactly11:37
jamielennoxi've very explicity said to use v3 auth and yet that's sourcing keystoneclient.auth.identity.v211:37
jamielennoxbut i can't replicate it locally and i've got no idea how to debug what happened11:37
*** e0ne has quit IRC11:41
*** e0ne has joined #openstack-glance11:44
jokke_jamielennox: by quick grep the auth_version gets never set anywhere to 3 for swiftclient11:46
jamielennoxjokke_: http://logs.openstack.org/51/209351/3/check/gate-tempest-dsvm-full/7c182e9/logs/etc/glance/glance-swift-store.conf.txt.gz11:47
jamielennoxthe version is still being set to 2 in the main conf file but that glance-swift file should override that11:48
jokke_jamielennox: sorry, this might be really far off, but try swift_store_auth_version = 3 there instead11:50
jokke_jamielennox: oh sorry, never mind ... was looking wrong function11:52
*** ishant_ has quit IRC11:53
jokke_jamielennox: I think I found it!11:59
jamielennoxjokke_: awesome, i was thinking of calling it a night otherwise12:00
jokke_jamielennox: https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/swift/store.py#L94012:00
jamielennoxjokke_: :O, wtf is that12:01
jokke_that's very much hardcoded auth_version value on get_connection :P12:02
jamielennoxjokke_: so how is it that locally that's not being triggered?12:03
*** mingdang1 has joined #openstack-glance12:03
jamielennoxjokke_: that's the multi-tenant store12:03
jokke_gate uses multitenant store vs you use locally single tenant12:03
jamielennoxwhich isn't good, but shouldn't matter here12:03
jamielennoxjokke_: ...12:03
jamielennoxwhen did that change12:03
jokke_I don't see any other place where that would be overriden12:04
jamielennoxmulti-tenant store should use the user token12:04
jokke_that's purely sophisticated quess from me12:04
jokke_I just can't find any other reference that would not be overriden bu the swift store conf auth_version12:05
jamielennox#swift_store_multi_tenant = False in http://logs.openstack.org/51/209351/3/check/gate-tempest-dsvm-full/7c182e9/logs/etc/glance/glance-api.conf.txt.gz12:05
jamielennoxunless the default in glance_store has changed recently12:05
jamielennoxbecause i had a review that i abandoned to switch devstack to multi-tenant mode to work around this issue12:06
jokke_glance_store/_drivers/swift/store.py:    cfg.BoolOpt('swift_store_multi_tenant', default=False,12:07
jamielennoxnone of the swift_store_* options are shown in the options debug list12:07
jamielennoxwhich i think means they aren't referenced in the glance oslo.config options list12:08
jokke_jamielennox: we load SingleTenantStore http://logs.openstack.org/51/209351/3/check/gate-tempest-dsvm-full/7c182e9/logs/screen-g-api.txt.gz#_2015-08-06_10_27_14_50612:12
openstackgerritNiall Bunting proposed openstack/glance: Removes *.sql files from git ignore  https://review.openstack.org/20986612:12
jamielennoxjokke_: ok, so that's not it12:12
jokke_sorry for false alarm12:13
jamielennoxjokke_: so what happens if i upload images to :9292/images without a version number12:14
jamielennoxhttp://logs.openstack.org/51/209351/3/check/gate-tempest-dsvm-full/7c182e9/logs/devstacklog.txt.gz#_2015-08-06_10_27_15_52612:14
jamielennoxi mean i didn't change that so i don't know why it would cause an issue now12:15
jokke_jamielennox: as I expected12:16
jokke_glance store does not have that on last release12:16
jamielennoxjokke_: no, but Depends-On: should take care of that12:16
jamielennoxjokke_: oh - but it doesn't12:17
jamielennoxwtf12:17
jokke_depends on just prevents merge before the depending changes has merged, it does not patch the component changed on those12:18
jokke_the tests are still ran against the packages pulled from pip12:19
jamielennoxjokke_: it should do more than that, and we can see that https://review.openstack.org/#/c/209774/ is used12:20
jamielennoxjokke_: because without that i didn't see an error at all12:20
jamielennoxjokke_: however it appears that it doesn't work for libraries12:20
jamielennoxjokke_: which would explain why my swfitclient fix wasn't showing up, even when i changed the ordering12:21
jokke_http://logs.openstack.org/51/209351/3/check/gate-tempest-dsvm-full/7c182e9/logs/pip-freeze.txt.gz  glance-store==0.7.112:23
jokke_yes, projects comes in from git so the can be cherry picked in ... libs does not12:24
jamielennoxjokke_: excellent, so once again people writing clients get ignored - wtf is the point of that12:24
*** dims has joined #openstack-glance12:24
jokke_jamielennox: tell me about it ... but the crying is insane when the lib release breaks the gate as those changes were never tested against everything else before12:25
jamielennoxjokke_: that would seem to be the point of testing against recent client12:25
*** vijendar has joined #openstack-glance12:26
*** vijendar has quit IRC12:26
*** vijendar has joined #openstack-glance12:26
*** dims_ has quit IRC12:27
jokke_jamielennox: IIUC that is to avoid the situation where deployer tries to install from master and the lib does not have the dependencies released12:27
jamielennoxjokke_: i don't think that applies in Depends-On in gate12:28
jokke_well the depends on just ensures that the change gets merged, not that it gets released12:28
*** mingdang_ has joined #openstack-glance12:29
jokke_what I would suggest 'though is that we would add 1 test run done against master ... so you could see that "Yes, my change fails gating because client x and z has not released their related changes, but them included this works"12:30
*** mingdang1 has quit IRC12:30
jokke_anyways ... call it night, we know why it fails and you will need to wait the release, before that change will merge. Based on the local testing it works anyways ;)12:31
jokke_will *not* merge12:32
jamielennoxjokke_: yea, checked in -infra as well12:33
jokke_OR if you change that swift_auth_version to 3 in gapi.conf it should pass also12:33
jamielennoxnothing we can do until we get a library release12:33
jamielennoxjokke_: no, because it's still not supported until the master branch of glance_store12:33
jokke_as that's what it's utilizing with the current code12:34
jokke_ah12:34
jamielennoxflaper87: ok - really wanting a glance_store release when you can12:34
jokke_nikhil_k: ^^12:34
flaper87jamielennox: ok, roger that12:34
flaper87sigmavirus24_awa: nikhil_k ^ Lets sync and schedule one!12:35
jokke_flaper87: yeah would make sense ... 0.8.0 ;)12:35
jokke_previous was already 3 weeks ago so :P12:35
*** edmondsw has joined #openstack-glance12:36
jamielennoxjokke_: thanks for your help, i'm done for the night12:37
nikhil_kflaper87: jokke_ : jamielennox : please sync with dhellmann or ttx as well for that. I am cool with whenever12:38
flaper87nikhil_k: ah jeez, you're right!12:39
flaper87nikhil_k: great point12:39
flaper87ETOOMANYTEAMS12:39
flaper87requesting a release out of https://github.com/openstack/glance_store/commit/29a84f7ced03ece1eb5b2128593dc84f8f95bec112:41
*** bapalm has joined #openstack-glance12:44
flaper87jokke_: https://review.openstack.org/#/c/209536/12:44
jamielennoxflaper87: how do we sync that with dhellmann? do you have to email or put in a review somewhere?12:45
flaper87jokke_: a) it'll take time and many other changes b) shipping the json file is more complicated than haivng it in a python module because then ou gotta figure out where the heck that file should be stored c) sure12:45
jamielennoxsend an email...12:45
flaper87jamielennox: I just pingged him on IRC, I'll let you know12:45
jamielennoxflaper87: cheerys12:46
jamielennoxcheers12:46
jamielennoxalright - night all12:46
flaper87jamielennox: night12:46
flaper87jokke_: actually, I'll put that in the review12:46
*** zul has joined #openstack-glance12:46
flaper87jokke_: replied on the review. I'd love to get some feedback when you can12:51
flaper87really want to fix his issue asap, it's really unfortunate and awful12:52
jokke_flaper87: just got back from bit of fresh air and fetching coffee13:02
jokke_flaper87: it was preference order, not reality check ;)13:02
jokke_flaper87: I'd be happy with image_schema module just to keep it bit tidier13:05
*** junhongl has left #openstack-glance13:08
*** abhishekk has joined #openstack-glance13:09
jokke_sigmavirus24_awa: wanna join to the fight _again_ http://logs.openstack.org/01/178201/2/check/gate-glance_store-python27/a52f322/console.html#_2015-08-06_12_02_34_94313:10
*** vijendar has quit IRC13:25
openstackgerritMarek Denis proposed openstack/glance-specs: Federated model for cloning glance objects  https://review.openstack.org/19189713:30
*** marcusvrn has joined #openstack-glance13:30
*** vokhrimenko has quit IRC13:35
marekdflaper87: rosmaita ^^13:35
*** ayoung has joined #openstack-glance13:39
buntingjokke_: https://review.openstack.org/#/c/207150/ here is my current patch for the problem if it helps13:44
*** edmondsw has quit IRC13:46
*** harshs has joined #openstack-glance13:49
openstackgerritNiall Bunting proposed openstack/glance: Bodies that are not dicts or lists return 400  https://review.openstack.org/20413813:50
*** jaypipes has quit IRC13:53
*** sigmavirus24_awa is now known as sigmavirus2413:55
*** sdake has joined #openstack-glance13:55
sigmavirus24jokke_: why are you concerned about that? It means the Python that's installed at the gate is super old14:00
*** r-daneel has joined #openstack-glance14:01
*** bpoulos has joined #openstack-glance14:02
jokke_sigmavirus24: well I'm worried about that because our stable/kilo gate is bonkers14:02
sigmavirus24jokke_: it's merely a warning14:02
sigmavirus24It's informational of sorts but also a "Hey we can't really give you the best security because this python is old and the ssl module is absolutely awful"14:02
jokke_sigmavirus24: sorry my bad14:03
sigmavirus24jokke_: No worries14:04
sigmavirus24I know it's a red-herring14:04
sigmavirus24Courtesy meeting reminder: ativelkov, cpallares, esheffield, flaper87, flwang1, hemanthm, ivasilevskaya, jokke_, kragniz, lakshmiS, mclaren, mfedosin, nikhil_k, Nikolay_St, Olena, pennerc, rosmaita, sigmavirus24, sabari, TravT, zhiyan, pkoniszewski, krykowski, ajayaa, GB21, bpoulos, harshs, abhishekk, bunting14:04
*** vijendar has joined #openstack-glance14:04
GB21sigmavirus24, thanks for the reminder :D14:04
sigmavirus24You're welcome GB2114:05
*** GB21 has quit IRC14:10
*** tpeoples has joined #openstack-glance14:13
*** sdake has quit IRC14:14
*** bpoulos has quit IRC14:16
*** dims has quit IRC14:25
*** dims has joined #openstack-glance14:26
*** edmondsw has joined #openstack-glance14:28
*** mingdang_ has quit IRC14:34
*** jasondot_ is now known as jasondotstar14:38
*** tsekiyama has joined #openstack-glance14:44
*** annegentle has joined #openstack-glance14:45
*** MVenesio has joined #openstack-glance14:45
*** delattec has joined #openstack-glance14:48
*** lakshmiS has joined #openstack-glance14:49
*** e0ne has quit IRC14:52
*** e0ne has joined #openstack-glance14:57
*** Fin1te has joined #openstack-glance14:58
*** r-daneel has quit IRC14:58
*** bapalm_ has joined #openstack-glance15:00
*** r-daneel has joined #openstack-glance15:01
*** bapalm has quit IRC15:01
flaper87nikhil_k: https://review.openstack.org/#/c/197899/ :D :P ;) PLEASE! (the spec the spec the spec) https://review.openstack.org/#/c/188388/15:02
flaper87rosmaita: ^15:02
* flaper87 waves15:02
jokke_gr8 so we have 2 security bugs discussed openly on today's meeting :|15:02
flaper87it's 1am here and I have a plane to catch tomorrow morning15:02
flaper87jokke_: sometimes happens, it's not always obvious how you can exploit a bug15:02
abhishekkjokke_, sigmavirus24, nikhil_k, sabari: please have a look at patch  https://review.openstack.org/#/c/207847/ once you get time15:02
jokke_flaper87: so you'll be here with us whole evening ;)15:02
flaper87jokke_: if you're buying drinks, I'll stay15:03
flaper87:P15:03
* flaper87 is doing research on headphones because he lost his15:04
*** ishant has joined #openstack-glance15:04
*** abhishekk has quit IRC15:04
rosmaitaflaper87: will put some comments on triggerd task spec ... soon15:05
rosmaitasorry i didn't get to it last night15:05
*** jaypipes has joined #openstack-glance15:05
rosmaitawill do it today new york time15:05
jokke_flaper87: sure deal! I go and buy drinks in local and you continue working.15:05
flaper87rosmaita: sounds perfect, thanks!15:07
flaper87jokke_: hey hey, don't try to fool me. You send me money over through paypal (you pay the fee) and I'll buy the drinks15:07
flaper87ok, I'm out! take care all15:08
marekdflaper87: nikhil_k rosmaita : Hi, please take a look when you have a free minute : https://review.openstack.org/#/c/191897/15:08
*** mtanino has joined #openstack-glance15:09
*** zul has quit IRC15:10
openstackgerritMatthew Edmonds proposed openstack/glance: clean up requirements  https://review.openstack.org/20324215:11
*** harshs has quit IRC15:21
openstackgerritKentaro Takeda proposed openstack/glance: Fix variable name in the second 'with' statement  https://review.openstack.org/20999015:25
*** sdake has joined #openstack-glance15:31
*** spzala has quit IRC15:38
*** jaypipes has quit IRC15:43
*** jaypipes has joined #openstack-glance15:56
nikhil_kflaper87: marekd : sure thing!15:56
*** GB21 has joined #openstack-glance16:00
*** ishant has quit IRC16:01
*** belmoreira has quit IRC16:03
*** GB21_ has joined #openstack-glance16:05
*** GB21 has quit IRC16:06
*** hogepodge has quit IRC16:06
*** hogepodge has joined #openstack-glance16:07
openstackgerritMerged openstack/python-glanceclient: Support for Metadata Definition Catalog for Tags  https://review.openstack.org/13861216:08
*** jistr has quit IRC16:12
*** sdake has quit IRC16:14
*** GB21_ is now known as GB2116:17
*** groen692 has quit IRC16:18
*** exploreshaifali has joined #openstack-glance16:24
*** annegentle has quit IRC16:38
*** smatzek has quit IRC16:41
*** bdossant has quit IRC16:41
*** takedakn has joined #openstack-glance16:42
*** Fin1te has quit IRC16:42
*** takedakn has quit IRC16:42
openstackgerritTomoki Sekiyama proposed openstack/glance_store: Implement get, add and delete for cinder store  https://review.openstack.org/16641416:51
openstackgerritErno Kuvaja proposed openstack/glance: Add mechanism to limit Request ID size  https://review.openstack.org/21002516:56
openstackgerritErno Kuvaja proposed openstack/glance: Setting default max_request_id_length to 64  https://review.openstack.org/21002616:56
*** lakshmiS has quit IRC16:57
*** bapalm_ has quit IRC17:01
openstackgerritGB21 proposed openstack/glance: Add a task_scrub function  https://review.openstack.org/20925517:03
*** harshs has joined #openstack-glance17:11
*** spzala has joined #openstack-glance17:12
*** chlong has quit IRC17:14
jokke_nikhil_k, sigmavirus24: https://bugs.launchpad.net/glance/+bug/1482301 fix proposed to all branches17:19
openstackLaunchpad bug 1482301 in Glance "'X-Openstack-Request-ID' leght limited only by header size" [Critical,In progress] - Assigned to Erno Kuvaja (jokke)17:19
jokke_hemanthm: ^^ you as well if still around17:23
nikhil_kjokke_: done17:23
nikhil_kat least the bug part17:24
jokke_nikhil_k: ;)17:24
jokke_nikhil_k: we gotta wait for the checks, but would be nice if you would look through the fix if and flag anything you think needs to change so I can do them asap17:25
nikhil_kjokke_: added a comment on master17:26
hemanthmjokke_: hey17:26
nikhil_khemanthm: ^17:26
jokke_nikhil_k:17:27
jokke_sorry17:27
hemanthmlooking17:27
jokke_is there standard size?17:27
nikhil_kis abhishek around?17:27
nikhil_kI thought it was uuid?17:28
nikhil_knot sure what's the latest on the consensus around the prefixing it with other things17:28
jokke_nikhil_k: yeah, and it's actually irrelevant on that first change as it's intentionally just bringing the mechanism without limit so it was easily backportable17:30
jokke_backwards compatible and all17:30
nikhil_kI see17:30
nikhil_kthat's great point17:30
nikhil_kok, +2 coming your way then17:30
nikhil_kbut good to get a +1 from abhishek first17:30
nikhil_kot tpatil17:30
nikhil_kor*17:30
nikhil_kfirst == before merge17:31
*** sdake has joined #openstack-glance17:33
jokke_I'm well willing to debate on the one that actually sets any limit, I just want to have that limiting mechanism in there so we can provide admins tools to plug that security hole ... the initial 0 does not limit17:41
*** marcusvrn has quit IRC17:43
*** eandersson has quit IRC17:47
*** exploreshaifali has quit IRC17:47
*** exploreshaifali has joined #openstack-glance17:52
*** markus_z has quit IRC17:53
*** exploreshaifali has quit IRC17:54
*** zul has joined #openstack-glance17:56
*** abhishekk has joined #openstack-glance18:01
abhishekkjokke_: hi you around?18:01
abhishekknikhil_k: hi18:02
nikhil_kabhishekk: o/18:02
nikhil_kI am around just for a bit, sorry got a plenty to handle today ://18:03
abhishekknikhil_k: sure no proble18:03
nikhil_kabhishekk: is this the req-id limit thin?18:03
abhishekkjust want to say https://bugs.launchpad.net/glance/+bug/1482301 is not there in juno and kilo18:03
openstackLaunchpad bug 1482301 in Glance "'X-Openstack-Request-ID' lenght limited only by header size" [Critical,In progress] - Assigned to Erno Kuvaja (jokke)18:03
abhishekkthen how it is just by design in liberty?18:04
*** bapalm has joined #openstack-glance18:04
abhishekksorry, https://bugs.launchpad.net/glance/+bug/148019118:04
openstackLaunchpad bug 1480191 in Glance "User can send 'request-id' from headers to glance api" [Undecided,Opinion] - Assigned to Abhijeet Malawade (abhijeet-malawade)18:04
nikhil_keh, that's my gtd got in the way18:04
nikhil_kah18:04
abhishekkI am not sure how x-openstack-request-id header passed by user is not accepted in juno and kilo and in current master it is accepting18:06
nikhil_kabhishekk: can you please elaborate more on this?18:08
abhishekknikhil_k: even if user pass 'x-openstack-request-id' from header, it is not accepted in glance18:09
abhishekknikhil_k: i ran this curl command on stable/juno curl -g -i -X GET -H 'Accept-Encoding: gzip, deflate' -H 'Accept: */*' -H 'Connection: keep-alive' -H 'X-Auth-Token: ff173629aa104fa9bdb7266347d05128' -H 'X-Openstack-Request-Id: testing--aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa123456' http://10.69.4.174:9292/v2/images?limit=118:09
abhishekkand got this response, HTTP/1.1 200 OK Content-Length: 737 Content-Type: application/json; charset=UTF-8 X-Openstack-Request-Id: req-6db4f65a-0440-4a49-9919-521b472506a2 Date: Thu, 06 Aug 2015 17:56:54 GMT Connection: keep-alive18:10
nikhil_khmm, is that oslo.middleware update that caused this?18:10
abhishekkthe 'X-Openstack-Request-Id' is generated by glance IMO18:11
nikhil_kbut teh middleware handles/can handle headers18:11
nikhil_kI don;t think we enabled anything in glance that would loosen header restrictions18:12
nikhil_kabhishekk: is your issue about letting people pass that header?18:13
abhishekknikhil_k: yes18:13
nikhil_kabhishekk: I think fixing that case makes sense18:13
nikhil_kwe are allowed to make api changes until liberty freeze18:14
abhishekknikhil_k: ok18:14
nikhil_klet's hear from jokke on that bug why he things e need to let pass that header18:14
nikhil_kabhishekk: thanks, catchya later18:14
abhishekknikhil_k: thank you18:15
*** abhishekk has quit IRC18:15
*** spzala has quit IRC18:17
jokke_I'm here18:17
*** pal has joined #openstack-glance18:20
*** Fin1te has joined #openstack-glance18:21
*** ayoung has quit IRC18:34
*** e0ne has quit IRC18:37
*** mtreinish has quit IRC18:37
*** mtreinish has joined #openstack-glance18:39
*** spzala has joined #openstack-glance18:40
*** lakshmiS has joined #openstack-glance18:45
*** harlowja has quit IRC18:56
*** lakshmiS has quit IRC18:58
*** GB21_ has joined #openstack-glance19:04
*** harshs has quit IRC19:05
*** GB21 has quit IRC19:07
*** annegentle has joined #openstack-glance19:09
*** wokuma has joined #openstack-glance19:13
*** harshs has joined #openstack-glance19:24
*** e0ne has joined #openstack-glance19:24
*** harshs has quit IRC19:25
*** boris-42 has joined #openstack-glance19:39
*** fifieldt_ has joined #openstack-glance19:39
*** fifieldt has quit IRC19:42
*** ayoung has joined #openstack-glance19:49
*** fnordahl has quit IRC20:10
*** daemontool_ has joined #openstack-glance20:10
*** Fin1te has quit IRC20:12
*** marzif has quit IRC20:12
*** tpeoples has quit IRC20:16
*** fnordahl has joined #openstack-glance20:17
*** DCWilliams_VA has joined #openstack-glance20:21
hemanthmjokke_, flaper87: created the v1 status bug we talked about earlier in the morning.20:23
*** dims_ has joined #openstack-glance20:23
hemanthmit's private for now, but I subscribed you both to it20:23
*** dims_ has quit IRC20:24
*** e0ne has quit IRC20:24
sigmavirus24hemanthm: if it was spoken about in the meeting and you're continuing to speak about it in public IRC channels there's no point keeping it private security20:25
*** dims_ has joined #openstack-glance20:25
*** dims has quit IRC20:26
*** DCWilliams_VA has quit IRC20:26
hemanthmsigmavirus24: well I realized certain things only after I tested it in devstack. It can't be worse to let the security team look at it first20:27
hemanthmanyway, +1 you have a point :)20:28
sigmavirus24hemanthm: yeah just stop using public logged channels to discuss it20:28
hemanthmaye20:30
*** pal has quit IRC20:37
*** sgotliv_ has quit IRC20:44
*** bapalm has quit IRC20:50
*** GB21_ has quit IRC20:59
jokke_thanks hemanthm21:02
jokke_I'll include that to my comments in it21:03
jokke_hemanthm: if you have the environment still available, mind to check how image-upload behaves21:06
*** e0ne has joined #openstack-glance21:09
*** ayoung has quit IRC21:11
*** e0ne has quit IRC21:16
*** bpoulos has joined #openstack-glance21:17
*** annegentle has quit IRC21:18
*** harshs has joined #openstack-glance21:24
*** MVenesio has quit IRC21:24
*** flwang1 has joined #openstack-glance21:29
flwang1nikhil_k: ping21:29
*** harlowja has joined #openstack-glance21:34
*** belmoreira has joined #openstack-glance21:35
*** harlowja has quit IRC21:39
*** harlowja has joined #openstack-glance21:39
openstackgerritErno Kuvaja proposed openstack/glance: Setting default max_request_id_length to 64  https://review.openstack.org/21002621:40
openstackgerritErno Kuvaja proposed openstack/glance: Add mechanism to limit Request ID size  https://review.openstack.org/21002521:40
*** alanf has joined #openstack-glance21:43
alanfanyone know if glance in fact support image creation from tar.gz file?21:44
*** zul has quit IRC21:45
*** vijendar has quit IRC21:47
*** zul has joined #openstack-glance21:55
*** delattec has quit IRC21:58
openstackgerritBrianna Poulos proposed openstack/glance-specs: Image Signing and Verification Support  https://review.openstack.org/17794822:03
jokke_alanf: how do you mean?22:12
jokke_alanf: iirc some compresser archives were supported on Images API v1 copy-from functionality, but we do not support that currently in v2 nor we do support it if user just uploads it (I'm not sure 'though if it works)22:13
jokke_compressed even22:14
alanfjokke_ i am trying to determine if glance can properly download and handle a remote image source that is compressed22:14
alanfi see22:14
openstackgerritFei Long Wang proposed openstack/python-glanceclient: Use API v2 as default  https://review.openstack.org/19393122:15
alanfcurrent in horizon there is help text:22:15
alanfCurrently only images available via an HTTP URL are supported. The image location must be accessible to the Image Service. Compressed image binaries are supported (.zip and .tar.gz.)22:15
alanfit has been around since 2012 i think, but recently a user reported the feature broken22:16
alanfi am trying to determine is this is a defect in horizon and/or glance, or this is deprecated and this helper text needs to be updated22:16
openstackgerritFei Long Wang proposed openstack/python-glanceclient: Use API v2 as default  https://review.openstack.org/19393122:18
jokke_alanf: it's supposed to work still with Images Api v122:18
jokke_Not sure if Horizon even uses that anymore and wouldn't be surprised if it had broken at some point and no-one just noticed it22:19
jokke_so feel free to file a bug if your investigations comes to the conclusion that the issue is indeed in glance22:19
alanfok thanks!22:19
*** edmondsw has quit IRC22:20
*** zul has quit IRC22:35
jamielennoxflaper87, nikhil_k, sigmavirus24: so in addition to glance_store we need a glanceclient release as well please22:42
*** dims_ has quit IRC22:50
*** dims_ has joined #openstack-glance23:10
*** shalq has quit IRC23:16
*** shalq has joined #openstack-glance23:16
*** r-daneel has quit IRC23:17
*** sdake has quit IRC23:38
« Wednesday, 2015-08-05 Index Friday, 2015-08-07 »

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