Wednesday, 2015-02-18

*** polew has joined #openstack-ceilometer00:01
*** r-daneel has quit IRC00:26
*** david-lyle has quit IRC00:44
*** polew has quit IRC00:52
*** rbak has quit IRC00:57
*** shengjiemin has joined #openstack-ceilometer01:12
*** hhuang has quit IRC01:15
*** liuhang has quit IRC01:35
*** promulo has joined #openstack-ceilometer01:36
*** ddieterly has joined #openstack-ceilometer01:37
*** promulo__ has quit IRC01:40
*** ddieterl_ has joined #openstack-ceilometer01:41
*** ddieterly has quit IRC01:41
*** rwsu is now known as rwsu-afk01:54
*** briancline has quit IRC02:09
*** briancline has joined #openstack-ceilometer02:09
*** changbl has joined #openstack-ceilometer02:10
*** david-lyle has joined #openstack-ceilometer02:11
*** _cjones_ has quit IRC02:18
*** shengjiemin has quit IRC02:34
*** boris-42 has quit IRC03:12
*** himangi has quit IRC03:21
*** harlowja_ is now known as harlowja_away03:22
*** david-lyle has quit IRC03:35
*** david-lyle has joined #openstack-ceilometer03:35
*** david-lyle has quit IRC03:40
*** deepthi has joined #openstack-ceilometer03:55
*** yatin has joined #openstack-ceilometer04:15
*** fnaval has quit IRC04:17
*** david-lyle has joined #openstack-ceilometer04:22
*** ddieterl_ has quit IRC04:23
*** david-lyle is now known as david-lyle_afk04:23
*** fnaval has joined #openstack-ceilometer04:52
*** cmyster has quit IRC04:54
*** _cjones_ has joined #openstack-ceilometer05:04
*** claudiub_ has quit IRC05:08
*** zhenq has quit IRC05:30
*** zhenq has joined #openstack-ceilometer05:30
*** zhenq has quit IRC05:34
*** _cjones_ has quit IRC05:38
*** _cjones_ has joined #openstack-ceilometer05:39
*** cmyster has joined #openstack-ceilometer05:40
*** X019 has joined #openstack-ceilometer05:40
*** ddieterly has joined #openstack-ceilometer05:54
*** ddieterly has quit IRC05:58
openstackgerritOpenStack Proposal Bot proposed openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/15519706:09
*** exploreshaifali has joined #openstack-ceilometer06:17
*** amalagon has joined #openstack-ceilometer06:20
*** ildikov has quit IRC06:22
*** exploreshaifali has quit IRC06:24
*** exploreshaifali has joined #openstack-ceilometer06:24
*** zhenq has joined #openstack-ceilometer06:31
*** exploreshaifali has quit IRC06:33
*** zhenq has quit IRC06:35
*** fnaval has quit IRC06:37
*** ildikov has joined #openstack-ceilometer06:54
*** ddieterly has joined #openstack-ceilometer06:55
*** ddieterly has quit IRC06:59
*** cmyster has quit IRC07:04
*** exploreshaifali has joined #openstack-ceilometer07:18
*** _cjones_ has quit IRC07:23
*** cmyster has joined #openstack-ceilometer07:23
*** _cjones_ has joined #openstack-ceilometer07:24
*** _cjones_ has quit IRC07:28
*** polew has joined #openstack-ceilometer07:46
*** IvanBerezovskiy has joined #openstack-ceilometer07:53
*** ddieterly has joined #openstack-ceilometer07:55
*** ddieterly has quit IRC08:00
*** boris-42 has joined #openstack-ceilometer08:00
*** zhenq has joined #openstack-ceilometer08:20
*** zhenq has quit IRC08:24
*** amalagon has quit IRC08:28
*** Ala has joined #openstack-ceilometer08:29
*** eglynn has joined #openstack-ceilometer08:29
*** himangi has joined #openstack-ceilometer08:34
*** himangi has quit IRC08:34
*** andreluiz has quit IRC08:40
*** andreluiz has joined #openstack-ceilometer08:40
*** ddieterly has joined #openstack-ceilometer08:56
*** ddieterly has quit IRC09:00
*** exploreshaifali has quit IRC09:17
openstackgerritSwami Reddy proposed openstack/ceilometer: Add ceph object storage meters  https://review.openstack.org/15176309:25
*** amalagon has joined #openstack-ceilometer09:39
*** amalagon has quit IRC09:43
*** maishsk has joined #openstack-ceilometer09:45
*** ddieterly has joined #openstack-ceilometer09:57
*** ddieterly has quit IRC10:02
*** exploreshaifali has joined #openstack-ceilometer10:03
*** zhenq has joined #openstack-ceilometer10:09
*** zhenq has quit IRC10:13
*** zhenq has joined #openstack-ceilometer10:34
*** zhenq has quit IRC10:38
*** yatin has quit IRC10:39
*** nellysmitt has joined #openstack-ceilometer10:49
*** Ala has quit IRC10:56
*** Ala has joined #openstack-ceilometer10:57
*** Ala has quit IRC10:57
*** Ala has joined #openstack-ceilometer10:57
*** ddieterly has joined #openstack-ceilometer10:58
*** polew has quit IRC10:58
*** polew has joined #openstack-ceilometer10:58
*** ddieterly has quit IRC11:02
*** kbyrne has quit IRC11:09
*** exploreshaifali has quit IRC11:11
*** kbyrne has joined #openstack-ceilometer11:13
*** cdent has joined #openstack-ceilometer11:13
*** exploreshaifali has joined #openstack-ceilometer11:24
*** exploreshaifali has quit IRC11:28
*** ddieterly has joined #openstack-ceilometer11:59
*** ddieterly has quit IRC12:03
*** zhenq has joined #openstack-ceilometer12:04
*** yatin has joined #openstack-ceilometer12:05
*** zhenq has quit IRC12:09
cdentthe ceilo tests seems to be spewing way more debug info than they used to, or am I'm just misremembering?12:10
ildikovcdent: on the gate?12:14
cdentno, locally12:14
ildikovcdent: I had this experience with elasticsearch tests IIRC12:15
*** nellysmitt has quit IRC12:15
cdentit looks like it might have something to do with network stats12:16
ildikovcdent: well, we have quite lots of them, so that can be a candidate too12:18
*** eglynn has quit IRC12:18
*** eglynn has joined #openstack-ceilometer12:20
*** NellyK has joined #openstack-ceilometer12:23
openstackgerritChris Dent proposed openstack/ceilometer: Destroy fixture database after each gabbi TestSuite  https://review.openstack.org/15696812:30
*** NellyK has quit IRC12:32
*** NellyK has joined #openstack-ceilometer12:33
*** NellyK has quit IRC12:40
*** fwdit has joined #openstack-ceilometer12:43
*** deepthi has quit IRC12:48
*** nellysmitt has joined #openstack-ceilometer12:53
*** ddieterly has joined #openstack-ceilometer12:59
*** yatin has quit IRC13:01
*** fwdit has quit IRC13:02
*** ddieterly has quit IRC13:04
*** fwdit has joined #openstack-ceilometer13:07
*** himangi has joined #openstack-ceilometer13:09
*** fwdit has quit IRC13:10
cdentsigh: https://bugs.launchpad.net/ceilometer/+bug/142317113:13
openstackLaunchpad bug 1423171 in Ceilometer "regex filtering with postgresql backend fails" [Undecided,New]13:13
cdentand that's not the only postgres bug13:13
*** zigo has quit IRC13:18
*** zigo has joined #openstack-ceilometer13:20
cdentjd__: If you've got time, if you could stick your +2 back on my wsme change that would be awesome: https://review.openstack.org/#/c/154066/13:31
*** ildikov has quit IRC13:36
*** gordc has joined #openstack-ceilometer13:37
gordchttps://lobste.rs/s/kjn5an/recommended_reading_for_building_time_series_databases13:41
gordcpretty funny evaluation of current tsdbs13:41
jd__cdent: you got it13:42
gordcjd__: maybe your canonical backend might be best.13:42
cdentthanks jd__13:42
jd__gordc: "The state of the art, apparently, is total shit." cool13:42
jd__the bar seems pretty low, that works for me :D13:42
gordc:)13:43
jd__well at least Carbonara goes down to the nanosecond13:44
cdentHuh, I'm surprised to see kragen cursing so much, he must be disappointed13:44
gordcnanosecond ftw!13:44
gordccdent: do you follow that site? i have zero context except that i googled "kairosdb vs influxdb"13:45
gordcand i found his review so entertaining13:45
cdentNo, I don't, but I do know kragen (the author of the comment)13:45
gordci see... i don't know if it's worth picking his mind? not sure what the current state of influxdb/opentsdb support is in gnocchi.13:47
cdentHe can be a bit hard to find, but I can see if I can dredge him up.13:50
*** himangi has quit IRC13:50
gordccdent: no worries. i was just throwing random idea out there in case he was without shooting distance.13:51
ityaptingordc: I work with support13:51
ityaptingordc oprntsdb and influx in current gnocchi13:51
gordcityaptin: have you noticed any issues? again i should add my entire experience is this (hilarious) thread13:52
cdentgordc: I've tweeted at him, will let you know if I get anything13:54
ityaptingordc: issues with aggregation methods in archive_policy and using a ArchivePolicy class instead dict13:54
gordccdent: you kids and your technology13:55
cdentgordc: I was all keen to ask you something this morning, when you weren't here, and now I can't remember what it was. very frustrating.13:55
jd__cdent: owned by pep813:55
cdentargh13:56
gordccdent: yes, i would like a free porsche. you have your answer13:56
cdentmaybe I just wanted to get you to provide me context on this bit of awesome+sad melange: https://www.openstack.org/vote-vancouver/presentation/openstack-is-doomed-and-it-is-your-fault13:57
cdentanyway: I'll bet you a beer that I'm older than you gordc13:57
gordccdent: TERMIE is back!13:57
cdentjd__: that's bizarre. passes in check but not gate13:58
gordccdent: yeah i know, i'm just joking since i don't understand twitter13:58
* cdent digs13:58
cdentpooh, clearly I need to work on my age ambiance13:58
cdentmy youthful vigor is not properly shining through13:58
cdentah yes, those are new pep8 rules13:59
*** ddieterly has joined #openstack-ceilometer14:00
gordccdent: the best way i can describe termie is would be to riff on a famous quote: some people see things and say why, termie see things and says why not... (and then throws a bomb at it).14:02
cdentthat's a nice description14:03
cdentThere's a lot to agree with in his descrption of doom.14:04
*** exploreshaifali has joined #openstack-ceilometer14:04
gordccdent: didn't read it but i will not argue.14:04
*** ddieterly has quit IRC14:05
linuxhermithow do you get access to lobste.rs?14:05
cdentIt's actually quite positive14:05
cdentlinuxhermit: you mean to comment? I think you have to ask an existing user to invite you14:05
cdentit is trying to create a social graph of responsibility14:05
cdentyou've been vouchsafed in some fashion14:05
* cdent doesn't have an account14:05
linuxhermitahh cdent14:05
linuxhermitdo we talk to postgres via sqlalchemy?14:06
linuxhermitif so I might be able to lend a hand there as well14:06
cdentlinuxhermit: yes14:06
cdentjd__: those problems were in master, so I fixed them: https://review.openstack.org/15700514:12
*** ildikov has joined #openstack-ceilometer14:21
*** ccrouch has joined #openstack-ceilometer14:22
*** ildikov is now known as ildikov_afk14:23
jd__did we lose eglynn this week btw?14:28
gordcjd__: i think he answered a ping on monday... but i might have been dreaming14:29
jd__gordc: do you often dream of people from #openstack-ceilometer?14:29
gordcjd__: i dream of electric sheep14:30
cdenti have dreams of gordc dreaming of electric sheep and there are paper unicorns there14:30
gordci wish there were unicorns... your dream is better.14:31
*** changbl has quit IRC14:32
ityaptin:-D14:32
idegtiarovsileht: hi! probably that patch could be landed https://review.openstack.org/#/c/149259/8 WDYT?14:36
gordcidegtiarov: he's on vacation this week14:39
idegtiarovgordc: thanks!14:39
gordcnp14:39
*** ddieterly has joined #openstack-ceilometer14:42
gordcif anyone is bored, please check out https://review.openstack.org/#/c/13086914:54
cdent"this patch aims to improve write performance" Does it?14:55
gordccdent: see patchset 8 comments for my local "performance" results14:56
openstackgerritMerged stackforge/gnocchi: Add volume resource  https://review.openstack.org/14925914:56
gordccdent: it's been a while since i've looked at this... but i don't think anything has changed.14:56
openstackgerritMerged stackforge/gnocchi: rest: implement complex query for resource listing  https://review.openstack.org/15667814:57
cdenthuh, better, but not awesomely so.14:57
cdentwell, I guess depends on the scale14:58
gordccdent: i'm not claiming miracles. :)14:58
idegtiaroveglynn: thank you for your +A14:58
cdentyou work miracles all the time gordc, given the available materials...14:59
gordccdent: i'm open to readjusting if needed... it's just kinda bothering me that i need to keep rebasing.14:59
eglynnidegtiarov: np! (... apols for the missed ping yesterday, caught up in some internal stuff)14:59
* cdent nods14:59
idegtiaroveglynn: i have one more question, do we need a spec for new transformers that we are going to use in event pipeline14:59
gordccdent: where's that unicorn... i need it for my miracles.15:00
cdentIT'S ONLY IN MY DREAMS15:00
* cdent weeps for the lost miracles15:00
eglynnidegtiarov: well the usual principle would apply ... a spec would be required if the change is complex, intrusive, and/or could benefit from further discussion before any code is written15:01
eglynnidegtiarov: ... and not be needed is it's relatively simple and/or the way forward is self-evident15:01
* gordc pours some booze on the ground... to what could have been but never was15:01
cdentone for me and one for my dead unicorns15:02
idegtiaroveglynn:  cool, will write a spec! ))15:03
gordccdent: noooo!!!! a dark twist.lol15:03
*** rbak has joined #openstack-ceilometer15:04
openstackgerritgordon chung proposed openstack/ceilometer: add an option to disable non-metric meters  https://review.openstack.org/15674715:07
*** Ephur has joined #openstack-ceilometer15:08
*** cdent_ has joined #openstack-ceilometer15:09
*** cdent has quit IRC15:09
*** cdent_ is now known as cdent15:09
openstackgerritgordon chung proposed openstack/ceilometer: add option to store raw notification  https://review.openstack.org/15379015:10
openstackgerritgordon chung proposed openstack/ceilometer: modify events sql schema to reduce empty columns  https://review.openstack.org/13086915:10
gordceglynn: thanks for review. added a response.15:16
eglynngordc: coolness, looking15:17
gordceglynn: thanks! i can finally move the card from my trello board. (hoping it doesn't fail)15:24
*** himangi has joined #openstack-ceilometer15:24
eglynngordc: np!15:26
*** devlaps has joined #openstack-ceilometer15:32
*** exploreshaifali has quit IRC15:32
*** rbak_ has joined #openstack-ceilometer15:35
*** rbak has quit IRC15:37
*** zhenq has joined #openstack-ceilometer15:42
cdentgordc these men are nihilists, there's nothing to be afraid of.15:45
gordccdent: had to google it... i should really watch that movie.15:46
* cdent dies15:46
gordc... and star wars...15:46
cdentmeh15:46
gordcok so big lebowski step one.15:47
gordccitizen kane?15:47
*** mstyne_ is now known as mstyne15:47
cdentnot managed to sit through the whole thing, only seen bits and pieces15:47
gordcbig lebowski it is then. weekend viewing15:48
cdentI expect a full report next week.15:48
gordcwill do :)15:49
*** shengjiemin has joined #openstack-ceilometer15:53
*** polew has quit IRC16:04
openstackgerritJulien Danjou proposed stackforge/gnocchi: rest: replace list+filter by search mechanism  https://review.openstack.org/15704616:07
openstackgerritJulien Danjou proposed stackforge/gnocchi: service: validate default values  https://review.openstack.org/15411116:11
openstackgerritJulien Danjou proposed stackforge/gnocchi: archive_policy: have a sane default list of agg methods  https://review.openstack.org/15410816:11
*** fnaval has joined #openstack-ceilometer16:12
cdentjd__: SEARCH? which RFC has that?16:15
openstackgerritIlya Tyaptin proposed stackforge/gnocchi: Add OpenTSDB storage driver  https://review.openstack.org/10798616:15
cdent(anything other than 5323?)16:16
jd__cdent: no just 532316:16
jd__that's enough for me to be creative16:16
cdentheh :)16:16
jd__since there's a builtin fallback on _method= I'm not too worried16:16
cdentAre you invested/interested in aligning with API work group proposals/guidelines?16:17
jd__cdent: yes16:17
jd__cdent: did they produce anything useful or is it just a group of bike shedder?16:18
cdentWell I think it is fair mixture of both16:18
cdentAnd not too far along16:18
jd__I don't want my productivity to go down16:18
cdentThey seem to be overly concerned with representations and not enough with healthy HTTP16:18
jd__cdent: aren'y you in it btw?16:18
cdentnever made official because I'm not core16:19
jd__"representations"?16:19
jd__core of what?16:19
*** ParsectiX has joined #openstack-ceilometer16:19
cdentceilo16:19
jd__pfff16:19
jd__is that you or them considering that? :)16:19
cdentOh I don't care, I considering myself a member of the group and I seem to be listened to when I say stuff.16:19
jd__with big tent'ing approaching you'll be core of anything so it seems lame to consider that16:19
jd__ah ok cool16:20
cdentBut I can't be the _official_ ceilo rep according to some guideline or other.16:20
jd__core are not specials said someone16:20
jd__lol16:20
jd__cdent: you want me to write a letter of delegation?16:20
cdentAnyway, I'll leave some comments on the spec.16:20
jd__hahaha bureaucracy to its top16:20
*** fnaval has quit IRC16:20
jd__s/to/at/16:20
cdentOnly if you sign it in gold ink and stamp it with a big seal and wax16:20
jd__(Totally open to correction, but I thought that required the search criteria to be specified in XML?)16:24
jd__that troll eglynn16:24
*** pradk has joined #openstack-ceilometer16:24
eglynnjd__: LOL ... I should really take to twitter and hone my trolling skills there :)16:25
* eglynn reads back16:25
eglynncdent: I thought we'd agreed with jaypipes that the core-only BS wouldn't be enforced on the api-wg?16:25
cdentI never heard any official word, but didn't care, and just carried on. Nobody minds.16:25
openstackgerritJulien Danjou proposed stackforge/gnocchi: rest: replace list+filter by search mechanism  https://review.openstack.org/15704616:25
jd__eglynn: yeah we'd have fun :p16:26
eglynncdent: ... yeah I remember discussing it with Jay on IRC, and him at least loosely aggreeing with the principle16:26
jd__I wish HTTP methods would be less rigid16:27
* cdent shuns jd__ 16:27
eglynnmeh, yeah, with the fallback to POST+_method=SEARCH seems reasonable16:28
cdentthe small number of methods is what makes it grand16:28
*** david-lyle_afk is now known as david-lyle16:28
* cdent shuns eglynn 16:28
eglynncdent: you fundamentalist, you!16:28
* cdent is a shunner16:28
cdentsomebody has to do it in this chaotic world!16:28
jd__I would have use GET instead of SEARCH with a body, but that's not even valid :(16:28
*** IvanBerezovskiy has left #openstack-ceilometer16:28
cdentmaybe the core problem is that the query logic is naff16:30
*** shengjiemin has quit IRC16:30
cdent(but I recall talking about that nearly a year ago)16:31
*** gzhai_mob has joined #openstack-ceilometer16:32
*** fnaval has joined #openstack-ceilometer16:32
*** gzhai_mob has quit IRC16:33
jd__cdent: re fixture, I was talking about config_fixture16:36
cdentyou have to reset it by hand, if you want to run another test suite in the same process16:37
jd__test suite manipulate the conf?16:37
cdent(at least that's been my experience)16:37
cdentyes, it changes the database that is being used16:37
cdentand various other things16:37
jd__ok16:38
cdentbasically the idea is that at the start and end of each suite we start fresh and then clean up16:39
*** changbl has joined #openstack-ceilometer16:39
openstackgerritChris Dent proposed stackforge/gnocchi: Base infrastructure to support gabbi tests  https://review.openstack.org/15660116:45
*** ddieterly has quit IRC16:45
*** rwsu-afk is now known as rwsu16:46
*** ParsectiX has quit IRC16:47
*** ParsectiX has joined #openstack-ceilometer16:48
*** ParsectiX has quit IRC16:50
*** ParsectiX has joined #openstack-ceilometer16:51
*** ddieterly has joined #openstack-ceilometer16:54
*** ddieterly has quit IRC16:55
*** ddieterly has joined #openstack-ceilometer16:55
eglynncdent: FYI good vibes on gabbi at the cross-project meeting last night16:57
eglynnhttp://eavesdrop.openstack.org/meetings/crossproject/2015/crossproject.2015-02-17-21.00.log.html#l-10316:57
cdentthat's good to hear16:58
*** Ala has quit IRC16:59
openstackgerritIgor Degtiarov proposed stackforge/gnocchi: refactor indexer.sqlalchemy  https://review.openstack.org/15706617:10
*** rbrooker has joined #openstack-ceilometer17:14
*** rbrooker has quit IRC17:23
*** rbrooker has joined #openstack-ceilometer17:30
*** _cjones_ has joined #openstack-ceilometer17:36
*** EmilienM is now known as EmilienM|afk17:40
*** harlowja_away is now known as harlowja_17:42
*** exploreshaifali has joined #openstack-ceilometer17:43
*** _cjones_ has quit IRC17:48
*** _cjones_ has joined #openstack-ceilometer17:48
swamireddygordc: Hi ...I have updated the code as per your comments and uploaded the patch set#8...Can you please let me know, if you other comments on this patch?18:15
swamireddygordc: patch url - https://review.openstack.org/#/c/151763/18:16
*** ParsectiX has quit IRC18:16
*** ParsectiX has joined #openstack-ceilometer18:17
openstackgerritMerged stackforge/gnocchi: Base infrastructure to support gabbi tests  https://review.openstack.org/15660118:18
*** ParsectiX has quit IRC18:19
*** ParsectiX has joined #openstack-ceilometer18:19
openstackgerritMerged openstack/ceilometer: cleanup measurements page  https://review.openstack.org/15610618:22
openstackgerritMerged openstack/ceilometer: check metering_connection attribute by default  https://review.openstack.org/15422218:23
*** maishsk has quit IRC18:28
*** maishsk has joined #openstack-ceilometer18:28
*** ParsectiX has quit IRC18:28
*** maishsk has quit IRC18:29
*** ParsectiX has joined #openstack-ceilometer18:29
*** maishsk has joined #openstack-ceilometer18:30
*** maishsk has quit IRC18:38
*** pas-ha has joined #openstack-ceilometer18:39
cdentgordc, jd__ : https://twitter.com/kragen/status/56811722920158003318:39
cdent"but for now we're using some files"18:39
pas-hahi all, looks like ceilometerclient release is long overdue, please see bug #142329118:40
openstackbug 1423291 in python-ceilometerclient "ceilometerclient needs a new release" [Undecided,New] https://launchpad.net/bugs/142329118:40
openstackgerritMerged openstack/ceilometer: start recording error notifications  https://review.openstack.org/15336218:40
*** fabiog has joined #openstack-ceilometer18:41
gordccdent: ah cool. "some files" - the best db there is.18:47
gordcswamireddy: sure thing18:48
cdentgiven that it's kragen it's entirely possible that when he says "some files" he means "a custom data structure that I made up in my mind that solves everything ever"18:48
swamireddygordc: Thank you.18:49
gordccdent: ... that's some serious reputation he has... i would open a bank account and have people drop money in with no conditions and i would decide what to work on... if i had that credibility.18:50
cdenti must ease my eyes18:52
cdentlater18:52
*** cdent has quit IRC18:52
gordcswamireddy: minor comment... i think it's good to go after that.18:55
gordclet me know if i need to clarify18:55
*** moravec has joined #openstack-ceilometer18:57
gordcswamireddy: one more comment.18:57
*** EmilienM|afk is now known as EmilienM19:02
swamireddygordc: /me checking the comments19:05
gordcswamireddy: cool cool19:06
*** pradk has quit IRC19:07
*** nellysmitt has quit IRC19:13
gordcfor anyone with release powers: ->> https://bugs.launchpad.net/python-ceilometerclient/+bug/1423291/19:13
openstackLaunchpad bug 1423291 in python-ceilometerclient "ceilometerclient needs a new release" [Undecided,New]19:13
*** nellysmitt has joined #openstack-ceilometer19:14
*** nellysmitt has quit IRC19:18
swamireddygordc: I will chang the namedtuple declaration to the module level and use the same in the test also..19:21
swamireddygordc: But with the package adding to test-requirement.txt needs clarification...can you plz?19:22
gordcswamireddy: sure. so generally, if ceilometer services only require a lib for an optional path, we don't add the new lib to requirements but rather test-requirements.19:23
gordcso for instance: https://github.com/openstack/ceilometer/blob/master/test-requirements.txt you see oslo.vmware in test-reqs only because if you're using libvirt you  don't really need oslo.vmware19:24
gordcbasically all the driver related libs are usually in test-requirements and not requirements.txt19:25
*** pradk has joined #openstack-ceilometer19:25
gordcswamireddy: ...i just read your comment in gerrit. apologies for dumbing it down so much. :)19:26
gordcswamireddy: let's move it to test-requirements... you were incorrectly told to move it to requirements.txt19:27
*** pas-ha has left #openstack-ceilometer19:27
swamireddygordc: cool...If the lib (ie requests-aws) moved to test-requirement.txt, then when a user really to get the ceph meters, then the rgw_client.py may fail, because it can not fin this lib...19:29
swamireddygordc: Do I miss somthing here??19:29
swamireddygordc: tox test works fine, the lib included in it...19:29
gordcswamireddy: yes. they would need to purposely install the client themselves19:29
gordcsorry client -> lib19:30
*** tasdomas` has joined #openstack-ceilometer19:30
gordci guess it's whether we want to install everything, even if the operator doesn't it need it... or install minimum and require operators to finish off the missing requirements.19:31
gordcwe took the latter.19:31
*** tasdomas has quit IRC19:31
swamireddygordc: ok, got it. Is this same for other libs also?19:31
swamireddygordc: sure, I got it...thanks19:32
gordcswamireddy: hmm.. i'm not sure. trying to think of another project which has as many different drivers as us.19:33
gordc(that isn't neutron)19:33
wolsengordc, cinder?19:34
gordcwolsen: sure. (won't lie, i've zero experience with contributing to cinder)19:35
gordccinder doesn't seem to have drivers in test-reqs only... i guess it's not a consistent rule across projects.19:36
wolsengordc, I was just suggesting it has the same kinds of issues - multitude of drivers with each driver having various dependencies... but I guess the dependencies are handled by the drivers not in the test harnesses19:36
gordci believe glance has their backends in test but i'm just guessing19:37
gordcwolsen: ah i see.19:37
openstackgerritMerged openstack/ceilometer: unicode error in event converter  https://review.openstack.org/15550119:49
openstackgerritgordon chung proposed openstack/ceilometer: add an option to disable non-metric meters  https://review.openstack.org/15674719:53
*** exploreshaifali has quit IRC20:13
*** maishsk has joined #openstack-ceilometer20:18
*** devlaps has quit IRC20:27
swamireddygordc: how about making the Bucket namedtuple to the class level instead of module level?20:32
swamireddygordc:  In the module, other class never use this namedtuple..so Its good to be @ class level....Is this make meaningful?20:34
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259220:38
*** ddieterly has quit IRC20:40
*** maishsk has quit IRC20:40
gordcswamireddy: sure... i guess my main point would be to have test not redeclare the namedtuple.20:41
swamireddygordc: thats possible with class level itself (not neede module level)..20:44
gordcyep20:44
*** devlaps has joined #openstack-ceilometer20:44
gordcwhichever way is fine with me.20:45
*** _cjones_ has quit IRC20:48
swamireddygordc:  Updated with class level as that class only use the Bucket namedtuple...and testing is in progress...once testing done..will submit the patch...20:56
gordcswamireddy: awesome20:57
*** ParsectiX has quit IRC21:03
*** ildikov_afk is now known as ildikov21:03
*** ParsectiX has joined #openstack-ceilometer21:04
*** nellysmitt has joined #openstack-ceilometer21:11
*** _cjones_ has joined #openstack-ceilometer21:14
openstackgerritSwami Reddy proposed openstack/ceilometer: Add ceph object storage meters  https://review.openstack.org/15176321:15
*** zhenq has quit IRC21:21
gordcfabiog: made some tweaks to the 'non-metric' meters patch... please have a look when you have time21:40
gordcthere's some discussion i patchset 3 as well.21:40
fabioggordc: sure, I will21:42
gordcfabiog: cool cool21:42
fabioggordc: regarding the pipeline granularity, I think when we implement the PATCH method in the API we can go to the level of Sinks and Sources21:43
fabioggordc: I am not so sure it is viable to apply changes a the meters: publishers: transformers: and so on21:43
fabioggordc: those stuff do not even have a unique id or name, so it will be difficult to map them in a REST API21:44
gordcfabiog: right. it won't be possible in your patch (and will add quite a bit of complexity).21:44
gordcfabiog: tbh, i'm fine with that because i don't have a requirement to support it... i can't remember who suggested the idea originally so they might be more critical of that decisions21:45
fabioggordc: maybe when I will work on the separation of notifications from polling we could re-factor the pipeline, adding unique identifiers for the different parts we want to modify21:45
gordcfabiog: sure. seems reasonable.21:46
openstackgerritMerged openstack/ceilometer: modify events sql schema to reduce empty columns  https://review.openstack.org/13086921:46
fabioggordc: also transformers are nearly impossible to map into sql ;-)21:46
gordcfabiog: yeah. agreed. the sql first approach would probably not be my first choice but i guess it's a TC requirement?21:47
*** claudiub has joined #openstack-ceilometer21:49
fabioggordc: I guess is to simplify the testing env22:01
fabioggordc: maybe we should ask to allow "special" deployment for the "big data" services, I guess in the future there will be more of those22:02
gordcfabiog: i'm not sure there is a real sql first requirement. tbh, right now, we don't really have a great alternative (mongodb will destroy you within a week if you don't know what you're doing)22:04
gordci guess that's another discussion22:05
*** ddieterly has joined #openstack-ceilometer22:06
*** ddieterly has quit IRC22:07
*** ddieterly has joined #openstack-ceilometer22:08
*** karimb has joined #openstack-ceilometer22:08
*** changbl has quit IRC22:09
*** nellysmitt has quit IRC22:18
*** ParsectiX has quit IRC22:23
*** ParsectiX has joined #openstack-ceilometer22:24
*** openstackgerrit has quit IRC23:00
*** openstackgerrit has joined #openstack-ceilometer23:00
*** claudiub has quit IRC23:01
*** pradk has quit IRC23:08
*** ddieterly has quit IRC23:21
*** karimb has quit IRC23:25
*** karimb has joined #openstack-ceilometer23:25
*** karimb has quit IRC23:41

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