Monday, 2014-02-17

« Sunday, 2014-02-16 Index Tuesday, 2014-02-18 »
*** Yathi has quit IRC00:10
*** Yathi has joined #openstack-ceilometer00:13
*** Yathi has quit IRC00:18
*** Yathi has joined #openstack-ceilometer00:41
*** Yathi has quit IRC00:45
*** Yathi has joined #openstack-ceilometer00:46
*** Yathi_ has joined #openstack-ceilometer00:47
*** Yathi has quit IRC00:50
*** Yathi_ has quit IRC00:52
*** Yathi has joined #openstack-ceilometer01:05
*** Yathi has quit IRC01:09
*** xianghui has joined #openstack-ceilometer02:02
*** Yathi has joined #openstack-ceilometer02:05
openstackgerritJia Dong proposed a change to openstack/python-ceilometerclient: py3kcompat: remove in python-ceilometerclient  https://review.openstack.org/7351102:10
*** Yathi has quit IRC02:10
openstackgerritFengqian.gao proposed a change to openstack/ceilometer: Keep py3.X compatibility for urllib.urlencode  https://review.openstack.org/7354102:41
openstackgerritFengqian.gao proposed a change to openstack/ceilometer: Keep py3.X compatibility for urllib.urlencode  https://review.openstack.org/7354102:43
*** thomasem has joined #openstack-ceilometer02:59
*** Yathi has joined #openstack-ceilometer03:05
*** Yathi has quit IRC03:10
openstackgerritlitong01 proposed a change to openstack/ceilometer: add notification alarm  https://review.openstack.org/6947303:12
*** flwang has joined #openstack-ceilometer03:39
*** _nadya_ has joined #openstack-ceilometer04:02
*** _nadya_ has quit IRC04:03
*** Yathi has joined #openstack-ceilometer04:05
*** Yathi has quit IRC04:10
*** Yathi has joined #openstack-ceilometer04:33
openstackgerritCyril Roelandt proposed a change to openstack/python-ceilometerclient: test_url_generation_with_proxy: do not use contextlib.nested()  https://review.openstack.org/7362205:02
*** AMike has joined #openstack-ceilometer05:10
*** AMike has joined #openstack-ceilometer05:10
openstackgerritCyril Roelandt proposed a change to openstack/python-ceilometerclient: test_url_generation_with_proxy: do not use contextlib.nested()  https://review.openstack.org/7362205:13
*** Yathi has quit IRC05:27
*** ildikov_ has quit IRC05:49
openstackgerritJenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/6280806:03
*** Yathi has joined #openstack-ceilometer06:09
*** _nadya_ has joined #openstack-ceilometer06:13
*** saju_m has joined #openstack-ceilometer06:15
*** _nadya_ has quit IRC06:21
*** _nadya_ has joined #openstack-ceilometer06:27
*** ildikov_ has joined #openstack-ceilometer06:39
*** jaypipes has quit IRC06:42
*** jaypipes has joined #openstack-ceilometer06:42
*** _nadya_ has quit IRC07:03
*** _nadya_ has joined #openstack-ceilometer07:05
*** _nadya_ has quit IRC07:09
*** mihgen has joined #openstack-ceilometer07:17
*** Alexei_987 has joined #openstack-ceilometer07:38
*** boris-42 has quit IRC07:40
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Fixed DateTime in PostgreSQL  https://review.openstack.org/7309208:03
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Run unit tests against MySQL  https://review.openstack.org/5948908:03
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Fix for get_statistics with postgresql  https://review.openstack.org/5921408:03
openstackgerritAlexei Kornienko proposed a change to openstack/ceilometer: Run tests against PostgreSQL  https://review.openstack.org/6304908:03
openstackgerritliusheng proposed a change to openstack/ceilometer: Modify the discription of combination alarm  https://review.openstack.org/7396608:28
openstackgerritliusheng proposed a change to openstack/ceilometer: Modify the discription of combination alarm  https://review.openstack.org/7396608:30
openstackgerritShuangtai Tian proposed a change to openstack/ceilometer: Adds gettextutils module in converter  https://review.openstack.org/7396808:31
*** saju_m has quit IRC08:31
*** saju_m has joined #openstack-ceilometer08:58
*** flwang has quit IRC09:15
*** yassine has joined #openstack-ceilometer09:17
Alexei_987jd__: Hi09:29
jd__Alexei_987: hi09:29
Alexei_987jd__: question: Can I create a patch removing testscenarios dependency? (reimplement same feature using test.run method)09:29
Alexei_987problem is that testscenarios doesn't work with nosetests09:30
jd__Alexei_987: yes and no; that doesn't depend on Ceilometer, propose that on the mailing list to every project09:32
jd__having a drop in replacement would be great for example09:32
jd__understand that I'm not against the idea, but we're trying to move the project as a whole so don't just look at Ceilometer09:33
Alexei_987jd__: ok I see09:33
*** mihgen has quit IRC09:42
*** eglynn has joined #openstack-ceilometer09:43
*** boris-42_ has joined #openstack-ceilometer09:52
*** sayali has joined #openstack-ceilometer10:11
*** mihgen has joined #openstack-ceilometer10:20
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: api: export recorded_at in returned samples  https://review.openstack.org/7399210:21
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: storage: store recording timestamp  https://review.openstack.org/7016610:21
*** sayali has quit IRC10:38
*** Yathi has quit IRC10:39
*** Yathi has joined #openstack-ceilometer10:41
*** Yathi has quit IRC10:46
*** sayali has joined #openstack-ceilometer10:47
*** xianghui has quit IRC11:00
*** Yathi has joined #openstack-ceilometer11:10
*** flwang has joined #openstack-ceilometer11:10
*** Yathi_ has joined #openstack-ceilometer11:12
*** sayali_ has joined #openstack-ceilometer11:13
*** sayali_ has quit IRC11:14
*** Yathi has quit IRC11:14
*** sayali_ has joined #openstack-ceilometer11:14
*** yassine has quit IRC11:15
*** yassine has joined #openstack-ceilometer11:15
*** Yathi_ has quit IRC11:16
*** sayali has quit IRC11:16
Alexei_987jd__: question about dbmanagers/fixtures11:22
Alexei_987jd__: as I mentioned fixtures may not always teadDown properly11:22
Alexei_987jd__: this will cause out of connections error11:22
Alexei_987jd__: I agree that patch needs some additional work. but I'm not sure that fixtures is the best option here11:23
*** mihgen has quit IRC11:47
*** mihgen has joined #openstack-ceilometer11:55
openstackgerritSwann Croiset proposed a change to openstack/ceilometer: Handle Heat notifications for stack CRUD  https://review.openstack.org/7193912:00
*** Yathi has joined #openstack-ceilometer12:12
*** Yathi has quit IRC12:16
jd__Alexei_987: AFAIK fixtures always calls their tearDown part12:20
Alexei_987jd__: they call it. but it's not guaranteed *when*12:20
jd__I guess it's no in the middle of the test12:21
Alexei_987no..12:21
Alexei_987consider following case:12:21
*** shengjiemin has joined #openstack-ceilometer12:21
Alexei_987test.tearDown calls super.tearDown and then uses db connection12:21
Alexei_987fixture will be shut down (and DB deleted)12:21
Alexei_987and we'll still have connections refering to it12:21
Alexei_987jd__: ^12:22
Alexei_987jd__: this may lead to errors that will be hard to debug12:22
jd__why tearDown() would use the db connection in the first place?12:22
jd__plus AFAIK fixtures uses cleanup which is called after tearDown12:23
jd__so I don't get how that would fail actually12:23
Alexei_987jd__: https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/db.py#L6812:23
jd__that code is going to be part of the fixture12:23
Alexei_987no it won't12:24
Alexei_987fixture is backend specific12:24
Alexei_987and this code is not12:24
Alexei_987jd__: one sec.. let me check fixture cleanup once again12:25
Alexei_987cleanups are running during tearDown, no?12:28
jd__Alexei_987: after12:28
Alexei_987what do mean after?12:28
Alexei_987from where the method is called?12:29
jd__cleanup are run at the end of the test, after tearDown(), even if tearDown() is not called because setUp fails12:29
jd__I just RTFM'ed unittest12:29
jd__fixtures uses addCleanup to clean things12:30
Alexei_987ah you are right12:30
Alexei_987I've found it12:30
openstackgerritBalazs Gibizer proposed a change to openstack/ceilometer: Implements "not" operator for complex query  https://review.openstack.org/6689212:31
openstackgerritBalazs Gibizer proposed a change to openstack/ceilometer: Implements metadata query for complex query feature  https://review.openstack.org/6689112:31
Alexei_987jd__: ok we can use a fixture here12:31
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Alarm support in HBase Part 1  https://review.openstack.org/7401912:46
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Alarms in HBase Part 2  https://review.openstack.org/7402012:46
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Fix work with metadata in HBase  https://review.openstack.org/6864112:46
Alexei_987jd__: can I ask an offtopic question?12:47
*** promulo has quit IRC12:48
openstackgerritIlya Tyaptin proposed a change to openstack/ceilometer: Add ttl to hbase storage  https://review.openstack.org/7359012:48
Alexei_987jd__: what's the point in supporting multiple backends in ceilometer? this approach has many cons and I don't see any valuable pros here12:49
jd__Alexei_987: avoiding arguments about whatever is the best12:51
jd__so precisely avoiding the kind of discussion you're trying to have; if you think one of the driver is the best, just use it12:51
jd__:)12:51
Alexei_987jd__: my opinion is not about the best driver12:52
Alexei_987jd__: trying to support all of them means that we provide bad support for all12:52
jd__same reason Nova supports Xen and KVM and VMware12:52
Alexei_987jd__: instead of focusing on good support for one12:52
jd__that's your opinion12:52
Alexei_987jd__: problem I see is that our model structure doesn't map cleanly on any backend12:52
Alexei_987jd__: and we have to do complex operations to implement it12:53
jd__Alexei_987: then propose something better?12:53
Alexei_987jd__: I'm thinking about it. Maybe I'll have something ready to discuss on summit12:53
jd__cool12:53
*** Yathi has joined #openstack-ceilometer13:10
*** Yathi_ has joined #openstack-ceilometer13:12
*** Yathi has quit IRC13:15
*** Yathi_ has quit IRC13:17
*** eglynn is now known as eglynn-lunch13:17
*** jdob has joined #openstack-ceilometer13:41
*** thomasem has joined #openstack-ceilometer13:54
*** promulo has joined #openstack-ceilometer13:57
*** prad_ has joined #openstack-ceilometer14:01
*** sayali_ has quit IRC14:03
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Fix work with metadata in HBase  https://review.openstack.org/6864114:06
openstackgerritNadya Privalova proposed a change to openstack/ceilometer: Make lists query-able during fetching samples and meters  https://review.openstack.org/6858314:07
*** Yathi has joined #openstack-ceilometer14:10
*** Yathi has quit IRC14:15
*** saju_m has quit IRC14:15
*** jdob has quit IRC14:19
*** jdob has joined #openstack-ceilometer14:19
*** sayali has joined #openstack-ceilometer14:19
gibi_jd__: hi! Do you think it is a valid effort to try to improve the the measurement doc here https://review.openstack.org/#/c/69637/ ?14:23
jd__gibi_: I don't think it's an improvement, CPU time is really well defined and known for centuries http://en.wikipedia.org/wiki/CPU_time14:26
*** sayali has quit IRC14:26
*** sayali has joined #openstack-ceilometer14:27
gibi_jd__: I had a different feeling about this after the irc discussion I linked in the review comment. I agree that the wikipedia is quite clear about the definition but it seems people don't aware of that14:29
nprivalovajd__, https://blueprints.launchpad.net/ceilometer/+spec/hbase-alarming , do I have a chance to see it approved? And is it ok to start with events in HBase?14:29
jd__I really think any sysadmin what CPU time really, so I'll stick to my -1 but you're free to try to convince somebody else :)14:29
jd__nprivalova: sure14:30
gibi_jd__: OK thanks for the clear statement. :)14:30
jd__gibi_: bah my sentence wasn't complete, I meant "I really think any syadmin knows what CPU time is, really"14:31
jd__but I guess you got it14:31
gibi_jd__: I got it thanks14:31
nprivalovajd__, I have a batch of hbase-related stuff on review, could you please advise reviewers :)? As I understand, not every person would like to see this. I'm trying to do refactoring anyway14:33
*** jdob_ has joined #openstack-ceilometer14:37
*** jdob has quit IRC14:37
jd__nprivalova: if you can split in small patches that'll probably help14:37
*** jdob_ is now known as jdob14:38
nprivalovajd__: I've tried. All of them is about 150-200 lines now14:39
jd__nprivalova: should be good enough then14:39
*** sayali has quit IRC15:07
*** rwsu has joined #openstack-ceilometer15:09
*** Yathi has joined #openstack-ceilometer15:10
*** _nadya_ has joined #openstack-ceilometer15:14
*** Yathi has quit IRC15:15
*** gordc has joined #openstack-ceilometer15:15
*** _nadya_ has quit IRC15:19
openstackgerritlitong01 proposed a change to openstack/ceilometer: add notification alarm  https://review.openstack.org/6947315:22
*** shengjiemin has quit IRC15:41
*** boris-42_ has quit IRC16:01
*** Yathi has joined #openstack-ceilometer16:10
*** Yathi has quit IRC16:15
*** urulama has joined #openstack-ceilometer16:15
ildikov_gordc: hi. are you around?16:15
gordcildikov_: sort of. whatsup?16:17
ildikov_gordc: I just wanted to ask, if you saw my answers to your questions on the alarms patch of the complex query16:19
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Refactor timestamp exsitence validation in V2 API  https://review.openstack.org/7380516:21
gordcildikov_: sorry. i hadn't looked at it since i last reviewed. i'll give a quick look.16:22
ildikov_gordc: thanks16:24
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Refactor timestamp exsitence validation in V2 API  https://review.openstack.org/7380516:27
openstackgerritA change was merged to openstack/ceilometer: Rename Openstack to OpenStack  https://review.openstack.org/7317116:33
openstackgerritA change was merged to openstack/ceilometer: Adds gettextutils module in converter  https://review.openstack.org/7396816:33
flwangjd__: ping16:38
flwangeglynn-lunch: any lucky you there?16:39
*** eglynn-lunch is now known as eglynn16:40
eglynnflwang: 'sup?16:40
flwangimage.download Delta B image ID notification Image is downloaded                                   image.serve Delta B image ID notification Image is served out16:41
flwangeglynn: may I know what's the difference between image.download and image.serve?16:41
flwangeglynn: based on my understanding from the code, admin/user can get a sum bytes consumed/served for a specific image with 'image.serve'. And he can also review the statistics of how the specific image is consumed by different tenant, different user, is it?16:43
eglynnflwang: yep, it's same info from the perspective of different roles pertaining to the image16:44
eglynnflwang: i.e. the owner think of his image being served out16:44
eglynnflwang: the user thinks of an image being downloaded16:44
eglynnflwang: the idea IIRC was to enable different charge-back based on those different perspectives16:45
flwangeglynn: got it, like the image provider and consumer POV, right?16:46
eglynnflwang: e.g. downloading might be cheap and serving out less so, or vice versa, depending on the cloud provider's policies16:46
eglynnflwang: yeap16:46
flwangeglynn: cool, thanks for the clarification16:46
eglynnflwang: np!16:46
*** prad_ has quit IRC16:47
*** sayali has joined #openstack-ceilometer16:52
*** mihgen has quit IRC16:54
*** _cjones_ has joined #openstack-ceilometer17:05
*** Alexei_987 has quit IRC17:05
*** jaypipes has quit IRC17:07
*** Yathi has joined #openstack-ceilometer17:10
openstackgerritBalazs Gibizer proposed a change to openstack/ceilometer: Fix evaluation_periods usage in DB2 engine  https://review.openstack.org/7410017:10
*** urulama has left #openstack-ceilometer17:14
*** Yathi has quit IRC17:14
*** _nadya_ has joined #openstack-ceilometer17:34
*** eglynn has quit IRC17:34
*** boris-42_ has joined #openstack-ceilometer17:40
openstackgerritlitong01 proposed a change to openstack/python-ceilometerclient: add notification alarm  https://review.openstack.org/6947417:44
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Refactor timestamp exsitence validation in V2 API  https://review.openstack.org/7380517:50
*** jaypipes has joined #openstack-ceilometer17:53
*** sayali has quit IRC17:55
*** prad_ has joined #openstack-ceilometer17:58
openstackgerritA change was merged to openstack/ceilometer: Removed useless prints that pollute tests log  https://review.openstack.org/7383117:59
*** Yathi has joined #openstack-ceilometer18:10
*** Yathi has quit IRC18:15
*** _cjones_ has quit IRC18:16
*** _cjones_ has joined #openstack-ceilometer18:16
*** Yathi_ has joined #openstack-ceilometer18:17
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Refactor timestamp exsitence validation in V2 API  https://review.openstack.org/7380518:18
*** Yathi_ has quit IRC18:21
openstackgerritIldiko Vancsa proposed a change to openstack/ceilometer: Refactor timestamp exsitence validation in V2 API  https://review.openstack.org/7380518:26
*** ildikov_ has quit IRC18:34
*** _nadya_ has quit IRC18:39
*** dhellmann is now known as dhellmann_18:40
*** openstackgerrit has quit IRC18:53
*** openstackgerrit has joined #openstack-ceilometer18:54
*** ildikov_ has joined #openstack-ceilometer19:04
*** mihgen has joined #openstack-ceilometer19:11
*** ildikov_ has quit IRC19:29
*** _nadya_ has joined #openstack-ceilometer19:38
*** _nadya_ has quit IRC19:52
*** mihgen has quit IRC20:18
*** rbowen has joined #openstack-ceilometer20:45
openstackgerritA change was merged to openstack/ceilometer: Added missing import  https://review.openstack.org/7382820:52
*** gordc has quit IRC20:53
*** gordc has joined #openstack-ceilometer20:59
*** raymondr has joined #openstack-ceilometer21:32
*** promulo has quit IRC21:40
*** jdob has quit IRC21:54
openstackgerritA change was merged to openstack/ceilometer: Implements complex query functionality for alarms  https://review.openstack.org/6215821:55
*** dhellmann_ is now known as dhellmann22:07
*** promulo has joined #openstack-ceilometer22:13
*** raymondr has quit IRC22:40
*** thomasem has quit IRC23:03
*** prad_ has quit IRC23:07
*** raymondr has joined #openstack-ceilometer23:15
*** yassine has quit IRC23:28
*** raymondr has quit IRC23:29
*** raymondr has joined #openstack-ceilometer23:31
*** raymondr_ has joined #openstack-ceilometer23:35
*** raymondr has quit IRC23:35
*** raymondr_ has quit IRC23:41
« Sunday, 2014-02-16 Index Tuesday, 2014-02-18 »

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