Monday, 2014-01-20

*** asalkeld has joined #openstack-ceilometer01:31
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Added resources support in pollster's interface  https://review.openstack.org/5848901:41
*** krast has joined #openstack-ceilometer02:19
*** flwang has quit IRC02:48
*** flwang has joined #openstack-ceilometer03:29
*** asalkeld has quit IRC03:43
*** asalkeld has joined #openstack-ceilometer03:56
*** metallsnab has joined #openstack-ceilometer04:05
*** Alexei_987 has left #openstack-ceilometer04:05
*** metallsnab has quit IRC04:14
*** yfujioka has quit IRC04:26
*** asalkeld has quit IRC04:31
*** asalkeld has joined #openstack-ceilometer05:39
*** SergeyLukjanov_ is now known as SergeyLukjanov05:40
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280806:03
*** asalkeld has quit IRC06:32
*** ildikov_ has quit IRC06:33
*** asalkeld has joined #openstack-ceilometer06:45
*** SergeyLukjanov is now known as SergeyLukjanov_a06:59
*** SergeyLukjanov_a is now known as SergeyLukjanov_07:00
*** SergeyLukjanov_ is now known as SergeyLukjanov07:08
*** SergeyLukjanov is now known as SergeyLukjanov_07:18
*** ildikov_ has joined #openstack-ceilometer07:37
*** SergeyLukjanov_ is now known as SergeyLukjanov07:53
openstackgerritDazhao Yu proposed a change to openstack/ceilometer: Make sure use IPv6 sockets for ceilometer in IPv6 environment  https://review.openstack.org/6778608:07
openstackgerritDazhao Yu proposed a change to openstack/ceilometer: Make sure use IPv6 sockets for ceilometer in IPv6 environment  https://review.openstack.org/6778608:12
*** Alexei_987 has joined #openstack-ceilometer08:36
openstackgerritJia Dong proposed a change to openstack/ceilometer: Implement  meter query by 'counter_volume' field.  https://review.openstack.org/6738408:39
openstackgerritIlya Tyaptin proposed a change to openstack/ceilometer: Add run pollsters on demand function to api  https://review.openstack.org/6655108:41
*** SergeyLukjanov is now known as SergeyLukjanov_08:47
Alexei_987sileht: ping08:48
silehtAlexei_987, hi !08:48
Alexei_987do you still working on sqla session fix?08:48
Alexei_987I would like to know the status of this issue08:49
Alexei_987sileht: I'm working on several patches intended to encapsulate sqla.get_session method08:50
silehtAlexei_987, I have no time allowed to this this week, I can work on it only on my empty work time08:50
Alexei_987sileht: ok so I still have time to finish my patches :)08:50
silehtAlexei_987, I haven't done more that you have seen last week08:50
Alexei_987sileht: ok thanks. I'll try to upload my patches in the next few days08:51
silehtAlexei_987, cool, go ahead08:51
*** krast has quit IRC09:00
*** yassine has joined #openstack-ceilometer09:16
*** yassine has quit IRC09:18
*** yassine has joined #openstack-ceilometer09:18
*** vrovachev has joined #openstack-ceilometer09:26
*** SergeyLukjanov_ is now known as SergeyLukjanov09:43
*** SergeyLukjanov is now known as SergeyLukjanov_09:48
*** IvanBerezovskiy has joined #openstack-ceilometer09:49
*** flwang has quit IRC10:09
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Exit expirer earlier is db-ttl is disabled  https://review.openstack.org/6780310:34
*** _ruhe is now known as ruhe10:42
*** yassine has quit IRC10:46
*** AMike has quit IRC11:16
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Exit expirer earlier is db-ttl is disabled  https://review.openstack.org/6780311:24
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Exit expirer earlier is db-ttl is disabled  https://review.openstack.org/6780311:25
*** boris-42 has quit IRC11:31
*** sayali has joined #openstack-ceilometer11:34
*** sayali_ has joined #openstack-ceilometer11:34
openstackgerritMehdi Abaakouk proposed a change to openstack/ceilometer: Exit expirer earlier if db-ttl is disabled  https://review.openstack.org/6780311:41
*** boris-42 has joined #openstack-ceilometer11:41
*** flwang has joined #openstack-ceilometer11:52
*** AMike has joined #openstack-ceilometer11:53
*** AMike has joined #openstack-ceilometer11:53
*** eglynn has joined #openstack-ceilometer11:58
*** asalkeld has left #openstack-ceilometer11:59
*** ruhe is now known as _ruhe12:09
*** ildikov__ has joined #openstack-ceilometer12:10
*** ildikov_ has quit IRC12:10
*** ildikov__ is now known as ildikov_12:11
*** yassine has joined #openstack-ceilometer12:11
*** _ruhe is now known as ruhe12:30
*** julienvey_ has joined #openstack-ceilometer12:35
*** RemyS has joined #openstack-ceilometer12:38
*** julienvey_ has quit IRC12:38
*** yassine has quit IRC12:39
*** yassine has joined #openstack-ceilometer12:40
*** Audi has joined #openstack-ceilometer12:54
*** xuhanp has joined #openstack-ceilometer12:57
*** SergeyLukjanov_ is now known as SergeyLukjanov12:58
*** Audi has quit IRC12:58
*** kwhitney has left #openstack-ceilometer13:03
*** kwhitney1 has quit IRC13:05
*** kwhitney has joined #openstack-ceilometer13:05
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Removes use of timeutils.set_time_override  https://review.openstack.org/6782613:08
openstackgerritlizheming proposed a change to openstack/ceilometer: use six.move.xrange replace xrange  https://review.openstack.org/6763513:10
*** ruhe is now known as _ruhe13:11
*** xuhanp has quit IRC13:12
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Removes use of timeutils.set_time_override  https://review.openstack.org/6782613:12
*** RemyS has quit IRC13:13
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Removes use of timeutils.set_time_override  https://review.openstack.org/6782613:14
*** tongli has joined #openstack-ceilometer13:15
tongli@eglynn, ping,13:15
eglynntongli: yep, your mail is on to-do list, will get to it shortly13:16
tongli@eglynn, thanks so much.13:16
tongli@eglynn, really appreciate it.13:17
*** SergeyLukjanov is now known as SergeyLukjanov_13:26
*** _ruhe is now known as ruhe13:29
*** jdob has joined #openstack-ceilometer13:33
*** kwhitney has quit IRC13:36
*** thomasem has joined #openstack-ceilometer13:37
*** thomasem has left #openstack-ceilometer13:37
*** thomasem has joined #openstack-ceilometer13:54
*** kwhitney has joined #openstack-ceilometer14:01
*** sayali has quit IRC14:04
*** SergeyLukjanov_ is now known as SergeyLukjanov14:04
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements in operator for complex query functionality  https://review.openstack.org/6668714:07
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements field validation for complex query functionality  https://review.openstack.org/6530214:07
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarms  https://review.openstack.org/6215814:07
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarm history  https://review.openstack.org/6215914:07
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for samples  https://review.openstack.org/6215714:07
ildikov_jd__: ping14:10
jd__ildikov_: pong14:10
ildikov_jd__: we have some results on Colander investigation14:10
gibijd__: here is a wiki with some code pasted https://wiki.openstack.org/wiki/Ceilometer/ComplexFilterExpressionsInAPIQueries/ValidationWithColander14:10
jd__ildikov_: tell me about it14:11
* jd__ reads14:11
ildikov_jd__: we've tried to show with examples that what can be done and what not really, or just in a complicated way14:11
jd__that looks interesting indeed14:13
jd__thanks for sharing14:13
thomasemjd__, ping14:13
jd__thomasem: you can see that I'm here14:13
thomasemjd__, Lol, oh hey! :P14:13
thomasemjd__, So, just wanted to give you (and thusly the CM team) a heads up that I'm moving on to another project and will no longer be allocated time to work on Ceilometer. I may still find time outside of work, but I didn't want to disappear without saying anything. :)14:14
jd__thomasem: ok! it was nice having you around14:15
jd__thomasem: is your next project OpenStack related?14:15
thomasemjd__, Yeah, man. It was great working with you and the rest of the team.14:15
thomasemjd__, It is. There will be a fair bit of neutron and nova work.14:16
*** iprokof has joined #openstack-ceilometer14:16
*** iprokof has quit IRC14:16
jd__thomasem: cool, so we'll still see you around :)14:16
thomasemjd__, I bet so! :)14:16
gibijd__: if you have any comment / question regarding our colander investigation do not hesitate to ask. :)14:18
*** kwhitney has quit IRC14:19
ildikov_jd__: also as we saw it, jsonschema is used in tests in other projects and Colander seems to be a bit heavy weight for changing jsonschema to it in those simple tests14:19
jd__ildikov_: that's not really a concern14:20
jd__I'm more interested to see why it's not a good candidate14:21
jd__and it seems you have valid points indeed14:21
jd__that'll help me digging a bit more around the subject :)14:21
ildikov_jd__: we hoped it will help a bit, as maybe currently the complex structures are not so popular, but it can change any time14:22
jd__I'm not sure I want "complex structures" to become popular OTOH :))14:23
gibijd__: a bit hard to make the complex query filters without complex structures14:23
jd__I guess it depends on the value of "complex", gibi14:24
gibijd__: of course :)14:24
ildikov_jd__: I thought so, but as so many new projects come into OpenStack, you can never know, what they will need, and as gibi jus said, we also do not have much chance here :)14:24
jd__though I acknowledge that not having something like OneOf() is likely a problem14:24
*** gordc has joined #openstack-ceilometer14:26
ildikov_jd__: btw, another question, not especially for Colander14:26
gibijd__: I think recursion and OneOf on node type is unavoidable for the filter14:26
ildikov_jd__: we've stucked a bit with our API design after the comments of dhellmann14:27
jd__gibi: yeah that makes sense to me14:27
*** eglynn is now known as eglynn-lunch14:29
jd__ildikov_: was that the question?14:29
ildikov_jd__: the question is that how we should go on as there were some comments about the design itself as JSON or python expression should be used14:30
Alexei_987jd__: Hi don't you mind if I'll rebase you Run against MySQL patch?14:30
Alexei_987jd__: I've implemented cleaner fix for common.session issue14:31
jd__ildikov_: I honestly with I could answer but I didn't have time to map Doug's comment and the code to move things forward so far14:31
jd__s/with/wish/14:31
jd__Alexei_987: please, go ahead14:31
jd__Alexei_987: did you take a look at the oslo.db original's bug?14:31
Alexei_987yes but the problem is that we cannot change oslo itself cause it will affect many projects14:32
Alexei_987at least we cannot change it now14:32
Alexei_987I've implemented a local fix14:32
Alexei_987more or less clean14:32
ildikov_jd__: the main point of the comment was to use something python expression like syntax for our query expressions instead of JSON14:34
ildikov_jd__: we just would like to know your opinion also14:35
jd__Alexei_987: that's the point of the incubator, to break things, so go ahead please :)14:36
ildikov_jd__: I know the timing is bad right now, I can add a topic about this to the meeting agenda, if it makes sense to you to bring the discussion there14:36
Alexei_987jd__: unfortunately that's not an easy one :(14:36
jd__ildikov_: I'm not sure I understand what python expression syntax is about14:36
Alexei_987jd__: the correct fix would be to remove this singleton object14:36
Alexei_987jd__: and it means huge refactoring of all projects14:37
jd__ildikov_: ho if we need to discuss this, it's definitely on the list considering the complexity14:37
jd__Alexei_987: I just want to fix it in oslo14:37
jd__Alexei_987: oslo.db is never going to get out of incubation without that done14:37
Alexei_987jd__: the problem is that it's only broken for us14:37
Alexei_987jd__: cause we open several connections in parallel14:38
jd__Alexei_987: oh no, it's broken for any sane program14:38
Alexei_987jd__: true but openstack isn't one :)14:38
jd__yeah, welcome to computer 101 where one should be allowed to do this :)14:38
jd__with such a design flaw it's not getting out anytime soon14:39
Alexei_987jd__:  do you agree that correct solution would be to remove Singleton from there?14:39
Alexei_987and let caller code to store engine objects14:39
jd__Alexei_987: the global variable _ENGINE and the use of the global cfg.CONF yeah14:39
Alexei_987ok14:39
jd__from what I saw14:39
jd__at least that would be a really good start14:39
Alexei_987jd__: to do that we'll need to update all projects that use this lib14:40
jd__Alexei_987: though your work-around might be still good enough for now anyway14:40
jd__Alexei_987: not a problem, again, that's what incubator is for14:40
ildikov_jd__: we do not have much chance to reduce code complexity, maybe we can split the codebase to more patch sets, with several not implemented errors in the first ones, to get smaller patch sets that need to be reviewed14:40
Alexei_987jd__: ok we'll try to think about how this can be done14:41
Alexei_987anyway my patch will be needed14:41
Alexei_987it will only become 4 lines smaller in such case14:41
jd__ildikov_: I don't know how/if it's possible, but that would likely help yeah14:42
jd__Alexei_987: great14:43
ildikov_jd__: is it ok to add the API design question to the meeting agenda for this week, or more likely to the one next week, regarding to the i-2 release?14:46
*** kwhitney has joined #openstack-ceilometer14:46
ildikov_jd__: and then we can discuss if the JSON format is acceptable or not, should we split the code to more patch sets, etc...14:46
jd__ildikov_: sure go ahead14:46
ildikov_jd__: ok, thanks, then I will add it to the next week meeting, as I assume, there will be enough things to discuss about i-2 this Thursday14:48
*** SergeyLukjanov is now known as SergeyLukjanov_a14:51
jd__indeed14:52
*** SergeyLukjanov_a is now known as SergeyLukjanov_14:52
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Refactored session access  https://review.openstack.org/6785015:08
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Handle engine creation inside of Connection object  https://review.openstack.org/6785115:08
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Convert alarm timestamp to PrecisionTimestamp  https://review.openstack.org/6313115:08
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Set the SQL Float precision  https://review.openstack.org/6313215:08
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Run unit tests against MySQL  https://review.openstack.org/5948915:08
Alexei_987jd__: I've accidentally re-uploaded approved changes *facepalm* :(15:09
Alexei_987jd__: However you can now review my fix in this 2 commits15:10
jd__Alexei_987: if you just rebased them it shouldn't be a problem15:10
Alexei_987jd__: it seems that they need to be approved once again to push them to zuul15:10
jd__just did15:10
Alexei_987really sorry for that :(15:11
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Aggregation for SQL backend  https://review.openstack.org/6785615:18
*** SergeyLukjanov_ is now known as SergeyLukjanov15:19
*** deform has joined #openstack-ceilometer15:26
tongli@nadya, ping.15:28
nprivalovahi tongli!15:31
*** galitskaja has joined #openstack-ceilometer15:31
*** nprivalova is now known as nadya_15:31
tongli@nprivalova, I remember you did some work to expand the query so that some complex queries are supported.15:32
tongli@nadya, are these patches merged?15:32
*** deform has quit IRC15:32
nadya_tongli, we have two ideas: about filtering on services (e.g. get all instances launched by Heat) and 'complex queries' on existing data. The second variant is here https://review.openstack.org/#/q/status:open+project:openstack/ceilometer+branch:master+topic:bp/complex-filter-expressions-in-api-queries,n,z . The first variant is not started yet15:35
ildikov_tongli: hi15:38
ildikov_tongli: if you need the complex query functionality that is implemented in the patches, linked by nadya_, then it seems that they will not be merged for i-215:39
ildikov_tongli: the details of the new feature is on this wiki page: https://wiki.openstack.org/wiki/Ceilometer/ComplexFilterExpressionsInAPIQueries15:40
*** blackpr has joined #openstack-ceilometer15:45
*** galitskaja has quit IRC15:51
*** blackpr has quit IRC15:53
tongli@ildikov_, thanks for these pointers and comments.15:56
ildikov_tongli: np, if you have any questions, feel free to ask16:02
*** SergeyLukjanov is now known as SergeyLukjanov_16:09
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: StringIO compatibility for python3  https://review.openstack.org/6786616:12
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Use aggregated values  https://review.openstack.org/6787016:21
*** eglynn-lunch is now known as eglynn16:35
*** s2r2_ has joined #openstack-ceilometer16:36
*** ruhe is now known as _ruhe16:59
*** sayali has joined #openstack-ceilometer17:08
*** vrovachev has left #openstack-ceilometer17:12
*** jaypipes has joined #openstack-ceilometer17:19
*** ildikov_ has quit IRC17:22
*** yassine has quit IRC17:27
*** Alexei_987 has quit IRC17:33
*** s2r2_ has quit IRC17:35
*** _ruhe is now known as ruhe17:45
*** SergeyLukjanov_ is now known as SergeyLukjanov17:52
*** eglynn has quit IRC17:55
*** ruhe is now known as _ruhe17:57
*** boris-42 has quit IRC18:00
*** boris-42 has joined #openstack-ceilometer18:04
*** ildikov has joined #openstack-ceilometer18:14
*** tongli has quit IRC18:15
*** nadya__ has joined #openstack-ceilometer18:16
*** jaypipes has quit IRC18:16
*** nadya__ has quit IRC18:17
*** ildikov_ has joined #openstack-ceilometer18:17
*** s2r2_ has joined #openstack-ceilometer18:20
*** SergeyLukjanov is now known as SergeyLukjanov_a18:24
*** SergeyLukjanov_a is now known as SergeyLukjanov_18:25
*** s2r2_ has quit IRC18:26
*** s2r2_ has joined #openstack-ceilometer18:28
*** jaypipes has joined #openstack-ceilometer18:37
*** s2r2__ has joined #openstack-ceilometer18:45
*** s2r2_ has quit IRC18:48
*** SergeyLukjanov_ is now known as SergeyLukjanov19:02
*** thomasem has quit IRC19:28
*** SergeyLukjanov is now known as SergeyLukjanov_a19:31
*** SergeyLukjanov_a is now known as SergeyLukjanov_19:32
*** SergeyLukjanov_ is now known as SergeyLukjanov19:35
*** nadya__ has joined #openstack-ceilometer19:37
*** julienvey_ has joined #openstack-ceilometer19:42
*** vshukina has joined #openstack-ceilometer19:49
*** nadya__ has quit IRC19:50
*** _ruhe is now known as ruhe19:59
*** nadya__ has joined #openstack-ceilometer20:05
*** prizyvnik has joined #openstack-ceilometer20:06
*** nadya__ has quit IRC20:10
*** prizyvnik has quit IRC20:13
*** thomasem has joined #openstack-ceilometer20:17
*** SergeyLukjanov is now known as SergeyLukjanov_20:22
*** sayali has quit IRC20:41
*** adriant has joined #openstack-ceilometer20:42
*** retroauto has joined #openstack-ceilometer20:56
*** gpocentek has joined #openstack-ceilometer20:57
*** vshukina has quit IRC20:57
gpocentekgood evening21:01
gpocentekI'm working on a doc patch for the installation guide, and a discussion came up about the ceilometer config for glance : http://docs.openstack.org/havana/install-guide/install/apt/content/ceilometer-install-glance.html21:02
gpocentekit seems that the notifier_strategy in glance-api.conf is not really needed21:02
gpocentekcould semeone confirm this (or not)? Thanks21:02
*** vihareva has joined #openstack-ceilometer21:03
*** s2r2__ has quit IRC21:05
*** ildikov has quit IRC21:09
*** jaypipes has quit IRC21:12
*** vihareva has quit IRC21:14
*** denwer has joined #openstack-ceilometer21:17
*** retroauto has quit IRC21:17
*** ruhe is now known as _ruhe21:34
*** asalkeld has joined #openstack-ceilometer21:48
*** denwer has quit IRC21:51
*** jaypipes has joined #openstack-ceilometer21:52
*** asalkeld has quit IRC21:57
*** asalkeld has joined #openstack-ceilometer21:58
gpocentekok, going to answer myself, notifier_strategy=rabbit is needed22:05
gpocentekaother question, I didn't get how to use the new notification_driver option in icehouse, any pointer would be apprciated, thanks!22:06
*** jdob has quit IRC22:10
*** thomasem has quit IRC22:33
*** sandywalsh has joined #openstack-ceilometer22:37
adriantAnyone know who'd be the best person to email in regards to transformer+pipeline questions?22:48
adriantBecause normally my working hours are times when this channel is fairly quiet, or the people I should talk to are sleeping/away. :(22:51
*** asalkeld has quit IRC23:50

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!