| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Correct testcase content https://review.openstack.org/85408 | 00:06 |
|---|---|---|
| *** _nadya_ has quit IRC | 00:25 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:29 | |
| *** _cjones_ has quit IRC | 00:50 | |
| *** _cjones_ has joined #openstack-ceilometer | 00:50 | |
| *** alexpilotti has quit IRC | 00:54 | |
| *** _cjones_ has quit IRC | 00:55 | |
| *** matsuhashi has quit IRC | 00:59 | |
| *** matsuhashi has joined #openstack-ceilometer | 01:01 | |
| *** yfujioka has quit IRC | 01:05 | |
| *** fnaval has joined #openstack-ceilometer | 01:10 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:21 | |
| *** flwang has quit IRC | 01:32 | |
| *** _cjones_ has quit IRC | 01:32 | |
| *** flwang has joined #openstack-ceilometer | 01:39 | |
| *** nosnos has joined #openstack-ceilometer | 01:41 | |
| *** flwang has quit IRC | 01:41 | |
| *** flwang has joined #openstack-ceilometer | 02:15 | |
| openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Disable reverse dns lookup https://review.openstack.org/79876 | 02:18 |
| *** flwang has quit IRC | 02:21 | |
| *** flwang has joined #openstack-ceilometer | 02:24 | |
| *** fnaval has quit IRC | 02:41 | |
| *** fnaval has joined #openstack-ceilometer | 02:41 | |
| *** fnaval has quit IRC | 02:46 | |
| *** fnaval has joined #openstack-ceilometer | 03:03 | |
| *** openstackgerrit has quit IRC | 03:04 | |
| *** openstackgerrit has joined #openstack-ceilometer | 03:04 | |
| *** matsuhashi has quit IRC | 03:14 | |
| *** matsuhashi has joined #openstack-ceilometer | 03:15 | |
| *** matsuhashi has quit IRC | 03:19 | |
| *** _cjones_ has joined #openstack-ceilometer | 03:29 | |
| *** nosnos has quit IRC | 03:29 | |
| *** changbl has joined #openstack-ceilometer | 03:32 | |
| *** _cjones_ has quit IRC | 03:34 | |
| *** rwsu has quit IRC | 03:50 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:19 | |
| *** nosnos has joined #openstack-ceilometer | 04:22 | |
| *** fnaval has quit IRC | 04:49 | |
| *** fnaval has joined #openstack-ceilometer | 04:50 | |
| *** fnaval has quit IRC | 04:54 | |
| *** ildikov has quit IRC | 05:01 | |
| *** shakamunyi has quit IRC | 05:14 | |
| *** ilyashakhat has quit IRC | 05:15 | |
| *** ilyashakhat has joined #openstack-ceilometer | 05:16 | |
| *** nati_ueno has quit IRC | 05:16 | |
| *** ildikov has joined #openstack-ceilometer | 05:23 | |
| *** matsuhashi has quit IRC | 05:37 | |
| *** matsuhashi has joined #openstack-ceilometer | 05:37 | |
| *** matsuhashi has quit IRC | 05:43 | |
| *** matsuhashi has joined #openstack-ceilometer | 05:44 | |
| *** matsuhas_ has joined #openstack-ceilometer | 05:50 | |
| *** matsuhashi has quit IRC | 05:53 | |
| *** matsuhas_ has quit IRC | 05:58 | |
| *** matsuhashi has joined #openstack-ceilometer | 05:58 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:03 |
| *** _nadya_ has joined #openstack-ceilometer | 06:09 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Support direct alarm_evaluator db access https://review.openstack.org/89756 | 06:19 |
| *** nati_ueno has joined #openstack-ceilometer | 06:19 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Support direct alarm_evaluator db access https://review.openstack.org/89756 | 06:25 |
| *** _nadya_ has quit IRC | 06:26 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:31 | |
| *** _nadya_ has quit IRC | 06:46 | |
| *** lsmola has joined #openstack-ceilometer | 06:48 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:51 | |
| *** idegtiarov has joined #openstack-ceilometer | 07:03 | |
| *** nati_ueno has quit IRC | 07:16 | |
| *** _nadya_ has quit IRC | 07:20 | |
| *** nacim has joined #openstack-ceilometer | 07:23 | |
| *** sdake_ has quit IRC | 07:25 | |
| *** _nadya_ has joined #openstack-ceilometer | 07:26 | |
| *** ildikov has quit IRC | 07:38 | |
| *** ildikov has joined #openstack-ceilometer | 07:43 | |
| *** _nadya_ has quit IRC | 07:45 | |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Support direct alarm_evaluator db access https://review.openstack.org/89756 | 07:50 |
| *** nati_ueno has joined #openstack-ceilometer | 07:55 | |
| *** tian has joined #openstack-ceilometer | 07:59 | |
| *** nacim has quit IRC | 08:11 | |
| *** nacim has joined #openstack-ceilometer | 08:15 | |
| *** Alexei_987 has quit IRC | 08:27 | |
| *** matsuhashi has quit IRC | 08:27 | |
| *** matsuhashi has joined #openstack-ceilometer | 08:34 | |
| openstackgerrit | Ionut Artarisi proposed a change to openstack/ceilometer: reconnect to mongodb on connection failure https://review.openstack.org/86609 | 08:58 |
| *** bada has quit IRC | 09:00 | |
| *** Alexei_987 has joined #openstack-ceilometer | 09:14 | |
| *** flwang has quit IRC | 09:25 | |
| *** eglynn has joined #openstack-ceilometer | 09:26 | |
| *** ildikov_ has joined #openstack-ceilometer | 09:28 | |
| *** ildikov has quit IRC | 09:31 | |
| openstackgerrit | Alexei Kornienko proposed a change to openstack/ceilometer: Made get_capabilities a classmethod instead of object method https://review.openstack.org/88027 | 09:32 |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Describe storage backends in the collector installation guide https://review.openstack.org/89398 | 09:37 |
| openstackgerrit | liusheng proposed a change to openstack/ceilometer: Support direct alarm_evaluator db access https://review.openstack.org/89756 | 09:40 |
| *** alexpilotti has joined #openstack-ceilometer | 09:46 | |
| openstackgerrit | Alexei Kornienko proposed a change to openstack/ceilometer: Replaced CONF object with url in storage engine creation https://review.openstack.org/87805 | 09:54 |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/ceilometer: Opencontrail network statistics driver https://review.openstack.org/85994 | 10:15 |
| *** matsuhashi has quit IRC | 10:16 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Change 200->404 for the history of non-existent alarm https://review.openstack.org/89566 | 10:19 |
| *** nacim has quit IRC | 10:21 | |
| *** matsuhashi has joined #openstack-ceilometer | 10:23 | |
| jd__ | eglynn: I've filled the form :) | 10:28 |
| eglynn | jd__: that was quick :) thank you sir! | 10:28 |
| jd__ | I know how painful it is to organize that | 10:29 |
| jd__ | I know you're getting tons of email with various more or less interesting stuff for the next summit | 10:30 |
| jd__ | it's like organizing a party but without the interesting part! | 10:30 |
| eglynn | jd__: yeah, someone else gets to choose the beer ;) | 10:32 |
| DinaBelova | :D well, such big event are always about crazy party organizing :) | 10:34 |
| *** _nadya_ has joined #openstack-ceilometer | 10:35 | |
| *** _nadya_ has quit IRC | 10:39 | |
| *** nacim has joined #openstack-ceilometer | 10:40 | |
| *** _nadya_ has joined #openstack-ceilometer | 10:44 | |
| *** Alexei_987 has quit IRC | 10:56 | |
| *** matsuhashi has quit IRC | 11:04 | |
| DinaBelova | eglynn, btw, may you take a look on the https://review.openstack.org/#/c/89566/ ? As discussed with ildikov_, there is no clear idea if that's the thing to fix really. I've caught it from the Launchpad (as it was triaged one), but after some discussions there is no solid opinion about this problem. | 11:22 |
| eglynn | DinaBelova: thanks, looking ... | 11:24 |
| ildikov_ | eglynn: hi | 11:25 |
| ildikov_ | eglynn: I will have another question too, when the bug topic is closed :) | 11:26 |
| eglynn | ildikov_: hey ... shoot | 11:26 |
| DinaBelova | :D | 11:27 |
| ildikov_ | eglynn: so I saw the doc about the sessions and there is the doc related one, which I registered under the Documentation topic | 11:27 |
| ildikov_ | eglynn: so it should be already under the right track or did I miss something? | 11:27 |
| ildikov_ | eglynn: I'm a bit overloaded nowadays, so it can easily happen :S :) | 11:28 |
| eglynn | ildikov_: yep, you submitted to the right track originally :) | 11:29 |
| eglynn | ildikov_: ... but I included it in the ceilo speadsheet anyway because we have to be sure it gets scheduled one way or the other | 11:29 |
| eglynn | ildikov_: ... so if annegentle doesn't have space for it on the docco track | 11:29 |
| eglynn | ildikov_: ... then we'll have to squeeze in somehow, seeing as it's one of the TC commitments | 11:29 |
| eglynn | ... squeeze *it in | 11:30 |
| ildikov_ | eglynn: do we know how many sessions are in the Doc track? | 11:32 |
| eglynn | ildikov_: ... docco has only 5 proposals for 4 slots | 11:33 |
| eglynn | ildikov_: ... (smaller over-subscription ratio than most other projects) | 11:33 |
| eglynn | ildikov_: ... so I'm still hopeful it can be accomodated there | 11:33 |
| ildikov_ | eglynn: hopefully in the worst case scenario we can get a shared session there | 11:34 |
| eglynn | ildikov_: ... yep, fingers crossed :) | 11:34 |
| ildikov_ | eglynn: mine too :) | 11:34 |
| eglynn | ildikov_: ... I'll reach out to anne about it today, at a time more convenient for her TZ | 11:34 |
| ildikov_ | eglynn: cool, thanks, please update me with the news | 11:35 |
| eglynn | ildikov_: ... will do! | 11:35 |
| ildikov_ | eglynn: ... cool, many thanks | 11:37 |
| ildikov_ | eglynn: now get back to the original bug, I'm the bad cop there too :) | 11:37 |
| eglynn | ildikov_: ... how uncharacteristic of you! ;) | 11:38 |
| ildikov_ | eglynn: ... FYI, currently not my statistics are the worst :) | 11:38 |
| eglynn | ildikov_: ... you must be going soft ;) | 11:39 |
| ildikov_ | eglynn: ... I blame the Easter Bunny ;) | 11:45 |
| ildikov_ | eglynn: do you think that we need an error message there? | 11:54 |
| ildikov_ | eglynn: for instance it will return an error also, when the user do not have permission to see the alarm history | 11:54 |
| eglynn | ildikov_: ... an explicit error msg in response to alarm history query, for never-existing alarms? | 11:55 |
| ildikov_ | eglynn: ... yes | 11:55 |
| ildikov_ | eglynn: ... I do not see why it would be more useful, than the empty list | 11:55 |
| ildikov_ | eglynn: ... also in case of the future TTL value never existing is a question | 11:56 |
| eglynn | ildikov_: ... isn't an error msg provided by Dina's patch? | 11:56 |
| eglynn | https://review.openstack.org/#/c/89566/3/ceilometer/api/controllers/v2.py line 94? | 11:56 |
| ildikov_ | eglynn: ... there is also a config option to turn off alarm history, in which case if we tell that the alarm never existed is totally misleading and not correct | 11:57 |
| ildikov_ | eglynn: .. in my view the bug itself is not valid | 11:57 |
| ildikov_ | eglynn: ... so if everyone sais that the bug itself is correct, than it will be hard to return correct error messag(es) | 11:58 |
| eglynn | ildikov_: ... but doesn't the error msg currentlu just say that history wasn't found? | 11:58 |
| ildikov_ | eglynn: ... maybe I haven't seen the latest version of the patch | 11:59 |
| eglynn | ildikov_: ... yeah it's "No history found for the alarm alarm_id" in https://review.openstack.org/#/c/89566/3/ceilometer/api/controllers/v2.py | 11:59 |
| ildikov_ | eglynn: ... but in this case, my question is that why we should report that back, when the result of the alarm history query is an empty list? | 11:59 |
| ildikov_ | eglynn: ... my first question is that the bug itself, for which this patch is proposed is valid or not? | 12:00 |
| ildikov_ | eglynn: ... this one I mean: https://bugs.launchpad.net/ceilometer/+bug/1288104 | 12:01 |
| eglynn | ildikov_: hmmm, do we currently return 404 for a constrained statistics query with no hits? | 12:01 |
| eglynn | ildikov_: ... IIRC we just return [] in that case | 12:01 |
| ildikov_ | eglynn: ... I'm pretty sure we do not return error message for any empty query result | 12:01 |
| ildikov_ | eglynn: ... and I also think that it is correct in that way, so the empty list is the best I think | 12:02 |
| sileht | eglynn, ildikov_ I agree too | 12:04 |
| ildikov_ | eglynn: ... the current error message in that patch does not have any added value compared it to the empty list, but if you write anything else in that error message it will not be correct for all cases | 12:04 |
| eglynn | ildikov_, sileht: ... don't we already return 404s for non-existence in several places? | 12:04 |
| * eglynn checks code .... | 12:04 | |
| eglynn | say https://github.com/openstack/ceilometer/blob/master/ceilometer/api/controllers/v2.py#1861 for non-existent alarm | 12:05 |
| sileht | eglynn, only when we are really sure | 12:05 |
| ildikov_ | eglynn: ... this is my strict opinion, but I thought it would be better to discuss, if I may miss something here | 12:05 |
| eglynn | ... or https://github.com/openstack/ceilometer/blob/master/ceilometer/api/controllers/v2.py#1452 for resource | 12:05 |
| eglynn | sileht: ... and the "not sure" case occurs say after the TTL has kicked in? | 12:06 |
| ildikov_ | eglynn: ... it's a different issue there | 12:06 |
| eglynn | ildikov_: ... is the core issue the fact that alarm history retention can be turned off? | 12:07 |
| sileht | eglynn, or when a normal user doesn't see items of the history because all changed are done by a admin | 12:07 |
| ildikov_ | eglynn: the core issue is that the empty list describes the best that there is no alarm history, because of any reason | 12:08 |
| eglynn | ildikov_: ok, I begin to see the light: keep it generic coz we can't be 100% sure why there's no history in any individual case | 12:09 |
| eglynn | BUT ... should we also apply that logic to a resource for which all samples have possibly been expired? | 12:10 |
| eglynn | i.e. as a result of https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/impl_mongodb.py#L593 | 12:10 |
| *** nosnos has quit IRC | 12:10 | |
| ildikov_ | eglynn: ... it's a waste of time and resorces there to figure out all the different cases and send back the proper error message, if we even can do that in every case... | 12:11 |
| *** promulo has quit IRC | 12:11 | |
| ildikov_ | eglynn: ... it's better to me to apply this for resources as well | 12:11 |
| ildikov_ | eglynn: ... in the example that you've sent one particular resource was requested with it's id | 12:12 |
| ildikov_ | eglynn: ... so in case of get all it works fine | 12:12 |
| eglynn | ildikov_: yep, which is just asking for history for an individual alarm, no? | 12:13 |
| eglynn | ... just *like | 12:13 |
| ildikov_ | eglynn: ... the get_one is a special case there, if there isn't any resource than there isn't anything to respond | 12:13 |
| ildikov_ | eglynn: ... no :) | 12:13 |
| ildikov_ | eglynn: ... it would be the same as requesting one particular alarm | 12:13 |
| ildikov_ | eglynn: ... but we can have alarm history for deleted alarms | 12:14 |
| ildikov_ | eglynn: ... it's an unlucky situation how the endpoint URL looks like in case of alarm history | 12:15 |
| ildikov_ | eglynn: ... but the query itself on that endpoint is a get_all request | 12:15 |
| eglynn | ildikov_: sure about the deleted alarm history, that's deliberate ... the alarm is gone but the history still occurred | 12:16 |
| ildikov_ | eglynn: ... I cannot see any benefit of having an error message here, but more problems because of it | 12:16 |
| eglynn | ildikov_: ... ok, that's a fair point | 12:17 |
| ildikov_ | eglynn: ... I'm not the expert of REST, so maybe someone who is, could mention some professional reasons and not just bad feelings like me :) | 12:17 |
| eglynn | ildikov_: ... TBH I don't fully the distinction between the empty history case and requesting a un-found resource that may never have existed *or* may have existed but been evicted on sample expiry | 12:19 |
| DinaBelova | sorry, was afk, reading the logs about the 404 problems | 12:19 |
| ildikov_ | eglynn, sileht: ... so as a summarization, I still think that the bug is invalid | 12:20 |
| ildikov_ | eglynn: ... in the example there I do not know how we could return nothing on that endpoint, which should return one resource | 12:20 |
| sileht | ildikov_, like you can see on the bug report history I have change my mind many times, but I prefer keep the current behavior | 12:21 |
| ildikov_ | eglynn: ... in case of the alarm history, you request for all the stored alarm history, with a default query, where the alarm_id is something and the user can specify additional query fields | 12:21 |
| ildikov_ | sileht: ... yes, I saw that, it is why I didn't mark it invalid and stated in the review also that it should be discussed | 12:22 |
| ildikov_ | eglynn: ... so these two cases are different, as the behavior/purpose of these two endpoints are different | 12:22 |
| * eglynn reads back over bug history ... | 12:23 | |
| ildikov_ | eglynn: ... for the get_one endpoint I do not know how to return nothing as there is no empty list there | 12:23 |
| * DinaBelova trying to get final opinion | 12:23 | |
| * ildikov_ will be back in a minute | 12:24 | |
| eglynn | DinaBelova: OK, I'm swayed by sileht & ildikov's arguments, particularly the final point on get_one versus get_all | 12:26 |
| *** ildikov has joined #openstack-ceilometer | 12:26 | |
| eglynn | (... i.e. there's being no analogue of [] to return from a get_one-style query) | 12:26 |
| DinaBelova | ildikov_, sileht, eglynn so about some final opinion about this moment - this exact moment is not the bug here | 12:26 |
| *** _nadya_ has quit IRC | 12:27 | |
| DinaBelova | that's just a way of presenting the things to the user - so if no history was found, well - just [] will be returned and that's it | 12:27 |
| eglynn | DinaBelova: yes, that about sums it up | 12:27 |
| DinaBelova | okay, in this case let's mark the bug as "not a bug" and I'll abandon the change not to be in the list of the changes) | 12:28 |
| DinaBelova | ildikov_, sileht, eglynn - ok with that? | 12:28 |
| *** ildikov_ has quit IRC | 12:28 | |
| ildikov | eglynn: ... kind of no analogue, so the response of get_one there is correct, but here (for alarm history or any other get_all request) we have a valid query response, which is an empty list | 12:28 |
| DinaBelova | ildikov, ok, got your idea, I think that's quite logical here | 12:28 |
| sileht | DinaBelova, ok for me | 12:29 |
| eglynn | DinaBelova: yes, please do that re. the launchpad housekeeping | 12:29 |
| ildikov | DinaBelova: I'm ok with that and if this will be the final decision, I'm sorry that you worked with this bug that much | 12:29 |
| * DinaBelova abandoning | 12:29 | |
| ildikov | DinaBelova: thanks, and sorry again :( | 12:29 |
| DinaBelova | ildikov, np - I think several hours is ok to make launchpad cleaner))) | 12:29 |
| eglynn | DinaBelova: ... nontheless, thanks for your attention to detail on this | 12:29 |
| DinaBelova | and to decide if that's the bug or not)) | 12:29 |
| ildikov | DinaBelova, eglynn, sileht: thanks all for the time for clarifying this | 12:30 |
| * eglynn needs to eat after all that ;) | 12:30 | |
| ildikov | DinaBelova: I can add a comment to the bug and link the log about this discussion, I think you can also set the bug to invalid, but I will if you cannot | 12:31 |
| ildikov | eglynn: ... bon apetite ;) | 12:31 |
| *** eglynn is now known as eglynn-lunch | 12:31 | |
| ildikov | eglynn-lunch: : ... you're the luckyer, I will have a half bloody meeting right now :) | 12:31 |
| DinaBelova | I've abandoned the change and marked https://bugs.launchpad.net/ceilometer/+bug/1288104 invalid with some comments :) | 12:33 |
| ildikov | DinaBelova: thanks :) | 12:34 |
| sileht | DinaBelova, thx | 12:35 |
| *** _nadya_ has joined #openstack-ceilometer | 12:35 | |
| *** alexpilotti has quit IRC | 12:35 | |
| DinaBelova | ildikov, sileht, eglynn-lunch (heh, funny to write *-lunch :)), btw - I have mentioned interesting moment while investigating the working process of the ceilometer rpc working >>> WARNING ceilometer.publisher.rpc [-] Failed to publish 5125 samples, queue them <<< - it looks like that while tests running messages are collected in some way and queued because of it, I'd like to investigate this moment :) | 12:39 |
| DinaBelova | are you ok with it? | 12:39 |
| *** _nadya_ has quit IRC | 12:40 | |
| *** alexpilotti has joined #openstack-ceilometer | 12:41 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:44 | |
| sileht | DinaBelova, yes you can take a look to the swift middleware tests | 12:44 |
| sileht | DinaBelova, it does this kinds of negative tests | 12:44 |
| DinaBelova | okay, thanks | 12:44 |
| *** _nadya_ has quit IRC | 12:44 | |
| *** fnaval has joined #openstack-ceilometer | 12:45 | |
| sileht | DinaBelova, This behavior is disable by default , but for swift you can want to postpone publishing of samples if rabbitmq is not avialable to not block swift request (but you have to deal with the memory queue instead :p) | 12:46 |
| *** jdob has joined #openstack-ceilometer | 12:48 | |
| DinaBelova | :D | 12:55 |
| *** promulo has joined #openstack-ceilometer | 13:08 | |
| *** prad has joined #openstack-ceilometer | 13:08 | |
| *** nekron99 has quit IRC | 13:18 | |
| *** nati_ueno has quit IRC | 13:21 | |
| *** gordc has joined #openstack-ceilometer | 13:25 | |
| *** promulo has quit IRC | 13:32 | |
| *** fnaval has quit IRC | 13:39 | |
| *** fnaval has joined #openstack-ceilometer | 13:39 | |
| *** joesavak has joined #openstack-ceilometer | 13:42 | |
| *** mengxd has joined #openstack-ceilometer | 13:42 | |
| *** fnaval has quit IRC | 13:44 | |
| ildikov | gordc: hi | 13:48 |
| gordc | ildikov: hi, whatsup? | 13:49 |
| ildikov | gordc: I commented on the ceilo client patch | 13:51 |
| ildikov | gordc: I think it would be better to revert the changes in that job, I asked eglynn-lunch about this earlier, but then this patch lost the priority as Jenkins needed some time to finish with the tests :) | 13:52 |
| gordc | ildikov: hmm... do you know if gate changed because of this bug? | 13:54 |
| *** fnaval has joined #openstack-ceilometer | 13:54 | |
| *** promulo has joined #openstack-ceilometer | 13:54 | |
| DinaBelova | folks, o/ I've noticed small problem with the ceilometer devstack - https://bugs.launchpad.net/devstack/+bug/1311666 - I've proposed small fix for that | 13:54 |
| gordc | ildikov: just wondering because if it was for another reason, they may change the non-voting later and we're broken again... i'll look at infra patches to see. | 13:54 |
| ildikov | gordc: the gate is set to non-voting because of that bug, I made the fix for, according to my knowledge | 13:56 |
| gordc | ildikov: ah i see. that's fine with me then. | 13:56 |
| ildikov | gordc: not just ceilo client was affected and when I saw that the job is non-voting, it seemed to be better to revert that temporary fix | 13:58 |
| ildikov | gordc: cool, thanks | 13:58 |
| *** fnaval has quit IRC | 13:59 | |
| *** thomasem has joined #openstack-ceilometer | 14:00 | |
| *** tongli has joined #openstack-ceilometer | 14:03 | |
| *** eglynn-lunch is now known as eglynn | 14:05 | |
| *** rwsu has joined #openstack-ceilometer | 14:14 | |
| *** flwang has joined #openstack-ceilometer | 14:23 | |
| *** julim has joined #openstack-ceilometer | 14:26 | |
| *** fnaval has joined #openstack-ceilometer | 14:27 | |
| *** Alexei_987 has joined #openstack-ceilometer | 14:28 | |
| gordc | ildikov: was there ever a plan to include groupby functionality in complex query? | 14:48 |
| *** bada has joined #openstack-ceilometer | 14:48 | |
| ildikov | gordc: we plan to extend complex query functionality for statistics | 14:49 |
| gordc | ildikov: ok... but no plans to add groupby for complex query against samples? | 14:50 |
| ildikov | gordc: hmm, I'm not sure I get what you mean | 14:51 |
| ildikov | gordc: do you have an example? | 14:51 |
| gordc | ildikov: hmm. let me take a look at use case... maybe it's covered by statistics. | 14:52 |
| ildikov | gordc: I have a session proposal for statistics, if you have a UC that is valid, doable, but not implemented yet | 14:54 |
| ildikov | gordc: I'm also open for complex query improvements, I just not clearly see how groupby could be used for samples query | 14:55 |
| ildikov | gordc: but maybe the reason for this is that we're close here to the end of the working day :) | 14:55 |
| gordc | ildikov: yeah. that's why i'm asking. i had someone ask me about grouping by resource last week. can't remember exact details. | 14:56 |
| gordc | ildikov: i'll ping you again if i remember and if its not covered by complex query for statistics. | 14:56 |
| ildikov | gordc: ok, cool, if you have a pastebin with an example or something like this, that would be the best, but anyway, ping me if you have more details :) | 14:57 |
| *** shakamunyi has joined #openstack-ceilometer | 14:59 | |
| gordc | ildikov: ah got it... they wanted the latest sample of a specific meter for each resource... so right now they've been calling v2/meters/<meter_name>?q.field=resource_id&q.value=resourceId&limit=1 for each resourceId (multiple calls). i couldn't think of a way to do this one call... can you? | 15:02 |
| ityaptin | eglynn, hi! May you take a look into our collector performance document, if you have time? Doc's url: https://docs.google.com/document/d/1ARpKiYW2WN94JloG0prNcLjMeom-ySVhe8fvjXG_uRU/edit?usp=sharing | 15:02 |
| eglynn | ityaptin: will do! | 15:02 |
| *** shakamunyi has quit IRC | 15:02 | |
| *** shakamunyi has joined #openstack-ceilometer | 15:03 | |
| *** idegtiarov has quit IRC | 15:04 | |
| *** cdent has joined #openstack-ceilometer | 15:05 | |
| ildikov | gordc: hmm, give me a few sec | 15:05 |
| *** mengxd has quit IRC | 15:05 | |
| ildikov | gordc: I'm not an SQL expert, so it does not seem to be an easy question on the first place :) | 15:10 |
| *** julim has quit IRC | 15:11 | |
| gordc | ildikov: agreed. maybe outside of the scope of our api. | 15:11 |
| ildikov | gordc: if it requires subquery, which it seems to do, it makes the problem complex, maybe more complex, than we would like it to be | 15:11 |
| ildikov | gordc: we were thinking about subqueries earlier, but we decided to skip it, because of the complexity it would bring into the system | 15:14 |
| ildikov | gordc: I'm not against talking a few words about this on the summit, if we will have a half session for CQ for statistics | 15:16 |
| *** gordc has quit IRC | 15:16 | |
| ildikov | gordc: there can be other opinions or someone with more SQL experience | 15:16 |
| *** prad has quit IRC | 15:30 | |
| *** prad has joined #openstack-ceilometer | 15:32 | |
| *** gordc has joined #openstack-ceilometer | 15:33 | |
| *** _cjones_ has joined #openstack-ceilometer | 15:35 | |
| gordc | ildikov: sorry about that, internet dropped. i don't believe it requires a subquery... but i'm not sure it would work regardless because we'd need some mechanism to drop ungrouped columns. | 15:40 |
| gordc | we can leave it off the table for no i think. | 15:41 |
| ildikov | gordc: I saw that, I hoped you will read the logs as I got distracted a bit in the meantime :) | 15:44 |
| ildikov | gordc: I will think about it, but I cannot see a way currently without subquery or additional logic in the db driver | 15:45 |
| gordc | ildikov: cool cool | 15:49 |
| openstackgerrit | A change was merged to openstack/ceilometer: Disable reverse dns lookup https://review.openstack.org/79876 | 15:54 |
| *** Ruetobas has quit IRC | 16:01 | |
| *** shakamunyi has quit IRC | 16:03 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:03 | |
| *** ildikov has quit IRC | 16:03 | |
| *** dperaza has joined #openstack-ceilometer | 16:04 | |
| *** shakamunyi has joined #openstack-ceilometer | 16:05 | |
| *** Ruetobas has quit IRC | 16:08 | |
| DinaBelova | eglynn, btw, are you ok with current version of the https://review.openstack.org/#/c/89398/ ? | 16:10 |
| *** jsavak has joined #openstack-ceilometer | 16:12 | |
| *** joesavak has quit IRC | 16:13 | |
| *** Ruetobas has joined #openstack-ceilometer | 16:13 | |
| *** joesavak has joined #openstack-ceilometer | 16:15 | |
| *** jsavak has quit IRC | 16:18 | |
| eglynn | DinaBelova: thanks ... on a call now, will look when that finishes | 16:18 |
| DinaBelova | eglynn, np, it's you who I should thank) | 16:18 |
| *** Anju_ has joined #openstack-ceilometer | 16:22 | |
| *** changbl has quit IRC | 16:24 | |
| *** erecio has joined #openstack-ceilometer | 16:33 | |
| *** cdent has quit IRC | 16:34 | |
| *** cdent has joined #openstack-ceilometer | 16:36 | |
| *** fnaval has quit IRC | 16:39 | |
| *** fnaval has joined #openstack-ceilometer | 16:39 | |
| *** fnaval has quit IRC | 16:44 | |
| *** fnaval has joined #openstack-ceilometer | 16:45 | |
| eglynn | ityaptin: an interesting doc! | 16:50 |
| eglynn | ityaptin: ... couple questions | 16:51 |
| eglynn | ityaptin: ... what exactly does the "Delta time" on the x-axis of the single collector graph actually represent? | 16:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/88714 | 16:52 |
| eglynn | (i.e. delta between what & what?) | 16:52 |
| eglynn | ityaptin: ... also any thoughts on explaining the saw-tooth pattern for hbase in the 3-collector case as the message per second is ramped up? | 16:53 |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: refactor sql backend for write speed https://review.openstack.org/80461 | 16:57 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/88730 | 17:01 |
| *** nacim has quit IRC | 17:03 | |
| ityaptin | eglynn, o/. let me clarify these moments: 1. Delta time - time from test starting) 2. I think it's happens because connections pool is empty and samples wait next free connection. | 17:04 |
| *** nati_ueno has joined #openstack-ceilometer | 17:06 | |
| eglynn | ityaptin: ... are the generated messages drip-fed into the test over time, or all generated together and injected from the get-go? | 17:07 |
| ityaptin | eglynn: messages are generated one by one with improving rate from 50 m/s to 1000 m/s during the test | 17:09 |
| eglynn | ityaptin: fair enough, thanks for the clarification | 17:10 |
| eglynn | ityaptin: one other quick point, have you captured the mongo "Overflow sort stage buffered data usage ..." issue seen in a bug on LP? | 17:11 |
| ityaptin | eglynn: collector + hbase cluster can process about 300 m/s and then rate exceed this number, messages accumulate in queue | 17:11 |
| *** nati_uen_ has joined #openstack-ceilometer | 17:12 | |
| eglynn | ityaptin: (that mongo error could be indicative of an issue with our indexing ... along the lines of https://jira.mongodb.org/browse/SERVER-13611 ) | 17:12 |
| eglynn | ityaptin: ... so it would be good for us to dig in and get to the bottom of it | 17:12 |
| *** nati_ueno has quit IRC | 17:15 | |
| *** eglynn is now known as eglynn-afk | 17:15 | |
| ityaptin | I have not captured this bug before. | 17:16 |
| ityaptin | I may take a look on this problem and try to catch it. | 17:19 |
| *** shakamunyi has quit IRC | 17:32 | |
| *** shakayumi has joined #openstack-ceilometer | 17:32 | |
| *** eglynn-afk is now known as eglynn | 17:36 | |
| eglynn | ityaptin: thanks, that would be great! | 17:37 |
| *** shakayumi has quit IRC | 17:40 | |
| *** lsmola has quit IRC | 17:51 | |
| *** cdent has quit IRC | 18:00 | |
| *** ildikov has joined #openstack-ceilometer | 18:02 | |
| *** promulo has quit IRC | 18:10 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Describe storage backends in the collector installation guide https://review.openstack.org/89398 | 18:13 |
| DinaBelova | eglynn, gordc - have updated docs change ^^ Please take a look) | 18:14 |
| eglynn | DinaBelova: thanks, looking ... | 18:14 |
| DinaBelova | I did not fix all comments, please take a look on my answers :) | 18:14 |
| *** jaypipes has quit IRC | 18:15 | |
| *** sdake_ has joined #openstack-ceilometer | 18:18 | |
| *** shakamunyi has joined #openstack-ceilometer | 18:19 | |
| DinaBelova | eglynn, you here? | 18:45 |
| *** promulo has joined #openstack-ceilometer | 18:45 | |
| eglynn | DinaBelova: yep | 18:45 |
| DinaBelova | I'm about your comment to the https://review.openstack.org/#/c/89398/6/doc/source/install/manual.rst | 18:46 |
| DinaBelova | s/Ceilometer services/central agent, notification agent, and API service/ | 18:46 |
| eglynn | DinaBelova: darn typo! | 18:46 |
| eglynn | DinaBelova: s/central agent/collector/ | 18:46 |
| eglynn | DinaBelova: sorry! | 18:46 |
| DinaBelova | oh, thank you :) | 18:46 |
| DinaBelova | I was having this face -> 0_o | 18:46 |
| DinaBelova | :D | 18:46 |
| eglynn | DinaBelova: I can well imagine :) | 18:47 |
| *** _nadya_ has joined #openstack-ceilometer | 18:47 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Describe storage backends in the collector installation guide https://review.openstack.org/89398 | 18:55 |
| DinaBelova | eglynn, ^^ :) | 18:56 |
| eglynn | DinaBelova: looking ... | 18:56 |
| DinaBelova | ildikov, you too :) ^^ | 18:56 |
| ildikov | DinaBelova: I'm faster, already looking ;) | 18:57 |
| DinaBelova | :D | 18:57 |
| openstackgerrit | A change was merged to openstack/ceilometer: cleanup virt pollster code https://review.openstack.org/80662 | 19:03 |
| *** changbl has joined #openstack-ceilometer | 19:03 | |
| ildikov | DinaBelova: for the first quick look it seems to be ok, I will wait for Jenkins to see the "built picture" | 19:04 |
| DinaBelova | ok, cool) | 19:04 |
| ildikov | DinaBelova: I mean that the links are working and how the final formatting looks, etc | 19:05 |
| ildikov | DinaBelova: so nothing serious :) | 19:05 |
| DinaBelova | ;) | 19:05 |
| *** nati_uen_ has quit IRC | 19:08 | |
| *** jdob_ has joined #openstack-ceilometer | 19:10 | |
| *** tongli has quit IRC | 19:12 | |
| *** nati_ueno has joined #openstack-ceilometer | 19:14 | |
| *** eglynn has quit IRC | 19:20 | |
| openstackgerrit | Dina Belova proposed a change to openstack/ceilometer: Describe storage backends in the collector installation guide https://review.openstack.org/89398 | 19:29 |
| *** jaypipes has joined #openstack-ceilometer | 19:30 | |
| *** nati_ueno has quit IRC | 19:30 | |
| *** julim has joined #openstack-ceilometer | 19:41 | |
| *** _nadya_ has quit IRC | 19:49 | |
| *** jsavak has joined #openstack-ceilometer | 20:01 | |
| *** joesavak has quit IRC | 20:04 | |
| *** nati_ueno has joined #openstack-ceilometer | 20:06 | |
| *** _nadya_ has joined #openstack-ceilometer | 20:09 | |
| *** nati_ueno has quit IRC | 20:11 | |
| *** _nadya_ has quit IRC | 20:21 | |
| *** nati_ueno has joined #openstack-ceilometer | 20:24 | |
| *** julim has quit IRC | 20:33 | |
| *** joesavak has joined #openstack-ceilometer | 20:37 | |
| *** prad_ has joined #openstack-ceilometer | 20:39 | |
| *** prad has quit IRC | 20:41 | |
| *** prad_ is now known as prad | 20:41 | |
| *** jsavak has quit IRC | 20:41 | |
| *** jdob_ has quit IRC | 20:44 | |
| *** yjiang5 is now known as yjiang5_away | 21:00 | |
| *** Anju_ has quit IRC | 21:01 | |
| *** erecio has quit IRC | 21:07 | |
| *** thomasem has quit IRC | 21:13 | |
| *** shakamunyi has quit IRC | 21:17 | |
| *** jdob has quit IRC | 21:17 | |
| openstackgerrit | Rob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation https://review.openstack.org/77957 | 21:28 |
| *** gordc has quit IRC | 21:32 | |
| *** joesavak has quit IRC | 21:41 | |
| *** changbl has quit IRC | 21:47 | |
| *** nati_ueno has quit IRC | 21:59 | |
| *** nati_ueno has joined #openstack-ceilometer | 22:15 | |
| *** shakamunyi has joined #openstack-ceilometer | 22:39 | |
| *** shakamunyi has quit IRC | 22:52 | |
| openstackgerrit | Rob Raymond proposed a change to openstack/ceilometer: Replace hard coded WSGI application creation https://review.openstack.org/77957 | 22:53 |
| *** shakamunyi has joined #openstack-ceilometer | 22:59 | |
| *** fnaval has quit IRC | 23:02 | |
| *** fnaval has joined #openstack-ceilometer | 23:18 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Describe storage backends in the collector installation guide https://review.openstack.org/89398 | 23:20 |
| openstackgerrit | A change was merged to openstack/ceilometer: Improve combination rule validation https://review.openstack.org/88355 | 23:28 |
| *** jaypipes has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!