Thursday, 2015-04-02

*** promulo__ has joined #openstack-ceilometer00:16
*** promulo has quit IRC00:20
*** amalagon has joined #openstack-ceilometer00:20
openstackgerritgordon chung proposed openstack/ceilometer: proposal to add ZhiQiang Fan to Ceilometer core  https://review.openstack.org/16995900:37
*** megh has quit IRC00:41
*** megh has joined #openstack-ceilometer00:44
*** jruano has joined #openstack-ceilometer00:47
*** haigang has joined #openstack-ceilometer00:59
*** jruano has quit IRC01:02
*** harlowja is now known as harlowja_away01:14
*** zhenq has left #openstack-ceilometer01:14
*** logan2 has quit IRC01:19
*** edmondsw has quit IRC01:20
*** _cjones_ has quit IRC01:21
*** logan2 has joined #openstack-ceilometer01:21
*** ankita_wagh has quit IRC01:26
*** jmanko has joined #openstack-ceilometer01:28
*** megh has quit IRC01:28
*** alexpilotti has quit IRC01:29
*** jmank has quit IRC01:31
*** Longgeek has quit IRC01:36
*** Longgeek has joined #openstack-ceilometer01:37
*** Longgeek has quit IRC01:43
*** Longgeek has joined #openstack-ceilometer01:44
*** Longgeek_ has joined #openstack-ceilometer01:57
*** Longgeek has quit IRC02:00
*** julim has joined #openstack-ceilometer02:05
*** Kennan2 is now known as Kennan02:14
*** fawadkhaliq has joined #openstack-ceilometer04:11
*** fnaval has quit IRC04:21
*** _cjones_ has joined #openstack-ceilometer04:47
*** fnaval has joined #openstack-ceilometer04:48
*** _cjones_ has quit IRC04:50
*** _cjones_ has joined #openstack-ceilometer04:50
*** megh has joined #openstack-ceilometer04:56
*** megh has quit IRC05:17
*** megh has joined #openstack-ceilometer05:19
*** megh has quit IRC05:23
*** _cjones_ has quit IRC05:29
*** fawadkhaliq has quit IRC05:42
*** yatin has joined #openstack-ceilometer05:47
*** _cjones_ has joined #openstack-ceilometer05:56
*** fawadkhaliq has joined #openstack-ceilometer06:00
*** haigang has quit IRC06:02
*** fawadk has joined #openstack-ceilometer06:05
*** thomasem has quit IRC06:09
*** fawadkhaliq has quit IRC06:09
*** ildikov_ has quit IRC06:12
*** haigang has joined #openstack-ceilometer06:12
*** thomasem has joined #openstack-ceilometer06:15
*** fawadkhaliq has joined #openstack-ceilometer06:16
*** fawadk has quit IRC06:19
*** yprokule has joined #openstack-ceilometer06:45
*** amalagon has quit IRC06:51
*** safchain has joined #openstack-ceilometer06:53
*** afazekas has joined #openstack-ceilometer06:54
openstackgerritSteve Martinelli proposed openstack/ceilometer: Add project and domain ID to event definition for identity CRUD  https://review.openstack.org/17002407:04
*** ildikov_ has joined #openstack-ceilometer07:04
*** Longgeek_ has quit IRC07:17
*** Longgeek has joined #openstack-ceilometer07:17
*** boris-42 has quit IRC07:18
*** yprokule has quit IRC07:21
*** yprokule has joined #openstack-ceilometer07:22
*** haigang has quit IRC07:30
*** eglynn has joined #openstack-ceilometer07:35
*** haigang has joined #openstack-ceilometer07:35
*** _cjones_ has quit IRC07:37
*** _cjones_ has joined #openstack-ceilometer07:37
*** haigang has quit IRC07:56
*** haigang has joined #openstack-ceilometer08:02
*** ildikov_ has quit IRC08:02
*** yprokule has quit IRC08:12
openstackgerritZhiQiang Fan proposed openstack/python-ceilometerclient: WIP: add region_name to auth plugin parameters  https://review.openstack.org/17003708:12
*** yprokule has joined #openstack-ceilometer08:13
*** ildikov_ has joined #openstack-ceilometer08:16
*** haigang has quit IRC08:24
*** haigang has joined #openstack-ceilometer08:28
*** yatin has quit IRC08:57
*** yatin has joined #openstack-ceilometer08:57
*** shardy has joined #openstack-ceilometer09:00
*** _cjones_ has quit IRC09:04
*** haigang has quit IRC09:24
*** msly has quit IRC09:38
*** msly has joined #openstack-ceilometer09:38
*** haigang has joined #openstack-ceilometer09:40
*** ildikov_ has quit IRC09:50
*** _nadya_ has joined #openstack-ceilometer10:03
*** ildikov_ has joined #openstack-ceilometer10:05
*** robsparker has quit IRC10:17
*** cdent has joined #openstack-ceilometer10:31
openstackgerritZhiQiang Fan proposed openstack/python-ceilometerclient: add region_name to auth plugin parameters  https://review.openstack.org/17003710:42
openstackgerritZhiQiang Fan proposed openstack/python-ceilometerclient: print user friendly error message for alarm update time constraints  https://review.openstack.org/16977410:43
*** fawadkhaliq has quit IRC10:45
*** haigang has quit IRC10:55
*** _nadya_ has quit IRC11:14
*** haigang has joined #openstack-ceilometer11:23
*** links has joined #openstack-ceilometer11:32
*** eglynn has quit IRC11:36
*** alexpilotti has joined #openstack-ceilometer11:43
*** msly has quit IRC11:44
*** fawadkhaliq has joined #openstack-ceilometer11:46
*** eglynn has joined #openstack-ceilometer11:49
*** fawadkhaliq has quit IRC11:51
*** tongli has joined #openstack-ceilometer11:52
openstackgerritChris Dent proposed openstack/ceilometer: Further mock adjustments to deal with intermittent failure  https://review.openstack.org/17009211:55
*** ildikov_ has quit IRC12:00
*** boris-42 has joined #openstack-ceilometer12:00
*** ildikov_ has joined #openstack-ceilometer12:01
*** eglynn has quit IRC12:03
*** _cjones_ has joined #openstack-ceilometer12:05
*** _cjones_ has quit IRC12:09
*** eglynn has joined #openstack-ceilometer12:13
*** claudiub has joined #openstack-ceilometer12:23
*** robsparker has joined #openstack-ceilometer12:25
*** gordc has joined #openstack-ceilometer12:32
*** yatin has quit IRC12:44
*** fawadkhaliq has joined #openstack-ceilometer12:56
*** _nadya_ has joined #openstack-ceilometer13:22
*** thomasem has quit IRC13:23
*** thomasem has joined #openstack-ceilometer13:24
openstackgerritMerged openstack/ceilometer: Fix the event type for trusts  https://review.openstack.org/16992913:33
*** zigo has quit IRC13:38
*** prad has joined #openstack-ceilometer13:39
*** zigo_ is now known as zigo13:41
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Move common resource attributes into a mixin  https://review.openstack.org/17012813:44
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/17012913:44
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840513:45
openstackgerritMehdi Abaakouk proposed openstack/gnocchi: Keep history of updated resource attributes  https://review.openstack.org/16840513:47
*** fawadk has joined #openstack-ceilometer13:47
*** ildikov_ has quit IRC13:47
*** fawadkhaliq has quit IRC13:48
*** gordc has quit IRC13:48
*** fawadkhaliq has joined #openstack-ceilometer13:52
*** fawadk has quit IRC13:53
*** prad_ has joined #openstack-ceilometer13:54
*** prad has quit IRC13:54
*** prad_ is now known as prad13:54
*** ildikov_ has joined #openstack-ceilometer13:59
*** gordc has joined #openstack-ceilometer14:04
*** haigang has quit IRC14:08
*** rbak has joined #openstack-ceilometer14:10
*** exploreshaifali has joined #openstack-ceilometer14:12
*** edmondsw has joined #openstack-ceilometer14:18
*** links has quit IRC14:26
edmondswam I reading correctly that message_signature is stored in the db but only ever checked before storing, so it would be fine to change telemetry_secret after storing some data?14:29
gordcedmondsw: right. it's really only used to ensure a message isn't tampered with during IPC.14:29
edmondswgordc, so pretty silly that we store that in the db... waste of space14:30
gordcedmondsw: i added a patch which you can effectively disable it if you feel that MQ security is enough14:30
gordcedmondsw: yes. i mean, it's possible that some people are using it to validate after the fact...14:31
edmondswgordc, wouldn't you just not specify telemetry_secret if you wanted to rely on MQ security?14:31
gordcedmondsw: yeah. that's what i did. i believe pre-kilo it still attempted to sign message iwth blank telemetry_secret so it added processing overhead still14:33
gordcnow if it's blank, it'll just skip signing and send.14:33
edmondswgotcha14:33
gordcedmondsw: this was the bug for reference: https://bugs.launchpad.net/ceilometer/+bug/143607714:34
openstackLaunchpad bug 1436077 in Ceilometer "minimise hmac calls to avoid performance degradation" [Medium,Fix committed] - Assigned to gordon chung (chungg)14:34
edmondswtx14:34
*** fawadk has joined #openstack-ceilometer14:37
*** jruano has joined #openstack-ceilometer14:39
*** fawadkhaliq has quit IRC14:41
*** ildikov_ has quit IRC14:53
*** ildikov_ has joined #openstack-ceilometer14:56
*** fawadk has quit IRC15:03
*** jaypipes has joined #openstack-ceilometer15:16
openstackgerritChris Dent proposed openstack/gnocchi: rest: enhance metric retrieval in controllers  https://review.openstack.org/16937515:19
*** alexpilotti_ has joined #openstack-ceilometer15:20
cdentsileht: i edited the commit message on that ^ so might have confused gerrit a bit but my goal was for it to be +A15:20
silehtcdent, k15:21
*** alexpilotti has quit IRC15:22
*** alexpilotti_ is now known as alexpilotti15:22
*** fawadkhaliq has joined #openstack-ceilometer15:30
openstackgerritIlya Tyaptin proposed openstack/ceilometer: Add a storing of resource id for events  https://review.openstack.org/17018015:42
*** kbyrne has quit IRC15:48
*** packet has joined #openstack-ceilometer15:52
*** _cjones_ has joined #openstack-ceilometer16:03
*** prad has quit IRC16:04
*** prad has joined #openstack-ceilometer16:05
openstackgerritSteve Martinelli proposed openstack/ceilometer: Tweak authenticate event definition  https://review.openstack.org/17019316:07
*** prad has quit IRC16:10
cdentbbl16:11
*** cdent has quit IRC16:11
ildikov_FYI, bug report for meter deprecation: https://bugs.launchpad.net/ceilometer/+bug/143976616:11
openstackLaunchpad bug 1439766 in Ceilometer "Deprecation process for meters need to be introduced" [High,New]16:11
*** prad has joined #openstack-ceilometer16:12
openstackgerritIlya Tyaptin proposed openstack/ceilometer-specs: Enable Liberty specs  https://review.openstack.org/16832116:14
*** kbyrne has joined #openstack-ceilometer16:15
*** afazekas has quit IRC16:16
openstackgerritMerged openstack/gnocchi: Update resource.yaml to reflect recent fixes  https://review.openstack.org/16985516:24
*** fawadkhaliq has quit IRC16:35
openstackgerritSteve Martinelli proposed openstack/ceilometer: Tweak authenticate event definition  https://review.openstack.org/17019316:39
*** jruano has quit IRC16:41
*** harlowja_away is now known as harlowja16:42
*** Longgeek has quit IRC16:50
*** fawadkhaliq has joined #openstack-ceilometer16:53
*** ildikov_ has quit IRC16:54
*** ankita_wagh has joined #openstack-ceilometer16:57
*** ankita_w_ has joined #openstack-ceilometer16:58
*** fnaval has quit IRC16:59
*** ankit____ has joined #openstack-ceilometer17:01
*** ankita_wagh has quit IRC17:01
*** EmilienM is now known as EmilienM|afk17:01
*** ankita_w_ has quit IRC17:03
*** ankit____ has quit IRC17:04
*** mahipal has joined #openstack-ceilometer17:06
*** sbfox has joined #openstack-ceilometer17:16
*** eglynn has quit IRC17:16
*** fnaval has joined #openstack-ceilometer17:18
*** _nadya_ has quit IRC17:20
*** exploreshaifali has quit IRC17:26
*** sbfox has quit IRC17:29
*** alexpilotti_ has joined #openstack-ceilometer17:31
*** alexpilotti has quit IRC17:32
*** megh has joined #openstack-ceilometer17:32
*** alexpilotti_ is now known as alexpilotti17:33
*** sbfox has joined #openstack-ceilometer17:33
openstackgerritDivya K Konoor proposed openstack/ceilometer: Adds support for default rule in ceilometer policy.json.  https://review.openstack.org/16737017:37
*** alexpilotti has quit IRC17:42
*** alexpilotti has joined #openstack-ceilometer17:44
*** safchain has quit IRC17:44
*** yprokule has quit IRC17:44
*** ildikov_ has joined #openstack-ceilometer17:52
*** jruano has joined #openstack-ceilometer17:53
*** Kennan has quit IRC18:03
*** Kennan2 has joined #openstack-ceilometer18:03
*** Kennan has joined #openstack-ceilometer18:06
*** Kennan2 has quit IRC18:07
*** ddieterly has joined #openstack-ceilometer18:08
*** ankita_wagh has joined #openstack-ceilometer18:10
*** ankita_w_ has joined #openstack-ceilometer18:18
*** ankita_wagh has quit IRC18:20
*** EmilienM|afk is now known as EmilienM18:30
*** ddieterly has quit IRC18:33
*** cdent has joined #openstack-ceilometer19:09
cdentgordc: the answer on that magicmock thing is basically "I dunno, it's magic"19:10
cdentI can't be certain it fixes it, just that it makes it fail less.19:10
cdentI can annotate with a pointer to where the failure happens if that's relevant?19:11
gordccdent: sure. it seems weird that that would do anything tbh.19:13
cdenttoo much magic19:14
cdentprad: you about?19:18
pradcdent, yep19:18
cdentsorry I'm a bit late, got distracted by magic19:18
pradno worries19:18
*** fawadkhaliq has quit IRC19:19
cdentif we're gonna talk gabbi we may as well do it here, and then if jasonamyers is around he can join in too, and there will be an artifact of some kind in the log if that turns out to be useful19:19
cdentthat work?19:19
pradcdent, sure19:20
cdentit would also inflict us on gordc and since it turns out everything is his fault, he deserves it19:20
pradhehe sure enough19:20
jasonamyersI'm here19:20
cdentearlier in the year I made this as a sort of plan of what to do with gabbi: https://tank.peermore.com/tanks/cdent-rhat/GabbiCeiloPlan19:21
gordci heard my name19:21
cdentwe've diverged from that somewhat19:21
cdentbut it still provides some useful background19:21
pradcool, i'll take a peek at that19:22
cdentin gnocchi, jasonamyers and I have been slowly but surely taking some segment of the api and building a suite of tests for it19:23
cdenthere's a recent example: https://review.openstack.org/#/c/169893/19:23
pradso you have been adding api coverage to gnocchi ? has it started for ceilo?19:24
cdentAs you can see, if the API allows you to create stuff you don't need to do anything other than create the yaml19:24
cdentyup, recent ceilo one:19:24
*** sbfox has quit IRC19:24
cdenthttps://review.openstack.org/#/c/161746/19:24
pradcdent, would it help if i look on the ceilo side of api to add coverage?19:25
pradalso did you identify missing areas to add coverage?19:25
pradok19:26
cdentyeah, so one of the recent habits has been to run the coverage tests, limiting it to just using the gabbi tests and then seeing what's missing19:26
cdentin ceilo that would be something like 'tox -ecover -- gabbi_'19:26
pradcool19:26
cdentand looking at the ceilometer/api/controllers files19:26
cdentthe policy we've been using thus far is we're not trying to go too far beyond the controller code19:27
jasonamyersI never got tox to work in gnochhi19:27
jasonamyerswith -ecover19:27
jasonamyersbut I'm sure it was me19:27
*** ddieterly has joined #openstack-ceilometer19:28
cdentjust into the controllers themselves to confirm that they talk HTTP properly and that bad args and such are handled properly19:28
cdentjasonamyers: in gnocchi it is py27-cover instead of "cover"19:28
cdentprad: it turns out that WSME can be _really bad_ about being a proper HTTP citizen so where it isn't behaving properly things like this are done:19:29
cdenthttps://github.com/openstack/ceilometer/blob/master/ceilometer/tests/gabbi/gabbits/meters.yaml#L3319:30
*** packet has quit IRC19:30
cdentand then as those issues are fixed (either in wsme, pecan or ceilo itself) going back and cleaing up the yaml19:31
pradunderstood.. i think i'll get a better hang of it once i start writing a couple of simple tests19:32
cdentyeah, I reckon the only complex part is how the fixtures work (if used)19:32
cdentthe basic idea is that there are 0 or more fixtures active per yaml file (which internally is a unittest.TestSuite)19:33
cdentso you can have a different (and fresh) database per yaml file19:33
*** edmondsw has quit IRC19:33
cdentand .testr.conf is set up so that each file will be considered a "group" when tests are being distributed for concurrency19:33
cdentWhen I'm writing the tests I like to pretend that I don't know the APIs at all and just throw random junk and see what happens19:35
cdenttends to be quite illuminating19:35
prad:)19:35
jasonamyers+1 cdent19:35
pradis there a specific area you like me to get started? to start getting a feel for it19:36
cdentIf you're thinking on the ceilo side, I don't really know what's missing. I'd say try the coverage trick and see where it points19:37
cdentIf there's an entire subpath that isn't covered at all, do that19:37
pradok19:37
cdentOtherwise just pick something that's thin19:37
cdentin ceilometer/tests/gabbi/gabbits/ there are files for most things, but some of them are more fleshed out than others19:38
pradcdent, on gnocchi side are there anything left? or you guys covered it?19:39
cdentthere's still stuff left19:39
cdentI think the big one is search19:39
pradok19:39
cdentwhich in order to be effective will probaby need some kind of fixture to install some real data19:39
cdentcomplex query tests are pretty non-existent for both19:40
cdentas well as any tests that exercise the policy enforcement system19:40
pradok, i'll run the cover and start with some small ones first and then expand19:41
cdentnote that all the tests at the moment bypass keystone: instead of a token being passed and tranformed into a user-id we add our own user and project ids to the headers as needed19:41
pradok19:41
cdentor x-roles19:41
cdentat some point we'll want to cover the various authz situations19:41
cdentoh, in case it wasn't clear there are convenience targets in tox.ini for both ceilo and gnocchi for running just the gabbi tests19:43
cdenton the gnocchi side pretty-tox is being used so the output is friendly19:44
cdentbut on the ceilo side less so19:44
*** packet has joined #openstack-ceilometer19:44
cdentso I often find myself doing 'testr last --subunit |subunit-trace' # subunit-trace is from tempest-lib19:44
cdentto get a better view on failures19:44
pradcool19:44
cdentif hit a roadblock, just shout, I ought to be around for a few more hours19:46
cdentor tuesday19:46
pradsure,i'll play around with gabbi a bit and poke ya if i get stuck19:46
pradthx for the info19:46
cdentyou're welcome prad, if you get converted into the church of gabbi, we might have enough people to properly shun gordc19:52
* cdent can no longer gordc19:59
cdentso it is failing again gordc20:00
*** exploreshaifali has joined #openstack-ceilometer20:09
*** exploreshaifali has quit IRC20:09
gordccdent: hmm.... i don't get prompted when my name isn't first word in message.20:10
gordcirssi has made me a narcissist20:10
*** ddieterly has quit IRC20:19
*** ddieterly has joined #openstack-ceilometer20:20
*** prad has quit IRC20:29
*** prad has joined #openstack-ceilometer20:30
*** jruano has quit IRC20:33
*** ankita_wagh has joined #openstack-ceilometer20:34
*** ankita_w_ has quit IRC20:38
*** sbfox has joined #openstack-ceilometer20:50
*** ankita_w_ has joined #openstack-ceilometer20:53
*** ankita_wagh has quit IRC20:56
*** ddieterly has quit IRC20:57
*** packet has quit IRC20:58
*** packet has joined #openstack-ceilometer21:04
*** packet has quit IRC21:07
*** _cjones_ has quit IRC21:11
*** _cjones_ has joined #openstack-ceilometer21:11
*** fnaval has quit IRC21:13
*** ankita_wagh has joined #openstack-ceilometer21:27
*** ankita___ has joined #openstack-ceilometer21:28
*** ankita_w_ has quit IRC21:30
*** ankita_wagh has quit IRC21:32
*** vishwana_ has joined #openstack-ceilometer21:32
*** adam_g has quit IRC21:34
*** dougwig has quit IRC21:35
*** julim_ has joined #openstack-ceilometer21:36
*** jasonamyers has quit IRC21:38
*** vishwanathj has quit IRC21:38
*** swamireddy has quit IRC21:38
*** julim has quit IRC21:38
*** adam_g has joined #openstack-ceilometer21:40
*** adam_g has quit IRC21:40
*** adam_g has joined #openstack-ceilometer21:40
*** swamireddy has joined #openstack-ceilometer21:42
*** jasonamyers has joined #openstack-ceilometer21:43
*** dougwig has joined #openstack-ceilometer21:45
*** fnaval has joined #openstack-ceilometer21:46
*** gordc has quit IRC21:53
*** gordc has joined #openstack-ceilometer21:54
*** alexpilotti has quit IRC21:59
*** ankita___ has quit IRC22:00
*** Kennan has quit IRC22:02
*** Kennan has joined #openstack-ceilometer22:03
*** gordc has quit IRC22:04
*** cdent has quit IRC22:08
*** prad has quit IRC22:11
*** packet has joined #openstack-ceilometer22:12
*** tongli has quit IRC22:23
*** ankita_wagh has joined #openstack-ceilometer22:28
*** packet has quit IRC22:33
*** shardy has quit IRC22:39
*** rbak has quit IRC22:53
*** ankita_w_ has joined #openstack-ceilometer23:08
*** sbfox has quit IRC23:09
*** ankita___ has joined #openstack-ceilometer23:09
*** ankita_wagh has quit IRC23:11
*** ankita_w_ has quit IRC23:12
*** ankita_wagh has joined #openstack-ceilometer23:16
*** ankita___ has quit IRC23:19
*** ankita_w_ has joined #openstack-ceilometer23:32
*** _cjones_ has quit IRC23:34
*** ankita_wagh has quit IRC23:35
*** jaypipes has quit IRC23:54
*** vishwana_ has quit IRC23:58
*** megh has quit IRC23:58

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