| *** egallen has joined #openstack-metering | 00:54 | |
| *** SergeyLukjanov has joined #openstack-metering | 00:57 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 00:57 | |
| *** _SergeyLukjanov has quit IRC | 00:58 | |
| *** SergeyLukjanov has joined #openstack-metering | 01:17 | |
| *** SergeyLukjanov has quit IRC | 01:22 | |
| *** bingbu has joined #openstack-metering | 01:38 | |
| *** egallen has quit IRC | 01:46 | |
| *** egallen has joined #openstack-metering | 01:46 | |
| *** egallen has quit IRC | 01:47 | |
| *** egallen has joined #openstack-metering | 01:47 | |
| *** egallen has quit IRC | 01:48 | |
| *** egallen has joined #openstack-metering | 01:59 | |
| *** egallen has quit IRC | 02:03 | |
| *** suo has joined #openstack-metering | 02:09 | |
| *** ArcTanSusan has joined #openstack-metering | 02:39 | |
| *** ArcTanSusan has quit IRC | 02:53 | |
| *** ArcTanSusan has joined #openstack-metering | 03:26 | |
| *** egallen has joined #openstack-metering | 03:27 | |
| *** egallen has quit IRC | 03:33 | |
| *** sdake has quit IRC | 04:04 | |
| *** sdake has joined #openstack-metering | 04:05 | |
| *** evanjfraser has quit IRC | 04:24 | |
| *** ArcTanSusan has quit IRC | 05:10 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/54482 | 06:03 |
|---|---|---|
| *** sdake has quit IRC | 06:29 | |
| *** sdake has joined #openstack-metering | 06:32 | |
| *** sdake has joined #openstack-metering | 06:32 | |
| *** fnaval_ has joined #openstack-metering | 07:04 | |
| *** fnaval__ has joined #openstack-metering | 07:05 | |
| *** fnaval_ has quit IRC | 07:09 | |
| *** sdake has quit IRC | 07:26 | |
| *** lsmola has joined #openstack-metering | 07:33 | |
| *** tasdomas` is now known as tasdomas | 07:45 | |
| *** egallen has joined #openstack-metering | 08:38 | |
| *** egallen has quit IRC | 08:40 | |
| openstackgerrit | Victor Stinner proposed a change to openstack/ceilometer: Replace mox with mocks in tests.compute.pollsters https://review.openstack.org/54845 | 08:53 |
| *** egallen has joined #openstack-metering | 09:23 | |
| *** egallen has quit IRC | 09:23 | |
| *** egallen has joined #openstack-metering | 09:23 | |
| *** suo has quit IRC | 09:49 | |
| *** egallen has quit IRC | 10:01 | |
| *** bingbu has quit IRC | 10:15 | |
| *** Bada has joined #openstack-metering | 10:33 | |
| *** asalkeld has joined #openstack-metering | 13:11 | |
| *** asalkeld has left #openstack-metering | 13:12 | |
| *** thomasem has joined #openstack-metering | 13:32 | |
| *** lsmola has quit IRC | 13:38 | |
| thomasem | Hey all! | 14:03 |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Fix nova_notifiers tests https://review.openstack.org/55143 | 14:08 |
| jd__ | hey | 14:17 |
| *** beav has joined #openstack-metering | 14:23 | |
| *** boris-42 has joined #openstack-metering | 14:27 | |
| *** boris-42 has quit IRC | 14:35 | |
| *** lsmola has joined #openstack-metering | 14:41 | |
| *** Bada has quit IRC | 14:42 | |
| *** Bada has joined #openstack-metering | 14:45 | |
| *** Bada has quit IRC | 14:51 | |
| *** lsmola has quit IRC | 14:59 | |
| *** fnaval__ has quit IRC | 15:21 | |
| *** fnaval_ has joined #openstack-metering | 15:22 | |
| *** fnaval_ has quit IRC | 15:26 | |
| *** lsmola has joined #openstack-metering | 15:32 | |
| *** fnaval_ has joined #openstack-metering | 15:40 | |
| *** sayali_ has joined #openstack-metering | 15:48 | |
| *** sdake has joined #openstack-metering | 15:59 | |
| *** EmilienM has quit IRC | 16:00 | |
| *** lsmola has quit IRC | 16:00 | |
| *** EmilienM has joined #openstack-metering | 16:00 | |
| *** giroro_ has quit IRC | 16:01 | |
| *** Ruetobas has joined #openstack-metering | 16:03 | |
| *** Ruetobas has quit IRC | 16:07 | |
| openstackgerrit | Thomas Maddox proposed a change to openstack/ceilometer: Add Event API https://review.openstack.org/40248 | 16:10 |
| *** Ruetobas has joined #openstack-metering | 16:14 | |
| *** Bada has joined #openstack-metering | 16:26 | |
| *** Bada has quit IRC | 16:34 | |
| thomasem | jd__, Is your fix for conductor_api in the nova notifier tests because of this? http://logs.openstack.org/48/40248/12/check/gate-ceilometer-python27/ea26457/console.html | 16:45 |
| thomasem | =] | 16:45 |
| *** sphoorti has joined #openstack-metering | 17:00 | |
| sphoorti | thomasem, hello :) | 17:01 |
| thomasem | sphoorti, Hey there. How are you doing? :) | 17:01 |
| sphoorti | hello :) .. i am still struggling with the bug.. | 17:02 |
| sphoorti | so far i have been reading the collector logs | 17:03 |
| *** cflmarques_ has joined #openstack-metering | 17:04 | |
| sphoorti | thomasem, i have also found that the neutron code does not take the tenant id into consideration | 17:04 |
| sphoorti | i am also looking into the neutron code, trying to figure out what could be done. thomasem | 17:05 |
| thomasem | sphoorti, Did we get any conversation back from the Neutron folks? | 17:05 |
| sphoorti | no thomasem :( | 17:05 |
| sphoorti | i was searching for the q-api in my collector logs ... but i am not finding those | 17:06 |
| sphoorti | and with eglynn's help i relaize that the router.update.start notification generates the error | 17:07 |
| thomasem | Ahhh, yeah - so I think L3 support (router notifications) was just recently added to Ceilometer. | 17:07 |
| thomasem | Well, just recently added to Neutron, actually, we're just getting them now. | 17:08 |
| thomasem | Let's see who made that change. | 17:08 |
| sphoorti | https://github.com/openstack/neutron/blob/master/neutron/api/v2/base.py#L453 | 17:08 |
| sphoorti | thomasem, | 17:08 |
| sphoorti | here the router.update.start notification is triggered and then the requestbody is populated | 17:08 |
| sphoorti | which gives a call to populate tenant id | 17:08 |
| sphoorti | so i feel the tenant id should be populated first and then the event should be triggered. | 17:09 |
| sphoorti | thomasem, | 17:09 |
| thomasem | Oh, the tenant is set in prepare_request_body? | 17:10 |
| sphoorti | yes thomasem | 17:10 |
| thomasem | kk | 17:10 |
| thomasem | Let's see who added this code | 17:10 |
| thomasem | And see if we can get their eyes or rationale | 17:10 |
| thomasem | :) | 17:10 |
| sphoorti | https://github.com/openstack/neutron/blob/master/neutron/api/v2/base.py#L500 | 17:10 |
| thomasem | I'm thinking we don't want to bill someone for a failed router set up or anything. | 17:10 |
| thomasem | So, that could be it, but it's better for us to detect that it's failed and decrement their meter. | 17:11 |
| thomasem | i.e. we don't see an end event come through. | 17:11 |
| thomasem | or don't increment the meter until we see the .end come through. | 17:11 |
| thomasem | So, maybe we could write a separate plugin that doesn't expect tenant_id to be set until the end comes through. | 17:11 |
| thomasem | Seems another viable option. | 17:12 |
| sphoorti | okay. i am slowly slowly getting understanding :) | 17:12 |
| sphoorti | thomasem, | 17:12 |
| thomasem | So, all of these events are tied to billable actions. | 17:12 |
| thomasem | Potentially billable, I mean. Not every cloud owner wold charge its customers for every one of these things. It's just what is available for them to bill on from Ceilometer. | 17:13 |
| sphoorti | thomasem, so the current keyError is for only the router.update.start event right? | 17:13 |
| thomasem | So, in the case of billing for building virtual routers and what-not, it'd be important to know whether it's an active router or not, meaning we'd need to know that it was successfully created, which would be indicated by a '.end. event coming through. | 17:13 |
| thomasem | sphoorti, Correct. So, we could write another plugin to handle that particular event. | 17:14 |
| sphoorti | thomasem, | 17:14 |
| thomasem | But, idk exactly the use case for these events | 17:14 |
| sphoorti | yesterday i tried this | 17:14 |
| sphoorti | neutron router-update router_UUID --routes action=clear | 17:14 |
| sphoorti | which eglynn had suggested | 17:15 |
| thomasem | I understand. Okay, so that is potentially billable and needs a tenant. | 17:15 |
| sphoorti | this action generated the router.update.end event | 17:15 |
| sphoorti | then eglynn had advised me to search for q-api in my collector logs | 17:15 |
| sphoorti | but i could not find it there | 17:15 |
| sphoorti | something wrong with my devstack thomasem ? | 17:15 |
| thomasem | sphoorti, not certain. | 17:16 |
| sphoorti | thomasem, i am thinking of experimenting now.. making a slight change in my localrc enabling q-api | 17:17 |
| thomasem | Sure, go for it | 17:17 |
| sphoorti | lets see if that is of any help | 17:17 |
| sphoorti | just a question, after that addition in line, i should unstack and stack right? | 17:18 |
| sphoorti | thomasem, ? | 17:18 |
| thomasem | Yeah, if you want the startup of q-api to be from DevStack (and therefore auto configured). | 17:20 |
| sphoorti | thomasem, i am looking back at my localrc and i find the lines i had added for neutron installation missing | 17:20 |
| sphoorti | :( | 17:20 |
| sphoorti | sorry thomasem . I found those :) | 17:21 |
| sphoorti | my bad | 17:21 |
| sphoorti | thomasem, now my ./stack.sh is running. will ping you when it is done :) | 17:22 |
| thomasem | Okay. I'm about to head to lunch. | 17:23 |
| sphoorti | okay. would you be around after that? | 17:23 |
| sphoorti | thomasem, ? | 17:23 |
| thomasem | sphoorti, Most likely. I am working on hooking up the Event API for to the Ceilometer client, so I will be available on and off. | 17:24 |
| sphoorti | okay thomasem :) | 17:24 |
| sphoorti | thomasem, q-api still does not appear in collector logs | 17:43 |
| *** boris-42 has joined #openstack-metering | 17:52 | |
| *** sphoorti has quit IRC | 18:03 | |
| *** cflmarques_ has quit IRC | 18:06 | |
| *** sdake has quit IRC | 18:13 | |
| *** sayali_ has quit IRC | 18:34 | |
| *** krtaylor has quit IRC | 19:01 | |
| *** boris-42 has quit IRC | 19:13 | |
| *** yjiang5 has joined #openstack-metering | 19:15 | |
| *** evanjfraser has joined #openstack-metering | 19:17 | |
| *** krtaylor has joined #openstack-metering | 19:40 | |
| *** changbl has joined #openstack-metering | 19:59 | |
| openstackgerrit | Thomas Maddox proposed a change to openstack/ceilometer: Add Event API https://review.openstack.org/40248 | 20:01 |
| *** clouded_tux has quit IRC | 21:22 | |
| *** thomasem has quit IRC | 21:45 | |
| *** thomasem has joined #openstack-metering | 22:33 | |
| thomasem | I'm curious - does anyone remember any rationale for filter for a single event by ID in the DB as opposed to the message ID? It seems like the easier one to go with would be message ID to keep the terminology simple and there's already an inherent 1:1 mapping of event ID in the DB to message_id in the DB, thus sort of making Event ID only useful for indexing. | 22:34 |
| *** |_nikhil_ is now known as nikhil_ | 22:44 | |
| *** nikhil_ is now known as Guest26347 | 22:45 | |
| *** krtaylor has quit IRC | 22:57 | |
| *** Guest26347 is now known as nikhil___ | 23:04 | |
| *** thomasem has quit IRC | 23:05 | |
| *** sdake has joined #openstack-metering | 23:25 | |
| *** eglynn has joined #openstack-metering | 23:31 | |
| *** eglynn has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!