| *** matsuhashi has joined #openstack-ceilometer | 00:21 | |
| *** zhikunliu has joined #openstack-ceilometer | 00:31 | |
| *** nati_ueno has quit IRC | 00:53 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Change pipeline_manager to instance attribute in hooks https://review.openstack.org/98295 | 01:04 |
|---|---|---|
| *** liusheng has joined #openstack-ceilometer | 01:08 | |
| *** liusheng has quit IRC | 01:14 | |
| *** liusheng has joined #openstack-ceilometer | 01:16 | |
| *** nati_ueno has joined #openstack-ceilometer | 01:19 | |
| *** nati_ueno has quit IRC | 01:20 | |
| *** liusheng has quit IRC | 01:20 | |
| *** liusheng has joined #openstack-ceilometer | 01:21 | |
| *** nosnos has joined #openstack-ceilometer | 01:43 | |
| *** julim has quit IRC | 02:26 | |
| *** mgagne1 is now known as mgagne | 02:30 | |
| *** nosnos has quit IRC | 03:18 | |
| *** matsuhashi has quit IRC | 04:04 | |
| *** nosnos has joined #openstack-ceilometer | 04:06 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:13 | |
| *** r3pl4y has joined #openstack-ceilometer | 04:37 | |
| *** r3pl4y has quit IRC | 04:45 | |
| *** psharma has joined #openstack-ceilometer | 04:53 | |
| *** Longgeek has joined #openstack-ceilometer | 05:04 | |
| *** renlt has joined #openstack-ceilometer | 05:34 | |
| *** _nadya_ has joined #openstack-ceilometer | 05:41 | |
| *** _nadya_ has quit IRC | 05:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:01 |
| liusheng | Hi, is there anyone can run unit tests in ceilometer.tests.event.test_endpoint.TestEventEndpoint ? | 06:25 |
| zqfan | liusheng: the latest code seems ok, is there any problem? | 06:28 |
| liusheng | zqfan, but when I run it, it raise: NoSuchGroupError: no such group: notification | 06:29 |
| liusheng | zqfan, I do have no idea about that :( | 06:29 |
| *** _nadya_ has joined #openstack-ceilometer | 06:33 | |
| liusheng | zqfan, is it ok in your environment? | 06:33 |
| zqfan | liusheng: please paste the stack trace | 06:39 |
| zqfan | liusheng: it can be the same problem which I told you, somewhere in the source code uses an option before import it | 06:40 |
| liusheng | zqfan, trace has posted to you | 06:42 |
| liusheng | zqfan, Privately | 06:42 |
| liusheng | zqfan, the problem is occured when "self.CONF.set_override("store_events", True, group="notification")" in setup() method, but there is similar in ceilometer.tests.test_notification.TestNotification, it is OK! | 06:45 |
| liusheng | zqfan, a similar case | 06:45 |
| *** Longgeek has quit IRC | 06:46 | |
| zqfan | liusheng, that because test_notification imports ceilometer.notification which register(power than import) notification group | 06:49 |
| zqfan | liusheng, the problem is that, is it necessary that ceilometer project will support nosetests? currently, tox is ok to execute single test (it does something to solve the problem but I don't know what is it yet) | 06:52 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Add missing explicit cfg option import https://review.openstack.org/98698 | 06:56 |
| sileht | liusheng, zqfan does that ^ fix your issue | 06:56 |
| sileht | ? | 06:56 |
| liusheng | sileht, I am trying to use tox | 06:56 |
| *** nosnos has quit IRC | 06:57 | |
| sileht | zqfan, I don't fix we can use nose, because of an issue with testscenarions and nose | 06:57 |
| sileht | fix/thing | 06:57 |
| sileht | the nose/testscenarios issue: https://bugs.launchpad.net/testscenarios/+bug/872887 | 06:58 |
| zqfan | sileht, there are many such problems since tox has been used for a long while, and many test even source code doesn't obey the rule "import before use option", are we going to fix them all? | 06:58 |
| llu | sileht: ping | 06:59 |
| *** r3pl4y has joined #openstack-ceilometer | 06:59 | |
| sileht | zqfan, only we someone needs it, we are not looking for all missing opt import | 07:00 |
| sileht | llu, o/ | 07:00 |
| llu | about your comment on https://review.openstack.org/#/c/98295/2/ceilometer/api/hooks.py | 07:00 |
| llu | I added a new parameter just following the DBHook convention, but actually no one is using that. Are you suggesting removing that? | 07:01 |
| sileht | llu, yes I guess we can | 07:02 |
| llu | ok, thanks | 07:02 |
| sileht | llu, I'm ask the question is this way, in case I have missed something needed for future patch | 07:02 |
| *** lsmola_ has joined #openstack-ceilometer | 07:05 | |
| llu | sileht: sorry, what do you mean by saying 'new attribute'? Do you refer to the new parameter when calling __init__() or do you refer to something else? | 07:06 |
| llu | I changed the pipeline_manager from class attribute to instance attribute so that we can correct mock the unittest | 07:06 |
| sileht | llu, sorry, I means 'new argument' | 07:07 |
| llu | sileht, ok I got that. | 07:08 |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Change pipeline_manager to instance attribute in hooks https://review.openstack.org/98295 | 07:11 |
| *** zhikunliu has quit IRC | 07:13 | |
| *** Longgeek has joined #openstack-ceilometer | 07:16 | |
| *** Longgeek has quit IRC | 07:22 | |
| *** r0j4z0 has quit IRC | 07:55 | |
| *** amalagon_ has joined #openstack-ceilometer | 07:56 | |
| *** amalagon has quit IRC | 07:58 | |
| *** openstackgerrit has quit IRC | 08:02 | |
| *** vrovachev has joined #openstack-ceilometer | 08:03 | |
| *** openstackgerrit has joined #openstack-ceilometer | 08:06 | |
| *** renlt has quit IRC | 08:17 | |
| *** eglynn has joined #openstack-ceilometer | 08:19 | |
| *** kun_huang has joined #openstack-ceilometer | 08:20 | |
| *** _nadya_ has quit IRC | 08:30 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Fix notification for NotImplemented record_events https://review.openstack.org/97669 | 08:46 |
| *** Alexei_987 has joined #openstack-ceilometer | 08:48 | |
| *** _nadya_ has joined #openstack-ceilometer | 09:01 | |
| *** IvanBerezovskiy has joined #openstack-ceilometer | 09:02 | |
| *** kun_huang has quit IRC | 09:04 | |
| *** _cjones__ has joined #openstack-ceilometer | 09:06 | |
| *** Longgeek has joined #openstack-ceilometer | 09:07 | |
| DinaBelova | folks. o/ | 09:10 |
| DinaBelova | ildikov_, eglynn - may you please take a look on the https://review.openstack.org/#/c/91408 - and the change it depends on (due to testing issues). Igor has day off today - so I'll try to fix your comments and recheck failing gates :D | 09:12 |
| *** Longgeek has quit IRC | 09:12 | |
| DinaBelova | it's almost done, I guess - Igor fixed at least all my and Nadya's comments :) it were lots of them :) | 09:12 |
| *** _cjones_ has quit IRC | 09:14 | |
| *** zqfan has quit IRC | 09:14 | |
| *** zqfan has joined #openstack-ceilometer | 09:15 | |
| eglynn | DinaBelova: looking now | 09:16 |
| DinaBelova | eglynn, thank you sir! | 09:16 |
| DinaBelova | as we all want to land it in j1 :) | 09:16 |
| openstackgerrit | Rikimaru Honjo proposed a change to openstack/ceilometer: Adds caches for image and flavor in compute agent https://review.openstack.org/98301 | 09:17 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits hbase storage code base https://review.openstack.org/98721 | 09:22 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits hbase storage code base https://review.openstack.org/98721 | 09:22 |
| DinaBelova | eglynn, I don't want to run recheck on the https://review.openstack.org/#/c/95640/ and https://review.openstack.org/#/c/96699/ - as they need to be reverified. If you'll add your +2/A it might help to run gate jobs without the rechecks.... | 09:25 |
| DinaBelova | eglynn, may you do that, sir? as gate is still unstable really... | 09:25 |
| eglynn | DinaBelova: no probs, doing it now | 09:25 |
| DinaBelova | eglynn, thanks! | 09:26 |
| *** matsuhashi has quit IRC | 09:31 | |
| *** matsuhashi has joined #openstack-ceilometer | 09:31 | |
| DinaBelova | sileht, thanks for the review of hbase events :) I've published the answer in the comments | 09:32 |
| *** kun_huang has joined #openstack-ceilometer | 09:33 | |
| *** zqfan is now known as zqfan_sleep | 09:33 | |
| ildikov_ | DinaBelova: why does the HBase patch depend on the DBManagers refactor one? | 09:35 |
| *** matsuhashi has quit IRC | 09:39 | |
| *** matsuhashi has joined #openstack-ceilometer | 09:40 | |
| *** erecio has quit IRC | 09:42 | |
| *** erecio has joined #openstack-ceilometer | 09:43 | |
| DinaBelova | ildikov_, the answer is in the sileht's quesiton really | 09:46 |
| DinaBelova | as currently (in our code) it's almost impossible to pass some (one) test for one backend | 09:47 |
| DinaBelova | Alexei_987's change makes it easy | 09:47 |
| DinaBelova | and available for the classes, methods, etc. | 09:47 |
| ildikov_ | DinaBelova: hmm, I remembered that we had workarounds for this before, but maybe I mix it with smth else | 09:50 |
| DinaBelova | well, yes, I don't know for sure, but Alexei_987 and Igor said me that's the best way to do this here | 09:51 |
| *** _nadya_ has quit IRC | 09:51 | |
| ildikov_ | DinaBelova: ok | 09:56 |
| *** kun_huang has quit IRC | 09:58 | |
| *** kun_huang has joined #openstack-ceilometer | 10:09 | |
| *** _nadya_ has joined #openstack-ceilometer | 10:17 | |
| DinaBelova | eglynn, thanks for the comments! please take a look on my answers - ityaptin will provide new patch set. As for the literal '+' - he'll investigate this moment too :) | 10:17 |
| eglynn | DinaBelova: cool, thanks! | 10:17 |
| DinaBelova | really we forgot that Igor (and all Ukranians) have day-off today :D | 10:18 |
| DinaBelova | so it was like a surprise :) | 10:18 |
| *** kun_huang has quit IRC | 10:22 | |
| *** Longgeek has joined #openstack-ceilometer | 10:31 | |
| *** r3pl4y has quit IRC | 10:38 | |
| *** r3pl4y has joined #openstack-ceilometer | 10:39 | |
| openstackgerrit | Pooja Tiwari proposed a change to openstack/ceilometer: Alarm default description update https://review.openstack.org/97777 | 10:41 |
| openstackgerrit | Pooja Tiwari proposed a change to openstack/ceilometer: Alarm default description update https://review.openstack.org/97777 | 10:50 |
| *** matsuhashi has quit IRC | 10:51 | |
| *** r3pl4y_ has joined #openstack-ceilometer | 10:55 | |
| *** r3pl4y has quit IRC | 10:57 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:58 | |
| *** kun_huang has joined #openstack-ceilometer | 11:03 | |
| *** _nadya__ has joined #openstack-ceilometer | 11:16 | |
| *** _nadya_ has quit IRC | 11:16 | |
| *** matsuhashi has quit IRC | 11:18 | |
| *** matsuhashi has joined #openstack-ceilometer | 11:19 | |
| *** kun_huang has quit IRC | 11:20 | |
| DinaBelova | Hehe :) | 11:22 |
| DinaBelova | what I see in the qa channel :) | 11:22 |
| DinaBelova | [15:16:48] <vrovachev> I see several strange issues with devstack+tempest now (at least speaking about ceilometer testing) | 11:22 |
| DinaBelova | [15:17:33] <vrovachev> https://review.openstack.org/#/c/98344/ and https://review.openstack.org/#/c/95640/ seem to have one root for failings | 11:22 |
| DinaBelova | [15:17:54] <vrovachev> so tempest job and ceilometerclient gate are failing because of one reason | 11:22 |
| DinaBelova | [15:18:01] <vrovachev> although tests are different | 11:22 |
| DinaBelova | [15:18:21] <vrovachev> in console.log it's seens that "wrong nova credentials" are provided | 11:22 |
| DinaBelova | So it looks like our ceilometerclient changes cannot be merged due to https://bugs.launchpad.net/tempest/+bug/1298472 somehow | 11:23 |
| *** dmsimard_away is now known as dmsimard | 11:24 | |
| *** matsuhashi has quit IRC | 11:24 | |
| *** ityaptin has quit IRC | 11:25 | |
| openstackgerrit | Arthur Svechnikov proposed a change to openstack/python-ceilometerclient: Add methods to resource classes https://review.openstack.org/91554 | 11:27 |
| *** Ruetobas has quit IRC | 11:51 | |
| *** Ruetobas has joined #openstack-ceilometer | 11:57 | |
| *** Ruetobas has quit IRC | 12:01 | |
| *** jdob has joined #openstack-ceilometer | 12:07 | |
| *** erecio has quit IRC | 12:08 | |
| *** Ruetobas has joined #openstack-ceilometer | 12:08 | |
| *** inc0 has joined #openstack-ceilometer | 12:12 | |
| *** _nadya__ has quit IRC | 12:12 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits hbase storage code base https://review.openstack.org/98721 | 12:14 |
| *** _nadya_ has joined #openstack-ceilometer | 12:14 | |
| *** _nadya_ has quit IRC | 12:23 | |
| *** thomasem has quit IRC | 12:25 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:25 | |
| *** erecio has joined #openstack-ceilometer | 12:35 | |
| *** ildikov_ has quit IRC | 12:38 | |
| *** gordc has joined #openstack-ceilometer | 12:48 | |
| DinaBelova | gordc, eglynn - btw - may you please take a look on the https://review.openstack.org/#/c/95613/3 as it's the dependency (as said, to avoid dirty code for tests) for hbase events change :) | 12:55 |
| *** prad has joined #openstack-ceilometer | 12:55 | |
| *** rbowen has joined #openstack-ceilometer | 12:59 | |
| *** julim has joined #openstack-ceilometer | 13:01 | |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: Refactor tests to remove direct access to test DBManagers https://review.openstack.org/95613 | 13:03 |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: [HBase] Implement events on HBase https://review.openstack.org/91408 | 13:03 |
| eglynn | DinaBelova: looking | 13:08 |
| DinaBelova | eglynn, thanks! | 13:08 |
| *** thomasem has joined #openstack-ceilometer | 13:15 | |
| _nadya_ | eglynn, DinaBelova, oh, please, take a look at my comments in patch set 8 | 13:16 |
| * DinaBelova looking | 13:16 | |
| DinaBelova | _nadya_, hehe, ityaptin provided other timestamp method at all :) | 13:18 |
| DinaBelova | is it ok? :) | 13:18 |
| *** ityaptin has joined #openstack-ceilometer | 13:18 | |
| _nadya_ | no | 13:18 |
| _nadya_ | no.jpg | 13:18 |
| DinaBelova | :D | 13:19 |
| *** jaypipes has joined #openstack-ceilometer | 13:19 | |
| DinaBelova | ityaptin - please fix Nadya's comment about timestamp method :) | 13:19 |
| _nadya_ | ityaptin: Ilya please rename EVENTS_table to EVENT_table | 13:20 |
| ityaptin | _nadya_, ok) | 13:21 |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: Change using of limit argument in get_sample https://review.openstack.org/93585 | 13:21 |
| DinaBelova | ityaptin, you're quick :D | 13:22 |
| DinaBelova | oh, that's other change, sorry | 13:22 |
| ityaptin | It's other CR) | 13:22 |
| *** palar has joined #openstack-ceilometer | 13:24 | |
| *** cmart has joined #openstack-ceilometer | 13:25 | |
| *** ccrouch has joined #openstack-ceilometer | 13:26 | |
| *** Alexei_987 has quit IRC | 13:27 | |
| *** jaypipes has quit IRC | 13:28 | |
| _nadya_ | eglynn: one more comment from me to events patch-set 8. Please take a look when you have time | 13:28 |
| *** palar has quit IRC | 13:29 | |
| *** jaypipes has joined #openstack-ceilometer | 13:29 | |
| *** palar has joined #openstack-ceilometer | 13:30 | |
| eglynn | _nadya_: do you mean in https://review.openstack.org/#/c/91408/8/ceilometer/storage/impl_hbase.py ? | 13:30 |
| eglynn | _nadya_: looking | 13:30 |
| eglynn | _nadya_: ... a-ha, k, so addressed in a subsequent patch, that's cool | 13:31 |
| _nadya_ | eglynn: yep, I'll create a bug for this to be sure that it will be fixed | 13:31 |
| eglynn | _nadya_: cool, thanks! | 13:31 |
| amalagon_ | morning eglynn! | 13:32 |
| eglynn | _nadya_: are you going to also review version 9 of that events patchset? (... just pushed) | 13:33 |
| eglynn | _nadya_: i.e. ... https://review.openstack.org/#/c/91408/9/ceilometer/storage/impl_hbase.py | 13:33 |
| eglynn | amalagon_: goo afternoon! | 13:33 |
| eglynn | amalagon_: 1:1 time? | 13:33 |
| amalagon_ | sure | 13:33 |
| _nadya_ | eglynn: I'm ok with it except timestamp method. Ilya will upload a new one soon so I will wait for it | 13:34 |
| eglynn | _nadya_: cool, I'll hold off until patchset-10 in that case also | 13:34 |
| *** jaypipes has quit IRC | 13:34 | |
| *** jaypipes has joined #openstack-ceilometer | 13:35 | |
| *** jaypipes has quit IRC | 13:36 | |
| openstackgerrit | Christian Martinez proposed a change to openstack/ceilometer: Adding alarm list filtering by state and meter https://review.openstack.org/96582 | 13:36 |
| *** jaypipes has joined #openstack-ceilometer | 13:38 | |
| *** vrovachev has quit IRC | 13:39 | |
| *** jaypipes has quit IRC | 13:45 | |
| *** jaypipes has joined #openstack-ceilometer | 13:46 | |
| *** jaypipes has quit IRC | 13:49 | |
| *** jaypipes has joined #openstack-ceilometer | 13:50 | |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: [HBase] Implement events on HBase https://review.openstack.org/91408 | 13:54 |
| *** Longgeek has quit IRC | 13:55 | |
| *** Longgeek has joined #openstack-ceilometer | 13:55 | |
| *** rwsu has joined #openstack-ceilometer | 14:04 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Metering LoadBalancer as a Service https://review.openstack.org/95784 | 14:09 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer-specs: Minor formatting and typo fixes for spec https://review.openstack.org/98165 | 14:09 |
| *** _nadya_ has quit IRC | 14:10 | |
| *** ildikov has joined #openstack-ceilometer | 14:11 | |
| amalagon_ | hey jd__ o/ | 14:13 |
| *** _nadya_ has joined #openstack-ceilometer | 14:13 | |
| jd__ | hey amalagon_ | 14:13 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Update Measurement Docs for LBaaS https://review.openstack.org/98513 | 14:14 |
| amalagon_ | hi ~ I was wondering what the plans were on enabling keystone for gnocchi-api | 14:14 |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: [HBase] Implement events on HBase https://review.openstack.org/91408 | 14:23 |
| *** Alexei_987 has joined #openstack-ceilometer | 14:34 | |
| *** r3pl4y_ has quit IRC | 14:41 | |
| *** fnaval has joined #openstack-ceilometer | 14:45 | |
| jd__ | amalagon_: no plan for now | 14:45 |
| jd__ | but it's a 3 lines patches to enable authentication as we do in Ceilo | 14:45 |
| *** psharma has quit IRC | 14:48 | |
| *** erecio has quit IRC | 14:48 | |
| *** erecio has joined #openstack-ceilometer | 14:49 | |
| *** erecio has quit IRC | 14:49 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Metering LoadBalancer as a Service https://review.openstack.org/95784 | 14:51 |
| *** _nadya_ has quit IRC | 14:52 | |
| *** rongze has joined #openstack-ceilometer | 14:56 | |
| *** r3pl4y has joined #openstack-ceilometer | 14:59 | |
| *** thomasem has quit IRC | 15:00 | |
| *** fnaval has quit IRC | 15:00 | |
| *** fnaval has joined #openstack-ceilometer | 15:01 | |
| *** thomasem has joined #openstack-ceilometer | 15:03 | |
| cmart | Hello | 15:05 |
| *** thomasem has quit IRC | 15:06 | |
| cmart | Do you guys know if our alarm-list API supports pagination? I saw the DB drivers and they had this: raise NotImplementedError('Pagination not implemented') . Is there any plan to implement this in the near future? | 15:09 |
| *** jaypipes has quit IRC | 15:09 | |
| *** cdent has joined #openstack-ceilometer | 15:10 | |
| *** erecio has joined #openstack-ceilometer | 15:12 | |
| openstackgerrit | Sean Dague proposed a change to openstack/ceilometer: Change pipeline_manager to instance attribute in hooks https://review.openstack.org/98295 | 15:20 |
| *** jaypipes has joined #openstack-ceilometer | 15:21 | |
| *** thomasem has joined #openstack-ceilometer | 15:22 | |
| eglynn | sileht: o/ | 15:23 |
| eglynn | cmart: see https://github.com/openstack/ceilometer/blob/master/ceilometer/api/controllers/v2.py#2130 | 15:24 |
| *** julim has quit IRC | 15:24 | |
| eglynn | cmart: also note the pagination capability is not set for alarms | 15:25 |
| eglynn | cmart: so short answer is no | 15:25 |
| cmart | eglynn: OK! And there are no plans to do that, right? | 15:26 |
| cmart | (at least for medium term) | 15:26 |
| eglynn | cmart: not that I'm aware of ... but as always, patches are welcome | 15:27 |
| DinaBelova | eglynn, I guess cmart found some nice thing to do here :) | 15:28 |
| cmart | eglynn: because I was thinking on the alarm page, and the amount of alarms that could be retrieved on the admin side.. and then I thought about pagination | 15:28 |
| openstackgerrit | Victor Stinner proposed a change to openstack/ceilometer: Respect $TMPDIR environment variable to run tests https://review.openstack.org/98818 | 15:28 |
| cmart | DinaBelova: hahaa.. Well, as soon as I finished the alarm page, I would be glad to do it | 15:29 |
| DinaBelova | Ж) | 15:29 |
| DinaBelova | oh, sorry | 15:29 |
| DinaBelova | wrong layout | 15:29 |
| DinaBelova | it was ":)" | 15:29 |
| cmart | eglynn: but maybe it is better to just show the entire table and then add some "cool filtering options" to it :) | 15:30 |
| sileht | eglynn, o/ | 15:30 |
| *** inc0 has quit IRC | 15:31 | |
| eglynn | sileht: just wondering about https://bugs.launchpad.net/ceilometer/+bug/1323524 | 15:32 |
| eglynn | sileht: ... you'd an idea on the source of the error and a possible fix, right? | 15:32 |
| eglynn | sileht: ... just wondering if you could propose a patch, if it's just a simple testr config change? | 15:32 |
| eglynn | sileht: ... I ask because sdague is getting concerned about the rate of ceilo unit test failures | 15:33 |
| sileht | eglynn, ok I will revert just the timeout part of testr configuration | 15:34 |
| eglynn | sileht: excellent, thank you sir! | 15:34 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Removes per test testr timeout. https://review.openstack.org/98820 | 15:43 |
| sileht | eglynn, ^^ | 15:43 |
| eglynn | sileht: thank you sir! | 15:43 |
| sileht | eglynn, my though about the real issue into the commit msg | 15:43 |
| *** promulo_ has quit IRC | 15:43 | |
| *** rongze has quit IRC | 15:44 | |
| *** Ruetobas has quit IRC | 16:01 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| *** IvanBerezovskiy has left #openstack-ceilometer | 16:04 | |
| *** Ruetobas has quit IRC | 16:08 | |
| *** ildikov_ has joined #openstack-ceilometer | 16:08 | |
| *** dmsimard is now known as dmsimard_away | 16:08 | |
| *** ildikov has quit IRC | 16:10 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:14 | |
| *** _nadya_ has joined #openstack-ceilometer | 16:18 | |
| *** _nadya_ has quit IRC | 16:18 | |
| *** Longgeek has quit IRC | 16:19 | |
| *** jaypipes has quit IRC | 16:21 | |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: [HBase] Implement events on HBase https://review.openstack.org/91408 | 16:23 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Update Measurement Docs for LBaaS https://review.openstack.org/98513 | 16:23 |
| openstackgerrit | Christian Martinez proposed a change to openstack/ceilometer: Adding alarm list filtering by state and meter https://review.openstack.org/96582 | 16:26 |
| *** rbowen has quit IRC | 16:42 | |
| *** rbowen has joined #openstack-ceilometer | 16:42 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix list of modules not included in auto-gen docs https://review.openstack.org/97268 | 16:48 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix Sphinx directive name in session.py https://review.openstack.org/98834 | 16:48 |
| eglynn | ityaptin: I've +2'd https://review.openstack.org/91408 but I wonder if your latest patchset just missed Nadya leaving for the evening | 16:57 |
| eglynn | ityaptin: ... if poss, it would be good to get her blessings also (as the hbase domain expert) | 16:57 |
| ildikov_ | eglynn: BTW, I guess that patch cannot be mreged without its dependency, but correct me if I'm wrong here | 16:59 |
| eglynn | ildikov_: you are not wrong | 16:59 |
| eglynn | ildikov_: ... that dependency is somewhat unfortunate | 17:01 |
| *** gordc has quit IRC | 17:01 | |
| ildikov_ | eglynn: yeap, this is my opinion too | 17:01 |
| *** gordc has joined #openstack-ceilometer | 17:03 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix notification for NotImplemented record_events https://review.openstack.org/97669 | 17:09 |
| *** harlowja_away is now known as harlowja | 17:09 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Avoid unnecessary stderr message when run test https://review.openstack.org/95640 | 17:15 |
| openstackgerrit | A change was merged to openstack/ceilometer: Add missing explicit cfg option import https://review.openstack.org/98698 | 17:16 |
| *** _nadya_ has joined #openstack-ceilometer | 17:18 | |
| *** nati_ueno has joined #openstack-ceilometer | 17:27 | |
| _nadya_ | ildikov_: hi! could you please take a look on my comment here https://review.openstack.org/#/c/91408/10/ceilometer/storage/impl_hbase.py ? | 17:28 |
| *** eglynn has quit IRC | 17:31 | |
| _nadya_ | ildikov_: problem is solved :) | 17:36 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Metering LoadBalancer as a Service https://review.openstack.org/95784 | 17:36 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Update Measurement Docs for LBaaS https://review.openstack.org/98513 | 17:36 |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Metering LoadBalancer as a Service https://review.openstack.org/95784 | 17:38 |
| *** _nadya_ has quit IRC | 17:38 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Update Measurement Docs for LBaaS https://review.openstack.org/98513 | 17:38 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Removes per test testr timeout https://review.openstack.org/98820 | 17:43 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Splits hbase storage code base https://review.openstack.org/98721 | 17:45 |
| *** cdent has quit IRC | 17:54 | |
| *** dmsimard_away is now known as dmsimard | 18:01 | |
| *** fnaval has quit IRC | 18:06 | |
| *** gibi has quit IRC | 18:10 | |
| *** jaypipes has joined #openstack-ceilometer | 18:15 | |
| *** erecio has quit IRC | 18:19 | |
| *** erecio has joined #openstack-ceilometer | 18:19 | |
| *** ildikov has joined #openstack-ceilometer | 18:27 | |
| *** ildikov|2 has joined #openstack-ceilometer | 18:29 | |
| *** ildikov has quit IRC | 18:29 | |
| *** ildikov_ has quit IRC | 18:30 | |
| *** nati_ueno has quit IRC | 18:31 | |
| *** r0j4z0 has joined #openstack-ceilometer | 18:34 | |
| *** nati_uen_ has joined #openstack-ceilometer | 18:34 | |
| *** thomasem has quit IRC | 18:37 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:43 | |
| *** promulo has joined #openstack-ceilometer | 18:49 | |
| ildikov|2 | _nadya_: hi, sorry for not answering, sometimes my IRC client forgets to inform me about new messages... | 18:54 |
| *** ildikov|2 is now known as ildikov | 18:54 | |
| ildikov | _nadya_: I added a few nits and questions as comments to that patch | 18:55 |
| *** _nadya_ has quit IRC | 18:55 | |
| *** eglynn has joined #openstack-ceilometer | 19:00 | |
| *** nati_ueno has joined #openstack-ceilometer | 19:02 | |
| *** erecio has quit IRC | 19:04 | |
| *** erecio has joined #openstack-ceilometer | 19:04 | |
| *** nati_uen_ has quit IRC | 19:06 | |
| *** rob_s_parker has quit IRC | 19:07 | |
| *** gibi has joined #openstack-ceilometer | 19:15 | |
| *** nati_ueno has quit IRC | 19:15 | |
| *** fnaval has joined #openstack-ceilometer | 19:16 | |
| *** thomasem has joined #openstack-ceilometer | 19:18 | |
| *** palar_ has joined #openstack-ceilometer | 19:19 | |
| *** thomasem has quit IRC | 19:20 | |
| *** thomasem has joined #openstack-ceilometer | 19:20 | |
| *** palar has quit IRC | 19:23 | |
| *** gibi has quit IRC | 19:23 | |
| *** palar_ has quit IRC | 19:25 | |
| *** gibi has joined #openstack-ceilometer | 19:27 | |
| *** r3pl4y has quit IRC | 19:33 | |
| *** yfujioka has joined #openstack-ceilometer | 19:40 | |
| *** thomasem_ has joined #openstack-ceilometer | 19:44 | |
| *** thomasem_ has quit IRC | 19:44 | |
| *** thomasem_ has joined #openstack-ceilometer | 19:45 | |
| *** thomasem has quit IRC | 19:46 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix list of modules not included in auto-gen docs https://review.openstack.org/97268 | 19:47 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Fix list of modules not included in auto-gen docs https://review.openstack.org/97268 | 19:48 |
| *** yfujioka has quit IRC | 19:51 | |
| flwang | jd__: ping | 19:55 |
| flwang | dhellmann: ping | 19:56 |
| jd__ | flwang: ? | 19:57 |
| flwang | jd__: question about the db migration | 19:58 |
| flwang | jd__: just wanna know is there any best practice about migrating mysql to mongodb against Ceilometer | 19:58 |
| *** eglynn has quit IRC | 19:58 | |
| jd__ | flwang: no | 19:59 |
| flwang | so seems it's not related to ceilometer, but a different database migrating issue, is it? | 20:00 |
| *** julim has joined #openstack-ceilometer | 20:03 | |
| *** boris-42 has quit IRC | 20:08 | |
| *** boris-42 has joined #openstack-ceilometer | 20:13 | |
| *** jdob has quit IRC | 20:25 | |
| *** jdob has joined #openstack-ceilometer | 20:25 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Metering LoadBalancer as a Service https://review.openstack.org/95784 | 20:33 |
| *** julim has quit IRC | 20:34 | |
| *** fnaval has quit IRC | 20:36 | |
| *** nati_ueno has joined #openstack-ceilometer | 20:40 | |
| *** fnaval has joined #openstack-ceilometer | 20:42 | |
| *** cmart has quit IRC | 20:53 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Update Measurement Docs for LBaaS https://review.openstack.org/98513 | 20:54 |
| *** jdob has quit IRC | 20:59 | |
| *** flwang has quit IRC | 21:18 | |
| *** gordc1 has joined #openstack-ceilometer | 21:18 | |
| *** Underyx|off is now known as underyx | 21:18 | |
| *** gordc has quit IRC | 21:18 | |
| *** gordc1 is now known as gordc | 21:18 | |
| *** rbowen has quit IRC | 21:23 | |
| *** promulo has quit IRC | 21:29 | |
| *** thomasem_ has quit IRC | 21:39 | |
| *** thomasem has joined #openstack-ceilometer | 21:39 | |
| *** thomasem has quit IRC | 21:44 | |
| *** _nadya_ has joined #openstack-ceilometer | 21:56 | |
| *** _nadya_ has quit IRC | 22:00 | |
| *** nati_ueno has quit IRC | 22:08 | |
| *** isviridov_ has joined #openstack-ceilometer | 22:10 | |
| *** aviau has quit IRC | 22:15 | |
| *** isviridov has quit IRC | 22:17 | |
| *** eglynn has joined #openstack-ceilometer | 22:17 | |
| *** gordc has quit IRC | 22:25 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Metering LoadBalancer as a Service https://review.openstack.org/95784 | 22:30 |
| *** dmsimard is now known as dmsimard_away | 22:42 | |
| *** eglynn has quit IRC | 22:46 | |
| *** nati_ueno has joined #openstack-ceilometer | 22:47 | |
| *** aviau has joined #openstack-ceilometer | 22:55 | |
| *** jogo has joined #openstack-ceilometer | 23:01 | |
| jogo | hi | 23:01 |
| jogo | so https://bugs.launchpad.net/ceilometer/+bug/1327344 | 23:01 |
| jogo | 5 hits in the gate in the past 24 hours | 23:01 |
| jogo | I think that warrents higher then medium | 23:02 |
| jogo | eglynn-office: ^ | 23:02 |
| jogo | same for https://bugs.launchpad.net/ceilometer/+bug/1323524 | 23:02 |
| jogo | it looks like your guys are the are the top gate breaker of gate after the ssh issues which are subsiding | 23:17 |
| jogo | dhellmann: ^ | 23:18 |
| *** skelpter has joined #openstack-ceilometer | 23:21 | |
| *** nati_ueno has quit IRC | 23:22 | |
| *** promulo has joined #openstack-ceilometer | 23:23 | |
| *** nati_ueno has joined #openstack-ceilometer | 23:28 | |
| *** nati_uen_ has joined #openstack-ceilometer | 23:31 | |
| *** nati_ueno has quit IRC | 23:32 | |
| jogo | anyone around? | 23:45 |
| *** fnaval has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!