*** flwang has quit IRC | 00:02 | |
*** ok_delta__ has quit IRC | 00:08 | |
*** ok_delta has quit IRC | 00:08 | |
*** gordc has quit IRC | 00:09 | |
*** flwang has joined #openstack-ceilometer | 00:20 | |
*** yassine has quit IRC | 00:21 | |
*** zul has joined #openstack-ceilometer | 00:37 | |
*** Alexei_987 has quit IRC | 01:16 | |
openstackgerrit | Lee Li proposed a change to openstack/python-ceilometerclient: Using common method 'bool_from_string' from oslo strutils https://review.openstack.org/68310 | 03:30 |
---|---|---|
openstackgerrit | Marc Solanas proposed a change to openstack/python-ceilometerclient: Checking the type of auth_token, and fixing it if necessary https://review.openstack.org/58051 | 05:00 |
*** AMike has joined #openstack-ceilometer | 05:06 | |
*** AMike has joined #openstack-ceilometer | 05:06 | |
*** flwang has quit IRC | 05:36 | |
*** flwang has joined #openstack-ceilometer | 05:59 | |
*** ildikov_ has quit IRC | 06:02 | |
*** AMike has quit IRC | 06:43 | |
*** ildikov_ has joined #openstack-ceilometer | 07:01 | |
*** nadya__ has joined #openstack-ceilometer | 07:05 | |
*** sayali has joined #openstack-ceilometer | 07:27 | |
openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/62808 | 07:56 |
*** s2r2_ has joined #openstack-ceilometer | 08:10 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 08:23 | |
openstackgerrit | A change was merged to openstack/ceilometer: Implement the /v2/samples/<sample-id> API https://review.openstack.org/61814 | 08:30 |
*** Alexei_987 has joined #openstack-ceilometer | 08:44 | |
Alexei_987 | jd__: 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,z | 08:46 |
Alexei_987 | sileht: ^ | 08:46 |
openstackgerrit | Nadya Privalova proposed a change to openstack/ceilometer: Fix recursive_keypairs output https://review.openstack.org/67704 | 08:49 |
*** yassine has joined #openstack-ceilometer | 09:06 | |
*** eglynn has joined #openstack-ceilometer | 09:08 | |
*** sayali has quit IRC | 09:11 | |
*** sayali has joined #openstack-ceilometer | 09:13 | |
*** sayali has quit IRC | 09:19 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 09:27 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov_ | 09:28 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 09:40 | |
*** flwang has quit IRC | 09:47 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 09:47 | |
*** boris-42 has quit IRC | 10:02 | |
*** boris-42 has joined #openstack-ceilometer | 10:04 | |
*** sayali has joined #openstack-ceilometer | 10:10 | |
*** sayali has quit IRC | 10:20 | |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: eventlet: stop monkey patching https://review.openstack.org/68145 | 10:24 |
*** flwang has joined #openstack-ceilometer | 10:31 | |
eglynn | llu: 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-ceilometer | 10:58 | |
*** yassine has quit IRC | 10:59 | |
openstackgerrit | A change was merged to openstack/ceilometer: Update oslo https://review.openstack.org/65661 | 11:10 |
*** sayali has joined #openstack-ceilometer | 11:17 | |
*** sayali_ has joined #openstack-ceilometer | 11:17 | |
openstackgerrit | A change was merged to openstack/ceilometer: Fix to handle null threshold_rule values https://review.openstack.org/66757 | 11:23 |
openstackgerrit | A change was merged to openstack/ceilometer: Use DEFAULT section for dispatcher in doc https://review.openstack.org/66908 | 11:24 |
openstackgerrit | A change was merged to openstack/ceilometer: Return trait type from Event api https://review.openstack.org/66936 | 11:25 |
openstackgerrit | A change was merged to openstack/ceilometer: Convert alarm timestamp to PrecisionTimestamp https://review.openstack.org/63131 | 11:25 |
jd__ | some merge finally, nice | 11:31 |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Use swift master https://review.openstack.org/68150 | 11:33 |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Allow pytidylib to be installed from external source https://review.openstack.org/68149 | 11:33 |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: tests: kill all started processes on exit https://review.openstack.org/65509 | 11:33 |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: storage: bases of a Cassandra driver https://review.openstack.org/62779 | 11:33 |
*** nadya__ has quit IRC | 11:33 | |
*** sayali_ has quit IRC | 11:42 | |
*** asalkeld has left #openstack-ceilometer | 11:47 | |
*** boris-42 has quit IRC | 11:53 | |
*** RemyS has joined #openstack-ceilometer | 11:54 | |
*** yassine_ has quit IRC | 11:56 | |
*** boris-42 has joined #openstack-ceilometer | 11:56 | |
*** yassine has joined #openstack-ceilometer | 11:57 | |
*** RemyS has quit IRC | 11:59 | |
*** AMike has joined #openstack-ceilometer | 12:12 | |
*** AMike has joined #openstack-ceilometer | 12:12 | |
*** s2r2_ has quit IRC | 12:24 | |
*** s2r2_ has joined #openstack-ceilometer | 12:26 | |
eglynn | from 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 MST | 12:29 |
eglynn | ... so we're at T minus ~90mins for i-2 | 12:30 |
eglynn | any 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 |
eglynn | folks, is there anything else currently targeted at i-2 with outstanding reviews? | 12:35 |
eglynn | jd__: ... 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 |
sileht | eglynn, I think GET /sample-statistics/?q= is not implemented | 13:03 |
sileht | eglynn, I take a look to your review | 13:04 |
eglynn | sileht: a-ha, gotcha ... thank you sir! | 13:04 |
eglynn | I guess we might as well hustle to get as much approved as poss before i-2 is tagged | 13:05 |
jd__ | I'm going to mark this as implemented, we can create a new bp is someone wants to work on /sample-statistics | 13:05 |
eglynn | ... in the faint hope that the gate swampage magically resolves | 13:06 |
eglynn | jd__: fair enough | 13:06 |
ekarlso | is 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 :p | 13:08 |
ekarlso | or doesn't the current devstack integration allow this | 13:08 |
ildikov_ | jd__, eglynn: the POST /sample is ready for sample-api? | 13:09 |
jd__ | ildikov_: I think so | 13: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 it | 13:13 |
jd__ | ildikov_: I also might be wrong | 13:16 |
jd__ | eglynn: convenience wraper | 13:17 |
jd__ | wrapper | 13:17 |
ekarlso | http://paste.fedoraproject.org/70632/13903965 < that's my local.conf and this is the failure: http://fpaste.org/70633/13903966/ | 13:17 |
eglynn | jd__: cool, gotcha! | 13:17 |
eglynn | ekarlso: I'd imagine the devstack ceilo setup assume an allinone-stylee deployment | 13:18 |
jd__ | eglynn: I'd even say correct REST interface, POSTing in /meters a sample is not really RESTy… | 13:18 |
eglynn | jd__: yep, the REST purists would shudder at that ;) | 13:18 |
ekarlso | eglynn: that's *sadface* | 13:19 |
eglynn | ... or are they known as RESTafarians? ;) | 13:19 |
eglynn | ekarlso: easy to comment out the cleanup_ceilometer function from devstack/lib/ceilometer to avoid the DB drop | 13:20 |
ekarlso | will do :) | 13:20 |
jd__ | eglynn: is there a corollary that makes me a REST purist thn? :) | 13:20 |
jd__ | *then | 13: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 it | 13:21 |
jd__ | RESTafarians, I like that :) | 13:21 |
jd__ | we should arREST you for that | 13:21 |
eglynn | jd__: 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-lunch | 13:24 | |
jd__ | haha | 13:25 |
jd__ | don't care, roll it man | 13:25 |
ekarlso | ok, trying to run a ceilometer service by itself in devstack is just totally broken | 13:33 |
ekarlso | now it attempts to start ceilometer api :p | 13:33 |
*** yfujioka has joined #openstack-ceilometer | 13:35 | |
*** jdob has joined #openstack-ceilometer | 13:39 | |
*** sayali has quit IRC | 13:41 | |
*** thomasem has joined #openstack-ceilometer | 13:43 | |
*** sayali has joined #openstack-ceilometer | 13:45 | |
*** yfujioka has quit IRC | 13:52 | |
*** yfujioka has joined #openstack-ceilometer | 13:53 | |
*** prad has joined #openstack-ceilometer | 13:54 | |
*** ildikov_ has quit IRC | 13:57 | |
*** ildikov_ has joined #openstack-ceilometer | 13:59 | |
*** nadya__ has joined #openstack-ceilometer | 14:00 | |
*** boris-42 has quit IRC | 14:06 | |
*** yfujioka has quit IRC | 14:09 | |
*** boris-42 has joined #openstack-ceilometer | 14:10 | |
*** yassine has quit IRC | 14:15 | |
*** yassine has joined #openstack-ceilometer | 14:15 | |
*** jaypipes has quit IRC | 14:27 | |
*** jaypipes has joined #openstack-ceilometer | 14:28 | |
*** xuhanp has joined #openstack-ceilometer | 14:34 | |
*** tongli has joined #openstack-ceilometer | 14:39 | |
*** prad has quit IRC | 14:52 | |
*** prad has joined #openstack-ceilometer | 14:53 | |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Revert "Ensure we are not exhausting the sqlalchemy pool" https://review.openstack.org/68399 | 14:56 |
*** dhellmann is now known as dhellmann_ | 15:22 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 15:33 | |
*** xuhanp has quit IRC | 15:42 | |
*** nadya__ has quit IRC | 15:47 | |
prad | jd__: can i get your input on https://review.openstack.org/#/c/66746/ | 15:54 |
jd__ | prad: reviewed | 15:57 |
sileht | jd__, 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 know | 15:58 |
prad | jd__: thanks! | 15:59 |
sileht | jd__, 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#n165 | 16:01 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 16: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 end | 16:09 |
jd__ | but it's different in term of achievement | 16:09 |
sileht | jd__, sure | 16:11 |
sileht | jd__, I think for collector, notification consumer and alarming, we should implements the 'worker' parameter of os_service.launch() to keep the same performance without evenlet | 16:12 |
jd__ | sileht: on my roadmap :) | 16:13 |
sileht | jd__, cool | 16:13 |
*** _ruhe is now known as ruhe | 16:20 | |
*** nadya__ has joined #openstack-ceilometer | 16:26 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 16:28 | |
*** herndon has joined #openstack-ceilometer | 16:31 | |
*** nadya__ has quit IRC | 16:32 | |
*** nadya__ has joined #openstack-ceilometer | 16:35 | |
*** eglynn-lunch has quit IRC | 16:35 | |
*** eglynn has joined #openstack-ceilometer | 16:41 | |
*** herndon has quit IRC | 17:05 | |
openstackgerrit | Nadya Privalova proposed a change to openstack/ceilometer: Fix happybase version https://review.openstack.org/68436 | 17:05 |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 17:10 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 17:18 | |
*** ildikov_ has quit IRC | 17:24 | |
*** s2r2_ has quit IRC | 17:26 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 17:28 | |
*** jmckind has joined #openstack-ceilometer | 17:30 | |
*** eglynn has quit IRC | 17:44 | |
*** julienvey has left #openstack-ceilometer | 17:46 | |
*** s2r2_ has joined #openstack-ceilometer | 18:10 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 18:21 | |
*** sayali has quit IRC | 18:25 | |
*** nadya__ has quit IRC | 18:26 | |
*** Alexei_987 has quit IRC | 18:30 | |
*** nadya__ has joined #openstack-ceilometer | 18:34 | |
*** nadya__ has quit IRC | 18:35 | |
*** s2r2_ has quit IRC | 18:37 | |
*** julienvey_ has joined #openstack-ceilometer | 18:38 | |
*** ruhe is now known as _ruhe | 18:39 | |
*** _ruhe is now known as ruhe | 19:01 | |
*** julienvey_ has quit IRC | 19:06 | |
*** s2r2_ has joined #openstack-ceilometer | 19:12 | |
tongli | question to anyone working on alarms. | 19:14 |
tongli | when you create an alarm and specify the query conditions, where was the query persisted? | 19:14 |
*** nadya__ has joined #openstack-ceilometer | 19:16 | |
*** yassine has quit IRC | 19:19 | |
*** nadya__ has quit IRC | 19:30 | |
*** julienvey_ has joined #openstack-ceilometer | 19:46 | |
*** julienvey_ has quit IRC | 19:48 | |
*** s2r2__ has joined #openstack-ceilometer | 20:23 | |
*** s2r2_ has quit IRC | 20:23 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 20:28 | |
*** ildikov has joined #openstack-ceilometer | 20:32 | |
*** ildikov_ has joined #openstack-ceilometer | 20:34 | |
*** julienvey_ has joined #openstack-ceilometer | 20:48 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 20:51 | |
*** julienvey_ has quit IRC | 20:54 | |
*** julienvey__ has joined #openstack-ceilometer | 20:54 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 21:02 | |
*** ruhe is now known as _ruhe | 21:23 | |
*** ildikov has quit IRC | 21:28 | |
*** s2r2__ has quit IRC | 21:30 | |
*** gordc has joined #openstack-ceilometer | 21:45 | |
*** jdob has quit IRC | 22:27 | |
*** julienvey__ has quit IRC | 22:28 | |
*** nadya__ has joined #openstack-ceilometer | 22:31 | |
*** nadya__ has quit IRC | 22:35 | |
*** tongli has quit IRC | 22:42 | |
*** thomasem has quit IRC | 22:48 | |
*** gordc1 has joined #openstack-ceilometer | 23:04 | |
*** gordc1 has quit IRC | 23:05 | |
*** gordc has quit IRC | 23:05 | |
*** jmckind has quit IRC | 23:36 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!