Wednesday, 2014-02-05

*** gordc1 has quit IRC00:01
*** julienvey_ has quit IRC00:08
_cjones_gordc1, you around?00:18
*** xmltok has quit IRC00:48
*** terriyu has quit IRC02:15
*** _cjones_ has quit IRC02:47
*** yfujioka has joined #openstack-ceilometer02:59
*** promulo has quit IRC04:12
*** jd__ has quit IRC04:21
*** jd__ has joined #openstack-ceilometer04:22
*** gordc has joined #openstack-ceilometer04:48
*** gordc has quit IRC05:37
*** mihgen has joined #openstack-ceilometer05:49
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280806:02
*** krast has joined #openstack-ceilometer06:14
*** boris-42_ has quit IRC07:05
*** sayali has joined #openstack-ceilometer07:39
*** sayali_ has joined #openstack-ceilometer07:39
*** sayali_ has quit IRC07:40
*** julienvey_ has joined #openstack-ceilometer08:04
*** eglynn-lunch has joined #openstack-ceilometer08:05
openstackgerritNejc Saje proposed a change to openstack/ceilometer: Adds additional details to alarm notifications  https://review.openstack.org/7010308:07
*** eglynn-lunch has quit IRC08:08
*** eglynn has joined #openstack-ceilometer08:10
*** yfujioka has quit IRC08:16
*** julienvey_ has quit IRC08:21
*** lsmola has quit IRC08:46
*** swann has joined #openstack-ceilometer08:48
*** mihgen has quit IRC08:58
*** lsmola has joined #openstack-ceilometer08:59
*** vrovachev has joined #openstack-ceilometer09:01
*** yassine has joined #openstack-ceilometer09:18
*** lsmola has quit IRC09:19
*** boris-42_ has joined #openstack-ceilometer09:23
*** lsmola has joined #openstack-ceilometer09:32
jd__sileht, eglynn https://review.openstack.org/#/c/70736/09:44
eglynnjd__: looking10:11
eglynnjd__: ... a-ha, too late :)10:12
jd__too late :)10:12
*** mihgen has joined #openstack-ceilometer10:12
jd__thanks though10:12
*** boris-42_ has quit IRC10:28
*** boris-42_ has joined #openstack-ceilometer10:40
openstackgerritMehdi Abaakouk proposed a change to openstack/python-ceilometerclient: Ensure url sent to proxy don't have redundant /  https://review.openstack.org/7036810:50
*** sayali has quit IRC11:22
*** Alexei_987 has joined #openstack-ceilometer11:25
*** lsmola has quit IRC11:46
*** ruhe has joined #openstack-ceilometer11:59
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: nova notifier: disable tests + update sample conf  https://review.openstack.org/7073612:01
*** lsmola has joined #openstack-ceilometer12:02
*** vrovachev has quit IRC12:02
*** ruhe is now known as ruhe_12:06
*** ruhe_ is now known as ruhe12:06
jd__sileht, eglynn https://review.openstack.org/70736 again12:16
silehtjd__, done12:20
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Added debug environment to tox  https://review.openstack.org/7125512:27
*** ruhe is now known as ruhe_12:44
*** ruhe_ has quit IRC12:48
*** lsmola has quit IRC12:53
*** lsmola has joined #openstack-ceilometer12:56
eglynnlsmola: are you around?13:08
lsmolaeglynn: hello13:08
eglynnlsmola: hey ... just wanted a sanity check that I'm actually addressing your "distinct resource counting" requirement13:09
lsmolaeglynn: cool :-)13:09
eglynn(I brought it up on #openstack-horizon on Friday, but sayali didn't know and you weren't around)13:09
eglynnone sec I'll dig up a link to a paste of that convo ...13:09
eglynnhere we go, from the BP whiteboard ... http://paste.openstack.org/show/62294/13:10
lsmolaeglynn: the horizon yesterday?13:10
lsmolaeglynn: oh sorry I thought the meeting13:10
lsmolaeglynn: yeah, sorry I've been on Fosdem13:10
lsmolaeglynn: :-)13:10
eglynnlsmola: ... yeah I guess that fruity Belgian beer might have been involved somehow ;)13:10
lsmolaeglynn: hehe13:11
* lsmola reading13:11
eglynnlsmola: FWIW here's a WIP patch (sqlalchemy only) that gives a flavour of where I'm going with it13:11
eglynn... https://review.openstack.org/7045413:11
*** promulo has joined #openstack-ceilometer13:15
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Fix string-to-boolean casting in queries  https://review.openstack.org/7126213:16
eglynnlsmola: ... the cardinality aggregate sounds like what you were thinking of?13:20
lsmolaeglynn: seems like for 'instance' meter, it does13:21
eglynnlsmola: same principle should apply to any "existence" meter, or?13:22
lsmolaeglynn: I believe yes13:22
*** ruhe has joined #openstack-ceilometer13:22
lsmolaeglynn: if there are no samples with value 0, this should be great13:23
lsmolaeglynn: yeah I think it's great13:25
eglynnlsmola: ... do you mean no *aggregates* with value 0?13:25
lsmolaeglynn: well13:25
eglynnlsmola: ... i.e. s/samples/aggregates/13:25
eglynnlsmola: (... since this is on the statistics API we're not reporting samples directly)13:25
lsmolaeglynn: not sure how we can deal with example when e.g. one resoruce has 10 instance samples and other only 1 in the period13:25
lsmolaeglynn: hm13:26
eglynnlsmola: so cardinality = 2 in that period if both resources in the same group-by bucket13:26
lsmolaeglynn: I wonder if there are cases when the presence meter is not filled, e.g. when suspending? or only when I delete the VM?13:27
eglynnlsmola: ... that I don't know off-hand, easy to check13:27
eglynnlsmola: ... would depend on the filtering done by the instance pollster13:27
lsmolaeglynn: ok, I wonder if there can be short holes in any entity presence13:28
eglynnlsmola: ... if the period is choosen to the be say 50% longer than the polling interval (e.g. 900s by default), that should avoid such "holes"13:28
eglynnlsmola: ... if IIUC what you meant there13:29
lsmolaeglynn: yes13:29
eglynncool enough13:29
lsmolaeglynn: and13:29
lsmolaeglynn: or not, that should be enough13:30
*** jdob has joined #openstack-ceilometer13:30
eglynnlsmola: ... k, thanks, I just wanted to get some initial feedback before investing too much time in the mongo implementation13:30
lsmolaeglynn: though for larger timespans, we might need bigger periods13:30
eglynn(mongo's gonna be messier, /me coming up against the limits of string substitution on the M-R bson code fragments)13:31
eglynnlsmola: sure longer periods are fine, distinct is still distinct ... it's too short periods that are problematic from a spurious zeros PoV13:32
lsmolaeglynn: yeah, but for longer periods, it will give us just that the instance was up13:32
lsmolaeglynn: but since we are not tracking it was down, it is probably impossible to tell it was up 63% of the period13:33
eglynnlsmola: yep, exactly ... you're asking was it up for *any* time during that period, and the API is answering correctly ;)13:34
eglynn... that's just the nature of long periods, the data are distilled down to a coarse grain and some detail is lost13:35
lsmolaeglynn: ok13:35
eglynnlsmola: grandeo, thanks for your time!13:35
eglynnlsmola: ... BTW how was FOSDEM?13:35
lsmolaeglynn: excellent, going to check the review :-)13:36
lsmolaeglynn: it was great13:36
eglynnlsmola: ... was there much openstackery going on in the IaaS/virt room?13:36
lsmolaeglynn: the party was quite overclouded, so we were drinking beers outside13:36
lsmolaeglynn: just watched thiery's presentation, otherwise not that many openstack was there13:37
nprivalova'overclouded' sound good :)13:37
nprivalova*sounds13:37
lsmolaeglynn: more of openstack services used somewhere else13:37
lsmolanprivalova: lol, what a typo13:37
eglynnlsmola, nprivalova: ...  Freudian slip ;)13:37
lsmolaeglynn: and the python room was full, they put it into very small room, so i missed most of the presentations i wanted to see :-)13:38
lsmolaeglynn: but the beer was good :-)13:38
eglynnlsmola: \o/ ... a man with his priorities straight! ;)13:39
lsmolaeglynn: and i brought a lot of chocolate to my wife :-) so everybody were happy in the end13:39
lsmolaeglynn: hehe13:39
lsmolaeglynn: are you coming for devconf?13:40
eglynn.... after a withdrawal from the favour bank, always good to start building up the brownie-points straight away! :)13:40
eglynnlsmola: nope, but I may be in Brno for the w/c March 17th13:41
lsmolaeglynn: ok great13:41
eglynn... /me is assuming that they dye the pivo green for Patrick's Day ;)13:42
lsmolaeglynn: they do13:43
lsmolaeglynn: though it's not very good :-)13:43
*** thomasem has joined #openstack-ceilometer13:43
eglynn... I knew the good barkeep's of Brno wouldn't let an Irishman down in his time of need ;)13:43
lsmolaeglynn: hehe13:44
*** ruhe is now known as ruhe_13:46
*** yassine has quit IRC13:49
*** thomasem has quit IRC13:49
*** yassine_ has joined #openstack-ceilometer13:49
*** thomasem has joined #openstack-ceilometer13:50
*** ruhe_ is now known as ruhe13:54
*** ruhe has quit IRC13:54
nprivalovaeglynn: I remember you were fixing nova-table in docs, weren't you?13:55
eglynnnprivalova: yeap, one sec I'll dig out the review13:55
eglynnnprivalova: https://review.openstack.org/6891813:55
nprivalovaeglynn: ah, it is still on review, ok. We created a bug today about this, I was sure it is merged:) Will update bug's description13:57
eglynnnprivalova: yeah, it's been in gerrit limbo for a while13:58
eglynn(... would be good to get it landed, given how trivial the fix is and how visible the breakage)13:58
nprivalovaeglynn: I cannot assign to you https://bugs.launchpad.net/ceilometer/+bug/127655414:01
eglynnnprivalova: thanks I've taken it14:07
*** eglynn is now known as eglynn-lunch14:10
ityaptinHi! In ceilometer test often happens what expected and actual arguments are found at wrong places in assertEqual. I've found it more than 1700 times. it is ok to fix this?14:13
*** prad_ has joined #openstack-ceilometer14:15
*** tongli has joined #openstack-ceilometer14:16
*** boris-42_ has quit IRC14:31
*** eglynn-lunch is now known as eglynn14:35
*** ruhe has joined #openstack-ceilometer14:35
*** terriyu has joined #openstack-ceilometer14:36
eglynnityaptin: that one always trips me up, so first=expected, second=actual, right?14:38
eglynnityaptin: ... and the ill-efect of transposing them is just a slightly confusing error message, right?14:38
eglynnityaptin: ... well, I would say knock yourself out if you've a reliable way of automating detecting where the atg swap is required14:39
eglynnityaptin: ... but not a good use of your time to do it manually14:40
eglynn(... just my $0.02)14:40
ityaptineglynn: Ok, I can do it automatically)14:43
ityaptinThen I'll register bug for it.14:46
*** ruhe is now known as ruhe_14:46
eglynnityaptin: cool, thanks!14:46
*** ruhe_ is now known as ruhe14:49
tongli@eglynn, do we have some kind of introduction for alarms in doc directory? I could not find any.14:51
openstackgerritA change was merged to openstack/ceilometer: nova notifier: disable tests + update sample conf  https://review.openstack.org/7073614:52
*** ruhe is now known as ruhe_14:53
*** ruhe_ is now known as ruhe14:53
tongli@eglynn, or some sort of manual on how to create an alarm.14:53
eglynntongli: I included some basic alarms usage pointers here ... http://openstack.redhat.com/CeilometerQuickStart14:54
tongli@eglynn, since I am adding new alarms, I was thinking I need to add some doc somewhere to indicate that14:55
tongli@eglynn, but I could not find anything existed, so I thought I do better to ask.14:56
tongli@eglynn, I've put up the client for review, wonder if you have time to take a look. and make sure that I am moving in the right direction.14:57
tongli@eglynn, https://review.openstack.org/#/c/69474/14:57
eglynntongli: tanks ... that's on my todo list14:58
eglynn*thanks14:58
tongli@eglynn, cool, thanks, I will put the server impl in review next couple of days as well.14:58
*** ruhe is now known as ruhe_15:01
*** ruhe_ is now known as ruhe15:05
*** ruhe is now known as ruhe_15:07
*** ruhe_ is now known as ruhe15:09
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Fix formating of compute-nova measurements table  https://review.openstack.org/6891815:10
openstackgerritNejc Saje proposed a change to openstack/ceilometer: Adds additional details to alarm notifications  https://review.openstack.org/7010315:15
nsaje_Hey eglynn, would you please take a look at  https://review.openstack.org/70103 when you have time?15:20
eglynnnsaje_: sure, it's on my list15:26
*** mihgen has quit IRC15:26
nsaje_eglynn: Great, thank you!15:27
*** ruhe is now known as ruhe_15:27
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Handle engine creation inside of Connection object  https://review.openstack.org/6785115:31
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Added debug environment to tox  https://review.openstack.org/7125515:31
*** ruhe_ has quit IRC15:32
*** boris-42_ has joined #openstack-ceilometer15:32
flwangjd__: ping15:41
flwangdhellmann: ping15:44
jd__flwang: pong15:47
flwangjd__: maybe here is not the best place to request, but would you mind taking a look at this https://review.openstack.org/#/c/70935/ ?15:48
jd__approved15:48
*** gordc has joined #openstack-ceilometer15:48
flwangjd__: cool, thanks a lot15:49
*** dhellmann is now known as dhellmann_15:52
openstackgerritgordon chung proposed a change to openstack/ceilometer: sync oslo common code  https://review.openstack.org/7112415:52
*** mihgen has joined #openstack-ceilometer15:56
*** dhellmann_ is now known as dhellmann16:15
tongli@Alexei_987, ping16:29
*** Alexei_987 has quit IRC16:40
*** terriyu has quit IRC16:52
*** _cjones_ has joined #openstack-ceilometer17:10
*** _cjones_ has joined #openstack-ceilometer17:10
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Revert "Revert "Ensure we are not exhausting the sqlalchemy pool""  https://review.openstack.org/7033717:16
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: storage: store recording timestamp  https://review.openstack.org/7016617:17
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Move ceilometer-send-counter to a console script  https://review.openstack.org/7008317:17
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: service: re-enable eventlet just for sockets  https://review.openstack.org/7033817:25
promulohey _cjones_ gordc, after a full re-install of the host machine I'm now able to see the meter list using stable/havana17:31
promulonot sure why but at least now it's working17:31
*** yassine_ has quit IRC17:42
*** yassine has joined #openstack-ceilometer17:43
*** yassine has quit IRC17:43
nsaje_promulo: good to hear :)17:49
*** viktors has quit IRC17:54
*** dhellmann is now known as dhellmann_17:56
*** nadya has joined #openstack-ceilometer18:00
*** nadya is now known as Guest8386318:01
_cjones_promulo, Nice.18:03
_cjones_promulo, This is why I suggested doing this all in a VM... so you can skip that step.18:04
promulo_cjones_: yeah... I guess that all the clone->stack->unstack->clean cycle somehow messed up the system instalation18:09
promuloprobably it came to a point that a simple fallback from master to havana wouldn't be enough :)18:09
_cjones_promulo, unless you really need to.... I'd say keep your host prestine, do work in a VM.18:09
_cjones_promulo, I've noticed that sometimes "unstack" doesn't clean up its mess. Neither does "clean".18:10
*** yassine has joined #openstack-ceilometer18:11
promulome too... btw a lot of ceilometer processes keep running after unstack18:11
_cjones_promulo, yes. I always do a "ps -ef | grep python" after an unstack, and manually clean up messes.18:20
_cjones_promulo, I have yet to debug why those things linger around. I suspect they are just bugs in stack/unstack process.18:20
promuloyeah I was thinking of filling a bug if there isn't one already... and later try to fix.. shouldn't be very hard I guess18:21
*** zul has quit IRC18:33
*** zul has joined #openstack-ceilometer18:37
_cjones_promulo, prolly a good idea. Might be worth while doing a root cause analysis to tack on with the bug in case someone is more familiar and can squash it prior to you... but I've found that for stuff like that it is usually left to folks like us to fix.18:39
*** yassine has quit IRC18:39
*** Guest83863 is now known as _nadya_18:41
_nadya_gordc, hi! please take a look on my comemnts https://review.openstack.org/#/c/67704/18:43
*** gordc1 has joined #openstack-ceilometer18:46
gordc1_nadya_: sure. i'll take a look18:46
_nadya_gordc1: thanks!18:47
*** gordc has quit IRC18:48
*** _nadya_ has quit IRC18:49
openstackgerritJenkins proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/6990519:34
*** mihgen has quit IRC19:39
*** ildikov_ has joined #openstack-ceilometer20:02
*** eglynn has quit IRC20:20
*** ildikov_ has quit IRC20:25
*** xmltok has joined #openstack-ceilometer20:43
*** tongli has quit IRC21:18
*** promulo has quit IRC21:50
*** jdob has quit IRC22:07
*** promulo has joined #openstack-ceilometer22:23
*** thomasem has quit IRC22:30
openstackgerritCyril Roelandt proposed a change to openstack/ceilometer: Replace 'a alarm' by 'an alarm'  https://review.openstack.org/7140222:47
*** jhurlbert has joined #openstack-ceilometer23:31
*** promulo has quit IRC23:36
*** yfujioka has joined #openstack-ceilometer23:38
openstackgerritgordon chung proposed a change to openstack/ceilometer: meter table contains redundant/duplicate data  https://review.openstack.org/6578623:39
*** gordc1 has quit IRC23:45
openstackgerritLuis A. Garcia proposed a change to openstack/ceilometer: Allow individual lazy _() imports  https://review.openstack.org/7082423:48
yfujiokajd__: hello, I hope you to a review of blueprints. What I have to do?23:48
*** prad_ has quit IRC23:50
openstackgerritYuuichi Fujioka proposed a change to openstack/ceilometer: fix column name and alignment  https://review.openstack.org/6952423:52
openstackgerritYuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network  https://review.openstack.org/6047323:53
openstackgerritYuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network-from-opendaylight  https://review.openstack.org/6389023:53
yfujiokaui23:55
yfujiokasorry, I missed23:55
*** jhurlbert has quit IRC23:59

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