Monday, 2013-08-26

*** changbl has joined #openstack-metering00:25
*** sdake_ has joined #openstack-metering00:28
*** flwang has quit IRC00:32
*** tasdomas is now known as tasdomas_afk00:35
*** sdake_ has quit IRC00:48
*** dina_belova has joined #openstack-metering00:49
*** dina_belova has quit IRC00:54
*** alexb_ has joined #openstack-metering01:16
*** shang has joined #openstack-metering01:30
*** Prince_Africanus has joined #openstack-metering01:33
*** flwang has joined #openstack-metering01:34
*** Prince_Africanus has quit IRC01:41
*** dina_belova has joined #openstack-metering01:50
*** dina_belova has quit IRC01:55
*** d34dh0r53 has joined #openstack-metering01:59
*** d34dh0r53 has quit IRC02:03
*** d34dh0r53 has joined #openstack-metering02:03
*** sdake_ has joined #openstack-metering02:19
*** sdake_ has quit IRC02:19
*** sdake_ has joined #openstack-metering02:19
*** dina_belova has joined #openstack-metering02:51
*** dina_belova has quit IRC02:55
*** alexb_ has quit IRC03:16
*** SergeyLukjanov has joined #openstack-metering03:19
*** shang_ has joined #openstack-metering03:39
*** shang has quit IRC03:39
*** shang_ is now known as shang03:39
*** alexb_ has joined #openstack-metering03:51
*** dina_belova has joined #openstack-metering03:51
*** dina_belova has quit IRC03:55
*** alexb_ has quit IRC04:06
*** alexb_ has joined #openstack-metering04:29
openstackgerritShuangtai Tian proposed a change to openstack/ceilometer: Handle the metrics sent by nova notifier  https://review.openstack.org/4283804:47
openstackgerritShuangtai Tian proposed a change to openstack/ceilometer: Handle the metrics sent by nova notifier  https://review.openstack.org/4283804:50
*** dina_belova has joined #openstack-metering04:52
*** dina_belova has quit IRC04:57
*** alexb_ has quit IRC05:06
*** evanjfraser has quit IRC05:11
*** SergeyLukjanov has quit IRC05:13
*** shang_ has joined #openstack-metering05:35
*** shang has quit IRC05:37
*** sdake_ has quit IRC05:51
*** dina_belova has joined #openstack-metering05:52
*** tasdomas_afk is now known as tasdomas05:54
*** dina_belova has quit IRC05:57
*** terriyu has quit IRC06:02
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/4340506:11
*** evanjfraser has joined #openstack-metering06:19
*** yolanda has joined #openstack-metering06:49
*** dina_belova has joined #openstack-metering06:53
*** dina_belova has quit IRC06:56
*** maksimov has joined #openstack-metering07:35
*** evanjfraser has quit IRC07:41
*** d34dh0r53 has quit IRC07:50
*** dina_belova has joined #openstack-metering07:53
*** dina_belova has quit IRC07:59
*** shang has joined #openstack-metering08:01
*** shardy is now known as shardy_afk08:02
*** d34dh0r53 has joined #openstack-metering08:02
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: doc: replace GitHub by git.openstack.org  https://review.openstack.org/4366108:23
*** eglynn has joined #openstack-metering08:24
*** yolanda has quit IRC08:32
*** shang has quit IRC08:39
*** shang has joined #openstack-metering08:41
openstackgerritA change was merged to openstack/ceilometer: Raise Error when pagination support is missing  https://review.openstack.org/4288908:48
*** dina_belova has joined #openstack-metering08:54
*** dina_belova has quit IRC08:58
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: api: allow usage of resource_metadata in query  https://review.openstack.org/4331008:59
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: doc: replace GitHub by git.openstack.org  https://review.openstack.org/4366109:01
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Support for wildcard in pipeline  https://review.openstack.org/4363009:24
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Extra indexes cleanup  https://review.openstack.org/4271609:28
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name  https://review.openstack.org/4271509:32
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Support for wildcard in pipeline  https://review.openstack.org/4363009:36
*** yolanda has joined #openstack-metering09:47
flwangjd__: ping09:51
*** dina_belova has joined #openstack-metering09:55
flwangGuangyu Suo: around?09:55
jd__flwang: o/09:58
flwangthis is regarding the bug 1202749: missing some metadata09:59
*** dina_belova has quit IRC09:59
flwangjd__: though you and gordc approved my patch, but the jenkins failed since the other patch introduced an issue. so I'd like to have short discussion with you.10:00
jd__listening10:00
flwangjd__: see line 88: https://review.openstack.org/#/c/40098/11/nova_tests/test_notifier.py10:01
flwangit's using util.make_sample_from_instance but passing the incorrect instance10:02
flwangThe instance object collected by pollster is <novaclient.v1_1.servers.Server>,  like the #1 in http://paste.openstack.org/show/43144/.  my pre patch-set was trying to access the flavor attribute to get more  info, so that's why my pre patchset failed, since the flavor attribute  is missing in <compute.nova_notifier.Instance>10:03
flwangjd__: I don't understand why we need calling util.make_sample_from_instance to build the sample. and if we really need to do that, I think we do need to pass the correct instance type. Just want confirm with you since it's approved by you :)10:06
*** lsmola has joined #openstack-metering10:06
jd__ouh la la10:07
* jd__ needs some time to rewire its brain on this part10:08
flwangjd__: sure, i understand10:09
jd__flwang: problem is that we can't have this same instance type10:12
jd__I think we're trying to have only one function converting all instance from either nova or novaclient to a sample10:12
flwangjd__: are you talking about transform novaclient.v1_1.servers.Server to compute.nova_notifier.Instance?10:13
flwangjd__: in https://review.openstack.org/#/c/41818/7/ceilometer/compute/pollsters/util.py I'm trying to read flavor info from instance.flavor, but in the unit test code, since it's passing the wrong instance type, so it will be failed due to key error.10:15
flwangjd__: so I'd like to get the thought from you,  do we really need to build the sample by util.make_sample_from_instance?10:17
flwangjd__: if so, seems I may need to build a novaclient.v1_1.servers.Server object in test_notifier.py10:18
jd__I think it would be good to use it yes10:20
jd__maybe add a compatibility layer10:20
jd__in the end one of th goal will be to remove the nova_notifier altogether10:20
jd__does that make sense?10:20
flwangjd__: yep, I do agree we need an abstract layer10:23
flwangjd__: so for now, I need to create a novaclient.v1_1.servers.Server object to pass in, agree?10:24
jd__flwang: or something that looks that way, you don't have to use *that* type10:25
flwangjd__: sorry?10:25
jd__flwang: you can create a class that mimics the behviour of novaclient.v1_1.servers.Server10:26
jd__not necessarily use novaclient.v1_1.servers.Server10:26
jd__your call :)10:26
flwanggot it, so just a mock stuff only used in test_notifier.py, is it?10:27
openstackgerritA change was merged to openstack/ceilometer: Raise Error when pagination/groupby is missing  https://review.openstack.org/4342710:35
openstackgerritA change was merged to openstack/ceilometer: Move sqlachemy tests only in test_impl_sqlachemy  https://review.openstack.org/4289010:35
flwangjd__: will submit a new patch.10:35
flwangthanks for the suggestion10:37
*** flwang has quit IRC10:37
*** chuckieb has joined #openstack-metering10:45
openstackgerritA change was merged to openstack/ceilometer: doc: replace GitHub by git.openstack.org  https://review.openstack.org/4366110:53
*** maksimov has quit IRC10:55
*** dina_belova has joined #openstack-metering10:55
openstackgerritA change was merged to openstack/ceilometer: Remove useless doc/requirements  https://review.openstack.org/4357610:56
*** dina_belova has quit IRC10:59
eglynnjd__: day before I headed off on vacation, we were discussing the impact on devstack of ceilo's new strict requirement for mongo version 2.2+11:23
eglynnjd__: just wondering what the latest story is wrt. to enabling devstack run out-of-the-box with the expected mongo version?11:24
jd__eglynn: we switched the default to Oracle finally11:24
eglynni.e. are infra folks on board with requiring the 10gen package in the devstack gate?11:24
jd__to be corporate compliant11:24
silehtjd__, ahah11:24
eglynnLOL :)11:24
jd__infra isn't thrilled about using 10gen, so we still don't have mongodb on gate-ceilometer-python2711:25
jd__but I've https://review.openstack.org/#/c/42642/ for devstack ready to go in11:25
eglynnnot thrilled, but accepting of the inevitable?11:26
jd__this is needed because nsaje has a patch adding test on tempest that needs that11:26
eglynnyeah, that's the case I had in mind, cool11:26
jd__eglynn: we're all hung to Canonical willingness to backport MongoDB 2.411:27
eglynnyep, we'll have to lean on some of the Canonical folks around these parts ...11:28
*** flwang has joined #openstack-metering11:29
jd__I keep pocking zul :]11:30
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Extra indexes cleanup  https://review.openstack.org/4271611:37
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name  https://review.openstack.org/4271511:42
*** dina_belova has joined #openstack-metering11:56
*** bpokorny has quit IRC11:58
*** dina_belova has quit IRC12:00
*** evanjfraser has joined #openstack-metering12:03
*** bpokorny has joined #openstack-metering12:18
*** bpokorny1 has joined #openstack-metering12:19
*** boris-42 has joined #openstack-metering12:20
*** bpokorny has quit IRC12:23
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios  https://review.openstack.org/4307112:24
*** bpokorny1 has left #openstack-metering12:25
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Support for wildcard in pipeline  https://review.openstack.org/4363012:30
openstackgerritA change was merged to openstack/ceilometer: Refactored storage tests to use testscenarios  https://review.openstack.org/4307112:37
*** evanjfraser has quit IRC12:41
*** sandywalsh has joined #openstack-metering12:52
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Support for wildcard in pipeline  https://review.openstack.org/4363012:55
*** dina_belova has joined #openstack-metering12:56
*** thomasm has joined #openstack-metering12:57
*** fnaval_ has quit IRC12:57
*** dina_belova has quit IRC12:59
*** anteaya has joined #openstack-metering12:59
thomasmHello!13:00
*** eglynn is now known as eglynn-lunch13:04
*** jergerber has joined #openstack-metering13:06
*** jergerber has quit IRC13:06
jd__flwang: I've a better patch for https://review.openstack.org/#/c/4363013:07
jd__flwang: can I hijack it?13:07
silehtjd__, that smells a oneline with fnmatch to catch all cases :)13:09
jd__almost13:09
*** jergerber has joined #openstack-metering13:09
flwangmay I know what's your solution? :)13:16
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Support for wildcard in pipeline  https://review.openstack.org/4370213:16
flwangso seems I have to abandon mine13:18
jd__flwang: :) we just crossed13:18
jd__I proposed a new change since I did'nt want to steal your patch without your approval though :)13:19
jd__I removed a lot of special case and simplified the code so it's easier to comprehend13:19
flwangjd__: so, what should I do for my patch?13:22
jd__flwang: well if you agree with my implementation you can abandon and +1 me13:22
jd__or we can fight for votes, as you prefer :-P13:22
*** gordc has joined #openstack-metering13:23
*** fnaval_ has joined #openstack-metering13:23
flwangjd__: haha, I can abandon mine13:23
*** eglynn-lunch is now known as eglynn13:24
flwangif you can say my patch triggered your thought :)13:24
jd__sure :-)13:26
*** fnaval_ has quit IRC13:27
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Support for wildcard in pipeline  https://review.openstack.org/4370213:28
jd__flwang: done ;-)13:28
gordcjd__: did you prefer something like this for requestnotifier? http://paste.openstack.org/show/45143/13:30
jd__gordc: yay13:30
gordcok, i'll make that change then. we don't need to check the global cfg file right? just asking because the ContextMiddleware uses that.13:31
*** zul has quit IRC13:32
*** fnaval_ has joined #openstack-metering13:32
*** yolanda has quit IRC13:32
gordcjd__: ^13:32
flwangjd__: just kidding :)13:33
flwangjd__: I have +1 for your patch13:33
*** fnaval__ has joined #openstack-metering13:33
jd__flwang: :D13:35
*** fnaval_ has quit IRC13:37
*** dina_belova has joined #openstack-metering13:37
*** SergeyLukjanov has joined #openstack-metering13:38
*** maksimov has joined #openstack-metering13:38
openstackgerritAlessandro Pilotti proposed a change to openstack/ceilometer: Adds Hyper-V compute inspector  https://review.openstack.org/4370913:39
*** yolanda has joined #openstack-metering13:39
*** yolanda has quit IRC13:46
gordcjd__: wow, that jenkins test run was quick. i've added the change to use paste ini variable. let me know if you see anything wrong. also, rebased the audit stuff.13:51
openstackgerritgordon chung proposed a change to openstack/ceilometer: missing resource in middleware notification  https://review.openstack.org/4358513:55
*** dina_belova has quit IRC13:56
openstackgerritgordon chung proposed a change to openstack/ceilometer: missing resource in middleware notification  https://review.openstack.org/4358513:59
*** yolanda has joined #openstack-metering14:04
openstackgerritgordon chung proposed a change to openstack/ceilometer: missing resource in middleware notification  https://review.openstack.org/4358514:06
openstackgerritgordon chung proposed a change to openstack/ceilometer: missing resource in middleware notification  https://review.openstack.org/4358514:07
*** litong has joined #openstack-metering14:08
*** yolanda has quit IRC14:19
*** zul has joined #openstack-metering14:28
openstackgerritAlessandro Pilotti proposed a change to openstack/ceilometer: Adds Hyper-V compute inspector  https://review.openstack.org/4370914:31
*** maksimov has quit IRC14:31
*** yolanda has joined #openstack-metering14:35
*** thomasm has quit IRC14:40
*** thomasm has joined #openstack-metering14:44
*** flwang has quit IRC14:44
*** krtaylor has joined #openstack-metering14:56
*** dina_belova has joined #openstack-metering14:57
*** boris-42 has quit IRC14:59
*** dina_belova has quit IRC15:01
openstackgerritA change was merged to openstack/ceilometer: api: allow usage of resource_metadata in query  https://review.openstack.org/4331015:03
*** dina_belova has joined #openstack-metering15:07
*** dina_belova has quit IRC15:11
*** herndon has joined #openstack-metering15:14
*** shaneduan is now known as shaneduan[afk]15:23
*** shaneduan[afk] is now known as shaneduan15:24
*** gdusbabek has quit IRC15:41
*** Ruetobas has joined #openstack-metering15:41
*** giroro_ has joined #openstack-metering15:44
*** Ruetobas has quit IRC15:45
*** shang has quit IRC15:47
*** shang_ has quit IRC15:48
*** shaneduan is now known as shaneduan[afk]15:50
*** zul has quit IRC15:50
*** alexb_ has joined #openstack-metering16:03
*** bmqq123 is now known as yjiang516:05
*** dina_belova has joined #openstack-metering16:08
*** dina_belova has quit IRC16:13
*** litong has quit IRC16:14
*** maksimov has joined #openstack-metering16:17
*** zul has joined #openstack-metering16:18
*** alexb_ has quit IRC16:22
openstackgerritAlessandro Pilotti proposed a change to openstack/ceilometer: Adds Hyper-V compute inspector  https://review.openstack.org/4370916:38
*** boris-42 has joined #openstack-metering16:41
*** dina_belova has joined #openstack-metering16:42
gordcjd__: do you think filtering by counter_volume has any value? https://bugs.launchpad.net/ceilometer/+bug/120256916:43
*** SergeyLukjanov has quit IRC16:43
*** dina_belova has quit IRC16:43
jd__gordc: I can't think about any good example of that16:44
gordcjd__: same here, was going to add support for it but couldn't think of a good use case i'd want to filter on just that.16:45
jd__a good use case is pike detection, but it is more complicated than that16:45
thomasmjd__. gordc: Maybe if someone were trying to figure out where a billing mistake happened?16:45
jd__thomasm: maybe16:45
*** SergeyLukjanov has joined #openstack-metering16:45
thomasmjd__: Of course? that'd just be a sample query16:45
jd__OTOH adding it doesn't cost much16:45
thomasmyeah16:46
gordcthomasm: do you mean search for a random spike in volume to find relating info? timestamp/resource_metadata?16:47
thomasmgordc: Yeah, that's what popped into my head.16:47
*** terriyu has joined #openstack-metering16:47
gordcthomasm: i guess that is a case. i'll look to add support for it.16:47
*** shaneduan[afk] is now known as shaneduan16:51
*** SergeyLukjanov has quit IRC16:52
*** alexb_ has joined #openstack-metering16:55
*** maksimov has quit IRC16:56
* lsmola is away: Away16:57
*** sdake has joined #openstack-metering17:07
*** alexb_ has quit IRC17:07
*** dina_belova has joined #openstack-metering17:12
openstackgerritJay Pipes proposed a change to openstack/ceilometer: Add a specialized Event Type model and db table  https://review.openstack.org/4131617:13
*** sandywalsh has quit IRC17:16
openstackgerritlitong01 proposed a change to openstack/ceilometer: install manual last few sections format needs to be fixed  https://review.openstack.org/4349917:20
*** SergeyLukjanov has joined #openstack-metering17:22
*** tasdomas is now known as tasdomas_afk17:31
*** sandywalsh has joined #openstack-metering17:32
*** sandywalsh has quit IRC17:38
*** sdake_ has joined #openstack-metering17:47
*** sdake_ has quit IRC17:47
*** sdake_ has joined #openstack-metering17:47
*** sandywalsh has joined #openstack-metering17:51
openstackgerritAlessandro Pilotti proposed a change to openstack/ceilometer: Adds Hyper-V compute inspector  https://review.openstack.org/4370918:01
thomasmgordc: Yeppers. I am not certain of how often that would happen, but it may be useful.18:07
*** sdake_ has quit IRC18:10
openstackgerritBrad Pokorny proposed a change to openstack/ceilometer: Use system locale when Accept-Language header is not provided  https://review.openstack.org/4374718:13
*** litong has joined #openstack-metering18:16
*** zul has quit IRC18:31
*** alexb_ has joined #openstack-metering18:36
*** bpokorny has joined #openstack-metering18:39
*** d34dh0r53 has quit IRC18:45
*** eglynn has quit IRC18:57
*** sdake_ has joined #openstack-metering19:22
*** sdake_ has joined #openstack-metering19:22
*** maksimov has joined #openstack-metering19:34
*** d34dh0r53 has joined #openstack-metering19:37
*** SergeyLukjanov has quit IRC19:38
*** d34dh0r53 has quit IRC19:39
*** nealph has joined #openstack-metering20:18
*** evanjfraser has joined #openstack-metering20:33
*** evanjfraser has joined #openstack-metering20:37
openstackgerritlitong01 proposed a change to openstack/ceilometer: install manual last few sections format needs to be fixed  https://review.openstack.org/4349920:46
*** litong has quit IRC20:48
*** yolanda has quit IRC20:50
sandywalshso, run_tests has a --stop option (stop after first error) ... anyone know if that exists for testr?20:58
openstackgerritBrad Pokorny proposed a change to openstack/ceilometer: Use system locale when Accept-Language header is not provided  https://review.openstack.org/4374720:59
jd__sandywalsh: testr help run show --until-failure21:01
sandywalshjd__, thanks!21:03
*** boris-42 has quit IRC21:03
gordcjd__: if you feel like doing some late night reviews: https://review.openstack.org/#/c/41423/ https://review.openstack.org/#/c/42179/ , thought i'd just throw it out there :)21:08
*** thomasm has quit IRC21:10
nealphAnyone familiar enough with the new dispatchers setup to tell me what this error indicates:21:19
nealphRuntimeError: No ceilometer.dispatcher extensions found21:19
*** maksimov has quit IRC21:19
sandywalshnealph, there are some new .conf options needed for that I think? Perhaps reinstalling ceilometer (if a devstack setup). Or look at the patch with those changes and see what .conf changes are needed?21:19
nealphsandywalsh: good thought on examining the patch. I'm thinking that I got out of sync with some dependencies in my latest rebase or else am missing something in the conf. Hoping it's the latter. :(21:20
*** dina_belova has quit IRC21:20
*** dina_belova has joined #openstack-metering21:21
nealphI thought that all was needed was 'dispatchers=database'21:21
*** dina_belova has quit IRC21:25
gordcnealph: it's dispatcher (singular) that might be the problem.21:31
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Implement 'reset_on' for mongodb  https://review.openstack.org/4235521:43
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Statistics: Add a "reset_on" query parameter  https://review.openstack.org/4189921:43
*** shaneduan is now known as shaneduan[afk]21:50
*** bpokorny has quit IRC22:10
*** dina_belova has joined #openstack-metering22:31
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Implement 'reset_on' for mongodb  https://review.openstack.org/4235522:33
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Statistics: Add a "reset_on" query parameter  https://review.openstack.org/4189922:33
*** dina_belova has quit IRC22:36
*** yjiang5 is now known as yjiang5_away22:38
openstackgerritgordon chung proposed a change to openstack/ceilometer: add tests for _query_to_kwargs func  https://review.openstack.org/4379622:41
*** gordc has quit IRC22:51
*** shaneduan[afk] is now known as shaneduan22:54
*** shaneduan is now known as shaneduan[afk]23:05
*** herndon has quit IRC23:13
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Implement 'reset_on' for mongodb  https://review.openstack.org/4235523:24
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Statistics: Add a "reset_on" query parameter  https://review.openstack.org/4189923:24
*** shaneduan[afk] is now known as shaneduan23:30
*** dina_belova has joined #openstack-metering23:32
*** fnaval_ has joined #openstack-metering23:34
*** dina_belova has quit IRC23:37
*** fnaval__ has quit IRC23:37
*** fnaval_ has quit IRC23:39
*** zul has joined #openstack-metering23:47

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