Tuesday, 2014-01-21

*** julienvey_ has quit IRC00:13
*** gordc has quit IRC00:16
*** sandywalsh has quit IRC00:42
*** asalkeld has joined #openstack-ceilometer01:10
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Drop foreign key contraints of alarm in sqlalchemy  https://review.openstack.org/6501801:14
openstackgerritJia Dong proposed a change to openstack/ceilometer: Implement meter query by 'counter_volume' field  https://review.openstack.org/6738401:24
*** jaypipes has quit IRC01:37
*** flwang has quit IRC02:02
*** gordc has joined #openstack-ceilometer02:12
*** gordc has quit IRC02:40
*** flwang has joined #openstack-ceilometer02:46
*** asalkeld has quit IRC02:51
*** flwang has quit IRC03:16
*** flwang has joined #openstack-ceilometer03:16
*** asalkeld has joined #openstack-ceilometer03:45
*** asalkeld has left #openstack-ceilometer03:45
openstackgerritZhiQiang Fan proposed a change to openstack/ceilometer: Removes use of timeutils.set_time_override  https://review.openstack.org/6782604:22
*** adriant has quit IRC04:53
*** SergeyLukjanov_ is now known as SergeyLukjanov05:06
*** adriant has joined #openstack-ceilometer05:16
*** asalkeld has joined #openstack-ceilometer05:22
*** asalkeld has left #openstack-ceilometer05:24
*** ildikov_ has quit IRC06:01
*** SergeyLukjanov is now known as SergeyLukjanov_a06:12
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280806:13
*** SergeyLukjanov_a is now known as SergeyLukjanov_06:13
*** SergeyLukjanov_ is now known as SergeyLukjanov06:42
*** gcb has joined #openstack-ceilometer07:18
*** gcb has left #openstack-ceilometer07:23
*** SergeyLukjanov is now known as SergeyLukjanov_07:29
*** nadya__ has joined #openstack-ceilometer07:31
*** nadya__ has quit IRC07:31
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Added hardware agent's inspector and snmp implementation  https://review.openstack.org/4307307:39
*** ildikov_ has joined #openstack-ceilometer07:51
*** s2r2_ has joined #openstack-ceilometer08:05
*** _ruhe is now known as ruhe08:11
*** julienvey_ has joined #openstack-ceilometer08:19
*** julienvey_ has quit IRC08:28
openstackgerritDazhao Yu proposed a change to openstack/ceilometer: Make sure use IPv6 sockets for ceilometer in IPv6 environment  https://review.openstack.org/6778608:29
openstackgerritDazhao Yu proposed a change to openstack/ceilometer: Make sure use IPv6 sockets for ceilometer in IPv6 environment  https://review.openstack.org/6778608:31
openstackgerritLianhao Lu proposed a change to openstack/ceilometer: Added hardware pollsters for the central agent  https://review.openstack.org/4307408:31
lluanyone around? https://review.openstack.org/58489 needs approval. It was approved before, but have to go the 2nd round due to some merge conflict.08:32
*** SergeyLukjanov_ is now known as SergeyLukjanov08:37
*** yassine has joined #openstack-ceilometer08:47
nadya_Ceilometer team, we need approval https://review.openstack.org/#/c/55276/30 . Do we have Tempest core team members here :)?08:52
*** krast has joined #openstack-ceilometer08:57
*** boris-42 has quit IRC09:00
*** boris-42 has joined #openstack-ceilometer09:00
*** s2r2_ has quit IRC09:01
*** Alexei_987 has joined #openstack-ceilometer09:18
Alexei_987jd__: Hi. I can answer your questions about my patch if needed09:19
jd__Alexei_987: "later this code will be valid" is never going to convince anyone09:24
jd__:)09:24
Alexei_987jd__: I don't understand why you want to use static methods everywhere09:24
Alexei_987jd__: It don't make much sense for me09:24
jd__Alexei_987: because that's correct09:24
Alexei_987why?09:24
Alexei_987my IMHO that statis methods are the methods that are static by design and are not connected logically to any class instance09:25
Alexei_987s/statis/static09:25
Alexei_987why should I make method static and in the next commit make it non static?09:26
Alexei_987especially in python where static is just a way to drop self variable09:26
jd__Alexei_987: http://julien.danjou.info/blog/2013/guide-python-static-class-abstract-methods09:27
Alexei_987Ok I've read it... "seeing @staticmethod, we know that the method does not depend on the state of object itself;" that's exactly my point this method does depend on the state of the object09:30
Alexei_987by design09:30
jd__Alexei_987: it does not since it's not using self09:30
Alexei_987but it does "by design"09:30
Alexei_987even not using self09:30
jd__wat?09:30
jd__that doesn't make any sense09:31
Alexei_987I've created it to connect db session to Connection object09:31
jd__the method is not different for each instance09:31
jd__so it's _static_09:31
jd__i.e. you could put it outside the class itself, it wouldn't make any difference09:31
Alexei_987*facepalm* ok so how should I fix it?09:31
Alexei_987make it static and in the next patch make it non static?09:31
jd__lol09:31
jd__yes!09:31
Alexei_987*double-facepalm*09:32
Alexei_987ok09:32
jd__but as I said, the next patch should likely remove that methods09:32
jd__-s09:32
jd__(I wrote that in the review)09:32
Alexei_987yes but do you think it will improve readability?09:32
jd__readability is subjective, I don't argue on that09:32
jd__I argue on code correctness09:32
Alexei_987self._get_db_session.close_all()09:32
Alexei_987and I do argue about readability09:33
jd__then do it alone :)09:33
Alexei_987user will have to read __init__ method09:33
jd__s/user/developer/09:33
Alexei_987user - developer who uses your class09:33
Alexei_987and he will also have to read several other methods to find out how it can be used09:34
Alexei_987so I would like to leave it :)09:34
Alexei_987ok I'll update the patches in 30 minutes..09:35
nadya_jd__, hi! I've already asked you about parallel tests for dbs. You said that for Mongo and SQL the separate db is created. But I cannot find it in the code :( Could you please help me?09:46
silehtnadya_, on bottom of ceilometer/tests/db.py you have the scenarions that define each db backends and some of them create a random database_connection uri by implementing the __str__ method09:50
nadya_sileht, thanks you a lot! My colleagues have started to work on another OS project and they are trying to create pecan tests. Ceilometer is a good example09:53
llujd__:  https://review.openstack.org/58489 needs re-approval. It was approved before, but have to go 2nd round due to some merge conflict09:54
jd__nadya_: ceilometer/tests/db.py check the class used for mongo, we use a different db each time09:54
jd__for sql, the patches are still under review09:54
jd__llu: taking care thx09:54
llujd__: thx hope no need for another round. Seems a very very long gate queue09:55
jd__60 hours :/09:55
llujd__: and about the -2 https://review.openstack.org/58747, can you give me a brief description about what's your concern of the 'generalization issue'? Can I add this topic into this week's IRC meeting for further discussion?09:58
jd__llu: yeah, I think you send a mail about that, didn't have time to reply yet09:58
llujd__: ok, i'll add that topic into this week's IRC meeting schedule, for further discussion.10:00
ruhejd__: speaking of database stuff. what would you say if we make all the DB tests opportunistic? that's how they run tests for nova migrations - run tests for specific DB only if DB is present, otherwise skip10:02
ruhewith exception to sqlite of course10:03
jd__ruhe: did you check the work we did and that is under review in this regard?10:03
ruhejd__: i see 2 patches from you to run mysql and postgres. but the still raise exception if url is not found in env. is there another patch?10:04
jd__ruhe: no10:05
jd__ruhe: as you can see in the patch, not raising an exception if the URL is not set is pretty trivial, the big challenge is to have this patches pass first anyway10:05
ruheagree10:07
*** sileht has quit IRC10:11
*** Alexei_9871 has joined #openstack-ceilometer10:11
ruhejd__: i was going to introduce a patch which would make mongodb optional for ceilometer tests10:12
Alexei_9871jd__: sorry I have another stupid question10:12
*** Alexei_987 has quit IRC10:12
Alexei_9871jd__: If I'll make _get_db_session static does this means that I should also use it as static and don't change other methods to be non-static?10:13
jd__ruhe: that wouldn't help, what would help is splitting test so we can run things independently rather, sileht has been working on that a bit already10:13
jd__Alexei_9871: no, you need to use self.method() to use it, otherwise that would break inheritance10:14
jd__Alexei_9871: if I undertand your question correctly10:14
Alexei_9871no10:14
Alexei_9871https://review.openstack.org/#/c/67850/1/ceilometer/storage/impl_sqlalchemy.py10:14
Alexei_9871I remove static method decorator from  def _real_record_metering_data(self, data): for example10:15
Alexei_9871to use _get_db_session from self10:15
ruhejd__: ok, thank you. i'll see if there is smth i could help with10:15
Alexei_9871but it doesn't makes sense in case _get_db_session is actually static10:15
*** Alexei_9871 has left #openstack-ceilometer10:16
*** Alexei_9871 has joined #openstack-ceilometer10:16
*** sileht has joined #openstack-ceilometer10:21
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Handle engine creation inside of Connection object  https://review.openstack.org/6785110:28
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Refactored session access  https://review.openstack.org/6785010:28
*** flwang has quit IRC10:30
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Handle engine creation inside of Connection object  https://review.openstack.org/6785110:33
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Refactored session access  https://review.openstack.org/6785010:33
*** krast has quit IRC10:34
*** krast has joined #openstack-ceilometer10:36
*** Alexei_9871 has left #openstack-ceilometer10:36
*** boris-42 has quit IRC10:38
*** asalkeld has joined #openstack-ceilometer10:38
*** yassine has quit IRC10:38
*** ruhe is now known as _ruhe10:39
*** Alexei_9871 has joined #openstack-ceilometer10:41
*** Alexei_9871 has left #openstack-ceilometer10:41
*** yassine has joined #openstack-ceilometer10:42
*** yassine has quit IRC10:50
*** yassine has joined #openstack-ceilometer10:50
*** _ruhe is now known as ruhe10:54
*** Alexei_9871 has joined #openstack-ceilometer10:55
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Fix the Alarm documentation of Web API V2  https://review.openstack.org/6807110:55
*** yassine has quit IRC10:57
*** Alexei_9871 is now known as Alexei_98710:58
*** kwhitney has quit IRC11:03
*** yassine has joined #openstack-ceilometer11:10
*** kwhitney has joined #openstack-ceilometer11:11
*** sayali has joined #openstack-ceilometer11:12
*** SergeyLukjanov is now known as SergeyLukjanov_11:15
*** SergeyLukjanov_ is now known as SergeyLukjanov11:19
ildikov_jd__: ping11:19
*** yassine has quit IRC11:21
*** yassine has joined #openstack-ceilometer11:23
*** eglynn has joined #openstack-ceilometer11:28
*** s2r2_ has joined #openstack-ceilometer12:01
*** ruhe is now known as _ruhe12:05
*** boris-42 has joined #openstack-ceilometer12:05
*** _ruhe is now known as ruhe12:11
*** s2r2_ has quit IRC12:13
eglynnjd__: good afternoon, sir!12:20
eglynnjd__: WRT the Atlanta summit CFP ... thinking about a ceilo submission?12:20
eglynnI had the semi-formed notion to propose a 'Quo Vadis Ceilometer?' style session12:21
eglynn(a lot acheived, loads more to do, here's we're thinking of heading next at a high level, yada yada ...)12:21
eglynnmeh! ... /me mixes latin with greek etymology12:22
eglynnclassy-like ... as we say in Dublin ;)12:22
eglynnjd__: anyhoo, if you've an interest in doing a joint preso as per HK, gimme a shout ...12:23
jd__eglynn: why not :)12:27
jd__no idea on the topic per say, but we can probably do one12:28
eglynnjd__: cool! ... how does the 'what next' idea above grab you?12:28
*** flwang has joined #openstack-ceilometer12:29
eglynn... could be a placeholder for many different things ;)12:29
* eglynn feels an etherpad coming on ... ;)12:31
eglynnone sec ...12:31
eglynnhttps://etherpad.openstack.org/p/QuoVadisCeilo12:33
eglynn jd__: feel free to hack on that ^^^ ... change the language or even topic if you've a better idea12:33
jd__eglynn: that looks good12:35
eglynnjd__: awesome sauce! ... I'll go ahead and propose with us both as co-presenters12:36
eglynn... unless nijaba would also like to reprise his MC role?12:37
jd__eglynn: I think we'll be enough :)12:38
eglynnjd__: grandeo, I'll pull the trigger on it so12:38
*** sayali has quit IRC12:41
*** sayali has joined #openstack-ceilometer12:42
*** asalkeld has quit IRC12:51
*** xuhanp has joined #openstack-ceilometer12:52
eglynnjd__: FYI ... http://www.openstack.org/summit/openstack-summit-atlanta-2014/call-for-speakers/TalkDetails/70812:52
jd__ack :)12:53
*** SergeyLukjanov is now known as SergeyLukjanov_a13:02
*** SergeyLukjanov_a is now known as SergeyLukjanov_13:03
*** asalkeld has joined #openstack-ceilometer13:03
*** asalkeld has quit IRC13:04
*** SergeyLukjanov_ is now known as SergeyLukjanov13:09
*** eglynn is now known as eglynn-lunch13:17
*** AMike has quit IRC13:19
*** Alexei_987 has quit IRC13:30
*** jdob has joined #openstack-ceilometer13:31
*** ruhe is now known as _ruhe13:42
*** _ruhe is now known as ruhe13:45
*** thomasem has joined #openstack-ceilometer13:52
*** tongli has joined #openstack-ceilometer13:54
*** flwang has quit IRC13:59
*** jdob_ has joined #openstack-ceilometer14:08
*** sayali has quit IRC14:09
*** yassine has quit IRC14:21
*** yassine has joined #openstack-ceilometer14:22
*** gordc has joined #openstack-ceilometer14:27
nadya_gordc, hi!14:27
gordcnadya_: hi, whatsup?14:27
nadya_gordc, I decided to get rid of recursive_keypairs instead of dict_to_keyval. To make query by list items available. and change flatten_metadata in next API. what do you think? I've created a patch already14:29
gordcchange flatten_metadata in next API... you mean a v3 API?14:31
nadya_gordc, yes14:33
gordcnadya_: that seems fine for retreiving data.14:34
*** jdob_ has quit IRC14:34
gordci think we need to look at how lists are currently being saved in db when posting samples. if it's broken already then it's not really an issue. but if it's storing samples in a certain way right now we might need to do some db migration.14:34
nadya_gordc, I think it will be clear for you from my patch. I will publish it in 5 min14:35
gordcok cool. i'll take a look at that.14:35
gordci doubt it'll get into icehouse-2 based on how slowly patches are merging.14:36
*** SergeyLukjanov is now known as SergeyLukjanov_a14:42
*** SergeyLukjanov_a is now known as SergeyLukjanov_14:43
*** prad has joined #openstack-ceilometer14:44
*** prad has quit IRC14:46
*** SergeyLukjanov_ is now known as SergeyLukjanov14:50
nadya_ gordc, looks like I cannot upload new change :( I need https://review.openstack.org/#/c/52670/11 merged first. Cannot rebase in a proper way without update approved patch14:53
*** boris-42 has quit IRC14:54
*** terriyu has joined #openstack-ceilometer14:57
*** flwang has joined #openstack-ceilometer15:01
*** chmouel has quit IRC15:05
*** chmouel_ has joined #openstack-ceilometer15:05
*** chmouel_ is now known as chmouel15:06
*** prad has joined #openstack-ceilometer15:07
*** IvanBerezovskiy has left #openstack-ceilometer15:08
*** yassine has quit IRC15:08
*** yassine has joined #openstack-ceilometer15:09
gordcnadya_: np. we can wait on the patch anyways.15:10
openstackgerritA change was merged to openstack/ceilometer: Name and unit mapping for rate_of_change transformer  https://review.openstack.org/6665715:16
*** eglynn-lunch is now known as eglynn15:17
*** yassine has quit IRC15:35
*** yassine has joined #openstack-ceilometer15:35
nadya_eglynn, hi! Sorry for delay in aggregation activity. I've added some comments here https://etherpad.openstack.org/p/ceilometer-aggregation and created two patches for sql https://review.openstack.org/#/c/67856/ and https://review.openstack.org/#/c/67870/ . I think it is more easy for reviewers to start with normal SQL instead of HBase. Please, take a look if you have time. I need the decision about this functionality, because I want to be sur15:40
eglynncool, I'll look at the etherpad and sql patch by EoD15:40
eglynnnadya_: ^^^15:40
*** adriant has quit IRC15:41
nadya_eglynn, ok, thanks! I will answer any of your concerns faster then before, because it has the highest priority for me15:44
*** sandywalsh has joined #openstack-ceilometer15:50
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: eventlet: stop monkey patching  https://review.openstack.org/6814515:50
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Allow pytidylib to be installed from external source  https://review.openstack.org/6814915:54
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Use swift master  https://review.openstack.org/6815015:54
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: tests: kill all started processes on exit  https://review.openstack.org/6550915:54
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: storage: bases of a Cassandra driver  https://review.openstack.org/6277915:54
*** sandywalsh has left #openstack-ceilometer15:57
*** SergeyLukjanov is now known as SergeyLukjanov_15:57
*** kwhitney has quit IRC16:02
*** boris-42 has joined #openstack-ceilometer16:02
*** xuhanp has quit IRC16:10
*** prad has quit IRC16:11
*** prad has joined #openstack-ceilometer16:12
*** kwhitney has joined #openstack-ceilometer16:14
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Aggregation for SQL backend  https://review.openstack.org/6785616:40
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Use aggregated values when get statistics  https://review.openstack.org/6787016:40
openstackgerritBalazs Gibizer proposed a change to openstack/ceilometer: Move enable_acl and debug config to ceilometer.conf  https://review.openstack.org/6816616:42
openstackgerritgordon chung proposed a change to stackforge/pycadf: adjust typeURI to capture target better  https://review.openstack.org/6816916:50
eglynnnadya_: FYI I've left some further comments on etherpad and gerrit, I'll check back later17:00
nadya_eglynn, thank you very much! I need to go now. Will take a look tomorrow17:01
eglynnnadya_: sure, no rush! ... chat further tmrw so17:02
*** eglynn is now known as eglynn-afk17:03
*** s2r2_ has joined #openstack-ceilometer17:06
*** ildikov_ has quit IRC17:18
*** SergeyLukjanov_ is now known as SergeyLukjanov17:22
*** eglynn-afk is now known as eglynn17:33
eglynnjd__, dhellmann, sileht, gordc: this i-2 targeted patch is still a +1 short of getting over the line in time for the milestone: https://review.openstack.org/6716117:39
eglynn(if any core has some spare review cycles today ...)17:39
eglynnthanks!17:39
*** ildikov has joined #openstack-ceilometer17:50
*** eglynn has quit IRC17:51
*** ruhe is now known as _ruhe17:57
*** s2r2_ has quit IRC18:00
*** nadya__ has joined #openstack-ceilometer18:09
openstackgerritMarc Solanas proposed a change to openstack/python-ceilometerclient: Checking the type of auth_token, and fixing it if necessary  https://review.openstack.org/5805118:11
*** nadya__ has quit IRC18:18
*** sayali has joined #openstack-ceilometer18:27
*** sayali_ has joined #openstack-ceilometer18:27
*** sayali has quit IRC18:27
*** xmltok has joined #openstack-ceilometer18:30
*** ildikov_ has joined #openstack-ceilometer18:42
*** Alexei_987 has joined #openstack-ceilometer18:44
*** ildikov_ has quit IRC18:47
*** sayali_ has quit IRC19:06
*** ok_delta has joined #openstack-ceilometer19:08
*** ok_delta__ has joined #openstack-ceilometer19:08
*** terriyu has quit IRC19:11
*** julienvey_ has joined #openstack-ceilometer19:11
*** julienvey_ has quit IRC19:16
openstackgerritEoghan Glynn proposed a change to openstack/ceilometer: Exclude weak datapoints from alarm threshold evaluation  https://review.openstack.org/6716119:30
*** ildikov_ has joined #openstack-ceilometer19:38
*** nadya__ has joined #openstack-ceilometer20:01
*** nadya__ has quit IRC20:15
*** prad_ has joined #openstack-ceilometer20:16
*** prad has quit IRC20:17
*** prad_ is now known as prad20:17
openstackgerritJenkins proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/6823720:23
*** s2r2_ has joined #openstack-ceilometer20:27
*** jdob has quit IRC20:35
*** jdob has joined #openstack-ceilometer20:36
*** jdob has quit IRC20:47
*** jdob has joined #openstack-ceilometer20:47
*** tongli has quit IRC21:01
*** gordc has quit IRC21:04
*** prad has quit IRC21:05
*** zul has quit IRC21:06
*** ildikov has quit IRC21:08
*** s2r2_ has quit IRC21:16
*** prad has joined #openstack-ceilometer21:20
*** gordc has joined #openstack-ceilometer21:25
*** asalkeld has joined #openstack-ceilometer21:44
*** jdob has quit IRC21:58
*** asalkeld has quit IRC22:20
*** yassine has quit IRC22:23
*** prad has quit IRC22:27
*** prad has joined #openstack-ceilometer22:28
*** gordc has quit IRC22:30
*** thomasem has quit IRC22:31
*** asalkeld has joined #openstack-ceilometer22:33
*** yassine has joined #openstack-ceilometer22:41
*** SergeyLukjanov is now known as SergeyLukjanov_a22:50
*** SergeyLukjanov_a is now known as SergeyLukjanov_22:51
*** prad has quit IRC22:56
*** jaypipes has joined #openstack-ceilometer23:03
*** xmltok has quit IRC23:39
*** flwang has quit IRC23:42
*** gordc has joined #openstack-ceilometer23:44
*** flwang has joined #openstack-ceilometer23:57

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