| openstackgerrit | Joe Gordon proposed a change to openstack/ceilometer: Disable oslo.messaging debug logs https://review.openstack.org/82671 | 00:01 |
|---|---|---|
| *** Alexei_987 has joined #openstack-ceilometer | 00:10 | |
| *** promulo has quit IRC | 00:17 | |
| *** promulo has joined #openstack-ceilometer | 00:17 | |
| *** openstack has joined #openstack-ceilometer | 00:19 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:26 | |
| *** _cjones_ has quit IRC | 00:52 | |
| *** llu_linux has joined #openstack-ceilometer | 00:55 | |
| *** llu has quit IRC | 00:56 | |
| *** creiht has left #openstack-ceilometer | 01:17 | |
| *** rwsu has quit IRC | 01:26 | |
| *** nosnos has joined #openstack-ceilometer | 01:29 | |
| *** matsuhashi has quit IRC | 01:30 | |
| *** matsuhashi has joined #openstack-ceilometer | 01:30 | |
| *** rwsu has joined #openstack-ceilometer | 01:42 | |
| *** yjiang5_1 has joined #openstack-ceilometer | 01:57 | |
| *** yjiang5 has quit IRC | 01:57 | |
| *** tasdomas has quit IRC | 01:58 | |
| *** tasdomas has joined #openstack-ceilometer | 02:00 | |
| *** kun_huang has joined #openstack-ceilometer | 02:19 | |
| kun_huang | can I create a meter by myself? | 02:19 |
| liusheng | well, you can create a sample with the meter_name not implemented in ceilometer, this meter_name will be included in meter-list | 02:50 |
| kun_huang | liusheng this is not implemented in v2 now? | 02:50 |
| kun_huang | liusheng POST /v2/meters/(meter_name) | 02:51 |
| kun_huang | liusheng not exist one I think | 02:51 |
| flwang | kun_huang: IIUC, you can post sample to Ceilometer | 02:55 |
| flwang | and IIRC, the meter you set in the sample will be saved automatically | 02:56 |
| kun_huang | flwang could I delete one meter? | 02:57 |
| kun_huang | the one I created or not | 02:57 |
| flwang | kun_huang: I don't think so. I'm not aware there is an api can do that | 02:59 |
| flwang | kun_huang: btw, congrats for your Rally core :) | 02:59 |
| liusheng | kun_huang, deleting sample, this has be discussed in mailing-list about a month ago | 02:59 |
| kun_huang | flwang haha thank you | 02:59 |
| flwang | kun_huang: looking forward to seeing more contributions | 03:01 |
| kun_huang | liusheng POST /v2/meters/(meter_name) this API is doing "create a sample"? | 03:02 |
| kun_huang | flwang to rally? | 03:02 |
| liusheng | kun_huang, yes | 03:02 |
| kun_huang | liusheng got it, thank you :) | 03:02 |
| liusheng | kun_huang, congratulations for you:) | 03:04 |
| kun_huang | liusheng flwang :) | 03:05 |
| *** matsuhashi has quit IRC | 03:12 | |
| *** nosnos has quit IRC | 03:29 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:30 | |
| *** matsuhashi has quit IRC | 03:36 | |
| *** ddutta has quit IRC | 03:44 | |
| *** ddutta has joined #openstack-ceilometer | 03:54 | |
| *** nati_ueno has quit IRC | 04:13 | |
| *** piyushmasrani has joined #openstack-ceilometer | 04:15 | |
| *** ddutta has quit IRC | 04:15 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:23 | |
| *** nosnos has joined #openstack-ceilometer | 04:25 | |
| *** nati_ueno has joined #openstack-ceilometer | 04:40 | |
| *** _nadya_ has joined #openstack-ceilometer | 05:01 | |
| *** matsuhashi has quit IRC | 05:14 | |
| *** matsuhashi has joined #openstack-ceilometer | 05:18 | |
| *** _nadya_ has quit IRC | 05:21 | |
| *** mihgen has joined #openstack-ceilometer | 05:29 | |
| *** _nadya_ has joined #openstack-ceilometer | 05:41 | |
| *** _nadya_ has quit IRC | 05:44 | |
| *** nati_ueno has quit IRC | 05:47 | |
| *** nati_ueno has joined #openstack-ceilometer | 05:51 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/76762 | 06:08 |
| *** ildikov_ has quit IRC | 06:15 | |
| *** Ruetobas has quit IRC | 06:28 | |
| *** Ruetobas has joined #openstack-ceilometer | 06:34 | |
| *** Ruetobas has quit IRC | 06:39 | |
| *** Ruetobas has joined #openstack-ceilometer | 06:39 | |
| *** ildikov_ has joined #openstack-ceilometer | 06:59 | |
| *** lsmola_ has quit IRC | 07:02 | |
| *** lsmola_ has joined #openstack-ceilometer | 07:03 | |
| *** bogdando has quit IRC | 07:05 | |
| *** ityaptin has quit IRC | 07:16 | |
| *** eglynn has joined #openstack-ceilometer | 07:19 | |
| eglynn | ildikov_: good morning! | 07:19 |
| eglynn | ildikov_: thanks for the recheck comment on https://review.openstack.org/82672 earlier | 07:19 |
| eglynn | ildikov_: nice to wake up to a green report from jenkins :) | 07:20 |
| eglynn | (... I jumped the gun with my own recheck late last night when I saw one of the tempest runs fail in realtime via the zuul status) | 07:21 |
| eglynn | jd__: any idear when ttx is planning to cut the milestone-proposed for RC1? | 07:22 |
| eglynn | (... I ask coz /me would really like to get https://bugs.launchpad.net/ceilometer/+bug/1297004 in for RC1) | 07:23 |
| ildikov_ | eglynn: good morning :) | 07:23 |
| ildikov_ | eglynn: np, I do not have problems with red color, but I like green more in case of tests too, it's better to review a code if Jenkins had already some positive opinion about it ;) | 07:24 |
| eglynn | ildikov_: wise words :) | 07:25 |
| eglynn | ildikov_: ... I hope that late-night bug report reads sensibly in the cold light of morning | 07:25 |
| eglynn | ildikov_: ... I was scratching my head for a long while yesterday before I realized what was going on | 07:25 |
| eglynn | ... debugging javascript run via mongo map-reduce is no fun :( | 07:26 |
| ildikov_ | eglynn: here the sun is shining, so I'm just about to dig deeper in that bug and the fix | 07:27 |
| eglynn | ildikov_: nice! ... and thanks :) | 07:27 |
| ildikov_ | eglynn: I have experience only with pure javascript debugging, so I know that there isn't much fun in it :( | 07:28 |
| ildikov_ | eglynn: BTW, I have a nice review for you, small and simple, I wrote som enew tests too, hopefully I wrote enough: https://review.openstack.org/#/c/82110/ | 07:29 |
| eglynn | ildikov_: ... my own fault, I convinced myself first there was a subtle bug in the original JS, only later realized that it was actually a bug in eglynn's understanding of M-R | 07:29 |
| ildikov_ | eglynn: so if you may have time between a map and a reduce, it would be good, if you could take a look ;) | 07:29 |
| *** saju_m has joined #openstack-ceilometer | 07:30 | |
| eglynn | ildikov_: cool, looking at https://review.openstack.org/82110 now | 07:30 |
| ildikov_ | eglynn: I'm not an expert of M-R, but it's never too late :) | 07:30 |
| eglynn | ildikov_: ... well, I'm less inexpert than I was yesterday | 07:30 |
| ildikov_ | eglynn: just a dumb question, do we have any coverage report somewhere? | 07:30 |
| ildikov_ | eglynn: than I have to say that it is a beautiful day to learn some more tricks about M-R :) | 07:31 |
| eglynn | ildikov_: ... /me has vague memories of a --coverage option to run-tests.sh back in the day | 07:32 |
| eglynn | ildikov_: ... but not sure if that carries over to tox/testr | 07:32 |
| ildikov_ | eglynn: I was just asking, because as you may know I'm the great fan of code refactor and I realize time to time that we do not have enough tests | 07:32 |
| *** gibi has quit IRC | 07:33 | |
| ildikov_ | eglynn: I made a mistake during the implementation of that small patch and there wasn't any failing test to show it to me, so now I wrote one, but I'm just wondering, how many similar situations could we have | 07:34 |
| eglynn | ildikov_: ... I see a --coverage in tox.ini under the cover testenv | 07:34 |
| eglynn | ildikov_: ... so I guess tox -ecover ...? | 07:34 |
| ildikov_ | eglynn: cool, I will try, tnx | 07:35 |
| ildikov_ | eglynn: is it running on the gate too? | 07:36 |
| eglynn | ildikov_: ... not AFAIK, but don't take that as gospel (I could be wrong) | 07:36 |
| *** nati_ueno has quit IRC | 07:37 | |
| ildikov_ | eglynn: tnx, I will check, if I will have some free time it's not that extremely important, but it would be good to have an initial value about it | 07:38 |
| eglynn | the iron law of nit-picking in action | 07:38 |
| eglynn | ... /me was just about to nitpick on the lack of i18n in the exception messages | 07:38 |
| eglynn | ... lazy _() for the win! :) | 07:38 |
| ildikov_ | eglynn: if you mean that install lazy _() patch, I had some review effort to get it in, so the guy really deserved that real +1 after my endless questions :) | 07:40 |
| eglynn | ildikov_: yeah that's exactly the patch I mean, community service at its best! :) | 07:41 |
| ildikov_ | eglynn: LOL :) | 07:42 |
| eglynn | ildikov_: minor question inline on https://review.openstack.org/82110 ... looks good to me otherwise | 07:42 |
| ildikov_ | eglynn: that nitpick is an API level check, which should not face with this NotImplementedError issue | 07:45 |
| ildikov_ | eglynn: but I can modify that check to the list version, if you think that it can be affected once by this issue | 07:46 |
| eglynn | ildikov_: a-ha, of course, the call never makes it as far as the storage driver | 07:46 |
| eglynn | ildikov_: ... yep, carry on, ignore my babbling ;) | 07:47 |
| ildikov_ | eglynn: np, I assume you're before your early morning coffee :) | 07:47 |
| ildikov_ | eglynn: I had a thought to backport this patch to stable/havana | 07:47 |
| ildikov_ | eglynn: what do you think about this weird thought of mine? | 07:48 |
| eglynn | ildikov_: looks like a perfect backport candidate to me, thanks! | 07:48 |
| ildikov_ | cool, tnx :) | 07:48 |
| eglynn | ... /me scurries off to get kids out the door to school | 07:48 |
| ildikov_ | eglynn: so everyone survived the yesterday's fun program, good to read that :) | 07:50 |
| eglynn | ildikov_: ... yep no fatalities or spinal injuries :) | 07:50 |
| ildikov_ | eglynn: cool, I hope it was fun too and not just you're screaming that "don't do this! don't do that!" ;) | 07:51 |
| eglynn | ildikov_: ... nah, no shouting from me, I was just trying to figure out how to sneak onto the trampoline myself :) | 07:58 |
| *** eglynn has quit IRC | 07:59 | |
| *** bogdando has joined #openstack-ceilometer | 08:03 | |
| *** mihgen has quit IRC | 08:19 | |
| *** admin0 has joined #openstack-ceilometer | 08:28 | |
| *** inc0 has joined #openstack-ceilometer | 08:30 | |
| *** kun_huang has quit IRC | 08:35 | |
| *** matsuhashi has quit IRC | 08:49 | |
| *** saju_m has quit IRC | 08:50 | |
| *** yassine has joined #openstack-ceilometer | 08:52 | |
| *** saju_m has joined #openstack-ceilometer | 08:52 | |
| *** nacim has joined #openstack-ceilometer | 08:55 | |
| *** matsuhashi has joined #openstack-ceilometer | 08:56 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix validation error for invalid field name in simple query https://review.openstack.org/82110 | 09:00 |
| *** kun_huang has joined #openstack-ceilometer | 09:06 | |
| *** matsuhashi has quit IRC | 09:14 | |
| *** matsuhashi has joined #openstack-ceilometer | 09:18 | |
| *** eglynn has joined #openstack-ceilometer | 09:34 | |
| *** saju_m has quit IRC | 09:38 | |
| *** nosnos has quit IRC | 09:39 | |
| *** saju_m has joined #openstack-ceilometer | 09:57 | |
| *** kun_huang has quit IRC | 10:08 | |
| *** matsuhashi has quit IRC | 10:08 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:10 | |
| openstackgerrit | Jia Dong proposed a change to openstack/python-ceilometerclient: Replace assertTrue with assertIsNotNull for check an object https://review.openstack.org/82764 | 10:16 |
| openstackgerrit | Peter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests https://review.openstack.org/82572 | 10:23 |
| openstackgerrit | A change was merged to openstack/ceilometer: Fixes duplicated names in alarm time constraints https://review.openstack.org/81264 | 10:31 |
| *** mihgen has joined #openstack-ceilometer | 11:01 | |
| *** saju_m has quit IRC | 11:07 | |
| *** saju_m has joined #openstack-ceilometer | 11:08 | |
| openstackgerrit | Nadya Privalova proposed a change to openstack/ceilometer: Make Resource able to store several meters https://review.openstack.org/78244 | 11:08 |
| *** saju_m has quit IRC | 11:09 | |
| openstackgerrit | Nadya Privalova proposed a change to openstack/ceilometer: Make Resource able to store several meters https://review.openstack.org/78244 | 11:12 |
| openstackgerrit | Nadya Privalova proposed a change to openstack/ceilometer: Make Resource able to store several meters https://review.openstack.org/78244 | 11:15 |
| *** saju_m has joined #openstack-ceilometer | 11:15 | |
| openstackgerrit | Peter Portante proposed a change to openstack/ceilometer: Use swob instead of webob in swift unit tests https://review.openstack.org/82572 | 11:21 |
| *** saju_m has quit IRC | 11:38 | |
| *** saju_m has joined #openstack-ceilometer | 11:38 | |
| *** julim has joined #openstack-ceilometer | 12:13 | |
| *** matsuhashi has quit IRC | 12:16 | |
| *** matsuhashi has joined #openstack-ceilometer | 12:23 | |
| *** promulo has quit IRC | 12:23 | |
| inc0 | hmm, can you gather event types "compute.instance.exists" per flavor? so instance:myflavor will also have samples from compute.instance.exist? | 12:34 |
| *** zul has quit IRC | 12:38 | |
| *** zul has joined #openstack-ceilometer | 12:41 | |
| *** saju_m has quit IRC | 12:41 | |
| *** saju_m has joined #openstack-ceilometer | 12:55 | |
| *** julienvey has joined #openstack-ceilometer | 12:57 | |
| *** julienvey has quit IRC | 12:58 | |
| *** julienvey has joined #openstack-ceilometer | 12:58 | |
| *** saju_m has quit IRC | 12:59 | |
| *** saju_m has joined #openstack-ceilometer | 13:00 | |
| *** heyongli has joined #openstack-ceilometer | 13:02 | |
| *** matsuhashi has quit IRC | 13:03 | |
| *** saju_m has quit IRC | 13:04 | |
| *** saju_m has joined #openstack-ceilometer | 13:05 | |
| *** ildikov_ has quit IRC | 13:06 | |
| *** heyongli has quit IRC | 13:07 | |
| *** heyongli has joined #openstack-ceilometer | 13:07 | |
| *** saju_m has quit IRC | 13:09 | |
| *** saju_m has joined #openstack-ceilometer | 13:10 | |
| *** gordc has joined #openstack-ceilometer | 13:11 | |
| *** ildikov_ has joined #openstack-ceilometer | 13:13 | |
| *** julienvey has quit IRC | 13:16 | |
| flwang | jd__: ping | 13:17 |
| jd__ | flwang: pong | 13:18 |
| flwang | jd__: a question about metering the public traffic in/out of the cloud for a given tenant | 13:19 |
| flwang | jd__: does ceilometer support this? | 13:19 |
| flwang | jd__: and if not, does this requirement depends on your patch https://review.openstack.org/#/c/70090 ? | 13:20 |
| flwang | s/your/this | 13:20 |
| *** ChanServ changes topic to "the gerrit event stream is currently hung, blocking all testing. troubleshooting is in progress (next update at 14:00 utc)" | 13:20 | |
| flwang | jd__: sorry, I know you may not the right person I should ask, but just try if I'm lucky man :) | 13:21 |
| *** changbl has quit IRC | 13:22 | |
| jd__ | yeah you need to take a look at neutron | 13:24 |
| jd__ | it's the one doing that | 13:24 |
| flwang | jd__: ok, seems we didn't touch much against Neutron metering | 13:27 |
| flwang | jd__: thanks all the same :) | 13:29 |
| *** ChanServ changes topic to "OpenStack Telemetry (Ceilometer) | https://wiki.openstack.org/wiki/Ceilometer" | 13:29 | |
| openstackstatus | NOTICE: the issue with gerrit cleared on its own before any corrective action was taken | 13:29 |
| eglynn | good afternoon folks ... | 13:39 |
| eglynn | a call for reviews in python-ceilometerclient if any cores have bandwidth | 13:39 |
| eglynn | ... /me would like to get https://review.openstack.org/78356 https://review.openstack.org/80499 https://review.openstack.org/79973 https://review.openstack.org/75076 landed at least | 13:40 |
| eglynn | ... before cutting a new 1.0.10 release to co-incide roughly with RC1 | 13:40 |
| *** nati_ueno has joined #openstack-ceilometer | 13:40 | |
| *** ddutta has joined #openstack-ceilometer | 13:42 | |
| *** promulo has joined #openstack-ceilometer | 13:55 | |
| *** liusheng has quit IRC | 13:58 | |
| *** nati_ueno has quit IRC | 13:59 | |
| *** nacim has quit IRC | 14:02 | |
| *** heyongli has quit IRC | 14:02 | |
| *** prad has joined #openstack-ceilometer | 14:05 | |
| *** prad has quit IRC | 14:14 | |
| *** prad has joined #openstack-ceilometer | 14:16 | |
| openstackgerrit | Stéphane Albert proposed a change to openstack/python-ceilometerclient: Statistics groupby handling improvement https://review.openstack.org/80256 | 14:20 |
| openstackgerrit | Akhil Hingane proposed a change to openstack/ceilometer: VMware vSphere: Improve the accuracy of queried samples https://review.openstack.org/80314 | 14:20 |
| *** sayalilunkad has joined #openstack-ceilometer | 14:21 | |
| *** alexpilotti has quit IRC | 14:28 | |
| *** admin0 has quit IRC | 14:32 | |
| *** luisg has joined #openstack-ceilometer | 14:35 | |
| *** jmckind has joined #openstack-ceilometer | 14:35 | |
| *** kun_huang has joined #openstack-ceilometer | 14:35 | |
| nprivalova | jd__: hi! I'm sorry but I still have static related question. if we have base-class and several childs. Let's consider one method from base-class. We know that this method may not be implemented in childs but if it is, it should use 'self', so all child-implementations is not static. Why should we declare this method as static in base-class? | 14:44 |
| *** terriyu has joined #openstack-ceilometer | 14:53 | |
| *** sayalilunkad has quit IRC | 14:53 | |
| inc0 | nprivalova, I don't understand. If you're using self the method is not static, why you declare it as static in the first place? | 15:03 |
| inc0 | in my opinion it shouldn't be static in base class | 15:04 |
| nprivalova | inc0: I have the same vision. it's from here https://review.openstack.org/#/c/80662/3/ceilometer/compute/virt/inspector.py | 15:04 |
| inc0 | ah, interfaces in python... | 15:06 |
| inc0 | quite frankly I can't see any point of such operation | 15:06 |
| inc0 | also, I'd use ABC here | 15:07 |
| *** nacim has joined #openstack-ceilometer | 15:09 | |
| nprivalova | inc0: abc may not be used because methods may not be implemented | 15:10 |
| inc0 | well, if they're not implemented (in this case) they won't be user | 15:11 |
| inc0 | used* | 15:11 |
| inc0 | or rather, they'll raise NotImplemented anyway | 15:11 |
| *** saju_m has quit IRC | 15:13 | |
| *** packet has joined #openstack-ceilometer | 15:14 | |
| nprivalova | gordc: https://review.openstack.org/#/c/82131/ . I even don't know how to comment that | 15:14 |
| nprivalova | gordc: hi :) | 15:15 |
| gordc | nprivalova: :) | 15:15 |
| gordc | nprivalova: writing up a comment now. | 15:15 |
| nprivalova | gordc: maybe have several patches with several variants.. Maybe Sean will not notice one of them :D | 15:16 |
| gordc | nprivalova: try creating a few more accounts in different names. | 15:17 |
| *** ddutta has quit IRC | 15:17 | |
| *** changbl has joined #openstack-ceilometer | 15:20 | |
| inc0 | tell me please, because I might be missing something | 15:22 |
| inc0 | I'd like to get statistics of flavors usage. I mean tenant x had 3 vms of flavor y in given time | 15:23 |
| inc0 | I can get number of vms | 15:23 |
| inc0 | using statistics and cardinality aggregate | 15:23 |
| inc0 | with "instance" metric | 15:24 |
| inc0 | since instance metric gets event_type "instance.exists" | 15:24 |
| inc0 | but instance:flavorname doesn't get this event_type, so I can't really tell how much of those are present at any given time | 15:25 |
| inc0 | I can try to parse metadata, because there is flavor there, but this query times out, its very long | 15:26 |
| inc0 | so, as I see it I could eighter try to optimize this query (np try to create some magical regexp to evaluate metadata json in single request) or make instance:flavorname get instance.exists event | 15:27 |
| inc0 | or I'm missing some other, obvious way to do it | 15:27 |
| inc0 | one I could actually implement without messing with code | 15:27 |
| nprivalova | inc0: how do you want to obtain this statistics? there are filters on metadata, why don't you use it? | 15:30 |
| inc0 | nprivalova, when I tried to use it (namely call metadata.instance_type eq myflavor) the query was very slow | 15:31 |
| inc0 | in sqlalchemy | 15:32 |
| nprivalova | inc0: ah, ok. the problem is in performance, right? | 15:32 |
| inc0 | I guess | 15:32 |
| inc0 | it timed out | 15:32 |
| nprivalova | inc0: I see...And you want to create your special meter with flavor to make it faster? | 15:33 |
| inc0 | on the other hand, query without this filter was about 2s, which is ok | 15:33 |
| inc0 | there is such meter already | 15:33 |
| inc0 | instance:flavorname | 15:33 |
| inc0 | but it doesn't get event "instance.exists" | 15:33 |
| inc0 | only created, updated, deleted etc | 15:34 |
| inc0 | but tracking vm lifespan by these events would be horrible | 15:34 |
| nprivalova | inc0: let me think about it, I'll check the code | 15:34 |
| inc0 | thank you | 15:35 |
| eglynn | inc0: just reading back over your question (... on a call just now) | 15:41 |
| eglynn | inc0: ... not entirely sure I understand problem | 15:41 |
| eglynn | inc0: ... so the instance.flavor meter is emmitted by the a compute agent pollster also | 15:41 |
| eglynn | inc0: ... https://github.com/openstack/ceilometer/blob/master/ceilometer/compute/pollsters/instance.py#L40 | 15:41 |
| eglynn | inc0: ... i.e. it is not dependent entirely on incoming notifications | 15:42 |
| eglynn | inc0: ... i.e. the meter is generated in *two* places | 15:42 |
| eglynn | inc0: ... the pollster above running periodicallay *and* when releavnt incoming events are received https://github.com/openstack/ceilometer/blob/master/ceilometer/compute/notifications/instance.py#147 | 15:43 |
| inc0 | eglynn, still if I'll call something like that: stats = ceilo_client.statistics.list('instance') in given period for given tenant I'll get let say 7 vms | 15:43 |
| nprivalova | eglynn: we have notifications on this meter too. but anyway event_type is written to metadata | 15:43 |
| inc0 | and instance:flavor would show me only one record when one of them was created | 15:43 |
| *** alexpilotti has joined #openstack-ceilometer | 15:44 | |
| inc0 | nprivalova, yes, it is, thats how I've noticed that event_type of instance.exists isn't present in any of instance:flavor samples | 15:44 |
| eglynn | inc0: ... hmmm, that seems wrong unless I've misread the compute pollster code | 15:44 |
| eglynn | one sec | 15:44 |
| eglynn | inc0: in a fresh mongo-based devstack, I'm see the same number of instance and instance:flavor samples | 15:46 |
| eglynn | > db.meter.find({counter_name: 'instance'}).count() | 15:46 |
| eglynn | 38 | 15:46 |
| eglynn | > db.meter.find({counter_name: 'instance:m1.tiny'}).count() | 15:46 |
| eglynn | 38 | 15:46 |
| *** alexpilotti has quit IRC | 15:46 | |
| inc0 | strange, for me the difference is very big | 15:47 |
| nprivalova | inc0: are you sure that you receive notifications? | 15:47 |
| inc0 | about 200 to 1100 | 15:47 |
| inc0 | well, I do have "instance" samples | 15:48 |
| eglynn | nprivalova: good point, have you looked at the notification agent logs? | 15:48 |
| eglynn | inc0: ^^^ | 15:48 |
| *** _cjones_ has joined #openstack-ceilometer | 15:48 | |
| inc0 | hmm, I'll take a look | 15:48 |
| *** _cjones__ has joined #openstack-ceilometer | 15:49 | |
| nprivalova | inc0: make sure that agent-notification service is running. It should, because it's start was added to devstack rather a long time ago | 15:49 |
| eglynn | inc0: ... I'm seeing exactly the same number of instance and instance:flavor samples with event_type set | 15:50 |
| eglynn | > db.meter.find({counter_name: 'instance', 'resource_metadata.event_type': {$exists: 1}}).count() | 15:50 |
| eglynn | 15 | 15:50 |
| eglynn | > db.meter.find({counter_name: 'instance:m1.tiny', 'resource_metadata.event_type': {$exists: 1}}).count() | 15:50 |
| eglynn | 15 | 15:50 |
| inc0 | now, thats strange | 15:50 |
| inc0 | I did get this event | 15:50 |
| inc0 | but it stopped...I get other events | 15:50 |
| inc0 | but not this one in this meter | 15:50 |
| eglynn | inc0: ... note the event is only set for samples that are derived from notifications, as opposed to polling | 15:51 |
| inc0 | yeah, seems like something is wrong with my notification agnet | 15:51 |
| eglynn | inc0: ... sometimes the same meter is driven my both models (push & pull) | 15:51 |
| eglynn | s/my/by/ | 15:51 |
| nprivalova | eglynn: btw, is it ok that we have the line "metadata['event_type'] = message['event_type']" during notification processing but no such action during polling? | 15:51 |
| eglynn | nprivalova: yeah, I guess there's no real analogue of event_type for polling | 15:52 |
| *** _cjones_ has quit IRC | 15:52 | |
| eglynn | nprivalova: ... unless we were to make one up, say event_type='compute_agent_poll' or something | 15:52 |
| inc0 | yeah...notification agent is throwing TRACES | 15:52 |
| *** kun_huang has quit IRC | 15:53 | |
| inc0 | thank you, I'll debug it | 15:53 |
| eglynn | (not sure if that even makes sense, expect for some sort of artificial consistency) | 15:53 |
| inc0 | still, later on I'll take a look at querying over metadata in sqlalchemy because it was just too long | 15:53 |
| *** jergerber has joined #openstack-ceilometer | 15:54 | |
| *** kun_huang has joined #openstack-ceilometer | 15:55 | |
| *** kun_huang has quit IRC | 15:55 | |
| nprivalova | inc0: :) I think performance on sqlalchemy bothers everybody who use Ceilometer. There is several patches on review regarding it | 15:56 |
| *** packet has quit IRC | 15:57 | |
| inc0 | https://github.com/openstack/ceilometer/blob/master/ceilometer/compute/util.py#L41 can someone explain me this line please? | 15:58 |
| inc0 | ah, never mind | 15:58 |
| eglynn | nprivalova: yeah so the perf issues with sqlalchemy were thrown into sharp relief by the mongo licensing fight around the Marconi failed graduation last week | 15:58 |
| eglynn | inc0: I can explain that if you're still interested | 15:58 |
| eglynn | ? | 15:58 |
| inc0 | eglynn, I didn't look close enought. I've noticed how its indented | 15:59 |
| *** packet has joined #openstack-ceilometer | 15:59 | |
| inc0 | sorry to bother you, its almost 5pm here;) | 15:59 |
| eglynn | inc0: basically the idea is to ensure that only certain user metadata are maintained in the resource metadata, plus to avoid an embedded period in the metadata key | 16:00 |
| eglynn | ... mongo doesn't like embedded periods, as these have special meaning to it | 16:00 |
| inc0 | yeah, and our flavor names has periods in them | 16:00 |
| inc0 | this line raises an error, I'm not sure why yet | 16:01 |
| *** Ruetobas has quit IRC | 16:01 | |
| eglynn | inc0: but surely that's only user metadata being transformed there? | 16:01 |
| eglynn | inc0: ... i.e. not system metadata | 16:01 |
| inc0 | I don't know yet | 16:02 |
| eglynn | inc0: ... and it's only keys, not values | 16:02 |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| inc0 | that would explain TypeError | 16:03 |
| eglynn | inc0: so here's an example of a period in *system* metadata values ... | 16:04 |
| eglynn | > db.meter.find({counter_name: 'instance:m1.tiny'}, {'resource_metadata.instance_type': 1}) | 16:04 |
| eglynn | { "_id" : ObjectId("53316ff92304bc7d2f01514d"), "resource_metadata" : { "instance_type" : "m1.tiny" } } | 16:04 |
| inc0 | attributeerror* | 16:04 |
| eglynn | inc0: the transformation code you referred to is applied to *user* metadata keys IIUC | 16:04 |
| eglynn | inc0: ... what's the context of the AttributeError? | 16:05 |
| nprivalova | eglynn: and what's wrong with license? TC don't want Mongo at all? Is there any requirement like "make your code work on sql first"? | 16:05 |
| eglynn | nprivalova: see the withdrawal mail from the Marconi folks ... http://lists.openstack.org/pipermail/openstack-dev/2014-March/030638.html | 16:06 |
| nprivalova | eglynn: thanks | 16:07 |
| eglynn | nprivalova: key phrase ... "The drivers currently supported by the project don't cover some important cases related to deploying it. One of them solves a licensing issue but introduces a scale issue whereas the other one solves the scale issue and introduces a licensing issue." | 16:07 |
| eglynn | ... the two drivers being spoken about are sqlalchemy and mongo IIUC | 16:08 |
| *** Ruetobas has quit IRC | 16:08 | |
| eglynn | nprivalova: ... here's one of the mails from an earlier thread that lobs the mongo licensing grenade | 16:09 |
| eglynn | http://lists.openstack.org/pipermail/openstack-dev/2014-March/030510.html | 16:09 |
| eglynn | TBH I have no idea why all these objections weren't raised over two years ago when ceilo was incubated | 16:10 |
| nprivalova | eglynn: hmm, 'great' news... | 16:11 |
| eglynn | nprivalova: yeah I kinda ignored it last week as I was off-site | 16:12 |
| eglynn | nprivalova: ... but we prolly need to discuss impact on ceilo | 16:12 |
| *** luisg has quit IRC | 16:12 | |
| nprivalova | eglynn: last week I was trying to move Mongo on gating and nobody tell me about 'license'. Only issues with repos | 16:13 |
| *** Ruetobas has joined #openstack-ceilometer | 16:13 | |
| eglynn | nprivalova: honestly I think it's a "storm in a teacup" but we may need to be defensive | 16:14 |
| eglynn | nprivalova: ... I'll add a topic to the weeking agenda | 16:14 |
| nprivalova | eglynn: cool | 16:14 |
| *** nati_ueno has joined #openstack-ceilometer | 16:17 | |
| inc0 | eglynn, sorry for late answer -> AttributeError is raised when src_metadata is list | 16:20 |
| *** mihgen has quit IRC | 16:21 | |
| eglynn | inc0: can you enable debug and copy the stack trace to paste.openstack.org ? | 16:21 |
| inc0 | eglynn, maybe later on, I'd like to take look at it myself first. I need to learn this code myself;) | 16:25 |
| inc0 | also, currently our cloud controler was taken down so I can't really reproduce it at this moment | 16:25 |
| eglynn | inc0: cool, np ... a most admirable sentiment! :) | 16:26 |
| inc0 | thank you very much anyway | 16:26 |
| *** nati_ueno has quit IRC | 16:30 | |
| *** sayalilunkad has joined #openstack-ceilometer | 16:35 | |
| *** inc0 has quit IRC | 16:44 | |
| *** Shaan7 has quit IRC | 16:44 | |
| *** Shaan7 has joined #openstack-ceilometer | 16:44 | |
| *** sayalilunkad has quit IRC | 16:46 | |
| *** nati_ueno has joined #openstack-ceilometer | 17:02 | |
| *** _cjones__ has quit IRC | 17:05 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:05 | |
| *** sayalilunkad has joined #openstack-ceilometer | 17:12 | |
| *** nacim has quit IRC | 17:15 | |
| *** sayalilunkad has quit IRC | 17:52 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Ensure idempotency of cardinality reduction in mongo https://review.openstack.org/82672 | 17:54 |
| *** eglynn has quit IRC | 17:57 | |
| *** sayalilunkad has joined #openstack-ceilometer | 18:00 | |
| *** changbl has quit IRC | 18:00 | |
| *** packet has quit IRC | 18:07 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:10 | |
| *** packet has joined #openstack-ceilometer | 18:11 | |
| *** changbl has joined #openstack-ceilometer | 18:19 | |
| *** _nadya_ has quit IRC | 18:20 | |
| *** shakamunyi has joined #openstack-ceilometer | 18:25 | |
| *** shakamunyi has quit IRC | 18:25 | |
| *** shakamunyi has joined #openstack-ceilometer | 18:26 | |
| *** shakamunyi has quit IRC | 18:26 | |
| *** ildikov_ has quit IRC | 18:29 | |
| *** yassine has quit IRC | 18:33 | |
| *** yassine has joined #openstack-ceilometer | 18:39 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:50 | |
| *** shakayumi has joined #openstack-ceilometer | 18:50 | |
| *** shakayumi has quit IRC | 18:50 | |
| *** shakayumi has joined #openstack-ceilometer | 18:51 | |
| *** _nadya_ has quit IRC | 18:59 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:05 | |
| *** shakayumi has quit IRC | 19:05 | |
| *** mdickson has left #openstack-ceilometer | 19:09 | |
| *** openstackgerrit has quit IRC | 19:18 | |
| *** openstackgerrit has joined #openstack-ceilometer | 19:18 | |
| *** zul has quit IRC | 19:26 | |
| *** zul has joined #openstack-ceilometer | 19:30 | |
| *** _nadya_ has quit IRC | 19:44 | |
| *** shakayumi has joined #openstack-ceilometer | 19:45 | |
| *** shakayumi has quit IRC | 19:45 | |
| *** shakayumi has joined #openstack-ceilometer | 19:54 | |
| *** shakamunyi has joined #openstack-ceilometer | 19:56 | |
| *** eglynn has joined #openstack-ceilometer | 19:57 | |
| *** shakayumi has quit IRC | 19:58 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:59 | |
| *** changbl has quit IRC | 20:00 | |
| *** sayalilunkad has quit IRC | 20:02 | |
| *** shakamunyi has quit IRC | 20:03 | |
| *** shakamunyi has joined #openstack-ceilometer | 20:06 | |
| *** yassine has quit IRC | 20:12 | |
| *** _nadya_ has quit IRC | 20:20 | |
| *** changbl has joined #openstack-ceilometer | 20:51 | |
| *** shakayumi has joined #openstack-ceilometer | 21:08 | |
| *** shakayumi has quit IRC | 21:09 | |
| *** shakamunyi has quit IRC | 21:11 | |
| *** ildikov_ has joined #openstack-ceilometer | 21:13 | |
| *** thomasem has joined #openstack-ceilometer | 21:32 | |
| *** promulo has quit IRC | 21:34 | |
| *** alexpilotti has joined #openstack-ceilometer | 21:44 | |
| *** zigo has quit IRC | 21:47 | |
| *** zigo has joined #openstack-ceilometer | 21:49 | |
| *** thomasem has quit IRC | 21:58 | |
| *** promulo has joined #openstack-ceilometer | 21:59 | |
| *** jmckind has quit IRC | 22:00 | |
| *** changbl has quit IRC | 22:17 | |
| *** prad has quit IRC | 22:22 | |
| *** gordc has quit IRC | 22:26 | |
| *** jergerber has quit IRC | 22:44 | |
| *** promulo has quit IRC | 22:48 | |
| openstackgerrit | A change was merged to openstack/ceilometer: VMware vSphere: Improve the accuracy of queried samples https://review.openstack.org/80314 | 23:04 |
| *** promulo has joined #openstack-ceilometer | 23:07 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Add complex query support for alarms https://review.openstack.org/75076 | 23:07 |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Add complex query support for alarm history https://review.openstack.org/79973 | 23:09 |
| *** _cjones_ has quit IRC | 23:14 | |
| *** _cjones_ has joined #openstack-ceilometer | 23:14 | |
| *** eglynn has quit IRC | 23:17 | |
| *** anamalagon has joined #openstack-ceilometer | 23:27 | |
| *** julim has quit IRC | 23:30 | |
| *** anamalagon has quit IRC | 23:33 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!