Friday, 2015-07-31

« Thursday, 2015-07-30 Index Saturday, 2015-08-01 »
*** jruano has joined #openstack-ceilometer00:00
*** htruta has quit IRC00:05
*** ankita_wagh has quit IRC00:54
*** thorst has joined #openstack-ceilometer00:55
*** boris-42 has joined #openstack-ceilometer00:57
*** thorst has quit IRC00:57
*** ddieterly has joined #openstack-ceilometer01:22
*** pauldix has joined #openstack-ceilometer01:23
openstackgerritliusheng proposed openstack/aodh: Add alembic support for aodh  https://review.openstack.org/20645901:28
*** ddieterl_ has joined #openstack-ceilometer01:43
*** ddieterly has quit IRC01:44
*** ankita_wagh has joined #openstack-ceilometer01:48
*** ankita_wagh has quit IRC01:48
*** ankita_wagh has joined #openstack-ceilometer01:49
*** pauldix has quit IRC01:53
*** pauldix has joined #openstack-ceilometer01:55
*** pauldix has quit IRC01:59
*** ankita_wagh has quit IRC02:19
*** jwcroppe has quit IRC02:28
*** jwcroppe has joined #openstack-ceilometer02:29
*** openstackgerrit has quit IRC02:31
*** openstackgerrit has joined #openstack-ceilometer02:31
*** jwcroppe has quit IRC02:33
*** ankita_wagh has joined #openstack-ceilometer02:47
*** ankita_wagh has quit IRC02:47
*** ankita_wagh has joined #openstack-ceilometer02:48
*** ddieterl_ has quit IRC02:52
*** h00327910__ has quit IRC03:28
*** jwcroppe has joined #openstack-ceilometer03:43
*** fabian1 has joined #openstack-ceilometer03:48
*** ddieterly has joined #openstack-ceilometer03:53
*** edmondsw has quit IRC03:56
*** ddieterly has quit IRC03:57
*** pauldix has joined #openstack-ceilometer03:59
*** pauldix has quit IRC04:04
*** haigang has joined #openstack-ceilometer04:07
*** liusheng has quit IRC04:09
*** liusheng has joined #openstack-ceilometer04:10
*** yprokule has joined #openstack-ceilometer04:22
*** htruta___ has joined #openstack-ceilometer04:23
*** deepthi has joined #openstack-ceilometer04:24
*** htruta___ has quit IRC04:26
*** htruta_____ has joined #openstack-ceilometer04:29
*** htruta_____ has quit IRC04:30
openstackgerritMerged openstack/ceilometer: fix gnocchi resources yaml  https://review.openstack.org/20623504:48
*** ddieterly has joined #openstack-ceilometer04:54
*** fawadkhaliq has joined #openstack-ceilometer04:55
*** ddieterly has quit IRC04:58
openstackgerritMerged openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/20689705:03
openstackgerritMerged openstack/ceilometer: distributed coordinated notifications  https://review.openstack.org/20381805:04
*** jaosorior has joined #openstack-ceilometer05:23
*** ig0r_ has joined #openstack-ceilometer05:25
*** jruano has quit IRC05:29
openstackgerritRohit Jaiswal proposed openstack/python-ceilometerclient: tenant_id not required with keystone v3  https://review.openstack.org/20772405:31
*** jwcroppe has quit IRC05:43
*** ddieterly has joined #openstack-ceilometer05:55
*** ddieterly has quit IRC05:59
*** ig0r__ has joined #openstack-ceilometer06:11
*** ig0r_ has quit IRC06:12
*** jwcroppe has joined #openstack-ceilometer06:15
*** dikonoor has joined #openstack-ceilometer06:28
*** jiaxi has joined #openstack-ceilometer06:43
jiaxiliusheng: Hello06:43
*** lsmola has joined #openstack-ceilometer06:44
*** belmoreira has joined #openstack-ceilometer06:52
*** ddieterly has joined #openstack-ceilometer06:56
*** jwcroppe has quit IRC06:59
*** ddieterly has quit IRC07:00
*** Ala has joined #openstack-ceilometer07:08
*** boris-42 has quit IRC07:10
*** jiaxi has quit IRC07:27
*** exploreshaifali has joined #openstack-ceilometer07:37
*** jaosorior has quit IRC07:44
*** jaosorior has joined #openstack-ceilometer07:47
openstackgerritMerged openstack/ceilometer: Make the partition coordinator log more readable  https://review.openstack.org/20733807:47
silehthum, the kafka publisher is well buggy07:52
haypohi. i have issues with python 3 and gabbi. it looks like cdent wrote gabbi, i guess that he can help :)07:52
haypomaybe i should wait for him07:52
silehthaypo, perhaps I can help, I have already do one patch for gabbi :p07:53
haypowith my latest python 3 patch, https://review.openstack.org/207661 , only gabbi tests are failing07:53
hayposileht: it looks like the unit test expects Unicode, whereas the HTTP response is bytes07:53
hayposileht: the error is truncated, but you should only see types: str not in bytes, http://paste.openstack.org/show/406552/07:54
hayposileht: the pattern _is_ in the output07:55
silehtI see, it fails because we compare bytes with str07:56
haypoceilometer/tests/gabbi/gabbits/resources-empty.yaml:07:56
haypo      response_strings:07:56
haypo          - unrecognized field in query07:56
hayposileht: ^^ it's gabbi "response_strings" check07:56
hayposileht: PyPI says that gabbi is compatible with Python 307:57
hayposo maybe we misuse gabbi?07:57
hayposileht: to be clear, if we fix the 16 gabbi failures, all unit tests with pass on Python 3. maybe it can motivate you :-D07:58
haypoall unit tests _will_ pass, sorry07:58
silehtAt first glance the body is decoded to utf8 by gabbi, I wonder why we get bytes07:59
* sileht continues digging07:59
*** _nadya_ has joined #openstack-ceilometer08:01
*** afazekas has joined #openstack-ceilometer08:09
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: Fixes the kafka publisher  https://review.openstack.org/20776908:21
*** deepthi has quit IRC08:24
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: Fixes the kafka publisher  https://review.openstack.org/20776908:24
*** exploreshaifali_ has joined #openstack-ceilometer08:28
*** exploreshaifali has quit IRC08:30
*** yassine has joined #openstack-ceilometer08:40
*** exploreshaifali_ has quit IRC08:54
*** exploreshaifali has joined #openstack-ceilometer08:54
*** ddieterly has joined #openstack-ceilometer08:57
*** nijaba has quit IRC09:01
*** ddieterly has quit IRC09:02
*** exploreshaifali has quit IRC09:02
*** ityaptin has joined #openstack-ceilometer09:02
*** exploreshaifali has joined #openstack-ceilometer09:02
ityaptinjd__: Hi!09:03
ityaptinjd__: Are you working on InlfuxDb right now? Do you mind if I update this CR with get_cross_metric functional today?09:04
jd__ityaptin: go ahead09:05
jd__ityaptin: though It'd be more comfortable if we fix this patch, merge it, and then add new features09:06
ityaptinjd__: Ok, understand. Then I will start to research failing tests now.09:08
jd__thanks ityaptin09:08
*** ankita_wagh has quit IRC09:26
*** hhuang has joined #openstack-ceilometer09:28
*** ankita_wagh has joined #openstack-ceilometer09:32
openstackgerritIlya Tyaptin proposed openstack/ceilometer: Initial separating unit and functional tests  https://review.openstack.org/20412809:47
*** ankita_wagh has quit IRC09:51
*** _nadya_ has quit IRC09:59
*** _nadya_ has joined #openstack-ceilometer09:59
*** pauldix has joined #openstack-ceilometer10:02
*** _nadya_ has quit IRC10:04
*** pauldix has quit IRC10:06
*** fabian1 has left #openstack-ceilometer10:08
ityaptinjd__: Also I found a bug in SearchMetricController. We don't cast unicode time str to datetime. How do you think it will be enough if i fix it in InlfuxDb patch or I should create separate CR?10:50
*** cdent has joined #openstack-ceilometer10:51
*** eglynn has joined #openstack-ceilometer10:54
*** ddieterly has joined #openstack-ceilometer10:59
*** _nadya_ has joined #openstack-ceilometer11:02
*** ddieterly has quit IRC11:03
*** eglynn_ has joined #openstack-ceilometer11:06
*** eglynn has quit IRC11:09
*** ig0r__ has quit IRC11:15
openstackgerritChris Dent proposed openstack/aodh: Clean up permissions in function post_test_hook  https://review.openstack.org/20784011:16
*** ig0r_ has joined #openstack-ceilometer11:19
*** afazekas has quit IRC11:31
*** dikonoor has quit IRC11:34
*** pauldix has joined #openstack-ceilometer11:40
*** eglynn_ has quit IRC11:49
*** exploreshaifali has quit IRC11:50
openstackgerritMerged openstack/aodh: Port remaining tests to Python 3  https://review.openstack.org/20754711:55
*** thorst has joined #openstack-ceilometer11:55
*** bdossant has joined #openstack-ceilometer11:55
*** pauldix has quit IRC11:58
*** ddieterly has joined #openstack-ceilometer11:59
openstackgerritIlya Tyaptin proposed openstack/gnocchi: InfluxDB storage driver  https://review.openstack.org/16540712:04
*** ddieterly has quit IRC12:04
*** gordc has joined #openstack-ceilometer12:09
jd__ityaptin: do you want a hand to fix those last tests in InfluxDB?12:12
*** ddieterly has joined #openstack-ceilometer12:14
*** ddieterly has quit IRC12:24
silehtcdent, https://github.com/cdent/python3-wsgi-intercept/pull/3212:24
*** haigang has quit IRC12:25
silehthaypo, ^12:26
* cdent looks12:26
*** thumpba has joined #openstack-ceilometer12:28
silehtcdent, ceilometer does that changing header after start_response have been run12:29
cdenta) that's a very nice catch b) shame on ceilometer for doing that12:29
silehtswift does that too12:30
cdentis weak sauce12:30
cdentswift is like the poster child of "does weird stuff"12:30
* cdent merges12:30
silehtcdent, haypo found the bug, because on py34 we can't compare bytes and str12:30
silehtcdent, and because the headers are missed, the body is not decoded12:31
silehthaypo, only 4 tests fails now on py3412:31
*** eglynn_ has joined #openstack-ceilometer12:31
silehtcdent, who should handle this exception http://paste.openstack.org/show/406586/ ?12:32
silehtseems come from test with xfail: true12:33
*** thumpba has quit IRC12:33
cdentsileht: that'll be a bug from 1.1.5 release last night12:33
silehtoh cool12:33
cdentroll back to gabbi 1.1.4 while I fix it12:33
* sileht downgrade12:33
cdentunexpected success is I think the main area of not proper coverage in gabbi12:34
*** dikonoor has joined #openstack-ceilometer12:35
*** edmondsw has joined #openstack-ceilometer12:36
*** _nadya_ has quit IRC12:36
*** _nadya_ has joined #openstack-ceilometer12:36
cdentsileht: new wsgi-intercept released12:37
*** eglynn_ has quit IRC12:39
*** eglynn_ has joined #openstack-ceilometer12:39
*** _nadya_ has quit IRC12:40
*** _nadya_ has joined #openstack-ceilometer12:41
jd__cdent: there are +2A holes in my branch :D12:41
jd__cdent: in my clean up conf things for aodh12:41
cdentjd__: noted12:41
haypo"14:37 < cdent> sileht: new wsgi-intercept released" woooot12:43
cdenthaypo: now working on fixing the gabbi unexpected success problem12:44
silehthaypo, just one test fail now, I'm fixing it12:44
haypo"unexpected success problem" haha, i like this concept12:45
hayposileht: ok. i let you fix remaining issues12:45
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: mongo: fix lastest python3 bug  https://review.openstack.org/20787212:45
silehthaypo,  ^12:46
hayposileht: latest or last?12:46
silehthaypo, both is more powerfull ;p12:46
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: mongo: fix latest python3 bug  https://review.openstack.org/20787212:47
hayposileht: i used sort(key=lambda obj: sorted(obj.items()) in some projets12:47
haypoprojects12:47
hayposileht: but i also like your choice to fix it ;)12:48
hayposileht: hum. i'm not sure that dict items will always be in the same order12:49
hayposileht: dict buckets are filled randomly12:49
silehthaypo, I will change for you version so12:50
*** jiaxi has joined #openstack-ceilometer12:50
*** ddieterly has joined #openstack-ceilometer12:52
*** ddieterly has quit IRC12:53
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: fix latest python3 bugs  https://review.openstack.org/20787212:53
silehthaypo, with your version ^12:54
silehtit's more readable12:54
silehthaypo, does ceilometer the first application that have all unittests that pass py34 ?12:54
hayposileht: sorry, o12:56
haypono12:56
hayposileht: https://wiki.openstack.org/wiki/Python3#OpenStack_applications12:57
silehtsniff12:57
hayposileht: gnocchi, ironic, rally have a full python3 support12:57
hayposileht: aodh maybe, i don't know if my patch is accepted or not ;)12:57
hayposileht: ah, it's merged ;)12:57
silehthaypo, http://paste.openstack.org/show/406590/ \o/12:58
* sileht run py34 test on mysql13:00
*** eglynn_ has quit IRC13:04
silehtall py34 tests pass mysql backend too :)13:04
hayposileht: oh, you event wrote a uni test for python3-wsgi-intercept13:05
hayposileht, cdent : there are one version of wsgi-intercept for python 3, and another for python 2? two code bases?13:06
cdenthaypo: no, same version13:06
cdentit's just the repo name that is python313:06
cdentit is because at the time there were several version of wsgi-intercept sort of working towards the future13:06
cdentand I started working on a pure python3 one13:07
cdentand it turned out that it was easy to make work in both and I sort of ended up the official maintainer13:07
cdentI should probably change the name of the repo13:07
cdentwhat do you think?13:07
haypocdent: change the name of the repo to have the same name than PyPI13:07
*** eglynn_ has joined #openstack-ceilometer13:08
*** eglynn_ has quit IRC13:08
hayposorry, i didn't check the name of these projects13:08
hayposileht was faster than me13:08
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: api: dogpile cache for getting resource  https://review.openstack.org/20752113:10
hayposileht: did you fix the remaining "/ 100"?13:15
silehthaypo, no, tests are passing :p13:15
hayposileht: i see, you hate opendaylight13:15
silehthaypo, I hate untested stuffs13:16
hayposileht: ok, i'm writing a fix13:16
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: gate: fixup gabbi live tests  https://review.openstack.org/20651013:18
*** htruta has joined #openstack-ceilometer13:18
hayposileht: in ceilometer/compute/virt/vmware/inspector.py, "cpu_util = cpu_util / 100" is supposed to return a float or an int?13:19
silehthaypo, float I guess13:19
openstackgerritMehdi Abaakouk (sileht) proposed openstack/gnocchi: Don't update resource if nothing has changed  https://review.openstack.org/20752213:22
openstackgerritVictor Stinner proposed openstack/ceilometer: Fix Python 3 issue in opendaylight client  https://review.openstack.org/20789613:23
cdentsileht, haypo: new gabbi just released, ought to fix the unexpected success problem13:23
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: Fixes the kafka publisher  https://review.openstack.org/20776913:23
*** fabian4 has joined #openstack-ceilometer13:24
openstackgerritMehdi Abaakouk (sileht) proposed openstack/ceilometer: mongo: fix last python3 bugs  https://review.openstack.org/20787213:26
*** primechuck has joined #openstack-ceilometer13:26
silehtcdent, cool https://review.openstack.org/#/c/207872/ should pass the py34 job on next pypi mirror sync, so13:26
jiaxihttps://github.com/openstack/ceilometer/blob/master/ceilometer/agent/base.py L221-L223  What does the code mean ?13:31
cdentjiaxi: it's possible to run the polling agent with a limited set of pollster plugins13:32
cdentso that's basically saying: only give me the extensions declared in pollster_list13:33
cdentpollster_list isn't often used, from what I can tell13:33
jiaxicdent: the code means running the pollster plugins ? Not like13:34
jiaxicdent: only use namespace ?13:34
cdentI'm sorry, I don't really understand your question.13:34
jiaxidef __init__(self, namespaces, pollster_list, group_prefix=None):13:35
jiaxipollster_list is often None13:35
jiaxinamespace == 'computer' or 'central'13:35
jiaxicomputer -> compute13:35
cdentyes13:36
jiaxicdent:  https://github.com/openstack/ceilometer/blob/master/ceilometer/agent/base.py L225  What does L225 mean ?13:37
cdentthe previous few lines are determing which extensions (PollsterPlugins) are going to be available to the polling agent, line 225 establishes them for later use by the polling agent13:38
cdentlater in the code when polling tasks are being created, the extension will be used to determine how they are created13:39
gordcsileht: is your patch suppose to be dependent on https://review.openstack.org/#/c/207896/13:41
gordcpy3 patch13:41
jiaxicdent: Great  explain . Thank you . cdent.13:41
silehtgordc, yes13:41
silehtgordc, no13:41
gordclol13:41
silehtgordc, int(r.status_code / 100) and r.status_code // 10013:42
silehtare the same thing13:42
silehtgordc,just the opendaylight is a really fix13:42
gordcright... so you don't need that to enable all py3 tests?13:43
silehtgordc, no13:43
silehtgordc, opendaylight tests just doesn't catch that py3 bug13:43
gordci'm just going to trust you. too lazy to wait for check.13:44
gordcsileht: i don' think this is possible but is there a way to update only the endpoints of a listener? or do you have to recreate listener with new endpoints?13:48
* sileht thinks13:48
silehtgordc, no you have to recreate the listener because the endpoints define the queue/topic/exchange to listen on13:49
*** eglynn_ has joined #openstack-ceilometer13:49
gordcsileht: yeah, that's what i figured. thanks for confirmation.13:50
*** jiaxi has quit IRC13:51
silehtgordc, with kombu it's totally realisty to subscribe/unsubscribe some stuffs, but oslo.messaging is really inflexible for that13:52
gordcis it because the endpoints all share one connection?13:53
*** jiaxi has joined #openstack-ceilometer13:54
silehtgordc, this is true, but that's not really the problem13:54
*** jaosorior has quit IRC13:54
silehtgordc, this is because we browse endpoint to get the list of the implemented method, to listen only those implements13:56
silehtie: you impement 'info', so we listen '<the-target-or-config-topics>-info'13:57
*** IBerezovskiy has joined #openstack-ceilometer13:58
gordcsileht: ah. got it. it does this all at startup.13:58
silehtgordc, yes, I think we could do later but we need to add a new driver API, I see how this can be done for rabbit, but I have no idea for other.13:59
cdentjd__: I think I got most of them, there are two that still have unresolved comments13:59
gordcsileht: no worries. not that important.14:00
*** rbak has joined #openstack-ceilometer14:00
gordcsileht: i think if you swap endpoints while listener is active you might end up missing notifications on some endpoints anyways14:01
silehtgordc, when your notification ring need to rebuilded, all connections to rabbit are closed, then reopen, when we just need to send to rabbit msgs14:01
sileht*to send some rabbit msgs*14:01
silehtgordc, anyways that just an improvements14:01
gordcsileht: agreed.14:02
openstackgerritDrew Thorstensen proposed openstack/ceilometer: Provide base method for inspect_memory_resident  https://review.openstack.org/20790714:03
*** IBerezovskiy has quit IRC14:03
*** eglynn_ has quit IRC14:04
*** IBerezovskiy has joined #openstack-ceilometer14:04
jd__cdent: thanks I'll reply/fix14:05
silehtgordc, this you can take a look to https://review.openstack.org/#/c/207137/ , I'm sure a PTL like this kind of tests :p14:07
gordcsileht: i don't like tests14:08
gordcsileht: are we going to create gate/check on this?14:09
silehtgordc, yes sir14:09
gordc:)14:09
*** IBerezovskiy has quit IRC14:11
*** fabian4 has quit IRC14:17
*** jruano has joined #openstack-ceilometer14:21
*** h00327910__ has joined #openstack-ceilometer14:24
*** prad has joined #openstack-ceilometer14:26
gordchmmm we need a way to have both gnocchi enabled and events...14:27
*** Ala has quit IRC14:28
*** nijaba has joined #openstack-ceilometer14:31
cdentgordc: isn't it good we are actually trying to use the things we are creating. so many holes we find. is good14:33
silehtgordc, create two topic on client side (nova/glance/...) and 1 collector for each purpose14:33
cdentsileht: that's an onerous requirement for exisiting installations14:34
gordcsileht: that's the workaround i proposed: https://bugs.launchpad.net/ceilometer/+bug/148033314:37
openstackLaunchpad bug 1480333 in Ceilometer "event consumption needs to be disabled on gnocchi dispatcher" [Undecided,New]14:37
gordcwe need to publish to a non-default topic and have a second collector14:37
gordcthe only way to get this working in default is to have gnocchi dispatch also set up a event connection...14:38
openstackgerritMerged openstack/aodh: storage: always use get_connection_from_config()  https://review.openstack.org/20556114:38
*** fabian4 has joined #openstack-ceilometer14:38
openstackgerritMerged openstack/aodh: tests.db: simplify connection handling  https://review.openstack.org/20556214:39
openstackgerritMerged openstack/aodh: mongodb: stop using global config object  https://review.openstack.org/20556314:39
*** fawadkhaliq has quit IRC14:41
*** dikonoor has quit IRC14:42
gordcsileht: what's the local.conf configuration i need to give more workers to swift for gnocchi?14:43
*** exploreshaifali has joined #openstack-ceilometer14:45
*** ddaskal has joined #openstack-ceilometer14:50
gordcalso, i think i asked last week. but i believe gnocchi has a hard requirement on valid uuid... most of OpenStack is non-sense with ids... what do we do?14:51
*** fawadkhaliq has joined #openstack-ceilometer14:54
openstackgerritMerged openstack/aodh: api: fix alarm deletion and update  https://review.openstack.org/20714014:58
ityaptinjd__: I found what we can fix it if add  lazy='joined' parameter to sqalachemy_base.Metric: https://github.com/openstack/gnocchi/blob/master/gnocchi/indexer/sqlalchemy_base.py#L15314:59
*** fawadkhaliq has quit IRC14:59
ityaptinjd__: Aslo, this issue affects not only tests but real running too.15:00
*** jiaxi has quit IRC15:00
ityaptinjd__: If we add lazy parameter then inspect from [1] think what archive_policy loaded and show it in json output. [1] https://github.com/openstack/gnocchi/blob/master/gnocchi/indexer/sqlalchemy_base.py#L17615:03
openstackgerritRohit Jaiswal proposed openstack/python-ceilometerclient: tenant_id not required with keystone v3  https://review.openstack.org/20772415:03
*** exploreshaifali has quit IRC15:04
*** fawadkhaliq has joined #openstack-ceilometer15:06
openstackgerritIlya Tyaptin proposed openstack/gnocchi: InfluxDB storage driver  https://review.openstack.org/16540715:08
ityaptinjd__: ^^^ Tests pass, but even we query metrics without loading `archive_policy` it loads in jsonify.15:09
jd__ityaptin: but all tests pass?15:10
ityaptinjd__: yep15:11
jd__ityaptin: so I don't get what the problem is? :)15:13
ityaptinjd__: it affects expect output of several requests :(15:14
jd__ityaptin: and that's not caught by tests?15:14
ityaptinjd__: Not15:14
jd__ityaptin: could you write a test then?15:14
jd__I'd fix the issue15:15
ityaptinjd__: I afraid I don't have time to write the test today :(15:15
gordcjd__: is there some configuration you need to do to get influxdb running test?15:16
gordci always see:  requests.exceptions.ConnectionError: ('Connection aborted.', error(111, 'Connection refused'))15:16
jd__ityaptin: ok no worry, that can wait next week :)15:16
jd__gordc: are you sure you have influx 0.9?15:16
ityaptinjd__: And next week :( I am going to vacation since Monday.15:17
jd__gordc: likely the bash script does not start it and the test continues – I already noticed that15:17
jd__ityaptin: ah damn it – do you still have time to at least give me an example in the review so I can investigate?15:17
gordcjd__: hmm.. i have 0.9.2. i'll check script15:17
ityaptinjd__: Yes, give me a minute.15:18
*** ankita_wagh has joined #openstack-ceilometer15:18
ityaptinjd__: http://pastebin.com/sUxmphcv15:20
*** haypo has left #openstack-ceilometer15:21
gordcjd__: stupid path issue. https://github.com/influxdb/influxdb/issues/293315:21
ityaptinjd__: Metric.archive_policy[1] is loaded in function Metric.jsonify(), because Metric inspection[3] thinks what archive_policy is loaded [3].15:22
openstackgerritRohit Jaiswal proposed openstack/python-ceilometerclient: tenant_id not required with keystone v3  https://review.openstack.org/20772415:22
ityaptinjd__: [1]https://github.com/openstack/gnocchi/blob/master/gnocchi/indexer/sqlalchemy_base.py#L15315:22
ityaptinjd_: [3] https://github.com/openstack/gnocchi/blob/master/gnocchi/indexer/sqlalchemy_base.py#L17215:23
*** belmoreira has quit IRC15:24
jd__ityaptin: where do you see that what you mark as "expected" is actually expected?15:24
jd__AFAIK what's expected is what is returned, i.e. the full archive policy description15:25
jd__andI don't think it's a problem with InfluxDB actually15:25
ityaptinjd__: It's not Influxdb fail, but I fix issue with adding lazy="joined" to Metric.archive_policy relationship15:27
ityaptinjd__: expected I see at gnocchi master15:27
jd__ok I'll check that out15:28
ityaptinjd__: By default we expected a short metric definition output in /v1/metric/, but after my patch /v1/metric show full definition15:28
jd__ityaptin: yeah I get that I just don't see the relationship with Influx yet but I'll take a look into it don't worry :)15:29
jd__I've added a note on the patch so I check that on Monday15:29
jd__if the rest works, we'll merge the patch :)15:30
ityaptinjd__: ok, cool, thanx!15:30
jd__thanks to you for your work ityaptin :)15:30
openstackgerritMerged openstack/ceilometer: Fix WSGI replacement_start_response() on Python 3  https://review.openstack.org/20755115:31
*** ankita_wagh has quit IRC15:31
*** ankita_wagh has joined #openstack-ceilometer15:32
*** fabian4 has left #openstack-ceilometer15:34
openstackgerritSaba proposed openstack/ceilometer: Corrected unit of snmp based harware disk and memory meters  https://review.openstack.org/20719915:37
*** jruano has quit IRC15:37
openstackgerritJulien Danjou proposed openstack/aodh: tests.api: remove unused argument/config option  https://review.openstack.org/20596915:43
openstackgerritJulien Danjou proposed openstack/aodh: api: fix alarm group declaration  https://review.openstack.org/20647015:43
openstackgerritJulien Danjou proposed openstack/aodh: api: remove global conf and local pecan config  https://review.openstack.org/20597115:44
openstackgerritJulien Danjou proposed openstack/aodh: gnocchi: stop using global conf object for Gnocchi evaluator  https://review.openstack.org/20647115:44
openstackgerritJulien Danjou proposed openstack/aodh: api: remove force_canonical option  https://review.openstack.org/20597015:44
openstackgerritJulien Danjou proposed openstack/aodh: api: stop using a global Enforcer object  https://review.openstack.org/20557215:44
openstackgerritJulien Danjou proposed openstack/aodh: evaluator: remove global conf usage from threshold evaluator  https://review.openstack.org/20597315:44
openstackgerritJulien Danjou proposed openstack/aodh: api.hooks: stop using global conf object  https://review.openstack.org/20557115:44
openstackgerritJulien Danjou proposed openstack/aodh: rpc: remove global conf usage from notifier  https://review.openstack.org/20597215:44
openstackgerritJulien Danjou proposed openstack/aodh: mongodb: stop relying on global conf object in utils  https://review.openstack.org/20597515:44
openstackgerritJulien Danjou proposed openstack/aodh: mongodb: replace custom retry code by retrying  https://review.openstack.org/20597415:44
openstackgerritJulien Danjou proposed openstack/aodh: storage: only retry connection, not driver finding  https://review.openstack.org/20658915:44
openstackgerritJulien Danjou proposed openstack/aodh: Stop using global conf object  https://review.openstack.org/20657315:44
*** IBerezovskiy has joined #openstack-ceilometer15:47
*** dikonoor has joined #openstack-ceilometer15:51
cdentjd__: I +A those two I was being whiny about15:52
jd__thanks cdent I actually fixed your comments :)15:52
cdentyeah, and I exclaimed on gerrit just how much better that's going to make my weekend15:53
cdentwithout those changes I would have been in a deep blue funk15:53
*** bdossant has quit IRC16:03
*** ildikov has quit IRC16:09
openstackgerritSaba proposed openstack/ceilometer: Corrected unit of snmp based harware disk and memory meters  https://review.openstack.org/20719916:10
*** _cjones_ has joined #openstack-ceilometer16:11
*** rbak has quit IRC16:17
*** ildikov has joined #openstack-ceilometer16:21
*** dikonoor has quit IRC16:23
*** yprokule has quit IRC16:25
*** ddieterly has joined #openstack-ceilometer16:31
*** primechuck has quit IRC16:33
*** primechuck has joined #openstack-ceilometer16:33
*** primechuck has quit IRC16:33
cdentjd__ ++ on the blog post, really nice. I'll give it a review for typos and the like later but the content is good.16:33
*** primechuck has joined #openstack-ceilometer16:33
jd__thanks cdent!16:33
openstackgerritOpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements  https://review.openstack.org/20565016:35
*** rbak has joined #openstack-ceilometer16:35
*** ddieterly has quit IRC16:37
*** briancline has joined #openstack-ceilometer16:43
*** ddieterly has joined #openstack-ceilometer16:45
*** rbak has quit IRC16:46
*** ddieterly has quit IRC16:49
*** _nadya_ has quit IRC16:52
*** _nadya_ has joined #openstack-ceilometer16:53
*** _nadya_ has quit IRC16:57
*** rbak has joined #openstack-ceilometer17:00
*** rbak has quit IRC17:03
*** rbak has joined #openstack-ceilometer17:05
*** IBerezovskiy has quit IRC17:05
openstackgerritChris Dent proposed openstack/aodh: Correct database functional tests  https://review.openstack.org/20784017:07
cdentjd__: if you're still around that ^ is pretty important17:07
cdentthe mysql unit tests had been skipping, the pymsql driver wasn't able to be used, and a whole slew of stuff with the tests was not actually doing what I think we thought they were17:08
cdenta test run without my changes http://logs.openstack.org/40/207840/1/check/gate-aodh-python27/9d7aed5/console.html#_2015-07-31_13_31_08_969 shows too many skips17:09
*** Ephur has joined #openstack-ceilometer17:10
*** cdent_ has joined #openstack-ceilometer17:13
*** cdent has quit IRC17:14
*** cdent_ is now known as cdent17:14
cdentgordc: you still around17:18
cdentI have some findings to report on long running ceilo+gnocchi17:18
*** pauldix has joined #openstack-ceilometer17:19
*** ankita_wagh has quit IRC17:22
*** _nadya_ has joined #openstack-ceilometer17:23
*** fawadkhaliq has quit IRC17:23
*** fawadkhaliq has joined #openstack-ceilometer17:23
openstackgerritMerged openstack/aodh: api.hooks: stop using global conf object  https://review.openstack.org/20557117:26
openstackgerritMerged openstack/aodh: api: stop using a global Enforcer object  https://review.openstack.org/20557217:27
openstackgerritMerged openstack/aodh: tests.api: remove unused argument/config option  https://review.openstack.org/20596917:27
openstackgerritMerged openstack/aodh: api: remove force_canonical option  https://review.openstack.org/20597017:27
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Fixes querying alarm history with severity field  https://review.openstack.org/20023817:28
*** _nadya_ has quit IRC17:32
openstackgerritMerged openstack/aodh: api: remove global conf and local pecan config  https://review.openstack.org/20597117:34
openstackgerritMerged openstack/aodh: rpc: remove global conf usage from notifier  https://review.openstack.org/20597217:34
openstackgerritMerged openstack/aodh: evaluator: remove global conf usage from threshold evaluator  https://review.openstack.org/20597317:35
*** ankita_wagh has joined #openstack-ceilometer17:42
*** rbak has quit IRC17:52
*** exploreshaifali has joined #openstack-ceilometer17:52
*** ankita_w_ has joined #openstack-ceilometer18:02
*** rbak has joined #openstack-ceilometer18:04
*** ankita_wagh has quit IRC18:05
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Masks messaging_urls in logs during debug mode  https://review.openstack.org/20801118:06
*** rbak has quit IRC18:07
openstackgerritMerged openstack/aodh: mongodb: replace custom retry code by retrying  https://review.openstack.org/20597418:09
openstackgerritMerged openstack/aodh: mongodb: stop relying on global conf object in utils  https://review.openstack.org/20597518:09
openstackgerritMerged openstack/aodh: api: fix alarm group declaration  https://review.openstack.org/20647018:10
openstackgerritMerged openstack/aodh: gnocchi: stop using global conf object for Gnocchi evaluator  https://review.openstack.org/20647118:10
openstackgerritMerged openstack/aodh: Stop using global conf object  https://review.openstack.org/20657318:10
openstackgerritMerged openstack/aodh: storage: only retry connection, not driver finding  https://review.openstack.org/20658918:10
openstackgerritMerged openstack/ceilometer: Remove the compute inspector choice restriction  https://review.openstack.org/20759218:11
openstackgerritMerged openstack/ceilometer: tests: add an integration test  https://review.openstack.org/20713718:11
*** ankita_w_ has quit IRC18:11
*** ankita_wagh has joined #openstack-ceilometer18:11
*** ildikov has quit IRC18:19
*** pauldix has quit IRC18:27
*** pauldix has joined #openstack-ceilometer18:27
openstackgerritMichael Bayer proposed openstack/gnocchi: Switch to facade  https://review.openstack.org/20466918:32
*** ildikov has joined #openstack-ceilometer18:36
openstackgerritChris Dent proposed openstack/aodh: Correct database functional tests  https://review.openstack.org/20784018:39
*** fawadkhaliq has quit IRC18:40
*** jruano has joined #openstack-ceilometer18:46
*** exploreshaifali has quit IRC18:49
*** rbak has joined #openstack-ceilometer18:52
*** ddieterly has joined #openstack-ceilometer18:56
*** ddieterly has quit IRC18:57
*** ddieterly has joined #openstack-ceilometer18:58
*** fawadkhaliq has joined #openstack-ceilometer19:05
*** fawadkhaliq has quit IRC19:09
*** rjaiswal has joined #openstack-ceilometer19:13
rjaiswalgordc: put some comments on https://review.openstack.org/#/c/207724/19:14
gordcrjaiswal: k. i'll take a look19:16
rjaiswalthanks gordc19:17
*** afazekas has joined #openstack-ceilometer19:18
*** _nadya_ has joined #openstack-ceilometer19:20
gordcrjaiswal: k. thanks for pointing out shell part19:20
gordci don't know understand where we are validating it in client19:20
gordcit's not actually validated in the part you linked. it just uses one or the other19:21
gordchttps://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/client.py#L232-L23519:21
rjaiswalfrom here, i think:https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/client.py#L25919:24
rjaiswalwhich in turn from here - https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/v2/client.py#L6819:25
*** ig0r_ has quit IRC19:26
*** afazekas has quit IRC19:26
gordcrjaiswal: right, but if we don't pass in project or tenant then it'll just be None.19:27
gordchttps://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/client.py#L232-L235 doesn't actually validate. it just takes one or the other19:27
gordcbut if both are None. it'll still keep going... and i think we always need tenant or project.19:28
rjaiswalYes, the shell does the validation, i think19:28
rjaiswalhttps://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/shell.py#L21919:28
rjaiswalhttps://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/shell.py#L18419:29
gordcright. so the shell is validated, so if you run command line it'll be valided. if i run client in python it won't go through shell part.19:30
rjaiswalgordc: sure, makes sense19:30
rjaiswali will add the validation of the shell in the client as well19:30
rjaiswalwould that work19:31
gordcrjaiswal: cool :). i think that makes sense... or it could just be friday.19:31
gordcrjaiswal: had a question but why is the test change needed?19:31
gordci think i forgot to ask that.19:31
rjaiswalso the test was passing initially with a command error because of the bug, but with the fix it raised an unauthorized error and not a command error19:32
*** openstack has joined #openstack-ceilometer19:33
*** ChanServ sets mode: +v openstackstatus19:33
rjaiswalwhen we mocked it with the AuthPlugin, it actually started testing the V3 code19:33
gordchm that's so weird. i'll just accept it as so weird test we have.19:34
rjaiswalso now its similar to https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/tests/unit/test_shell.py#L11719:34
rjaiswalthe bug fix uncovered a test regression19:35
gordcah cool. nice19:35
rjaiswalthanks gordc, will make the change and upload a new patchset19:36
gordcrjaiswal: also regarding your listener patch. i have no idea how to do it. i think it's something we have to settle with... or wait for pool feature.19:36
gordcsounds good19:36
rjaiswalok19:36
rjaiswalgordc: yeah, i wanted to have that fixed before introducing dynamic event pipeline19:37
gordcyeah... it'd be better to do it that way. but i think as long as pipeline's aren't adjusted a lot it should be ok.19:38
gordcyou can raise priority if you find the above isn't true isn't reality.19:39
gordcin reality*19:39
rjaiswalok19:44
*** ddaskal has quit IRC19:50
*** eglynn_ has joined #openstack-ceilometer19:52
*** cdent has quit IRC20:11
*** eglynn_ has quit IRC20:17
*** cdent has joined #openstack-ceilometer20:22
*** ildikov has quit IRC20:24
*** _nadya_ has quit IRC20:33
*** _nadya_ has joined #openstack-ceilometer20:33
*** hhuang has quit IRC20:34
openstackgerritMerged openstack/ceilometer: Provide base method for inspect_memory_resident  https://review.openstack.org/20790720:35
openstackgerritRohit Jaiswal proposed openstack/python-ceilometerclient: tenant_id not required with keystone v3  https://review.openstack.org/20772420:35
*** _nadya_ has quit IRC20:38
*** ildikov has joined #openstack-ceilometer20:41
*** ankita_w_ has joined #openstack-ceilometer20:42
*** ankita_w_ has quit IRC20:43
*** ankita_wagh has quit IRC20:44
*** hhuang has joined #openstack-ceilometer20:47
*** thorst has quit IRC20:50
openstackgerritChris Dent proposed openstack/aodh: Correct database functional tests  https://review.openstack.org/20784021:01
*** fawadkhaliq has joined #openstack-ceilometer21:06
*** changbl has quit IRC21:09
*** fawadkhaliq has quit IRC21:11
*** prad has quit IRC21:14
*** cdent has quit IRC21:15
*** ddaskal has joined #openstack-ceilometer21:19
*** boris-42 has joined #openstack-ceilometer21:23
*** superflyy has joined #openstack-ceilometer21:31
*** yassine has quit IRC21:31
*** primechuck has quit IRC21:43
*** thorst has joined #openstack-ceilometer21:51
*** thorst has quit IRC21:53
*** ildikov has quit IRC21:56
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Adds support for dynamic event pipeline  https://review.openstack.org/20806822:01
*** gordc has quit IRC22:04
*** primechuck has joined #openstack-ceilometer22:04
openstackgerritMerged openstack/ceilometer: gnocchi: reduce the number of patch to gnocchi API  https://review.openstack.org/20258922:05
*** htruta_____ has joined #openstack-ceilometer22:07
*** pauldix has quit IRC22:11
*** ddaskal has quit IRC22:11
*** pauldix has joined #openstack-ceilometer22:12
*** htruta_____ has quit IRC22:20
*** htruta has quit IRC22:21
*** ddieterly has quit IRC22:21
*** jruano has quit IRC22:40
*** superflyy has quit IRC22:52
*** primechuck has quit IRC23:10
*** _cjones_ has quit IRC23:18
*** pauldix has quit IRC23:31
*** julim has quit IRC23:37
*** rjaiswal has quit IRC23:50
« Thursday, 2015-07-30 Index Saturday, 2015-08-01 »

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