| *** rwsu has quit IRC | 00:31 | |
| *** thomasem has joined #openstack-ceilometer | 00:33 | |
| *** promulo has joined #openstack-ceilometer | 00:33 | |
| *** openstackgerrit has quit IRC | 00:35 | |
| *** openstackgerrit has joined #openstack-ceilometer | 00:35 | |
| openstackgerrit | Claudiu Belu proposed a change to openstack/ceilometer: Fixes Hyper-V metrics units https://review.openstack.org/78802 | 00:38 |
|---|---|---|
| *** raymondr has quit IRC | 00:39 | |
| *** shakayumi has quit IRC | 00:42 | |
| *** fnaval has quit IRC | 00:46 | |
| openstackgerrit | Adrian Turjak proposed a change to openstack/ceilometer: Adds flavor.id metric https://review.openstack.org/75728 | 00:48 |
| *** ironic has joined #openstack-ceilometer | 00:53 | |
| *** thomasem has quit IRC | 00:57 | |
| openstackgerrit | Adrian Turjak proposed a change to openstack/ceilometer: Adds a state metric to ceilometer https://review.openstack.org/75193 | 00:58 |
| *** xianghui has joined #openstack-ceilometer | 01:04 | |
| *** HT_Sergio has quit IRC | 01:05 | |
| *** Yathi has joined #openstack-ceilometer | 01:13 | |
| *** ironic has quit IRC | 01:16 | |
| *** Yathi has quit IRC | 01:21 | |
| *** Yathi has joined #openstack-ceilometer | 01:22 | |
| *** raymondr has joined #openstack-ceilometer | 01:22 | |
| *** xianghui has quit IRC | 01:37 | |
| *** Yathi has quit IRC | 01:48 | |
| openstackgerrit | Zhongyue Luo proposed a change to openstack/ceilometer: Removes use of timeutils.set_time_override https://review.openstack.org/78833 | 01:50 |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Fix updating alarm can specify existing alarm name https://review.openstack.org/78495 | 01:51 |
| *** _cjones_ has quit IRC | 01:56 | |
| *** prad_ has quit IRC | 01:58 | |
| *** nosnos has joined #openstack-ceilometer | 02:00 | |
| *** prad has joined #openstack-ceilometer | 02:01 | |
| *** mihgen has quit IRC | 02:02 | |
| *** dhellmann_ has joined #openstack-ceilometer | 02:05 | |
| *** changbl has joined #openstack-ceilometer | 02:05 | |
| *** bogdando has quit IRC | 02:06 | |
| *** dhellmann has quit IRC | 02:06 | |
| *** tasdomas has quit IRC | 02:07 | |
| *** nikhil___ has quit IRC | 02:07 | |
| *** dhellmann_ is now known as dhellmann | 02:07 | |
| *** bogdando has joined #openstack-ceilometer | 02:07 | |
| *** nikhil__ has joined #openstack-ceilometer | 02:07 | |
| *** tasdomas has joined #openstack-ceilometer | 02:09 | |
| *** nikhil___ has joined #openstack-ceilometer | 02:22 | |
| *** promulo_ has joined #openstack-ceilometer | 02:23 | |
| *** ondergetekende_ has joined #openstack-ceilometer | 02:24 | |
| *** nati_ueno has quit IRC | 02:29 | |
| *** openstackgerrit has quit IRC | 02:35 | |
| *** sbadia has quit IRC | 02:35 | |
| *** changbl has quit IRC | 02:35 | |
| *** jaypipes has quit IRC | 02:35 | |
| *** lsmola has quit IRC | 02:35 | |
| *** parkerr_ has quit IRC | 02:35 | |
| *** nprivalova has quit IRC | 02:35 | |
| *** llu has quit IRC | 02:35 | |
| *** bogdando has quit IRC | 02:35 | |
| *** janisg has quit IRC | 02:35 | |
| *** esheffie1d has quit IRC | 02:35 | |
| *** Kim-Chi-San has quit IRC | 02:35 | |
| *** sdake has quit IRC | 02:35 | |
| *** nikhil__ has quit IRC | 02:35 | |
| *** dhellmann has quit IRC | 02:35 | |
| *** promulo has quit IRC | 02:35 | |
| *** ondergetekende has quit IRC | 02:35 | |
| *** tianst20 has quit IRC | 02:35 | |
| *** sileht has quit IRC | 02:35 | |
| *** ondergetekende_ has quit IRC | 02:35 | |
| *** promulo_ has quit IRC | 02:35 | |
| *** nikhil___ has quit IRC | 02:35 | |
| *** nosnos has quit IRC | 02:35 | |
| *** gibi has quit IRC | 02:35 | |
| *** zul has quit IRC | 02:35 | |
| *** ryanpetrello has quit IRC | 02:35 | |
| *** ilyashakhat_ has quit IRC | 02:35 | |
| *** annegentle has quit IRC | 02:35 | |
| *** dragondm has quit IRC | 02:35 | |
| *** yjiang5 has quit IRC | 02:35 | |
| *** scroiset has quit IRC | 02:35 | |
| *** jd__ has quit IRC | 02:35 | |
| *** ashestakov has quit IRC | 02:35 | |
| *** nijaba has quit IRC | 02:35 | |
| *** tasdomas has quit IRC | 02:35 | |
| *** sdake_ has quit IRC | 02:35 | |
| *** Ruetobas has quit IRC | 02:35 | |
| *** nealph has quit IRC | 02:35 | |
| *** lakspace has quit IRC | 02:35 | |
| *** SergeyLukjanov has quit IRC | 02:35 | |
| *** ruhe has quit IRC | 02:35 | |
| *** jburkhart has quit IRC | 02:35 | |
| *** kin has quit IRC | 02:35 | |
| *** fc__ has quit IRC | 02:35 | |
| *** hhamalai has quit IRC | 02:35 | |
| *** gpocentek has quit IRC | 02:35 | |
| *** Daviey has quit IRC | 02:35 | |
| *** boris-42 has quit IRC | 02:35 | |
| *** nsaje_ has quit IRC | 02:35 | |
| *** EmilienM has quit IRC | 02:35 | |
| *** zigo has quit IRC | 02:35 | |
| *** Guest34377 has quit IRC | 02:35 | |
| *** ildikov_ has quit IRC | 02:35 | |
| *** ekarlso has quit IRC | 02:35 | |
| *** anteaya has quit IRC | 02:35 | |
| *** apmelton1 has quit IRC | 02:35 | |
| *** prad has quit IRC | 03:00 | |
| *** thomasem has joined #openstack-ceilometer | 03:02 | |
| *** thomasem has quit IRC | 03:03 | |
| *** raymondr has quit IRC | 03:08 | |
| *** raymondr has joined #openstack-ceilometer | 03:14 | |
| *** ondergetekende has joined #openstack-ceilometer | 03:22 | |
| *** yjiang5 has joined #openstack-ceilometer | 03:23 | |
| *** promulo has joined #openstack-ceilometer | 03:25 | |
| *** annegentle has joined #openstack-ceilometer | 03:31 | |
| *** zul has joined #openstack-ceilometer | 03:31 | |
| *** ryanpetrello has joined #openstack-ceilometer | 03:31 | |
| *** dragondm has joined #openstack-ceilometer | 03:31 | |
| *** scroiset has joined #openstack-ceilometer | 03:31 | |
| *** annegentle has quit IRC | 03:31 | |
| *** zul has quit IRC | 03:31 | |
| *** zul has joined #openstack-ceilometer | 03:32 | |
| *** annegentle has joined #openstack-ceilometer | 03:33 | |
| *** raymondr has quit IRC | 03:48 | |
| *** raymondr has joined #openstack-ceilometer | 03:48 | |
| *** raymondr has quit IRC | 03:53 | |
| *** raymondr has joined #openstack-ceilometer | 05:16 | |
| *** _nadya_ has joined #openstack-ceilometer | 05:32 | |
| *** _nadya_ has quit IRC | 05:37 | |
| *** sayalilunkad has joined #openstack-ceilometer | 05:42 | |
| *** raymondr has quit IRC | 06:03 | |
| *** sayali has joined #openstack-ceilometer | 06:24 | |
| *** sayalilunkad has quit IRC | 06:28 | |
| *** AMike has joined #openstack-ceilometer | 06:42 | |
| *** sayali has quit IRC | 06:46 | |
| *** saju_m has joined #openstack-ceilometer | 07:01 | |
| *** stzilli has joined #openstack-ceilometer | 07:53 | |
| *** ildikov_ has joined #openstack-ceilometer | 07:53 | |
| *** eglynn has joined #openstack-ceilometer | 07:53 | |
| *** piyushmasrani has joined #openstack-ceilometer | 07:53 | |
| *** xianghui has joined #openstack-ceilometer | 07:53 | |
| *** shakayumi has joined #openstack-ceilometer | 07:53 | |
| *** fnaval has joined #openstack-ceilometer | 07:53 | |
| *** flwang has joined #openstack-ceilometer | 07:53 | |
| *** gibi has joined #openstack-ceilometer | 07:53 | |
| *** nikhil__ has joined #openstack-ceilometer | 07:53 | |
| *** ilyashakhat has joined #openstack-ceilometer | 07:53 | |
| *** sbadia has joined #openstack-ceilometer | 07:53 | |
| *** tianst20 has joined #openstack-ceilometer | 07:53 | |
| *** sileht has joined #openstack-ceilometer | 07:53 | |
| *** dhellmann has joined #openstack-ceilometer | 07:53 | |
| *** tasdomas has joined #openstack-ceilometer | 07:53 | |
| *** bogdando has joined #openstack-ceilometer | 07:53 | |
| *** changbl has joined #openstack-ceilometer | 07:53 | |
| *** sdake_ has joined #openstack-ceilometer | 07:53 | |
| *** Ruetobas has joined #openstack-ceilometer | 07:53 | |
| *** nealph has joined #openstack-ceilometer | 07:53 | |
| *** jaypipes has joined #openstack-ceilometer | 07:53 | |
| *** lsmola has joined #openstack-ceilometer | 07:53 | |
| *** janisg has joined #openstack-ceilometer | 07:53 | |
| *** parkerr_ has joined #openstack-ceilometer | 07:53 | |
| *** nprivalova has joined #openstack-ceilometer | 07:53 | |
| *** ekarlso has joined #openstack-ceilometer | 07:53 | |
| *** lakspace has joined #openstack-ceilometer | 07:53 | |
| *** llu has joined #openstack-ceilometer | 07:53 | |
| *** jd__ has joined #openstack-ceilometer | 07:53 | |
| *** boris-42 has joined #openstack-ceilometer | 07:53 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 07:53 | |
| *** ruhe has joined #openstack-ceilometer | 07:53 | |
| *** jburkhart has joined #openstack-ceilometer | 07:53 | |
| *** nsaje_ has joined #openstack-ceilometer | 07:53 | |
| *** esheffie1d has joined #openstack-ceilometer | 07:53 | |
| *** Kim-Chi-San has joined #openstack-ceilometer | 07:53 | |
| *** ashestakov has joined #openstack-ceilometer | 07:53 | |
| *** sdake has joined #openstack-ceilometer | 07:53 | |
| *** nijaba has joined #openstack-ceilometer | 07:53 | |
| *** EmilienM has joined #openstack-ceilometer | 07:53 | |
| *** kin has joined #openstack-ceilometer | 07:53 | |
| *** fc__ has joined #openstack-ceilometer | 07:53 | |
| *** anteaya has joined #openstack-ceilometer | 07:53 | |
| *** hhamalai has joined #openstack-ceilometer | 07:53 | |
| *** gpocentek has joined #openstack-ceilometer | 07:53 | |
| *** Daviey has joined #openstack-ceilometer | 07:53 | |
| *** zigo has joined #openstack-ceilometer | 07:53 | |
| *** Guest34377 has joined #openstack-ceilometer | 07:53 | |
| *** apmelton1 has joined #openstack-ceilometer | 07:53 | |
| *** AMike has quit IRC | 07:56 | |
| *** AMike has joined #openstack-ceilometer | 07:56 | |
| *** taplax has joined #openstack-ceilometer | 07:59 | |
| *** eglynn has quit IRC | 08:18 | |
| *** shakayumi has quit IRC | 08:19 | |
| *** admin0 has joined #openstack-ceilometer | 08:28 | |
| admin0 | good morning | 08:34 |
| *** raymondr has joined #openstack-ceilometer | 08:35 | |
| *** raymondr has quit IRC | 08:39 | |
| *** nosnos has joined #openstack-ceilometer | 08:45 | |
| *** shakayumi has joined #openstack-ceilometer | 08:46 | |
| *** nacim has joined #openstack-ceilometer | 08:58 | |
| *** ildikov_ has quit IRC | 09:02 | |
| *** yassine has joined #openstack-ceilometer | 09:06 | |
| *** eglynn has joined #openstack-ceilometer | 09:07 | |
| *** mihgen has joined #openstack-ceilometer | 09:10 | |
| *** ildikov_ has joined #openstack-ceilometer | 09:18 | |
| *** flwang has quit IRC | 09:20 | |
| *** shakayumi has quit IRC | 09:26 | |
| *** shalmaliss has joined #openstack-ceilometer | 09:29 | |
| *** mihgen has quit IRC | 09:30 | |
| shalmaliss | hi, I am trying to install ceilometer in three-node set up using http://docs.openstack.org/havana/install-guide/install/apt/content/ceilometer-install.html | 09:31 |
| shalmaliss | But I am getting error "ServiceCatalogException: Invalid service catalog service: compute" | 09:31 |
| shalmaliss | in apache logs | 09:32 |
| shalmaliss | can anyone plz help me with this | 09:32 |
| shalmaliss | I could loging for the first time but now I can't login to horizon | 09:33 |
| *** Alexei_987 has joined #openstack-ceilometer | 09:34 | |
| *** admin0 has quit IRC | 09:38 | |
| *** admin0 has joined #openstack-ceilometer | 09:48 | |
| *** raymondr has joined #openstack-ceilometer | 10:04 | |
| *** Kim-Chi-San has quit IRC | 10:06 | |
| *** raymondr has quit IRC | 10:08 | |
| *** shakayumi has joined #openstack-ceilometer | 10:17 | |
| *** saju_m has quit IRC | 10:19 | |
| *** shakayumi has quit IRC | 10:22 | |
| *** flwang has joined #openstack-ceilometer | 10:42 | |
| *** admin0 has quit IRC | 10:50 | |
| *** admin0 has joined #openstack-ceilometer | 11:29 | |
| *** saju_m has joined #openstack-ceilometer | 11:30 | |
| *** admin0 has quit IRC | 11:35 | |
| *** shakayumi has joined #openstack-ceilometer | 11:41 | |
| *** xianghui has quit IRC | 11:43 | |
| *** piyushmasrani has quit IRC | 11:48 | |
| *** nosnos has quit IRC | 11:51 | |
| nsaje_ | guys, can someone run `ceilometer statistics -m cpu` and `ceilometer statistics -m cpu_util` ? I get almost the same output and would like to verify this as a bug | 11:57 |
| nsaje_ | i.e. it looks to me like the statistics query for cpu_util is taking cpu metric's values | 11:58 |
| eglynn | nsaje_: hmmm, that's kinda weird | 12:01 |
| eglynn | ... /me checks | 12:01 |
| *** admin0 has joined #openstack-ceilometer | 12:02 | |
| nsaje_ | in fact, every meter returns the same statistics on my devstak | 12:02 |
| nsaje_ | *devstack | 12:02 |
| *** shakayumi has quit IRC | 12:05 | |
| eglynn | nsaje_: does the raw query you see when running with --debug look sane? | 12:05 |
| nsaje_ | yes | 12:05 |
| nsaje_ | eglynn: the query is to http://192.169.33.15:8777/v2/meters/instance/statistics | 12:06 |
| nsaje_ | and I get | 12:06 |
| nsaje_ | https://www.irccloud.com/pastebin/iY73XTOJ | 12:06 |
| *** promulo_ has joined #openstack-ceilometer | 12:07 | |
| *** promulo has quit IRC | 12:08 | |
| eglynn | nsaje_: ok, that looks very wrong :( | 12:08 |
| nsaje_ | eglynn: unfortunately, yes. Could it have something to do with new aggregates changes? | 12:09 |
| eglynn | nsaje_: I'm thinking more likely the pipeline changes TBH | 12:09 |
| nsaje_ | eglynn: This is why we need tempest tests asap :D | 12:10 |
| eglynn | nsaje_: yep | 12:10 |
| eglynn | nsaje_: ok, let me look into it, I don't have a fresh devstack running so it'll take me a while to get set up | 12:11 |
| nsaje_ | eglynn: ok, I'm looking into it also, I'll see if I can find anything | 12:11 |
| nsaje_ | eglynn: should I report a bug or wait? | 12:11 |
| eglynn | nsaje_: yep, please file a bug | 12:12 |
| eglynn | nsaje_: which storage driver, mongo or sqlalchemy? | 12:17 |
| nsaje_ | sqlalchemy | 12:17 |
| eglynn | nsaje_: ... have you checked the counter names in the DB? | 12:17 |
| nsaje_ | yes, they look alright | 12:17 |
| nsaje_ | eglynn: anything specific I should be looking for? | 12:17 |
| nsaje_ | eglynn: even listing the samples through CLI returns the correct data | 12:18 |
| eglynn | nsaje_: "correct data"? | 12:18 |
| eglynn | nsaje_: correct volume but wrong meter names? | 12:18 |
| nsaje_ | e.g. cpu_util returns data samples in the range 0-100 | 12:18 |
| nsaje_ | eglynn: no, everything is correct. Only statistics command is problematic | 12:19 |
| nsaje_ | eglynn: so it's not a problem of persisting the samples | 12:20 |
| nsaje_ | eglynn: the problem should be when the samples are retrieved for statistics computation | 12:20 |
| eglynn | nsaje_: yep, that sounds likely | 12:20 |
| *** promulo_ has quit IRC | 12:22 | |
| *** saju_m has quit IRC | 12:49 | |
| *** shakayumi has joined #openstack-ceilometer | 12:57 | |
| *** julim has joined #openstack-ceilometer | 13:06 | |
| eglynn | nsaje_: finally got a devstack up, yep I'm seeing the same against sqlalchemy but not mongo | 13:07 |
| eglynn | nsaje_: looking into it now | 13:07 |
| nsaje_ | eglynn: I think the problem could be this line: https://git.openstack.org/cgit/openstack/ceilometer/tree/ceilometer/storage/impl_sqlalchemy.py#n172 | 13:11 |
| nsaje_ | eglynn: but I haven't yet figured out what that line should really be :) | 13:11 |
| eglynn | nsaje_: yeah, thinking the same ... I just confirmed that constraint is actually being added to the query | 13:12 |
| eglynn | nsaje_: ... I wonder could it related to the recent re-org of the samples table? | 13:12 |
| nsaje_ | eglynn: seems very likely, since [counter|meter|sample]_[id|name] were thrown around | 13:12 |
| nprivalova | eglynn: I have an update about notifications. Looks like it's just too long. In tempest logs I see that the last attempt for retrieving notifications was at 13:37:45. In collector logs I see the message 13:43:19.361 28730 DEBUG ceilometer.openstack.common.rpc.amqp [-] received {message}. The message is about vcpus so it cannot be from pollsters. We continue investigations, will write email today | 13:13 |
| eglynn | nprivalova: great, thanks for the update! ... I wonder has nova extended the notification payload size beyond some magical threshold? | 13:14 |
| eglynn | nprivalova: ... or maybe some variable aspect of the notification (e.g. user metadata for instance) is causing it to bloat | 13:15 |
| nprivalova | eglynn: do you think that the problem is message's size? | 13:15 |
| eglynn | nprivalova: sorry, I mis-interpreted what you were saying ... "Looks like it's just too long" | 13:16 |
| nprivalova | eglynn: I meant that notifications are processed for a long time. Or is sending for a long time...I think the first variant is more likely | 13:17 |
| eglynn | nprivalova: a-ha, I see ... my bad! ... apols for the noise :) | 13:17 |
| eglynn | nsaje_: just rolled back my devstack to prior to the pipeline and aggregate changes landing, and still seeing the issue | 13:18 |
| eglynn | nsaje_: ... so looks like they're off the hook for now | 13:19 |
| eglynn | nsaje_: ... looking into that latest migration now | 13:19 |
| *** shakayumi has quit IRC | 13:20 | |
| eglynn | nsaje_: i.e. this commit https://github.com/openstack/ceilometer/commit/604fc825 | 13:22 |
| nsaje_ | eglynn: I'm not intimately familiar with sqlalchemy, but are we missing a join? Since meter is now a separate table | 13:24 |
| eglynn | nsaje_: yep exactly what I was thinking | 13:25 |
| eglynn | nsaje_: ... but I'm confused as to why sqlalchemy would consider (models.Meter.name == sample_filter.meter) constantly true in the absence of a join | 13:26 |
| *** Kim-Chi-San has joined #openstack-ceilometer | 13:31 | |
| eglynn | nsaje_: hmmm, maybe the join is implicit ... quick attempt to add an explicit join failed for me with an error that implies the join was already in place | 13:33 |
| nsaje_ | eglynn: I got it working | 13:35 |
| eglynn | nsaje_: cool, what was the fix? | 13:35 |
| nsaje_ | eglynn: for sqlalchemy the 'base' model is Meter | 13:36 |
| nsaje_ | so I joined a Sample to it | 13:36 |
| nsaje_ | and it works | 13:36 |
| nsaje_ | (probably because the Meter.unit is listed first in the select parameters) | 13:36 |
| eglynn | nsaje_: excellent! | 13:36 |
| eglynn | nsaje_: can you get a fix up on gerrit? | 13:36 |
| nsaje_ | eglynn: ASAP | 13:37 |
| eglynn | nsaje_: unfortunately we've missed the boat on the i3 tag | 13:37 |
| nsaje_ | eglynn: broken alarming in i3 :D | 13:37 |
| eglynn | nsaje_: ... I'm guessin a re-spin is out of the question | 13:37 |
| eglynn | nsaje_: ... all statistics broken, even worse :( | 13:37 |
| eglynn | nsaje_: got a bug number? | 13:38 |
| eglynn | nsaje_: I'll mark critical and target to icehouse-rc1 | 13:38 |
| nsaje_ | https://bugs.launchpad.net/ceilometer/+bug/1289325 | 13:38 |
| *** admin0 has quit IRC | 13:38 | |
| *** raymondr has joined #openstack-ceilometer | 13:40 | |
| eglynn | jd__: around? | 13:40 |
| sileht | hum, annoying bug ... | 13:41 |
| *** ityaptin has joined #openstack-ceilometer | 13:42 | |
| nsaje_ | eglynn: we probably should take a closer look at that filtering function, it filters mostly on Sample and it isn't joined anywhere | 13:44 |
| eglynn | nsaje_: yep, lets get the initial fix out for review anyway | 13:44 |
| eglynn | nsaje_: we can have a mpore detailed discussion when gordc comes online | 13:44 |
| eglynn | jd__: nsaje_ has unearthed a serious breakage in statistics for sqlalchemy | 13:45 |
| eglynn | ... samples for *all* meters are included in all stats queries | 13:45 |
| eglynn | ... defect has on master since https://github.com/openstack/ceilometer/commit/604fc825 (i.e. circa 10 days) | 13:45 |
| eglynn | ... and is in icehouse-3 unfortunately :( | 13:45 |
| eglynn | jd__: ... would asking for a re-spin of icehouse-3 be out of the question? | 13:46 |
| *** admin0 has joined #openstack-ceilometer | 13:47 | |
| nsaje_ | eglynn: https://review.openstack.org/#/c/78962/ | 13:52 |
| eglynn | nsaje_: looking | 13:52 |
| sileht | eglynn, nsaje_ I wonder why no unit tests have catched that | 13:53 |
| sileht | w+2: Julien Danjou <julien@danjou.info> | 13:54 |
| sileht | Approved+1: Mehdi Abaakouk <mehdi.abaakouk@enovance.com> | 13:54 |
| sileht | oups | 13:54 |
| eglynn | sileht: prolly coz the statistics scenario tests all use a constant meter name | 13:54 |
| sileht | eglynn, we needs to add a new test so | 13:54 |
| eglynn | sileht: yep | 13:54 |
| *** raymondr has quit IRC | 13:55 | |
| *** shalmaliss has quit IRC | 13:57 | |
| *** admin0 has quit IRC | 13:58 | |
| *** thomasem has joined #openstack-ceilometer | 13:59 | |
| *** nsaje_ is now known as nsaje_on_the_bus | 14:00 | |
| *** admin0 has joined #openstack-ceilometer | 14:01 | |
| *** admin0 has quit IRC | 14:02 | |
| *** bowdengl has joined #openstack-ceilometer | 14:06 | |
| *** yfujioka has quit IRC | 14:07 | |
| *** raymondr has joined #openstack-ceilometer | 14:08 | |
| ildikov_ | nsaje_: hi | 14:11 |
| ildikov_ | nsaje_: I've posted a comment to that fix, could you please check? | 14:11 |
| *** raymondr has quit IRC | 14:12 | |
| jd__ | eglynn: that's what rc1 is for, fixing those bugs :) | 14:21 |
| eglynn | jd__: yeah, I guess I'm thinking there may be folks actually deploying milestones with some expectation of non-borked-ness | 14:23 |
| *** gordc has joined #openstack-ceilometer | 14:23 | |
| eglynn | jd__: ... well more of an expectation that when just plain trunk chasing | 14:23 |
| eglynn | jd__: ... any-hoo, for the i3-based spin of the RDO distro I guess we'll just carry an extra patch | 14:24 |
| jd__ | eglynn: yeah sounds reasonable | 14:24 |
| eglynn | jd__: cool enough, /me unpresses the panic button ;) | 14:25 |
| *** nsaje_on_the_bus is now known as nsaje | 14:26 | |
| *** raymondr has joined #openstack-ceilometer | 14:26 | |
| *** parkerr_ has quit IRC | 14:27 | |
| ildikov_ | nsaje: hey | 14:28 |
| *** admin0 has joined #openstack-ceilometer | 14:28 | |
| nsaje | eglynn, ildikov_ : looking at comments now, sorry I had to commute | 14:28 |
| eglynn | nsaje: np ... I twigged your on_the_bus nick and assumed as much :) | 14:28 |
| ildikov_ | nsaje: /me sorry, I've just realized when you changed back your name that you wre on the bus :) | 14:29 |
| *** gordc has quit IRC | 14:29 | |
| nsaje | ildikov_: great suggestion, MeterSampleModel looks like the class to use | 14:30 |
| *** raymondr has quit IRC | 14:30 | |
| eglynn | nsaje: .. prolly an idea to add gordc as a reviewer on that patch also | 14:31 |
| ildikov_ | eglynn, nsaje: I've just wanted to suggest that | 14:31 |
| *** gordc has joined #openstack-ceilometer | 14:31 | |
| ildikov_ | eglynn, nsaje: gordc can confirm my suggestion too, if MeterSample looks like the one, or it is the one | 14:32 |
| *** gordc has quit IRC | 14:33 | |
| *** gordc has joined #openstack-ceilometer | 14:33 | |
| nsaje | eglynn: if this is urgent to fix today, perhaps someone with more sqlalchemy experience should take over this patch | 14:35 |
| eglynn | gordc: hey | 14:35 |
| gordc | eglynn: whatsup? | 14:36 |
| eglynn | gordc: you might want to read the baklog http://eavesdrop.openstack.org/irclogs/%23openstack-ceilometer/%23openstack-ceilometer.2014-03-07.log | 14:36 |
| eglynn | gordc: (issue with statistics queries in sqlalchemy) | 14:36 |
| eglynn | nsaje: actually its not urgent that it gets in today | 14:36 |
| eglynn | nsaje: let's take our time and get it right | 14:36 |
| gordc | eglynn: i saw the patch. i'll read the log to see what's being discussed. | 14:37 |
| eglynn | nsaje: the urgency was me initially hoping to sneak the fix into a icehouse-3 respin | 14:37 |
| eglynn | gordc: cool thanks! | 14:37 |
| *** prad_ has joined #openstack-ceilometer | 14:37 | |
| eglynn | nsaje: but that's not going to happen, so I think we have time to get this right | 14:37 |
| ildikov_ | eglynn: in get it right we also need to include the tests | 14:38 |
| eglynn | ildikov_: true dat | 14:38 |
| *** parkerr_ has joined #openstack-ceilometer | 14:44 | |
| *** gordc1 has joined #openstack-ceilometer | 14:53 | |
| *** gordc has quit IRC | 14:54 | |
| nsaje | gordc1: what say you, should MeterSample be used in https://git.openstack.org/cgit/openstack/ceilometer/tree/ceilometer/storage/impl_sqlalchemy.py#n172 ? | 14:55 |
| gordc1 | nsaje: testing it out right now. (or trying to... my internet keeps dying on me) | 14:57 |
| gordc1 | nsaje: switching all the aggregate items to MeterSample should work but i'm trying to test if that's needed or if it's broken elsewhere. | 14:58 |
| nsaje | gordc1: well, feel free to take over fixing this, I don't have much sqlalchemy experience :) | 14:58 |
| *** gordc1 is now known as gordc | 14:59 | |
| gordc | nsaje: cool cool. i'll continue looking at it and see what i can dig up | 14:59 |
| *** stzilli has quit IRC | 15:01 | |
| *** fnaval has quit IRC | 15:10 | |
| *** gordc has quit IRC | 15:16 | |
| *** gordc has joined #openstack-ceilometer | 15:16 | |
| *** openstackgerrit has joined #openstack-ceilometer | 15:22 | |
| *** admin0 has quit IRC | 15:29 | |
| *** admin0 has joined #openstack-ceilometer | 15:30 | |
| *** ryanpetrello has quit IRC | 15:31 | |
| *** ryanpetrello has joined #openstack-ceilometer | 15:32 | |
| *** fnaval has joined #openstack-ceilometer | 15:34 | |
| *** bowdengl has quit IRC | 15:38 | |
| *** shakayumi has joined #openstack-ceilometer | 15:39 | |
| *** admin0 has quit IRC | 15:56 | |
| *** Ruetobas has quit IRC | 16:04 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:05 | |
| *** mihgen has joined #openstack-ceilometer | 16:05 | |
| *** shakayumi has quit IRC | 16:08 | |
| *** AMike has quit IRC | 16:09 | |
| *** Ruetobas has quit IRC | 16:09 | |
| *** admin0 has joined #openstack-ceilometer | 16:10 | |
| *** admin0 has quit IRC | 16:13 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:14 | |
| *** admin0 has joined #openstack-ceilometer | 16:19 | |
| *** admin0 has quit IRC | 16:21 | |
| ildikov_ | gordc: hi. are you around? | 16:26 |
| gordc | ildikov_: yep | 16:26 |
| ildikov_ | gordc: I just have a quick question about negative api tests | 16:26 |
| ildikov_ | grodc: I was wondering if we should test the error messgae or not as those usually have translation | 16:27 |
| *** raymondr has joined #openstack-ceilometer | 16:27 | |
| ildikov_ | gordc: I'm not sure about if it is handled by the unit tests or not | 16:27 |
| gordc | ildikov_: hmm. good question. i've no idea how the translation work... i'd assume the unit tests take the raw 'untranslated' messages | 16:29 |
| gordc | ildikov_: are you adding tests or is this relating to a bug? | 16:30 |
| ildikov_ | gordc: I added a review comment to a test earlier and I got back this issue as a reason for not testing the expected error message | 16:31 |
| *** raymondr has quit IRC | 16:31 | |
| ildikov_ | gordc: in case of that patch it was a minor issue and that was about to merge into I-3, so I let it go, now I have another review, where I have already a few comments and this would also be on the list, so I thought to ask about it | 16:32 |
| ildikov_ | gordc: do you have any hints, where should I start the investigation of this topic? | 16:33 |
| ildikov_ | gordc: in my view it is better to test the error message too as in many cases it is generated and it is better to konw if the text that was generated after an exception is correct or not | 16:33 |
| gordc | ildikov_: ah i see... to be honest i don't know when the translation happens... i would think it would happen during packaging. | 16:34 |
| ildikov_ | gordc: and also we already have some test cases that checks these messages | 16:34 |
| gordc | ildikov_: search ceilometer code, i vaguely remember that we have a few tests that check the error message. | 16:34 |
| gordc | ildikov_: if so, i would assume translation is a non-issue in regards to unit tests. | 16:35 |
| ildikov_ | gordc: yes we have, I wrote some tests of this kind and the patch is already merged | 16:35 |
| ildikov_ | gordc: and there were some already, when I wrote mine | 16:36 |
| ildikov_ | gordc: that sounds logical | 16:36 |
| gordc | ildikov_: yeah... i've never heard of anyone complain about tests not running on their machine because of translation... | 16:36 |
| gordc | ildikov_: that said, i couldn't tell you anyone who uses the translations so that could be the reason. | 16:37 |
| ildikov_ | gordc: me neither, but we also have en_us locale on every of our test machines | 16:38 |
| gordc | ildikov_: same. | 16:39 |
| *** raymondr has joined #openstack-ceilometer | 16:40 | |
| ildikov_ | gordc: and what is your opinion about testing the error messages in general, independently from this translation question? | 16:40 |
| ildikov_ | gordc: in my view it is important, but maybe I'm just too maniac :) | 16:41 |
| gordc | ildikov_: for me, it's not really important... unless there's a bunch of different exception cases and we particularly want to make sure it's getting caught by right clause. | 16:44 |
| *** raymondr has quit IRC | 16:45 | |
| gordc | ildikov_: although i'm probably not a good person to ask about tests (i should probably improve my tests). | 16:45 |
| ildikov_ | gordc: in the review I'm just in, there three ClientSideErrors in that validate function and all of them with the default error code (HTTP 400) | 16:47 |
| ildikov_ | s/there three/there are three/ | 16:48 |
| ildikov_ | gordc: maybe this can be one case, when it would be useful to check | 16:50 |
| gordc | ildikov_: i see... i'd probably be indifferent to it since they all have same error (if that's right). that said, i'm sure there are people who share your opinion that it should be verified... | 16:51 |
| gordc | ildikov_: for me, if you added that tests in i wouldn't mind since it doesn't hurt to test for more things... whether it's useful or not, you'll get different opinions on. :) | 16:52 |
| ildikov_ | gordc: ok, I get it :) | 16:53 |
| *** _cjones_ has joined #openstack-ceilometer | 16:53 | |
| ildikov_ | gordc: I do not want to steal your time, so thanks for the opinions and the discussion | 16:55 |
| *** yassine has quit IRC | 16:56 | |
| gordc | ildikov_: np. good discussion. i typically just put comments on how i'd do it. if people disagree they'll say so... not a big deal. if you feel like it's really important, you can put it to a bigger forum (ie. meeting or mailinglist) to see what community thinks. | 16:59 |
| ildikov_ | gordc: ok, thanks, it's a good idea, improving tests is always a good topic I think :) | 17:01 |
| ildikov_ | gordc: sorry, I need to run now, thanks again for guiding me :) | 17:01 |
| gordc | ildikov_: agreed. even though i hate writing tests. | 17:01 |
| gordc | ildikov_: cool cool, ttyl | 17:01 |
| ildikov_ | gordc: sometimes me too, but during development I feel that it is useful to have tests | 17:03 |
| ildikov_ | gordc: laters | 17:03 |
| *** ildikov_ has quit IRC | 17:07 | |
| *** nacim has quit IRC | 17:07 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Enable monkeypatch for select module https://review.openstack.org/78654 | 17:10 |
| *** rwsu has joined #openstack-ceilometer | 17:12 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: Fix broken statistics in sqlalchemy https://review.openstack.org/78962 | 17:14 |
| *** promulo has joined #openstack-ceilometer | 17:15 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: Fix broken statistics in sqlalchemy https://review.openstack.org/78962 | 17:15 |
| *** lsmith has joined #openstack-ceilometer | 17:18 | |
| lsmith | I am working on setting up MySQL backend for ceilometer. I have the connection working but no tables. I've looked for setup files, etc back can't seem to find any reference on how the MySQL tables get created. | 17:19 |
| eglynn | lsmith: ceilometer-dbsync --config-file /etc/ceilometer/ceilometer.conf | 17:20 |
| eglynn | lsmith: after creating the DB | 17:20 |
| eglynn | lsmith: i.e. fire up mysql client and run "create database ceilometer;" | 17:21 |
| *** _nadya_ has joined #openstack-ceilometer | 17:21 | |
| eglynn | lsmith: in /etc/ceilometer/ceilometer.conf ensure you've got ... | 17:21 |
| eglynn | [database] | 17:21 |
| eglynn | connection = mysql://$USER:$PASSWD@$IP_ADDR/ceilometer?charset=utf8 | 17:22 |
| eglynn | lsmith: ^^^ with the appropriate substitutions | 17:22 |
| lsmith | eglynn Awesome Thanks that what I was looking for. Was not able to find it in the docs or google search. Thanks again | 17:23 |
| eglynn | lsmith: np! | 17:23 |
| *** nati_ueno has joined #openstack-ceilometer | 17:29 | |
| *** _nadya_ has quit IRC | 17:29 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:30 | |
| *** sayalilunkad has joined #openstack-ceilometer | 17:42 | |
| *** nati_ueno has quit IRC | 17:46 | |
| *** _nadya_ has quit IRC | 17:46 | |
| *** thomasem has quit IRC | 17:47 | |
| *** raymondr has joined #openstack-ceilometer | 17:56 | |
| *** gordc has quit IRC | 18:00 | |
| *** raymondr has quit IRC | 18:00 | |
| *** gordc has joined #openstack-ceilometer | 18:05 | |
| *** thomasem has joined #openstack-ceilometer | 18:06 | |
| *** sayalilunkad has quit IRC | 18:08 | |
| *** thomasem has quit IRC | 18:09 | |
| *** sayalilunkad has joined #openstack-ceilometer | 18:09 | |
| *** sayalilunkad has quit IRC | 18:09 | |
| *** eglynn has quit IRC | 18:10 | |
| *** sayalilunkad has joined #openstack-ceilometer | 18:10 | |
| *** nati_ueno has joined #openstack-ceilometer | 18:20 | |
| *** mihgen has quit IRC | 18:23 | |
| *** shakayumi has joined #openstack-ceilometer | 18:24 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:25 | |
| *** mihgen has joined #openstack-ceilometer | 18:35 | |
| *** sayali has joined #openstack-ceilometer | 18:40 | |
| *** sayalilunkad has quit IRC | 18:43 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: Fix broken statistics in sqlalchemy https://review.openstack.org/78962 | 18:45 |
| *** gordc has quit IRC | 18:54 | |
| *** eglynn has joined #openstack-ceilometer | 18:55 | |
| eglynn | sayali: back | 18:58 |
| eglynn | sayali: did you run that mysql command? | 18:58 |
| *** admin0 has joined #openstack-ceilometer | 18:58 | |
| sayali | eglynn: mysql shell is giving me errors | 18:59 |
| eglynn | sayali: what kind of errors? | 18:59 |
| sayali | eglynn: syntax errors | 18:59 |
| *** admin0 has quit IRC | 19:00 | |
| eglynn | sayali: copy and paste the following | 19:00 |
| eglynn | select volume, resource_id from sample where meter_id in (select id from meter where name = 'cpu_util'); | 19:00 |
| sayali | nope the first command only fails | 19:00 |
| sayali | for authenticating to the database | 19:00 |
| eglynn | sayali: paste? | 19:00 |
| eglynn | that's not a syntax error | 19:01 |
| sayali | eglynn : http://paste.openstack.org/show/72918 | 19:02 |
| eglynn | sayali: try ... | 19:02 |
| eglynn | mysql -u root -p$(grep DATABASE_PASSWORD ~/devstack/localrc | cut -f2 -d=) ceilometer | 19:02 |
| eglynn | sayali: arggh! | 19:03 |
| eglynn | sayali: infinite recursion :) | 19:03 |
| eglynn | sayali: ... you've tried to fire up mysql *within* mysql | 19:04 |
| eglynn | sayali: ctrl+D | 19:04 |
| eglynn | sayali: at the *shell* prompt paste: mysql -u root -p$(grep DATABASE_PASSWORD ~/devstack/localrc | cut -f2 -d=) ceilometer | 19:04 |
| eglynn | sayali: note *no space* after the -p | 19:05 |
| sayali | eglynn: ok now that got me to a database shell | 19:06 |
| eglynn | sayali: w00t! | 19:06 |
| eglynn | sayali: select volume, resource_id from sample where meter_id in (select id from meter where name = 'cpu_util'); | 19:06 |
| eglynn | sayali: can you paste the output of that command? | 19:08 |
| sayali | eglynn: Error:- table ceilometer.sample does not exist | 19:08 |
| eglynn | sayali: show tables; | 19:08 |
| eglynn | sayali: how old is your devstack? | 19:09 |
| sayali | eglynn: http://paste.openstack.org/show/72919 | 19:09 |
| eglynn | sayali: ctrl+D | 19:10 |
| eglynn | sayali: cd /opt/stack/ceilometer/ | 19:11 |
| eglynn | sayali: git log | head -10 | 19:11 |
| sayali | eglynn: maybe a month old | 19:11 |
| eglynn | sayali: really? we're debugging a *month* old devstack? | 19:11 |
| sayali | Feb 12th:| | 19:12 |
| eglynn | sayali: that's ancient history I'm afraid | 19:13 |
| sayali | eglynn: I'll spin a new one and check | 19:13 |
| eglynn | sayali: I can't even remember the sqlalchemy schema from back then | 19:13 |
| eglynn | sayali: yes that would be wise | 19:13 |
| eglynn | sayali: but before you do so | 19:13 |
| eglynn | sayali: go back into mysql client | 19:13 |
| sayali | eglynn: though I had done that a few days back on another VM when I first approached you with this problem | 19:14 |
| sayali | eglynn: ok and ? | 19:14 |
| eglynn | sayali: select counter_volume, resource_id from meter where counter_name = 'cpu_util' | 19:15 |
| eglynn | ; | 19:15 |
| eglynn | sayali: ... /me is confused as to why you went back to such an old devstack | 19:15 |
| eglynn | sayali: ... why not just use the new-ish one you spun up a few days ago? | 19:16 |
| eglynn | sayali: devstacks should be treated like cattle, as opposed to beloved pets ;) | 19:17 |
| eglynn | sayali: ... just shoot them in the head when they start showing their age | 19:17 |
| sayali | eglynn: that one kept crashing when I did stack.sh | 19:17 |
| eglynn | sayali: ... don't try to keep nursing an old one along | 19:17 |
| eglynn | sayali: ... anyway, any joy with that SQL statement above? | 19:17 |
| sayali | eglynn: http://paste.openstack.org/show/72920 | 19:17 |
| eglynn | sayali: what's your instance ID, 2fcbf418-d62f-427c-ab14-b109c1bbcff9 perchance? | 19:18 |
| sayali | eglynn: yes | 19:19 |
| eglynn | sayali: ok so the samples exist in the metering store | 19:19 |
| eglynn | sayali: which means that the sample gathering pipeline is all working | 19:19 |
| eglynn | sayali: which is good | 19:20 |
| sayali | eglynn: yes | 19:20 |
| eglynn | sayali: but for some reason you don't see those samples via the ceilo-api service | 19:20 |
| eglynn | sayali: now we could drill down and try to figure out why | 19:20 |
| eglynn | sayali: but debugging a month-old devstack is not a good use of your time | 19:20 |
| sayali | eglynn: before that do you want me to start fresh? | 19:20 |
| eglynn | sayali: or of mine, for that matter | 19:20 |
| eglynn | sayali: yeap, you got it in one! :) | 19:21 |
| sayali | eglynn: ok cool! how long would you be around? | 19:21 |
| eglynn | sayali: it's friday evening getting on for 7:30pm in my TZ, so not for too long | 19:22 |
| sayali | eglynn: ok, will ping if your still around:) | 19:22 |
| eglynn | sayali: k | 19:22 |
| *** thomasem has joined #openstack-ceilometer | 19:29 | |
| *** yassine has joined #openstack-ceilometer | 19:32 | |
| *** shakayumi has quit IRC | 19:33 | |
| *** gordc has joined #openstack-ceilometer | 19:43 | |
| *** raymondr has joined #openstack-ceilometer | 19:45 | |
| *** zul has quit IRC | 19:55 | |
| *** _nadya_ has quit IRC | 19:58 | |
| *** raymondr has quit IRC | 20:01 | |
| *** ryanpetrello has quit IRC | 20:02 | |
| *** ryanpetrello has joined #openstack-ceilometer | 20:04 | |
| *** sayali has quit IRC | 20:05 | |
| *** _nadya_ has joined #openstack-ceilometer | 20:22 | |
| *** raymondr has joined #openstack-ceilometer | 20:38 | |
| *** _nadya_ has quit IRC | 20:39 | |
| *** eglynn has quit IRC | 20:39 | |
| *** admin0 has joined #openstack-ceilometer | 21:19 | |
| *** ryanpetrello_ has joined #openstack-ceilometer | 21:21 | |
| *** ryanpetrello has quit IRC | 21:23 | |
| *** ryanpetrello_ is now known as ryanpetrello | 21:23 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: Fix broken statistics in sqlalchemy https://review.openstack.org/78962 | 21:28 |
| *** promulo has quit IRC | 21:31 | |
| *** admin0 has quit IRC | 21:35 | |
| openstackgerrit | Rob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation https://review.openstack.org/77957 | 21:40 |
| *** admin0 has joined #openstack-ceilometer | 21:50 | |
| *** lsmith has quit IRC | 21:52 | |
| *** promulo has joined #openstack-ceilometer | 21:55 | |
| *** admin0 has quit IRC | 22:01 | |
| *** ryanpetrello has quit IRC | 22:04 | |
| *** thomasem has quit IRC | 22:09 | |
| *** gordc has quit IRC | 22:26 | |
| *** yassine has quit IRC | 22:32 | |
| *** julim has quit IRC | 22:42 | |
| openstackgerrit | Rob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation https://review.openstack.org/77957 | 22:51 |
| *** yanuha has joined #openstack-ceilometer | 22:53 | |
| *** rwsu has quit IRC | 23:15 | |
| *** zigo has quit IRC | 23:17 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!