Wednesday, 2014-09-10

openstackgerritAndrey Grebennikov proposed a change to openstack/ceilometer: Added Cinder volume/snapshot pollster  https://review.openstack.org/12028700:04
*** gordc has quit IRC00:07
*** DrBacchus has quit IRC00:07
*** DrBacchus has joined #openstack-ceilometer00:08
*** sdake_ has quit IRC00:12
openstackgerritgordon chung proposed a change to openstack/ceilometer: fix network discovery meters  https://review.openstack.org/11705100:16
*** DrBacchus has quit IRC00:17
*** prad has joined #openstack-ceilometer00:17
*** ViswaV has quit IRC00:18
*** sbfox has joined #openstack-ceilometer00:18
*** prad has quit IRC00:19
*** sdake_ has joined #openstack-ceilometer00:23
*** Longgeek has quit IRC00:26
*** sdake_ has quit IRC00:34
*** DrBacchus has joined #openstack-ceilometer00:34
*** dzwiedziu has joined #openstack-ceilometer00:40
*** dzwiedziu has quit IRC00:45
*** Ephur has quit IRC00:46
*** russellb has quit IRC00:46
*** russellb has joined #openstack-ceilometer00:49
*** Ephur has joined #openstack-ceilometer00:49
*** dzwiedziu has joined #openstack-ceilometer00:52
*** dzwiedziu has quit IRC00:58
*** DrBacchus has quit IRC01:00
*** _cjones_ has quit IRC01:07
*** _cjones_ has joined #openstack-ceilometer01:07
*** sbfox has quit IRC01:09
*** _cjones_ has quit IRC01:12
*** ccrouch has quit IRC01:21
*** DrBacchus has joined #openstack-ceilometer01:21
*** ccrouch has joined #openstack-ceilometer01:22
*** ccrouch has quit IRC01:24
*** ccrouch has joined #openstack-ceilometer01:25
*** sbfox has joined #openstack-ceilometer01:26
*** ccrouch has quit IRC01:26
*** changbl has joined #openstack-ceilometer01:27
*** ccrouch has joined #openstack-ceilometer01:28
*** DrBacchus has quit IRC01:31
*** ccrouch1 has joined #openstack-ceilometer01:34
*** ccrouch1 has quit IRC01:34
*** ccrouch1 has joined #openstack-ceilometer01:34
*** ccrouch2 has joined #openstack-ceilometer01:36
*** ccrouch has quit IRC01:36
*** ccrouch1 has quit IRC01:39
*** ccrouch2 has quit IRC01:42
*** ccrouch has joined #openstack-ceilometer01:45
*** rmcall has quit IRC01:47
*** nosnos has joined #openstack-ceilometer01:54
*** DrBacchus has joined #openstack-ceilometer02:01
*** r-daneel has quit IRC02:02
*** ccrouch1 has joined #openstack-ceilometer02:04
*** ccrouch has quit IRC02:05
*** ViswaV has joined #openstack-ceilometer02:05
*** DrBacchus has quit IRC02:06
*** flwang1 has joined #openstack-ceilometer02:07
*** ViswaV has quit IRC02:21
*** harlowja is now known as harlowja_away02:25
openstackgerritAndrey Grebennikov proposed a change to openstack/ceilometer: Added Cinder volume/snapshot pollster  https://review.openstack.org/12028702:26
openstackgerritHisashi Osanai proposed a change to openstack/ceilometer: enable to get swift's service type from configuration file  https://review.openstack.org/11277102:37
*** dzwiedziu has joined #openstack-ceilometer02:42
*** dzwiedziu has quit IRC02:46
*** _nadya_ has joined #openstack-ceilometer02:46
*** ccrouch1 has quit IRC02:48
*** ccrouch has joined #openstack-ceilometer02:48
openstackgerritRen Qiaowei proposed a change to openstack/ceilometer: Inspector: add log hints  https://review.openstack.org/12030403:02
openstackgerritgordon chung proposed a change to openstack/ceilometer: fix network discovery meters  https://review.openstack.org/11705103:29
*** _nadya_ has quit IRC03:49
*** rmcall has joined #openstack-ceilometer04:31
*** dzwiedziu has joined #openstack-ceilometer04:39
*** dzwiedziu has quit IRC04:43
*** cmyster has quit IRC04:46
*** yatin_ has joined #openstack-ceilometer04:53
*** ildikov has quit IRC04:54
openstackgerritAndrey Grebennikov proposed a change to openstack/ceilometer: Added Cinder volume/snapshot pollster  https://review.openstack.org/12028704:55
*** _cjones_ has joined #openstack-ceilometer05:07
*** _cjones_ has quit IRC05:11
*** _cjones_ has joined #openstack-ceilometer05:11
*** julim has quit IRC05:17
*** ildikov has joined #openstack-ceilometer05:24
*** nsaje has quit IRC05:27
*** cmyster has joined #openstack-ceilometer05:27
*** cmyster has joined #openstack-ceilometer05:27
*** nsaje has joined #openstack-ceilometer05:28
*** flwang1 has quit IRC05:40
*** k4n0 has joined #openstack-ceilometer05:45
*** arezmerita has joined #openstack-ceilometer05:51
*** _cjones_ has quit IRC06:16
*** _cjones_ has joined #openstack-ceilometer06:16
*** _cjones_ has quit IRC06:21
*** ildikov_ has joined #openstack-ceilometer06:22
*** ildikov has quit IRC06:23
*** ildikov has joined #openstack-ceilometer06:29
*** ildikov_ has quit IRC06:29
*** deepthi has joined #openstack-ceilometer06:32
*** rmcall has quit IRC06:45
*** melonsoda has joined #openstack-ceilometer06:46
melonsodaHOLA06:46
*** melonsoda has left #openstack-ceilometer06:46
*** arezmerita_ has joined #openstack-ceilometer06:48
*** arezmerita has quit IRC06:48
*** ifarkas has joined #openstack-ceilometer06:53
*** k4n0 has quit IRC07:07
*** ildikov has quit IRC07:08
*** _nadya_ has joined #openstack-ceilometer07:11
*** safchain has joined #openstack-ceilometer07:12
*** k4n0 has joined #openstack-ceilometer07:22
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/11009607:25
*** ildikov has joined #openstack-ceilometer07:57
*** jdandrea has quit IRC07:59
*** jdandrea has joined #openstack-ceilometer08:00
*** sbfox has quit IRC08:01
*** dzwiedziu has joined #openstack-ceilometer08:15
*** jbuckley has joined #openstack-ceilometer08:19
*** yassine has joined #openstack-ceilometer08:19
*** dzwiedziu has quit IRC08:20
*** alexpilotti has joined #openstack-ceilometer08:25
*** amalagon has quit IRC08:30
openstackgerritA change was merged to openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/11854308:33
openstackgerritHisashi Osanai proposed a change to openstack/ceilometer: Enable to get swift's service type from configuration file  https://review.openstack.org/11277108:40
*** _nadya_ has quit IRC08:48
*** r3pl4y has joined #openstack-ceilometer08:49
openstackgerritLadislav Smola proposed a change to openstack/ceilometer: Allow to pass dict from resource discovery  https://review.openstack.org/11980308:54
*** _nadya_ has joined #openstack-ceilometer08:56
openstackgerritZhai, Edwin proposed a change to openstack/ceilometer: Add IPMI support  https://review.openstack.org/11584509:26
openstackgerritZhai, Edwin proposed a change to openstack/ceilometer: Add IPMI pollster  https://review.openstack.org/11903809:26
*** ccrouch has quit IRC09:28
*** ccrouch has joined #openstack-ceilometer09:28
*** flwang1 has joined #openstack-ceilometer09:30
*** cdent has joined #openstack-ceilometer09:34
openstackgerritRen Qiaowei proposed a change to openstack/ceilometer: Add message translate module in app module  https://review.openstack.org/12035609:36
boris-42jd__ hi09:38
*** nosnos has quit IRC09:41
*** nosnos has joined #openstack-ceilometer09:42
cdentI wish people would learn to trim and quote properly.09:44
*** claudiub has joined #openstack-ceilometer09:45
*** nosnos has quit IRC09:47
*** yassine has quit IRC09:58
*** Longgeek has joined #openstack-ceilometer09:59
*** dzwiedziu has joined #openstack-ceilometer10:16
*** dzwiedziu has quit IRC10:21
*** dzwiedziu has joined #openstack-ceilometer10:23
*** dzwiedziu has quit IRC10:50
*** dzwiedziu has joined #openstack-ceilometer10:50
*** dzwiedziu has quit IRC10:55
*** deepthi has quit IRC11:08
openstackgerritZhai, Edwin proposed a change to openstack/ceilometer: Add IPMI support  https://review.openstack.org/11584511:11
openstackgerritZhai, Edwin proposed a change to openstack/ceilometer: Add IPMI pollster  https://review.openstack.org/11903811:11
*** yassine has joined #openstack-ceilometer11:14
*** Longgeek_ has joined #openstack-ceilometer11:14
*** Longgeek has quit IRC11:18
*** dzwiedziu has joined #openstack-ceilometer11:18
*** ssakhamuri__ has quit IRC11:21
openstackgerritjustinsb proposed a change to openstack/ceilometer: Timestamp bounds need not be tight (per ceilometer 1288372)  https://review.openstack.org/11974711:29
*** ekarlso- has quit IRC11:37
*** ekarlso- has joined #openstack-ceilometer11:37
*** dzwiedziu has quit IRC11:41
*** dzwiedziu has joined #openstack-ceilometer11:42
*** isviridov_away is now known as isviridov11:42
*** dzwiedziu has quit IRC11:43
*** dzwiedziu has joined #openstack-ceilometer11:44
*** asalkeld has quit IRC11:49
*** dzwiedziu has quit IRC11:54
*** cmyster is now known as cmyster_away11:56
cdentboris-42: you about, I'd like to get a quick primer on how to do some rally+ceilometer explorations12:00
boris-42cdent hi there12:01
cdentAh, cool, I've got a devstack set up with rally enabled12:01
boris-42cdent so what you are going to test?)12:02
cdentand I want to benchmark various api commands (as a starting point)12:02
cdentI want to compare different database setups12:02
cdentOnce I have a handle on how things work I want to go deeper, but I figure that's a good starting point.12:02
boris-42cdent okay yep12:03
boris-42cdent so what you need is to run "rally deployment check"12:04
boris-42cdent to ensure that rally successfully installed12:04
cdentyup lots of stuff listed as "available"12:04
boris-42cdent great12:04
cdent(including ceilo)12:05
boris-42cdent so now you can run some benchmarks12:05
boris-42cdent lemme find some sample12:05
*** dzwiedziu has joined #openstack-ceilometer12:05
cdentthank you very much boris-4212:05
boris-42cdent so all samples are here https://github.com/stackforge/rally/tree/master/doc/samples/tasks/scenarios12:05
boris-42cdent you have ceilometer samples https://github.com/stackforge/rally/tree/master/doc/samples/tasks/scenarios/ceilometer12:06
boris-42cdent unfroutantelly during some bug not all works =)12:06
cdentthat's fair12:06
cdentso those are on my disk, how do I call them?12:06
boris-42cdent so create_meter_and_get_stats: and .create_and_query_samples doesn't work12:07
boris-42cdent other you can run12:07
boris-42cdent so you can run these benchmarks with next command12:07
boris-42cdent rally -v task start <some_file_like_in_doc_samples>12:08
boris-42cdent that is all=)12:08
cdentsweet12:09
boris-42cdent then you will get at the end hints12:09
cdentyup, seeing that12:09
boris-42cdent to get all results (in json) or (html report)12:09
cdentif I want to adjust concurrency and count and such I just modify the scenario file to my liking, yes?12:09
boris-42cdent yep load generators are plugable12:10
boris-42cdent different plugins can be used changing "type" key12:10
boris-42cdent and they have all own arguments12:10
*** claudiub has quit IRC12:10
*** claudiub has joined #openstack-ceilometer12:10
boris-42cdent so actually just to make it clear in constant runner "times" means total times12:10
boris-42cdent e.g. if you specify times: 10 and concurrency: 512:11
boris-42it will call 10 times benchmark scenario (in total)12:11
* cdent nods12:11
boris-42but at any moment of time there will be exactly 5 running scenarios12:11
* cdent is looking at html output12:11
boris-42cdent I am going to refactor it12:11
boris-42cdent cause now it sux =(12:11
cdentI sux a whole lot less than nothing at all.12:11
boris-42cdent =)12:12
boris-42cdent so actually it will be full report12:12
boris-42cdent with detailed description of what was benchmarked12:12
boris-42cdent like we will put docstrings from benchmarks/runners to it12:12
boris-42cdent so it will be "full" report not just some strange graphs=)12:12
cdentI can see how, once you get a hang of how to set up scenarios, this could be _very_ useful12:13
boris-42cdent ?12:14
boris-42cdent you mean how to write own scenarios?12:14
cdentWhat I mean is: One _I_ figure how to write scenarios, it will be very useful to me.12:14
cdents/One/Once/12:14
boris-42cdent so it's really super extra easy12:14
cdent:)12:15
boris-42cdent actually whole my team is working on making framwrok12:15
boris-42cdent so we don't write any scenarios just making everything simple easy and flexible=)12:15
boris-42cdent here are ceilometer scenarios https://github.com/stackforge/rally/tree/master/rally/benchmark/scenarios/ceilometer12:15
boris-42cdent basically what you need is next12:16
*** dzwiedziu has quit IRC12:16
boris-42cdent you should create a class that is subclass of base.Scenario12:17
boris-42cdent in this case, ceilometerutils.CeilometerScenario - is subclass of base.Scenario12:17
* cdent nods12:17
boris-42cdent then any method that has this @base.scenario decorator https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/ceilometer/alarms.py#L2412:17
boris-42cdent is benchmark it will be autodiscovered12:18
boris-42cdent and name of this will be CeilometerAlarms.create_alarm12:18
* cdent nods12:18
boris-42cdent it will accept in "args" meter_name and threshold12:18
boris-42cdent and very last thing that you should know is what is in utils12:18
boris-42cdent https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/ceilometer/utils.py#L49-L6212:19
boris-42cdent so basically we are calling this from benchmark12:19
boris-42cdent https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/ceilometer/utils.py#L49 this will measure duration of this metho12:19
* cdent nods12:19
boris-42cdent so you can have more granularity then just (whole duration of whole benchmark scenario)12:19
boris-42cdent inide it you are calling https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/ceilometer/utils.py#L6112:19
boris-42cdent self.clients("ceilometer") - is initialized plain ceilometer python clinet12:20
boris-42cdent there is as well sefl.admin_clinets("ceilometer") - that will be initialized by admin12:20
cdentmakes sense12:20
*** dzwiedziu has joined #openstack-ceilometer12:21
boris-42cdent heh ceilometer samples should be fixed lol12:21
boris-42cdent I'll do that =)12:22
cdentThis gives me enough to experiment with. If I hit a roadblock I'll give you a shout. I think at this point the best thing for me to do is make some more coffee and do some experimentation. Thanks very much for the detailed info.12:22
boris-42cdent one more thing12:22
boris-42cdent about this https://github.com/stackforge/rally/blob/master/doc/samples/tasks/scenarios/authenticate/keystone.json#L9-L1412:23
cdentyes12:23
boris-42cdent "users" context will create users and tenants that will be used by self.clinets()12:23
boris-42cdent e.g. every time when we are calling benchmark scenario, it will get one randomly passed user12:23
cdentmakes sense12:23
boris-42cdent so you can play with amount of users and tenatns12:23
boris-42cdent for now by default even if you don't specify12:24
boris-42cdent it will create 1 tenant and 1 user12:24
cdentright, thanks. I'm get food and get to it.12:25
boris-42cdent btw we have own chat #openstack-rally12:26
* cdent adds it to the list12:26
boris-42cdent but I'll be here as well=)12:26
openstackgerritAndrey Grebennikov proposed a change to openstack/ceilometer: Added Cinder volume/snapshot pollster  https://review.openstack.org/12028712:37
openstackgerritJulien Danjou proposed a change to stackforge/gnocchi: Fix tests with latest releases of testtools  https://review.openstack.org/12040412:38
*** gordc has joined #openstack-ceilometer12:45
*** DrBacchus has joined #openstack-ceilometer12:55
*** ifarkas has quit IRC12:57
*** sdake_ has joined #openstack-ceilometer12:58
*** dzwiedziu has quit IRC13:02
*** ifarkas has joined #openstack-ceilometer13:05
*** thomasem has joined #openstack-ceilometer13:15
*** ildikov has quit IRC13:21
*** sdake_ has quit IRC13:24
*** dzwiedziu has joined #openstack-ceilometer13:28
*** julim has joined #openstack-ceilometer13:28
*** _nadya_ has quit IRC13:28
*** yatin_ has quit IRC13:29
openstackgerritgordon chung proposed a change to openstack/python-ceilometerclient: keystone discovery fallback support  https://review.openstack.org/11893113:32
*** dzwiedziu has quit IRC13:43
*** Longgeek_ has quit IRC13:56
*** yassine has quit IRC13:58
*** _nadya_ has joined #openstack-ceilometer13:59
*** flwang1 has quit IRC14:03
*** rbak has joined #openstack-ceilometer14:08
*** dzwiedziu has joined #openstack-ceilometer14:10
*** Longgeek has joined #openstack-ceilometer14:11
*** rbak has quit IRC14:13
*** yassine has joined #openstack-ceilometer14:13
*** rwsu has joined #openstack-ceilometer14:30
*** Longgeek has quit IRC14:34
*** nsaje has quit IRC14:35
*** dhellmann is now known as dhellmann_14:39
*** _nadya_ has quit IRC14:39
*** dhellmann_ is now known as dhellmann14:40
*** _nadya_ has joined #openstack-ceilometer14:40
openstackgerritJulien Danjou proposed a change to openstack/ceilometer: Switch to using pbr's autodoc capability  https://review.openstack.org/12044114:40
openstackgerritAndrey Grebennikov proposed a change to openstack/ceilometer: Added Cinder volume/snapshot pollster  https://review.openstack.org/12028714:40
*** Longgeek has joined #openstack-ceilometer14:42
*** _nadya_ has quit IRC14:42
*** rwsu has quit IRC14:48
*** Longgeek has quit IRC14:49
*** rmcall has joined #openstack-ceilometer14:59
*** sbfox has joined #openstack-ceilometer15:03
*** _nadya_ has joined #openstack-ceilometer15:12
openstackgerritPrabhakar Kudva proposed a change to openstack/ceilometer: Predictive Fail metric for host with a unit test with doc  https://review.openstack.org/11647015:15
*** sbfox has quit IRC15:31
*** k4n0 has quit IRC15:34
openstackgerritA change was merged to stackforge/gnocchi: Fix tests with latest releases of testtools  https://review.openstack.org/12040415:34
*** arezmerita_ has quit IRC15:36
*** sbfox has joined #openstack-ceilometer15:44
*** amalagon has joined #openstack-ceilometer15:44
*** ifarkas has quit IRC15:44
*** underyx is now known as underyx|off15:47
*** underyx|off is now known as underyx15:50
*** arezmerita has joined #openstack-ceilometer15:51
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Add RSS feed  https://review.openstack.org/12045615:56
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Remove templates from toctrees  https://review.openstack.org/12045715:56
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Use the current date for the copyright statement  https://review.openstack.org/12045815:56
*** stevelle_ has joined #openstack-ceilometer15:58
*** IvanBerezovskiy has left #openstack-ceilometer15:59
*** sbfox has quit IRC15:59
*** sbfox1 has joined #openstack-ceilometer15:59
*** sbfox1 has quit IRC16:01
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Remove templates from toctrees  https://review.openstack.org/12045716:02
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Add RSS feed  https://review.openstack.org/12045616:02
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Use the current date for the copyright statement  https://review.openstack.org/12045816:02
*** sbfox has joined #openstack-ceilometer16:02
*** _cjones_ has joined #openstack-ceilometer16:02
*** _cjones_ has quit IRC16:08
*** _cjones_ has joined #openstack-ceilometer16:09
*** rwsu has joined #openstack-ceilometer16:12
*** _cjones_ has quit IRC16:13
*** dzwiedziu has quit IRC16:19
*** _nadya_ has quit IRC16:19
*** _nadya_ has joined #openstack-ceilometer16:19
*** yassine has quit IRC16:26
*** cdent_ has joined #openstack-ceilometer16:27
*** cdent has quit IRC16:29
*** cdent_ is now known as cdent16:29
*** ViswaV has joined #openstack-ceilometer16:34
*** sbfox has quit IRC16:34
*** sbfox has joined #openstack-ceilometer16:34
*** ViswaV_ has joined #openstack-ceilometer16:37
*** claudiub has quit IRC16:40
*** ViswaV has quit IRC16:40
*** rmcall has quit IRC16:48
*** _cjones_ has joined #openstack-ceilometer16:49
*** DrBacchus has quit IRC16:49
*** rmcall has joined #openstack-ceilometer16:50
*** jbuckley has quit IRC16:50
*** ViswaV_ has quit IRC16:52
*** stevelle_ has quit IRC16:53
*** _cjones_ has quit IRC16:54
*** ViswaV has joined #openstack-ceilometer16:55
*** _cjones_ has joined #openstack-ceilometer16:56
*** rmcall has quit IRC16:56
*** rmcall has joined #openstack-ceilometer16:57
*** harlowja_away is now known as harlowja16:58
*** rbowen has joined #openstack-ceilometer16:58
openstackgerritJoe Hakim Rahme proposed a change to openstack/ceilometer: Run unit tests against MySQL  https://review.openstack.org/11775316:59
*** _cjones_ has quit IRC17:10
openstackgerritSrinivas Sakhamuri proposed a change to openstack/ceilometer: Handle invalid JSON filters from the input gracefully  https://review.openstack.org/12015017:10
*** _cjones_ has joined #openstack-ceilometer17:10
*** _cjones_ has quit IRC17:12
*** _cjones_ has joined #openstack-ceilometer17:13
*** _nadya_ has quit IRC17:15
openstackgerritAndrey Grebennikov proposed a change to openstack/ceilometer: Added Cinder volume/snapshot pollster  https://review.openstack.org/12028717:17
*** rmcall has quit IRC17:18
*** rmcall has joined #openstack-ceilometer17:20
*** _cjones_ has quit IRC17:37
*** _cjones_ has joined #openstack-ceilometer17:38
*** dzwiedziu has joined #openstack-ceilometer17:39
*** safchain has quit IRC17:40
*** _cjones_ has quit IRC17:42
*** dzwiedziu has quit IRC17:43
*** _cjones_ has joined #openstack-ceilometer17:43
*** _nadya_ has joined #openstack-ceilometer17:45
*** joshw42 has joined #openstack-ceilometer17:51
*** joshw42 is now known as jwilmes17:51
*** jwilmes has quit IRC17:52
*** r3pl4y has quit IRC17:54
*** joshw42 has joined #openstack-ceilometer17:54
*** joshw42 has quit IRC17:57
*** rmcall has quit IRC17:59
*** rmcall has joined #openstack-ceilometer18:01
cdentgordc or nealph: If you're around can you confirm success or failure with the following in a devstack: ceilometer sample-create -r 'house' -m cpu_util --meter-type gauge --meter-unit '%' --sample-volume 818:05
nealphcdent: related to a recent patch? (my devtest is a bit out-of-date)18:11
cdentI don't think so, no. If its a problem I think it will be an old one.18:11
*** _nadya_ has quit IRC18:13
nealphcdent: checking. Gimme a bit.18:13
cdentthank you18:13
gordccdent: error for me18:21
gordcTypeError: string indices must be integers, not str18:21
cdentyup, that's the one18:22
cdentwsme is expecting the sent json to be wrapped as a list18:22
cdentbut the client isn't doing that18:22
cdentI don't know if the bug is in the client or in the strong-typing in the server18:22
gordccan't recall if ryanpetrello is wsme or pecan expert.18:23
ryanpetrelloo/18:23
ryanpetrelloI'm generally familiar w/ wsme18:23
ryanpetrelloI'm the maintainer of pecan18:24
gordcryanpetrello: ^^ the above mean anything to you?18:24
gordccdent: i don't know if you have enough yet to explain the issue.18:25
*** arezmerita has quit IRC18:25
cdentI think to avoid going down yet another rabbit hole I'm going to just report the bug, as I'm several layers deep already :)18:25
gordci probably should've asked that first before i bugged ryan.lol18:25
cdent;)18:25
ryanpetrelloyea, if you can report a bug w/ a traceback I can take a peek18:25
cdentcan do18:26
*** safchain has joined #openstack-ceilometer18:28
*** safchain has quit IRC18:30
cdentryanpetrello: I put it here: https://bugs.launchpad.net/python-ceilometerclient/+bug/136786718:31
cdentlet me know if there's additional info I can provide18:32
*** dzwiedziu has joined #openstack-ceilometer18:32
cdentthanks gordc, ryanpetrello, nealph for the help18:32
nealphcdent: sorry for the delay...same result.18:32
ryanpetrelloyea, from the traceback, this looks squarely in the wsme side of things18:32
ryanpetrello*reading18:32
* nealph needs to do some env cleanup18:32
*** _cjones_ has quit IRC18:34
*** _cjones_ has joined #openstack-ceilometer18:34
ryanpetrellocdent: based on "TypeError: string indices must be integers, not str"18:37
ryanpetrelloI read this as `value is anticipated to be a dict, but in this situation, it's a str`18:37
cdentthat's right18:37
cdentbut if you go upstream slightly and wrap the incoming data in a []18:38
cdentit works just fine18:38
ryanpetrellointeresting18:38
*** _cjones_ has quit IRC18:38
ryanpetrellowhich API argument is it?18:39
*** ssakhamuri has joined #openstack-ceilometer18:39
cdentthe posted data is a json dict18:40
cdentif you make it a dict inside a list, disco!18:40
cdentI'm not too savvy on how the type validation on the server works, but I'm guess that if there is a SampleController a POST to it looks for a list18:41
cdentand for whatever reason a list is not being sent18:41
cdentsorry s/SampleController/SamplesController/18:42
*** ViswaV has quit IRC18:44
*** _cjones_ has joined #openstack-ceilometer18:48
*** julim has quit IRC18:51
*** julim has joined #openstack-ceilometer18:53
*** _cjones_ has quit IRC18:56
*** _cjones_ has joined #openstack-ceilometer18:57
*** _cjones_ has quit IRC18:58
*** _cjones_ has joined #openstack-ceilometer18:58
*** rmcall has quit IRC18:59
openstackgerritgordon chung proposed a change to openstack/ceilometer: ceilometer spamming syslog  https://review.openstack.org/12053719:01
openstackgerritgordon chung proposed a change to openstack/ceilometer: ceilometer spamming syslog  https://review.openstack.org/12053719:02
*** dzwiedziu has quit IRC19:05
*** dzwiedziu has joined #openstack-ceilometer19:10
*** _cjones_ has quit IRC19:12
*** _cjones_ has joined #openstack-ceilometer19:12
*** _cjones_ has quit IRC19:14
*** _cjones_ has joined #openstack-ceilometer19:14
*** ViswaV has joined #openstack-ceilometer19:17
cdentryanpetrello: I've updated that bug with some conversation with boris*42, giving a (loose) window of time when the create_sample code did work.19:18
*** mgagne has quit IRC19:18
ryanpetrellocdent: I know I released a new pecan a week or so back19:18
ryanpetrellohave you tried downgrading?19:19
ryanpetrelloor can you report on what version of pecan/wsme you're running here (latest?)19:19
*** promulo has joined #openstack-ceilometer19:19
*** mgagne has joined #openstack-ceilometer19:19
*** mgagne is now known as Guest7525019:19
cdentI think it is the latest, I can try downgrading and see what that does. Suggested version to try?19:19
ryanpetrello< 0.7.0 ?19:21
ryanpetrellothat's what I released about 2 weeks ago19:21
ryanpetrelloI don't think it's responsible for this, but it's worth a try19:21
ryanpetrelloWSME hasn't really changed much since May19:21
ryanpetrellowsme 0.6.1 is May 219:22
cdentActually it appears I've got 0.5.019:22
cdent(of pecan)19:22
ryanpetrellohrm19:22
ryanpetrellotry *upgrading* then :)?19:22
ryanpetrelloI still don't think that's it19:22
ryanpetrellobut worth a shot19:22
cdentyeah, I'm trying that now19:22
*** _nadya_ has joined #openstack-ceilometer19:22
ryanpetrellocould be an incompatibility between the latest wsme and some older version of pecan19:22
cdentmeh, apparently devstack doesn't pip -U. That's frustrating.19:27
cdentryanpetrello: same error present with pecan 0.7.0 and WSME 0.6.119:33
ryanpetrellookay :\19:33
ryanpetrelloyea, I'm guessing this is just some edge case of WSME you've hit :\19:33
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Remove templates from toctrees  https://review.openstack.org/12045719:37
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Use the current date for the copyright statement  https://review.openstack.org/12045819:37
cdentokay ryanpetrello, boris-42 has done some stellar work to narrow down when the problem started for them: 7th Aug, between 11am and 1pm19:39
boris-42cdent ryanpetrello it's better to check between 11a.m. and 5p.m.19:39
cdentthat's based on data of benchmark tests run in the gagte19:39
boris-42cdent ya they started failing at that moment19:40
cdentso something changed then19:40
boris-42cdent and still failing19:40
boris-42cdent btw at that moment there were no changes in rally19:40
cdentyeah guessed that19:40
boris-42cdent related to that part19:40
boris-42cdent okay https://review.openstack.org/#/c/68939/19:42
boris-42cdent this was merged19:42
boris-42cdent I remember that this break some stuff19:42
boris-42cdent end this was partial fix https://review.openstack.org/#/c/112943/19:43
cdentactually you can see the problem in the diff: https://review.openstack.org/#/c/68939/17/ceilometerclient/v2/samples.py,cm19:43
*** rmcall has joined #openstack-ceilometer19:43
cdentnew is not wrapped in []19:43
boris-42cdent that sux19:44
boris-42=)19:44
cdentryanpetrello: you wanna or shall I?19:44
cdentvery good sleuthing boris-4219:44
boris-42cdent sooo19:45
cdentyup, doing the [] wrap fixes it19:53
cdentle sigh19:53
cdentbad tests somewhere19:53
cdentthis change will fix both broken ceilo scenarios in rally19:53
cdentryanpetrello: I'll push it19:53
*** ViswaV has quit IRC19:56
boris-42cdent ryanpetrello great19:59
*** _nadya_ has quit IRC19:59
boris-42cdent maybe we need rally job in ceilo ^_^20:00
boris-42cdent to catch such stuff20:00
cdentyes20:00
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Remove templates from toctrees  https://review.openstack.org/12045720:00
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Add RSS feed  https://review.openstack.org/12045620:00
openstackgerritDoug Hellmann proposed a change to openstack/ceilometer-specs: Use the current date for the copyright statement  https://review.openstack.org/12045820:00
cdentI'm headed that direction, slowly but surely.20:00
*** ViswaV has joined #openstack-ceilometer20:02
openstackgerritChris Dent proposed a change to openstack/python-ceilometerclient: Fix sample-create in v2 api calls  https://review.openstack.org/12056720:03
*** sdake has joined #openstack-ceilometer20:03
*** sdake has joined #openstack-ceilometer20:03
*** sdake has quit IRC20:04
*** sdake has joined #openstack-ceilometer20:05
cdentyup boris-42 with that change I'm running the create_and_query_samples scenario just fine20:05
*** flwang1 has joined #openstack-ceilometer20:08
*** flwang1 has quit IRC20:16
*** ViswaV has quit IRC20:17
ryanpetrellocdent: catching back up20:24
ryanpetrellocdent: makes sense, good catch20:25
cdentit's really a ++ for boris-42, he narrowed it down20:26
*** sbfox has quit IRC20:33
boris-42cdent so we can make a patch in rally, that will break rally gates if this occurs one more time20:39
boris-42cdent wanna try?20:39
cdentI'm down several rabbitholes at the moment20:40
boris-42cdent it's just a few lines of config20:40
boris-42cdent put here https://github.com/stackforge/rally/blob/master/rally-scenarios/rally.yaml#L28520:40
boris-42cdent sla check like here https://github.com/stackforge/rally/blob/master/rally-scenarios/rally.yaml#L304-L30520:40
cdentthat's great, and can do it later, but need to focus on what I'm doing right now, now20:41
boris-42cdent okay =) so when you have time just ping me=20:41
*** sbfox has joined #openstack-ceilometer20:44
*** dzwiedziu has quit IRC20:50
*** ViswaV has joined #openstack-ceilometer20:50
*** dzwiedziu has joined #openstack-ceilometer20:51
*** dzwiedziu has quit IRC20:53
*** dzwiedzi_ has joined #openstack-ceilometer20:53
*** promulo has quit IRC20:53
*** dzwiedzi_ has quit IRC21:07
*** dzwiedziu has joined #openstack-ceilometer21:07
*** dzwiedziu has quit IRC21:11
*** dzwiedzi_ has joined #openstack-ceilometer21:12
*** sbfox has quit IRC21:19
openstackgerritgordon chung proposed a change to openstack/ceilometer: ceilometer spamming syslog  https://review.openstack.org/12053721:20
*** _cjones_ has quit IRC21:21
*** _cjones_ has joined #openstack-ceilometer21:21
*** asalkeld has joined #openstack-ceilometer21:22
*** ViswaV has quit IRC21:23
*** _cjones_ has quit IRC21:26
cdentgordc: my bad on https://review.openstack.org/#/c/120537/21:27
cdentthanks for discovering and fixing21:27
gordccdent: no worries. i didn't notice it myself.21:28
*** _cjones_ has joined #openstack-ceilometer21:28
cdentThe logging part was one of the things that was requiring the import from swift itself (which my changes were trying to fix), so it was a bit clumsy getting it out.21:29
cdentIn other news, gordc, I've been playing with the switch to sqlcore patch + rally21:29
cdentand so far it is about the same with no concurrency (for create_sample) but somewhat worse with a concurrency of ten21:30
cdentI need to redo the tests tomorrow to be sure of my methods (most of today was getting things to work properly)21:31
*** r3pl4y has joined #openstack-ceilometer21:31
gordccdent: that sucks...21:32
cdentalso rather surprising21:32
cdentthus my desire to redo and be sure21:32
cdentthis is based on comparing with today's master, which has the big-sql schema changes21:32
cdent(I guess they merged yesterday?)21:32
gordchow does concurrency work with our api? isn't it a single thread.21:33
gordcyeah, i think the normalised data work merged today.21:33
cdentThat's something I need to clarify: both in terms of how rally is managing concurrency and how ceilometer-api is too21:34
cdentbut if the api is single threaded the tests are kind of meaningless21:34
cdentso, as usual, more poking required21:34
gordccdent: ok, cool cool.21:34
gordccdent: yeah, i haven't really tried multiple requests at once agianst our api... maybe it does work.21:35
cdentbut if the devstack api is single threaded by default, I'll spin it up under mod_wsgi and let apache handle concurrency and see what that does21:35
*** sbfox has joined #openstack-ceilometer21:36
*** _cjones_ has quit IRC21:39
*** _cjones_ has joined #openstack-ceilometer21:40
*** _cjones_ has quit IRC21:42
*** _cjones_ has joined #openstack-ceilometer21:42
*** ViswaV has joined #openstack-ceilometer21:43
gordccdent: cool cool. the extent of my testing has been running make_test_data but that only hits record_metering_data method21:43
gordcthat and flooding it with samples in devstack21:44
openstackgerritgordon chung proposed a change to openstack/ceilometer: ceilometer spamming syslog  https://review.openstack.org/12053721:50
cdentah, gordc, using mod_wsgi makes a huge difference so that sends me back to the drawing board for my comparisons22:03
cdenti'll investigate again tomorrow22:03
gordccdent: cool cool. enjoy your evening... or sleep.22:03
cdenthopefully sleep22:03
gordc:) cheers22:03
*** thomasem has quit IRC22:06
asalkeldhi22:08
asalkeldi have a question22:08
*** flwang1 has joined #openstack-ceilometer22:09
asalkeldi want to make a plugin that can receive all notifications/samples and filter on some thing in the metadata22:09
asalkeldand do what ever i want (I don't want to disrupt the ceilometer pipeline)22:10
asalkeldany suggestions?22:10
*** sdake has quit IRC22:11
*** gordc has quit IRC22:20
*** sbfox has quit IRC22:27
*** sbfox has joined #openstack-ceilometer22:27
*** sbfox has quit IRC22:29
*** sbfox has joined #openstack-ceilometer22:29
*** dzwiedzi_ has quit IRC22:36
*** dzwiedziu has joined #openstack-ceilometer22:37
boris-42asalkeld like for osprofiler?22:39
boris-42asalkeld https://github.com/openstack/ceilometer/blob/master/ceilometer/profiler/notifications.py22:39
asalkeldboris-42, kinda - i want all notifications tho' (compute, image, volume)22:41
asalkeldnot from one queue22:41
boris-42asalkeld so to filter notifications22:41
boris-42asalkeld but it's exactly what you want22:41
*** dzwiedziu has quit IRC22:42
boris-42asalkeld https://github.com/openstack/ceilometer/blob/master/ceilometer/profiler/notifications.py#L45-L5522:42
asalkeldI'll have a look - just taking kids to school22:42
boris-42asalkeld exchanges from different projects22:42
boris-42asalkeld so and then you can use query-samples22:42
boris-42asalkeld to query metadta22:42
asalkeldthanks22:42
boris-42asalkeld http://docs.openstack.org/developer/ceilometer/webapi/v2.html#complex-query22:42
*** nealph has quit IRC22:47
*** sbfox1 has joined #openstack-ceilometer22:59
*** sbfox has quit IRC22:59
*** _nadya_ has joined #openstack-ceilometer23:00
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements  https://review.openstack.org/11009623:02
*** r3pl4y has quit IRC23:03
*** _nadya_ has quit IRC23:04
*** r3pl4y has joined #openstack-ceilometer23:20
*** sbfox1 has quit IRC23:40
*** _cjones_ has quit IRC23:42
*** _cjones_ has joined #openstack-ceilometer23:43
*** _cjones_ has quit IRC23:44
*** _cjones_ has joined #openstack-ceilometer23:44
*** harlowja has quit IRC23:48
*** harlowja_ has joined #openstack-ceilometer23:48

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