Tuesday, 2015-03-10

« Monday, 2015-03-09 Index Wednesday, 2015-03-11 »
sabarikragniz: Congrats on your shiny new button. You are welcome to give it a try on my patches ;).00:11
sabarijk !.00:11
*** smc7 has joined #openstack-glance00:23
*** mfedosin_ has quit IRC00:24
*** annashen has quit IRC00:27
*** mfedosin_ has joined #openstack-glance00:28
*** smc7 has quit IRC00:28
openstackgerritSabari proposed openstack/glance: Update config and docs for multiple datastores support  https://review.openstack.org/16197000:45
openstackgerritMerged openstack/glance: SemVer utility to store object versions in DB  https://review.openstack.org/10767600:49
*** r-daneel has quit IRC00:49
*** kfox1111 has quit IRC00:52
*** flwang has quit IRC00:58
openstackgerritMerged openstack/glance: Typos fixed in the comments  https://review.openstack.org/15112901:02
*** vilobhmm1 has quit IRC01:20
*** bkopilov has joined #openstack-glance01:24
*** browne has quit IRC01:25
*** annashen has joined #openstack-glance01:27
*** annashen has quit IRC01:32
*** mfedosin_ has quit IRC01:42
*** rwsu has quit IRC01:47
stevellesmells like merge velocity in here.01:54
openstackgerritJun Hong Li proposed openstack/glance_store: Throw NotFound exception when template is gone  https://review.openstack.org/15316901:54
stevelleburnin it up01:54
*** diegows has quit IRC02:02
*** anki80 has quit IRC02:24
*** hemanth has joined #openstack-glance02:37
*** browne has joined #openstack-glance02:37
*** flwang has joined #openstack-glance02:42
*** haomaiwang has joined #openstack-glance03:19
openstackgerritIan Cordasco proposed openstack/glance: Use graduated oslo.policy  https://review.openstack.org/16236803:29
*** bkopilov has quit IRC03:30
openstackgerritMerged openstack/glance: Updated from global requirements  https://review.openstack.org/16138603:33
*** flwang has quit IRC03:51
*** Longgeek has joined #openstack-glance03:59
*** vilobhmm has joined #openstack-glance04:00
*** smc7 has joined #openstack-glance04:01
*** smc7 has quit IRC04:06
*** rajesht has joined #openstack-glance04:21
*** rajesht has quit IRC04:22
*** hemanth has quit IRC04:54
*** bkopilov has joined #openstack-glance05:16
openstackgerritZhi Yan Liu proposed openstack/glance: Add capabilities to storage driver  https://review.openstack.org/14182505:31
openstackgerritMerged openstack/glance: Switch to latest oslo-incubator  https://review.openstack.org/16190705:44
*** smc7 has joined #openstack-glance05:50
*** smc7 has quit IRC05:55
*** harlowja_ is now known as harlowja_away05:57
openstackgerritOpenStack Proposal Bot proposed openstack/glance: Imported Translations from Transifex  https://review.openstack.org/15857006:01
*** Longgeek has quit IRC06:04
openstackgerritMerged openstack/glance_store: Deprecate VMware store single datastore options  https://review.openstack.org/16230806:11
*** vilobhmm1 has joined #openstack-glance06:22
*** Longgeek has joined #openstack-glance06:23
*** vilobhmm has quit IRC06:23
*** david-lyle has quit IRC06:30
*** iwao has joined #openstack-glance06:38
*** david-lyle has joined #openstack-glance06:39
*** sgotliv has quit IRC06:53
*** vilobhmm1 has quit IRC06:53
*** vilobhmm has joined #openstack-glance06:54
*** bangara has joined #openstack-glance06:55
bangarahello all06:55
bangara:)06:55
bangarahow do we make image to "active state" using create image API of glance service? It is always coming to  "queued state". and also how to specify the location value of the image in API parameter.06:55
bangara<kragniz> hello,you there?06:56
*** smc7 has joined #openstack-glance07:00
iwaoHello :)07:02
*** groen692 has joined #openstack-glance07:03
iwaoDid you tried to upload image ? I think, If image-data is uploaded, the STATUS will be changed to ACTIVE.07:03
*** bangara has quit IRC07:05
*** abhishekk has joined #openstack-glance07:05
*** bkopilov has quit IRC07:07
*** david-lyle is now known as david-lyle_afk07:11
*** browne has quit IRC07:11
*** buzztroll has quit IRC07:17
*** buzztroll has joined #openstack-glance07:17
*** ParsectiX has joined #openstack-glance07:20
*** buzztroll has quit IRC07:21
*** buzztroll has joined #openstack-glance07:21
*** chlong has quit IRC07:21
*** bkopilov has joined #openstack-glance07:22
*** flwang has joined #openstack-glance07:27
iwaoHi, Could I ask about implementation of member-create method ?07:29
*** jistr has joined #openstack-glance07:31
*** openstackgerrit has quit IRC07:35
*** openstackgerrit has joined #openstack-glance07:35
openstackgerritRajesh Tailor proposed openstack/glance: Raise exception if store location URL not found  https://review.openstack.org/13751507:43
*** iwao has quit IRC07:47
*** markus_z has joined #openstack-glance08:03
*** flwang has quit IRC08:11
*** sgotliv has joined #openstack-glance08:15
*** sgotliv has quit IRC08:20
*** haomaiwang has quit IRC08:32
*** sgotliv has joined #openstack-glance08:33
*** haomaiwang has joined #openstack-glance08:37
*** nellysmitt has joined #openstack-glance09:06
*** rajesht has joined #openstack-glance09:10
ativelkovmiguelgrinberg: hi, are you around?09:19
*** aix has joined #openstack-glance09:19
*** pdb has quit IRC09:20
*** pdb has joined #openstack-glance09:20
*** david-lyle_afk has quit IRC09:30
*** david-lyle_afk has joined #openstack-glance09:30
*** jistr has quit IRC09:34
*** smc7 has quit IRC09:34
*** jistr has joined #openstack-glance09:46
*** vilobhmm has quit IRC09:48
*** ericpeterson has joined #openstack-glance09:50
openstackgerritAlexander Tivelkov proposed openstack/glance: Declarative definitions of Artifact Types  https://review.openstack.org/11917409:50
openstackgerritAlexander Tivelkov proposed openstack/glance: A mixin for jsonpatch requests validation  https://review.openstack.org/14858809:50
openstackgerritAlexander Tivelkov proposed openstack/glance: REST API layer for Artifact Repository  https://review.openstack.org/13662909:50
openstackgerritAlexander Tivelkov proposed openstack/glance: Database layer for Artifact Repository  https://review.openstack.org/11599809:50
openstackgerritAlexander Tivelkov proposed openstack/glance: Domain layer for Artifact Repository  https://review.openstack.org/13289809:50
openstackgerritAlexander Tivelkov proposed openstack/glance: Artifact Plugins Loader  https://review.openstack.org/13430009:50
ativelkovfixed a merge conflict, so had to rebase09:50
openstackgerritLakshmi N Sampath proposed openstack/glance: Catalog Index Service  https://review.openstack.org/13826909:55
*** ericpeterson has quit IRC10:04
*** sgotliv has quit IRC10:07
*** bkopilov has quit IRC10:15
*** ParsectiX has left #openstack-glance10:15
*** ParsectiX has joined #openstack-glance10:15
*** chlong has joined #openstack-glance10:18
*** sgotliv has joined #openstack-glance10:20
junhonglflwang and nikhil_k, i just rebased https://review.openstack.org/#/c/153169/5 for the VMware store, template NotFound bug, it seems that the related modules are modified quite frequently, so i appreciate if you can help to review and approve the patch. thank you guys10:29
*** aix has quit IRC10:32
*** buzztroll has quit IRC10:33
*** buzztroll has joined #openstack-glance10:33
*** tshefi has joined #openstack-glance10:35
*** annashen has joined #openstack-glance10:36
*** buzztroll has quit IRC10:38
*** pkoniszewski has joined #openstack-glance10:39
openstackgerritLakshmi N Sampath proposed openstack/glance: Catalog Index Service - Index Update  https://review.openstack.org/16162110:39
*** annashen has quit IRC10:40
*** nellysmitt has quit IRC10:41
*** MattMan has quit IRC10:46
*** chlong has quit IRC10:51
*** haomaiwang has quit IRC10:53
*** ericpeterson has joined #openstack-glance10:54
*** aix has joined #openstack-glance10:57
*** nellysmitt has joined #openstack-glance11:09
*** Krast has quit IRC11:09
*** ericpeterson has quit IRC11:10
*** Longgeek has quit IRC11:14
*** ericpeterson has joined #openstack-glance11:20
*** diegows has joined #openstack-glance11:20
*** stpierre has joined #openstack-glance11:23
*** bkopilov has joined #openstack-glance11:25
*** Longgeek has joined #openstack-glance11:36
openstackgerritLakshmi N Sampath proposed openstack/glance: Catalog Index Service  https://review.openstack.org/13826911:38
openstackgerritMerged openstack/glance_store: Use oslo_utils.units where appropriate  https://review.openstack.org/15734811:47
*** ericpeterson has quit IRC11:47
*** ParsectiX has left #openstack-glance11:49
*** ParsectiX has joined #openstack-glance11:50
openstackgerritMerged openstack/python-glanceclient: Remove duplicate 'a' in the help string of --os-image-url  https://review.openstack.org/14846711:50
openstackgerritMike Fedosin proposed openstack/glance_store: Fix timeout during upload from slow resource  https://review.openstack.org/16259912:13
*** nlevinki has joined #openstack-glance12:13
*** sgotliv has quit IRC12:16
openstackgerritLouis Taylor proposed openstack/glance: Move to graduated oslo.log module  https://review.openstack.org/14934412:28
*** sgotliv has joined #openstack-glance12:29
openstackgerritMichal Dulko proposed openstack/python-glanceclient: Add unit tests for log_curl_request  https://review.openstack.org/10546412:31
*** jistr has quit IRC12:47
*** rmartyn has joined #openstack-glance12:48
*** jistr has joined #openstack-glance12:48
rmartynMorning guys, anyone free for a quick query about ceph intergration with glance?12:48
*** nellysmitt has quit IRC12:49
flaper87kragniz: https://review.openstack.org/#/c/156249/ oks :)12:50
flaper87pls*12:50
*** annegentle has joined #openstack-glance12:51
flaper87sigmavirus24_awa: https://review.openstack.org/#/c/156249/ make good use of your "+2"12:52
flaper87and if possible, your "+A"12:52
flaper87:P12:52
*** ericpeterson has joined #openstack-glance12:54
abhishekkflaper87: could you please have a look at https://review.openstack.org/#/c/122181/, I have replied to your comment on PS #1013:01
abhishekkflaper87: Thank you13:01
*** cdelatte has joined #openstack-glance13:01
flaper87abhishekk: yes, sir!13:02
rmartynCeph COW anyone ? :D13:03
abhishekkflaper87: :)13:04
abhishekkrmartyn: what is your query?13:04
*** bkopilov has quit IRC13:04
*** zul has quit IRC13:04
*** ericpeterson has quit IRC13:04
*** jistr has quit IRC13:04
*** jistr has joined #openstack-glance13:05
rmartynI note that enabling the show_direct_url=true creates some security considerations, but there is not much information. Is there a way to enable this whilst keeping our 'backend location' secure? our ceph cluster is in no way 'public' should I be worried about enabling this functionality?13:07
*** smc7 has joined #openstack-glance13:07
rmartynthe ceph docs show as follows " Note that this exposes the back end location via Glance’s API, so the endpoint with this option enabled should not be publicly accessible"13:07
*** nlevinki has quit IRC13:08
rmartynappreciate any and all information!13:08
*** zul has joined #openstack-glance13:09
kragnizflaper87: some little nits :(13:09
flaper87kragniz: really, really? after so many +1's ?13:09
flaper87>.>13:09
kragniz;_;13:10
*** nidhi has joined #openstack-glance13:10
kragnizI'm sorry flaper8713:10
*** smc7 has quit IRC13:12
nidhihi! I'm new to glance. Is there a fairly simple bug I could try fixing to get started?13:12
*** ericpeterson has joined #openstack-glance13:13
flaper87also, we really need to auto-generate this f!@#@! sample config file13:14
flaper87:D13:15
* flaper87 random rant13:15
kragnizflaper87: soon!13:15
*** ericpeterson has quit IRC13:15
kragnizit's driving me crazy13:15
*** ericpeterson has joined #openstack-glance13:15
*** bkopilov has joined #openstack-glance13:18
kragniznidhi: I'm having a look, but I don't think there are any nice low-hanging-fruit bugs open at the moment13:19
flaper87kragniz: assertEqual('active', image.status) <- that's (expected, actual)13:20
*** peristeri has joined #openstack-glance13:20
nidhiOh okay, thanks anyway :)13:20
kragnizflaper87: durrr13:20
*** nlevinki has joined #openstack-glance13:21
kragnizflaper87: I'm very sorry :(13:21
openstackgerritFlavio Percoco proposed openstack/glance: Glance Image Introspection  https://review.openstack.org/15624913:22
openstackgerritFlavio Percoco proposed openstack/glance: Basic support for image conversion  https://review.openstack.org/15912913:22
flaper87kragniz: np, good catch about the missing config option in the conf file ^13:22
openstackgerritMerged openstack/python-glanceclient: Unit tests covering missing username or password  https://review.openstack.org/11689413:22
openstackgerritIan Cordasco proposed openstack/glance_store: Move from oslo.utils to oslo_utils (supplement)  https://review.openstack.org/16017813:22
*** rmartyn has left #openstack-glance13:25
*** sigmavirus24_awa is now known as sigmavirus2413:25
*** changbl has quit IRC13:26
*** rmartyn04 has joined #openstack-glance13:26
sigmavirus24flaper87: I'll give it a proper review first =P13:29
*** pkoniszewski has quit IRC13:31
rajeshtflaper87, sigmavirus24: please have a look at this patch, the dependent glance_store library version is released.13:31
rajeshtflaper87, sigmavirus24: https://review.openstack.org/13751513:31
rajeshtflaper87, sigmavirus24: thank you very much13:31
sigmavirus24rajesht: consider it queue'd13:32
rajeshtsigmavirus24: np, :)13:32
* sigmavirus24 wants to -2 one of flaper87's patches as a joke but I won't13:32
*** thangp has joined #openstack-glance13:33
kragniz"-2 idk"13:33
kragniz"-2 lol idk"13:34
openstackgerritInessa Vasilevskaya proposed openstack/glance: artifact property -> json schema proof of concept  https://review.openstack.org/13225313:48
nikhil_ksigmavirus24: kragniz mfedosin hemanthm : good morning!13:49
nikhil_kI've a kind request13:49
nikhil_kPlease keep the reviews to spec until Thursday as bugs can be dealt with later13:49
mfedosinokay13:50
nikhil_kSo, we can reduce the BP queue on https://launchpad.net/glance/+milestone/kilo-313:50
nikhil_k(side requests can wait)13:50
*** annegentle has quit IRC13:51
flaper87kragniz: sigmavirus24 mfedosin what nikhil_k is trying to say is that you should approve my patches13:51
flaper87:P13:51
sigmavirus24flaper87: lolol13:51
sigmavirus24flaper87: I have a -1.5 coming for you =P13:51
flaper87jokes apart, it'd be nice to get some reviews on the conversion one. I've some nits to fix there but I'll wait until it gets more reviews13:51
flaper87sigmavirus24: pls, tell me you're joking13:51
nikhil_kno, review them and put -2 on his patches :P13:52
nikhil_kjk13:52
sigmavirus24I has some nits13:52
rajesht:D13:52
* flaper87 is frustrated13:52
flaper87we can't iterate over nits forever :(13:52
nikhil_klol13:52
nikhil_kflaper87: btw, I've suggested to fix bugs after 12th13:52
nikhil_kso you can propose a new patch and say that it fixes those issues13:53
flaper87nikhil_k: awesome, I'll be tackling some13:53
flaper87LOL13:53
sigmavirus24nikhil_k: you're killing me =P13:53
sigmavirus24All my fun gone13:53
nikhil_kheh13:53
nikhil_ksigmavirus24: there's always scope for more , right?13:53
nikhil_kput one bugs in every patch review13:53
*** peristeri has quit IRC13:53
nikhil_kbug*13:53
sigmavirus24heh13:53
nikhil_kzhiyan: o/13:54
nikhil_kzhiyan: from your comment, we may need to release store before 12th https://review.openstack.org/#/c/161970/13:54
flaper87sigmavirus24: seriously, though. lets not go crazy with nits. I'm not saying this because this is my patch but because I've so much nitpicking lately (not from you, but since we're talking about this)13:54
nikhil_kas that's a spec that can't be merged later13:54
flaper87s/so/seen so/13:54
* flaper87 keeps eating words13:55
zhiyannikhil_k: hi. indeed13:55
flaper87(although I already had breakfast)13:55
sigmavirus24flaper87: really only one nit13:55
flaper87then don't :P13:55
nikhil_ksabari: hope you're around13:55
flaper87hahahaha13:55
sigmavirus24but other things are concerns I have for problems in the future13:55
nikhil_kzhiyan: ok, I will plan to release one today13:55
nikhil_kzhiyan: thanks!13:55
zhiyannikhil_k: thanks13:55
sigmavirus24nikhil_k: zhiyan about the version number, I didn't see any feature additions but I could be wrong13:55
sigmavirus24nikhil_k: in general, bug fixes ~> 0.2.1, any new features ~> 0.3.013:56
nikhil_ksigmavirus24: where?13:56
sigmavirus24in glance_store13:56
nikhil_kah13:56
sigmavirus24(for the next release of glance_store)13:56
sigmavirus24(on that card)13:56
sigmavirus24(it's too early)13:56
* sigmavirus24 needs more coffee13:56
flaper87sigmavirus24: click reply so I can address whatever you wrote there13:56
flaper87:D13:56
nikhil_kthis one https://review.openstack.org/#/c/161970/ ? BP https://blueprints.launchpad.net/openstack/?searchtext=vmware-store-multiple-datastores13:56
nikhil_ksigmavirus24: ^13:56
sigmavirus24flaper87: still reviewing. I'll note that they should be done in a follow-up13:56
sigmavirus24nikhil_k: oh that probably warrants an 0.3.0 since it's deprecating old behaviour13:57
nikhil_kflaper87: btw, I keep missing out on your reviews as the BP is not in the kilo-3 list13:57
* sigmavirus24 should have looked closer-er-er-er-er13:57
nikhil_ksigmavirus24: yeah :/13:57
flaper87nikhil_k: ooooooooh, isn't it?13:57
flaper87damnit13:57
flaper87:(13:57
nikhil_ksigmavirus24: np, happens while multi-tasking13:57
nikhil_kit's alllll goooood13:57
flaper87nikhil_k: can you schedule it? :D13:57
* flaper87 can do it13:57
kragnizflaper87: btw I think the rest of your patch is good to merge13:57
sigmavirus24yeah, reviewing flaper87's patches for -2's and IRC is hard =P13:57
nikhil_kflaper87: can you give me the link13:58
*** r-daneel has joined #openstack-glance13:58
nikhil_kelse I may need to click 5 times to find it :P13:58
*** rajesht has quit IRC13:58
*** MattMan has joined #openstack-glance13:58
flaper87nikhil_k: https://blueprints.launchpad.net/glance/+spec/new-upload-workflow https://blueprints.launchpad.net/glance/+spec/introspection-of-images https://blueprints.launchpad.net/glance/+spec/basic-import-conversion13:58
flaper87also, assign them to me13:59
nikhil_kflaper87: lol , this is abandoned https://review.openstack.org/#/c/137992/ :P13:59
flaper87nikhil_k: yeah, something weird happened with these specs13:59
kragnizwait, why did it get abandoned?13:59
flaper87The blueprints were approved, then specs required, then specs weren't required anymore because the bps were approved before the specs thing14:00
flaper87then Erno said we should have the specs14:00
sigmavirus24flaper87: you got your -2 =P14:00
flaper87sigmavirus24: volunteered to write them, we haven't had a chance to sync on that14:00
flaper87that said, I still think we don't need them for this thing14:00
sigmavirus24flaper87: I've been here =P14:00
sigmavirus24I've been waiting patiently ;P14:01
flaper87sigmavirus24: liar14:01
flaper87>.>14:01
sigmavirus24lol14:01
kragnizthat doesn't look like a -2, sigmavirus2414:01
sigmavirus24kragniz: math.abs(-2)14:01
sigmavirus24bam14:01
* kragniz would +2A but fears the wrath of nikhil_k 14:01
nikhil_kflaper87: put in a copy paste spec please14:01
sigmavirus24kragniz: +214:01
flaper87kragniz: go and +2 :)14:01
flaper87nikhil_k: will do14:02
nikhil_kwe need to find a way to do away with launchpad BPs14:02
nikhil_kand track everything on the specs somehow :/14:02
kragniznikhil_k: launchpad is going to be removed at some point14:03
kragniznikhil_k: it's all about storyboard now14:03
nikhil_k:)14:03
nikhil_kI'm waiting for that14:03
nikhil_kstoryboad does not quite help with release mgmt14:03
nikhil_kflaper87: I can assign you the upload workflow one given you promise to implement it for swift14:04
flaper87nikhil_k: wait, what? :D14:04
kragnizlol nikhil_k bringing in the -2 at last14:05
flaper87nikhil_k: so we'll make the spec a hard requirement for this work?14:05
* flaper87 disagrees14:06
nikhil_kflaper87: specs come in the release updates and this is big feature14:06
nikhil_kBPs are pointless except for tracking purposed (so both are neeeded now)14:06
flaper87right, I'm not saying we won't have it but it seems silly to block a patch that is ready to land on that14:06
nikhil_kpurposes14:06
flaper87A patch that implements a feature that has already been approved on LP14:07
flaper87and it'll likely have merge clonflicts by the time the spec lands14:07
nikhil_kflaper87: that's way just cope paste it14:07
nikhil_kcopy14:07
nikhil_kflaper87: well, we can make an exception in that case if someone is writing a spec14:07
flaper87sigmavirus24: ^14:08
*** annegentle has joined #openstack-glance14:08
flaper87you said you wanted to help with that spec, it should be enough to copy/paste what's in LP14:08
sigmavirus24flaper87: I still need information to write the spec with =P14:08
sigmavirus24link to bp?14:08
kragnizI agree with flaper87 here, originally there wasn't going to be a new spec at all14:08
flaper87sigmavirus24: https://blueprints.launchpad.net/openstack/?searchtext=introspection-of-images14:08
openstackgerritLakshmi N Sampath proposed openstack/glance: Catalog Index Service - Index Update  https://review.openstack.org/16162114:09
nikhil_kwhy are we not making doc changes?14:09
sigmavirus24just for funsies, I'll agree with nikhil_k because specs are helpful for people like me14:09
kragnizhaving a book keeping spec is kind of different, since the design of the patch isn't going to change based on that new spec14:09
nikhil_kyeah, this is a heavy weigted feature on operators and specs is the go to place14:09
kragnizsigmavirus24: in this case the spec is being written based on the implementation, though14:10
kragnizI agree there should be a spec, but I don't think it's a hard requirement to merge this14:10
nikhil_kthat's true for most of the specs14:11
abhishekkhi, in between what is FFE date?14:11
nikhil_k12th14:11
*** annegentle has quit IRC14:11
nikhil_kwe don't follow the first spec approval then code and then docs14:11
kragniznikhil_k: I guess you're right about that :(14:11
abhishekknikhil_k: thanks, so after 12th do I need to raise FFE?14:11
nikhil_kit's all hand in hand because there can be conflicts at any point of time and we don't want to go oops we missed in spec so we can't do anything about it now...14:12
flaper87nikhil_k: I'll have to go afk in a bit, will we make an exception for this patch?14:12
flaper87sigmavirus24: I can help you with the conversion spec14:12
flaper87sigmavirus24: if you get the introspection one up, that'd be very helpful14:12
*** ericpeterson has quit IRC14:12
nikhil_kif someone is willing to do it, surely flaper8714:12
flaper87nikhil_k: sigmavirus24 said he's going to14:12
flaper87sigmavirus24: confirm ^ ?14:13
flaper87nikhil_k: thanks for understanding14:13
nikhil_kabhishekk: I think FFE for anything besides CIS, Artifacts might be difficult. Deactivate has already proposed one but that may or may not happen14:13
flaper87nikhil_k: btw, just to make sure we're on the same page, we'll work on getting the conversion patch in for Kilo 2, right? The patch is up, the old bp was approved. I'll work on that spec and address the nits.14:14
*** smc7 has joined #openstack-glance14:14
flaper87I'll likely ask for FFE14:14
flaper87since I don't think it'll make it before that14:14
flaper87but I want to make sure this is something we want (since that's what we said 2 weeks ago :P)14:14
* flaper87 wants it for sure14:14
nikhil_kkilo-3*14:14
nikhil_kSo, since we did not get enough reviews on Artifacts and CIS until now they are likely candidates for FFE14:15
nikhil_kif I'm asked to pick, it would be a really hard choise otherwise14:15
abhishekknikhil_k: could you also add zero config reload for FFE if possible?14:15
nikhil_kabhishekk: what's blocking it now?14:16
sigmavirus24Yep14:16
abhishekknikhil_k: nothing, I need reviews14:16
*** cpallares has joined #openstack-glance14:16
nikhil_kabhishekk: I think we'd be able to get it merge by today/tomorrow, no?14:16
openstackgerritAlexander Tivelkov proposed openstack/glance: REST API layer for Artifact Repository  https://review.openstack.org/13662914:17
abhishekknikhil_k:most likely14:17
nikhil_kabhishekk: do you know who the active cores are?14:17
abhishekknikhil_k: :D14:18
nikhil_kabhishekk: the issue is that until now people kind left it upto me to +2A and I can't review all the patches14:20
abhishekknikhil_k: I can understand,14:20
nikhil_kabhishekk: just to clarify a bit more, giving a +2 is harder and sometimes risky14:21
*** nidhi has quit IRC14:21
abhishekknikhil_k: yes I know14:22
nikhil_kabhishekk: so, it's better to have varied opinions on bigger/drastic patches before +214:22
nikhil_kthat's the main reason for the delay on your review14:22
nikhil_k(people are either distracted or hesitent to have +1ed that after reviewing it before)14:23
abhishekknikhil_k: I know14:24
nikhil_k:)14:24
abhishekknikhil_k: I will wait :)14:24
abhishekknikhil_k: thank you14:25
*** ericpeterson has joined #openstack-glance14:26
nikhil_kthanks for understanding14:26
*** vijendar has joined #openstack-glance14:27
*** abhishekk has quit IRC14:29
*** Guest75125 has joined #openstack-glance14:30
*** pkoniszewski has joined #openstack-glance14:33
*** esheffield has quit IRC14:33
*** zz_jgrimm is now known as jgrimm14:35
flaper87sigmavirus24: hey, sorry for so much ping/bothering. Just to make sure I'm not missing anything and we're on the same page. You're working on the introspection spec, right?14:37
openstackgerritIan Cordasco proposed openstack/glance-specs: Add spec for Image Introspection blueprint  https://review.openstack.org/16302314:37
flaper87sigmavirus24: ^ LOL14:37
*** ericpeterson has quit IRC14:37
flaper87nikhil_k: ^14:37
kragnizlol14:37
sigmavirus24BOOM14:37
flaper87sigmavirus24: HAHAHAH, literally14:37
sigmavirus24It's skinny but hopefully helpful14:37
kragnizbest timing I've seen for a patch14:38
*** esheffield has joined #openstack-glance14:39
nikhil_ksigmavirus24 == TGV14:39
flaper87kragniz: nikhil_k sigmavirus24 https://twitter.com/flaper87/status/57530496965976883214:39
nikhil_knice!14:40
flaper87some things have to be captured forever14:40
kragnizflaper87: my client put it on the same second: http://i.imgur.com/kppTQ4V.png14:42
*** Guest75125 is now known as notsogentle14:43
*** notsogentle is now known as agentle_14:43
*** ericpeterson has joined #openstack-glance14:43
*** rwsu has joined #openstack-glance14:44
sigmavirus24TGV?14:44
sigmavirus24kragniz: mine too14:44
nikhil_khttp://www.sncf.com/en/trains/tgv14:44
nikhil_ksigmavirus24: ^14:44
nikhil_kflaper87: sigmavirus24 kragniz reviewing introspection code14:45
sigmavirus24oooh14:46
sigmavirus24nikhil_k: americans don't have high speed rail so all references to good trains are lost on us14:46
sigmavirus24=P14:46
nikhil_ksigmavirus24: heh, took a rare chance14:47
nikhil_kprolly should have referred to Bugati veron14:48
nikhil_kflaper87: oh, I found something that might be breaking some things14:49
nikhil_khttps://review.openstack.org/#/c/156249/24/glance/api/middleware/context.py14:49
sigmavirus24kragniz: flaper87 http://i.imgur.com/IuD3Hbc.png (same second proof)14:49
nikhil_kwhy do we need change to unauth middleware?14:49
flaper87nikhil_k: why is that breaking things?14:50
flaper87it's really not changing the default behavior14:50
nikhil_kI think it breaks pipeline which has unauth setup14:50
nikhil_kand then needs those kwargs14:50
flaper87it just allows me to pass a tenant for tasks with the unauth middleware14:50
nikhil_kwhat if it's not there14:51
flaper87mmh, but `.get()` returns None if the key doesn't exist14:51
nikhil_kstill seems weird14:51
flaper87why?14:51
nikhil_klemme double check14:51
nikhil_kso one scenario14:51
* sigmavirus24 had the same concerns as nikhil_k lol14:52
nikhil_kregistry is setup with unauth middleware14:52
*** TravT has quit IRC14:52
nikhil_kthen you don't expect tenant ID stuff there14:52
flaper87FWIW, I think NOONE should be using the unauth middleware in production14:52
flaper87if we don't do that, we need to allow task creation without tenant14:52
flaper87otherwise, you can't create tasks with the unaout middleware14:53
nikhil_kwell, that's not our call . it exists and with real world use cases14:53
nikhil_khmm14:53
flaper87it's not our call but we should advice people properly14:53
*** jaypipes has quit IRC14:53
flaper87and if people are using in production something we advice shouldn't be used in production, then that's not our problem14:53
nikhil_kflaper87:  there is a really good use case for having unauth middleware for long running upload processes14:54
nikhil_kso, that advice won't work on them14:54
flaper87again, not our problem, but that's a different discussion. The long uploads should use longer tokens or even better, we should fix the client library14:55
flaper87really, I don't think this is bad, I can revert it but again, it makes tasks unusale w/ the unaouth middleware14:56
flaper87And I found this out when I actually tried to, while testing the task workflow14:56
nikhil_kflaper87: token issue is not that easy, and you can't refetch token from Nova14:57
nikhil_kI'm looking further14:57
flaper87I know it's not simple but that's the right fix14:57
flaper87and we should really fix our client14:57
flaper87well14:58
flaper87to be fair14:58
openstackgerritAlexander Tivelkov proposed openstack/glance-specs: Artifact Repository  https://review.openstack.org/10096814:58
flaper87it's not just glanceclient14:58
openstackgerritMerged openstack/glance-specs: Add spec for Image Introspection blueprint  https://review.openstack.org/16302314:58
openstackgerritLouis Taylor proposed openstack/glance: Move to graduated oslo.log module  https://review.openstack.org/14934414:58
*** ParsectiX has quit IRC15:00
*** cpallares has quit IRC15:00
nikhil_khow can fixing the client help with token issue? Also, longer token do not necessarily help with uploads, snapshots can be taken at any point during the 24hr window of regular token (for eg. 23:59) and fail. Morever, tokens that big are pointless they need to have shorter window (different problem)15:00
ndonegankragniz: Enough changes in one commit? :)15:00
ativelkovFolks, I've addressed some of miguelgrinberg's comments to the artifact spec (mostly related to type version in URI string), mfedosin will add more on sorting/filtering (we do already implement the "modern" approach, it just has to be documented in spec)15:01
flaper87ndonegan: because the client does not renew the token when it expires and the upload has not been completed15:01
flaper87erm nikhil_k ^15:01
ativelkovIt would be great if we merge https://review.openstack.org/#/c/100968/ today15:01
flaper87there's a long thread on this token thing from last year, I think15:01
flaper87mmh, I should re-read it15:02
nikhil_kyep, so for such issues no-auth is right approach15:02
nikhil_kglance is not always public facing deployment so, this is a real world thing15:02
flaper87I still think no-auth shouldn't be used in production and even less for the registry service. I may be wrong about the glance-api side15:03
flaper87nikhil_k: right, that's a valid point15:03
nikhil_kflaper87: https://review.openstack.org/#/c/156249/24/glance/common/scripts/utils.py15:03
nikhil_kare we adding back file:// imports?15:03
nikhil_ksorry, if I misunderstood there15:04
*** agentle_ has quit IRC15:05
flaper87nikhil_k: nope, that's  just to get the data iterator. The real check is done in another function before getting to that point15:05
nikhil_kflaper87: ok, so just a diskIO vs in memory conversion?15:06
flaper87nikhil_k: mmh, nope, just a single way to get a data iterator (IO yes) for an URL (http or file)15:07
flaper87but it won't read it into memory, the caller of that function will15:07
nikhil_kflaper87: yeah, so it will cache it on disk15:08
*** vilobhmm has joined #openstack-glance15:08
*** TravT has joined #openstack-glance15:10
*** TravT has quit IRC15:11
*** TravT has joined #openstack-glance15:12
*** TravT has quit IRC15:12
*** browne has joined #openstack-glance15:12
*** ativelkov has quit IRC15:12
*** tsufiev has quit IRC15:12
*** vilobhmm has left #openstack-glance15:12
*** TravT has joined #openstack-glance15:12
*** tsufiev has joined #openstack-glance15:13
*** tsufiev has quit IRC15:14
*** ativelkov has joined #openstack-glance15:14
*** ativelkov_ has joined #openstack-glance15:15
*** ativelkov_ has quit IRC15:15
*** ativelkov_ has joined #openstack-glance15:18
flaper87nikhil_k: https://review.openstack.org/#/c/156249/24/glance/common/scripts/utils.py,cm <- re this, it's noted in the `work_dir` config option quite extensively.15:18
*** krykowski has joined #openstack-glance15:18
flaper87I'll revert the unauth context thing but again, that just makes tasks unusable with unauth15:18
kragnizndonegan: just about!15:18
flaper87hence, it makes it really hard to tes this thing without setting up keystone and a whole bunch of other things15:19
*** boris-42 has quit IRC15:22
*** ativelkov has quit IRC15:22
*** ativelkov_ is now known as ativelkov15:22
nikhil_kflaper87: if it's just for testing , we can add it later right?15:22
*** david-lyle_afk is now known as david-lyle15:23
nikhil_kI think we can just go with old setting for now15:23
*** tsufiev_ has joined #openstack-glance15:24
openstackgerritFlavio Percoco proposed openstack/glance: Glance Image Introspection  https://review.openstack.org/15624915:29
openstackgerritFlavio Percoco proposed openstack/glance: Basic support for image conversion  https://review.openstack.org/15912915:29
*** nlevinki has quit IRC15:29
openstackgerritFlavio Percoco proposed openstack/glance: Glance Image Introspection  https://review.openstack.org/15624915:31
openstackgerritFlavio Percoco proposed openstack/glance: Basic support for image conversion  https://review.openstack.org/15912915:31
*** ericpeterson has quit IRC15:35
*** changbl has joined #openstack-glance15:35
*** delattec has joined #openstack-glance15:37
*** cdelatte has quit IRC15:39
*** boris-42 has joined #openstack-glance15:40
*** annashen has joined #openstack-glance15:40
*** nlevinki has joined #openstack-glance15:44
*** annashen has quit IRC15:44
*** changbl has quit IRC15:47
*** geguileo has joined #openstack-glance15:48
nikhil_kflaper87: sigmavirus24 : are we not having doc changes in the introspection patch?15:49
sigmavirus24nikhil_k: good point. The docs are frozen this week, right?15:49
nikhil_ksigmavirus24: hm, I didn't think so15:50
nikhil_ksigmavirus24: is that a trnding topic?15:50
nikhil_ksigmavirus24: the doc freeze date is 19th as per the schedule on the wiki15:51
nikhil_k(called string freeze)15:51
sigmavirus24nikhil_k: I think I got the dates backwards. What freeze is on the 12th?15:51
sigmavirus24Or is there no freeze on the 12th?15:51
nikhil_ksigmavirus24: feature freze is 12th (sorry getting a lot og jitter due to vidyo)15:52
nikhil_kafter 12th we are accepting only FFE and bugs reviews15:52
nikhil_kso, doc changes if missed can be adjusted until 19th :)15:53
sigmavirus24nikhil_k: ah okay15:54
sigmavirus24Either I misunderstood during the last meeting or someone swapped them15:54
nikhil_kheh15:54
*** pkoniszewski has quit IRC15:57
*** rmartyn04 has quit IRC16:00
*** bkopilov has quit IRC16:02
*** changbl has joined #openstack-glance16:03
*** nlevinki has quit IRC16:03
openstackgerritInessa Vasilevskaya proposed openstack/glance: REST API layer for Artifact Repository  https://review.openstack.org/13662916:05
*** browne has quit IRC16:08
*** agentle_ has joined #openstack-glance16:10
*** delatte has joined #openstack-glance16:10
*** delattec has quit IRC16:13
*** changbl has quit IRC16:13
*** ericpeterson has joined #openstack-glance16:17
openstackgerritOlena Logvinova proposed openstack/glance: Glance basic architecture section  https://review.openstack.org/16044116:35
*** delattec has joined #openstack-glance16:35
*** delatte has quit IRC16:38
*** jaypipes has joined #openstack-glance16:40
*** harlowja_away is now known as harlowja_16:44
*** browne has joined #openstack-glance16:50
*** delatte has joined #openstack-glance16:51
*** delattec has quit IRC16:51
*** buzztroll has joined #openstack-glance16:53
*** agentle_ has quit IRC16:58
*** jistr has quit IRC17:01
*** changbl has joined #openstack-glance17:06
*** Longgeek has quit IRC17:06
*** jaypipes has quit IRC17:07
*** annashen has joined #openstack-glance17:09
*** ericpeterson has quit IRC17:11
*** krtaylor has quit IRC17:14
*** ericpeterson has joined #openstack-glance17:15
*** ericpeterson has quit IRC17:16
*** ericpeterson has joined #openstack-glance17:17
*** ericpeterson has quit IRC17:19
*** changbl has quit IRC17:21
*** ericpeterson has joined #openstack-glance17:23
*** changbl has joined #openstack-glance17:24
*** krtaylor has joined #openstack-glance17:27
*** changbl has quit IRC17:33
sabarinikhil_k: I am around now.17:38
sabaritrying to look up the context.17:38
nikhil_ksabari: we need a new glance_store release to enable your bp17:40
nikhil_ksabari: can you try to clearn things around it like creating global req patch and get it reviewed?17:41
*** changbl has joined #openstack-glance17:41
sabarinikhil_k: sure, I can do that.17:42
sabariwhen are we tagging the new version and uploading to pypi ?17:42
nikhil_ksabari: I will do that in a bit and let you know17:43
sabarigreat, thanks!17:43
openstackgerritLouis Taylor proposed openstack/glance: Avoid usability regression when generating config  https://review.openstack.org/16310617:50
kragnizflaper87: ^17:50
*** peristeri has joined #openstack-glance17:53
*** markus_z has quit IRC17:56
*** jdurgin has joined #openstack-glance17:56
*** krykowski has quit IRC18:00
*** jaypipes has joined #openstack-glance18:04
*** bkopilov has joined #openstack-glance18:05
*** changbl has quit IRC18:05
nikhil_ksabari: just send you an email18:05
nikhil_ksent*18:05
sabarinikhil_k: thanks, let me take a look.18:06
*** sgotliv has quit IRC18:06
*** tsufiev_ has quit IRC18:07
*** ativelkov has quit IRC18:07
*** ativelkov has joined #openstack-glance18:08
*** ativelkov has quit IRC18:08
*** ativelkov has joined #openstack-glance18:08
openstackgerritLakshmi N Sampath proposed openstack/glance: Catalog Index Service  https://review.openstack.org/13826918:10
*** ativelkov_ has joined #openstack-glance18:10
*** ericpeterson has quit IRC18:10
*** ativelkov_ has quit IRC18:10
*** ericpeterson has joined #openstack-glance18:12
*** ericpeterson has quit IRC18:12
*** ativelkov_ has joined #openstack-glance18:13
*** MattMan has quit IRC18:13
*** peristeri has quit IRC18:14
*** tsufiev_ has joined #openstack-glance18:16
*** ericpeterson has joined #openstack-glance18:16
*** browne has left #openstack-glance18:16
openstackgerritMike Fedosin proposed openstack/glance-specs: Artifact Repository  https://review.openstack.org/10096818:17
*** ericpeterson has quit IRC18:23
*** takedakn has joined #openstack-glance18:26
sabarinikhil_k: Added you on https://review.openstack.org/#/c/163121/18:26
*** takedakn has quit IRC18:27
*** smc7 has quit IRC18:31
*** annashen has quit IRC18:34
ativelkovnikhil_k: any chances to land the artifact spec today?18:37
nikhil_kativelkov: will try18:38
mfedosinnikhil_k, awesome! and what about sorting? :)18:39
nikhil_ksometime on/before thursday18:39
ativelkovWe've addressed most of miguelgrinberg's concerns. The rest (mostly related to actions) will need further discussions later. Having the API as experimental leaves us the room for that18:40
*** ericpeterson has joined #openstack-glance18:41
nikhil_ksounds good18:41
nikhil_kI've just been having hard time to find enough contiguous cycles to review all of these major ones18:41
nikhil_khaving more core reviewers actually reviewing specs related code helps18:42
*** changbl has joined #openstack-glance18:44
*** agentle_ has joined #openstack-glance18:50
openstackgerritLakshmi N Sampath proposed openstack/glance: Catalog Index Service - Index Update  https://review.openstack.org/16162118:51
mfedosinnikhil_k, we changed the doc based on Fei and Zhi suggestions https://review.openstack.org/#/c/160441/ Could you remove -2 now? :)18:52
*** agentle_ has quit IRC18:55
*** agentle_ has joined #openstack-glance18:55
*** sgotliv has joined #openstack-glance19:00
*** ericpeterson has quit IRC19:04
*** ericpeterson has joined #openstack-glance19:05
*** buzztroll has quit IRC19:05
*** ativelkov has quit IRC19:09
*** jaypipes is now known as jay-afk19:09
*** jay-afk is now known as jaypipes-afk19:10
*** smc7 has joined #openstack-glance19:12
*** ericpeterson has quit IRC19:23
*** ericpeterson has joined #openstack-glance19:24
*** annashen has joined #openstack-glance19:27
*** ericpeterson has quit IRC19:28
*** ericpeterson has joined #openstack-glance19:28
*** buzztroll has joined #openstack-glance19:29
*** ericpeterson has quit IRC19:36
*** vishy has joined #openstack-glance19:38
*** ericpeterson has joined #openstack-glance19:39
*** ericpete_ has joined #openstack-glance19:41
*** ericpeterson has quit IRC19:41
*** ericpete_ has quit IRC19:46
*** ericpeterson has joined #openstack-glance19:47
*** ericpeterson has quit IRC19:47
*** ericpeterson has joined #openstack-glance19:48
*** gokrokve has joined #openstack-glance19:51
*** ericpeterson has quit IRC19:51
*** ericpeterson has joined #openstack-glance19:51
openstackgerritIan Cordasco proposed openstack/glance: Use graduated oslo.policy  https://review.openstack.org/16236819:56
*** aix has quit IRC19:57
openstackgerritSabari proposed openstack/glance: Update config and docs for multiple datastores support  https://review.openstack.org/16197020:04
*** groen692 has quit IRC20:11
openstackgerritSteve Lewis proposed openstack/python-glanceclient: Override patch for 'tags' in v2.images.update method  https://review.openstack.org/8705020:12
*** mfedosin_ has joined #openstack-glance20:14
*** annashen has quit IRC20:15
*** takedakn has joined #openstack-glance20:16
*** buzztroll has quit IRC20:18
*** delatte has quit IRC20:19
sigmavirus24stevelle: https://review.openstack.org/#/c/87050/20:28
*** ericpeterson has quit IRC20:34
*** annashen has joined #openstack-glance20:34
*** ericpeterson has joined #openstack-glance20:35
*** ericpeterson has quit IRC20:36
*** cdelatte has joined #openstack-glance20:41
*** boris-42 has quit IRC20:42
*** tshefi has quit IRC20:42
*** pkoniszewski has joined #openstack-glance20:44
*** TravT has quit IRC20:47
*** buzztroll has joined #openstack-glance20:51
*** pkoniszewski has quit IRC20:51
*** ericpeterson has joined #openstack-glance20:57
*** TravT has joined #openstack-glance20:57
*** changbl has quit IRC20:58
*** ericpeterson has quit IRC21:05
*** ericpeterson has joined #openstack-glance21:11
*** boris-42 has joined #openstack-glance21:22
*** ericpeterson has quit IRC21:28
*** vijendar has quit IRC21:37
*** stpierre has quit IRC21:40
*** chlong has joined #openstack-glance21:40
*** flwang has joined #openstack-glance21:42
*** flwang has quit IRC21:44
*** flwang has joined #openstack-glance21:44
*** thangp has quit IRC21:52
*** buzztrol_ has joined #openstack-glance21:53
stevellesigmavirus24: thx for the review, I'll go back and push on this bit further.21:54
*** buzztroll has quit IRC21:54
sigmavirus24stevelle: I hope my dissertation helps21:58
sigmavirus24=P21:58
*** jaypipes-afk is now known as JAYPIPES21:59
*** JAYPIPES is now known as jaypipes21:59
openstackgerritIan Cordasco proposed openstack/glance: Pass a real image target to the policy enforcer  https://review.openstack.org/14665121:59
stevelleI'm sure it will.21:59
*** takedakn has quit IRC22:00
*** sigmavirus24 is now known as sigmavirus24_awa22:00
*** takedakn has joined #openstack-glance22:00
*** takedakn has quit IRC22:02
*** smc7_ has joined #openstack-glance22:03
*** smc7 has quit IRC22:05
*** chlong has quit IRC22:07
*** agentle_ has quit IRC22:07
*** smc7_ has quit IRC22:13
*** ativelkov_ has quit IRC22:14
*** tsufiev_ has quit IRC22:14
*** spzala has quit IRC22:19
*** annashen has quit IRC22:20
*** ativelkov has joined #openstack-glance22:24
*** ativelkov has quit IRC22:25
*** agentle_ has joined #openstack-glance22:25
*** ativelkov has joined #openstack-glance22:30
*** tsufiev_ has joined #openstack-glance22:36
*** flwang has quit IRC22:37
*** flwang has joined #openstack-glance22:38
*** pkoniszewski has joined #openstack-glance22:39
*** buzztrol_ has quit IRC22:40
*** buzztroll has joined #openstack-glance22:40
*** flwang has quit IRC22:46
*** pkoniszewski has quit IRC22:46
*** agentle_ has quit IRC22:47
*** flwang has joined #openstack-glance22:54
*** smc7 has joined #openstack-glance23:11
*** smc7 has quit IRC23:12
*** annashen has joined #openstack-glance23:13
*** tsufiev_ has quit IRC23:16
*** ativelkov has quit IRC23:16
*** tsufiev_ has joined #openstack-glance23:21
*** ativelkov has joined #openstack-glance23:21
*** ativelkov has quit IRC23:27
*** tsufiev_ has quit IRC23:27
*** david-lyle is now known as david-lyle_afk23:33
*** ativelkov has joined #openstack-glance23:35
*** ativelkov has quit IRC23:36
*** ativelkov has joined #openstack-glance23:41
*** ativelkov has quit IRC23:42
*** tsufiev_ has joined #openstack-glance23:46
*** ativelkov_ has joined #openstack-glance23:47
*** mfedosin_ has quit IRC23:51
*** chlong has joined #openstack-glance23:54
*** zul has quit IRC23:59
« Monday, 2015-03-09 Index Wednesday, 2015-03-11 »

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