Wednesday, 2014-01-22

*** flwang has quit IRC00:02
*** ok_delta__ has quit IRC00:08
*** ok_delta has quit IRC00:08
*** gordc has quit IRC00:09
*** flwang has joined #openstack-ceilometer00:20
*** yassine has quit IRC00:21
*** zul has joined #openstack-ceilometer00:37
*** Alexei_987 has quit IRC01:16
openstackgerritLee Li proposed a change to openstack/python-ceilometerclient: Using common method 'bool_from_string' from oslo strutils  https://review.openstack.org/6831003:30
openstackgerritMarc Solanas proposed a change to openstack/python-ceilometerclient: Checking the type of auth_token, and fixing it if necessary  https://review.openstack.org/5805105:00
*** AMike has joined #openstack-ceilometer05:06
*** AMike has joined #openstack-ceilometer05:06
*** flwang has quit IRC05:36
*** flwang has joined #openstack-ceilometer05:59
*** ildikov_ has quit IRC06:02
*** AMike has quit IRC06:43
*** ildikov_ has joined #openstack-ceilometer07:01
*** nadya__ has joined #openstack-ceilometer07:05
*** sayali has joined #openstack-ceilometer07:27
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280807:56
*** s2r2_ has joined #openstack-ceilometer08:10
*** SergeyLukjanov_ is now known as SergeyLukjanov08:23
openstackgerritA change was merged to openstack/ceilometer: Implement the /v2/samples/<sample-id> API  https://review.openstack.org/6181408:30
*** Alexei_987 has joined #openstack-ceilometer08:44
Alexei_987jd__: Hi could you please review db.session patches? https://review.openstack.org/#/q/status:open+project:openstack/ceilometer+branch:master+topic:oslo-db-session,n,z08:46
Alexei_987sileht: ^08:46
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Fix recursive_keypairs output  https://review.openstack.org/6770408:49
*** yassine has joined #openstack-ceilometer09:06
*** eglynn has joined #openstack-ceilometer09:08
*** sayali has quit IRC09:11
*** sayali has joined #openstack-ceilometer09:13
*** sayali has quit IRC09:19
*** SergeyLukjanov is now known as SergeyLukjanov_a09:27
*** SergeyLukjanov_a is now known as SergeyLukjanov_09:28
*** SergeyLukjanov_ is now known as SergeyLukjanov09:40
*** flwang has quit IRC09:47
*** SergeyLukjanov is now known as SergeyLukjanov_09:47
*** boris-42 has quit IRC10:02
*** boris-42 has joined #openstack-ceilometer10:04
*** sayali has joined #openstack-ceilometer10:10
*** sayali has quit IRC10:20
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: eventlet: stop monkey patching  https://review.openstack.org/6814510:24
*** flwang has joined #openstack-ceilometer10:31
eglynnllu: I wonder could I ask you to take another look at https://review.openstack.org/67161 ?10:45
eglynn(... earlier patch set +2'd by you, latest just has very minor deltas from that version)10:45
*** yassine_ has joined #openstack-ceilometer10:58
*** yassine has quit IRC10:59
openstackgerritA change was merged to openstack/ceilometer: Update oslo  https://review.openstack.org/6566111:10
*** sayali has joined #openstack-ceilometer11:17
*** sayali_ has joined #openstack-ceilometer11:17
openstackgerritA change was merged to openstack/ceilometer: Fix to handle null threshold_rule values  https://review.openstack.org/6675711:23
openstackgerritA change was merged to openstack/ceilometer: Use DEFAULT section for dispatcher in doc  https://review.openstack.org/6690811:24
openstackgerritA change was merged to openstack/ceilometer: Return trait type from Event api  https://review.openstack.org/6693611:25
openstackgerritA change was merged to openstack/ceilometer: Convert alarm timestamp to PrecisionTimestamp  https://review.openstack.org/6313111:25
jd__some merge finally, nice11:31
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Use swift master  https://review.openstack.org/6815011:33
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Allow pytidylib to be installed from external source  https://review.openstack.org/6814911:33
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: tests: kill all started processes on exit  https://review.openstack.org/6550911:33
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: storage: bases of a Cassandra driver  https://review.openstack.org/6277911:33
*** nadya__ has quit IRC11:33
*** sayali_ has quit IRC11:42
*** asalkeld has left #openstack-ceilometer11:47
*** boris-42 has quit IRC11:53
*** RemyS has joined #openstack-ceilometer11:54
*** yassine_ has quit IRC11:56
*** boris-42 has joined #openstack-ceilometer11:56
*** yassine has joined #openstack-ceilometer11:57
*** RemyS has quit IRC11:59
*** AMike has joined #openstack-ceilometer12:12
*** AMike has joined #openstack-ceilometer12:12
*** s2r2_ has quit IRC12:24
*** s2r2_ has joined #openstack-ceilometer12:26
eglynnfrom the project meeting last night ...12:29
eglynn"#agreed I2 branch cut around 7am MST tomorrow. Tags expected toward the end of day"12:29
eglynn... it's currently ~5:29am MST12:29
eglynn... so we're at T minus ~90mins for i-212:30
eglynnany chance of core eyes on https://review.openstack.org/67161 before then?12:30
eglynn(given the current "stuck" state of the gate, it's a long shot for anything to land ... but /me would like to see stuff +1'd in time nonetheless)12:32
eglynnfolks, is there anything else currently targeted at i-2 with outstanding reviews?12:35
eglynnjd__: ... for example should https://blueprints.launchpad.net/ceilometer/+spec/sample-api be moved from "needs code review" to "implemented"?12:37
eglynn(... as all the proposed patches seem to have landed)12:37
silehteglynn, I think GET /sample-statistics/?q=  is not implemented13:03
silehteglynn, I take a look to your review13:04
eglynnsileht: a-ha, gotcha ... thank you sir!13:04
eglynnI guess we might as well hustle to get as much approved as poss before i-2 is tagged13:05
jd__I'm going to mark this as implemented, we can create a new bp is someone wants to work on /sample-statistics13:05
eglynn... in the faint hope that the gate swampage magically resolves13:06
eglynnjd__: fair enough13:06
ekarlsois there any reason why ceilometer tries to clear out my database when I have enabled ceilometer-acompute ? I only want it to install the compute agent in a compute host :p13:08
ekarlsoor doesn't the current devstack integration allow this13:08
ildikov_jd__, eglynn: the  POST /sample is ready for sample-api?13:09
jd__ildikov_: I think so13:09
eglynn... /me has forgotten the functional difference between the new POST /v2/sample and the old POST /v2/meters/<meter_name>13:12
eglynn... is the one just a convenience wrapper for the other?13:12
ildikov_jd__: I haven't seen it in the SamplesController, but probably I missed it13:13
jd__ildikov_: I also might be wrong13:16
jd__eglynn: convenience wraper13:17
jd__wrapper13:17
ekarlsohttp://paste.fedoraproject.org/70632/13903965 < that's my local.conf and this is the failure: http://fpaste.org/70633/13903966/13:17
eglynnjd__: cool, gotcha!13:17
eglynnekarlso: I'd imagine the devstack ceilo setup assume an allinone-stylee deployment13:18
jd__eglynn: I'd even say correct REST interface, POSTing in /meters a sample is not really RESTy…13:18
eglynnjd__: yep, the REST purists would shudder at that ;)13:18
ekarlsoeglynn: that's *sadface*13:19
eglynn... or are they known as RESTafarians? ;)13:19
eglynnekarlso: easy to comment out the cleanup_ceilometer function from devstack/lib/ceilometer to avoid the DB drop13:20
ekarlsowill do :)13:20
jd__eglynn: is there a corollary that makes me a REST purist thn? :)13:20
jd__*then13:20
ildikov_jd__: I've checked 3 patches and I haven't seen the post under SamplesController, I assume that would be right place for it13:21
jd__RESTafarians, I like that :)13:21
jd__we should arREST you for that13:21
eglynnjd__: the shibboleth for membership of the Chruch of RESTafarianism ... do you prefer PATCH or PUT?13:22
ildikov_jd__: LOL :)13:22
*** eglynn is now known as eglynn-lunch13:24
jd__haha13:25
jd__don't care, roll it man13:25
ekarlsook, trying to run a ceilometer service by itself in devstack is just totally broken13:33
ekarlsonow it attempts to start ceilometer api :p13:33
*** yfujioka has joined #openstack-ceilometer13:35
*** jdob has joined #openstack-ceilometer13:39
*** sayali has quit IRC13:41
*** thomasem has joined #openstack-ceilometer13:43
*** sayali has joined #openstack-ceilometer13:45
*** yfujioka has quit IRC13:52
*** yfujioka has joined #openstack-ceilometer13:53
*** prad has joined #openstack-ceilometer13:54
*** ildikov_ has quit IRC13:57
*** ildikov_ has joined #openstack-ceilometer13:59
*** nadya__ has joined #openstack-ceilometer14:00
*** boris-42 has quit IRC14:06
*** yfujioka has quit IRC14:09
*** boris-42 has joined #openstack-ceilometer14:10
*** yassine has quit IRC14:15
*** yassine has joined #openstack-ceilometer14:15
*** jaypipes has quit IRC14:27
*** jaypipes has joined #openstack-ceilometer14:28
*** xuhanp has joined #openstack-ceilometer14:34
*** tongli has joined #openstack-ceilometer14:39
*** prad has quit IRC14:52
*** prad has joined #openstack-ceilometer14:53
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Revert "Ensure we are not exhausting the sqlalchemy pool"  https://review.openstack.org/6839914:56
*** dhellmann is now known as dhellmann_15:22
*** SergeyLukjanov_ is now known as SergeyLukjanov15:33
*** xuhanp has quit IRC15:42
*** nadya__ has quit IRC15:47
pradjd__: can i get your input on https://review.openstack.org/#/c/66746/15:54
jd__prad: reviewed15:57
silehtjd__, really removed eventlet  everywhere ? ;)15:58
jd__sileht: not at all, but I think/hope the use case fixed by that patch is now gone, though I might be wrong, let me know15:58
pradjd__: thanks!15:59
silehtjd__, many other code can be removed if monkey patch is disabled, but what about other application that monkey patch and load the publisher, some part of code should continue to be eventlet compliant ?16:01
* sileht think about this crap: http://git.openstack.org/cgit/openstack/ceilometer/tree/ceilometer/publisher/rpc.py#n16516:01
*** SergeyLukjanov is now known as SergeyLukjanov_a16:07
*** SergeyLukjanov_a is now known as SergeyLukjanov_16:08
jd__sileht: it shouldn't be eventlet compliant, _but_ it should be made thread safe – that will result in the same safe behaviour at the end16:09
jd__but it's different in term of achievement16:09
silehtjd__, sure16:11
silehtjd__, I think for collector, notification consumer and alarming, we should implements the 'worker' parameter of os_service.launch() to keep the same performance without evenlet16:12
jd__sileht: on my roadmap :)16:13
silehtjd__, cool16:13
*** _ruhe is now known as ruhe16:20
*** nadya__ has joined #openstack-ceilometer16:26
*** SergeyLukjanov_ is now known as SergeyLukjanov16:28
*** herndon has joined #openstack-ceilometer16:31
*** nadya__ has quit IRC16:32
*** nadya__ has joined #openstack-ceilometer16:35
*** eglynn-lunch has quit IRC16:35
*** eglynn has joined #openstack-ceilometer16:41
*** herndon has quit IRC17:05
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Fix happybase version  https://review.openstack.org/6843617:05
*** SergeyLukjanov is now known as SergeyLukjanov_17:10
*** SergeyLukjanov_ is now known as SergeyLukjanov17:18
*** ildikov_ has quit IRC17:24
*** s2r2_ has quit IRC17:26
*** SergeyLukjanov is now known as SergeyLukjanov_17:28
*** jmckind has joined #openstack-ceilometer17:30
*** eglynn has quit IRC17:44
*** julienvey has left #openstack-ceilometer17:46
*** s2r2_ has joined #openstack-ceilometer18:10
*** SergeyLukjanov_ is now known as SergeyLukjanov18:21
*** sayali has quit IRC18:25
*** nadya__ has quit IRC18:26
*** Alexei_987 has quit IRC18:30
*** nadya__ has joined #openstack-ceilometer18:34
*** nadya__ has quit IRC18:35
*** s2r2_ has quit IRC18:37
*** julienvey_ has joined #openstack-ceilometer18:38
*** ruhe is now known as _ruhe18:39
*** _ruhe is now known as ruhe19:01
*** julienvey_ has quit IRC19:06
*** s2r2_ has joined #openstack-ceilometer19:12
tongliquestion to anyone working on alarms.19:14
tongliwhen you create an alarm and specify the query conditions, where was the query persisted?19:14
*** nadya__ has joined #openstack-ceilometer19:16
*** yassine has quit IRC19:19
*** nadya__ has quit IRC19:30
*** julienvey_ has joined #openstack-ceilometer19:46
*** julienvey_ has quit IRC19:48
*** s2r2__ has joined #openstack-ceilometer20:23
*** s2r2_ has quit IRC20:23
*** SergeyLukjanov is now known as SergeyLukjanov_20:28
*** ildikov has joined #openstack-ceilometer20:32
*** ildikov_ has joined #openstack-ceilometer20:34
*** julienvey_ has joined #openstack-ceilometer20:48
*** SergeyLukjanov_ is now known as SergeyLukjanov20:51
*** julienvey_ has quit IRC20:54
*** julienvey__ has joined #openstack-ceilometer20:54
*** SergeyLukjanov is now known as SergeyLukjanov_21:02
*** ruhe is now known as _ruhe21:23
*** ildikov has quit IRC21:28
*** s2r2__ has quit IRC21:30
*** gordc has joined #openstack-ceilometer21:45
*** jdob has quit IRC22:27
*** julienvey__ has quit IRC22:28
*** nadya__ has joined #openstack-ceilometer22:31
*** nadya__ has quit IRC22:35
*** tongli has quit IRC22:42
*** thomasem has quit IRC22:48
*** gordc1 has joined #openstack-ceilometer23:04
*** gordc1 has quit IRC23:05
*** gordc has quit IRC23:05
*** jmckind has quit IRC23:36

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