Monday, 2014-01-27

*** ok_delta has quit IRC01:36
*** ok_delta has joined #openstack-ceilometer01:40
*** xianghui has joined #openstack-ceilometer02:17
openstackgerritgordon chung proposed a change to openstack/ceilometer: Alembic migrations not tested  https://review.openstack.org/6926002:29
*** xianghui has quit IRC03:08
*** adriant has quit IRC03:22
*** xianghui has joined #openstack-ceilometer03:25
openstackgerritShuangtai Tian proposed a change to openstack/ceilometer: Use the module units to refer bytes type  https://review.openstack.org/6926903:47
*** ok_delta has quit IRC03:59
openstackgerritYuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network  https://review.openstack.org/6047304:34
openstackgerritYuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network-from-opendaylight  https://review.openstack.org/6389004:36
*** AMike has joined #openstack-ceilometer05:09
*** AMike has joined #openstack-ceilometer05:09
openstackgerritA change was merged to openstack/ceilometer: Re-enable lazy translation  https://review.openstack.org/6698305:14
*** sayali has joined #openstack-ceilometer05:38
openstackgerritShuangtai Tian proposed a change to openstack/ceilometer: Use the module units to refer bytes type  https://review.openstack.org/6926905:45
*** werebutt has joined #openstack-ceilometer05:59
*** werebutt has left #openstack-ceilometer05:59
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280806:02
*** SergeyLukjanov_ is now known as SergeyLukjanov06:04
*** xianghui has quit IRC06:11
*** xianghui has joined #openstack-ceilometer06:25
*** SergeyLukjanov is now known as SergeyLukjanov_06:31
*** xianghui has quit IRC06:32
*** boris-42 has quit IRC06:34
*** xianghui has joined #openstack-ceilometer06:56
*** SergeyLukjanov_ is now known as SergeyLukjanov06:57
*** sayali has quit IRC07:00
openstackgerritChmouel Boudjnah proposed a change to openstack/python-ceilometerclient: Remove ununsed httplib2 requirement  https://review.openstack.org/6923707:03
*** ildikov_ has joined #openstack-ceilometer07:05
*** SergeyLukjanov is now known as SergeyLukjanov_07:20
*** lsmola_ has joined #openstack-ceilometer07:31
*** SergeyLukjanov_ is now known as SergeyLukjanov07:48
*** SergeyLukjanov is now known as SergeyLukjanov_08:03
*** ildikov_ has quit IRC08:03
*** SergeyLukjanov_ is now known as SergeyLukjanov08:03
*** ildikov_ has joined #openstack-ceilometer08:04
*** frankbutt has joined #openstack-ceilometer08:11
*** frankbutt has left #openstack-ceilometer08:11
*** SergeyLukjanov is now known as SergeyLukjanov_08:17
*** boris-42 has joined #openstack-ceilometer08:36
*** _nadya_ has joined #openstack-ceilometer08:47
*** _nadya_ has quit IRC09:00
*** _nadya_ has joined #openstack-ceilometer09:01
*** xianghui has quit IRC09:03
*** yassine has joined #openstack-ceilometer09:06
*** Alexei_987 has joined #openstack-ceilometer09:07
*** _nadya_ has quit IRC09:13
*** xianghui has joined #openstack-ceilometer09:17
*** _nadya_ has joined #openstack-ceilometer09:19
*** _nadya_ has quit IRC09:20
*** vrovachev1 has joined #openstack-ceilometer09:22
jd__chmouel: if you want to track the work yeah, otherwise just go ahead09:30
chmoueljd__: bah done (filled blueprint and uploaded reviews) now it's time for the fun part for you guys (reviews YAY)09:31
jd__heh thanks chmouel09:32
*** _ruhe is now known as ruhe09:35
*** eglynn has joined #openstack-ceilometer09:37
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: eventlet: stop monkey patching  https://review.openstack.org/6814509:43
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Revert "Ensure we are not exhausting the sqlalchemy pool"  https://review.openstack.org/6839909:43
*** sayali has joined #openstack-ceilometer09:57
*** AMike has quit IRC10:08
*** AMike has joined #openstack-ceilometer10:09
*** AMike has joined #openstack-ceilometer10:09
*** Ajaeger has joined #openstack-ceilometer10:57
AjaegerHi!10:58
AjaegerJust came here via https://launchpad.net/ceilometer - which still references #openstack-metering.10:58
Ajaegeranybody around that can update the page, please?10:59
jd__Ajaeger: thanks, fixed the homepaghe11:00
Ajaegerjd__: Thanks - I fixed https://wiki.openstack.org/wiki/Ceilometer myself11:01
AjaegerIs there anybody around that is willing to review and approve this documentation only patch: https://review.openstack.org/#/c/67658/ ? This is information that shows up in the manuals and we'd like to have them somehow consistent and nice  - and thus need consistent help strings11:02
*** ruhe is now known as _ruhe11:12
*** SergeyLukjanov_ is now known as SergeyLukjanov11:17
Ajaegerjd__: There's also https://review.openstack.org/#/c/67646/ and https://review.openstack.org/#/c/67654/ for Oslo - same topic as https://review.openstack.org/#/c/67658/. Could you review these, please?11:18
*** kwhitney has joined #openstack-ceilometer11:18
*** xianghui has quit IRC11:29
jd__Ajaeger: I'll take a look11:29
Ajaegerjd__: Thanks a lot!11:30
*** SergeyLukjanov is now known as SergeyLukjanov_11:33
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Handle engine creation inside of Connection object  https://review.openstack.org/6785111:34
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Refactored session access  https://review.openstack.org/6785011:34
*** _ruhe is now known as ruhe11:39
Alexei_987jd__: Hi. Is there any chance that you'll take a look at this patches? ^11:39
Alexei_987jd__:  they are kinda needed for real backends tests :)11:39
jd__I take a look at all patches without discrimination11:43
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Remove newlines in base64.encodestring  https://review.openstack.org/6933311:46
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Remove unused db engine variable in api  https://review.openstack.org/6934012:18
Alexei_987sileht: Hi12:24
silehtAlexei_987, o/12:24
Alexei_987sileht: do you have time to discuss storage_engine.get_connection(cfg.CONF) from my patch?12:24
Alexei_987sileht: seems that I've thought that stevedore.driver.DriverManager will only load object once12:25
Alexei_987sileht: correct me if I'm wrong12:25
ruheAlexei_987: is there etherpad i could read to understand how you're going to overcome issues with backend testing?12:27
Alexei_987ruhe: I'm afraid that there is no etherpad for this12:28
Alexei_987ruhe: problem is that cause of oslo db issue we can have only 1 sqlalchemy connection in 1 process12:28
Alexei_987ruhe: this patches introduce a workaround for this issue12:29
ruheAlexei_987: ok, i see. you still plan to use separate DB with a random name for each test run?12:30
ruheand one more thing - run alembic migrations for each test might consume a lot of time12:32
Alexei_987ruhe: It's already implemented for mongodb12:32
Alexei_987ruhe: and sqlalchemy migrations does not consume a lot of time?12:32
Alexei_987unfortunately I don't understand the last question12:32
Alexei_987ruhe: migrations are applied in any case12:32
Alexei_987ruhe: we are slowly moving towards creating empty DB from models.metadata12:33
ruheAlexei_987: they're the same; i just want to notice, that i found that migrations take the most time in unit tests12:33
Alexei_987for the tests12:33
ruheAlexei_987: things are much worse for HBase. it takes 16 seconds to upgrade/downgrade12:33
Alexei_987ruhe: you are right but for now I don't see anyway to avoid it12:33
Alexei_987ruhe: what really helps is to move /tmp to tempfs12:34
Alexei_987it allows us to save a lot of time on disk io12:34
ruheAlexei_987: good idea! will try it with HBase12:34
ruheAlexei_987: thanks for explanation12:35
Alexei_987sileht: just wrote a test for get_engine method. you are right it's creating 2 separate objects12:36
Alexei_987sileht: basically it means that we should remove method get_connection from storage12:36
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Add documentation for pipeline configuration  https://review.openstack.org/6935012:46
silehtAlexei_987, yes or the engine I have propose this one: https://review.openstack.org/6934012:47
Alexei_987sileht: LGTM :)12:48
Alexei_987sileht: however I'm saying that method get_connections in storage/__init__ is simply not correct12:49
Alexei_987sileht: and it must be removed12:49
Alexei_987s/get_connections/get_connection/12:49
*** ruhe is now known as _ruhe12:52
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Handle engine creation inside of Connection object  https://review.openstack.org/6785112:53
*** _ruhe is now known as ruhe12:54
openstackgerritEugeniya Kudryashova proposed a change to openstack/python-ceilometerclient: Use HTTPClient from common Oslo code  https://review.openstack.org/6893912:57
*** xianghui has joined #openstack-ceilometer12:57
silehtAlexei_987, I agree, fortunatly it is never used twice13:00
Alexei_987well "fortunately" is not the thing I would like to rely on :)13:02
Alexei_987I'll make a patch to remove it13:02
*** eglynn has quit IRC13:23
*** SergeyLukjanov_ is now known as SergeyLukjanov13:31
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Clean .gitignore  https://review.openstack.org/6936613:40
*** eglynn has joined #openstack-ceilometer13:59
*** gordc has joined #openstack-ceilometer14:06
*** eglynn is now known as eglynn-afk14:07
*** boris-42 has quit IRC14:23
*** prad_ has joined #openstack-ceilometer14:54
*** sayali has quit IRC14:55
*** sayali has joined #openstack-ceilometer14:56
*** eglynn-afk has quit IRC14:56
*** jdob has joined #openstack-ceilometer14:56
*** ruhe is now known as _ruhe15:12
*** rwsu has joined #openstack-ceilometer15:16
openstackgerritgordon chung proposed a change to openstack/ceilometer: meter table contains redundant/duplicate data  https://review.openstack.org/6578615:23
*** xianghui has quit IRC15:25
*** boris-42 has joined #openstack-ceilometer15:26
*** sayali has quit IRC15:26
openstackgerritgordon chung proposed a change to openstack/ceilometer: meter table contains redundant/duplicate data  https://review.openstack.org/6578615:27
*** _ruhe is now known as ruhe15:33
*** lucas-hungry has joined #openstack-ceilometer15:36
*** SergeyLukjanov is now known as SergeyLukjanov_15:38
*** SergeyLukjanov_ is now known as SergeyLukjanov15:58
*** dhellmann is now known as dhellmann_16:00
*** feldmann has joined #openstack-ceilometer16:01
*** ildikov_ has quit IRC16:11
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Use explicit http error code for api v2  https://review.openstack.org/6877516:13
*** eglynn-afk has joined #openstack-ceilometer16:20
*** sayali has joined #openstack-ceilometer16:22
*** ruhe is now known as _ruhe16:25
*** SergeyLukjanov is now known as SergeyLukjanov_16:30
*** lucas-hungry is now known as lucasagomes16:37
*** lucasagomes has left #openstack-ceilometer16:40
*** lsmola_ has quit IRC16:53
*** SergeyLukjanov_ is now known as SergeyLukjanov16:54
*** SergeyLukjanov is now known as SergeyLukjanov_17:03
*** ildikov_ has joined #openstack-ceilometer17:05
*** Ajaeger has left #openstack-ceilometer17:06
*** _nadya_ has joined #openstack-ceilometer17:10
*** sayali has quit IRC17:20
*** sayali has joined #openstack-ceilometer17:22
*** sayali has quit IRC17:23
*** jonnalley has joined #openstack-ceilometer17:23
*** sayali has joined #openstack-ceilometer17:24
*** jonnalley has left #openstack-ceilometer17:24
*** SergeyLukjanov_ is now known as SergeyLukjanov17:26
*** tongli has joined #openstack-ceilometer17:30
*** sayali has quit IRC17:36
*** sayali has joined #openstack-ceilometer17:37
*** SergeyLukjanov is now known as SergeyLukjanov_17:41
*** Alexei_987 has quit IRC17:46
openstackgerritA change was merged to openstack/ceilometer: eventlet: stop monkey patching  https://review.openstack.org/6814517:49
*** vrovachev1 has quit IRC17:51
*** yassine has quit IRC18:04
*** yassine has joined #openstack-ceilometer18:04
*** yassine has quit IRC18:08
*** mst89 has joined #openstack-ceilometer18:09
*** mst89 has quit IRC18:17
*** msolanas has joined #openstack-ceilometer18:18
*** dhellmann_ is now known as dhellmann18:23
*** dhellmann is now known as dhellmann_18:23
*** boris-42 has quit IRC18:26
openstackgerritA change was merged to openstack/ceilometer: Revert "Ensure we are not exhausting the sqlalchemy pool"  https://review.openstack.org/6839918:27
*** boris-42 has joined #openstack-ceilometer18:29
*** _ruhe is now known as ruhe18:32
*** _nadya_ has quit IRC18:35
*** xmltok has joined #openstack-ceilometer18:35
*** sayali has quit IRC18:36
*** feldmann has quit IRC18:44
*** SergeyLukjanov_ is now known as SergeyLukjanov18:53
*** ruhe is now known as _ruhe19:15
*** urulama has joined #openstack-ceilometer19:25
*** msolanas has quit IRC19:31
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Add documentation for pipeline configuration  https://review.openstack.org/6935019:50
openstackgerritA change was merged to openstack/ceilometer: Improve consistency of help strings  https://review.openstack.org/6765819:58
*** openstackgerrit has quit IRC20:06
*** openstackgerrit has joined #openstack-ceilometer20:06
*** _ruhe is now known as ruhe20:06
openstackgerritPradeep Kilambi proposed a change to openstack/ceilometer: Fix docs on what an instance meter represents  https://review.openstack.org/6674620:18
openstackgerritgordon chung proposed a change to openstack/ceilometer: use common code for migrations  https://review.openstack.org/6945920:21
*** jmckind has joined #openstack-ceilometer20:25
*** DanD has joined #openstack-ceilometer20:37
*** SergeyLukjanov is now known as SergeyLukjanov_20:44
*** ruhe is now known as _ruhe20:46
*** _ruhe is now known as ruhe20:56
*** dhellmann_ is now known as dhellmann21:15
*** ok_delta has joined #openstack-ceilometer21:16
openstackgerritlitong01 proposed a change to openstack/ceilometer: add notification alarm  https://review.openstack.org/6947321:32
*** _nadya_ has joined #openstack-ceilometer21:35
openstackgerritlitong01 proposed a change to openstack/python-ceilometerclient: add notification alarm  https://review.openstack.org/6947421:35
*** tongli has quit IRC21:38
*** _nadya_ has quit IRC21:39
*** DanD has quit IRC21:42
*** prad has joined #openstack-ceilometer21:45
openstackgerritA change was merged to openstack/python-ceilometerclient: Fix typos picked up by misspellings  https://review.openstack.org/6858421:47
*** prad has quit IRC21:48
*** prad_ is now known as prad21:48
*** ruhe is now known as _ruhe21:48
openstackgerritgordon chung proposed a change to openstack/ceilometer: meter table contains redundant/duplicate data  https://review.openstack.org/6578622:13
*** _ruhe is now known as ruhe22:30
*** ruhe is now known as _ruhe22:38
*** jdob has quit IRC22:42
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer: Use stevedore's make_test_instance  https://review.openstack.org/6949122:53
rwsuHi, I'm interested in generating alerts when openstack related service/process goes down, I don't think ceilometer covers this use case right now.22:57
openstackgerritA change was merged to openstack/python-ceilometerclient: Using common methods from oslo cliutils  https://review.openstack.org/6738122:57
rwsuI'm thinking of writing a job to check if a an expected process on an instance is up or not. If it is down then send a message to the notification bus.22:57
rwsucan I then write something in ceilometer to transform these messages into a metric/event and send an alarm if a process failure is detected?22:58
openstackgerritA change was merged to openstack/python-ceilometerclient: return sample info when creating sample with CLI  https://review.openstack.org/6612522:59
*** dhellmann is now known as dhellmann_23:01
openstackgerritA change was merged to openstack/python-ceilometerclient: Remove print debugs statements.  https://review.openstack.org/6923423:04
openstackgerritA change was merged to openstack/python-ceilometerclient: Remove ununsed httplib2 requirement  https://review.openstack.org/6923723:04
*** prad has quit IRC23:18
*** jmckind has quit IRC23:18
*** flwang has quit IRC23:22
*** _ruhe is now known as ruhe23:22
*** ok_delta has quit IRC23:23
*** flwang has joined #openstack-ceilometer23:23
*** ruhe is now known as _ruhe23:44
*** jd__` has joined #openstack-ceilometer23:47
*** jd__ has quit IRC23:48
*** jd__` is now known as jd__23:48

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