| *** david-ly_ has joined #openstack-ceilometer | 00:06 | |
| *** jwcroppe has quit IRC | 00:07 | |
| *** jwcroppe has joined #openstack-ceilometer | 00:07 | |
| *** david-lyle has quit IRC | 00:08 | |
| *** ankita_w_ has quit IRC | 00:09 | |
| *** jwcroppe has quit IRC | 00:10 | |
| *** Longgeek has joined #openstack-ceilometer | 00:12 | |
| *** jwcroppe has joined #openstack-ceilometer | 00:13 | |
| *** edmondsw has joined #openstack-ceilometer | 00:16 | |
| *** edmondsw_ has joined #openstack-ceilometer | 00:16 | |
| *** edmondsw_ has quit IRC | 00:17 | |
| *** pauldix has joined #openstack-ceilometer | 00:20 | |
| *** Longgeek has quit IRC | 00:23 | |
| *** pauldix has quit IRC | 00:29 | |
| *** rwsu has quit IRC | 00:38 | |
| *** thorst has joined #openstack-ceilometer | 00:39 | |
| *** boris-42 has quit IRC | 00:42 | |
| *** ddieterly has joined #openstack-ceilometer | 00:43 | |
| *** jfluhmann has quit IRC | 00:47 | |
| *** jwcroppe_ has joined #openstack-ceilometer | 00:47 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Add support for posting samples to notification-agent via API https://review.openstack.org/193472 | 00:47 |
|---|---|---|
| *** edmondsw has quit IRC | 00:47 | |
| *** r-daneel has quit IRC | 00:48 | |
| *** jwcroppe has quit IRC | 00:49 | |
| *** dscottsf has joined #openstack-ceilometer | 00:50 | |
| *** dscottsf has quit IRC | 00:53 | |
| *** thorst has quit IRC | 01:00 | |
| *** ankita_wagh has joined #openstack-ceilometer | 01:04 | |
| *** jfluhmann has joined #openstack-ceilometer | 01:12 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Add support for posting samples to notification-agent via API https://review.openstack.org/193472 | 01:13 |
| *** david-ly_ is now known as david-lyle | 01:36 | |
| *** _cjones_ has quit IRC | 01:41 | |
| *** dontalton has quit IRC | 01:56 | |
| openstackgerrit | Ryota MIBU proposed openstack/ceilometer: Add 'event' type and 'event_rule' to alarm API https://review.openstack.org/195376 | 01:59 |
| *** ddieterly has quit IRC | 02:07 | |
| *** ddieterly has joined #openstack-ceilometer | 02:09 | |
| *** jruano has joined #openstack-ceilometer | 02:11 | |
| *** ddieterly has quit IRC | 02:14 | |
| *** ankita_wagh has quit IRC | 02:35 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: remove unused notifier https://review.openstack.org/195291 | 02:43 |
| *** ddieterly has joined #openstack-ceilometer | 02:52 | |
| *** jruano has quit IRC | 03:15 | |
| *** Longgeek has joined #openstack-ceilometer | 03:20 | |
| *** Longgeek has quit IRC | 03:21 | |
| *** Longgeek has joined #openstack-ceilometer | 03:21 | |
| *** jruano has joined #openstack-ceilometer | 03:34 | |
| openstackgerrit | Merged openstack/ceilometer: Add support for posting samples to notification-agent via API https://review.openstack.org/193472 | 03:37 |
| *** rjaiswal has quit IRC | 03:44 | |
| *** ankita_wagh has joined #openstack-ceilometer | 03:50 | |
| *** deepthi has joined #openstack-ceilometer | 03:55 | |
| *** ddieterly has quit IRC | 04:10 | |
| *** jruano has quit IRC | 04:13 | |
| *** jruano has joined #openstack-ceilometer | 04:34 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Add documentation about the usage of api-no-pipline https://review.openstack.org/195409 | 04:35 |
| *** Longgeek has quit IRC | 04:37 | |
| *** Longgeek has joined #openstack-ceilometer | 04:37 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Add documentation about the usage of api-no-pipeline https://review.openstack.org/195409 | 04:38 |
| *** Longgeek has quit IRC | 04:44 | |
| *** jfluhmann has quit IRC | 04:51 | |
| *** yatin has joined #openstack-ceilometer | 05:04 | |
| *** jruano has quit IRC | 05:05 | |
| *** ddieterly has joined #openstack-ceilometer | 05:10 | |
| *** ddieterly has quit IRC | 05:15 | |
| *** yprokule has joined #openstack-ceilometer | 05:17 | |
| *** Longgeek has joined #openstack-ceilometer | 05:17 | |
| *** ildikov has quit IRC | 05:26 | |
| *** ddieterly has joined #openstack-ceilometer | 06:11 | |
| *** ildikov has joined #openstack-ceilometer | 06:13 | |
| *** changbl_ has quit IRC | 06:15 | |
| *** ddieterly has quit IRC | 06:16 | |
| *** jaosorior has joined #openstack-ceilometer | 06:41 | |
| *** ankita_wagh has quit IRC | 06:42 | |
| *** liusheng_phone has joined #openstack-ceilometer | 06:46 | |
| *** belmoreira has joined #openstack-ceilometer | 06:53 | |
| *** exploreshaifali has joined #openstack-ceilometer | 06:53 | |
| *** yprokule has quit IRC | 06:57 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:58 | |
| *** yprokule has joined #openstack-ceilometer | 06:59 | |
| *** _nadya_ has quit IRC | 07:00 | |
| *** _nadya_ has joined #openstack-ceilometer | 07:00 | |
| *** _nadya_ has quit IRC | 07:05 | |
| *** safchain has joined #openstack-ceilometer | 07:09 | |
| *** fabian4 has quit IRC | 07:11 | |
| *** ddieterly has joined #openstack-ceilometer | 07:12 | |
| *** gzhai1 has joined #openstack-ceilometer | 07:14 | |
| *** ddieterly has quit IRC | 07:17 | |
| *** lan has quit IRC | 07:20 | |
| *** shardy has joined #openstack-ceilometer | 07:20 | |
| *** lan has joined #openstack-ceilometer | 07:21 | |
| *** fabian4 has joined #openstack-ceilometer | 07:24 | |
| *** gzhai1 has quit IRC | 07:25 | |
| *** lsmola has joined #openstack-ceilometer | 07:28 | |
| *** cdent has joined #openstack-ceilometer | 07:29 | |
| *** ankita_wagh has joined #openstack-ceilometer | 07:36 | |
| *** fabian4 has quit IRC | 07:36 | |
| *** jfluhmann has joined #openstack-ceilometer | 07:37 | |
| *** j_t has joined #openstack-ceilometer | 07:44 | |
| j_t | Once I'm monitoring a VMs cpu, memory, etc etc. If I find VMs that aren't "doing anything", for awhile.. is there some other openstack system I can use to delete them? or at least say send an email... and it gets deleted in a month. give the user a link to click to cancel the process. in a week it gets shut down, in 3 more weeks it gets deleted | 07:45 |
| j_t | We run a private cloud, and people spin up Vms and they forget about them | 07:45 |
| *** Ala has joined #openstack-ceilometer | 07:45 | |
| j_t | so it would be nice to clean up unused crap in an automated way | 07:45 |
| j_t | does anything like this exist? | 07:45 |
| *** fabian4 has joined #openstack-ceilometer | 07:48 | |
| cdent | j_t: sounds like maybe ceilometer's alarming functionality or the heat project (which uses ceilometer's data) might be able to help you | 07:50 |
| j_t | cdent: hmm, so I'm guessing this is something that has to be built from scratch? | 07:59 |
| *** gzhai1 has joined #openstack-ceilometer | 08:00 | |
| j_t | leveraging the ceilometer/heat stuff | 08:00 |
| cdent | that's a good question. I bet there are pre-assembled solutions out there, but I don't know about them. | 08:00 |
| cdent | much of the openstack stuff is bits of lego to put together in various ways | 08:00 |
| cdent | If you're on the operators mailing list you might ask there: there are often people who have shared the tools they've assembled | 08:01 |
| j_t | yep, but it's mixed south american legos, with the chinese knock offs too, and the expensive american legit ones | 08:01 |
| j_t | so the pieces don't always fit | 08:01 |
| cdent | ha! | 08:01 |
| j_t | so you have to use a hammer, with some glue and duct tape too | 08:01 |
| j_t | I did find this old krufty perl project: https://github.com/paypal/cloudminion from paypal | 08:02 |
| j_t | actually the last time I checked that, there were no recent commits | 08:03 |
| j_t | It didn't look like it worked | 08:03 |
| j_t | One of those "let's be opensourcey and throw our internal thing that is specific to our environment over the wall" situations | 08:04 |
| j_t | and no updates for years | 08:04 |
| j_t | but now when I look at it, there was a recent update a month ago | 08:04 |
| j_t | I wonder if that works. | 08:04 |
| cdent | good luck, if you need to use the hammer, start with light taps before swinging wildly | 08:04 |
| *** deepthi has quit IRC | 08:05 | |
| j_t | yep, especially when it comes to deleting | 08:05 |
| j_t | I'll delete devtest VMs first before running this in production. | 08:05 |
| *** ankita_wagh has quit IRC | 08:09 | |
| *** ddieterly has joined #openstack-ceilometer | 08:13 | |
| *** exploreshaifali has quit IRC | 08:16 | |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: Ensure location header urls account for script_name https://review.openstack.org/195446 | 08:17 |
| *** ddieterly has quit IRC | 08:18 | |
| *** _nadya_ has joined #openstack-ceilometer | 08:33 | |
| *** yassine_ has joined #openstack-ceilometer | 08:36 | |
| *** _nadya_ has quit IRC | 08:38 | |
| *** _nadya_ has joined #openstack-ceilometer | 08:38 | |
| *** _nadya_ has quit IRC | 08:43 | |
| openstackgerrit | Lianhao Lu proposed openstack/ceilometer: (DO NOT MERGE) https://review.openstack.org/193407 | 08:43 |
| *** deepthi has joined #openstack-ceilometer | 08:52 | |
| openstackgerrit | Merged openstack/gnocchi: Ensure live gabbi tests run in gate https://review.openstack.org/195040 | 08:58 |
| openstackgerrit | Merged openstack/gnocchi: Don't reclone the repo we already did https://review.openstack.org/195080 | 08:59 |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: Ensure location header urls account for script_name https://review.openstack.org/195446 | 09:01 |
| *** deepthi has quit IRC | 09:02 | |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: Allow gnocchi API at /gnocchi https://review.openstack.org/195149 | 09:03 |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: Allow gnocchi API at /metric https://review.openstack.org/195149 | 09:03 |
| *** liusheng_phone has quit IRC | 09:09 | |
| *** shardy has quit IRC | 09:14 | |
| *** ddieterly has joined #openstack-ceilometer | 09:14 | |
| *** yprokule has quit IRC | 09:14 | |
| therve | Hey there | 09:15 |
| *** shardy has joined #openstack-ceilometer | 09:15 | |
| therve | We're seeing some test failures in integration tests with ceilometer | 09:15 |
| therve | There is a suspicious commit yesterday, anyone has an idea what it does? | 09:15 |
| *** ddieterly has quit IRC | 09:18 | |
| *** deepthi has joined #openstack-ceilometer | 09:18 | |
| therve | Especially that new direct flag, not sure what it does | 09:20 |
| *** yprokule has joined #openstack-ceilometer | 09:26 | |
| llu | sileht: I dig more into the oslo.messaging pool issue | 09:29 |
| llu | sileht: it seems that the pool works fine | 09:30 |
| llu | sileht: it's other issues on my local environment and my patches, as well as bug#1467776, combined together caused the jenkins failure | 09:31 |
| llu | therve: what's the failure? | 09:31 |
| therve | llu, bug #1468697 | 09:32 |
| openstack | bug 1468697 in heat "Test ceilometer alarm scenario is failing on gates" [Medium,Confirmed] https://launchpad.net/bugs/1468697 - Assigned to Kairat Kushaev (kkushaev) | 09:32 |
| therve | llu, We're posting a sample, but it doesn't trigger an alarm notification as we expect | 09:32 |
| therve | And as it used to do I may say | 09:32 |
| llu | therve: ah | 09:32 |
| llu | you also need to have notification-agent running in order to have the POST sample working | 09:33 |
| therve | It seems to be running as expected | 09:34 |
| llu | hmm, then this would indicate some issues in the patch series https://review.openstack.org/#/q/topic:bp/api-no-pipeline,n,z | 09:37 |
| llu | liusheng: are you there? | 09:38 |
| llu | liusheng: could you look at bug #1468697 | 09:38 |
| openstack | bug 1468697 in heat "Test ceilometer alarm scenario is failing on gates" [Medium,Confirmed] https://launchpad.net/bugs/1468697 - Assigned to Kairat Kushaev (kkushaev) | 09:38 |
| liusheng | llu: yes | 09:38 |
| liusheng | llu: ok, looking | 09:38 |
| openstackgerrit | Thomas Herve proposed openstack/ceilometer: WIP: test revert https://review.openstack.org/195482 | 09:38 |
| *** alexpilotti has joined #openstack-ceilometer | 09:45 | |
| liusheng | llu: I guess the reason is in devstack senario, the configure option "notification_driver = messagingv2" is not configured which is needed for api-no-pipeline | 09:54 |
| liusheng | llu: I have upload documentation about the usage https://review.openstack.org/#/c/195409/2/doc/source/webapi/v2.rst | 09:55 |
| jd__ | llu: sileht is in vacation | 09:57 |
| jd__ | and I need him for the Ceph driver :( | 09:57 |
| liusheng | llu: I am working on ceilometerclient to add flag 'direct', the failed tests can work with 'drect' flag | 09:57 |
| liusheng | llu: or fix it in devstack script | 09:58 |
| *** edmondsw has joined #openstack-ceilometer | 10:06 | |
| *** edmondsw has quit IRC | 10:06 | |
| *** edmondsw has joined #openstack-ceilometer | 10:07 | |
| openstackgerrit | Merged openstack/ceilometer: remove unused notifier https://review.openstack.org/195291 | 10:10 |
| *** Kennan2 has joined #openstack-ceilometer | 10:14 | |
| *** Kennan has quit IRC | 10:14 | |
| *** Kennan2 is now known as Kennan | 10:17 | |
| *** exploreshaifali has joined #openstack-ceilometer | 10:20 | |
| *** gautamd has joined #openstack-ceilometer | 10:25 | |
| *** jfluhmann has quit IRC | 10:34 | |
| *** arun_ has joined #openstack-ceilometer | 10:40 | |
| *** arun_ has left #openstack-ceilometer | 10:40 | |
| *** deepthi has quit IRC | 10:42 | |
| openstackgerrit | Aleksey Ripinen proposed openstack/ceilometer: Samples with last timestamp in sqlalchemy backend https://review.openstack.org/189670 | 10:44 |
| openstackgerrit | Aleksey Ripinen proposed openstack/ceilometer: Cover impl_db2 by tests https://review.openstack.org/182253 | 10:48 |
| openstackgerrit | Aleksey Ripinen proposed openstack/ceilometer: Add complex query for statistic requests https://review.openstack.org/188333 | 10:49 |
| openstackgerrit | Aleksey Ripinen proposed openstack/ceilometer: Query statistics function for sql https://review.openstack.org/189673 | 10:49 |
| openstackgerrit | liusheng proposed openstack/python-ceilometerclient: Add ceilometerclient support for api-no-pipeline https://review.openstack.org/195515 | 10:54 |
| therve | liusheng, llu: So I confirmed that this commit is the culprit. What should we do? Revert? | 11:08 |
| liusheng | therve: I'm trying to fix it in devstack script, patch will be uploaded soon | 11:10 |
| therve | liusheng, Do you think that's enough? | 11:10 |
| therve | It sounds like a backward incompatible change | 11:10 |
| liusheng | therve: no, I think it should be add a configuration in ceilometer conf as default for api-no-pipeline feature | 11:11 |
| therve | Humm... | 11:12 |
| therve | Well it's breaking heat gate, so we'd like to have a quick fix | 11:13 |
| *** ddieterly has joined #openstack-ceilometer | 11:15 | |
| *** david-lyle has quit IRC | 11:17 | |
| therve | If only one value for notification_driver is supported, maybe there shouldn't be an option at all | 11:19 |
| *** gautamd has quit IRC | 11:20 | |
| *** ddieterly has quit IRC | 11:20 | |
| *** david-lyle has joined #openstack-ceilometer | 11:21 | |
| liusheng | therve: :(, actually, it is ok without the configuration but that need heat test add the 'direct' flag and need waiting ceilometerclient's patch merged | 11:25 |
| therve | liusheng, It should be the other way around to keep compatibility | 11:25 |
| liusheng | therve: https://review.openstack.org/#/c/195530/ | 11:26 |
| liusheng | therve: I have uploaded a patch in devstack for this | 11:26 |
| therve | liusheng, Why is the patch in devstack? Should you take care of this in ceilometer code base itself? | 11:27 |
| therve | People not using devstack will have the same issue as heat | 11:27 |
| *** kairat_kushaev has joined #openstack-ceilometer | 11:27 | |
| liusheng | therve: the notification_driver is also add in devstack for other projects (nova, neutron, cinder...) for ceilometer itself, it only can provide documentation to remind deployers to config this option | 11:30 |
| therve | Sorry it doesn't make sense to me | 11:31 |
| *** prad has joined #openstack-ceilometer | 11:32 | |
| therve | liusheng, If I'm running the kilo version of ceilometer, and then upgrade to liberty, I expect the behavior to stay the same without having to change a configuration option | 11:32 |
| liusheng | therve: for testing scenario, the feature suggest posting sample with 'direct=True' flag, if so, the configure option isn't needed. very sorry for this, if we cannot wait for others merged, please revert it | 11:37 |
| *** EmilienM|off is now known as EmilienM | 11:38 | |
| openstackgerrit | liusheng proposed openstack/python-ceilometerclient: Add ceilometerclient support for api-no-pipeline https://review.openstack.org/195515 | 11:57 |
| *** AJaeger has joined #openstack-ceilometer | 12:01 | |
| AJaeger | Hi ceilometer cores, it's time for another translation import IMHO - could somebody +A https://review.openstack.org/#/c/190497/, please? | 12:01 |
| cdent | AJaeger: done | 12:02 |
| AJaeger | thanks, cdent ! | 12:02 |
| cdent | That's the third time I've +2 that review :) | 12:02 |
| AJaeger | cdent: Since it's a automatic import, other projects don't use a second core. A single "+2A" is enough for those. Might be something you want to discuss... | 12:03 |
| cdent | that's a good idea | 12:04 |
| cdent | I'm a relatively new core and gordc gave me strict instructions to not +A anything that didn't already have a +2 | 12:04 |
| cdent | but that gordc, he's just a tyrant, I'm going to ignore him henceforth | 12:04 |
| cdent | :) | 12:04 |
| AJaeger | DON'T! ;) | 12:04 |
| AJaeger | The translation import is an exception - and the only one I'm aware off - for many projects | 12:05 |
| cdent | I was kidding! | 12:05 |
| AJaeger | cdent: I know | 12:05 |
| AJaeger | so, figure it out with your wise leader ;) | 12:06 |
| cdent | I gordc were actively listening I'd feel far better about all the teasing of him I want to do | 12:07 |
| cdent | wise! | 12:07 |
| cdent | ha! | 12:07 |
| AJaeger | cdent: I'll wave good-bye again. Have a great day! | 12:08 |
| * cdent waves | 12:08 | |
| *** AJaeger has left #openstack-ceilometer | 12:08 | |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: Allow gnocchi API at /metric https://review.openstack.org/195149 | 12:12 |
| *** htruta has joined #openstack-ceilometer | 12:12 | |
| gordc | i have no control | 12:15 |
| gordc | i grant thee permission to +2A translations | 12:16 |
| *** ddieterly has joined #openstack-ceilometer | 12:16 | |
| gordc | and also reqiurements | 12:16 |
| gordc | and also anything i post. | 12:16 |
| cdent | gordc++ | 12:17 |
| *** ddieterly has quit IRC | 12:20 | |
| *** exploreshaifali has quit IRC | 12:32 | |
| *** david-ly_ has joined #openstack-ceilometer | 12:35 | |
| *** david-lyle has quit IRC | 12:39 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:41 | |
| *** yatin has quit IRC | 12:48 | |
| *** liusheng_phone has joined #openstack-ceilometer | 12:50 | |
| *** _nadya_ has quit IRC | 12:51 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:52 | |
| *** Ctina has joined #openstack-ceilometer | 12:56 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: catch warnings from error tests https://review.openstack.org/195142 | 12:56 |
| *** julim has joined #openstack-ceilometer | 13:01 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: fix kafka tests from flooding logs https://review.openstack.org/195189 | 13:01 |
| openstackgerrit | gordon chung proposed openstack/ceilometer: test error log - catch dummy error https://review.openstack.org/195094 | 13:01 |
| therve | gordc, Hey, if we can get your feedback on that alarm issue that'd be cool | 13:04 |
| openstackgerrit | Chris Dent proposed openstack/gnocchi: Allow gnocchi API at /metric https://review.openstack.org/195149 | 13:04 |
| gordc | therve: sure. you have a link? i might have missed it | 13:04 |
| therve | gordc, Links you mean :) | 13:05 |
| therve | bug #1468697 is the test issue we're having in Heat | 13:05 |
| openstack | bug 1468697 in heat "Test ceilometer alarm scenario is failing on gates" [Medium,Confirmed] https://launchpad.net/bugs/1468697 - Assigned to Kairat Kushaev (kkushaev) | 13:05 |
| *** belmoreira has quit IRC | 13:05 | |
| therve | It seems I6a1587e4aae055cc559286367632249865008654 would be the culprit | 13:05 |
| therve | So I think it's introducing a regression, and apparently changing some options would fix it | 13:06 |
| therve | gordc, http://eavesdrop.openstack.org/irclogs/%23openstack-ceilometer/latest.log.html#t2015-06-25T11:25:37 for the discussion | 13:07 |
| gordc | kk, reading | 13:07 |
| *** afazekas has joined #openstack-ceilometer | 13:08 | |
| gordc | therve: at quick glance, i'd be surprised if that patch is affecting your alarms. | 13:08 |
| therve | gordc, I checked with https://review.openstack.org/195482 and it seems to do the trick | 13:10 |
| therve | You think that option is set properly? | 13:10 |
| gordc | oh. you're posting samples, i just read up | 13:11 |
| therve | Yeah | 13:11 |
| *** jruano has joined #openstack-ceilometer | 13:11 | |
| *** husanux1 has joined #openstack-ceilometer | 13:13 | |
| *** husanux1 has quit IRC | 13:13 | |
| gordc | therve: we can revert it for now. | 13:14 |
| gordc | so it unblocks you... i think the devstack change might be correct though. | 13:14 |
| therve | gordc, You don't think it's a backward incompatible change though? | 13:15 |
| therve | I'm okay to wait a bit, if the devstack change can land today. | 13:16 |
| therve | I'm just not convinced it's the right solution | 13:16 |
| gordc | therve: it'll probably take longer to land in devstack | 13:16 |
| gordc | want to post a proper revert? or i can | 13:17 |
| therve | Probably | 13:17 |
| *** shardy has quit IRC | 13:17 | |
| therve | I can, let me do this right now | 13:17 |
| gordc | kk i'll +2A | 13:17 |
| gordc | therve: i should probably mention there's a button in gerrit to revert it (assuming it works). | 13:20 |
| openstackgerrit | Thomas Herve proposed openstack/ceilometer: Revert "Add support for posting samples.." https://review.openstack.org/195482 | 13:20 |
| therve | gordc, ^^ :) | 13:21 |
| *** exploreshaifali has joined #openstack-ceilometer | 13:21 | |
| gordc | did you hit revert button actually? just so it's easier to track? | 13:22 |
| *** shardy has joined #openstack-ceilometer | 13:22 | |
| therve | gordc, Ah no I didn't | 13:24 |
| therve | Let's try that | 13:24 |
| gordc | therve: yeah, that'd be better. thanks! | 13:24 |
| openstackgerrit | Thomas Herve proposed openstack/ceilometer: Revert "Add support for posting samples to notification-agent via API" https://review.openstack.org/195570 | 13:25 |
| *** husanux9 has joined #openstack-ceilometer | 13:25 | |
| therve | gordc, There you go | 13:25 |
| gordc | therve: thanks! +A. we'll work on fixing that in the meantime. | 13:28 |
| therve | Thank you! | 13:28 |
| *** husanux9 has quit IRC | 13:29 | |
| *** liusheng has quit IRC | 13:35 | |
| *** liusheng has joined #openstack-ceilometer | 13:35 | |
| *** kun_huang has joined #openstack-ceilometer | 13:38 | |
| gordc | liusheng: added a comment to your devsatck patch. | 13:40 |
| *** changbl_ has joined #openstack-ceilometer | 13:40 | |
| gordc | liusheng: we probably don't need it. i think we just configured notifier wrong. (maybe we need to add a test case in test_notification for api?) | 13:40 |
| liusheng | gordc, thanks, but the notification_driver option is needed to get notifier | 13:42 |
| liusheng | the option has a [] as default | 13:42 |
| *** shardy has quit IRC | 13:42 | |
| gordc | liusheng: we already have our own value: https://github.com/openstack/ceilometer/blob/master/ceilometer/publisher/messaging.py#L55-L60 | 13:43 |
| *** shardy has joined #openstack-ceilometer | 13:44 | |
| *** julim has quit IRC | 13:44 | |
| liusheng | gordc, I got it, yes, I didn't notice this option, I will test it, thanks | 13:45 |
| *** _nadya_ has quit IRC | 13:46 | |
| *** _nadya_ has joined #openstack-ceilometer | 13:47 | |
| gordc | liusheng: cool cool | 13:48 |
| * gordc makes a mental note on how to create notifier. | 13:48 | |
| cdent | gordc you should share all mental notes of that ilk in some non-mental way | 13:51 |
| *** _nadya_ has quit IRC | 13:51 | |
| *** htruta has quit IRC | 13:51 | |
| gordc | cdent: 75% of it is not safe for work... but i guess safe for internet. | 13:54 |
| liusheng | gordc, I have noticed nova, cinder, heat... need this option manully to send notifications(http://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/nova#n546) to notification-agent, so I assume ceilometer also need this, my mistake | 13:54 |
| cdent | all the better gordc | 13:54 |
| gordc | liusheng: i feel like our option is some side effect to pre-oslo.messaging days. we might want to use notificaiton_driver eventually. | 13:55 |
| gordc | that said, i don't know what happens when you have both set. | 13:56 |
| gordc | cdent: don't encourage me. a lot of effort goes into containing the beast. | 13:56 |
| cdent | FREE THE BEAST | 13:56 |
| gordc | no! | 13:57 |
| * cdent aws | 13:57 | |
| liusheng | gordc, yeah, I wonder why notification_driver has a default value (messaging) than a empty list | 13:57 |
| liusheng | s /has/hasn't/ | 13:57 |
| gordc | liusheng: you'll need to ask sileht :) | 13:57 |
| liusheng | gordc, :) | 13:58 |
| gordc | cdent: i haven't saved enough yet to be able to live unemployed. | 13:58 |
| *** exploreshaifali has quit IRC | 13:58 | |
| cdent | okay, I'll let you off this time | 13:58 |
| gordc | 3 years? i'll open a bank account -- shoe box is getting full anyways | 13:59 |
| *** julim has joined #openstack-ceilometer | 14:04 | |
| *** llu has quit IRC | 14:07 | |
| *** llu has joined #openstack-ceilometer | 14:08 | |
| *** r-daneel has joined #openstack-ceilometer | 14:09 | |
| *** rbak has joined #openstack-ceilometer | 14:16 | |
| *** liusheng has quit IRC | 14:20 | |
| *** _nadya_ has joined #openstack-ceilometer | 14:29 | |
| *** shardy has quit IRC | 14:29 | |
| *** shardy has joined #openstack-ceilometer | 14:30 | |
| *** htruta has joined #openstack-ceilometer | 14:33 | |
| *** eglynn has joined #openstack-ceilometer | 14:33 | |
| *** pauldix has joined #openstack-ceilometer | 14:43 | |
| openstackgerrit | Alexandre Viau proposed openstack/python-ceilometerclient: Typo: Show an sample => Show a sample https://review.openstack.org/195612 | 14:45 |
| *** alextricity has joined #openstack-ceilometer | 14:47 | |
| *** pauldix has quit IRC | 14:48 | |
| alextricity | Hey guys. I'm working on getting ceilometer up on Kilo, but am having trouble with adding the following configs for configuring cinder to send notifications | 14:48 |
| alextricity | control_exchange = cinder | 14:48 |
| alextricity | notification_driver = messagingv2 | 14:48 |
| alextricity | My cinder volume creation hangs if I add those directives. | 14:49 |
| alextricity | However, if I remove them, cinder starts working again :/ | 14:49 |
| alextricity | I've checked the logs..and unfortunately I wasn't able to find anything significant. Can anybody help me out? | 14:49 |
| openstackgerrit | Merged openstack/ceilometermiddleware: Add test which asserts non-empty target_id https://review.openstack.org/193062 | 14:54 |
| *** david-ly_ is now known as david-lyle | 15:01 | |
| _nadya_ | alextricity: hi! Maybe anything interesting in the logs when Cinder doesn't work? | 15:02 |
| gordc | heads up. we have a meeting over at openstack-meeting | 15:02 |
| gordc | :) | 15:02 |
| *** shardy has quit IRC | 15:08 | |
| *** shardy has joined #openstack-ceilometer | 15:08 | |
| *** rbak has quit IRC | 15:09 | |
| *** idegtiarov_ has joined #openstack-ceilometer | 15:09 | |
| *** rbak has joined #openstack-ceilometer | 15:13 | |
| *** shardy has quit IRC | 15:13 | |
| *** shardy has joined #openstack-ceilometer | 15:13 | |
| *** pauldix has joined #openstack-ceilometer | 15:15 | |
| *** edmondsw has quit IRC | 15:16 | |
| *** edmondsw has joined #openstack-ceilometer | 15:25 | |
| openstackgerrit | Roman Vasilets proposed openstack/python-ceilometerclient: Change default values from [] to None https://review.openstack.org/195633 | 15:29 |
| openstackgerrit | Roman Vasilets proposed openstack/python-ceilometerclient: Change default values from [] to None https://review.openstack.org/195633 | 15:31 |
| *** Ala has quit IRC | 15:58 | |
| *** idegtiarov_ has quit IRC | 15:59 | |
| *** Longgeek has quit IRC | 15:59 | |
| *** Longgeek has joined #openstack-ceilometer | 16:00 | |
| gordc | _nadya_: why were you using notifier? we can add it back... it just didn't seem to be connected to anything when i was looking at it | 16:00 |
| gordc | oh, you just used it as a dump for tests... leaving it off now :) | 16:02 |
| _nadya_ | gordc: no, I don't think it's needed to get it back. It was the easiest way to reach plugin_base's to_samples_and_publish | 16:03 |
| _nadya_ | gordc: I think I will manage using several mocks | 16:03 |
| ildikov | gordc: _nadya_: sorry, on another meeting and I still cannot talk and write at the same time... :( | 16:04 |
| ildikov | so as I see the issue is that as we don't generate data, but receive it and because of the OpenStack madness we are periodically in trouble to properly consume notifications | 16:05 |
| ildikov | but as we are on the consumer side, we are not able to handle the root cause | 16:06 |
| _nadya_ | gordc: it's just funny that it was in the project for years and removed at the moment I started to use it :) | 16:06 |
| gordc | ildikov: yeah. but then you need to propose a schema. and ppl for some reason have allegiance to own schema | 16:07 |
| ildikov | gordc: Nova is about to move to versioned obejcts for notifications, I think that could be a starting point, at least they have now an intention to change | 16:08 |
| ildikov | gordc: it can be raised on the cross-project level/meeting that it is still an issue let's solve it anyhow | 16:09 |
| cdent | I've similarly heard that versioned objects could help the schematization process | 16:11 |
| * cdent waits to be convinced : | 16:11 | |
| gordc | but version object only guarantees that the notifications work amongst themselves no? if the content was never there, versionobjects doesn't solve that. | 16:11 |
| ildikov | I just wanted to point out that at least they are aiming for a schema and versioning | 16:12 |
| ildikov | but they don't have the spec and roadmap yet | 16:12 |
| gordc | ildikov: is it a schema or just versioning? | 16:12 |
| *** htruta has quit IRC | 16:12 | |
| ildikov | so this could be the time to collaborate a bit as if they change it anyway, then it could be changed in a way that's somehow good for us too | 16:12 |
| ildikov | gordc: as much as I undrestood they would like to have schema too, but they don't really have any material that would describe the plans yet | 16:13 |
| _nadya_ | I think that is_valid() addition doesn't block an action 'raise the question' | 16:13 |
| ildikov | I'm just saying that if we start to drop things now kind of silently that will be confusing from user perspective | 16:15 |
| gordc | ildikov: i see. yeah, i think our issue is schema right now... or in the case _nadya_ is addresssing, the middleware was giving unusable data for certain points | 16:15 |
| ildikov | but maybe I'm caring too much about users :) | 16:15 |
| gordc | _nadya_: did we ever figure out why it was missing a resource_id in payload | 16:15 |
| _nadya_ | the initial bug is here https://bugs.launchpad.net/ceilometer/+bug/1464265 . I can add handling to HBase of course but Mongo and HBase will act differently :( | 16:16 |
| openstack | Launchpad bug 1464265 in Ceilometer "hbase backend not collecting all Swift metric" [Undecided,In progress] - Assigned to Nadya Privalova (nprivalova) | 16:16 |
| gordc | sql should fail. i'm pretty sure i made resource_id mandatory field | 16:16 |
| _nadya_ | gordc: I tried to break it but no success. Maybe the issue happened on old version | 16:17 |
| gordc | _nadya_: yeah, i was looking at code, i couldn't figure out a scenario where it'd be empty | 16:17 |
| _nadya_ | gordc: so sql fails, HBase fails but Mongo lives | 16:18 |
| *** liusheng_phone has quit IRC | 16:19 | |
| *** afazekas has quit IRC | 16:23 | |
| gordc | _nadya_: can we not just create a unique index in mongo | 16:24 |
| gordc | _nadya_: i guess main concern i have is we're adding a check on every single datapoint now... this check is not needed most of the time... and is only useful in the case where we get unusable data | 16:25 |
| _nadya_ | so in fact move validation on storage lvl? | 16:25 |
| gordc | _nadya_: that's my thought. i don't know if anyone else has a differing opinion | 16:25 |
| gordc | iiuc, we'll never be able to recover these datapoints. | 16:26 |
| _nadya_ | gordc: could you please add it in review :)? I will ping someone else, just gather the thoughts | 16:27 |
| _nadya_ | gordc: and will see | 16:27 |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Make ceilometer work correctly when hosted with a SCRIPT_NAME https://review.openstack.org/195670 | 16:28 |
| cdent | Isn't the only way to get people to send good notifications to not accept them? | 16:30 |
| gordc | _nadya_: cool cool | 16:31 |
| ildikov | cdent: the only with this question is that those who send will not be the ones who face with the issue of missing samples | 16:31 |
| gordc | cdent: yeah, you'd think so. but then they say it's your fault and walk away | 16:31 |
| ildikov | only *issue | 16:31 |
| cdent | I agree with both of you :) | 16:31 |
| * cdent locates dinner | 16:32 | |
| * cdent waves | 16:32 | |
| ildikov | cdent: gordc: but I think we would need to be more pushy on this, then just let them walk away | 16:32 |
| *** cdent has quit IRC | 16:32 | |
| *** jaosorior has quit IRC | 16:35 | |
| *** yatin has joined #openstack-ceilometer | 16:36 | |
| gordc | ildikov: if we present an idea at cross project, what is the proposal? | 16:36 |
| ildikov | gordc: as for the unusable data from the middleware, can we address that? | 16:36 |
| gordc | you must include a resource_id (at a specific location) in your payload? | 16:36 |
| gordc | ildikov: we can... i actually don't know why it's missing resource_id | 16:37 |
| ildikov | if we can define what we consider as required field for "sane" samples, then it's a starting point | 16:37 |
| gordc | ildikov: i made this https://review.openstack.org/#/c/194397/ | 16:37 |
| ildikov | gordc: so I think we should validate those things that we actually can | 16:37 |
| gordc | cadf actually validates fields. i think we just let it slide with empty string | 16:37 |
| ildikov | cadf is good, but we have it only fo raudit data and not for everything | 16:39 |
| ildikov | why I'mpushing this is that we are trying to encourage every project to send notifications so we don't need to poll that much | 16:39 |
| *** htruta has joined #openstack-ceilometer | 16:43 | |
| *** shardy has quit IRC | 16:45 | |
| gordc | oh. i wasn't trying to sell cadf. i was just mentioning it is because the ceilometermiddleware uses it. | 16:45 |
| *** shardy has joined #openstack-ceilometer | 16:45 | |
| ildikov | ... but we don't give a guideline nor enforce to have required fields and schema etc to be able to validate | 16:45 |
| *** eglynn has quit IRC | 16:45 | |
| gordc | and that id that's missing in middleware shouldn't be missing which is why i'm confused | 16:46 |
| _nadya_ | sometimes resource_id is not needed in notifications. we construct it ourselves | 16:46 |
| gordc | ildikov: so i think every cycle for past 2 years, someone has tried to suggest a schema | 16:46 |
| gordc | ildikov: the general response is... 'maybe there's another alternative... i don't have a real example of an alternative, but maybe there is' | 16:47 |
| ildikov | gordc: I would buy it, but I'm already on your side :) | 16:47 |
| _nadya_ | and in the bug it's not missed actually. It's empty String | 16:47 |
| gordc | what notification would we consume that doesn't have a resource_id? | 16:48 |
| gordc | _nadya_: yeah. i don't know how... that mean that account doesn't have a name in swift? | 16:48 |
| *** r-mibu has quit IRC | 16:48 | |
| *** r-mibu has joined #openstack-ceilometer | 16:48 | |
| gordc | ildikov: meters/metrics definitely have a resource id... if it's not there, the datapoint makes no sense. | 16:49 |
| _nadya_ | gordc: yep. that's why cadf generates it | 16:49 |
| gordc | ildikov: events might not have a resource_id, but then you wonder what that event is actually about and why do i care about it. | 16:49 |
| gordc | _nadya_: you can have accounts without names in swift? | 16:50 |
| gordc | maybe we're using the wrong value as our resource_id in our middleware. | 16:50 |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Update alarm history only if change in alarm property https://review.openstack.org/195343 | 16:50 |
| *** jwcroppe_ has quit IRC | 16:51 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Implementation of dynamically reloadable pipeline https://review.openstack.org/190842 | 16:51 |
| openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer-specs: Spec for DBaaS(Trove) notification consumption https://review.openstack.org/192818 | 16:51 |
| *** jwcroppe has joined #openstack-ceilometer | 16:51 | |
| _nadya_ | gordc: don't think so. but if your reseller_prefix is wrong you may fetch empty line | 16:52 |
| *** rwsu has joined #openstack-ceilometer | 16:52 | |
| _nadya_ | gordc: in ipmi we do resource_id = '%(nodeid)s-%(sensorid)s' | 16:54 |
| ildikov | gordc: yeap, data without resource_id is mostly not useful | 16:55 |
| _nadya_ | in swift resource_id=message['payload']['target']['id'] . So the message may not contain 'resource_id' field | 16:55 |
| *** jwcroppe has quit IRC | 16:56 | |
| *** _cjones_ has joined #openstack-ceilometer | 16:58 | |
| gordc | _nadya_: well in cadf, (since that msg is in cadf) it's a mandatory field | 17:01 |
| gordc | _nadya_: if we can't get the value, it's either a bug in swift, or a issue with our middleware i think | 17:01 |
| _nadya_ | # Resource.id | 17:03 |
| _nadya_ | setattr(self, RESOURCE_KEYNAME_ID, id or identifier.generate_uuid()) | 17:03 |
| _nadya_ | gordc: it's cadf code | 17:04 |
| *** htruta has quit IRC | 17:04 | |
| *** dscottsf has joined #openstack-ceilometer | 17:05 | |
| _nadya_ | gordc: maybe just not send notification from our middleware if id is empty? | 17:06 |
| _nadya_ | gordc: solve problem with Swift only | 17:06 |
| *** dscottsf has quit IRC | 17:06 | |
| gordc | _nadya_: so if CADF generates an id if missing, how is it empty when it reaches the storage? | 17:09 |
| _nadya_ | gordc: I don't know :) I have only one thought: maybe the guy who filed the bug used very old version without cadf | 17:12 |
| _nadya_ | gordc: but in description (or a letter) he daid it's Kilo | 17:13 |
| _nadya_ | *said | 17:13 |
| gordc | _nadya_: i see... yeah we still had the old middleware in kilo package. | 17:14 |
| gordc | _nadya_: i'll comment on your patch. | 17:15 |
| *** _cjones_ has quit IRC | 17:17 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:17 | |
| *** shardy has quit IRC | 17:25 | |
| *** jfluhmann has joined #openstack-ceilometer | 17:25 | |
| *** shardy has joined #openstack-ceilometer | 17:26 | |
| *** ankita_wagh has joined #openstack-ceilometer | 17:28 | |
| *** rjaiswal has joined #openstack-ceilometer | 17:31 | |
| *** Ctina_ has joined #openstack-ceilometer | 17:31 | |
| *** Ctina has quit IRC | 17:35 | |
| *** Ctina_ has quit IRC | 17:36 | |
| *** ddieterly has joined #openstack-ceilometer | 17:39 | |
| *** jwcroppe has joined #openstack-ceilometer | 17:41 | |
| *** yassine_ has quit IRC | 17:45 | |
| kun_huang | guys, using ceilometer client, create a sample successfully and get [] from running samples.list() | 17:45 |
| kun_huang | what's the possible reasons | 17:45 |
| gordc | kun_huang: what does your pipeline look like? it's not explicit but even when you post to api, it gets filtered through api before going to db | 17:48 |
| *** yatin has quit IRC | 17:48 | |
| kun_huang | gordc: pipeline = request_id authtoken api-server | 17:49 |
| kun_huang | from api-paste.ini | 17:49 |
| *** afazekas has joined #openstack-ceilometer | 17:49 | |
| kun_huang | what I print from sample.create(...) is: | 17:51 |
| kun_huang | [<OldSample {u'counter_name': u'cpu_util', u'user_id': u'e7c841a59f5e45a2b95e1b4a8755dfd0', u'resource_id': u'rally_resource_QtHTpY9wcb', u'timestamp': u'2015-06-23T01:38:36.116869', u'resource_metadata': {}, u'source': u'ccd9013decc9485aaf69780436a1e59a:openstack', u'counter_unit': u'instance', u'counter_volume': 1.0, u'project_id': | 17:51 |
| kun_huang | u'ccd9013decc9485aaf69780436a1e59a', u'message_id': u'90bae1f8-1948-11e5-9639-000c29c6eec1', u'counter_type': u'gauge'}>] | 17:51 |
| gordc | kun_huang: oh i meant your pipeline.yaml | 17:51 |
| gordc | kun_huang: here's the bug i'm mentioning https://bugs.launchpad.net/ceilometer/+bug/1432077 | 17:51 |
| openstack | Launchpad bug 1432077 in Ceilometer "Ceilometer client fails to create samples with new (unsupported) meter/counter name" [Medium,Fix released] - Assigned to ZhiQiang Fan (aji-zqfan) | 17:51 |
| *** yprokule has quit IRC | 17:51 | |
| kun_huang | gordc: http://paste.openstack.org/show/321097/ | 17:52 |
| kun_huang | this one | 17:52 |
| *** boris-42 has joined #openstack-ceilometer | 17:54 | |
| *** afazekas has quit IRC | 17:55 | |
| kun_huang | gordc: btw rally calls ceilometer with counter_type, not meter_type. is that serious? | 17:57 |
| *** idegtiarov_ has joined #openstack-ceilometer | 18:00 | |
| gordc | kun_huang: meter_type? | 18:01 |
| kun_huang | the parameter of the command line "ceilometer sample-create" | 18:02 |
| *** dontalton has joined #openstack-ceilometer | 18:02 | |
| *** husanu4 has joined #openstack-ceilometer | 18:02 | |
| *** ildikov has quit IRC | 18:07 | |
| *** jwcroppe_ has joined #openstack-ceilometer | 18:08 | |
| *** jwcroppe_ has quit IRC | 18:09 | |
| gordc | kun_huang: sorry, just finishing lunch | 18:09 |
| gordc | kun_huang: i'm not actually familiar with exact names of param.. i'd need to check | 18:09 |
| *** jwcroppe_ has joined #openstack-ceilometer | 18:09 | |
| *** jwcroppe has quit IRC | 18:11 | |
| gordc | kun_huang: hmmm. you're right, the client expects meter-type not counter-type | 18:11 |
| *** husanu4 has quit IRC | 18:14 | |
| *** lsmola has quit IRC | 18:19 | |
| *** julim has quit IRC | 18:28 | |
| *** julim has joined #openstack-ceilometer | 18:28 | |
| *** jruano has quit IRC | 18:33 | |
| *** _nadya_ has quit IRC | 18:38 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:38 | |
| *** _nadya_ has quit IRC | 18:43 | |
| kun_huang | gordc: https://bugs.launchpad.net/heat/+bug/1468697 | 18:43 |
| openstack | Launchpad bug 1468697 in heat "Test ceilometer alarm scenario is failing on gates" [Medium,Confirmed] - Assigned to Kairat Kushaev (kkushaev) | 18:43 |
| kun_huang | this issue | 18:43 |
| *** dontalton has quit IRC | 18:44 | |
| kun_huang | not only rally gate is blocked, and heat's | 18:44 |
| *** j_t has left #openstack-ceilometer | 18:49 | |
| *** julim has quit IRC | 18:52 | |
| *** rbrooker has joined #openstack-ceilometer | 18:52 | |
| *** r-daneel has quit IRC | 18:54 | |
| *** r-daneel has joined #openstack-ceilometer | 18:55 | |
| *** jwcroppe_ has quit IRC | 18:55 | |
| *** jwcroppe has joined #openstack-ceilometer | 18:56 | |
| *** jwcroppe has quit IRC | 18:57 | |
| *** jwcroppe has joined #openstack-ceilometer | 18:57 | |
| *** julim has joined #openstack-ceilometer | 18:59 | |
| *** ddieterly has quit IRC | 19:02 | |
| *** julim has quit IRC | 19:02 | |
| *** julim has joined #openstack-ceilometer | 19:03 | |
| *** ddieterly has joined #openstack-ceilometer | 19:03 | |
| *** gzhai1 has quit IRC | 19:08 | |
| *** jruano has joined #openstack-ceilometer | 19:08 | |
| *** gzhai1 has joined #openstack-ceilometer | 19:09 | |
| *** htruta has joined #openstack-ceilometer | 19:13 | |
| *** ildikov has joined #openstack-ceilometer | 19:22 | |
| *** shardy has quit IRC | 19:42 | |
| *** Ephur has joined #openstack-ceilometer | 19:42 | |
| *** shardy has joined #openstack-ceilometer | 19:43 | |
| *** julim has quit IRC | 19:44 | |
| *** Ephur has quit IRC | 19:47 | |
| *** safchain has quit IRC | 19:48 | |
| boris-42 | gordc: hi | 19:52 |
| boris-42 | kun_huang: hi | 19:52 |
| boris-42 | gordc: kun_huang any progress regarding to finxing gates? | 19:52 |
| gordc | boris-42: kun_huang: we're waiting on: https://review.openstack.org/#/ | 19:54 |
| gordc | https://review.openstack.org/#/c/195570/ | 19:54 |
| gordc | boris-42: there are bigger breaks in the gate that blocking the revert | 19:59 |
| openstackgerrit | gordon chung proposed openstack/ceilometer: Revert "Add support for posting samples to notification-agent via API" https://review.openstack.org/195570 | 20:00 |
| *** nsaje_ has joined #openstack-ceilometer | 20:03 | |
| *** nsaje__ has joined #openstack-ceilometer | 20:04 | |
| boris-42 | gordc: oh | 20:05 |
| *** nsaje has quit IRC | 20:06 | |
| *** nsaje_ has quit IRC | 20:07 | |
| ankita_wagh | gordc : So if I choose Kafka, will the meters published in the form of flattened json | 20:08 |
| ankita_wagh | kafka as a publisher * | 20:08 |
| gordc | ankita_wagh: so yeah, when the publishers publish, the data will be serialised | 20:08 |
| gordc | ankita_wagh: let me check on that. i don't actually use the kafka publish | 20:08 |
| gordc | i'd assume it's in some format that is accept to kafka broker | 20:08 |
| ankita_wagh | I also need to add some fields before pushing into kafka | 20:09 |
| ankita_wagh | How can I do that | 20:09 |
| gordc | https://github.com/openstack/ceilometer/blob/master/ceilometer/publisher/kafka_broker.py#L104-L106 | 20:10 |
| gordc | meter_message_from_counter basically serialised a sample | 20:10 |
| gordc | to add fields you'll probably need to build your own transformer | 20:11 |
| gordc | there was a similar patch a while back https://review.openstack.org/#/c/152505/ | 20:11 |
| gordc | you could probably copy it... it was never merged | 20:11 |
| ankita_wagh | got it... thanku | 20:12 |
| *** Longgeek has quit IRC | 20:13 | |
| gordc | np | 20:15 |
| *** nsaje___ has joined #openstack-ceilometer | 20:15 | |
| *** nsaje__ has quit IRC | 20:18 | |
| *** ViswaV has joined #openstack-ceilometer | 20:19 | |
| *** nsaje____ has joined #openstack-ceilometer | 20:20 | |
| *** nsaje___ has quit IRC | 20:22 | |
| *** nsaje has joined #openstack-ceilometer | 20:22 | |
| gordc | ankita_wagh: i should mention, if you find that transformer useful, you can push to have it merged. | 20:24 |
| ankita_wagh | ok | 20:24 |
| *** nsaje____ has quit IRC | 20:25 | |
| *** julim has joined #openstack-ceilometer | 20:47 | |
| *** stevebaker has joined #openstack-ceilometer | 20:50 | |
| stevebaker | hey, I'm wondering if a ceilometer commit has caused a regression in check-heat-dsvm-functional-mysq | 20:52 |
| stevebaker | l | 20:52 |
| stevebaker | we have a voting job which is failing every run on the ceilometer alarm integration test http://logs.openstack.org/27/195627/1/check/check-heat-dsvm-functional-mysql/6f67bea/console.html#_2015-06-25_17_48_16_805 | 20:53 |
| stevebaker | hmm, http://logs.openstack.org/27/195627/1/check/check-heat-dsvm-functional-mysql/6f67bea/logs/screen-ceilometer-collector.txt.gz?level=ERROR | 20:57 |
| *** idegtiarov_ has quit IRC | 21:01 | |
| *** ankita_wagh has quit IRC | 21:03 | |
| *** prad has quit IRC | 21:05 | |
| stevebaker | gordc: https://bugs.launchpad.net/ceilometer/+bug/1468907 | 21:07 |
| openstack | Launchpad bug 1468907 in heat "heat_integrationtests CeilometerAlarmTest.test_alarm FAILED" [Critical,Triaged] | 21:07 |
| *** ddieterly has quit IRC | 21:11 | |
| *** ankita_wagh has joined #openstack-ceilometer | 21:16 | |
| *** Longgeek has joined #openstack-ceilometer | 21:21 | |
| *** xmltok_ has joined #openstack-ceilometer | 21:25 | |
| gordc | stevebaker sorry commented on bug. | 21:25 |
| *** xmltok_ has left #openstack-ceilometer | 21:25 | |
| gordc | stevebaker: it's being reverted... problem is there are bigger issues in gate which are blocking the revert | 21:26 |
| stevebaker | ok, I'll mark mine as a dupe | 21:26 |
| gordc | stevebaker: cool cool | 21:27 |
| stevebaker | gordc: is this a different issue? http://logs.openstack.org/27/195627/1/check/check-heat-dsvm-functional-mysql/6f67bea/logs/screen-ceilometer-collector.txt.gz?level=ERROR | 21:28 |
| gordc | stevebaker: yes that looks different | 21:29 |
| gordc | we recently made our id fields smaller. | 21:29 |
| stevebaker | ah, I'm pretty sure we've used the arn format since forever. it is big | 21:30 |
| gordc | we expect uuids... right now we still give quite a big buffer of 255 i believe | 21:30 |
| gordc | stevebaker: i'm not familiar with arn format... how big is that? we can revert the id fix but i'd rather not if the failure is because odd test data | 21:32 |
| stevebaker | arn:openstack:heat::22990abeee3941d8aec34c09bf78d009:stacks/AutoScalingSignalTest-1278811483-JobServerGroup-g3ohap5jraxp-ghgun47tpqcs-6gd2g27t75xy/ead09b07-4fac-45b0-ad9c-489f997925fe | 21:32 |
| stevebaker | the non-uuid bit in the middle could be up to 255 | 21:33 |
| stevebaker | we should probably just switch to uuid, which is the last element | 21:33 |
| gordc | oh... :( yeah, didn't think anyone was pushing ids that large | 21:34 |
| gordc | the reason we made it smaller was because it's impossible to build indexes when all our ids are varchar(1024) | 21:34 |
| stevebaker | I'll attach the bug to ceilo and heat, but likely the fix will be to heat | 21:35 |
| gordc | stevebaker: yes, that'd be ideal. let me know if that's not possible | 21:35 |
| gordc | we just merged it so it can be reverted... (it's in liberty-1 so i might need to figure out migration rules there) | 21:36 |
| *** shardy has quit IRC | 21:40 | |
| *** Longgeek has quit IRC | 21:40 | |
| stevebaker | I've raised https://bugs.launchpad.net/ceilometer/+bug/1468916 | 21:40 |
| openstack | Launchpad bug 1468916 in heat "stack arn is too big for shorter ceilometer resource_id column" [High,Triaged] | 21:40 |
| *** julim has quit IRC | 21:51 | |
| gordc | stevebaker: cool cool. just added a comment to it. | 21:52 |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Make ceilometer work correctly when hosted with a SCRIPT_NAME https://review.openstack.org/195670 | 22:03 |
| *** jwcroppe_ has joined #openstack-ceilometer | 22:05 | |
| *** jwcroppe has quit IRC | 22:07 | |
| *** edmondsw has quit IRC | 22:27 | |
| *** eglynn has joined #openstack-ceilometer | 22:31 | |
| *** dontalton has joined #openstack-ceilometer | 22:32 | |
| *** jruano has quit IRC | 22:35 | |
| *** ddieterly has joined #openstack-ceilometer | 22:37 | |
| *** Longgeek has joined #openstack-ceilometer | 22:37 | |
| *** ddieterly has quit IRC | 22:38 | |
| *** Longgeek has quit IRC | 22:42 | |
| openstackgerrit | Merged openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/190434 | 22:45 |
| *** dontalton has quit IRC | 22:45 | |
| *** dhellmann has quit IRC | 22:46 | |
| *** dhellmann has joined #openstack-ceilometer | 22:46 | |
| *** jfluhmann has quit IRC | 23:18 | |
| *** rbak has quit IRC | 23:21 | |
| *** rbrooker has quit IRC | 23:22 | |
| *** rbak has joined #openstack-ceilometer | 23:31 | |
| *** rbak has quit IRC | 23:34 | |
| *** ddieterly has joined #openstack-ceilometer | 23:43 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!