| *** nati_ueno has joined #openstack-ceilometer | 00:14 | |
| *** nati_ueno has quit IRC | 00:40 | |
| *** shakamunyi has joined #openstack-ceilometer | 00:41 | |
| *** nati_ueno has joined #openstack-ceilometer | 00:41 | |
| *** shakamunyi has quit IRC | 00:50 | |
| *** nati_ueno has quit IRC | 00:51 | |
| *** liusheng has quit IRC | 00:54 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:55 | |
| openstackgerrit | liusheng proposed a change to openstack/python-ceilometerclient: Reduce redundant parameter of some commands in CLI https://review.openstack.org/66776 | 00:58 |
|---|---|---|
| *** nosnos has joined #openstack-ceilometer | 00:59 | |
| *** liusheng has joined #openstack-ceilometer | 01:00 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Fix the floatingip pollster https://review.openstack.org/83676 | 01:03 |
| *** shakamunyi has joined #openstack-ceilometer | 01:17 | |
| *** raymondr has joined #openstack-ceilometer | 01:21 | |
| *** shakamunyi has quit IRC | 01:26 | |
| *** raymondr has quit IRC | 01:41 | |
| *** llu has joined #openstack-ceilometer | 01:42 | |
| *** shakamunyi has joined #openstack-ceilometer | 01:53 | |
| *** shakamunyi has quit IRC | 01:57 | |
| *** shakamunyi has joined #openstack-ceilometer | 01:58 | |
| *** shakamunyi has quit IRC | 02:05 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Improves the query_to_kwargs method https://review.openstack.org/92788 | 02:06 |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Improves the query_to_kwargs method https://review.openstack.org/92066 | 02:06 |
| *** raymondr_ has joined #openstack-ceilometer | 02:07 | |
| *** raymondr_ has quit IRC | 02:26 | |
| *** shakamunyi has joined #openstack-ceilometer | 02:30 | |
| *** nati_ueno has joined #openstack-ceilometer | 02:33 | |
| *** shakamunyi has quit IRC | 02:40 | |
| *** _cjones_ has joined #openstack-ceilometer | 02:43 | |
| *** nati_ueno has quit IRC | 02:43 | |
| *** _cjones_ has quit IRC | 02:48 | |
| *** zqfan has joined #openstack-ceilometer | 03:00 | |
| *** shakamunyi has joined #openstack-ceilometer | 03:07 | |
| *** shakamunyi has quit IRC | 03:17 | |
| *** matsuhashi has quit IRC | 03:19 | |
| *** nosnos has quit IRC | 03:33 | |
| *** shakamunyi has joined #openstack-ceilometer | 03:43 | |
| *** nati_ueno has joined #openstack-ceilometer | 03:47 | |
| openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Evaluate alarms by their dependency order https://review.openstack.org/89330 | 03:50 |
| *** shakamunyi has quit IRC | 03:53 | |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/ceilometer: Add the function of deleting alarm history https://review.openstack.org/87869 | 04:13 |
| *** nati_ueno has quit IRC | 04:14 | |
| *** nati_ueno has joined #openstack-ceilometer | 04:14 | |
| *** shakamunyi has joined #openstack-ceilometer | 04:20 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:21 | |
| *** nosnos has joined #openstack-ceilometer | 04:21 | |
| *** zqfan has quit IRC | 04:22 | |
| *** shakamunyi has quit IRC | 04:29 | |
| *** _cjones_ has joined #openstack-ceilometer | 04:45 | |
| *** nati_ueno has quit IRC | 04:45 | |
| *** _cjones_ has quit IRC | 04:50 | |
| *** shakamunyi has joined #openstack-ceilometer | 04:55 | |
| *** nati_ueno has joined #openstack-ceilometer | 05:03 | |
| *** shakamunyi has quit IRC | 05:05 | |
| *** raymondr has joined #openstack-ceilometer | 05:17 | |
| *** nati_ueno has quit IRC | 05:21 | |
| *** nati_uen_ has joined #openstack-ceilometer | 05:25 | |
| *** nati_ue__ has joined #openstack-ceilometer | 05:28 | |
| *** nati_ueno has joined #openstack-ceilometer | 05:30 | |
| *** nati_uen_ has quit IRC | 05:30 | |
| *** shakamunyi has joined #openstack-ceilometer | 05:32 | |
| *** nati_ue__ has quit IRC | 05:33 | |
| *** raymondr has quit IRC | 05:35 | |
| *** shakamunyi_ has joined #openstack-ceilometer | 05:37 | |
| *** shakamunyi has quit IRC | 05:39 | |
| *** shakamunyi_ has quit IRC | 05:39 | |
| *** alexizz has joined #openstack-ceilometer | 05:40 | |
| *** nati_ueno has quit IRC | 05:42 | |
| *** nati_ueno has joined #openstack-ceilometer | 05:43 | |
| *** nati_uen_ has joined #openstack-ceilometer | 05:46 | |
| *** nati_ueno has quit IRC | 05:47 | |
| *** nati_ueno has joined #openstack-ceilometer | 05:48 | |
| *** nati_uen_ has quit IRC | 05:50 | |
| *** nati_uen_ has joined #openstack-ceilometer | 05:53 | |
| *** nati_ueno has quit IRC | 05:53 | |
| *** nati_uen_ has quit IRC | 05:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:01 |
| *** nati_ueno has joined #openstack-ceilometer | 06:06 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:10 | |
| *** psharma has joined #openstack-ceilometer | 06:17 | |
| *** _nadya_ has quit IRC | 06:19 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/ceilometer: remove unneeded definitions of Python Source Code Encoding https://review.openstack.org/95388 | 06:20 |
| *** _nadya_ has joined #openstack-ceilometer | 06:22 | |
| *** nati_ueno has quit IRC | 06:27 | |
| *** mengxd has joined #openstack-ceilometer | 06:36 | |
| *** zqfan has joined #openstack-ceilometer | 06:40 | |
| *** _nadya_ has quit IRC | 06:44 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:47 | |
| *** nati_ueno has joined #openstack-ceilometer | 06:48 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer-specs: Specs for alarm-evaluator-storage-access https://review.openstack.org/95418 | 06:52 |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Support direct alarm_evaluator storage access https://review.openstack.org/89756 | 06:59 |
| *** ildikov has quit IRC | 07:03 | |
| *** ildikov_ has quit IRC | 07:05 | |
| *** nati_ueno has quit IRC | 07:11 | |
| *** nati_uen_ has joined #openstack-ceilometer | 07:12 | |
| *** nati_uen_ has quit IRC | 07:16 | |
| *** zul has quit IRC | 07:26 | |
| *** _nadya_ has quit IRC | 07:26 | |
| *** zul has joined #openstack-ceilometer | 07:41 | |
| *** ruhe has quit IRC | 07:49 | |
| *** DinaBelova has quit IRC | 07:49 | |
| *** underyx has quit IRC | 07:51 | |
| *** nacim has joined #openstack-ceilometer | 07:54 | |
| *** underyx has joined #openstack-ceilometer | 07:56 | |
| *** DinaBelova has joined #openstack-ceilometer | 07:56 | |
| *** ruhe has joined #openstack-ceilometer | 07:56 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Support direct alarm_evaluator storage access https://review.openstack.org/89756 | 08:03 |
| *** pnascimento has joined #openstack-ceilometer | 08:27 | |
| *** Infitialis has joined #openstack-ceilometer | 08:29 | |
| Infitialis | I'm trying to set up compute host meters with "compute_monitors = ComputeDriverCPUMonitor" in nova.conf but it doesn | 08:30 |
| Infitialis | ... show it | 08:30 |
| *** lsmola has joined #openstack-ceilometer | 08:39 | |
| *** _nadya_ has joined #openstack-ceilometer | 08:44 | |
| *** safchain has joined #openstack-ceilometer | 08:45 | |
| *** alexizz has quit IRC | 09:05 | |
| *** eglynn has joined #openstack-ceilometer | 09:09 | |
| *** idegtiarov has joined #openstack-ceilometer | 09:11 | |
| *** matsuhashi has quit IRC | 09:13 | |
| *** ildikov has joined #openstack-ceilometer | 09:17 | |
| *** ildikov is now known as ildikov_cee | 09:17 | |
| *** IvanBerezovskiy has joined #openstack-ceilometer | 09:19 | |
| *** matsuhashi has joined #openstack-ceilometer | 09:20 | |
| *** _nadya_ has quit IRC | 09:23 | |
| *** matsuhashi has quit IRC | 09:33 | |
| *** matsuhas_ has joined #openstack-ceilometer | 09:38 | |
| *** mengxd has quit IRC | 09:40 | |
| eglynn | jd__: nice one, the gnocchi has landed ;) ... https://github.com/stackforge/gnocchi | 09:41 |
| eglynn | \o/ :) | 09:41 |
| eglynn | jd__: ... BTW do you normally run the gnocchi-api svc alongside a full devstack, for the swift aspect? | 09:41 |
| eglynn | jd__: ... or just run gnocchi-api against a stand-alone swift install? | 09:42 |
| eglynn | jd__: (... /me just asked with the intention of giving amalagon some pointers) | 09:42 |
| pnascimento | hello stackers | 09:47 |
| eglynn | pnascimento: good morning | 09:47 |
| jd__ | eglynn: I've done both actually | 09:48 |
| jd__ | both work fine | 09:48 |
| pnascimento | I'm on a quest to develop my own ceilometer agent, I think I have finished that part, so now I would like to develop a Unit or a Set of Unit Tests for that agent, seeing the tests/ subpackage I have realised that is availble a test for an agent, for the pollsters, and the list goes on... | 09:49 |
| pnascimento | my question is, to be a well performed test, do I need this structure? | 09:49 |
| pnascimento | if yes, which is the best structure for a agent test suite? | 09:49 |
| eglynn | jd__: ... aha cool, thanks | 09:50 |
| eglynn | jd__: (... /me was just wondering whether there were any dark arts or secret incantations required to get it up & running, seems like not) | 09:50 |
| *** igor_ has joined #openstack-ceilometer | 09:50 | |
| eglynn | jd__: ... do you normally work on precise or trusty or something else? | 09:50 |
| eglynn | jd__: (might as well line anamalagon up with that to simplify dependencies etc.) | 09:51 |
| eglynn | pnascimento: have you looked at https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/agentbase.py ? | 09:52 |
| jd__ | eglynn: OS X :] | 09:52 |
| jd__ | eglynn: I barely use devstack | 09:53 |
| eglynn | pnascimento: ... that mainly asserts the agent's interaction with the pipeline config | 09:53 |
| eglynn | wow! jd__ is an outlier! ;) | 09:53 |
| jd__ | hehe | 09:53 |
| eglynn | pnascimento: ... that base test class is then extended for various concrete agents, e.g. compute: https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/compute/test_manager.py | 09:55 |
| pnascimento | eglynn: I'm looking to ceilometer/tests/compute/pollsters/{base.py, test_cpu.py} and to ceilometer/tests/central/test_manager.py | 09:55 |
| eglynn | pnascimento: the latter test you mention ceilometer/tests/central/test_manager.py extends the superclass I referenced above (i.e. agentbase.BaseAgentManagerTestCase) | 09:56 |
| eglynn | pnascimento: ... whereas the logic in ceilometer/tests/compute/pollsters/test_cpu.py etc. is intened to just exercise the individual pollster code | 09:56 |
| eglynn | pnascimento: ... you'll need both | 09:57 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Allow to have different DB for alarm and metering https://review.openstack.org/94829 | 09:57 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Allow to have different DB for alarm and metering https://review.openstack.org/94829 | 10:00 |
| pnascimento | eglynn: ok, I will look into that specific class and into mock, but firstly should I develop a test as-I-need-to (e.g.: just for one of my pollster)? Or it is a best practice to create, for the manager, pollsters, etc? That is was too, one of my doubts... | 10:00 |
| *** shakamunyi has joined #openstack-ceilometer | 10:01 | |
| eglynn | pnascimento: your initial patch should include both | 10:01 |
| eglynn | pnascimento: ... i.e. both tests for the new agent code and the new pollsters | 10:01 |
| pnascimento | eglynn: ok :) after, how can I execute my tests? There anything like Maven or Grunt? | 10:03 |
| eglynn | pnascimento: to run all tests: tox -epy27,pep8 | 10:03 |
| eglynn | pnascimento: you may need to install tox first if not already there | 10:04 |
| eglynn | pnascimento: ... IIRC the version of tox has gotta be <1.7.0 | 10:04 |
| eglynn | pnascimento: e.g.: $ sudo pip install --upgrade "tox>=1.6,<1.7" | 10:05 |
| eglynn | pnascimento: ... to run an individual test module: $ tox -epy27 package.subpackage.module | 10:06 |
| eglynn | pnascimento: ... wow, maven ... that's a blast from the past for me | 10:06 |
| pnascimento | eglynn: lol I have seen that command but I thought that would be used just for analyse Python syntax (against PEP8 rules) in whole project :P | 10:06 |
| eglynn | pnascimento: ... you've a Java background, or? | 10:07 |
| pnascimento | pnascimento: yes :P | 10:07 |
| * eglynn hasn't used maven since the, cough, cough, good old days of Apache CFX & JBoss | 10:07 | |
| eglynn | ... ah, the memories ;) | 10:07 |
| eglynn | pnascimento: the pep8 analysis is relatively quick, not a big deal to do it across the entire ceilometer code-tree | 10:08 |
| eglynn | pnascimento: what's slow is downing the dependencies on the first run | 10:08 |
| eglynn | s/downing/downloading/ | 10:09 |
| pnascimento | eglynn: as I have heard recently you are better now with Python! :P | 10:09 |
| pnascimento | eglynn: but thank you for your guidance :) | 10:10 |
| eglynn | pnascimento: I have learned the error of my ways, and been cured of my non-dynamic-language heresy ;) | 10:10 |
| *** shakamunyi has quit IRC | 10:15 | |
| *** denis_makogon has joined #openstack-ceilometer | 10:22 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/ceilometer: replaced unicode() with six.text_type() https://review.openstack.org/95488 | 10:24 |
| *** mengxd has joined #openstack-ceilometer | 10:29 | |
| *** mengxd has quit IRC | 10:30 | |
| *** shakamunyi has joined #openstack-ceilometer | 10:34 | |
| eglynn | sileht: just noticed that https://blueprints.launchpad.net/ceilometer/+spec/switch-to-oslo.messaging has undefined priority | 10:52 |
| openstackgerrit | ZhiQiang Fan proposed a change to openstack/python-ceilometerclient: Avoid empty entity field in uri path https://review.openstack.org/91301 | 10:52 |
| eglynn | sileht: (prolly just a side-effect of it being bumped from icehouse) | 10:52 |
| eglynn | sileht: ... would you agree with it being "Essential" | 10:53 |
| eglynn | (i.e. the highest priority)? | 10:53 |
| eglynn | (... not that it makes a great deal of difference now that it's already landed) | 10:53 |
| *** zqfan has quit IRC | 10:54 | |
| *** pnascimento has quit IRC | 10:54 | |
| eglynn | idegtiarov: good afternoon! | 11:05 |
| eglynn | idegtiarov: are you planning to propose a detailed spec for https://blueprints.launchpad.net/ceilometer/+spec/hbase-events-structure ? | 11:05 |
| eglynn | idegtiarov: ... is juno-1 milestone still realistic for this BP to be implemented? | 11:06 |
| eglynn | idegtiarov: ... the process of proposing a spec is described on the wiki https://wiki.openstack.org/wiki/Blueprints#Ceilometer | 11:07 |
| DinaBelova | eglynn, o/ idegtiarov seems to be not here, I'll try to catch him)) | 11:08 |
| DinaBelova | eglynn, uno momento) | 11:08 |
| eglynn | DinaBelova: grazie! :) | 11:08 |
| *** nacim has quit IRC | 11:11 | |
| *** boris-42 has quit IRC | 11:13 | |
| eglynn | liusheng: are you planning to propose a detailed spec for https://blueprints.launchpad.net/ceilometer/+spec/support-novaclient-v3 ? | 11:16 |
| *** julim has joined #openstack-ceilometer | 11:17 | |
| *** nacim has joined #openstack-ceilometer | 11:17 | |
| liusheng | eglynn: Hi eglynn, yes I have registerd this bp, but I am not sure ceilometer should surpport v3 client in juno:) | 11:20 |
| eglynn | liusheng: that's fine, I'll just untarget it from juno-1 in that case | 11:21 |
| *** julim has quit IRC | 11:21 | |
| eglynn | liusheng: ... ok that BP is now untargeted | 11:22 |
| *** boris-42 has joined #openstack-ceilometer | 11:23 | |
| idegtiarov | eglynn: good afternoon! I am going to propose detailed spec for events into HBase, and have started bp https://blueprints.launchpad.net/ceilometer/+spec/hbase-events-feature and spec for events are proposed in frames of this bp. | 11:23 |
| eglynn | idegtiarov: excellent! ... please add me as a reviewer on the spec once you've proposed it to gerrot | 11:24 |
| *** guest_____ has joined #openstack-ceilometer | 11:24 | |
| eglynn | s/gerrot/gerrit/ | 11:24 |
| eglynn | idegtiarov: thanks! | 11:24 |
| liusheng | eglynn: OK I will write a specs for that bp :-) | 11:24 |
| idegtiarov | eglynn: ok! :) | 11:25 |
| *** guest_____ has left #openstack-ceilometer | 11:27 | |
| eglynn | idegtiarov: ... does juno-1 still seem like a reasonable target for completion for hbase events? | 11:27 |
| eglynn | idegtiarov: (... realistically, you'd need to have patches landed by June 6th in order to make a milestone cut the following week) | 11:27 |
| eglynn | (or if not landed by the 6th, at least with reviews well progressed at that stage, capable of being landed very early on the milestone week) | 11:29 |
| eglynn | (juno-1 will be cut on Thurs Jun 12th IIRC) | 11:29 |
| idegtiarov | eglynn: actually I hope to finish with this patch in a week | 11:32 |
| openstackgerrit | Mitsuru Kanabuchi proposed a change to openstack/ceilometer: Add the function of deleting alarm history https://review.openstack.org/87869 | 11:33 |
| eglynn | idegtiarov: excellent, thank you sir! | 11:37 |
| *** _nadya_ has joined #openstack-ceilometer | 11:44 | |
| sileht | eglynn, qck | 11:45 |
| eglynn | sileht: cool | 11:45 |
| *** nosnos has quit IRC | 12:00 | |
| *** matsuhas_ has quit IRC | 12:08 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/ceilometer: replaced unicode() with six.text_type() https://review.openstack.org/95488 | 12:13 |
| *** inc0 has joined #openstack-ceilometer | 12:13 | |
| inc0 | good afternoon everyone. I have quick question. How do you handle large samble tables in sqlalchemy impl? I have over 60m records and well, I'd like to shrink it, can you configure ceilo to skip some measurments? | 12:14 |
| inc0 | or well, how do you handle this? | 12:15 |
| *** pnascimento has joined #openstack-ceilometer | 12:15 | |
| sileht | inc0, you can choose what meter you want to store in the pipeline configuration (usualy /etc/ceilometer/pipeline.yml) | 12:23 |
| sileht | inc0, this must be done on ceilometer-*-agent nodes | 12:24 |
| eglynn | inc0: use https://blueprints.launchpad.net/ceilometer/+spec/db-ttl | 12:24 |
| eglynn | inc0: ... a-ha, maybe I misunderstood the question, in the light of sileht's reply | 12:25 |
| eglynn | inc0: ... do you wanna expire existing metering data, or ensure less data is collected going forward? | 12:25 |
| inc0 | eglynn, well, I think I'll do both. | 12:25 |
| eglynn | inc0: cool enough | 12:26 |
| inc0 | however I saw bp regarding aggregating records | 12:26 |
| inc0 | that would be better in the longer run, I'll check it out when I'll have time | 12:26 |
| inc0 | thank you guys | 12:27 |
| eglynn | inc0: well we spoke about aggregation/roll-up at the icehouse summit, but it didn't get implemented | 12:27 |
| ildikov_cee | inc0: here is some doc for the pipeline config: http://docs.openstack.org/developer/ceilometer/configuration.html#pipelines | 12:27 |
| eglynn | inc0: ... the bringht new hope though is TSDaaS/gnocchi for Juno | 12:27 |
| *** matsuhashi has joined #openstack-ceilometer | 12:28 | |
| eglynn | inc0: s/bringht/bright/ ... https://github.com/openstack/ceilometer-specs/blob/master/specs/gnocchi.rst | 12:28 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer-specs: Move TSDaaS/gnoochi spec under juno https://review.openstack.org/95513 | 12:31 |
| inc0 | hmm looks good, hopefully I'll get some time to work on it | 12:32 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer-specs: Move TSDaaS/gnoochi spec under juno https://review.openstack.org/95513 | 12:32 |
| *** ildikov has joined #openstack-ceilometer | 12:38 | |
| openstackgerrit | A change was merged to openstack/ceilometer-specs: Move TSDaaS/gnoochi spec under juno https://review.openstack.org/95513 | 12:39 |
| *** ildikov_cee has quit IRC | 12:40 | |
| *** amalagon has joined #openstack-ceilometer | 12:41 | |
| *** ityaptin has joined #openstack-ceilometer | 12:41 | |
| *** ildikov_ has joined #openstack-ceilometer | 12:43 | |
| *** ildikov has quit IRC | 12:45 | |
| *** ildikov has joined #openstack-ceilometer | 12:47 | |
| *** ildikov is now known as ildikov_cee | 12:47 | |
| *** promulo has quit IRC | 12:47 | |
| *** ildikov_ has quit IRC | 12:47 | |
| *** denis_makogon has left #openstack-ceilometer | 12:51 | |
| *** aviau has joined #openstack-ceilometer | 12:59 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: [DO NOT MERGE YET] Migration to oslo.db https://review.openstack.org/93442 | 13:04 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: [DO NOT MERGE YET] Migration to oslo.db https://review.openstack.org/93442 | 13:06 |
| *** matsuhashi has quit IRC | 13:06 | |
| *** matsuhashi has joined #openstack-ceilometer | 13:06 | |
| *** inc0 has quit IRC | 13:07 | |
| *** inc0 has joined #openstack-ceilometer | 13:07 | |
| *** shakamunyi has quit IRC | 13:07 | |
| eglynn | sileht: I guess you could just -2 https://review.openstack.org/93442 yourself to ensure it's not merged | 13:10 |
| *** matsuhas_ has joined #openstack-ceilometer | 13:10 | |
| eglynn | sileht: (... i.e. instead of the note in the commit msg subject line) | 13:10 |
| sileht | eglynn, sure | 13:10 |
| *** matsuhashi has quit IRC | 13:10 | |
| * eglynn is getting infected with the dreaded process-nitpick-syndrome ... ;) | 13:11 | |
| *** cmart has joined #openstack-ceilometer | 13:20 | |
| cmart | hello | 13:24 |
| eglynn | cmart: hola | 13:27 |
| cmart | eglynn: hi! | 13:27 |
| cmart | I have a couple of questions regarding this bug: https://bugs.launchpad.net/ceilometer/+bug/1224298 | 13:28 |
| cmart | I don't know if you remember that we talked last week about this.. | 13:28 |
| eglynn | cmart: we decided that *non-complex* attrs be filterable, right? | 13:28 |
| cmart | right | 13:29 |
| cmart | I look and analyze the code and I'm stucked with the "meter" field.. | 13:29 |
| eglynn | cmart: stuck in what sense? | 13:30 |
| cmart | stuck in the way that I don't know which is the best way to implement the filtering by meter... I saw that the alarm model does not have a meter column. Instead, it has a rule column with information about the meter, threshold, etc.. | 13:30 |
| eglynn | amalagon: o/ ... ok to delay 1:1 today by 5 mins? | 13:31 |
| amalagon | sure ~ | 13:31 |
| *** psharma has quit IRC | 13:31 | |
| *** ildikov_cee has quit IRC | 13:32 | |
| cmart | in the mongo implementation, it shouldn't be a problem, but what about the sqlalchemy impl? | 13:32 |
| eglynn | cmart: in the "olden days", it did actually have a top-level field equivalent to "meter name" | 13:33 |
| eglynn | cmart: https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/sqlalchemy/migrate_repo/versions/007_add_alarm_table.py#L34 | 13:33 |
| eglynn | cmart: but when we added the concept of combination alarms | 13:33 |
| eglynn | cmart: we gnericized the alarm representation so that it wasn't so threshold oriented | 13:33 |
| cmart | exactly.. | 13:33 |
| eglynn | cmart: ... so the counter_name got pushed into the threshold rule | 13:34 |
| *** shakamunyi has joined #openstack-ceilometer | 13:34 | |
| eglynn | cmart: ... so now that the rule looks like https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/sqlalchemy/migrate_repo/versions/016_simpler_alarm.py#L44 | 13:35 |
| eglynn | cmart: ... i.e. persisted as text in the sql-a case | 13:35 |
| eglynn | cmart: ... I guess we'd have to do a "like" query? | 13:35 |
| *** raymondr has joined #openstack-ceilometer | 13:36 | |
| eglynn | cmart: ... i.e. the sqla like operator ... http://docs.sqlalchemy.org/en/rel_0_7/core/expression_api.html | 13:36 |
| eglynn | cmart: ... performance ain't gonna be pretty though :( | 13:37 |
| cmart | that was my same thought :) | 13:38 |
| eglynn | cmart: ... gotta run, could chat more on this laters | 13:38 |
| cmart | OK | 13:38 |
| cmart | thanks for the help.. ttyl | 13:38 |
| *** Alexei_9871 has joined #openstack-ceilometer | 13:44 | |
| *** shakamunyi has quit IRC | 13:47 | |
| _nadya_ | hi folks! | 13:51 |
| _nadya_ | I have a question about block in https://etherpad.openstack.org/p/ceilometer-schema . Maybe you have discussed that on summit | 13:52 |
| _nadya_ | eglynn, sileht, Alexei_987, DinaBelova, jd__ ^^ | 13:53 |
| eglynn | _nadya_: thanks! (... in a meeting now but I'll have a look at your note in the etherpad once that ends) | 13:55 |
| *** gordc has joined #openstack-ceilometer | 14:00 | |
| sileht | _nadya_, I have added two reviews that recently changes the sqlalchemy schema | 14:03 |
| *** shakamunyi has joined #openstack-ceilometer | 14:04 | |
| _nadya_ | sileht: ok, looking. but it's not only sql I guess | 14:05 |
| sileht | _nadya_, user/project have been removed everywhere and sqlalchemy have drop source/sourceassoc too | 14:05 |
| _nadya_ | sileht: aha, but resource still may have several sources, right? | 14:06 |
| sileht | _nadya_, previously sqlalchemy allows to retreive/query only the first source, other sources was not usable | 14:07 |
| *** gordc1 has joined #openstack-ceilometer | 14:08 | |
| *** gordc has quit IRC | 14:10 | |
| _nadya_ | sileht: and here https://review.openstack.org/#/c/94483/8/ceilometer/storage/sqlalchemy/models.py you store only one source now, as I see. So every time source is rewritten, right? | 14:10 |
| _nadya_ | sileht: ah, it's for Samples. Resource table is dropped for sql | 14:11 |
| _nadya_ | sileht: I mean your change | 14:11 |
| sileht | _nadya_, in sample table you have the source record for each sample, but when we build the resource only the latest one is used | 14:12 |
| _nadya_ | sileht: I always thought that Resource may have several sources. And the behaviour you described above "only latest is used" looks wrong to me | 14:16 |
| *** erecio has joined #openstack-ceilometer | 14:18 | |
| sileht | _nadya_, mongo driver already do that | 14:19 |
| _nadya_ | sileht: yep, I know, and in HBase I have to impl that. but it looks not correct and that's why I was asking about it in etherpad :) | 14:20 |
| *** pnascimento_ has joined #openstack-ceilometer | 14:22 | |
| *** pnascimento has quit IRC | 14:22 | |
| sileht | _nadya_, but a resource with different source don't really have a sense (for me at least) | 14:22 |
| sileht | _nadya_, I agree that the term 'source' is not well defined | 14:23 |
| _nadya_ | sileht: it's really hard to imagine, I agree. And it's far from real life :) but we have so many tests with 'multiple sources' | 14:24 |
| *** fc__ has quit IRC | 14:25 | |
| sileht | _nadya_, so lets me add the unit tests cleanup to my todo once the sqlalchemy review have landed :p | 14:25 |
| _nadya_ | sileht: and that's why I was thinking about dropping source completely. Besides Resource problem there is a Meter one. If two meters have all fields equal but source are different, they will be considered as equal | 14:26 |
| *** fc__ has joined #openstack-ceilometer | 14:26 | |
| _nadya_ | sileht: it's because 'ceilometer meter list' doesn't return source | 14:27 |
| *** fc__ has quit IRC | 14:29 | |
| *** fc__ has joined #openstack-ceilometer | 14:29 | |
| _nadya_ | sileht: so the conclusion: we build Resource using actually random source (the source from 'first' sample, it depends on storage now actually) and Meter doesn't need source too, we just ignore it | 14:31 |
| *** fc__ has quit IRC | 14:32 | |
| sileht | _nadya_, yes this is the current state (once the sqlalchemy are landed), it's not perfect yet, but this permits to remove from the sql driver the annoying relationship between all tables | 14:33 |
| *** fc__ has joined #openstack-ceilometer | 14:33 | |
| *** aviau has quit IRC | 14:35 | |
| *** matsuhas_ has quit IRC | 14:37 | |
| *** fc__ has quit IRC | 14:39 | |
| *** admin0 has joined #openstack-ceilometer | 14:39 | |
| *** fc__ has joined #openstack-ceilometer | 14:39 | |
| cmart | sileht, can I ask you a question? About sqlalchemy and the LIKE function | 14:45 |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/ceilometer: Add an alarm service based on taskflow https://review.openstack.org/91763 | 14:46 |
| sileht | cmart, sure (also don't hesitate to always drop your question on the chan, I can answer later if I'm afk or someone else can anwser too) | 14:47 |
| cmart | sileht: will do. Thanks | 14:47 |
| openstackgerrit | A change was merged to openstack/ceilometer: Update testrepository configuration https://review.openstack.org/95199 | 14:47 |
| cmart | the question is the following (take into account that I'm new with SQL:). I'm trying to do smth like this: SELECT * FROM ceilometer.alarm WHERE ceilometer.alarm.rule LIKE '%meter_name\": \"memory%', but with sqlalchemy | 14:48 |
| pnascimento_ | good afternoon stackers :) if you are available I need your help with Unit Tests on Ceilometer, I had som share of UTs regarding Java Code, but in this context (Openstack/Ceilometer), this UTs are little hard to crack :/ | 14:49 |
| pnascimento_ | I'm looking into compute/test_manager.py and central/test_manager.py | 14:49 |
| cmart | sileht: and I tried with this line: query.filter(models.Alarm.rule.like('%meter_name\": \"memory%')).all().. But I'm getting no results :S | 14:50 |
| *** aviau has joined #openstack-ceilometer | 14:50 | |
| pnascimento_ | as well, looking into test/agentbase.py | 14:50 |
| pnascimento_ | I will try to put this problem of mine into a question... | 14:50 |
| cmart | Is the pattern well written? what am I missing? | 14:51 |
| sileht | cmart, not sure you can use LIKE with the JSON column type | 14:51 |
| cmart | :( | 14:52 |
| sileht | cmart, if you want parse a alarm rule, I think the best option is to have a better schema to describe them | 14:53 |
| pnascimento_ | on: tests/newAgent/test_manager.py I have class TestManager and TestRunTasks, so as I could infer, the first one its a class to test the agent as one; the second class it's to test every "task" or "method" that belong to this agent, anyone confirms? | 14:53 |
| pnascimento_ | in class TestManager there is only a method (duplicated likewise on compute agent test) named "test_load_plugins", the function content it's very straightforward, the only question, why on compute do you "injection" of "@mock.patch('ceilometer.pipeline.setup_pipeline', mock.MagicMock())" | 14:56 |
| pnascimento_ | *? | 14:56 |
| *** aviau has quit IRC | 14:56 | |
| cmart | my idea (or the bug I'm trying to solve) is to allow filter by meter within Alarm's model.. | 14:57 |
| cmart | for this first step, only filter by meter name.. That's why I got in here... and eglynn suggested that I should use LIKE within rule's column | 14:58 |
| eglynn | cmart: one sec | 14:58 |
| cmart | from my point of view, I see a couple of options (every one of them has downsides): 1) just query all the alarm's rows and then, if the meter is part of the filter, then check the query contents and search for coincidences | 15:00 |
| cmart | 2) add new columns to alarm to identify which one of them is related to meters (but what about combinated alarms?) | 15:01 |
| cmart | I know that both are really bad, and that's why I'm asking you for help :\ | 15:02 |
| *** idegtiarov has quit IRC | 15:05 | |
| *** aviau has joined #openstack-ceilometer | 15:08 | |
| pnascimento_ | some questions and problems of mine, could be related to inexperience with Mock and Unit Tests on Python :( | 15:10 |
| eglynn | cmart: changing the database schema like that is pobably not justified just to support filtering alarms by meter name | 15:12 |
| eglynn | cmart: ... the use of the LIKE operator was just a suggestion | 15:12 |
| eglynn | cmart: ... are you 100% sure that's not a runner? | 15:13 |
| *** _nadya_ has quit IRC | 15:14 | |
| *** Infitialis has quit IRC | 15:15 | |
| *** shakamunyi has quit IRC | 15:16 | |
| *** kun_huang has joined #openstack-ceilometer | 15:17 | |
| openstackgerrit | A change was merged to openstack/ceilometer-specs: Updated gitreview file for repo rename https://review.openstack.org/95299 | 15:17 |
| *** kun_huan_ has joined #openstack-ceilometer | 15:24 | |
| *** kun_huang has quit IRC | 15:25 | |
| *** IvanBerezovskiy has left #openstack-ceilometer | 15:27 | |
| cmart | eglynn: the guys from sqlalchemy told me this: "if you're using postgres's json type, and you should, you want func. json_extract_path_text(models.User.data, "parent_name").like("%Josh%") or something like that." | 15:35 |
| cmart | guess it's another way of doing this.. | 15:36 |
| cmart | it's lunch time here.. I will try that and let you guys know tomorrow.. | 15:37 |
| cmart | thanks! | 15:37 |
| eglynn | cmart: I stand corrected! :) | 15:37 |
| eglynn | cmart: thank you sir! | 15:38 |
| *** pnascimento_ has quit IRC | 15:38 | |
| *** raymondr has quit IRC | 15:44 | |
| *** _nadya_ has joined #openstack-ceilometer | 15:44 | |
| *** kun_huan_ has quit IRC | 15:51 | |
| *** kun_huang has joined #openstack-ceilometer | 15:52 | |
| *** Ruetobas has quit IRC | 16:01 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| *** igor_ has quit IRC | 16:05 | |
| *** igor_ has joined #openstack-ceilometer | 16:06 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/python-ceilometerclient: WIP: replace dict.iteritems() with six.iteritems(dict) https://review.openstack.org/95557 | 16:07 |
| *** Ruetobas has quit IRC | 16:08 | |
| *** igor_ has quit IRC | 16:10 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:14 | |
| openstackgerrit | Eugeniya Kudryashova proposed a change to openstack/python-ceilometerclient: Use HTTPClient from common Oslo code https://review.openstack.org/68939 | 16:14 |
| *** Alexei_9871 has left #openstack-ceilometer | 16:17 | |
| *** promulo has joined #openstack-ceilometer | 16:19 | |
| *** raymondr has joined #openstack-ceilometer | 16:20 | |
| *** _cjones_ has joined #openstack-ceilometer | 16:20 | |
| *** raymondr has quit IRC | 16:24 | |
| *** igor_ has joined #openstack-ceilometer | 16:36 | |
| *** drjones has joined #openstack-ceilometer | 16:39 | |
| *** igor_ has quit IRC | 16:41 | |
| *** _cjones_ has quit IRC | 16:43 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Fix exception handling of CLI https://review.openstack.org/94554 | 16:43 |
| *** admin0 has quit IRC | 16:47 | |
| *** nacim has quit IRC | 16:49 | |
| *** eglynn has quit IRC | 17:02 | |
| *** ildikov has joined #openstack-ceilometer | 17:03 | |
| openstackgerrit | Artur Svechnikov proposed a change to openstack/python-ceilometerclient: Add methods to resource classes https://review.openstack.org/91554 | 17:07 |
| openstackgerrit | Cristian A Sanchez proposed a change to openstack/python-ceilometerclient: Make HTTP client aware of no_proxy variable https://review.openstack.org/95565 | 17:15 |
| *** _nadya_ has quit IRC | 17:16 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:16 | |
| *** drjones has quit IRC | 17:21 | |
| *** safchain_ has joined #openstack-ceilometer | 17:27 | |
| *** safchain has quit IRC | 17:28 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:30 | |
| *** jergerber has joined #openstack-ceilometer | 17:32 | |
| *** jergerber has quit IRC | 17:35 | |
| *** igor_ has joined #openstack-ceilometer | 17:37 | |
| *** zqfan has joined #openstack-ceilometer | 17:39 | |
| *** igor_ has quit IRC | 17:42 | |
| *** zqfan has quit IRC | 18:01 | |
| *** _nadya_ has quit IRC | 18:03 | |
| *** raymondr has joined #openstack-ceilometer | 18:16 | |
| *** inc0 has quit IRC | 18:24 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:31 | |
| *** igor_ has joined #openstack-ceilometer | 18:38 | |
| *** igor_ has quit IRC | 18:43 | |
| *** raymondr has quit IRC | 18:48 | |
| *** safchain_ has quit IRC | 18:58 | |
| *** ddieterly has joined #openstack-ceilometer | 19:04 | |
| *** underyx is now known as Underyx|off | 19:07 | |
| *** ddieterl_ has joined #openstack-ceilometer | 19:08 | |
| *** ddieterly has quit IRC | 19:10 | |
| *** raymondr has joined #openstack-ceilometer | 19:19 | |
| *** kun_huang has quit IRC | 19:22 | |
| *** raymondr has quit IRC | 19:24 | |
| *** _nadya_ has quit IRC | 19:37 | |
| *** igor_ has joined #openstack-ceilometer | 19:39 | |
| *** igor_ has quit IRC | 19:43 | |
| *** sayalilunkad has joined #openstack-ceilometer | 19:46 | |
| *** sayalilunkad has quit IRC | 19:49 | |
| *** ddieterl_ has quit IRC | 20:05 | |
| *** ddieterly has joined #openstack-ceilometer | 20:06 | |
| *** ddieterly has quit IRC | 20:07 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/python-ceilometerclient: replace dict.iteritems() with six.iteritems(dict) https://review.openstack.org/95557 | 20:20 |
| *** igor_ has joined #openstack-ceilometer | 20:40 | |
| *** igor_ has quit IRC | 20:44 | |
| *** erecio has quit IRC | 20:46 | |
| *** cmart has quit IRC | 20:53 | |
| *** Underyx|off is now known as underyx | 20:57 | |
| *** raymondr has joined #openstack-ceilometer | 21:14 | |
| *** raymondr has quit IRC | 21:27 | |
| *** promulo has quit IRC | 21:33 | |
| *** melodous has joined #openstack-ceilometer | 21:37 | |
| melodous | hi guys | 21:37 |
| melodous | someone here to give me a hand please? | 21:37 |
| melodous | im trying to understand how to work with events on ceilometer | 21:38 |
| melodous | but my little brain is not enougth... | 21:38 |
| melodous | i dont know how to view the events i got during time | 21:39 |
| melodous | if i run ceilometer event-list i dont get anything | 21:39 |
| *** igor_ has joined #openstack-ceilometer | 21:40 | |
| *** igor_ has quit IRC | 21:45 | |
| *** gordc1 has quit IRC | 21:50 | |
| *** igor_ has joined #openstack-ceilometer | 21:50 | |
| *** igor_ has quit IRC | 21:54 | |
| *** melodous_away has joined #openstack-ceilometer | 22:03 | |
| *** melodous has quit IRC | 22:04 | |
| -openstackstatus- NOTICE: stopping gerrit briefly to rebuild its search index in an attempt to fix post-rename oddities (will update with notices every 10 minutes until completed) | 22:28 | |
| *** ChanServ changes topic to "stopping gerrit briefly to rebuild its search index in an attempt to fix post-rename oddities (will update with notices every 10 minutes until completed)" | 22:28 | |
| *** ChanServ changes topic to "OpenStack Telemetry (Ceilometer) | https://wiki.openstack.org/wiki/Ceilometer" | 22:35 | |
| *** igor_ has joined #openstack-ceilometer | 22:51 | |
| *** igor_ has quit IRC | 22:55 | |
| *** packet has joined #openstack-ceilometer | 23:35 | |
| *** promulo has joined #openstack-ceilometer | 23:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/95609 | 23:48 |
| *** igor_ has joined #openstack-ceilometer | 23:51 | |
| *** igor_ has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!