Wednesday, 2015-02-04

*** rbak__ has quit IRC00:05
*** alexpilotti has quit IRC00:10
*** thomasem has quit IRC00:10
*** claudiub has quit IRC00:14
*** gzhai1 has quit IRC00:23
*** exploreshaifali has quit IRC00:28
*** theanalyst has quit IRC00:36
*** theanalyst has joined #openstack-ceilometer00:40
*** claudiub has joined #openstack-ceilometer00:43
*** shengjiemin has joined #openstack-ceilometer00:44
*** ddieterly has joined #openstack-ceilometer00:55
*** david-lyle is now known as david-lyle_afk00:57
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259201:04
*** promulo__ has joined #openstack-ceilometer01:09
*** nellysmitt has joined #openstack-ceilometer01:11
*** promulo has quit IRC01:13
*** nellysmitt has quit IRC01:16
*** _cjones_ has quit IRC01:29
*** _cjones_ has joined #openstack-ceilometer01:29
*** _cjones_ has quit IRC01:34
*** jmanko has joined #openstack-ceilometer01:38
openstackgerritmizeng proposed openstack/ceilometer: Shuffle agents to send request  https://review.openstack.org/14840401:40
*** jmankov has quit IRC01:42
*** jmanko has quit IRC01:42
*** jmanko has joined #openstack-ceilometer01:42
*** liuh has joined #openstack-ceilometer01:47
*** theanalyst has quit IRC01:48
*** theanalyst has joined #openstack-ceilometer01:51
*** zzz_jasonamyers is now known as jasonamyers01:59
*** jasonamyers is now known as zzz_jasonamyers02:01
*** fnaval has joined #openstack-ceilometer02:02
*** rwsu is now known as rwsu-afk02:11
openstackgerritZhiQiang Fan proposed openstack/ceilometer: use debug value for pecan_debug default  https://review.openstack.org/15207802:25
*** zzz_jasonamyers is now known as jasonamyers02:25
*** jasonamyers is now known as zzz_jasonamyers02:31
*** zzz_jasonamyers is now known as jasonamyers02:38
openstackgerritgordon chung proposed openstack/ceilometer: add elasticsearch events db  https://review.openstack.org/13713102:41
*** jasonamyers is now known as zzz_jasonamyers02:45
*** theanalyst has quit IRC02:53
*** theanalyst has joined #openstack-ceilometer02:56
*** claudiub has quit IRC03:00
*** yeungp has quit IRC03:03
*** nellysmitt has joined #openstack-ceilometer03:12
openstackgerritZhiQiang Fan proposed openstack/ceilometer: split api.controllers.v2  https://review.openstack.org/14632903:12
*** nellysmitt has quit IRC03:16
*** gzhai has joined #openstack-ceilometer03:24
*** ajc_ has joined #openstack-ceilometer03:25
*** harlowja is now known as harlowja_away03:28
*** Ephur has quit IRC03:30
*** deepthi has joined #openstack-ceilometer03:39
*** jmankov has joined #openstack-ceilometer03:41
*** jmank has joined #openstack-ceilometer03:42
*** zzz_jasonamyers is now known as jasonamyers03:43
*** jmanko has quit IRC03:44
*** jmankov has quit IRC03:45
*** jasonamyers is now known as zzz_jasonamyers03:47
*** mitz_ has quit IRC03:56
*** mitz has joined #openstack-ceilometer03:56
openstackgerritgordon chung proposed openstack/ceilometer: add elasticsearch events db  https://review.openstack.org/13713103:58
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259204:10
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259204:13
*** julim has quit IRC04:27
*** _cjones_ has joined #openstack-ceilometer04:30
openstackgerritZhiQiang Fan proposed openstack/ceilometer: split api.controllers.v2  https://review.openstack.org/14632904:32
*** _cjones_ has quit IRC04:32
*** _cjones_ has joined #openstack-ceilometer04:33
*** _cjones_ has quit IRC04:37
*** fnaval has quit IRC04:38
*** ddieterly has quit IRC04:46
*** changbl has joined #openstack-ceilometer04:57
*** yatin has joined #openstack-ceilometer04:59
*** david-lyle_afk has quit IRC05:11
*** david-lyle_afk has joined #openstack-ceilometer05:11
*** nellysmitt has joined #openstack-ceilometer05:13
*** nellysmitt has quit IRC05:17
*** harlowja_away has quit IRC05:19
*** ildikov has quit IRC05:51
*** j_t has joined #openstack-ceilometer05:53
j_thttps://github.com/paypal/cloudminion05:53
j_tI was looking at this project, we wanted a VM reaper.. to wipe out 'unused VMs', which can be an aribtrary definition.05:54
j_tThough I'd like whatever janitor process we have in our openstack system to be integrated with our openstack system.05:54
j_tIs ceilometer going to be involved at all with a janitor process for openstack?05:54
j_tIs there anywhere I can read about 'the plan' for janitor systems in openstack?05:55
j_tCan ceilometer be integrated with openstack-congress? and nodes get deleted through nova if they violate the congress policy06:07
j_tor what is a good janitor system to use in openstack06:07
openstackgerritOpenStack Proposal Bot proposed openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/15240206:10
*** _nadya_ has joined #openstack-ceilometer06:34
*** _nadya_ has quit IRC06:38
*** ddieterly has joined #openstack-ceilometer06:48
*** ddieterly has quit IRC06:52
openstackgerritZhiQiang Fan proposed openstack/ceilometer: enable oslo namespace check for ceilometer project  https://review.openstack.org/14643507:12
*** nellysmitt has joined #openstack-ceilometer07:13
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259207:17
*** nellysmitt has quit IRC07:18
*** shengjiemin has quit IRC07:32
*** amalagon has quit IRC07:36
*** shengjiemin has joined #openstack-ceilometer07:42
*** nellysmitt has joined #openstack-ceilometer07:42
*** liuh has quit IRC07:43
gzhaieglynn:hi07:47
*** ddieterly has joined #openstack-ceilometer07:49
*** shengjiemin has quit IRC07:50
*** IvanBerezovskiy has joined #openstack-ceilometer07:51
*** ddieterly has quit IRC07:53
gzhaieglynn-office:hi07:56
*** nsaje has quit IRC07:57
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259208:03
*** nsaje has joined #openstack-ceilometer08:04
*** ildikov has joined #openstack-ceilometer08:07
*** _nadya_ has joined #openstack-ceilometer08:12
*** nsaje_ has joined #openstack-ceilometer08:23
openstackgerritZhiQiang Fan proposed openstack/ceilometer: use oslo.log instead of oslo-incubator code  https://review.openstack.org/14781008:23
*** nsaje has quit IRC08:23
openstackgerritLena Novokshonova proposed openstack/ceilometer: [SQLalchemy] Add regex to complex queries  https://review.openstack.org/15282408:28
*** ddieterly has joined #openstack-ceilometer08:49
*** ddieterly has quit IRC08:54
openstackgerritZhai, Edwin proposed openstack/ceilometer: Remove no_resource hack for IPMI pollster  https://review.openstack.org/15159909:15
openstackgerritZhai, Edwin proposed openstack/ceilometer: Add dummy resource for IPMI pollsters  https://review.openstack.org/15159809:15
openstackgerritZhai, Edwin proposed openstack/ceilometer: Enabling self-disabled pollster  https://review.openstack.org/14630309:15
*** safchain has joined #openstack-ceilometer09:19
*** shengjiemin has joined #openstack-ceilometer09:20
*** _nadya_ has quit IRC09:23
*** shengjiemin has quit IRC09:36
*** boris-42 has joined #openstack-ceilometer09:39
*** shengjiemin has joined #openstack-ceilometer09:47
*** adelinatuvenie has joined #openstack-ceilometer09:49
*** llu has quit IRC09:50
*** llu has joined #openstack-ceilometer09:50
*** ddieterly has joined #openstack-ceilometer09:50
*** ddieterly has quit IRC09:54
*** amalagon has joined #openstack-ceilometer09:57
*** amalagon has quit IRC10:02
*** Ala has joined #openstack-ceilometer10:06
*** _nadya_ has joined #openstack-ceilometer10:07
*** shengjiemin has quit IRC10:22
*** cdent has joined #openstack-ceilometer10:26
*** alexpilotti has joined #openstack-ceilometer10:47
*** cdent has quit IRC10:49
*** ddieterly has joined #openstack-ceilometer10:51
*** Longgeek has joined #openstack-ceilometer10:55
*** ddieterly has quit IRC10:55
*** _nadya_ has quit IRC11:00
*** cdent has joined #openstack-ceilometer11:06
*** _nadya__ has joined #openstack-ceilometer11:11
*** ityaptin__ has quit IRC11:24
*** ityaptin__ has joined #openstack-ceilometer11:24
*** claudiub has joined #openstack-ceilometer11:24
*** ityaptin has joined #openstack-ceilometer11:25
*** _nadya__ has quit IRC11:27
*** Longgeek has quit IRC11:33
*** deepthi has quit IRC11:42
*** sbfox has joined #openstack-ceilometer11:59
*** david-lyle_afk is now known as david-lyle12:01
*** amalagon has joined #openstack-ceilometer12:10
*** _nadya_ has joined #openstack-ceilometer12:11
*** sbfox1 has joined #openstack-ceilometer12:11
*** gzhai1 has joined #openstack-ceilometer12:13
*** alexpilotti has quit IRC12:13
*** gibi_ has joined #openstack-ceilometer12:14
*** yatin_ has joined #openstack-ceilometer12:16
*** ilyashakhat_ has joined #openstack-ceilometer12:17
*** sbfox has quit IRC12:20
*** yatin has quit IRC12:20
*** gzhai has quit IRC12:20
*** zenpac has quit IRC12:20
*** gibi has quit IRC12:20
*** zul has quit IRC12:20
*** shadower has quit IRC12:20
*** ilyashakhat has quit IRC12:20
*** sbadia has quit IRC12:20
*** sbfox1 has quit IRC12:21
*** sbadia has joined #openstack-ceilometer12:23
*** claudiub has quit IRC12:25
*** EmilienM|afk is now known as EmilienM12:27
*** zenpac has joined #openstack-ceilometer12:28
*** zul has joined #openstack-ceilometer12:28
*** sbfox has joined #openstack-ceilometer12:30
gzhai1eglynn:hi12:34
*** david-lyle is now known as david-lyle_afk12:36
*** david-lyle_afk is now known as david-lyle12:36
*** david-lyle is now known as david-lyle_afk12:44
*** david-lyle_afk is now known as david-lyle12:45
* cdent shakes his head at pecan12:50
*** ddieterly has joined #openstack-ceilometer12:52
*** yatin_ has left #openstack-ceilometer12:53
*** _nadya_ has quit IRC12:53
*** _nadya_ has joined #openstack-ceilometer12:56
*** ddieterly has quit IRC12:56
*** claudiub has joined #openstack-ceilometer12:58
*** sbfox has quit IRC13:00
*** prad has joined #openstack-ceilometer13:19
*** ajc_ has quit IRC13:24
*** Ephur has joined #openstack-ceilometer13:28
*** prad has quit IRC13:31
*** ddieterly has joined #openstack-ceilometer13:53
*** thomasem has joined #openstack-ceilometer13:54
*** ddieterly has quit IRC13:57
IvanBerezovskiycdent, Hi :)13:58
IvanBerezovskiycdent, can we use smth like http://paste.openstack.org/show/166817/ in puppets?13:58
IvanBerezovskiycdent, I can't find better way to pass namespaces to ceilometer config, if some of namespaces are undef13:59
cdentI'm not sure, do you have a preference EmilienM ?13:59
IvanBerezovskiyEmilienM, ^13:59
eglynngzhai1: hey14:00
IvanBerezovskiyEmilienM, can you help me with my question above, please?14:00
EmilienMcdent: IvanBerezovskiy: /me looking14:01
EmilienMouch, this is ugly14:02
EmilienMbut it should work :)14:02
EmilienMlet me a bit of time to remind the context and maybe find something14:03
cdentIs there a more reasonable way?14:03
*** thomasem has quit IRC14:03
*** thomasem has joined #openstack-ceilometer14:04
*** thomasem has quit IRC14:05
IvanBerezovskiyEmilienM, thanks you. I tried to find better way, but I failed :(14:05
*** thomasem has joined #openstack-ceilometer14:05
openstackgerritLena Novokshonova proposed openstack/ceilometer: [SQLalchemy] Add regex to complex queries  https://review.openstack.org/15282414:09
*** prad has joined #openstack-ceilometer14:19
cdenteglynn: a) feeling any better b) Do you happen to know if the api-site wadl files are made with a tool (that inspects code) or by hand?14:19
*** julim has joined #openstack-ceilometer14:19
eglynncdent: still sick as a dog :(14:20
eglynncdent: ... do you mean the http://developer.openstack.org/api-ref-telemetry-v2.html etc.?14:21
cdentsorry to hear that, sorry for the dog too14:21
eglynnLOL :)14:21
cdentyeah, that stuff14:21
cdentis generated from this: https://github.com/openstack/api-site/blob/master/api-ref/src/wadls/telemetry-api/src/v2/os-telemetry-api-2.0.wadl14:21
*** julim has quit IRC14:21
cdentbut I'm wondering what generates that wadl14:21
cdent(or who)14:22
eglynncdent: TBH I don't know how that stuff is generated these days, whether staticly maintained by a human (I hope not!) or generated by tooling14:22
eglynncdent: ... annegentle obviously would know14:22
eglynncdent: ... or Andreas Jaager if she's not on-line yet14:22
cdentI'll see if I can locate her, thanks.14:22
eglynncdent: try #openstack-doc14:23
cdentthanks14:23
*** julim has joined #openstack-ceilometer14:24
openstackgerritLena Novokshonova proposed openstack/ceilometer: [SQLalchemy] Add regex to complex queries  https://review.openstack.org/15282414:26
*** gordc has joined #openstack-ceilometer14:29
*** shengjiemin has joined #openstack-ceilometer14:32
*** Longgeek has joined #openstack-ceilometer14:33
*** Longgeek has quit IRC14:35
claudiubDinaBelova: hello14:46
*** packet has joined #openstack-ceilometer14:47
*** ildikov has quit IRC14:48
*** zzz_jasonamyers is now known as jasonamyers14:51
*** ddieterly has joined #openstack-ceilometer14:54
*** ddieterly has quit IRC14:57
*** ddieterly has joined #openstack-ceilometer15:02
*** ddieterly has quit IRC15:02
*** ddieterly has joined #openstack-ceilometer15:05
*** ddieterly has quit IRC15:08
*** ddieterly has joined #openstack-ceilometer15:10
*** EmilienM has quit IRC15:14
*** julim has quit IRC15:15
*** ddieterly has quit IRC15:15
*** EmilienM has joined #openstack-ceilometer15:16
*** julim has joined #openstack-ceilometer15:18
*** ildikov has joined #openstack-ceilometer15:19
*** rbak has joined #openstack-ceilometer15:19
*** fnaval has joined #openstack-ceilometer15:22
*** _nadya_ has quit IRC15:23
*** _nadya_ has joined #openstack-ceilometer15:23
*** ddieterly has joined #openstack-ceilometer15:28
*** _nadya_ has quit IRC15:28
*** _nadya_ has joined #openstack-ceilometer15:29
*** ddieterly has quit IRC15:31
openstackgerritMehdi Abaakouk proposed stackforge/gnocchi: Add ceilometer alarm rule and evaluator  https://review.openstack.org/14918215:31
openstackgerritMerged openstack/ceilometermiddleware: add options to configure messaging  https://review.openstack.org/15227315:33
*** fnaval has quit IRC15:35
*** fnaval has joined #openstack-ceilometer15:35
*** EmilienM is now known as EmilienM|afk15:35
*** fabiog has joined #openstack-ceilometer15:38
*** kkxue has joined #openstack-ceilometer15:42
openstackgerritIlya Tyaptin proposed openstack/ceilometer: [MongoDB] Add regex to complex queries  https://review.openstack.org/15120815:43
*** exploreshaifali has joined #openstack-ceilometer15:57
*** EmilienM|afk is now known as EmilienM16:18
*** Ala has quit IRC16:20
*** IvanBerezovskiy has left #openstack-ceilometer16:22
fabiogidegtiarov: ping16:32
*** yatin has joined #openstack-ceilometer16:33
*** adelinatuvenie has quit IRC16:34
openstackgerritgordon chung proposed openstack/ceilometermiddleware: add README template  https://review.openstack.org/15297216:34
*** Ephur has quit IRC16:35
*** kkxue has quit IRC16:35
*** rwsu-afk is now known as rwsu16:40
*** lan_ has quit IRC16:40
*** Ala has joined #openstack-ceilometer16:41
*** lan has joined #openstack-ceilometer16:42
*** Ala has quit IRC16:52
gordcDinaBelova: sileht: _nadya_: ildikov: anyone want a free review? https://review.openstack.org/#/c/152972/ if not, i'm just going to +A myself assuming it passes so i can make a release.16:53
silehtgordc, huge change ;)16:53
fabioggordc: I gave the best I could ... +1 :-)16:53
gordclol thanks.16:54
*** shengjiemin has quit IRC16:55
fabiogDinaBelova: sileht: _nadya_: ildikov: gordc: eglynn: please review https://review.openstack.org/#/c/142592/16:57
fabiogall the pipeline related code is now inside the pipeline package16:58
eglynnfabiog: there's a lot of red in the CI report, do those failures look kosher?17:00
fabiogeglynn: rohit is looking at it right now, I am interested in see if there are major issues with the approach.17:01
eglynncool17:01
fabiogeglynn: we moved pipeline.py in the pipeline package because I think it makes sense now that we have a pipeline package, would you agree?17:01
gordcfabiog: is this related to this: https://review.openstack.org/#/c/128411/7/specs/kilo/notification_filter.rst?17:02
eglynnsure, that aspec seems reasonable17:02
eglynn*aspect17:03
fabioggordc: once we have a datastore for the pipeline and an API we can think about making the polling and notifications separate17:03
*** atan8 has joined #openstack-ceilometer17:04
fabioggordc: I don't think is good not to be able to avoid polling or notifications because some meters can be obtain in both ways17:04
gordcfabiog: good to hear... i was asking because the link for schema shows otherwise: http://www.gliffy.com/go/publish/681627517:04
fabioggordc: you will get duplicated data17:04
_nadya_fabiog: will do today17:05
gordcfabiog: not exactly duplicate... just extra data... did you consider the last comment i made regarding providing different pipeline files to notification and polling?17:06
gordcthe reason i suggested it was becasue when i first saw phil's schema it was really complex for something that can be solved today.17:06
openstackgerritRomain Soufflet proposed openstack/ceilometermiddleware: Add swift middleware entry point  https://review.openstack.org/15298417:07
fabioggordc: I didn't work on the spec lately because I am now trying to get the datastore in first17:08
fabioggordc: I would really like to split the pipeline into pipelines. We now have event, I would recommend to have polling and notification pipelines17:09
fabioggordc: and these can be stored in the same tables where the only difference is that sources have a polling interval or not17:10
gordcfabiog: yeah i get your point... it's just that after i saw phil's patch, it became more complex than i imagined... we can continue to discuss.17:12
*** _nadya_ has quit IRC17:13
*** safchain has quit IRC17:19
*** jasonamyers is now known as zzz_jasonamyers17:20
openstackgerritMerged openstack/ceilometer: Updated from global requirements  https://review.openstack.org/14862617:21
openstackgerritMerged openstack/ceilometer: add elasticsearch events db  https://review.openstack.org/13713117:21
*** atan8 has quit IRC17:22
*** yatin has quit IRC17:23
eglynnfolks: I'm gonna pull the trigger on kilo-2 shortly, now that the elasticsearch patch has landed17:27
*** prad has quit IRC17:36
*** prad has joined #openstack-ceilometer17:36
*** zzz_jasonamyers is now known as jasonamyers17:37
*** jasonamyers is now known as zzz_jasonamyers17:38
*** exploreshaifali has quit IRC17:38
fabiog_nadya_: thanks17:43
*** fabiog has quit IRC17:43
openstackgerritMerged stackforge/gnocchi: indexer: store aggregation methods in archive policies  https://review.openstack.org/14711417:45
*** ddieterly has joined #openstack-ceilometer17:47
*** _cjones_ has joined #openstack-ceilometer17:51
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259218:01
*** harlowja has joined #openstack-ceilometer18:11
*** dkingshott has joined #openstack-ceilometer18:11
j_tCan ceilometer be used as a VM reaper process? I'm trying to find a good way to reclaim 'unused VMs' in our environment, and that being an arbitrary definition.18:14
gordcj_t: lol that's an awesome name.18:18
gordcj_t: if you pair ceilometer with heat, it might accomplish what you're hoping for.18:19
gordchttp://techs.enovance.com/5991/autoscaling-with-heat-and-ceilometer18:19
cdentmight also be able to pull it off with a simple webhook?18:19
openstackgerritMerged openstack/ceilometermiddleware: add README template  https://review.openstack.org/15297218:24
*** zzz_jasonamyers is now known as jasonamyers18:29
gordceglynn: i'm not allowed to release ceilometermiddleware (says prohibited by gerrit when i run oslo-release) so you'll need to release it.18:49
*** Fabio has joined #openstack-ceilometer18:58
*** Fabio has quit IRC18:59
*** fabiog has joined #openstack-ceilometer18:59
fabioggordc: quick question19:00
gordcfabiog: sure.19:01
fabioggordc: can you please point me to the code where you can add a parameter to the ceilo conf?19:02
fabioggordc: we need to add a specific driver for the pipeline since this is not optional or linked to the other database choices19:02
gordcfabiog: adding a new option to conf?19:03
fabioggordc: something along the line of a pipeline_connection inside the [database] section of the conf19:03
fabioggordc: yes19:03
gordchttps://github.com/openstack/ceilometer/blob/fc7e78134458b39a07af163396aa4e43602202ae/ceilometer/data_processing/notifications.py#L23-L3019:03
gordchttps://github.com/openstack/ceilometer/blob/master/ceilometer/storage/__init__.py#L42-L7019:04
gordclike that? ^19:04
fabioggordc: and this will generate a default option in the conf? Because tempest will need it to correctly set ceilometer up19:04
fabioggordc: we actually did that, but we need a default string to be in the conf19:05
gordcfabiog: well it will use the default you define if it's not in conf.19:05
fabioggordc: so you are saying not to use None in the default but actually the connection string value?19:05
gordcfabiog: if you want to have real text in the conf file you will need to add it to devstack specifically19:05
gordcfabiog: right... assuming the default you want is generic19:06
gordcfabiog: alternatively you can set the value here: https://github.com/openstack-dev/devstack/blob/master/lib/ceilometer#L179-L18119:06
gordcnot there specifically but in that file... and in a similar format19:07
fabioggordc: but this is a devstack change19:08
gordcfabiog: right.19:08
fabioggordc: I will go with the default to sql and then maybe we will have a concurrent patch to address this. My understanding is that if the param is not set in the conf it will use the default19:09
fabioggordc: so when the devstack patch is in, then it will override the default and we can move the default back to None. Does it make sense to you?19:10
gordcfabiog: sure. i take it None means don't us db base pipeline?19:11
*** _nadya_ has joined #openstack-ceilometer19:12
fabioggordc: well if there was a value in the conf that would be overridden. You have to provide a database for the pipeline since now is decoupled from the other dbs19:12
gordcfabiog: so it is mandatory to have pipeline in db?19:14
gordcfabiog: there's no way to continue as is? ie. just static conf files?19:14
fabioggordc: well it will be, but the test are failing in devstack just for the db patch19:15
*** exploreshaifali has joined #openstack-ceilometer19:16
fabioggordc: let me check if we can do this with the patch19:17
fabioggordc: thanks for the pointers19:17
*** cdent has quit IRC19:18
gordcfabiog: the spec says the current file based configuration should continue to work... this is still true right?19:19
fabioggordc: yes, it will.19:20
fabioggordc: there will be a param to select if you want to load it in the db or not19:20
gordcfabiog: ok, cool cool. so db is not mandatory. i just wanted to make sure that was still true.19:21
fabioggordc: yes it that part will be in the "services" patch, Central Agent and Notification Agent19:21
gordccool cool. hopefully i'll get a chance to look at it tomorrow19:23
*** _nadya_ has quit IRC19:28
*** harlowja is now known as harlowja_away19:50
*** EmilienM is now known as EmilienM|afk19:53
*** llu has quit IRC19:55
*** dkingshott has quit IRC19:58
*** adelinatuvenie has joined #openstack-ceilometer20:02
*** dkingshott has joined #openstack-ceilometer20:06
*** prad has quit IRC20:06
*** prad has joined #openstack-ceilometer20:10
*** _nadya_ has joined #openstack-ceilometer20:30
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259220:32
*** harlowja_away is now known as harlowja20:33
ildikovfabiog: hi20:34
fabiogildikov: hi20:35
ildikovfabiog: is the patch ready for review that you linked earlier?20:35
ildikovfabiog: I mean this one just proposed by Rohit ^^^20:37
fabiogildikov: we did some changes to fix the build in tempest, it is building now, but feel free to check that there isn't anything major20:37
ildikovfabiog: it kinda looks half ready, and I haven't checked even a half of it yet20:38
ildikovfabiog: it looks a bit odd to have storage drivers with only __init__ ready or not much more, but maybe I don't get the point here20:40
*** fnaval_ has joined #openstack-ceilometer20:41
fabiogildikov: we are going to implement the sql only solution, so for the other we just provided a stub20:42
fabiogildikov: others can add implementations for MongoDB or HBase if they want20:43
ildikovfabiog: wouldn't there be enough the NotImplemented error?20:44
fabiogildikov: we needed the methods for some of the unit tests20:44
ildikovfabiog: it's just did not make much sense to me this way and makes the patch even bigger, which does not help in reviewing20:45
*** fnaval has quit IRC20:45
ildikovfabiog: unit tests can be controlled to run against specific DBs and not all the available types20:45
fabiogildikov: we moved the pipeline.py file in the pipeline package since now we have one, and that triggered the changes in the tests. I think in the future is beneficial to have all the pipeline related code inside the pipeline package20:46
ildikovfabiog: hmm, I still don't get it 100%, why it needs those stubs20:47
ildikovfabiog: do you mean that the same db is used for pipeline, which is used in general in case of testing?20:48
ildikovfabiog: sorry for the dumb questions, I haven't slept much this week :(20:48
fabiogildikov: now we have a different db for pipeline, so you can have sql for pipeline and MongoDB for samples20:48
fabiogildikov: we are just implementing sql, but the tests are generalized for all the dbs so we are creating those stubs20:49
fabiogildikov: hope makes sense20:49
ildikovfabiog: then I still don't get why you needs those stubs for unit tests20:49
fabiogildikov: there are two things: 1) moving pipeline.py created a lot of changes in all the tests that used that file, 2) having a separate database means that you need to have the basic impl for all the databases since the tests are now run with all of them20:50
ildikovfabiog: you mean the tests that are using pipelines without any kind of mocking?20:51
fabiogildikov: no we had to change the mockings because of the refactoring of the pipeline.py file20:52
fabiogildikov: you should see that all the code changes in the tests are related to that re-factoring20:53
fabiogildikov: changing the location of a file usually does this .. :-)20:53
ildikovfabiog: yes, I got the renaming of the imports and mocking20:54
fabiogildikov: we may move that file back. it will shorten the patch ...20:54
ildikovfabiog: I just don't get the stubs20:54
fabiogildikov: events and alarms have the same strategy20:55
ildikovfabiog: it's not about the placement of the file20:55
ildikovfabiog: I mean files20:55
ildikovfabiog: I kinda feel that we are talking about two different things now :)20:56
*** nellysmitt has quit IRC20:56
fabiogildikov: ok. Event and Alarms have been moved form the unified storage to the separate storage20:56
fabiogildikov: if you look at these are very similar to what we have done20:56
fabiogildikov: you need the upgrade and clear to prepare and get rid of the sql tables20:57
*** fnaval_ has quit IRC20:57
*** fnaval has joined #openstack-ceilometer20:57
ildikovfabiog: I just don't get why db2 and other storage stuff is needed20:58
ildikovfabiog: I thought you mean that the tests are using it, but now I'm lost20:59
fabiogildikov: please comment in the patch, I want Rohit to be able to respond. My understanding is that unit tests are across databases and they will fail if you don't have at least a stab for the various db21:00
ildikovfabiog: well, I think it's better if i have some rest, check the whole patch and add my questions and concerns as review comments21:00
*** eglynn has quit IRC21:00
ildikovfabiog: ok, that's exactly what I meant and we are also on the same page regarding the next step, so thanks :)21:01
ildikovfabiog: sorry that it took so long, I wanted to be sure that I get the intention, so I can have a better insight when reviewing21:01
ildikovfabiog: so thanks much for your time here21:02
ildikovfabiog: sorry if I looked a bit impatient or smth like, it really was not an intention, I'm just too tired21:03
gordcgo to sleep21:03
fabiogildikov: no apologies, you have valid concerns, get some rest and we can talk about it tomorrow21:03
gordcproblem solved... pay on the way out21:04
ildikovgordc: will do :)21:04
gordc:)21:04
ildikovfabiog: cool, thanks, sounds good :)21:05
ildikovgordc: fabiog: have a nice afternoon/evening :)21:05
*** zul has quit IRC21:08
*** zul has joined #openstack-ceilometer21:08
*** claudiub has quit IRC21:12
*** packet has quit IRC21:30
*** adelinatuvenie has quit IRC21:31
*** _nadya_ has quit IRC21:40
*** julim has quit IRC21:45
*** EmilienM|afk is now known as EmilienM21:45
*** hhuang has quit IRC21:56
*** hhuang has joined #openstack-ceilometer21:56
*** claudiub has joined #openstack-ceilometer22:08
*** jasonamyers is now known as zzz_jasonamyers22:34
gordcclarkb: you around? /apologies i have no idea what timezone you're in.22:44
clarkbsort of not in front of a computer this moment22:45
gordcclarkb: cool cool. nm then. i'll give you a shout tomorrow. wanted to run elasticsearch stuff by you but i'm pretty sure i'm doing it wrong.22:47
clarkbI think it came up in -infra because no trusty packages yet (I thought they existed)22:47
* gordc hadn't checked... i'll try looking at that.22:49
gordcclarkb: hmm guess not... i'll try finding another way to test it. thanks for inpu22:51
gordcinput*22:51
*** nellysmitt has joined #openstack-ceilometer22:57
*** jaypipes has quit IRC23:00
*** nellysmitt has quit IRC23:01
*** ddieterly has quit IRC23:01
*** prad has quit IRC23:10
*** gordc has quit IRC23:26
*** fnaval has quit IRC23:29
*** ddieterly has joined #openstack-ceilometer23:37
*** Kennan has joined #openstack-ceilometer23:43
*** ViswaV has joined #openstack-ceilometer23:45
*** lan has quit IRC23:57
*** lan has joined #openstack-ceilometer23:57
*** ddieterly has quit IRC23:57

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