Thursday, 2014-06-12

*** julim has quit IRC00:05
*** _nadya_ has quit IRC00:25
*** matsuhashi has joined #openstack-ceilometer00:27
*** nosnos has joined #openstack-ceilometer00:32
*** ildikov_ has joined #openstack-ceilometer00:48
*** ildikov has quit IRC00:51
*** liusheng has quit IRC00:51
*** liusheng has joined #openstack-ceilometer00:52
*** _nadya_ has joined #openstack-ceilometer00:52
*** skelpter has joined #openstack-ceilometer00:57
*** dmsimard is now known as dmsimard_away01:08
*** zqfan has joined #openstack-ceilometer01:10
*** nati_ueno has quit IRC01:30
*** fnaval has joined #openstack-ceilometer01:41
*** rwsu has quit IRC01:42
*** _nadya_ has quit IRC01:50
*** _cjones_ has quit IRC01:55
*** _cjones_ has joined #openstack-ceilometer01:56
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/pycadf: Bump hacking to 0.9.x series  https://review.openstack.org/9954201:57
*** thomasem has joined #openstack-ceilometer01:59
*** thomasem has quit IRC01:59
*** _cjones_ has quit IRC02:00
*** skelpter has quit IRC02:11
*** _nadya_ has joined #openstack-ceilometer02:17
*** _cjones_ has joined #openstack-ceilometer02:27
*** _cjones_ has quit IRC02:34
*** fnaval has quit IRC03:01
*** zhikunliu has joined #openstack-ceilometer03:05
*** Longgeek has joined #openstack-ceilometer03:21
*** _cjones_ has joined #openstack-ceilometer03:29
*** _cjones_ has quit IRC03:34
*** nosnos has quit IRC03:44
*** r3pl4y has joined #openstack-ceilometer04:07
*** matsuhashi has quit IRC04:11
*** harlowja is now known as harlowja_away04:16
*** noorul has joined #openstack-ceilometer04:23
openstackgerritA change was merged to openstack/python-ceilometerclient: Remove © and remove unnecessary encoding lines  https://review.openstack.org/9588004:27
*** r3pl4y has quit IRC04:28
*** _cjones_ has joined #openstack-ceilometer04:30
*** _cjones_ has quit IRC04:35
*** yatin has joined #openstack-ceilometer04:38
*** yatin_ has joined #openstack-ceilometer04:46
*** yatin_ has joined #openstack-ceilometer04:47
*** matsuhashi has joined #openstack-ceilometer04:48
*** yatin has quit IRC04:48
*** nosnos has joined #openstack-ceilometer04:48
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/9681904:53
*** psharma has joined #openstack-ceilometer04:57
openstackgerritOpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements  https://review.openstack.org/9956704:59
*** Longgeek_ has joined #openstack-ceilometer05:21
*** Longgeek has quit IRC05:24
*** _nadya_ has quit IRC05:51
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/8850606:01
*** _nadya_ has joined #openstack-ceilometer06:17
*** ildikov_ has quit IRC06:20
*** eglynn has joined #openstack-ceilometer06:35
*** ildikov has joined #openstack-ceilometer06:38
*** r3pl4y has joined #openstack-ceilometer06:46
*** inc0 has joined #openstack-ceilometer06:50
*** _nadya_ has quit IRC06:51
*** manoj has joined #openstack-ceilometer06:52
*** _nadya_ has joined #openstack-ceilometer06:52
manojhi all06:52
manojis the instance level analytics available in openstack ceilometer06:54
eglynnmanoj: good morning07:04
eglynnmanoj: did you mean instance *metrics*?07:05
manojeglynn: good morning07:05
manojeglynn: yes07:05
eglynnmanoj: (... as opposed to *analytics*)07:05
*** _nadya_ has quit IRC07:05
eglynnmanoj: here are the instance-level metrics that ceilometer gathers ... http://docs.openstack.org/developer/ceilometer/measurements.html#compute-nova07:06
eglynnmanoj: ... does that answer your question?07:06
manojeglynn: yes, i need to know what is the instance duration?07:07
*** ildikov has quit IRC07:07
*** safchain has joined #openstack-ceilometer07:08
eglynnmanoj: ... I guess you can derive that from the first and last sample timestamp associated with the resource07:09
eglynnmanoj: http://docs.openstack.org/developer/ceilometer/webapi/v2.html#resources07:09
eglynnmanoj: (though that would not accout for any "downtime" when the instance was suspended)07:09
eglynnbbiab07:09
*** eglynn is now known as eglynn-commute07:10
zqfanhi, the bot uploaded patch https://review.openstack.org/#/c/91238/14 has been reviewed for nearly 6 weeks, is it good to fly now?07:10
*** zhikunliu1 has joined #openstack-ceilometer07:21
*** zhikunliu has quit IRC07:23
*** ildikov has joined #openstack-ceilometer07:24
*** zhikunliu1 has quit IRC07:26
*** zhikunliu has joined #openstack-ceilometer07:31
*** _nadya_ has joined #openstack-ceilometer07:38
*** eglynn-commute has quit IRC07:38
*** _cjones_ has joined #openstack-ceilometer07:43
*** _cjones_ has quit IRC07:47
*** _nadya_ has quit IRC08:03
*** eglynn-commute has joined #openstack-ceilometer08:07
*** asvechnikov has quit IRC08:16
*** vrovachev has quit IRC08:16
*** ilyashakhat has quit IRC08:17
*** ityaptin has quit IRC08:17
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Improve api_paste_config file searching  https://review.openstack.org/9959908:20
*** eglynn-commute is now known as eglynn08:34
*** zhikunliu has quit IRC08:38
lluzqfan: thanks for the patch08:42
*** _cjones_ has joined #openstack-ceilometer08:43
zqfanhi, llu, the bot uploaded patch https://review.openstack.org/#/c/91238/14 has been reviewed for nearly 6 weeks, is it good to fly now?08:46
*** pnascimento has joined #openstack-ceilometer08:48
zqfanllu, and since I upload a workaround for that patch (accidently modify the topic), jenkins uploads another patch which does same job....08:48
*** _cjones_ has quit IRC08:48
*** Alexei_987 has quit IRC08:53
*** manoj has quit IRC08:56
lluI think the previous patch sets didn't get merged in because of failure requirement tests, I think the infra has reverted the patch which caused that problem08:56
lluzqfan: sileht just approved that :)08:57
zqfanllu, sileht, thanks08:58
silehtllu, zqfan, but gate have a tone of patch that waiting for merging, we needs to wait at least 1 days before it get merged :(08:59
zqfansileht, that's ok to me09:01
*** zhikunliu has joined #openstack-ceilometer09:01
*** zhikunliu has quit IRC09:01
llusileht: i've got much more patience after I step into the openstack world :)09:02
*** zhikunliu has joined #openstack-ceilometer09:02
*** _nadya_ has joined #openstack-ceilometer09:14
eglynnliusheng: o/09:16
liushengeglynn, hi!09:17
eglynnliusheng: good morning / evening!09:18
eglynnliusheng: ... just wondering if you saw this thread on the ML http://lists.openstack.org/pipermail/openstack-dev/2014-June/037304.html09:18
liushengeglynn, yes I just saw that09:19
eglynnliusheng: yeah ... so the usage of the nova floating_ip API all_tenants flag has been reverted on the ceilometer side09:19
*** _nadya_ has quit IRC09:19
eglynnliusheng: the suggestion on that thread is that we could possibly derive this information from notifications instead09:20
eglynnliusheng: ... does that sound reasonable to you?09:20
eglynnliusheng: ... in any case, could you chime in on the thread?09:20
liushengeglynn, I have found that, and I am working on the patch https://review.openstack.org/9925109:20
liushengeglynn, yeah absolutely:)09:21
eglynnliusheng: cool, so definitely the underlying bug needs to be fixed ...09:22
eglynnliusheng: ... so thanks for proposing https://review.openstack.org/9925109:22
eglynnliusheng: the other aspect is whether ceilometer can acquire the same info via notifications, this putting less load on nova-api09:23
liushengeglynn, it is my fault for the effection :909:23
liushengeglynn, aggree !09:23
eglynnliusheng: cool, so replying on the ML thread with your thoughts and a reference to your nova fix would be great!09:24
eglynnliusheng: thank you sir!09:24
liushengeglynn, previously, I found the the floatingip pollster is designed in ceilometer, but not avaliable09:24
liushengeglynn, ok, thanks09:25
*** KurtRao has quit IRC09:38
*** fc__ has quit IRC09:39
*** fc__ has joined #openstack-ceilometer09:40
*** Alexei_987 has joined #openstack-ceilometer09:52
silehteglynn, we continue to have the TestRealNotification that fail sometimes: https://github.com/enovance/ceilometer/commit/c12274ca2dfd270919608001e5bc5ab55ccfde55 doesn't fix the issue10:01
silehtlatest gate issue: http://logs.openstack.org/40/99140/1/gate/gate-ceilometer-python26/df56e26/console.html10:01
eglynnsileht: darn! :(10:01
openstackgerritChinmaya Bharadwaj proposed a change to openstack/ceilometer: VMware:Update the ceilometer doc with VMware opts  https://review.openstack.org/9813210:01
* eglynn looks ...10:01
silehteglynn, I guess the 'atexit' error come from a unrelease semaphore10:01
openstackgerritChinmaya Bharadwaj proposed a change to openstack/ceilometer: Fixes ceilometer-compute service start failure  https://review.openstack.org/9783110:02
eglynnsileht: i.e. the semaphore introduced in that initial "fix"?10:03
eglynnsileht: would it make sense to revert c12274ca?10:03
eglynnsileht: ... i..e pending a more correct fix10:04
silehteglynn, I will try the find an other solution to ensure notification are published10:05
* sileht needs to resetup its py26 VM10:05
eglynnsileht: thank you sir!10:05
*** zhikunliu has quit IRC10:05
* eglynn feels guilty every time someone mentions py26 ...10:05
silehtthis error occurs only in py26...10:06
*** asvechnikov has joined #openstack-ceilometer10:11
*** ityaptin has joined #openstack-ceilometer10:12
*** idegtiarov has joined #openstack-ceilometer10:18
*** yatin__ has joined #openstack-ceilometer10:19
*** matsuhashi has quit IRC10:20
*** matsuhashi has joined #openstack-ceilometer10:20
*** yatin_ has quit IRC10:23
*** matsuhashi has quit IRC10:25
*** zqfan is now known as zqfan_afk10:34
*** underyx is now known as Underyx10:34
*** zqfan_afk is now known as zqfan_bot10:34
openstackgerritA change was merged to openstack/ceilometer: Revert "Fix the floatingip pollster"  https://review.openstack.org/9921210:43
*** _cjones_ has joined #openstack-ceilometer10:45
openstackgerritYatin Kumbhare proposed a change to openstack/ceilometer-specs: Specs for email alarm notification  https://review.openstack.org/9901910:49
*** _cjones_ has quit IRC10:50
*** _nadya_ has joined #openstack-ceilometer10:56
*** pnascimento has quit IRC10:57
*** _nadya_ has quit IRC11:05
EmilienMeglynn: hey, just to let you know that ceilometer gating has now grenade jobs11:08
EmilienMeglynn: and also the fix for grenade upgrade is here: https://review.openstack.org/#/c/99412/11:08
EmilienMthanks to sileht for the highlight11:09
silehtEmilienM, your welcome11:10
EmilienMsileht: :)11:10
eglynnEmilienM: excellent! :)11:12
eglynnEmilienM: ... so the lack of an dbsync was the cause of the ceilo-api silent failure after the upgrade?11:12
EmilienMeglynn: we think so yes11:12
eglynnEmilienM: cool, makes sense ... good catch guys!11:12
EmilienMall the credits to sileht11:13
*** nosnos has quit IRC11:13
EmilienMeglynn: so now, you should have gating for each ceilometer patch11:13
eglynnkudos to sileht's keen eyes! :)11:13
silehteglynn, EmilienM even it does fix the ceilo-api, the dbsync is missing :p11:13
EmilienMand I'm scared until the dbsync stuff is not merged, all the ceilometer patches are blocked11:13
eglynnEmilienM: a-ha so it's currently voting?11:14
eglynnEmilienM: ... so we need to get https://review.openstack.org/99412 landed asap in that case11:14
EmilienMeglynn: i think yes, it's voting11:15
EmilienMif I understand correctly11:15
eglynnEmilienM: ... sdague has +2'd, I've just added dtroyer as an additional reviewer11:15
eglynnEmilienM: ... let's see how it plays out for a day or two11:16
eglynnEmilienM: ... if it's problematic, would it be easy to revert to non-voting for a bedding-in period?11:16
EmilienMeglynn: of course11:16
openstackgerritChinmaya Bharadwaj proposed a change to openstack/ceilometer: Fixes ceilometer-compute service start failure  https://review.openstack.org/9783111:17
*** shakayumi has joined #openstack-ceilometer11:23
*** shakayumi has quit IRC11:23
*** dmsimard_away is now known as dmsimard11:23
*** shakayumi has joined #openstack-ceilometer11:24
*** r3pl4y has quit IRC11:27
*** r3pl4y has joined #openstack-ceilometer11:28
*** kun_huang has joined #openstack-ceilometer11:33
openstackgerritDirk Mueller proposed a change to openstack/ceilometer: Downgrade publisher logging to debug level again  https://review.openstack.org/9670611:38
*** yatin__ is now known as yatin11:39
*** yatin has quit IRC11:45
*** yatin has joined #openstack-ceilometer11:45
*** _cjones_ has joined #openstack-ceilometer11:46
*** kun_huang has quit IRC11:48
*** _cjones_ has quit IRC11:50
*** shakayumi has quit IRC11:50
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/pycadf: Bump hacking to 0.9.x series  https://review.openstack.org/9954211:52
yatinhello folks11:58
yatini found that ceilometer zeromq support in havana is broken....11:59
yatinI want to fix that, but don;t know how do i go about it?12:00
inc0yatin, what exactly do you mean?12:01
yatininc0: create_worker and join_consumer method implementation not present for impl_zmq12:02
yatininc0: hence ceilometer will not capture notificaton-info queue data12:02
silehtyatin, all rpc stuffs with zeromq is broken12:02
inc0yatin, you could fix it in master12:02
silehtyatin, nobody use/maintain zmq driver of openstack12:02
EmilienMeglynn: sileht : so I confirm that grenade is now part of ceilometer gate: http://is.gd/CoaIH012:02
inc0then it could be merged to stable/havaba but I doubt it12:03
EmilienMand is votin12:03
EmilienMand is voting12:03
yatininc0: ok. i will check master then...12:03
yatininc0: just thought ceilometer is collecting lots of data, hence zmq will be good fit for faster messaging12:04
inc0yatin, well, it only listens on messages really12:04
inc0so you would have to set up your whole openstack on zeromq12:05
yatininc0: that's right12:05
inc0and that might be...well...full of surprises and unwanted fun12:05
silehtyatin, inc0 zeromq in openstack have 0 unittests ...12:05
inc0I'd strongly reccomend to stay with rabbit12:05
silehtor qpid12:05
yatinsileht: i'm new contributer, and read docs that openstack supports zeromq, qpid and rabbtmq hence, i guess, all should be well supported12:06
silehtyatin, do you have a link ? (this must be fixed :p)12:07
yatinsileht: i checkout havana ceilometer code, and those two methods, not fould implemented12:08
yatinsileht: in havana, collecter/service.py calls create_worker and join_consumer12:08
yatinsileht: from initialize_service_hook method12:09
silehtyatin, I known, I just say the doc must be fixed zeromq don't work with any openstack componments12:10
EmilienMeglynn: i think we should manage to have this patch merged12:10
silehtyatin, I will be happy to change that If I have a link :)12:10
yatinsileht: yes. you are right. docs does says that.. i remember seen this in docs12:10
silehtyatin, the first step to get zmq works with openstack is to write unittests in oslo.messaging project (and fix the tone of issue) and then impls the TODO and then perhaps looks in other project12:12
yatinsileht: sure. understood.12:15
yatinsileht: is that true, the fixes can't happen on havana.12:17
yatinsileht: ?12:17
*** jdob has joined #openstack-ceilometer12:17
eglynnEmilienM: ... thanks for the confirmation12:17
silehtyatin, I guess, this is not just bug fix, zmq driver doesn't implements some features, so backport will be not possible12:19
yatinsileht: yes. backport not possible.12:20
EmilienMeglynn: the patch is going to be merged, it has been approved by sdague12:21
eglynnEmilienM: cool :)12:22
yatinsileht: in that case, what you said earlier is the only choice12:22
*** gordc has joined #openstack-ceilometer12:29
*** psharma has quit IRC12:41
*** openstackgerrit has quit IRC12:46
*** _cjones_ has joined #openstack-ceilometer12:46
*** openstackgerrit has joined #openstack-ceilometer12:47
*** rbowen has joined #openstack-ceilometer12:48
*** _cjones_ has quit IRC12:51
*** prad has joined #openstack-ceilometer12:56
*** julim has joined #openstack-ceilometer13:01
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: [TEST] try fix 1328679  https://review.openstack.org/9965413:04
*** thomasem has joined #openstack-ceilometer13:08
*** thomasem has quit IRC13:12
*** thomasem has joined #openstack-ceilometer13:13
inc0damn you ceilometer havana/stable, I've added ttl to samples (sqlalchemy impl) and I was happy13:19
inc0but now I've looked up this db13:19
inc0and figured out that there are no cascade on deletes between samples and sourceassoc and metadatas13:19
inc0which means, nothing got deleted because of constrains ><13:20
inc0and I have 20m records to delete manually13:20
*** KurtRao has joined #openstack-ceilometer13:31
*** jergerber has joined #openstack-ceilometer13:32
*** sileht has quit IRC13:43
*** yatin has quit IRC13:43
*** zhikunliu has joined #openstack-ceilometer13:44
*** yatin has joined #openstack-ceilometer13:44
eglynnfolks, realistically those 3 remaining juno-1 target'd patches are not gonna land today13:55
*** Longgeek has joined #openstack-ceilometer13:55
* eglynn bumps LP #1326250 #1326723 #1293337 to juno-213:55
eglynnFYI the juno-1 tag will be dropped on https://github.com/openstack/ceilometer/commit/5aa5cfd513:55
*** kun_huang has joined #openstack-ceilometer13:57
*** Longgeek_ has quit IRC13:57
*** kun_huang has quit IRC13:58
*** kun_huang has joined #openstack-ceilometer14:01
*** sileht has joined #openstack-ceilometer14:02
*** noorul has left #openstack-ceilometer14:05
*** zqfan_ has joined #openstack-ceilometer14:05
*** gordc has quit IRC14:06
*** Underyx is now known as underyx14:07
*** yatin has quit IRC14:08
*** yatin has joined #openstack-ceilometer14:08
silehtlaunchpad mail spam has started, ceilometer j-1 will be cutted soon  :)14:09
ildikovsileht: :)14:10
*** yatin has quit IRC14:21
eglynnsileht: yep, the juno-1 tag is out ... https://github.com/openstack/ceilometer/tree/2014.2.b114:21
*** r3pl4y has quit IRC14:23
*** rbowen is now known as DrBacchus14:28
*** kun_huang has quit IRC14:30
*** kun_huang has joined #openstack-ceilometer14:34
DinaBelovafolks, btw - all Russians have holiday today - so I guess you do not see us today :)14:35
*** chmartinez has joined #openstack-ceilometer14:36
ildikovDinaBelova: tnx for the info!14:37
DinaBelovaildikov, I'll be on the meeting but just lurking :)14:38
ildikovDinaBelova: you should enjoy your vacation, but I will not say a word as I guess I'm the second most workaholic here after eglynn ;)14:39
*** fnaval has joined #openstack-ceilometer14:39
DinaBelova:D14:39
* eglynn is more of a workaphobe ;)14:39
chmartinezDinaBelova: vacations?? (btw, I'm cmart)14:40
DinaBelova:) - well, national holidays :)14:41
DinaBelovaso all Russians are inactive :)14:41
chmartinezOh I see..14:41
ildikoveglynn: LOL :)14:41
chmartinezwell, here in Argentina we have smth similar.. When Soccer World Cup starts, the country is frozen14:42
chmartinezi.e: nobody works.. Or breathe14:42
chmartinez:D14:42
*** gordc has joined #openstack-ceilometer14:43
*** _cjones_ has joined #openstack-ceilometer14:48
*** admin0 has joined #openstack-ceilometer14:51
*** _cjones_ has quit IRC14:53
*** r3pl4y has joined #openstack-ceilometer15:03
*** zqfan_ is now known as zqfan15:03
*** r3pl4y has quit IRC15:07
*** r3pl4y has joined #openstack-ceilometer15:08
*** chmartinez has quit IRC15:12
*** fnaval has quit IRC15:17
*** _nadya_ has joined #openstack-ceilometer15:17
*** underyx is now known as Underyx|off15:17
*** fnaval has joined #openstack-ceilometer15:17
*** zhikunliu has quit IRC15:19
*** chmartinez has joined #openstack-ceilometer15:19
*** _nadya_ has quit IRC15:21
*** fnaval has quit IRC15:22
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Add an alarm service based on taskflow  https://review.openstack.org/9176315:23
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Improve api_paste_config file searching  https://review.openstack.org/9959915:25
*** alexpilotti has joined #openstack-ceilometer15:26
Alexei_987jd__: so off meeting question - why?15:29
Alexei_987jd__: you don't like the database?15:29
jd__Alexei_987: because we have better solution available15:29
jd__I like database to store data to request15:29
jd__not as a queue system15:29
Alexei_987jd__: it's not a queue15:30
Alexei_987jd__: it's a state holder15:30
Alexei_987so not a queue15:30
jd__right15:30
Alexei_987jd__: why it's better?15:30
jd__so you just reimplemented Taskflow15:30
Alexei_987jd__: if taskflow is implemented in 100 lines than yes15:31
Alexei_987jd__: and it doesn't require a separate service configured15:31
jd__no idea of the number of lines15:31
Alexei_987jd__: so you could try to check code before rejecting it15:31
jd__write a taskflow driver using a db (if there's not already?)15:31
*** aviau has quit IRC15:33
*** _cjones_ has joined #openstack-ceilometer15:44
*** aviau has joined #openstack-ceilometer15:47
*** llu-laptop has joined #openstack-ceilometer15:50
silehtllu, o/15:50
llu-laptopsileht: sorry, I still don't quite understand why the senario you mentioned would result that bug, in that senario, the test case should fail15:50
llu-laptopbut not at some I/O failure, but the assertion15:51
llu-laptopafter the attempt to acquire the sem, self.srv.stop() stops the notification service15:51
silehtllu-laptop, yes, I agree, It should fail15:51
*** fabiog has joined #openstack-ceilometer15:52
*** robsparker has joined #openstack-ceilometer15:52
silehtllu-laptop, in gate it doesn't fail because the tests never finish, and gate kill tox15:53
*** KurtRao has quit IRC15:53
silehtllu-laptop, this is just my guessing, perhaps I'm wrong15:54
llu-laptopthat's why I had the timeout, to make sure the acquire() returns15:54
llu-laptopsileht: are you saying that because the oslo.message's internal time.sleep, the sem.acquire(timeout) will never return?15:55
silehtllu-laptop, yes15:55
silehtllu-laptop, eventlet monkey patch time.sleep, so you can't know when the things happend15:56
silehtllu-laptop, previous I have tried to enable the testr timeout of each tests, I have put something 120 seconds, and all tests that really use rpc stuffs was fail sometimes15:57
llu-laptopsileht: that's the reason I think time.sleep shouldn't break eventlet's sem.acquire, becuase it monkey patch-ed time.sleep15:57
llu-laptopsileht: if it deosn't monkey patch time.sleep, then we should have the problem you mentioned15:58
*** fnaval has joined #openstack-ceilometer15:58
llu-laptopsileht: even in that case, I don't think remove the timeout would help, sem.acquire() just means wait for ever15:59
silehtllu-laptop, if time.sleep is not monkeypatch, when oslo.msg use eventlet.spawn, some tasks are never executed16:00
*** Ruetobas has quit IRC16:01
*** admin0 has quit IRC16:01
silehtllu-laptop, but your right that eventlet semaphore and eventlet time should work together correcly16:01
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Add an alarm service based on taskflow  https://review.openstack.org/9176316:03
silehtllu-laptop, the only things we are sure is a test never returns16:03
*** Ruetobas has joined #openstack-ceilometer16:03
silehtllu-laptop, and each time I have got this backtrace , it was due to a unreleased semphare before app exit16:04
eglynnllu-laptop: so are you concerned that removing the timeout on the sem.acquire() will cause the test to hang in other circumstances?16:05
llu-laptopeglynn: yes, that's my concern16:05
*** gordc has quit IRC16:06
llu-laptopmaybe we should try to use other mechanism than eventlet.semaphore if that's where the devil hide16:06
ildikovdo we have this timeout value set from the beginning?16:07
llu-laptopildikov: sorry, what do you mean by 'beginning'?16:07
ildikovllu-laptop: since that test case was created16:08
* eglynn is being simplistic here ... but would a really long timeout, like say 600s, make any sense?16:08
llu-laptopildikov: no, this was introduced in https://review.openstack.org/#/c/98953/ to fix another gate bug16:08
*** Ruetobas has quit IRC16:08
eglynnsileht: ... or is it the case that setting *any* timeout is problematic?16:09
ildikovllu-laptop: a-ha, ok16:09
silehteglynn, the worse case is the the first runned test started by testr is the latest to finish16:10
ildikovllu-laptop: and did you have a specific reason to set that timeout value or it was for preparing to anything that goes wrong?16:10
silehtbecause testr run tests in concurrency manner16:10
*** idegtiarov has quit IRC16:11
eglynnsileht: ... would that worst-case scenario cause any failures, or just an unexpected ordering of test completion?16:11
llu-laptopildikov: I set the timeout in case some error occurs, so the test case has a chance to report the error and exit, not just wait there for ever16:11
ildikovllu-laptop: ok, I see16:13
*** Ruetobas has joined #openstack-ceilometer16:13
silehteglynn, because of this worst-case we cannot determine how long a test needs to run16:14
ildikoveglynn: I guess that worst case scenarion would cause the same failure in case of even the longer timeout value16:14
silehteglynn, only tests that really use oslo messaging are impacted16:15
silehtthe semaphore just hide the real issue16:15
ildikoveglynn: ... and if we set a that long timeout that it could not happen, then it's more clear to not use timeout at all16:15
silehtthe fact is we have at least one test that sometime never returns and gate timeout16:16
eglynnsileht: what's the real issue being hidden? ... an oslo-messaging bug, or expected oslo-messaging behavior?16:16
silehtI don't know yet16:17
eglynnsileht: a-ha, ok ... do you think that removing the timeout will help unmask the real failure?16:18
eglynnsileht: i.e. maybe give us better clues/evidence to suggest the real root cuase?16:18
ildikovwhat about using the semaphore without timeout, if that really works, until we know what's the real issue?16:18
llu-laptopif there is a real issue between timeout sempahore and oslo.messaging's internal time.sleep, do you guys think using active polling/yield would help:16:23
llu-laptoptime_start = get start time16:23
llu-laptopwhile now() - time_start < certain time:16:23
llu-laptoppoll16:23
llu-laptopyield16:23
eglynnllu-laptop: "poll" == "check if expected notification was received"?16:25
llu-laptoppoll=='check if expected number of notification was received'16:26
eglynnyeah16:26
eglynnsileht: ^^^ does llu-laptop's suggestion sound like a plausible approach to you?16:28
*** Longgeek has quit IRC16:29
llu-laptopb.t.w does this bug happen in the check queue?16:29
llu-laptopor only in the gate-queue?16:29
*** inc0 has quit IRC16:29
llu-laptopI see, it happens in the check queue too16:30
eglynnllu-laptop: whay would that make a difference? (... i.e. check versus gate)16:30
eglynn*why16:30
llu-laptopllu-laptop: don't know. that's why i'm asking16:31
silehteglynn, yes but this add a timeout too, but why not16:31
eglynnllu-laptop: a-ha, k16:31
llu-laptopsileht: by saying timeout, you are referring to yield?16:31
*** _nadya_ has joined #openstack-ceilometer16:32
eglynnsileht: yeah, but not the "bad" sort of timeout with the atexit side effect, right?16:32
llu-laptopeventlet.greenthread.sleep(seconds=0)16:32
sileht"certain time"16:32
eglynnllu-laptop: now() - time_start < certain time16:32
*** safchain has quit IRC16:32
llu-laptopthat would make the test case at assertion, not killed by CI16:33
llu-laptopmake the test case fail at assertion, not failed with atexit16:33
silehtsound good16:33
eglynnllu-laptop: could you code up that idea, so we can run it thru' the gate a few times?16:34
llu-laptopok, i'll try to cook a pathc tomorrow, and do some recheck to see if this is real devil16:34
eglynnllu-laptop: excellent :)16:34
eglynnsileht, llu-laptop: thank you sirs!16:34
*** gordc has joined #openstack-ceilometer16:36
*** llu-laptop has quit IRC16:36
*** openstackgerrit has quit IRC16:38
*** isviridov is now known as isviridov|away16:47
*** zqfan has quit IRC16:50
*** _nadya_ has quit IRC16:50
*** zqfan has joined #openstack-ceilometer16:53
*** harlowja_away is now known as harlowja16:54
*** zqfan has quit IRC16:55
*** zqfan has joined #openstack-ceilometer16:56
*** erecio has quit IRC16:58
*** erecio has joined #openstack-ceilometer16:59
*** rwsu has joined #openstack-ceilometer17:00
*** erecio has quit IRC17:06
*** nati_ueno has joined #openstack-ceilometer17:06
zqfaneglynn: is havana only accept security backport now? I have some backport patch not related to security, if that is true, seems better to abandon them17:06
eglynnzqfan: yes, stable/havana is only for security patches17:08
*** erecio has joined #openstack-ceilometer17:09
*** blinky_ghost has joined #openstack-ceilometer17:17
blinky_ghosthi all, I'm looking for a solution for monitoring my openstack deployment/services. My question is if ceilometer does this job or only monitors resources that run on the cloud, like instances, networks, storage? Thanks17:19
*** rwsu has quit IRC17:30
*** rwsu has joined #openstack-ceilometer17:32
*** fnaval has quit IRC17:36
*** fnaval has joined #openstack-ceilometer17:37
*** fnaval has quit IRC17:41
*** admin0 has joined #openstack-ceilometer17:51
*** admin0 has quit IRC17:55
*** blinky_ghost has quit IRC18:00
*** zul has quit IRC18:03
*** zul has joined #openstack-ceilometer18:03
*** eglynn has quit IRC18:06
*** admin0 has joined #openstack-ceilometer18:11
*** ildikov has quit IRC18:12
*** jdob has quit IRC18:12
*** openstackgerrit has joined #openstack-ceilometer18:12
*** jdob has joined #openstack-ceilometer18:13
*** admin0 has quit IRC18:17
*** admin0 has joined #openstack-ceilometer18:19
*** greghaynes has joined #openstack-ceilometer18:22
*** jdob has quit IRC18:25
*** jdob has joined #openstack-ceilometer18:25
*** admin0 has quit IRC18:28
*** admin0 has joined #openstack-ceilometer18:30
*** erecio has quit IRC18:34
*** erecio has joined #openstack-ceilometer18:34
*** fnaval has joined #openstack-ceilometer18:38
greghaynesHello ceilometerers, I am working on leader election code in Tripleo and someone linked me to https://etherpad.openstack.org/p/icehouse-summit-ceilometer-central-agent - Does anyone know if any of the information there has been acted upon?18:44
greghaynesspecifically, it would be less than ideal for tripleo and ceilometer to use two different systems for this, so hoping to prevent that18:45
*** admin0 has quit IRC18:46
*** zqfan has quit IRC18:47
*** admin0 has joined #openstack-ceilometer18:47
*** erecio has quit IRC18:53
*** Alexei_987 has quit IRC19:00
*** Shaan7 has quit IRC19:02
*** changbl has quit IRC19:02
*** hoangdo has quit IRC19:02
*** erecio has joined #openstack-ceilometer19:03
*** changbl has joined #openstack-ceilometer19:03
*** hoangdo has joined #openstack-ceilometer19:03
*** r3pl4y has quit IRC19:07
*** skelpter has joined #openstack-ceilometer19:08
*** Shaan7 has joined #openstack-ceilometer19:10
*** fnaval has quit IRC19:17
*** _nadya_ has joined #openstack-ceilometer19:17
*** ildikov has joined #openstack-ceilometer19:24
*** skelpter has quit IRC19:25
*** admin0 has quit IRC19:46
*** nati_ueno has quit IRC19:47
*** admin0 has joined #openstack-ceilometer19:47
*** admin0 has quit IRC19:48
*** _nadya_ has quit IRC19:49
openstackgerritFabio Giannetti proposed a change to openstack/python-ceilometerclient: Update python-ceilometerclient to support Keystone V3 API  https://review.openstack.org/9632319:49
*** admin0 has joined #openstack-ceilometer19:51
*** admin0 has quit IRC19:52
*** chmartinez has left #openstack-ceilometer19:53
*** nati_ueno has joined #openstack-ceilometer19:56
*** admin0 has joined #openstack-ceilometer19:59
*** fnaval has joined #openstack-ceilometer20:06
*** Alexei_987 has joined #openstack-ceilometer20:07
*** admin0 has quit IRC20:12
*** admin0 has joined #openstack-ceilometer20:13
*** admin0 has quit IRC20:14
*** boris-42 has quit IRC20:18
*** boris-42 has joined #openstack-ceilometer20:18
openstackgerritA change was merged to openstack/ceilometer: Adds caches for image and flavor in compute agent  https://review.openstack.org/9830120:19
openstackgerritA change was merged to openstack/ceilometer: Use TYPE_GAUGE rather than TYPE_CUMULATIVE  https://review.openstack.org/9807520:23
*** admin0 has joined #openstack-ceilometer20:23
*** nati_ueno has quit IRC20:28
*** julim has quit IRC20:35
*** openstackgerrit has quit IRC20:35
*** openstackgerrit has joined #openstack-ceilometer20:35
*** admin0 has quit IRC20:39
*** nati_ueno has joined #openstack-ceilometer20:44
*** erecio has quit IRC20:53
*** admin0 has joined #openstack-ceilometer20:56
*** jdob has quit IRC20:59
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/9681921:01
*** DrBacchus has quit IRC21:03
*** nati_ueno has quit IRC21:07
*** admin0 has quit IRC21:12
*** thomasem has quit IRC21:35
*** thomasem_ has joined #openstack-ceilometer21:35
*** thomasem_ has quit IRC21:39
openstackgerritA change was merged to openstack/ceilometer: remove default=None for config options  https://review.openstack.org/9936821:42
*** ccrouch has quit IRC21:51
*** kun_huang has quit IRC21:55
*** admin0 has joined #openstack-ceilometer22:01
*** nati_ueno has joined #openstack-ceilometer22:03
*** ccrouch has joined #openstack-ceilometer22:04
*** ccrouch has quit IRC22:04
*** dmsimard is now known as dmsimard_away22:05
*** gordc has quit IRC22:07
*** ccrouch has joined #openstack-ceilometer22:12
*** jergerber has quit IRC22:12
openstackgerritA change was merged to openstack/ceilometer: Adding alarm list filtering by state and meter  https://review.openstack.org/9658222:28
*** admin0 has quit IRC22:33
*** robbybb111 has quit IRC23:03
*** fnaval has quit IRC23:11
*** ccrouch has quit IRC23:44

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