Tuesday, 2015-09-08

openstackgerritZhiQiang Fan proposed openstack/ceilometer: Prevent ceilometer expirer from causing deadlocks  https://review.openstack.org/16556600:19
*** ankita_wagh has joined #openstack-ceilometer00:35
*** ankita_wagh has quit IRC00:40
*** r-mibu has joined #openstack-ceilometer00:47
*** ankita_wagh has joined #openstack-ceilometer01:03
*** jd__ has quit IRC01:12
*** jd__ has joined #openstack-ceilometer01:12
openstackgerritTakashi NATSUME proposed openstack/ceilometer: Fix typos in gnocchi.py and converter.py  https://review.openstack.org/22113001:13
gzhair-mibu: hi01:16
r-mibugzhai, hi01:16
gzhaiHave you ever thought about the event-alarm fire policy? one-shot or periodic?01:16
r-mibuperiodic as the same way wa existing evaluators?01:17
r-mibus/wa/as/01:17
gzhaiJust want to discuss it :)01:18
*** ankita_w_ has joined #openstack-ceilometer01:18
r-mibuok01:19
gzhaithreshold alarm is limited periodic: reset to "unknown" when get insufficient data, so alarm again for new valid data.01:19
gzhaiBut only fire once for continuous valid data.01:19
r-mibuyes, if you set 'repeat_actions=False'01:20
gzhaifrom end user point view, I think event-alarm should be periodic01:20
gzhaiat least for a period.01:20
gzhaiWhat do you think about it?01:21
r-mibuyou mean repeat?01:21
r-mibuand not periodic evaluation, right?01:21
*** ankita_wagh has quit IRC01:21
gzhainot repeat. repeat means continuous fire even for one event, right?01:22
r-mibuok, so periodic notification like "fire alarm only once per 10 min"?01:22
gzhaionce trigger, fire for ever.01:22
gzhaiyes, in 10 min, fire upon each event comply with event-rule.01:23
gzhaiis it reasonable?01:23
r-mibuinteresting!01:23
r-mibuthat would be nice, if there are lots of alarming01:24
gzhaiFrom end user point of view, I want know when the event happen. e.g. "instance.crash" event, one-shot seems not enough.01:24
r-mibui think such mechanism can be implemented as one of notification driver, so that we can use in other alarming01:25
gzhainotification driver do periodic alarm?01:26
r-mibuno just verify last fired time, and ignore if it already sent in certein period01:27
r-mibus/no/no,/01:27
r-mibuumm, creating check in event-alarm evaluator might be smarter01:29
r-mibuwhat do you think?01:29
r-mibuor, how about creating periodic evaluator for events?01:30
gzhaiperiodic evaluator is not perfect, as we already have event-driven evaluator.01:31
r-mibuyeah, but event-driven evaluator cannot make sure that can send alarm periodically, since it depend on events01:31
gzhailet's have our goal first. e.g. create a event-alarm for one event1, then event1, event1, event2 and event1 comes. So how many times should it fired? maybe periodic is misleading here:(01:32
gzhaicurrent only 1 fire.01:32
gzhaiwe need think if we need 2 or 3 fires.01:33
r-mibuumm, i may miss something, could you tell more detail why repeat_actions doesn7t work for this use case?01:36
r-mibus/doesn7t/dosen't/01:36
gzhairepeat_actions seems fire for ever as long as it remains "alarm" state?01:41
*** nadya has joined #openstack-ceilometer01:42
r-miburight01:42
gzhaiso for the case, (event1, event1, event2 and event1 ) and (event1) have same result with repeat_actions.01:43
gzhaiend user just want one fire for each event.01:43
*** nadya has quit IRC01:46
r-mibuyes, we can do by setting alarms for each event01:46
gzhaiIs it reasonable for event-alarm?01:47
r-mibuyes, i think so01:47
r-mibuwhat is the case user wants to receive event alarm once again?01:48
r-mibuthe last event1 occured after 10 min passed?01:49
gzhaie.g end user want to catch instance crash, so create a alarm for "instance.crash"(just for e.g.). But only get alarm for 1st crash. Other crashes are missed.01:49
gzhailast event1 is in 10 mins.01:50
r-miburight if repeat_actions=False01:50
r-mibunote you can specify instance_id in alarm definition as query01:51
*** david-ly_ has joined #openstack-ceilometer01:51
*** david-lyle has quit IRC01:54
gzhair-mibu: I'll check the code again. Thanks for you explanation.01:59
r-mibunp, thank you for showing interest to event-alarm!01:59
openstackgerritRyota MIBU proposed openstack/aodh: event-alarm: add event wrapper class  https://review.openstack.org/22108002:18
openstackgerritRyota MIBU proposed openstack/aodh: event-alarm: add alarm wrapper class  https://review.openstack.org/22108102:18
openstackgerritRyota MIBU proposed openstack/aodh: event-alarm: add unit tests for various trait types  https://review.openstack.org/22108202:18
openstackgerritMerged openstack/aodh: event-alarm: fix unit tests to check stored alrams  https://review.openstack.org/22042203:01
openstackgerritMerged openstack/aodh: Refactor event-alarm caching  https://review.openstack.org/22013303:01
openstackgerritTakashi NATSUME proposed openstack/ceilometer: Fix typos in gnocchi.py and converter.py  https://review.openstack.org/22113003:02
*** nijaba has joined #openstack-ceilometer03:10
*** nijaba has quit IRC03:10
*** nijaba has joined #openstack-ceilometer03:10
*** liusheng has quit IRC03:31
*** liusheng has joined #openstack-ceilometer03:32
*** liusheng_ has joined #openstack-ceilometer03:33
*** r-mibu has quit IRC03:44
*** r-mibu has joined #openstack-ceilometer03:44
*** julim has quit IRC03:47
openstackgerritRyota MIBU proposed openstack/python-ceilometerclient: refactor: move 'repeat_actions' to common args  https://review.openstack.org/21715903:52
*** deepthi has joined #openstack-ceilometer04:03
*** dikonoor has joined #openstack-ceilometer04:14
*** liusheng_ has quit IRC04:25
*** liusheng_ has joined #openstack-ceilometer04:26
*** yprokule has joined #openstack-ceilometer04:29
*** ildikov has quit IRC04:53
*** prnk28 has joined #openstack-ceilometer04:59
*** ankita_w_ has quit IRC05:06
*** nadya has joined #openstack-ceilometer05:14
*** ankita_wagh has joined #openstack-ceilometer05:30
*** ankita_wagh has quit IRC05:48
*** nadya has quit IRC05:55
*** nadya has joined #openstack-ceilometer05:57
*** nadya has quit IRC05:58
*** alexpilotti_ has joined #openstack-ceilometer06:01
*** mahatic has joined #openstack-ceilometer06:06
*** nadya has joined #openstack-ceilometer06:13
*** mahatic has left #openstack-ceilometer06:21
openstackgerritMehdi Abaakouk (sileht) proposed openstack/aodh: Cleanup keystonemiddleware configuration  https://review.openstack.org/20898906:30
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: TEST: Validate integration test  https://review.openstack.org/21999906:40
*** liusheng_ has quit IRC06:42
*** liusheng_ has joined #openstack-ceilometer06:43
*** lsmola has joined #openstack-ceilometer06:47
*** ildikov has joined #openstack-ceilometer07:10
*** liusheng has quit IRC07:23
*** yassine__ has joined #openstack-ceilometer07:30
*** cdent has joined #openstack-ceilometer07:42
*** shardy has joined #openstack-ceilometer07:44
cdentsileht: new gabbi should be available in <5min. Thanks for the fix.07:48
silehtcdent, welcome07:48
cdentits up now07:48
*** safchain has joined #openstack-ceilometer07:51
*** dikonoor has quit IRC07:55
openstackgerritZhiQiang Fan proposed openstack/ceilometer: remove duplicate log exception message  https://review.openstack.org/22118307:59
*** Ala has joined #openstack-ceilometer08:16
*** gzhai has quit IRC08:30
*** exploreshaifali has joined #openstack-ceilometer08:31
openstackgerritMerged openstack/ceilometer: Fix typos in gnocchi.py and converter.py  https://review.openstack.org/22113008:52
*** prnk28 has quit IRC08:56
openstackgerritMerged openstack/ceilometer: Updated from global requirements  https://review.openstack.org/22110908:56
*** ildikov has quit IRC09:02
*** afazekas_ has joined #openstack-ceilometer09:22
*** nadya has quit IRC09:22
*** afazekas_ has joined #openstack-ceilometer09:23
*** shardy_ has joined #openstack-ceilometer09:33
openstackgerritlixiangjun proposed openstack/ceilometer: Spelling mistake of comment in api/controllers/v2/query.py  https://review.openstack.org/22121709:34
*** shardy has quit IRC09:34
*** nadya has joined #openstack-ceilometer09:37
*** shardy_ has quit IRC09:38
*** shardy has joined #openstack-ceilometer09:39
*** ildikov has joined #openstack-ceilometer09:39
openstackgerritJulien Danjou proposed openstack/ceilometer: doc: update devstack usage  https://review.openstack.org/22123010:00
*** ildikov has quit IRC10:00
*** guy_rodrigue has joined #openstack-ceilometer10:06
*** guy_rodrigue has left #openstack-ceilometer10:06
jd__connection = postgresql+pymysql://root:foobar@127.0.0.1/keystone?client_encoding=utf810:09
jd__ah, devstack…10:09
* jd__ sighs10:09
*** ildikov has joined #openstack-ceilometer10:14
*** julim has joined #openstack-ceilometer10:29
*** julim has quit IRC10:33
openstackgerritRyota MIBU proposed openstack/aodh: event-alarm: add unit tests for various trait types  https://review.openstack.org/22108210:47
*** ildikov has quit IRC11:04
*** ildikov has joined #openstack-ceilometer11:08
*** thorst has joined #openstack-ceilometer11:26
*** jaypipes has joined #openstack-ceilometer11:31
*** gordc has joined #openstack-ceilometer11:31
*** dobson has quit IRC11:39
*** dobson has joined #openstack-ceilometer11:45
*** shardy_ has joined #openstack-ceilometer11:49
*** shardy has quit IRC11:50
*** deepthi has quit IRC11:51
*** shardy_ has quit IRC11:55
*** shardy has joined #openstack-ceilometer11:55
*** safchain has quit IRC12:02
*** alexpilotti_ has quit IRC12:03
*** eandersson has joined #openstack-ceilometer12:11
*** liusheng has joined #openstack-ceilometer12:21
*** liusheng has quit IRC12:27
*** safchain has joined #openstack-ceilometer12:29
*** edmondsw has joined #openstack-ceilometer12:41
*** ildikov has quit IRC12:51
*** yprokule has quit IRC12:54
*** yprokule has joined #openstack-ceilometer12:56
*** ildikov has joined #openstack-ceilometer13:02
openstackgerritMerged openstack/ceilometer: integration: Add debugging information  https://review.openstack.org/21914913:02
*** tongli has joined #openstack-ceilometer13:10
*** changbl has quit IRC13:11
*** prnk28 has joined #openstack-ceilometer13:21
*** pradk has joined #openstack-ceilometer13:22
*** exploreshaifali has quit IRC13:27
*** rbak has joined #openstack-ceilometer13:50
*** rbak has quit IRC13:50
*** r-daneel has joined #openstack-ceilometer13:57
*** shardy_ has joined #openstack-ceilometer14:01
*** shardy has quit IRC14:03
*** shardy_ has quit IRC14:07
*** shardy has joined #openstack-ceilometer14:07
openstackgerritRyota MIBU proposed openstack/aodh: event-alarm: add event wrapper class  https://review.openstack.org/22108014:08
openstackgerritRyota MIBU proposed openstack/aodh: event-alarm: add alarm wrapper class  https://review.openstack.org/22108114:08
openstackgerritRyota MIBU proposed openstack/aodh: event-alarm: add unit tests for various trait types  https://review.openstack.org/22108214:08
*** rbak has joined #openstack-ceilometer14:13
*** rbak has quit IRC14:15
*** rbak has joined #openstack-ceilometer14:18
*** vishwanathj has joined #openstack-ceilometer14:32
gordcAodh 1.0.0 is live14:44
gordcildikov: any progress on the aodh doc?14:45
*** david-ly_ is now known as david-lyle14:53
*** yprokule has quit IRC14:59
*** yprokule has joined #openstack-ceilometer15:00
openstackgerritgordon chung proposed openstack/python-ceilometerclient: do not generate meter links on resource-list  https://review.openstack.org/22134815:03
openstackgerritMerged openstack/aodh: event-alarm: add event wrapper class  https://review.openstack.org/22108015:03
*** nadya has quit IRC15:04
*** changbl has joined #openstack-ceilometer15:14
*** prnk28 has quit IRC15:21
*** afazekas_ has quit IRC15:28
*** thorst has quit IRC15:35
*** dobson has quit IRC15:36
*** prnk28 has joined #openstack-ceilometer15:38
*** dobson has joined #openstack-ceilometer15:39
cdentjd__: you around, have a couple of metricd questions15:41
*** devlaps has joined #openstack-ceilometer15:43
*** r-mibu has left #openstack-ceilometer15:45
*** safchain has quit IRC15:45
jd__cdent: o/15:47
cdentyo, I just want to confirm that the reason you are using asyncio in the process() loop in metricd is because you explicitly want to be able to be process()-ing more than once, not just that you wanted a convenient way to sleep between process()-es?15:48
cdentthe assumption being that one process() may last longer than the delay15:49
cdentactually nevermind15:49
cdentsince you're not yielding it can't happen15:50
cdentso, then, why did you use asyncio instead of sleeping?15:50
*** yassine__ has quit IRC15:51
*** nadya has joined #openstack-ceilometer15:55
jd__cdent: in case we'd need to schedule more than one task15:58
jd__cdent: but currently you're right there's no need for it15:58
*** r-daneel has quit IRC15:58
cdentI'm changing things pretty drastically in there to fix: https://bugs.launchpad.net/gnocchi/+bug/1491339 and https://bugs.launchpad.net/gnocchi/+bug/149306015:58
openstackLaunchpad bug 1491339 in Gnocchi "If gnocchi-metricd has import errors in startup they are swallowed" [Undecided,New] - Assigned to Chris Dent (chdent)15:58
openstackLaunchpad bug 1493060 in Gnocchi "if gnocchi-metricd fails start the storage client it processes no metrics and does not retry" [Undecided,New]15:58
jd__cdent: if you're not happy with it you can ask haypo is a master of it15:58
cdent(partially fix)15:58
cdentPool doesn't work well for what we're trying to do there, easier to just use a collection of Processes15:59
jd__cdent: replacing it I think he's also an option15:59
jd__s/he/it/15:59
jd__I'm not really opiniated – whatever works best15:59
cdentI'll just push the code and we can look at it15:59
jd__cool15:59
*** promulo__ has quit IRC16:03
*** safchain has joined #openstack-ceilometer16:04
openstackgerritJulien Danjou proposed openstack/ceilometer: doc: update devstack usage  https://review.openstack.org/22123016:20
*** nadya has quit IRC16:26
*** Ala has quit IRC16:29
*** ankita_wagh has joined #openstack-ceilometer16:30
*** thorst has joined #openstack-ceilometer16:31
openstackgerritChris Dent proposed openstack/gnocchi: Make metricd use a collection of Process not Pool  https://review.openstack.org/22140016:39
cdentjd__: ^16:39
*** Guest60469 is now known as mgagne16:40
*** mgagne has joined #openstack-ceilometer16:40
*** prnk28 has quit IRC16:55
*** harlowja has joined #openstack-ceilometer17:03
*** ankita_wagh has quit IRC17:05
*** afazekas_ has joined #openstack-ceilometer17:05
*** eandersson has quit IRC17:07
*** yprokule has quit IRC17:15
*** afazekas_ has quit IRC17:24
*** lsmola has quit IRC17:29
*** ankita_wagh has joined #openstack-ceilometer17:32
*** safchain has quit IRC17:37
*** prnk28 has joined #openstack-ceilometer17:39
*** exploreshaifali has joined #openstack-ceilometer17:42
*** liusheng_ has quit IRC17:43
*** liusheng_ has joined #openstack-ceilometer17:44
openstackgerritChris Dent proposed openstack/gnocchi: Retry on initial contact with swift  https://review.openstack.org/22143017:56
*** cdent has quit IRC18:02
*** r-daneel has joined #openstack-ceilometer18:02
*** nadya has joined #openstack-ceilometer18:07
*** sachin has quit IRC18:18
*** openstackgerrit has quit IRC18:31
*** openstackgerrit has joined #openstack-ceilometer18:31
openstackgerritJulien Danjou proposed openstack/ceilometer: doc: update devstack usage  https://review.openstack.org/22123018:35
ildikovgordc: sorry, snowed under meetings today fully :(18:45
ildikovgordc: in progress, will upload some material to gerrit this week18:45
gordcildikov: cool cool. no worries just wanted to check19:02
gordcildikov: i'll draft up an announcement email and mention the upgrade/migration doc will be published soon.19:02
openstackgerritgordon chung proposed openstack/python-ceilometerclient: add limit support  https://review.openstack.org/22146119:04
ildikovgordc: ok, cool, tnx19:06
ildikovgordc: although the versioned OpenStack Manuals will be available later I think along with the official Liberty release19:06
gordcildikov: sure. i think we just need to send it to openstack-dev and openstack-operators mailing list once ready19:07
*** ankita_wagh has quit IRC19:10
*** ankita_wagh has joined #openstack-ceilometer19:12
ildikovgordc: yeap, sounds like a plan :)19:17
gordczigo: pradk: http://lists.openstack.org/pipermail/openstack-dev/2015-September/073897.html19:45
gordcjust an fyi19:45
gordcaodh 1.0.0 has been tagged and release19:46
pradkgordc, great, thx19:46
zigogordc: Ok. I'll try to find the time to package it.19:47
zigogordc: It's on my todo...19:48
zigogordc: I'm currently trying to finish all of Liberty b3.19:48
zigoSo far, I've done Keystone, Nova, Neutron, Cinder, Ironic, Heat, and I'm nearly done with Horizon.19:48
zigoCeilometer is next on my list ...19:48
zigoI believe I did all dependencies for it already.19:48
zigoIs all of the alarming out of Ceilometer in Liberty b3 ?19:49
gordczigo: cool cool. thanks very much!19:49
openstackgerritMerged openstack/ceilometer: Spelling mistake of comment in api/controllers/v2/query.py  https://review.openstack.org/22121719:49
gordczigo: so alarming code still exists in Ceilometer for liberty. but it is deprecated19:49
zigoOk, so I wont remove the packages just yet, then?19:49
zigoMaybe do that in Mitaka ?19:49
gordczigo: correct, don't remove just yet. we are targetting removal for Mitaka.19:50
zigoOk, good to know.19:50
gordcthanks again zigo.19:51
pradkhttps://pypi.python.org/pypi/aodh still points to  0.0.1.dev4004 ?19:52
*** ddieterly has joined #openstack-ceilometer19:53
gordcpradk: strange. i can see the tarball. http://tarballs.openstack.org/aodh/19:54
*** prnk28 has quit IRC19:58
gordcpradk: apparently openstack-server-release-jobs only pushes to tarballs.openstack.org. we need another job for pypi20:01
pradkgordc, ah ok, thought that happened automatically20:01
openstackgerritMerged openstack/ceilometer: polling: remove deprecated agents  https://review.openstack.org/21249820:05
openstackgerritMerged openstack/ceilometer: doc: update notification_driver  https://review.openstack.org/21919720:05
gordcpradk: from packaging pov, does it make a difference if something exists in pypi?20:05
pradkgordc, tag should be enough, but it would be good to have it on pypi20:11
gordcpradk: kk20:13
*** nadya has quit IRC20:14
*** cdent has joined #openstack-ceilometer20:20
*** exploreshaifali has quit IRC20:31
*** tiny-hands has joined #openstack-ceilometer20:35
*** ekarlso has quit IRC20:38
*** shardy_ has joined #openstack-ceilometer20:41
*** shardy has quit IRC20:42
*** tongli has quit IRC20:44
*** clduser_ has quit IRC20:44
*** shardy_ has quit IRC20:46
*** ankita_wagh has quit IRC20:46
*** shardy has joined #openstack-ceilometer20:47
*** cdent has quit IRC20:54
*** nihilifer has quit IRC20:55
*** nihilifer1 has joined #openstack-ceilometer20:55
*** ekarlso has joined #openstack-ceilometer20:57
*** ekarlso has quit IRC20:57
*** ekarlso has joined #openstack-ceilometer20:57
*** ankita_wagh has joined #openstack-ceilometer21:01
*** clduser has joined #openstack-ceilometer21:07
*** changbl has quit IRC21:07
*** nadya has joined #openstack-ceilometer21:14
*** nadya has quit IRC21:19
*** tiny-hands has quit IRC21:31
*** devlaps has quit IRC21:38
*** edmondsw has quit IRC21:57
*** Ephur has joined #openstack-ceilometer22:01
*** pradk has quit IRC22:22
*** r-daneel has quit IRC22:26
*** gordc has quit IRC22:31
*** ankita_w_ has joined #openstack-ceilometer22:38
*** ankita_wagh has quit IRC22:38
*** nihilifer1 is now known as nihilifer22:41
*** thorst has quit IRC22:43
*** ankita_wagh has joined #openstack-ceilometer22:51
*** ankita___ has joined #openstack-ceilometer22:52
*** ankita_w_ has quit IRC22:55
*** ankita_wagh has quit IRC22:56
*** ddieterly has quit IRC23:01
*** rbak has quit IRC23:02
*** zqfan_ has joined #openstack-ceilometer23:05
openstackgerritZhiQiang Fan proposed openstack/ceilometer: remove duplicate log exception message  https://review.openstack.org/22118323:12
*** nadya has joined #openstack-ceilometer23:16
*** nadya has quit IRC23:20
*** patrickeast has joined #openstack-ceilometer23:35
openstackgerritPatrick East proposed openstack/ceilometer: Update event_definitions for Cinder Image Cache  https://review.openstack.org/22154523:51
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Fix for AttributeError in discovery when keystone unavailable  https://review.openstack.org/22154623:55
*** ddieterly has joined #openstack-ceilometer23:56

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