| *** thomasm has quit IRC | 00:01 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/ceilometer: Implement 'reset_on' for mongodb https://review.openstack.org/42355 | 00:02 |
|---|---|---|
| openstackgerrit | Cyril Roelandt proposed a change to openstack/ceilometer: Statistics: Add a "reset_on" query parameter https://review.openstack.org/41899 | 00:02 |
| *** jergerber has joined #openstack-metering | 00:18 | |
| *** alexb_ has joined #openstack-metering | 00:39 | |
| *** dina_belova has joined #openstack-metering | 00:41 | |
| *** dina_belova has quit IRC | 00:46 | |
| *** changbl has joined #openstack-metering | 00:56 | |
| *** gordc has joined #openstack-metering | 01:07 | |
| *** sdake_ has quit IRC | 01:34 | |
| *** dina_belova has joined #openstack-metering | 01:42 | |
| *** alexb_ has quit IRC | 01:43 | |
| *** dina_belova has quit IRC | 01:46 | |
| *** flwang has quit IRC | 01:48 | |
| *** d34dh0r53 has joined #openstack-metering | 01:59 | |
| *** jergerber has quit IRC | 01:59 | |
| *** d34dh0r53 has quit IRC | 02:02 | |
| *** shang has joined #openstack-metering | 02:05 | |
| *** sdake_ has joined #openstack-metering | 02:11 | |
| *** shaneduan is now known as shaneduan[afk] | 02:14 | |
| *** fnaval_ has joined #openstack-metering | 02:14 | |
| *** flwang has joined #openstack-metering | 02:33 | |
| *** gordc has quit IRC | 02:41 | |
| *** dina_belova has joined #openstack-metering | 02:42 | |
| *** shang has quit IRC | 02:44 | |
| *** dina_belova has quit IRC | 02:47 | |
| *** shang has joined #openstack-metering | 02:59 | |
| *** evanjfraser has quit IRC | 03:17 | |
| *** shang has quit IRC | 03:18 | |
| *** d34dh0r53 has joined #openstack-metering | 03:20 | |
| *** d34dh0r53 has quit IRC | 03:22 | |
| *** dina_belova has joined #openstack-metering | 03:43 | |
| *** dina_belova has quit IRC | 03:48 | |
| *** yjiang5 is now known as yjiang5_away | 04:34 | |
| *** shaneduan[afk] is now known as shaneduan | 04:38 | |
| *** boris-42 has joined #openstack-metering | 04:39 | |
| *** dina_belova has joined #openstack-metering | 04:43 | |
| *** dina_belova has quit IRC | 04:47 | |
| *** shaneduan is now known as shaneduan[afk] | 04:57 | |
| *** shaneduan[afk] is now known as shaneduan | 04:59 | |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Extra indexes cleanup https://review.openstack.org/42716 | 05:00 |
| *** SergeyLukjanov has joined #openstack-metering | 05:16 | |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Add test for checking models with migrations https://review.openstack.org/43872 | 05:26 |
| *** dina_belova has joined #openstack-metering | 05:44 | |
| openstackgerrit | Terri Yu proposed a change to openstack/ceilometer: Adds else clause and TODO in storage tests https://review.openstack.org/43993 | 05:48 |
| *** dina_belova has quit IRC | 05:48 | |
| openstackgerrit | Terri Yu proposed a change to openstack/ceilometer: Adds else and TODO in statistics storage tests https://review.openstack.org/43993 | 05:52 |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Add test for checking models with migrations https://review.openstack.org/43872 | 05:53 |
| *** eglynn has joined #openstack-metering | 05:57 | |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name https://review.openstack.org/42715 | 05:59 |
| openstackgerrit | Shuangtai Tian proposed a change to openstack/ceilometer: Handle the metrics sent by nova notifier https://review.openstack.org/42838 | 06:04 |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/43405 | 06:04 |
| *** alexb_ has joined #openstack-metering | 06:14 | |
| *** alexb_ has quit IRC | 06:26 | |
| *** maksimov has quit IRC | 06:38 | |
| *** tasdomas_afk is now known as tasdomas | 06:39 | |
| *** dina_belova has joined #openstack-metering | 06:44 | |
| *** shaneduan is now known as shaneduan[afk] | 06:46 | |
| *** dina_belova has quit IRC | 06:49 | |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Add test for checking models with migrations https://review.openstack.org/43872 | 06:52 |
| *** dina_belova has joined #openstack-metering | 07:01 | |
| *** boris-42 has quit IRC | 07:24 | |
| openstackgerrit | Terri Yu proposed a change to openstack/ceilometer: Adds else and TODO in statistics storage tests https://review.openstack.org/43993 | 07:26 |
| *** yolanda has joined #openstack-metering | 07:27 | |
| *** shardy_afk is now known as shardy | 07:27 | |
| openstackgerrit | Fei Long Wang proposed a change to openstack/ceilometer: Fix empty metadata issue of instance https://review.openstack.org/41818 | 07:39 |
| flwang | jd__: ping | 07:43 |
| jd__ | flwang: pong | 07:49 |
| flwang | jd__: about the metadata missing issue, I'm going to remove the call util.make_sample_from_instance so as to just fake it. please take a look when you have time https://review.openstack.org/#/c/41818 | 07:53 |
| flwang | jd__: thanks | 07:53 |
| jd__ | k | 07:53 |
| *** eglynn has quit IRC | 07:53 | |
| flwang | jd__: is it acceptable to change the migration scripts for each table to add the mysql engine as "InnoDB"? | 08:03 |
| flwang | jd__: since there is a index creating issue | 08:03 |
| jd__ | flwang: I don't know MySQL enough to answer | 08:04 |
| flwang | jd__: let me ask in another way, can I change the migration script for each table? | 08:05 |
| flwang | instead of adding a new script? | 08:06 |
| jd__ | I don't think so | 08:08 |
| flwang | jd__: hmm... then I think it's difficult to resolve | 08:10 |
| flwang | since the db_sync will fail | 08:10 |
| jd__ | flwang: if it's a bug with current migrations scripts, then it's fair to fix the existing | 08:19 |
| jd__ | if they never worked… | 08:19 |
| flwang | jd__: ok, will investigate it more deeper, thanks for the comments | 08:20 |
| *** SergeyLukjanov has quit IRC | 08:24 | |
| openstackgerrit | A change was merged to openstack/ceilometer: install manual last few sections format needs to be fixed https://review.openstack.org/43499 | 08:25 |
| *** infiniti has joined #openstack-metering | 08:33 | |
| *** eglynn has joined #openstack-metering | 08:36 | |
| *** dina_belova has quit IRC | 08:45 | |
| *** lsmola has quit IRC | 08:46 | |
| *** dina_belova has joined #openstack-metering | 08:46 | |
| *** Alexei_987 has joined #openstack-metering | 08:50 | |
| *** dina_belova has quit IRC | 08:50 | |
| *** dina_belova has joined #openstack-metering | 08:53 | |
| flwang | Guangya Suo: around? | 08:57 |
| *** lsmola has joined #openstack-metering | 09:01 | |
| *** SergeyLukjanov has joined #openstack-metering | 09:04 | |
| *** dina_belova has quit IRC | 09:12 | |
| *** dina_belova has joined #openstack-metering | 09:13 | |
| *** dina_belova has quit IRC | 09:15 | |
| *** dina_belova has joined #openstack-metering | 09:15 | |
| *** lsmola has quit IRC | 09:32 | |
| *** dina_belova has quit IRC | 09:36 | |
| *** dina_belova has joined #openstack-metering | 09:36 | |
| *** flwang has quit IRC | 09:36 | |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name https://review.openstack.org/42715 | 09:37 |
| *** boris-42 has joined #openstack-metering | 09:37 | |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Add test for checking models with migrations https://review.openstack.org/43872 | 09:39 |
| sileht | jd__, I have a question, I'm reviewing this: https://review.openstack.org/#/c/42838/9/ceilometer/compute/notifications.py , and the source of the Sample in set, can you confirm that source field of the sample should be always 'openstack' if it come from the os componment ? | 09:39 |
| sileht | s/Sample in set/Sample is set | 09:40 |
| *** dina_belova has quit IRC | 09:40 | |
| jd__ | sileht: yes, this is wrong | 09:41 |
| sileht | jd__, thanks | 09:41 |
| jd__ | -1 it | 09:41 |
| jd__ | :D | 09:41 |
| jd__ | maybe we should remove source from the constructor | 09:42 |
| sileht | jd__, we need it for the post-sample REST API | 09:44 |
| jd__ | fair enough | 09:44 |
| *** lsmola has joined #openstack-metering | 09:46 | |
| *** SergeyLukjanov has quit IRC | 09:54 | |
| *** SergeyLukjanov has joined #openstack-metering | 09:59 | |
| nijaba | sileht: I think the source value to use is defined in the configuration file: sample_source | 10:08 |
| nijaba | (ie it should not be hard coded) | 10:08 |
| sileht | nijaba, yes it is, I'm just checking I have properly understanded the purpose the source field | 10:09 |
| nijaba | sileht: purpose is to be able to specify that the user referential is. You could have some component reporting tickets that are for users defined somewhere else than keystone | 10:10 |
| nijaba | or even imagine that 2 differents openstack deployments report in the same db | 10:11 |
| sileht | nijaba, yes I see | 10:14 |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Fix wrong UniqueConstraint name https://review.openstack.org/42715 | 10:14 |
| openstackgerrit | Fengqian.gao proposed a change to openstack/ceilometer: Add pagination parameter to the database backends of storage https://review.openstack.org/42582 | 10:18 |
| openstackgerrit | Fengqian.gao proposed a change to openstack/ceilometer: Add pagination parameter to the database backends of storage https://review.openstack.org/42582 | 10:23 |
| openstackgerrit | Fengqian.gao proposed a change to openstack/ceilometer: Change pagination query method in mongodb https://review.openstack.org/41869 | 10:23 |
| openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Add test for checking models with migrations https://review.openstack.org/43872 | 10:23 |
| *** chuckieb has quit IRC | 10:41 | |
| *** flwang has joined #openstack-metering | 10:44 | |
| *** dina_belova has joined #openstack-metering | 10:47 | |
| *** chuckieb has joined #openstack-metering | 10:49 | |
| *** dina_belova has quit IRC | 10:51 | |
| *** dina_belova has joined #openstack-metering | 11:04 | |
| sileht | ww/12 | 11:29 |
| sileht | oups | 11:30 |
| *** yolanda has quit IRC | 11:49 | |
| *** zul has quit IRC | 11:55 | |
| *** gordc has joined #openstack-metering | 12:03 | |
| *** yolanda has joined #openstack-metering | 12:03 | |
| *** anteaya has joined #openstack-metering | 12:40 | |
| *** thomasm has joined #openstack-metering | 12:49 | |
| *** dina_belova has quit IRC | 12:50 | |
| *** dina_belova has joined #openstack-metering | 12:51 | |
| thomasm | Hello all! | 12:54 |
| jd__ | boris-42: how's that oslo bug? | 12:55 |
| jd__ | hey thomasm | 12:55 |
| boris-42 | jd__ I send you link to bug fix | 12:55 |
| *** dina_belova has quit IRC | 12:55 | |
| boris-42 | jd__ wait a sec | 12:55 |
| jd__ | boris-42: oh, on IRC? I might have missed it | 12:55 |
| jd__ | add me to the reviewer otherwise | 12:55 |
| *** jergerber has joined #openstack-metering | 12:55 | |
| jd__ | (or I missed that too maybe) | 12:56 |
| *** SergeyLukjanov has quit IRC | 12:56 | |
| boris-42 | jd__ https://review.openstack.org/#/c/43864/ | 12:56 |
| boris-42 | jd__ so there is pretty simple reason why it makes problem | 12:56 |
| jd__ | haha! I was review this actually without knowing it fixes the problem | 12:56 |
| boris-42 | jd__ because if you run only this test you use non patches "migration" which tries to run | 12:57 |
| boris-42 | ALTER in sqlite | 12:57 |
| jd__ | makes sense | 12:57 |
| jd__ | cool thanks | 12:57 |
| *** bpokorny has joined #openstack-metering | 12:57 | |
| jd__ | I was working on getting testr run in // when I stumbled upon this bug | 12:57 |
| boris-42 | instead of "create new table, copy from old to new, drop old, rename new " logic | 12:57 |
| boris-42 | jd__ now it should work=) | 12:58 |
| boris-42 | jd__ it is great that bug is in tests and not in code=) | 12:58 |
| jd__ | clearly | 12:58 |
| *** SergeyLukjanov has joined #openstack-metering | 13:02 | |
| *** dina_belova has joined #openstack-metering | 13:02 | |
| *** fnaval_ has quit IRC | 13:10 | |
| *** zul has joined #openstack-metering | 13:15 | |
| *** sandywalsh has quit IRC | 13:18 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Plug alarm history logic into the API https://review.openstack.org/43849 | 13:20 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Alarm history storage implementation for mongodb https://review.openstack.org/43850 | 13:20 |
| *** shang has joined #openstack-metering | 13:23 | |
| thomasm | jd__, did you get my message regarding the bug fix dependency on HBase storing historical metadata? | 13:30 |
| thomasm | =] | 13:30 |
| jd__ | thomasm: yes but I didn't get into it actually | 13:30 |
| jd__ | do you need me to do something? | 13:30 |
| thomasm | jd__: the DB2 driver also wasn't able to handle out of order messages, so need to figure that piece out too. | 13:30 |
| thomasm | jd__: Nah, I was just polling for concerns if you had any about me making that change too. :) | 13:31 |
| jd__ | definitely no concern | 13:31 |
| jd__ | do the best as you can with these drivers | 13:31 |
| thomasm | Alright, cool. =] | 13:31 |
| *** sandywalsh has joined #openstack-metering | 13:31 | |
| sandywalsh | jd__, (resend from yesterday in case you missed it) ... any thoughts of jumping in on this mid-icehouse meetup movement going on the ML? | 13:42 |
| jd__ | I did miss that | 13:43 |
| *** fnaval_ has joined #openstack-metering | 13:43 | |
| jd__ | I did not really think about it actually | 13:43 |
| sandywalsh | it's nova specific right now, but a CM thing would be great ... especially where the team size has grown | 13:44 |
| sandywalsh | the form that russell made is quite nice | 13:45 |
| jd__ | yup | 13:45 |
| jd__ | well if there's enough people gathering at the same place, e.g. FOSDEM we could do something | 13:46 |
| thomasm | ls | 13:46 |
| thomasm | oops, sorry | 13:46 |
| jd__ | otherwise I don't know, it seems a bit complicated considering the amount of people in Ceilometer and the dispersion | 13:46 |
| *** dina_belova has quit IRC | 13:47 | |
| *** dina_belova has joined #openstack-metering | 13:47 | |
| *** changbl has quit IRC | 13:50 | |
| openstackgerrit | Fei Long Wang proposed a change to openstack/ceilometer: Fix empty metadata issue of instance https://review.openstack.org/41818 | 13:52 |
| *** dina_belova has quit IRC | 13:52 | |
| sandywalsh | jd__, good point. fosdem might be a good idea. | 13:56 |
| sandywalsh | especially given the $$$ for hong kong | 13:56 |
| jd__ | I'm pretty sure that'd work for me nick and eglynn | 13:58 |
| eglynn | yep a meetup at FOSDEM is likely to work for me | 13:59 |
| eglynn | (travel is nice & cheap from Dublin ...) | 14:00 |
| thomasm | Heyyy, I'm going to be in Dublin this December. :P | 14:02 |
| *** annegentle has quit IRC | 14:15 | |
| *** dina_belova has joined #openstack-metering | 14:18 | |
| *** dina_belova has quit IRC | 14:21 | |
| *** dina_belova has joined #openstack-metering | 14:21 | |
| *** annegentle_ has joined #openstack-metering | 14:25 | |
| jd__ | thomasm: well stay a little longer to join FOSDEM then! :) | 14:26 |
| thomasm | jd__: I'll look into that, actually. I would like to go, for sure. =] | 14:29 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Plug alarm history logic into the API https://review.openstack.org/43849 | 14:32 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Alarm history storage implementation for mongodb https://review.openstack.org/43850 | 14:32 |
| *** tasdomas is now known as tasdomas_afk | 14:35 | |
| eglynn | thomasm: going to Dublin, for work or for fun? | 14:36 |
| eglynn | ... or for both? ;) | 14:36 |
| jd__ | eglynn: :-) | 14:36 |
| jd__ | eglynn: so, good holidays despite everything? :) | 14:37 |
| eglynn | jd__: oh yeah, great holiday, just a bit strange being off the grid for so long ... | 14:38 |
| * eglynn missed the constant access to the interwebs ;) | 14:38 | |
| jd__ | hehe | 14:38 |
| thomasm | eglynn: Fun, I've gone to Dublin once a year for the past few years to visit some really good friends of mine and drink entirely too much of my favorite beverage. =] | 14:39 |
| eglynn | thomasm: cool! ... it's everyone's favorite beverage over here ;) | 14:40 |
| jd__ | I hope you're talking about whiskey | 14:42 |
| thomasm | jd__: Haha, it varies from day to day. | 14:43 |
| eglynn | jd__: I was think of a darker brew ;) | 14:44 |
| jd__ | hehe ;) | 14:45 |
| dragondm | Heh, as I recall, last time I was in Ireland, every 4th truck I saw was a Guinness truck :-> Gotta love it. | 14:48 |
| *** changbl has joined #openstack-metering | 14:51 | |
| thomasm | Haha, yep! Good times. | 14:52 |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: add tests for _query_to_kwargs func https://review.openstack.org/43796 | 14:52 |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: alarm: generate alarm_id in API https://review.openstack.org/44065 | 15:02 |
| *** dina_belova has quit IRC | 15:06 | |
| *** dina_belova has joined #openstack-metering | 15:07 | |
| *** maksimov has joined #openstack-metering | 15:07 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/43405 | 15:10 |
| *** dina_belova has quit IRC | 15:11 | |
| sileht | eglynn, ping | 15:12 |
| eglynn | sileht: hey | 15:13 |
| sileht | eglynn, I have answered about your remarks about meta alarm: https://review.openstack.org/#/c/42832/7/ceilometer/alarm/threshold_evaluation.py | 15:13 |
| eglynn | sileht: thanks! | 15:14 |
| * eglynn looking now ... | 15:14 | |
| sileht | eglynn, I was waiting your vacantion return to ask some questions, but it seems you have already catch what I'm looking for | 15:15 |
| eglynn | sileht: cool, so there are two separate issues here possibly? | 15:15 |
| eglynn | 1. whether period/eval_periods should apply for combo alarms | 15:16 |
| eglynn | 2. whether matching_metadata should apply for combo alarms | 15:16 |
| sileht | exactly | 15:16 |
| *** herndon has joined #openstack-metering | 15:16 | |
| eglynn | so for #1 | 15:16 |
| eglynn | my feeling is that the eval window shouldn't apply | 15:17 |
| eglynn | otherwise alarms caught in a certain state for too long as you point out leads to insufficient data | 15:18 |
| sileht | same for me I have tried to find a useful use case but I have failed | 15:18 |
| eglynn | yep, that feels wrong | 15:18 |
| eglynn | for #2, the matching_metadata seems to clash with the idea of specifying the underlying alarm upfront | 15:18 |
| eglynn | i.e. what exactly are we matching on, if we know the alarm IDs already? | 15:19 |
| eglynn | I suppose it could be useful as a way of referring to larger variable-sized groups of alarms | 15:20 |
| eglynn | e.g. all alarms based on a certain instance | 15:20 |
| eglynn | ( ... but that seems like a real stretch, no?) | 15:21 |
| sileht | yes, the evaluator can be extended later for this | 15:23 |
| sileht | eglynn, so you are ok that for now I remove this three fields from the meta alarm evaluator ? | 15:25 |
| eglynn | sileht: yep, that seems reasonable | 15:25 |
| sileht | eglynn, I have seen you use combo-alarm, I agree to say that it sound better than meta-alarm | 15:25 |
| sileht | eglynn, what do you thing if I rename this too ? | 15:26 |
| eglynn | sileht: well both are good for me | 15:27 |
| eglynn | sileht: but yeah, I'd be fine with renaming | 15:28 |
| *** shaneduan[afk] is now known as shaneduan | 15:40 | |
| *** zul has quit IRC | 15:41 | |
| *** changbl has quit IRC | 15:59 | |
| *** changbl has joined #openstack-metering | 16:05 | |
| *** maksimov has quit IRC | 16:08 | |
| *** dina_belova has joined #openstack-metering | 16:17 | |
| *** shaneduan is now known as shaneduan[afk] | 16:18 | |
| *** shang has quit IRC | 16:18 | |
| *** boris-42 has quit IRC | 16:21 | |
| *** dina_belova has quit IRC | 16:22 | |
| *** zul has joined #openstack-metering | 16:28 | |
| *** shaneduan[afk] is now known as shaneduan | 16:31 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: 'and' and 'or' operations for alarms combination https://review.openstack.org/42832 | 16:33 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Add alarm combination API https://review.openstack.org/41971 | 16:33 |
| *** SergeyLukjanov has quit IRC | 16:40 | |
| *** briancline has quit IRC | 16:48 | |
| *** dina_belova has joined #openstack-metering | 16:54 | |
| *** SergeyLukjanov has joined #openstack-metering | 17:05 | |
| *** shaneduan is now known as shaneduan[afk] | 17:07 | |
| *** Alexei_987 has quit IRC | 17:23 | |
| *** boris-42 has joined #openstack-metering | 17:42 | |
| *** yjiang5_away is now known as yjiang5 | 17:47 | |
| *** shaneduan[afk] is now known as shaneduan | 18:06 | |
| *** herndon has quit IRC | 18:08 | |
| *** shaneduan is now known as shaneduan[afk] | 18:08 | |
| *** herndon has joined #openstack-metering | 18:12 | |
| nealph | jd__: I'm chasing down a mongodb write error from the collector....do you know if there's any way to get a more verbose error response from the db write action? | 18:26 |
| *** herndon has quit IRC | 18:26 | |
| jd__ | nealph: do you get an exception? | 18:27 |
| nealph | yes, "not okForStorage", which is a mongo error I think. | 18:27 |
| nealph | (not really an exception I guess) | 18:27 |
| *** tasdomas_afk is now known as tasdomas | 18:29 | |
| *** tasdomas is now known as tasdomas_afk | 18:29 | |
| nealph | (more of a bubbled-up error) | 18:29 |
| *** shaneduan[afk] is now known as shaneduan | 18:33 | |
| nealph | I suspect the issue is one of our field names that contains a dot | 18:35 |
| *** yolanda has quit IRC | 18:40 | |
| *** openstack has joined #openstack-metering | 18:46 | |
| thomasm | gordc: Got a moment? | 18:51 |
| *** yolanda has joined #openstack-metering | 18:58 | |
| *** SergeyLukjanov has quit IRC | 18:58 | |
| *** SergeyLukjanov has joined #openstack-metering | 18:59 | |
| gordc | thomasm: sorry about that. i heard my computer make a sound... didn't realize it was my irc client warning me. | 19:06 |
| gordc | thomasm: whatsup? | 19:06 |
| thomasm | gordc, Haha no worries. So, I have some tests I just wrote to find all of the drivers get_resources methods that don't support ordering of samples (since they can be out of order when stored) and I've fixed up SQLAlchemy, MongoDB, and HBase to order on their Meter collections/tables for this instead to accomplish the more advanced filtering. With that, it looks like there's some spotty support for DB2 in PyMongo, and I was | 19:08 |
| thomasm | wondering if you had any suggestions, thoughts, etc. on the quirks there? | 19:08 |
| thomasm | Or if I ought to just tell those tests to run for SQLAlchemy, MongoDB, and HBase. | 19:09 |
| openstackgerrit | Terri Yu proposed a change to openstack/ceilometer: Add group by statistics tests in API v2 tests https://review.openstack.org/44130 | 19:09 |
| gordc | thomasm: the spotty support does not surprise me.lol | 19:09 |
| thomasm | Lol | 19:09 |
| gordc | thomasm: i've actually never played with the nosql implementation of db2, litong has been working with the db2 team to support that | 19:10 |
| gordc | do you have enough error information to debug it? if so, you can open a bug and i can tell the db2 team to look at it. | 19:10 |
| thomasm | I can get it relatively easily, yep. | 19:11 |
| gordc | thomasm: i think it'd be best to skip over db2 for now. unless you have knowledge of how the nosql impl works. | 19:11 |
| thomasm | gordc, not a clue | 19:11 |
| thomasm | =] | 19:11 |
| thomasm | gordc, I honestly have never worked with any of these frameworks/driver types before. | 19:11 |
| thomasm | gordc, the ones I did fix I did so from reading LOTS of documentation, hehe. | 19:11 |
| thomasm | gordc, I'll toss up the test cases that are failing to show the deficiencies. :) Thanks for the advice. | 19:12 |
| gordc | that's what i do everyday :) once google goes down, i'm pretty much ineffective. | 19:12 |
| thomasm | Hahahaha, right? | 19:12 |
| gordc | thomasm: cool. send me a link to the defects as well. i'll make sure the right ppl look at it. | 19:13 |
| thomasm | gordc, You got it. | 19:14 |
| gordc | thomasm: thanks very much. | 19:14 |
| thomasm | gordc, thank you as well. =] | 19:14 |
| *** eglynn has quit IRC | 19:15 | |
| *** evanjfraser has joined #openstack-metering | 19:34 | |
| *** herndon has joined #openstack-metering | 19:38 | |
| thomasm | gordc, https://bugs.launchpad.net/ceilometer/+bug/1218047 | 19:41 |
| gordc | thomasm: awesome | 19:43 |
| *** maksimov has joined #openstack-metering | 19:43 | |
| *** zul has quit IRC | 19:49 | |
| *** litong has joined #openstack-metering | 19:53 | |
| *** dina_belova has quit IRC | 20:00 | |
| *** dina_belova has joined #openstack-metering | 20:01 | |
| *** SergeyLukjanov has quit IRC | 20:05 | |
| *** dina_belova has quit IRC | 20:05 | |
| *** gordc has quit IRC | 20:07 | |
| annegentle_ | I wonder how hard this is, to get full responses from plugins? https://bugs.launchpad.net/openstack-manuals/+bug/1216716 | 20:08 |
| annegentle_ | is it a matter of code and scrape? | 20:08 |
| openstackgerrit | Terri Yu proposed a change to openstack/ceilometer: Adds group by statistics for MongoDB driver https://review.openstack.org/43043 | 20:13 |
| *** evanjfraser_ has joined #openstack-metering | 20:13 | |
| *** evanjfraser has quit IRC | 20:15 | |
| *** nealph has quit IRC | 20:16 | |
| *** eglynn has joined #openstack-metering | 20:34 | |
| *** d34dh0r53 has joined #openstack-metering | 20:37 | |
| *** gordc has joined #openstack-metering | 20:37 | |
| *** gordc has quit IRC | 20:38 | |
| *** gordc has joined #openstack-metering | 20:38 | |
| *** d34dh0r53 has quit IRC | 20:39 | |
| *** yolanda has quit IRC | 20:41 | |
| *** boris-42 has quit IRC | 20:51 | |
| gordc | jd__: i just realize what time it was for you guys, i was thinking of moving my meeting item to mailinglist (might be better suited for that anyways). any objection? | 20:52 |
| *** sandywalsh has quit IRC | 20:55 | |
| *** sandywalsh has joined #openstack-metering | 20:59 | |
| jd__ | gordc: nop | 21:00 |
| gordc | jd__, mailinglist it is then :) | 21:01 |
| *** dina_belova has joined #openstack-metering | 21:01 | |
| *** chuckieb has quit IRC | 21:03 | |
| *** dina_belova has quit IRC | 21:05 | |
| *** boris-42 has joined #openstack-metering | 21:08 | |
| *** dina_belova has joined #openstack-metering | 21:11 | |
| *** dina_belova has quit IRC | 21:15 | |
| *** boris-42 has quit IRC | 21:17 | |
| *** gordc has quit IRC | 21:28 | |
| *** maksimov has quit IRC | 21:33 | |
| openstackgerrit | Jay Pipes proposed a change to openstack/ceilometer: Add a specialized Event Type model and db table https://review.openstack.org/41316 | 21:51 |
| *** nealph has joined #openstack-metering | 22:00 | |
| *** litong has quit IRC | 22:00 | |
| *** dhellmann is now known as dhellmann_ | 22:00 | |
| *** thomasm has quit IRC | 22:02 | |
| *** bpokorny has quit IRC | 22:02 | |
| *** thomasm has joined #openstack-metering | 22:02 | |
| *** thomasm has quit IRC | 22:03 | |
| *** dina_belova has joined #openstack-metering | 22:12 | |
| *** dina_belova has quit IRC | 22:16 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: alarm: generate alarm_id in API https://review.openstack.org/44065 | 22:24 |
| *** changbl has quit IRC | 22:26 | |
| *** eglynn has quit IRC | 22:51 | |
| *** herndon has quit IRC | 22:59 | |
| *** dina_belova has joined #openstack-metering | 23:12 | |
| *** dina_belova has quit IRC | 23:16 | |
| *** jergerber has quit IRC | 23:24 | |
| *** zul has joined #openstack-metering | 23:33 | |
| *** changbl has joined #openstack-metering | 23:38 | |
| *** fnaval_ has quit IRC | 23:46 | |
| *** herndon has joined #openstack-metering | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!