openstackgerrit | Jay Pipes proposed a change to openstack/ceilometer: Add a specialized Event Type model and db table https://review.openstack.org/41316 | 00:06 |
---|---|---|
*** thomasm has quit IRC | 00:28 | |
*** dina_belova has joined #openstack-metering | 00:37 | |
*** dina_belova has quit IRC | 00:42 | |
*** sdake_ has joined #openstack-metering | 00:52 | |
*** sdake_ has quit IRC | 01:08 | |
*** anteaya has quit IRC | 01:08 | |
*** changbl has joined #openstack-metering | 01:11 | |
*** sdake_ has joined #openstack-metering | 01:14 | |
*** alexb_ has quit IRC | 01:30 | |
*** sdake_ has quit IRC | 01:37 | |
*** dina_belova has joined #openstack-metering | 01:37 | |
*** fnaval_ has joined #openstack-metering | 01:41 | |
*** fnaval_ has quit IRC | 01:41 | |
*** fnaval_ has joined #openstack-metering | 01:41 | |
*** dina_belova has quit IRC | 01:42 | |
*** d34dh0r53 has joined #openstack-metering | 01:59 | |
*** d34dh0r53 has quit IRC | 02:01 | |
*** shang has joined #openstack-metering | 02:06 | |
*** terriyu has quit IRC | 02:07 | |
*** alexb_ has joined #openstack-metering | 02:09 | |
*** shang_ has joined #openstack-metering | 02:12 | |
*** shang has quit IRC | 02:15 | |
*** maksimov has quit IRC | 02:24 | |
*** maksimov has joined #openstack-metering | 02:24 | |
*** dina_belova has joined #openstack-metering | 02:38 | |
*** dina_belova has quit IRC | 02:43 | |
*** maksimov has quit IRC | 02:53 | |
*** maksimov has joined #openstack-metering | 02:54 | |
*** dina_belova has joined #openstack-metering | 03:39 | |
*** dina_belova has quit IRC | 03:44 | |
*** shaneduan[afk] is now known as shaneduan | 04:37 | |
*** dina_belova has joined #openstack-metering | 04:39 | |
*** boris-42 has joined #openstack-metering | 04:42 | |
*** dina_belova has quit IRC | 04:44 | |
*** alexb_ has quit IRC | 04:57 | |
*** dina_belova has joined #openstack-metering | 05:40 | |
*** dina_belova has quit IRC | 05:44 | |
openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name https://review.openstack.org/42715 | 05:47 |
*** tasdomas_afk is now known as tasdomas | 06:01 | |
*** shaneduan is now known as shaneduan[afk] | 06:01 | |
openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/43405 | 06:04 |
openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name https://review.openstack.org/42715 | 06:06 |
*** shaneduan[afk] is now known as shaneduan | 06:07 | |
*** yolanda has joined #openstack-metering | 06:32 | |
*** dina_belova has joined #openstack-metering | 06:40 | |
*** SergeyLukjanov has joined #openstack-metering | 06:42 | |
*** dina_belova has quit IRC | 06:43 | |
*** maksimov has quit IRC | 06:47 | |
*** SergeyLukjanov has quit IRC | 06:58 | |
*** evanjfraser has quit IRC | 07:10 | |
*** shaneduan is now known as shaneduan[afk] | 07:20 | |
*** dina_belova has joined #openstack-metering | 07:41 | |
*** shardy_afk is now known as shardy | 07:41 | |
*** dina_belova has quit IRC | 07:45 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation https://review.openstack.org/43413 | 07:48 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: 'and' and 'or' operations for alarms combination https://review.openstack.org/42832 | 07:48 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Add alarm combination API https://review.openstack.org/41971 | 07:48 |
*** shaneduan[afk] is now known as shaneduan | 07:50 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: 'and' and 'or' operations for alarms combination https://review.openstack.org/42832 | 07:52 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Add alarm combination API https://review.openstack.org/41971 | 07:52 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation https://review.openstack.org/43413 | 07:52 |
*** shaneduan is now known as shaneduan[afk] | 08:04 | |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: api: allow usage of resource_metadata in query https://review.openstack.org/43310 | 08:31 |
*** dina_belova has joined #openstack-metering | 08:42 | |
*** Alexei_987 has joined #openstack-metering | 08:45 | |
*** dina_belova has quit IRC | 08:46 | |
*** SergeyLukjanov has joined #openstack-metering | 08:51 | |
*** dina_belova has joined #openstack-metering | 08:52 | |
openstackgerrit | A change was merged to openstack/ceilometer: Doc: improve doc about Nova measurements https://review.openstack.org/42842 | 08:55 |
*** shaneduan[afk] is now known as shaneduan | 08:56 | |
*** shaneduan is now known as shaneduan[afk] | 09:05 | |
*** vvechkanov2 has joined #openstack-metering | 09:11 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: 'and' and 'or' operations for alarms combination https://review.openstack.org/42832 | 09:11 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Add alarm combination API https://review.openstack.org/41971 | 09:11 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation https://review.openstack.org/43413 | 09:11 |
openstackgerrit | A change was merged to openstack/ceilometer: db2 support https://review.openstack.org/41123 | 09:18 |
*** sdake_ has joined #openstack-metering | 09:51 | |
*** shaneduan[afk] is now known as shaneduan | 09: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 IRC | 10:15 | |
*** nsaje1 has joined #openstack-metering | 10:20 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Add missing import of _ https://review.openstack.org/43426 | 10:21 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination/groupby is missing https://review.openstack.org/43427 | 10:21 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination support is missing https://review.openstack.org/42889 | 10:21 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios https://review.openstack.org/43071 | 10:21 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Move sqlachemy tests only in test_impl_sqlachemy https://review.openstack.org/42890 | 10:21 |
sileht | jd__, I get some issues with the testscenarios and db2, some tests won't pass... in db2 | 10:22 |
jd__ | sileht: tell me more about it | 10:23 |
sileht | jd__, I have uploded the review to share code and logs | 10:23 |
sileht | jd__, for example test_get_resources don't return last/first_sample_timestamp | 10:23 |
sileht | jd__, before my testscenarios change the test was overrided to skip the last/first_sample_timestamp check | 10:25 |
jd__ | ah | 10:25 |
jd__ | can you make the test return a NotImplemented for DB2? | 10:26 |
sileht | jd__, 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 |
sileht | jd__, I haven't find a clean ways to do this | 10:28 |
sileht | jd__, but this is the first time I need to do this | 10: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 solid | 10:29 |
sileht | jd__, 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 |
sileht | sniff :( | 10:31 |
jd__ | the db2 driver will not run these test and will not be compliant | 10:32 |
jd__ | OTOH we don't really care about DB2, only litong does, and he'll be able to fix it | 10:32 |
jd__ | we'll just have to point this at him I guess | 10:32 |
sileht | I have enabled the functionnal API to db2 too and most of them failed too, their return ugly 'list index out of range' | 10:34 |
openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Added pollsters for the hardware agent https://review.openstack.org/43074 | 10:34 |
openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Adding hardware-agent https://review.openstack.org/43072 | 10:34 |
openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Added hardware agent's inspector and snmp implementation https://review.openstack.org/43073 | 10: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 worried | 10:36 |
jd__ | s/are/have/ | 10:36 |
sileht | jd__, what do you think about I just disable the db2 scenarios until it pass all tests ? | 10:37 |
jd__ | sileht: that's my preferred solution | 10:37 |
jd__ | well at least the one failing :) | 10:37 |
sileht | I will add a big warning so | 10:37 |
*** sdake_ has quit IRC | 10:41 | |
sileht | jd__, 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 |
openstackgerrit | Fei Long Wang proposed a change to openstack/ceilometer: Fix empty metadata issue of instance https://review.openstack.org/41818 | 10:50 |
sileht | jd__, ok I have found: maxConns=32 | 10:51 |
*** shaneduan[afk] is now known as shaneduan | 10:56 | |
*** dina_belova has quit IRC | 10:56 | |
*** complexmind has joined #openstack-metering | 11:00 | |
*** dina_belova has joined #openstack-metering | 11:02 | |
*** SergeyLukjanov has quit IRC | 11:04 | |
*** shaneduan is now known as shaneduan[afk] | 11:05 | |
*** dina_belova has quit IRC | 11:06 | |
openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name https://review.openstack.org/42715 | 11:28 |
openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Added pollsters for the hardware agent https://review.openstack.org/43074 | 11:31 |
openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Added hardware agent's inspector and snmp implementation https://review.openstack.org/43073 | 11:31 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination/groupby is missing https://review.openstack.org/43427 | 11:33 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios https://review.openstack.org/43071 | 11:33 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Move sqlachemy tests only in test_impl_sqlachemy https://review.openstack.org/42890 | 11:33 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Raise Error when pagination/groupby is missing https://review.openstack.org/43427 | 11:38 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios https://review.openstack.org/43071 | 11:38 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Move sqlachemy tests only in test_impl_sqlachemy https://review.openstack.org/42890 | 11:38 |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: rpc: remove useless sleep https://review.openstack.org/43440 | 11:40 |
*** anteaya has joined #openstack-metering | 11:50 | |
*** shaneduan[afk] is now known as shaneduan | 11:56 | |
*** dina_belova has joined #openstack-metering | 12:04 | |
*** SergeyLukjanov has joined #openstack-metering | 12:05 | |
*** shaneduan is now known as shaneduan[afk] | 12:06 | |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: rpc: reduce sleep time https://review.openstack.org/43440 | 12:10 |
*** thomasm has joined #openstack-metering | 12:21 | |
*** zul has joined #openstack-metering | 12:24 | |
openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Extra indexes cleanup https://review.openstack.org/42716 | 12:29 |
sileht | jd__, 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.py | 12:30 |
jd__ | sileht: fair enough | 12:31 |
jd__ | if anyone wants to fix it after, it can anywya | 12:32 |
sileht | jd__, and the testr backtrace for litong http://paste.openstack.org/show/45016/ | 12:33 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Refactored storage tests to use testscenarios https://review.openstack.org/43071 | 12:35 |
thomasm | Hey all | 12:43 |
thomasm | jd__ 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-metering | 12:45 | |
boris-42 | jd__ ping | 12:48 |
jd__ | o/ | 12:48 |
boris-42 | jd__ about this thing https://review.openstack.org/#/c/43289/1/openstack/common/jsonutils.py | 12:48 |
boris-42 | jd__ I mean this is not real fix.. =( | 12:49 |
boris-42 | jd__ because you don't fix place where we are getting this object | 12:50 |
jd__ | boris-42: where? | 12:50 |
jd__ | boris-42: what would be an example that would fail? | 12:50 |
boris-42 | I mean how we get in value this object with type xmlrpclib.DateTime | 12:51 |
boris-42 | jd__ we should fix code that there will be no such check at all | 12:52 |
jd__ | oh you mean remove this altogether? | 12:52 |
jd__ | I don't know where we get this kind of object from | 12:53 |
boris-42 | jd__ yeah analyze where we call this method with such type | 12:53 |
jd__ | IIRC I spotted use in nova | 12:53 |
boris-42 | jd__ and cleanup all | 12:53 |
jd__ | but admittedly that is not the purpose on the patch in the first place | 12:53 |
jd__ | that's another problem, requiring another fix | 12:53 |
boris-42 | jd__ could you pls add some warning?) | 12:53 |
jd__ | I don't think this patch deserve any warning | 12:54 |
boris-42 | jd__ ok | 12: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-42 | ok | 12:54 |
boris-42 | but code becomes more dirty =) | 12:55 |
*** shaneduan[afk] is now known as shaneduan | 12:57 | |
*** bpokorny has joined #openstack-metering | 13:04 | |
*** shaneduan is now known as shaneduan[afk] | 13:06 | |
*** complexmind_ has joined #openstack-metering | 13:14 | |
*** complexmind has quit IRC | 13:14 | |
*** complexmind_ is now known as complexmind | 13:14 | |
*** complexmind has quit IRC | 13:15 | |
*** complexmind has joined #openstack-metering | 13:15 | |
flwang | sileht: ping | 13:19 |
sileht | flwang, pong | 13:20 |
flwang | about your coments if I will update my nickname | 13:20 |
flwang | sileht: I'm not sure what's the best practice | 13:20 |
flwang | can I submit a new separate patch to fix it? given you and jd__ has given +2 | 13:21 |
flwang | or i have to fix it in this patch? | 13:21 |
*** dina_belova has quit IRC | 13:22 | |
sileht | flwang, just fix this patch I'm sure you will get your +2 again | 13:22 |
flwang | sileht: okay | 13:23 |
flwang | I will submit a new patch soon and let you know, thanks for your reviewing | 13:23 |
*** SergeyLukjanov has quit IRC | 13:30 | |
*** alexb_ has joined #openstack-metering | 13:43 | |
*** changbl has quit IRC | 13:48 | |
*** fnaval_ has quit IRC | 13:54 | |
*** SergeyLukjanov has joined #openstack-metering | 13:56 | |
*** dina_belova has joined #openstack-metering | 13:56 | |
*** shaneduan[afk] is now known as shaneduan | 13:57 | |
openstackgerrit | Fei Long Wang proposed a change to openstack/ceilometer: Fixes non-string metadata query issue https://review.openstack.org/38795 | 13:57 |
thomasm | dhellmann_, 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 IRC | 14:05 | |
*** shaneduan is now known as shaneduan[afk] | 14:06 | |
*** dina_belova has joined #openstack-metering | 14:07 | |
*** fnaval_ has joined #openstack-metering | 14:12 | |
*** nsaje1 has quit IRC | 14:12 | |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: api: update v1 for Flask >= 0.10 https://review.openstack.org/43478 | 14:21 |
*** SergeyLukjanov has quit IRC | 14:24 | |
*** dina_belova has quit IRC | 14:24 | |
*** terriyu has joined #openstack-metering | 14:25 | |
*** SergeyLukjanov has joined #openstack-metering | 14:25 | |
*** dina_belova has joined #openstack-metering | 14:26 | |
openstackgerrit | A change was merged to openstack/ceilometer: Use timeutils.utcnow in alarm threshold evaluation https://review.openstack.org/43413 | 14:29 |
flwang | jd__: ping | 14:38 |
flwang | dhellmann_: ping | 14:39 |
jd__ | flwang: ? | 14:39 |
flwang | jd__: may I get your approval again for this https://review.openstack.org/#/c/38795/ ? I fixed my nickname issue :) | 14:40 |
flwang | jd__: got, thanks | 14:42 |
jd__ | you're welcome | 14:42 |
*** changbl has joined #openstack-metering | 14:46 | |
*** anteaya is now known as anTeaya | 14:51 | |
*** anTeaya is now known as ColinMochrie | 14:56 | |
*** SergeyLukjanov has quit IRC | 15:01 | |
*** SergeyLukjanov has joined #openstack-metering | 15:02 | |
*** dina_bel_ has joined #openstack-metering | 15:03 | |
*** dina_belova has quit IRC | 15:03 | |
jd__ | boris-42: btw did you have a chance to look at the sqlite test in oslo? | 15:11 |
*** litong has joined #openstack-metering | 15:11 | |
openstackgerrit | Monsyne Dragon proposed a change to openstack/ceilometer: Add configuration-driven conversion to Events https://review.openstack.org/42713 | 15:15 |
openstackgerrit | A change was merged to openstack/ceilometer: Fixes non-string metadata query issue https://review.openstack.org/38795 | 15:18 |
*** lsmola has quit IRC | 15:21 | |
*** alexb_ has quit IRC | 15:25 | |
*** SergeyLukjanov has quit IRC | 15:29 | |
*** tasdomas is now known as tasdomas_afk | 15:39 | |
*** shaneduan[afk] is now known as shaneduan | 15:39 | |
*** SergeyLukjanov has joined #openstack-metering | 15:44 | |
*** ColinMochrie has quit IRC | 15:49 | |
*** complexmind has quit IRC | 15:54 | |
openstackgerrit | litong01 proposed a change to openstack/ceilometer: install manual last few sections format needs to be fixed. https://review.openstack.org/43499 | 15:58 |
boris-42 | jd__ didn't have sorry but I will check!=) | 16:05 |
*** shaneduan is now known as shaneduan[afk] | 16:11 | |
*** dina_bel_ has quit IRC | 16:12 | |
*** dina_belova has joined #openstack-metering | 16:20 | |
*** dina_belova has quit IRC | 16:22 | |
*** dina_belova has joined #openstack-metering | 16:22 | |
*** anteaya has joined #openstack-metering | 16:29 | |
*** shaneduan[afk] is now known as shaneduan | 16:32 | |
*** anteaya is now known as ColinMochrie | 16:33 | |
*** dina_belova has quit IRC | 16:39 | |
*** SergeyLukjanov has quit IRC | 16:41 | |
*** Alexei_987 has quit IRC | 17:14 | |
*** dina_belova has joined #openstack-metering | 17:14 | |
*** sdake_ has joined #openstack-metering | 17:16 | |
*** sdake_ has quit IRC | 17:28 | |
*** sdake_ has joined #openstack-metering | 17:36 | |
*** zul has quit IRC | 17:58 | |
*** lovzan has joined #openstack-metering | 18:01 | |
lovzan | hi | 18:01 |
*** maksimov has joined #openstack-metering | 18:04 | |
*** SergeyLukjanov has joined #openstack-metering | 18:07 | |
*** dina_belova has quit IRC | 18:15 | |
*** zul has joined #openstack-metering | 18:34 | |
openstackgerrit | A change was merged to openstack/ceilometer: rpc: reduce sleep time https://review.openstack.org/43440 | 18:39 |
*** dina_belova has joined #openstack-metering | 18:40 | |
*** shaneduan is now known as shaneduan[afk] | 19:07 | |
*** changbl has quit IRC | 19:18 | |
*** maksimov has quit IRC | 19:25 | |
*** zul has quit IRC | 19:53 | |
*** yolanda has quit IRC | 19:55 | |
*** shaneduan[afk] is now known as shaneduan | 19:58 | |
*** tasdomas_afk is now known as tasdomas | 19:58 | |
*** alexb_ has joined #openstack-metering | 20:30 | |
*** changbl has joined #openstack-metering | 20:35 | |
*** tasdomas is now known as tasdomas_afk | 20:43 | |
*** SergeyLukjanov has quit IRC | 20:54 | |
*** thomasm has quit IRC | 20:57 | |
*** dina_belova has quit IRC | 21:04 | |
*** alexb_ has quit IRC | 21:10 | |
*** yjiang5 is now known as yjiang5_afk | 21:18 | |
*** yjiang5_afk is now known as bmqq123 | 21:21 | |
*** litong has quit IRC | 21:28 | |
openstackgerrit | litong01 proposed a change to openstack/ceilometer: install manual last few sections format needs to be fixed https://review.openstack.org/43499 | 21:31 |
*** gordc1 has joined #openstack-metering | 21:32 | |
*** gordc has quit IRC | 21:35 | |
*** lovzan_ has joined #openstack-metering | 21:48 | |
*** lovzan has quit IRC | 21:50 | |
*** sdake_ has quit IRC | 22:03 | |
*** gordc1 has quit IRC | 22:26 | |
*** bpokorny has quit IRC | 22:31 | |
*** changbl has quit IRC | 22:33 | |
*** shaneduan is now known as shaneduan[afk] | 22:33 | |
*** shaneduan[afk] is now known as shaneduan | 22:34 | |
*** boris-42 has quit IRC | 22:41 | |
*** sdake_ has joined #openstack-metering | 23:04 | |
*** fnaval_ has quit IRC | 23:28 | |
*** changbl has joined #openstack-metering | 23:37 | |
*** alexb_ has joined #openstack-metering | 23:38 | |
*** lovzan_ has quit IRC | 23: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!