Wednesday, 2013-09-25

*** maksimov has joined #openstack-metering00:10
*** krtaylor has joined #openstack-metering00:15
*** yfujioka has joined #openstack-metering00:15
*** nosnos has joined #openstack-metering00:16
*** maksimov has quit IRC00:19
*** matsuhashi has joined #openstack-metering00:35
*** changbl has joined #openstack-metering00:36
*** thomasm has quit IRC00:37
*** fnaval_ has joined #openstack-metering00:58
*** anteaya has quit IRC01:02
*** llu_away is now known as llu01:07
openstackgerritA change was merged to openstack/ceilometer: add more test cases to improve the test code coverage #1  https://review.openstack.org/4760801:32
*** krtaylor has quit IRC01:56
*** shang has joined #openstack-metering01:58
*** SergeyLukjanov has joined #openstack-metering02:03
*** SergeyLukjanov has quit IRC02:03
*** SergeyLukjanov has joined #openstack-metering02:04
*** Shaan7 has quit IRC03:01
*** SergeyLukjanov has quit IRC03:05
*** sandywalsh has quit IRC03:14
*** sandywalsh has joined #openstack-metering03:26
*** fnaval_ has quit IRC03:30
*** fnaval_ has joined #openstack-metering03:30
*** Shaan7 has joined #openstack-metering03:33
*** nosnos has quit IRC03:40
*** nosnos has joined #openstack-metering03:41
*** nosnos has quit IRC03:42
*** nosnos has joined #openstack-metering03:44
*** maksimov has joined #openstack-metering03:48
*** litong has quit IRC03:49
*** maksimov has quit IRC03:52
*** zbitter has joined #openstack-metering03:55
*** zaneb has quit IRC03:59
*** flwang has quit IRC04:09
*** zbitter has quit IRC04:15
*** zbitter has joined #openstack-metering04:15
*** zbitter has quit IRC04:21
*** zbitter has joined #openstack-metering04:23
*** flwang has joined #openstack-metering04:25
*** sdake__ has joined #openstack-metering04:35
*** sdake__ has joined #openstack-metering04:35
*** sdake_ has quit IRC04:35
*** evanjfraser has quit IRC04:42
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Incorrect error code impacted by debug level  https://review.openstack.org/4777504:44
*** nadya has joined #openstack-metering04:45
*** nadya has quit IRC04:55
*** nadya has joined #openstack-metering05:01
*** nadya has quit IRC05:10
*** SergeyLukjanov has joined #openstack-metering05:18
*** SergeyLukjanov has quit IRC05:18
*** Shaan7 has quit IRC05:35
*** maksimov has joined #openstack-metering05:37
*** maksimov has quit IRC05:41
*** Shaan7 has joined #openstack-metering05:48
*** flwang has quit IRC05:51
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/4796706:03
openstackgerritchenxiao proposed a change to openstack/ceilometer: Fixing ceilometer-dbsync errors when using DB2 and sqlalchemy  https://review.openstack.org/4697406:16
*** nsaje has joined #openstack-metering06:23
*** zbitter has quit IRC06:27
*** zbitter has joined #openstack-metering06:28
*** maksimov has joined #openstack-metering06:32
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Incorrect error code impacted by debug level  https://review.openstack.org/4777506:32
*** maksimov has quit IRC06:39
*** nadya has joined #openstack-metering06:40
openstackgerritchenxiao proposed a change to openstack/ceilometer: Fixing ceilometer-dbsync errors when using DB2 and sqlalchemy  https://review.openstack.org/4697406:58
*** flwang has joined #openstack-metering07:15
*** nadya has quit IRC07:19
*** eglynn-AFK has quit IRC07:24
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Don't publish samples if resource_id in missing  https://review.openstack.org/4604807:40
openstackgerritMehdi Abaakouk proposed a change to openstack/python-ceilometerclient: Improve the CM shell client alarm visualisation  https://review.openstack.org/4800507:45
*** eglynn-AFK has joined #openstack-metering07:52
*** boris-42 has joined #openstack-metering08:20
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: tests: fix places check for timestamp equality  https://review.openstack.org/4819908:33
*** yfujioka has quit IRC08:34
*** xiayu has joined #openstack-metering08:36
*** xiayu has quit IRC08:37
*** SergeyLukjanov has joined #openstack-metering08:58
openstackgerritA change was merged to openstack/ceilometer: Don't send unuseful rpc alarm notification  https://review.openstack.org/4798909:30
openstackgerritA change was merged to openstack/ceilometer: Require oslo.config 1.2.0 final  https://review.openstack.org/4813809:31
*** zbitter is now known as zaneb09:37
*** maksimov has joined #openstack-metering09:37
openstackgerritA change was merged to openstack/ceilometer: Add source field to Meter model  https://review.openstack.org/4663910:11
*** maksimov has quit IRC10:28
openstackgerritA change was merged to openstack/ceilometer: Adding region name to service credentials  https://review.openstack.org/4734110:44
openstackgerritA change was merged to openstack/python-ceilometerclient: Improve the CM shell client alarm visualisation  https://review.openstack.org/4800510:44
*** ruhe has joined #openstack-metering11:05
*** nsaje has quit IRC11:37
*** nsaje has joined #openstack-metering11:37
*** krtaylor has joined #openstack-metering11:40
*** ruhe has quit IRC12:00
*** sandywalsh has quit IRC12:07
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Avoid imposing alembic 6.0 requirement on all distros  https://review.openstack.org/4806912:17
*** sandywalsh has joined #openstack-metering12:20
*** thomasm has joined #openstack-metering12:28
*** raghuvrao has quit IRC12:32
*** raghuvrao has joined #openstack-metering12:33
*** eglynn-AFK is now known as eglynn-lunch12:39
*** ruhe has joined #openstack-metering12:39
*** eglynn-lunch has quit IRC12:41
*** annegentle has quit IRC12:42
*** annegentle has joined #openstack-metering12:43
*** gordc has joined #openstack-metering12:48
*** krtaylor has quit IRC12:49
thomasmHey all12:50
*** bpokorny has joined #openstack-metering12:52
sandywalshAll the Monitorama.eu slides are up https://speakerdeck.com/monitorama12:54
thomasmAhhh, nice. Thanks, sandywalsh  =]12:56
*** nsaje has quit IRC12:56
*** Ruetobas has quit IRC13:00
*** krtaylor has joined #openstack-metering13:01
*** Ruetobas has joined #openstack-metering13:02
openstackgerritCyril Roelandt proposed a change to openstack/python-ceilometerclient: Help messages: specify which options are required  https://review.openstack.org/4587013:03
*** anteaya has joined #openstack-metering13:09
flwangjd__: ping13:10
*** eglynn-lunch has joined #openstack-metering13:11
jd__flwang: ?13:11
*** Ruetobas has quit IRC13:12
*** julim has joined #openstack-metering13:12
chmouelsandywalsh: nice13:12
flwangjd__: as for this fix https://review.openstack.org/#/c/47775/ , I would say almost all the exceptions of wsme.exc will run into the issue. do you think we need update all of them to ClientSideError in v2 api?13:13
sandywalshany oslo.config experts in the house?13:13
flwangjd__: I think yes, but wanna confirm with you13:13
jd__flwang: since you write a test for one, I guess you can port it for the others and you will have your answer? :)13:13
jd__flwang: OTOH I think it'd be better to fix this into WSME, I'll look into it13:14
*** SergeyLukjanov has quit IRC13:15
flwangjd__: but do you think we really need test cases for each of them?13:15
jd__flwang: oh that I do13:15
flwangjd__: yep, I agree. that's why I'm hesitating to update all of them.13:15
flwangjd__: i have talked with dhellmann and he has opened a but against WSME, but seems no outlook when it can be fixed13:16
*** Ruetobas has joined #openstack-metering13:16
jd__flwang: yeah, but I've started tackling stuff in WSME recently13:21
jd__flwang: so I'll do this one as well13:21
flwangjd__: cool, so what's your suggestion for this?13:22
jd__flwang: blame WSME and drop the patch :)13:22
jd__you'll be able to fix it with a requirement on a higher version13:22
flwangjd__: and waiting the your patch from WSME ? :)13:22
jd__yes13:22
flwangjd__: sounds great13:23
flwangjd__: will do that, pls let me know if you fixed this problem13:23
flwangjd__: thanks13:23
*** changbl has quit IRC13:31
*** fnaval_ has quit IRC13:38
sandywalshjd__, any thoughts on a bug I'm running into with oslo.config? Details here: http://paste.openstack.org/show/47494/13:40
*** julim has quit IRC13:45
* jd__ looks13:47
jd__sandywalsh: ho that's an easy one13:47
sandywalshjd__, excellent :)13:48
jd__sandywalsh: you can't import nova.openstack.common in ceilometer, so use the rabbit notifier driver from ceilometer.openstack.common13:48
jd__this is a short coming of the way oslo.config.CFG is used globally by everyone unfortunately13:48
jd__s/CFG/cfg.CONF/13:49
sandywalshah! haha ... good catch. Never even noticed that in my .conf13:50
sandywalshthanks jd__ !13:50
jd__you're welcome :)13:50
*** fnaval_ has joined #openstack-metering14:03
*** fnaval__ has joined #openstack-metering14:07
*** matsuhashi has quit IRC14:07
*** fnaval_ has quit IRC14:07
*** nosnos has quit IRC14:08
*** ruhe has quit IRC14:09
*** eglynn-lunch is now known as eglynn14:15
*** shang has quit IRC14:20
*** julim has joined #openstack-metering14:21
*** bpokorny1 has joined #openstack-metering14:25
*** bpokorny has quit IRC14:27
*** ruhe has joined #openstack-metering14:28
*** changbl has joined #openstack-metering14:28
*** Ruetobas has quit IRC14:33
*** Ruetobas has joined #openstack-metering14:34
*** maksimov has joined #openstack-metering14:42
*** ruhe has quit IRC14:51
*** changbl has quit IRC14:55
*** bpokorny1 has quit IRC14:56
*** bpokorny has joined #openstack-metering15:00
*** changbl has joined #openstack-metering15:03
openstackgerritlitong01 proposed a change to openstack/ceilometer: add more test cases to improve the test code coverage #3  https://review.openstack.org/4827515:09
*** litong has joined #openstack-metering15:11
*** bpokorny has quit IRC15:23
openstackgerritCyril Roelandt proposed a change to openstack/python-ceilometerclient: Help messages: specify which options are required  https://review.openstack.org/4587015:25
*** bpokorny has joined #openstack-metering15:27
litonghi, folks, can any helping soul review this patch and get it merged? do not like to keep rebasing it. https://review.openstack.org/#/c/47629/15:32
*** shang has joined #openstack-metering15:32
*** maksimov has quit IRC15:33
eglynnlitong: looking now ...15:37
litong@eglynn, thanks so much.15:37
litong@eglynn, I also have this one, https://review.openstack.org/#/c/48275/15:37
litongthey are all series of new test cases to make our test coverage percentage higher.15:38
beavif i wanted to notify a rest interface in another app if an intance was created or terminated, is the best way to do that to create an alarm that fires a request?15:40
thomasmlitong, Thanks for working on the tests. =]15:40
eglynnlitong: super-quick question on the use of repr.repr? is it the size limits?15:41
litong@thomasm, thanks, our test coverage is quite low. only around 55%.15:41
litong@eglynn, repr.repr basically calls the object __repr__ method and serialize that object.15:41
eglynnbeav: ceilometer alarms are more oriented towards thresholds being crossed by statistics, as opposed to resource lifecycle events15:42
litong@eglynn, not ware it has a limit.15:42
thomasmlitong, Yeah, I'm really wanting to address that also. We probably could have avoided the issues with the drivers if we had our use-cases better documented in tests. =]15:42
eglynnlitong: fair enough15:42
litong@eglynn, wondering why you mentioned the size, does it have anything to do with object size?15:43
eglynnlitong: well I was just wondering what the motivation for using the alternative repr was15:44
eglynnlitong: repr.repr brings some size limitations to the table IIRC15:44
litong@eglynn, ah, yes, the purpose of having a method named __repr__() is to basically overwrite the standard repr behavior.15:44
litong@eglynn, so calling repr.repr() to make sure that behavir is preserved, that is the class __repr__() gets called.15:45
eglynnlitong: k, got it, thanks15:45
beaveglynn: would it make sense for me to add a publisher, similar to http://docs.openstack.org/developer/ceilometer/architecture.html#multi-publisher ?15:45
eglynnbeav: simpler just to directly consume the nova notification15:46
litong@heav, Ceilometer currently can be configured to have many publishers, is there a particular publisher you want to develop?15:47
litong@heav, or you are thinking adding the framework which allowes multiple publishers?15:47
litong@heav, if it is the later, then we have it already.15:47
beavlitong: i'd like to feed instance creates/terminate events into katello. i'm still getting familiar with the various pieces of openstack though, so i'm trying to figure out the best way to do it15:50
eglynnbeav: if all you want is the raw instance-lifecycle events as they arise, then you could just consume the nova AMQP notifications directly15:52
eglynnbeav: (as opposed to these data being mediated by ceilometer)15:52
litong@beav, there is also another way to do that, you can look at the file dispatcher and get an idea how to easily add a hook to process the raw data.15:54
litong@beav, that way, you do not even need to mess with reading from the queue.15:54
litong@beav, take a look at this file ceilometer/ceilometer/collector/dispatcher/file.py15:55
litong@beav, that file dispatcher writes everything into a file, you can easily overwrite record_metering_data to do whatever you need.15:55
beavi see, thx15:56
beavlitong: i will give that a shot15:57
beaveglynn: litong: thx for your help:)15:57
*** julim has quit IRC15:57
litong@beav, we also have document in install/manual.rst to tell you how to configure a new dispatcher.15:58
litong@beav, just in case you want to give it a spin.15:58
*** ashestakov_ has joined #openstack-metering16:00
*** Ruetobas has quit IRC16:01
*** Ruetobas has joined #openstack-metering16:04
*** SergeyLukjanov has joined #openstack-metering16:05
*** Ruetobas has quit IRC16:09
*** Ruetobas has joined #openstack-metering16:14
openstackgerritA change was merged to stackforge/pycadf: Switch to oslo.config 1.2.0 final  https://review.openstack.org/4815716:17
*** julim has joined #openstack-metering16:19
*** julim has quit IRC16:21
openstackgerritA change was merged to openstack/ceilometer: Fix failed downgrade in migrations  https://review.openstack.org/4541016:32
*** sdake__ is now known as sdake_16:40
*** sandywalsh has quit IRC17:02
openstackgerritA change was merged to openstack/ceilometer: tests: fix places check for timestamp equality  https://review.openstack.org/4819917:12
*** sandywalsh has joined #openstack-metering17:18
*** SergeyLukjanov has quit IRC17:32
*** ashestakov_ has quit IRC17:34
*** sandywalsh_ has joined #openstack-metering17:34
*** sandywalsh has quit IRC17:38
*** maksimov has joined #openstack-metering17:38
*** zaneb has quit IRC17:39
*** zaneb has joined #openstack-metering17:41
*** maksimov has quit IRC17:43
*** julim has joined #openstack-metering17:45
*** maksimov has joined #openstack-metering17:54
*** julim has quit IRC18:03
*** ashestakov_ has joined #openstack-metering18:05
*** julim has joined #openstack-metering18:05
*** julim has quit IRC18:10
*** maksimov has quit IRC18:15
*** SergeyLukjanov has joined #openstack-metering18:17
*** julim has joined #openstack-metering18:42
litonganybody know how to run run-tests.sh with coverage but excluding some common libraries/modules?18:52
litongwanted to have an overall test code coverage without modules that insert from common libraries.18:53
*** maksimov has joined #openstack-metering18:55
*** evanjfraser has joined #openstack-metering18:58
*** evanjfraser has quit IRC19:04
*** julim has quit IRC19:05
*** julim has joined #openstack-metering19:06
*** nati_ueno has joined #openstack-metering19:12
*** SergeyLukjanov has quit IRC19:47
*** julim has quit IRC19:49
*** julim has joined #openstack-metering19:52
*** maksimov has quit IRC20:00
beavlitong: i copied file.py and made some minor edits, but i'm having some trouble getting my new dispatcher to load. I see it mentioned in collector.log, but i dont see the new file being created that it should be logging to. i also edited setup.cfg to add the new dispatcher to ceilometer.dispatcher20:05
beavright now i'm only bouncing openstack-ceilometer-collector to try to get the changes picked up20:06
litong@beav, can you confirm if the dispatcher gets called?20:06
litongthere need to be similar lines like these in your entry_points.txt20:08
litongceilometer.dispatcher =20:08
litong    database = ceilometer.collector.dispatcher.database:DatabaseDispatcher20:08
litong    file = ceilometer.collector.dispatcher.file:FileDispatcher20:08
litongif you have added to setup.cfg, you will need to run the following command.20:08
litongsudo python setup.py develop20:09
litongto make it take affect.20:09
beavah:) that is probably what i am missing20:10
litongafter that, you will need to enable it in your ceilometer.conf file.20:12
beavlitong: i think there was some issue with having python-ceilometer installed via rpm and also installed via ./setup.py. the rpm version of entry_points.txt was being picked up instead of the setup.py installed version20:13
litongthe entry point file is just to make it available to others.20:13
beavi edited the other entry_points.txt and it works now, thx20:13
litonggood.20:13
openstackgerritMonsyne Dragon proposed a change to openstack/ceilometer: Add configuration-driven conversion to Events  https://review.openstack.org/4271320:14
litong@jd__, ping20:14
*** ashestakov_ has quit IRC20:21
*** nati_ueno has quit IRC20:24
openstackgerritgordon chung proposed a change to stackforge/pycadf: fix conf file settings  https://review.openstack.org/4833720:25
*** nati_ueno has joined #openstack-metering20:33
*** harmw has quit IRC20:40
*** harmw has joined #openstack-metering20:46
*** terriyu has joined #openstack-metering20:52
*** tri2sing has joined #openstack-metering21:01
*** tri2sing has quit IRC21:01
*** zbitter has joined #openstack-metering21:01
*** tri2sing has joined #openstack-metering21:02
*** tri2sing has quit IRC21:03
*** tri2sing has joined #openstack-metering21:04
*** zaneb has quit IRC21:05
*** sadhikar_ has joined #openstack-metering21:05
*** sadhikar__ has joined #openstack-metering21:05
*** sadhikar__ has quit IRC21:06
*** sadhikar_ has quit IRC21:06
*** zbitter is now known as zaneb21:07
*** tri2sing has left #openstack-metering21:08
*** thomasm has quit IRC21:08
*** sadhikar_ has joined #openstack-metering21:08
*** sadhikar_ has left #openstack-metering21:08
*** julim has quit IRC21:09
terriyudo we have a meeting today?21:09
*** thomasm has joined #openstack-metering21:12
*** thomasm has quit IRC21:13
litong@terriyu, I guess not.21:18
terriyulitong: ok, thanks21:23
eglynnterriyu, litong: today's meeting was cancelled ... http://lists.openstack.org/pipermail/openstack-dev/2013-September/015613.html21:30
litong@eglynn, thanks.21:31
litonggate-tempest-devstack-vm-neutron kept having problems21:32
litongcan someone please fix that problem?21:32
litongit kept approved patchset from being merged.21:33
terriyuthanks eglynn21:36
*** changbl has quit IRC21:41
openstackgerritlitong01 proposed a change to openstack/ceilometer: add coverage config file to control module coverage report  https://review.openstack.org/4835421:52
*** gordc has quit IRC21:53
*** boris-42 has quit IRC21:55
*** bpokorny has quit IRC22:06
*** eglynn has quit IRC22:09
*** litong has quit IRC22:22
*** nati_ueno has quit IRC22:32
*** dhellmann is now known as dhellmann_22:42
*** fnaval__ has quit IRC22:46
*** boris-42 has joined #openstack-metering23:26
*** julim has joined #openstack-metering23:27
*** julim has quit IRC23:40

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