Friday, 2013-08-23

openstackgerritJay Pipes proposed a change to openstack/ceilometer: Add a specialized Event Type model and db table  https://review.openstack.org/4131600:06
*** thomasm has quit IRC00:28
*** dina_belova has joined #openstack-metering00:37
*** dina_belova has quit IRC00:42
*** sdake_ has joined #openstack-metering00:52
*** sdake_ has quit IRC01:08
*** anteaya has quit IRC01:08
*** changbl has joined #openstack-metering01:11
*** sdake_ has joined #openstack-metering01:14
*** alexb_ has quit IRC01:30
*** sdake_ has quit IRC01:37
*** dina_belova has joined #openstack-metering01:37
*** fnaval_ has joined #openstack-metering01:41
*** fnaval_ has quit IRC01:41
*** fnaval_ has joined #openstack-metering01:41
*** dina_belova has quit IRC01:42
*** d34dh0r53 has joined #openstack-metering01:59
*** d34dh0r53 has quit IRC02:01
*** shang has joined #openstack-metering02:06
*** terriyu has quit IRC02:07
*** alexb_ has joined #openstack-metering02:09
*** shang_ has joined #openstack-metering02:12
*** shang has quit IRC02:15
*** maksimov has quit IRC02:24
*** maksimov has joined #openstack-metering02:24
*** dina_belova has joined #openstack-metering02:38
*** dina_belova has quit IRC02:43
*** maksimov has quit IRC02:53
*** maksimov has joined #openstack-metering02:54
*** dina_belova has joined #openstack-metering03:39
*** dina_belova has quit IRC03:44
*** shaneduan[afk] is now known as shaneduan04:37
*** dina_belova has joined #openstack-metering04:39
*** boris-42 has joined #openstack-metering04:42
*** dina_belova has quit IRC04:44
*** alexb_ has quit IRC04:57
*** dina_belova has joined #openstack-metering05:40
*** dina_belova has quit IRC05:44
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name  https://review.openstack.org/4271505:47
*** tasdomas_afk is now known as tasdomas06:01
*** shaneduan is now known as shaneduan[afk]06:01
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/4340506:04
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name  https://review.openstack.org/4271506:06
*** shaneduan[afk] is now known as shaneduan06:07
*** yolanda has joined #openstack-metering06:32
*** dina_belova has joined #openstack-metering06:40
*** SergeyLukjanov has joined #openstack-metering06:42
*** dina_belova has quit IRC06:43
*** maksimov has quit IRC06:47
*** SergeyLukjanov has quit IRC06:58
*** evanjfraser has quit IRC07:10
*** shaneduan is now known as shaneduan[afk]07:20
*** dina_belova has joined #openstack-metering07:41
*** shardy_afk is now known as shardy07:41
*** dina_belova has quit IRC07:45
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation  https://review.openstack.org/4341307:48
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: 'and' and 'or' operations for alarms combination  https://review.openstack.org/4283207:48
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Add alarm combination API  https://review.openstack.org/4197107:48
*** shaneduan[afk] is now known as shaneduan07:50
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: 'and' and 'or' operations for alarms combination  https://review.openstack.org/4283207:52
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Add alarm combination API  https://review.openstack.org/4197107:52
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation  https://review.openstack.org/4341307:52
*** shaneduan is now known as shaneduan[afk]08:04
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: api: allow usage of resource_metadata in query  https://review.openstack.org/4331008:31
*** dina_belova has joined #openstack-metering08:42
*** Alexei_987 has joined #openstack-metering08:45
*** dina_belova has quit IRC08:46
*** SergeyLukjanov has joined #openstack-metering08:51
*** dina_belova has joined #openstack-metering08:52
openstackgerritA change was merged to openstack/ceilometer: Doc: improve doc about Nova measurements  https://review.openstack.org/4284208:55
*** shaneduan[afk] is now known as shaneduan08:56
*** shaneduan is now known as shaneduan[afk]09:05
*** vvechkanov2 has joined #openstack-metering09:11
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: 'and' and 'or' operations for alarms combination  https://review.openstack.org/4283209:11
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Add alarm combination API  https://review.openstack.org/4197109:11
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation  https://review.openstack.org/4341309:11
openstackgerritA change was merged to openstack/ceilometer: db2 support  https://review.openstack.org/4112309:18
*** sdake_ has joined #openstack-metering09:51
*** shaneduan[afk] is now known as shaneduan09:56
* sileht is sad, db2 was merged before testscenarios and impl_db2 don't implement the abstract class correctly :(09:57
*** shaneduan is now known as shaneduan[afk]10:05
*** shang_ has quit IRC10:15
*** nsaje1 has joined #openstack-metering10:20
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Add missing import of _  https://review.openstack.org/4342610:21
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination/groupby is missing  https://review.openstack.org/4342710:21
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination support is missing  https://review.openstack.org/4288910:21
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios  https://review.openstack.org/4307110:21
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Move sqlachemy tests only in test_impl_sqlachemy  https://review.openstack.org/4289010:21
silehtjd__, I get some issues with the testscenarios and db2, some tests won't pass... in db210:22
jd__sileht: tell me more about it10:23
silehtjd__, I have uploded the review to share code and logs10:23
silehtjd__, for example test_get_resources don't return last/first_sample_timestamp10:23
silehtjd__, before my testscenarios change the test was overrided to skip the last/first_sample_timestamp check10:25
jd__ah10:25
jd__can you make the test return a NotImplemented for DB2?10:26
silehtjd__, do you I put a NotImplementedError in  impl_db2.get_ressource storage API ?10:27
jd__sileht: you can't override the test in a particular scenario?10:28
silehtjd__, I haven't find a clean ways to do this10:28
silehtjd__, but this is the first time I need to do this10:28
jd__sileht: disable the DB2 scenario for this class then?10:29
jd__litong will have to fix it if he wants the driver to be more solid10:29
silehtjd__, the value of testscenarios is to ensure that all backends return the same data, no ?10:30
sileht(and raise NotImplementedError if it can't)10:31
jd__sileht: the alternative is that you fix the DB2 driver, do you prefer that I suggest that? :)10:31
silehtsniff :(10:31
jd__the db2 driver will not run these test and will not be compliant10:32
jd__OTOH we don't really care about DB2, only litong does, and he'll be able to fix it10:32
jd__we'll just have to point this at him I guess10:32
silehtI have enabled the functionnal API to db2 too and most of them failed too, their return ugly 'list index out of range'10:34
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Added pollsters for the hardware agent  https://review.openstack.org/4307410:34
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Adding hardware-agent  https://review.openstack.org/4307210:34
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Added hardware agent's inspector and snmp implementation  https://review.openstack.org/4307310:34
jd__sileht: yeah, there's still work to do obviously…10:36
jd__fortunately we are a smart architecture where drivers are isolated, so we don't have to be worried10:36
jd__s/are/have/10:36
silehtjd__, what do you think about I just disable the db2 scenarios until it pass all tests ?10:37
jd__sileht: that's my preferred solution10:37
jd__well at least the one failing :)10:37
silehtI will add a big warning so10:37
*** sdake_ has quit IRC10:41
silehtjd__, a other issue, when I enable db2 functionnal api  tests some of them failed because of mongod http://paste.openstack.org/show/45008/10:46
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Fix empty metadata issue of instance  https://review.openstack.org/4181810:50
silehtjd__, ok I have found: maxConns=3210:51
*** shaneduan[afk] is now known as shaneduan10:56
*** dina_belova has quit IRC10:56
*** complexmind has joined #openstack-metering11:00
*** dina_belova has joined #openstack-metering11:02
*** SergeyLukjanov has quit IRC11:04
*** shaneduan is now known as shaneduan[afk]11:05
*** dina_belova has quit IRC11:06
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name  https://review.openstack.org/4271511:28
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Added pollsters for the hardware agent  https://review.openstack.org/4307411:31
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Added hardware agent's inspector and snmp implementation  https://review.openstack.org/4307311:31
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination/groupby is missing  https://review.openstack.org/4342711:33
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios  https://review.openstack.org/4307111:33
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Move sqlachemy tests only in test_impl_sqlachemy  https://review.openstack.org/4289011:33
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination/groupby is missing  https://review.openstack.org/4342711:38
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios  https://review.openstack.org/4307111:38
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Move sqlachemy tests only in test_impl_sqlachemy  https://review.openstack.org/4289011:38
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: rpc: remove useless sleep  https://review.openstack.org/4344011:40
*** anteaya has joined #openstack-metering11:50
*** shaneduan[afk] is now known as shaneduan11:56
*** dina_belova has joined #openstack-metering12:04
*** SergeyLukjanov has joined #openstack-metering12:05
*** shaneduan is now known as shaneduan[afk]12:06
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: rpc: reduce sleep time  https://review.openstack.org/4344012:10
*** thomasm has joined #openstack-metering12:21
*** zul has joined #openstack-metering12:24
openstackgerritSvetlana Shturm proposed a change to openstack/ceilometer: Extra indexes cleanup  https://review.openstack.org/4271612:29
silehtjd__, finally I have just disabled get_meter_statistics and keep db2 tests enabled, because I get 25 tests failed, all due this this method: https://review.openstack.org/#/c/43071/5/ceilometer/storage/impl_db2.py12:30
jd__sileht: fair enough12:31
jd__if anyone wants to fix it after, it can anywya12:32
silehtjd__, and the testr backtrace for litong http://paste.openstack.org/show/45016/12:33
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios  https://review.openstack.org/4307112:35
thomasmHey all12:43
thomasmjd__ I replied to your comment on that bug. I hope it makes more sense now. Please let me know if I should post things like that elsewhere. :)12:43
*** gordc has joined #openstack-metering12:45
boris-42jd__ ping12:48
jd__o/12:48
boris-42jd__ about this thing https://review.openstack.org/#/c/43289/1/openstack/common/jsonutils.py12:48
boris-42jd__ I mean this is not real fix.. =(12:49
boris-42jd__ because you don't fix place where we are getting this object12:50
jd__boris-42: where?12:50
jd__boris-42: what would be an example that would fail?12:50
boris-42I mean how we get in value this object with type xmlrpclib.DateTime12:51
boris-42jd__ we should fix code that there will be no such check at all12:52
jd__oh you mean remove this altogether?12:52
jd__I don't know where we get this kind of object from12:53
boris-42jd__ yeah analyze where we call this method with such type12:53
jd__IIRC I spotted use in nova12:53
boris-42jd__ and cleanup all12:53
jd__but admittedly that is not the purpose on the patch in the first place12:53
jd__that's another problem, requiring another fix12:53
boris-42jd__ could you pls add some warning?)12:53
jd__I don't think this patch deserve any warning12:54
boris-42jd__ ok12:54
jd__i.e. I'm trying to tackle Python 3 issue, the rest is up to who ever cares in this case ;-)12:54
boris-42=)12:54
boris-42ok12:54
boris-42but code becomes more dirty =)12:55
*** shaneduan[afk] is now known as shaneduan12:57
*** bpokorny has joined #openstack-metering13:04
*** shaneduan is now known as shaneduan[afk]13:06
*** complexmind_ has joined #openstack-metering13:14
*** complexmind has quit IRC13:14
*** complexmind_ is now known as complexmind13:14
*** complexmind has quit IRC13:15
*** complexmind has joined #openstack-metering13:15
flwangsileht: ping13:19
silehtflwang, pong13:20
flwangabout your coments if I will update my nickname13:20
flwangsileht: I'm not sure what's the best practice13:20
flwangcan I submit a new separate patch to fix it? given you and jd__ has given +213:21
flwangor i have to fix it in this patch?13:21
*** dina_belova has quit IRC13:22
silehtflwang, just fix this patch I'm sure you will get your +2  again13:22
flwangsileht: okay13:23
flwangI will submit a new patch soon and let you know, thanks for your reviewing13:23
*** SergeyLukjanov has quit IRC13:30
*** alexb_ has joined #openstack-metering13:43
*** changbl has quit IRC13:48
*** fnaval_ has quit IRC13:54
*** SergeyLukjanov has joined #openstack-metering13:56
*** dina_belova has joined #openstack-metering13:56
*** shaneduan[afk] is now known as shaneduan13:57
openstackgerritFei Long Wang proposed a change to openstack/ceilometer: Fixes non-string metadata query issue  https://review.openstack.org/3879513:57
thomasmdhellmann_, Thanks for your comments on that bug. That makes sense. So, then more precision on the timestamp would hardly be of any use when we're expecting duplicates anyway?13:59
*** dina_belova has quit IRC14:05
*** shaneduan is now known as shaneduan[afk]14:06
*** dina_belova has joined #openstack-metering14:07
*** fnaval_ has joined #openstack-metering14:12
*** nsaje1 has quit IRC14:12
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: api: update v1 for Flask >= 0.10  https://review.openstack.org/4347814:21
*** SergeyLukjanov has quit IRC14:24
*** dina_belova has quit IRC14:24
*** terriyu has joined #openstack-metering14:25
*** SergeyLukjanov has joined #openstack-metering14:25
*** dina_belova has joined #openstack-metering14:26
openstackgerritA change was merged to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation  https://review.openstack.org/4341314:29
flwangjd__: ping14:38
flwangdhellmann_: ping14:39
jd__flwang: ?14:39
flwangjd__: may I get your approval again for this https://review.openstack.org/#/c/38795/ ? I fixed my nickname issue :)14:40
flwangjd__: got, thanks14:42
jd__you're welcome14:42
*** changbl has joined #openstack-metering14:46
*** anteaya is now known as anTeaya14:51
*** anTeaya is now known as ColinMochrie14:56
*** SergeyLukjanov has quit IRC15:01
*** SergeyLukjanov has joined #openstack-metering15:02
*** dina_bel_ has joined #openstack-metering15:03
*** dina_belova has quit IRC15:03
jd__boris-42: btw did you have a chance to look at the sqlite test in oslo?15:11
*** litong has joined #openstack-metering15:11
openstackgerritMonsyne Dragon proposed a change to openstack/ceilometer: Add configuration-driven conversion to Events  https://review.openstack.org/4271315:15
openstackgerritA change was merged to openstack/ceilometer: Fixes non-string metadata query issue  https://review.openstack.org/3879515:18
*** lsmola has quit IRC15:21
*** alexb_ has quit IRC15:25
*** SergeyLukjanov has quit IRC15:29
*** tasdomas is now known as tasdomas_afk15:39
*** shaneduan[afk] is now known as shaneduan15:39
*** SergeyLukjanov has joined #openstack-metering15:44
*** ColinMochrie has quit IRC15:49
*** complexmind has quit IRC15:54
openstackgerritlitong01 proposed a change to openstack/ceilometer: install manual last few sections format needs to be fixed.  https://review.openstack.org/4349915:58
boris-42jd__ didn't have sorry but I will check!=)16:05
*** shaneduan is now known as shaneduan[afk]16:11
*** dina_bel_ has quit IRC16:12
*** dina_belova has joined #openstack-metering16:20
*** dina_belova has quit IRC16:22
*** dina_belova has joined #openstack-metering16:22
*** anteaya has joined #openstack-metering16:29
*** shaneduan[afk] is now known as shaneduan16:32
*** anteaya is now known as ColinMochrie16:33
*** dina_belova has quit IRC16:39
*** SergeyLukjanov has quit IRC16:41
*** Alexei_987 has quit IRC17:14
*** dina_belova has joined #openstack-metering17:14
*** sdake_ has joined #openstack-metering17:16
*** sdake_ has quit IRC17:28
*** sdake_ has joined #openstack-metering17:36
*** zul has quit IRC17:58
*** lovzan has joined #openstack-metering18:01
lovzanhi18:01
*** maksimov has joined #openstack-metering18:04
*** SergeyLukjanov has joined #openstack-metering18:07
*** dina_belova has quit IRC18:15
*** zul has joined #openstack-metering18:34
openstackgerritA change was merged to openstack/ceilometer: rpc: reduce sleep time  https://review.openstack.org/4344018:39
*** dina_belova has joined #openstack-metering18:40
*** shaneduan is now known as shaneduan[afk]19:07
*** changbl has quit IRC19:18
*** maksimov has quit IRC19:25
*** zul has quit IRC19:53
*** yolanda has quit IRC19:55
*** shaneduan[afk] is now known as shaneduan19:58
*** tasdomas_afk is now known as tasdomas19:58
*** alexb_ has joined #openstack-metering20:30
*** changbl has joined #openstack-metering20:35
*** tasdomas is now known as tasdomas_afk20:43
*** SergeyLukjanov has quit IRC20:54
*** thomasm has quit IRC20:57
*** dina_belova has quit IRC21:04
*** alexb_ has quit IRC21:10
*** yjiang5 is now known as yjiang5_afk21:18
*** yjiang5_afk is now known as bmqq12321:21
*** litong has quit IRC21:28
openstackgerritlitong01 proposed a change to openstack/ceilometer: install manual last few sections format needs to be fixed  https://review.openstack.org/4349921:31
*** gordc1 has joined #openstack-metering21:32
*** gordc has quit IRC21:35
*** lovzan_ has joined #openstack-metering21:48
*** lovzan has quit IRC21:50
*** sdake_ has quit IRC22:03
*** gordc1 has quit IRC22:26
*** bpokorny has quit IRC22:31
*** changbl has quit IRC22:33
*** shaneduan is now known as shaneduan[afk]22:33
*** shaneduan[afk] is now known as shaneduan22:34
*** boris-42 has quit IRC22:41
*** sdake_ has joined #openstack-metering23:04
*** fnaval_ has quit IRC23:28
*** changbl has joined #openstack-metering23:37
*** alexb_ has joined #openstack-metering23:38
*** lovzan_ has quit IRC23:43
*** shaneduan is now known as shaneduan[afk]23:53

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