Tuesday, 2015-02-24

*** rbak has quit IRC00:02
*** thomasem has quit IRC00:05
*** thomasem has joined #openstack-ceilometer00:10
*** ccrouch has quit IRC00:15
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer  https://review.openstack.org/15851900:42
*** zul has quit IRC00:51
*** zul has joined #openstack-ceilometer00:52
*** promulo has joined #openstack-ceilometer00:56
*** promulo__ has quit IRC00:56
*** drjones has quit IRC01:06
*** adam_g is now known as adam_g_out01:09
*** devlaps has quit IRC01:11
openstackgerritEric Brown proposed openstack/ceilometer: Use secure default for bind interfaces  https://review.openstack.org/15852301:11
*** zhenq1 has joined #openstack-ceilometer01:14
*** ddieterly has joined #openstack-ceilometer01:15
*** sbfox has quit IRC01:17
*** thomasem has quit IRC01:18
*** david-lyle is now known as david-lyle_afk01:19
*** promulo__ has joined #openstack-ceilometer01:29
*** promulo has quit IRC01:33
*** ParsectiX has quit IRC01:35
*** ParsectiX has joined #openstack-ceilometer01:36
*** _cjones_ has joined #openstack-ceilometer01:43
*** himangi has joined #openstack-ceilometer02:03
*** ybathia has quit IRC03:05
*** _cjones_ has quit IRC03:14
*** ddieterly has quit IRC03:23
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration  https://review.openstack.org/14259203:28
*** amalagon has quit IRC03:30
*** deepthi has joined #openstack-ceilometer03:50
*** fnaval has quit IRC03:52
*** himangi has quit IRC03:55
*** claudiub has quit IRC03:56
*** zhenq1 has quit IRC03:57
*** amalagon has joined #openstack-ceilometer03:59
*** fnaval has joined #openstack-ceilometer04:02
*** harlowja is now known as harlowja_away04:14
*** harlowja_away is now known as harlowja04:15
*** devlaps has joined #openstack-ceilometer04:17
*** yatin has joined #openstack-ceilometer04:21
*** ddieterly has joined #openstack-ceilometer04:23
*** ddieterly has quit IRC04:29
*** fnaval has quit IRC04:33
*** ParsectiX has quit IRC04:55
*** ParsectiX has joined #openstack-ceilometer04:55
*** ParsectiX has quit IRC04:57
*** ParsectiX has joined #openstack-ceilometer04:58
*** devlaps has quit IRC05:17
*** ddieterly has joined #openstack-ceilometer05:25
*** ddieterly has quit IRC05:29
*** ParsectiX has quit IRC05:47
*** ParsectiX has joined #openstack-ceilometer05:47
*** ildikov has quit IRC06:00
openstackgerritOpenStack Proposal Bot proposed openstack/ceilometer: Imported Translations from Transifex  https://review.openstack.org/15519706:07
*** harlowja is now known as harlowja_away06:11
*** ParsectiX has quit IRC06:13
*** _cjones_ has joined #openstack-ceilometer06:14
*** _cjones_ has quit IRC06:20
*** ddieterly has joined #openstack-ceilometer06:26
*** ddieterly has quit IRC06:30
*** ildikov has joined #openstack-ceilometer06:46
*** ybathia_ has joined #openstack-ceilometer06:57
openstackgerritEric Brown proposed openstack/ceilometer: Use oslo_config choices support  https://review.openstack.org/15858407:11
*** ddieterly has joined #openstack-ceilometer07:26
*** ddieterly has quit IRC07:31
*** shengjiemin has joined #openstack-ceilometer07:48
*** Ala has joined #openstack-ceilometer07:55
*** ybathia_ has quit IRC08:04
*** eglynn has joined #openstack-ceilometer08:16
*** promulo__ has quit IRC08:18
*** ddieterly has joined #openstack-ceilometer08:27
swamireddyeglynn: dhellmann: Thanks for gate issue fixed... I will try to add exception in my client code and if the import error can be handled before coming to agent...and resubmit the patch08:31
*** ddieterly has quit IRC08:31
eglynnswamireddy: excellent! ... if you can resubmit your patch soon, we'll fast-track the approval08:31
ildikovswamireddy: hi08:32
ildikovswamireddy: could you please remove the docco part from your patch and check this one, if it is correct or not: http://docs-draft.openstack.org/50/158250/3/check/gate-openstack-manuals-tox-doc-publish-checkbuild/c7aa5be//publish-docs/admin-guide-cloud/content/section_telemetry-ceph-object-storage-metrics.html08:33
ildikovswamireddy: here is the patch if you found any issues with that docco: https://review.openstack.org/#/c/158250/08:34
ildikovswamireddy: thanks in advance08:34
ildikoveglynn: hi08:35
eglynnildikov: hey08:35
ildikoveglynn: is there a chance that you will be able to have a look at Gordon's docco patches?08:35
eglynnildikov: sure08:35
swamireddyeglynn: doing some more testing locally...and re-submit the patch..08:35
eglynnswamireddy: thank you sir!08:36
ildikoveglynn: cool, many thanks08:36
eglynnildikov: np!08:36
swamireddyildikov:  checking it...08:38
ildikovswamireddy: cool, thanks much08:39
swamireddyildikov: This patch looks fine...and I can remove the doc patch from my patch set...08:55
ildikovswamireddy: great, thanks for your feedback08:56
*** IvanBerezovskiy_ has joined #openstack-ceilometer08:57
ildikovswamireddy: and sorry that I asked you in last minute, but the process of moving the Measurements page to OpenStack Manuals is still ongoing and it's actually a big help to me08:57
swamireddyildikov: Thank you. Please let me know, if I need do review of this page...09:03
ildikovswamireddy: do you mean the OS Manuals one I linked above?09:03
*** vissborg has quit IRC09:04
*** Kennan has quit IRC09:05
swamireddyildikov: Yep...09:07
ildikovswamireddy: it's not a must, but if you have time, it would be good09:08
ildikovswamireddy: it helps the doc team to let them know that the content is ok or not09:08
swamireddyildikov: cool...09:08
ildikovswamireddy: and also it helps me too, as I'm not a Ceph expert... :)09:09
swamireddyildikov: Sure, will do that.09:10
ildikovswamireddy: thanks much09:10
swamireddyeglynn:  can you please confirm that - I can remove the doc changes in my patch...09:11
eglynnswamireddy: I'll defer to ildikov on that matter :)09:12
eglynnildikov: your preference is to remove docco from swamireddy's patch, amiright?09:12
ildikoveglynn: that docco will be removed anyway and the content is already up for review in OS Manuals, so it just makes life easier as it will not cause additional merge conflict in my removal patch on Ceilo side09:13
ildikoveglynn: this is the background and the answer to your question is yes09:14
eglynnildikov: coolness, agreed09:14
swamireddyeglynn: Yes, thats correct..09:15
ildikoveglynn: cool :)09:15
eglynnsounds like we're all on the same page here :)09:16
ildikoveglynn: it is the joy of a Tuesday morning as opposed to Monday mornings ;)09:16
eglynn... think of it as, Monday part II09:17
ildikoveglynn: ... as my Grafield part says09:17
ildikovok, I give it up, you won :)09:18
swamireddyeglynn: ildikov: To summarize - I will keep the same doc info on my patch during my resubmit and ildikov can continue with doc change is in place..09:19
*** karimb has joined #openstack-ceilometer09:21
eglynnswamireddy: the opposite I think09:21
eglynnswamireddy: i.e. if you remove the docco from your patch09:22
ildikovswamireddy: hmm, I thought the agreement was about eglyn confirmed that you can remove the docco part09:22
eglynnswamireddy: ... then ildikov doesn't have to rebase her general docco-removal patch09:22
eglynnswamireddy: ... and meanwhile, your new meters get added to os-manuals09:22
*** karimb has quit IRC09:23
*** karimb has joined #openstack-ceilometer09:27
*** ddieterly has joined #openstack-ceilometer09:28
*** ddieterly has quit IRC09:32
swamireddyeglynn: OK. Will remove the doc changes from the patch in my patch submition..09:33
swamireddyildikov: cool...agreed09:33
*** openstackgerrit has quit IRC09:36
*** openstackgerrit has joined #openstack-ceilometer09:36
ildikovswamireddy: cool, thanks :)09:40
*** himangi has joined #openstack-ceilometer09:40
*** nellysmitt has joined #openstack-ceilometer09:44
eglynnswamireddy: thank you sir!09:44
*** _nadya_ has joined #openstack-ceilometer10:00
*** harlowja_away has quit IRC10:04
*** shengjiemin has quit IRC10:05
*** himangi has quit IRC10:08
swamireddyeglynn: with dhellmannś patch - the ImportError handled in the agent/base.py and displaying the an error msg as below:10:10
swamireddyeglynn: ERROR ceilometer.agent.base [-] Failed to import extension for rgw.usage: No module named awsauth10:10
swamireddyeglynn: so Now, if I add a importError handle in the rgw_client, what type of error message expected there?10:11
eglynnswamireddy: all dhellmann's patch does is to explicitly catch any ImportErrors that occur when a plugin is loaded10:13
eglynnswamireddy: ... and then log these, as opposed to allowing then to bubble up10:14
eglynnswamireddy: ... the aws-requests dependency still needs to be loadable10:14
eglynnswamireddy: ... previously you had this in the test-requirements.txt10:14
swamireddyeglynn: Thats correct. If I add the ImportError in the rgw client, it will duplicate work, so I am not preferring it...10:16
*** _nadya_ has quit IRC10:16
*** sbfox has joined #openstack-ceilometer10:17
swamireddyeglynn: yes, the requests-aws is in test-requirements.txt only10:17
eglynnswamireddy: "add the ImportError" ... do you mean explicitly catching the ImportError?10:17
eglynnswamireddy: no need to do that in two places10:17
swamireddyeglynn: yes, I agree with 100%. No need to repeat the catching the ImportError in 2 places.10:18
swamireddyeglynn: As the rgw use the requests-aws lib, who want to use the ceph rgw meters, those will install the rquests-aws and procced to use that. Thats my understanding. Please correct me if I miss some thing.10:20
eglynnswamireddy: if your tempest tests are going to run in 3rd party CI against a devstack instance with ceph installed, then surely the requests-aws is needed in the test-requirements?10:25
eglynnswamireddy: ... but yes, otherwise it can be treated as an optional requirement by openstack distributions10:26
eglynnswamireddy: ... in the same way as such distributions might decide not to ship happybase as they don't support the hbase storage driver10:26
*** ddieterly has joined #openstack-ceilometer10:29
swamireddyeglynn: ceilometer unit tests also use the requests-aws pkg, so I have added to test-requirements.txt10:31
eglynnswamireddy: cool10:31
swamireddyeglynn: So now, I can re-submit the same patch (by removing the doc changes) without adding any checks...is this OK?10:32
*** ddieterly has quit IRC10:33
*** himangi has joined #openstack-ceilometer10:33
*** cdent has joined #openstack-ceilometer10:34
eglynnswamireddy: yes, please fo10:35
*** exploreshaifali has joined #openstack-ceilometer10:35
eglynnswamireddy: yes, please *do10:35
swamireddyeglynn: Thank you very much Sir...10:36
eglynnswamireddy: well, thank *you* sir!10:38
swamireddyeglynn:  Shall I use the same commit id for the patch submittion?10:40
*** shengjiemin has joined #openstack-ceilometer10:41
eglynnswamireddy: nope, I'd say different as the original was reverted ... so that review is now closed on master10:43
eglynnswamireddy: IIUC the ChangeId can only reused once per branch10:43
eglynne.g. master & stable/juno10:43
swamireddyeglynn: OK. will use the new commit id.10:44
*** himangi has quit IRC10:58
*** deepthi has quit IRC11:09
*** shardy is now known as shardy_afk11:13
*** ildikov has quit IRC11:15
*** eglynn is now known as eglynn-afk11:16
*** eglynn-afk has quit IRC11:26
*** ddieterly has joined #openstack-ceilometer11:29
*** ddieterly has quit IRC11:34
*** cdent has quit IRC11:40
*** exploreshaifali has quit IRC11:40
*** exploreshaifali has joined #openstack-ceilometer11:43
*** sbfox has quit IRC11:43
*** cmyster has quit IRC11:44
*** cdent has joined #openstack-ceilometer11:48
*** himangi has joined #openstack-ceilometer11:54
openstackgerritSwami Reddy proposed openstack/ceilometer: Add ceph object storage meters  https://review.openstack.org/15865611:56
*** ildikov has joined #openstack-ceilometer11:59
*** exploreshaifali has quit IRC12:08
*** eglynn-afk has joined #openstack-ceilometer12:19
*** ddieterly has joined #openstack-ceilometer12:30
*** eglynn-afk is now known as eglynn12:31
*** ddieterly has quit IRC12:35
*** akrivoka has joined #openstack-ceilometer12:40
openstackgerritChris Dent proposed stackforge/gnocchi: Add a py27-cover tox target  https://review.openstack.org/15867212:42
*** _nadya_ has joined #openstack-ceilometer12:43
akrivokahi ceilometer!12:43
akrivokaI am trying to find some info/docs on IPMI agent12:44
akrivokaI have ceilometer installed on one node, and would like to get IPMI metrics from another node12:44
akrivokaI read this spec http://specs.openstack.org/openstack/ceilometer-specs/specs/juno/ipmi.html12:44
akrivokabut there's no actual configuration instruction there12:44
akrivokawhat do I need to do in order to see the IPMI metrics from my second node, in my first node's ceilometer?12:45
ildikovakrivoka: here is some docs: http://docs.openstack.org/admin-guide-cloud/content/section_telemetry-IPMI-agent.html12:56
ildikovakrivoka: you will need to deploy IPMI agent on node 212:57
ildikovakrivoka: but only in case it is not an Ironic-managed node and IPMI-capable12:58
akrivokaildikov: what if it is Ironic-managed?12:59
ildikovakrivoka: in that case you need to configure Ironic to send notifications to Ceilometer by setting ‘conductor.send_sensor_data’ option to True13:00
ildikovakrivoka: and should not deploy IPMI agent as that will cause duplication13:01
ildikovakrivoka: here is the Ironic install guide: http://docs.openstack.org/developer/ironic/deploy/install-guide.html13:02
ildikovakrivoka: you should find here if anything additional will be needed in order to get the IPMI sensor data13:02
*** yatin has quit IRC13:04
*** himangi has quit IRC13:09
*** Longgeek has joined #openstack-ceilometer13:12
*** exploreshaifali has joined #openstack-ceilometer13:17
eglynnakrivoka: BTW cdent and gzhai1 are the domain experts in IPMI metrics13:18
eglynn(Chris for the notification-driven option, i.e. the preferred approach agreed with the ironic team, and Edwin for the polling approach)13:18
*** gordc has joined #openstack-ceilometer13:28
openstackgerritRomain Soufflet proposed stackforge/gnocchi: Add HTML rendering for "/v1/resource/generic/<resource_id>" URL  https://review.openstack.org/15868613:29
*** ddieterly has joined #openstack-ceilometer13:31
*** ddieterly has quit IRC13:35
*** ddieterly has joined #openstack-ceilometer13:35
swamireddyeglynn: gordc: cdent: I have re-submitted the (same) patch - https://review.openstack.org/#/c/158656/ (Add ceph object storage meters).13:37
gordcswamireddy: cool stuff. did you ever figure out how to hide the error?13:38
*** _nadya_ has quit IRC13:40
swamireddygordc: Are the errors related to ImportErrors??13:46
*** sbfox has joined #openstack-ceilometer13:47
swamireddygordc:  As the importError has been handled by the dhellmann´s patch, so I tought not include the same check in rgw clint..13:47
*** sbfox has quit IRC13:47
swamireddygordc: ..and the these importError are needed to user, because, user will know that import failed, so install the pkg mannually and restart the service. If the error msg hide, this could be problem..as user may not know what happening.13:49
gordcswamireddy: so i think we added a way to make it verify this without throwing an error... something along this line: https://review.openstack.org/#/c/146303/13:51
linuxhermitgood morning13:52
gordcswamireddy: tbh, i'm ok with merging your patch, i just think it's best not to have it as an error.13:52
gordclinuxhermit: mornin'13:52
linuxhermitwoot another tiny doc change made it in13:53
*** moravec has quit IRC13:55
*** moravec has joined #openstack-ceilometer13:55
gordclinuxhermit: premature celebration angers the gate gods... be ready for an elastic-recheck.13:56
linuxhermitdoh13:56
linuxhermitI'm more interested in the results of the gabbi tests reviews13:57
swamireddygordc: Cool one - Enabling self-disabled pollster....I was looking for similar stuff. Let me know, if this will help for ceph rgw pollsters also..13:57
linuxhermitsince there are more of those to be done13:57
gordcswamireddy: did you leave out measurements on purpose?13:57
gordcnot a big deal since ildikov is moving it all to doc.openstack.org anyways.13:58
ildikovgordc: no worries, I asked for this change13:58
swamireddygordc: Quick Q: Is this ¨Enabling self-disabled pollster¨  ...some check need to add on setup.cfg or so?13:58
gordcildikov: cool cool. makes sense to me13:58
swamireddygordc:  Yes, after discussion with ildikov and eglynn, I have left that doc changes (this dicussion a few hours back today)13:59
gordcswamireddy: tbh, not exactly sure with details. i just noticed it has some functionality that is similar to what you'd want.13:59
gordcswamireddy: cool cool. i reviewed it. i'm ok with fixing other issue in separate patch.14:00
swamireddygordc: Thank you...14:01
swamireddygordc: ..and I feel ¨Enabling self-disabled pollster¨ is very important one.. In my case - when I have enballed the ceph rgw pollster, still swift storage pollsters are on, which not needed14:02
gordcagreed.14:02
*** ddieterly has quit IRC14:07
swamireddygordc:  I will check the ¨Enabling self-disabled pollster¨ patch, and see if I can use it14:07
gordcswamireddy: awesome! i was just going to ask if you could look at it. :)14:07
swamireddygordc: cool...As you suggested, will do that in a separate patch...14:08
gordcswamireddy: perfect.14:09
*** moravec has quit IRC14:10
openstackgerritgordon chung proposed openstack/ceilometer: reorganise architecture page  https://review.openstack.org/15594114:10
*** changbl has quit IRC14:13
*** _nadya_ has joined #openstack-ceilometer14:14
openstackgerritVadim Rovachev proposed openstack/ceilometer-specs: Add specification for integration tests.  https://review.openstack.org/15079414:15
cdentthanks for the typo catch linuxhermit14:19
openstackgerritChris Dent proposed openstack/ceilometer: Initial gabbi testing for alarms  https://review.openstack.org/15329314:19
*** swamireddy is now known as swamireddy_afk14:20
*** zul has quit IRC14:23
linuxhermitsorry :/14:24
linuxhermitthe tests ran great!14:24
*** zul has joined #openstack-ceilometer14:24
vrovachevcdent: I answered for your comments: https://review.openstack.org/#/c/150794/ :-)14:28
*** nellysmitt has quit IRC14:28
cdentYeah, saw that vrovachev, and I +1d14:28
*** nellysmitt has joined #openstack-ceilometer14:29
*** nellysmitt has quit IRC14:29
vrovachevcdent: thank you so much :-)14:29
*** nellysmitt has joined #openstack-ceilometer14:29
*** _nadya_ has quit IRC14:39
*** rbak has joined #openstack-ceilometer14:44
*** ddieterly has joined #openstack-ceilometer14:44
gordceglynn: wasm14:49
gordcwasn't aware you still have core in glance14:49
eglynngordc: yeah, I should have been grandfathered out ages ago :)14:50
linuxhermitgordc you were right I angered the gate gods14:54
*** nellysmitt has quit IRC14:55
gordclinuxhermit: lol never anger the gate gods.14:55
gordcthey put you in recheck purgatory.14:55
*** thomasem has joined #openstack-ceilometer14:55
linuxhermit:/14:56
linuxhermithaha14:56
*** exploreshaifali has quit IRC14:57
*** eglynn has quit IRC15:02
*** eglynn has joined #openstack-ceilometer15:05
*** alexpilotti has quit IRC15:07
gordcsileht: are we deprecating the use of *.common.notifier.*? ie notification_driver = cinder.openstack.common.notifier.rpc_notifier15:07
openstackgerritChris Dent proposed stackforge/gnocchi: Add a py27-cover tox target  https://review.openstack.org/15867215:09
silehtgordc, not yet, but this is an oslo.msg todo15:10
gordcsileht: what's the recommended usage? notification_driver = rabbit?15:11
silehtnotification_driver = messagingv215:11
silehtgordc, ^15:11
gordcsileht: ah right. cool cool. thanks!15:12
openstackgerritOpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements  https://review.openstack.org/15764115:13
silehtgordc, this is a MultiStrOpt, so you can set many drivers now15:13
sileht:q15:13
*** ccrouch has joined #openstack-ceilometer15:14
gordcsileht: so i can have notification_driver = log as well if i have option of messagingv2 already?15:15
silehtgordc, yes15:15
gordccool cool15:16
openstackgerritMerged openstack/ceilometer: compress events notes  https://review.openstack.org/15634215:17
openstackgerritMerged openstack/ceilometer: Fix unittests for supporting py-pgsql env  https://review.openstack.org/15783615:17
*** nellysmitt has joined #openstack-ceilometer15:18
*** prad has joined #openstack-ceilometer15:19
*** devlaps has joined #openstack-ceilometer15:24
*** devlaps has quit IRC15:24
*** ddieterl_ has joined #openstack-ceilometer15:25
*** ddieterly has quit IRC15:25
*** Ephur has quit IRC15:30
*** ybathia has joined #openstack-ceilometer15:32
*** nellysmitt has quit IRC15:32
*** nellysmitt has joined #openstack-ceilometer15:32
*** _nadya_ has joined #openstack-ceilometer15:35
*** david-lyle_afk is now known as david-lyle15:47
*** zul has quit IRC15:50
*** zul has joined #openstack-ceilometer15:50
*** ddieterl_ has quit IRC16:01
*** ddieterly has joined #openstack-ceilometer16:02
*** shengjiemin has quit IRC16:03
*** sbfox has joined #openstack-ceilometer16:04
*** shengjiemin has joined #openstack-ceilometer16:05
*** fnaval has joined #openstack-ceilometer16:08
*** sbfox has quit IRC16:08
openstackgerritOpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements  https://review.openstack.org/15764116:13
*** changbl has joined #openstack-ceilometer16:16
*** ybathia has quit IRC16:20
*** amalagon has quit IRC16:21
*** _cjones_ has joined #openstack-ceilometer16:40
*** _cjones_ has quit IRC16:41
*** _cjones_ has joined #openstack-ceilometer16:42
*** _cjones_ has quit IRC16:46
*** _cjones_ has joined #openstack-ceilometer16:46
*** shengjie_ has joined #openstack-ceilometer16:58
*** shengjiemin has quit IRC16:58
*** shengjie_ is now known as shengjiemin16:58
*** shengjiemin has quit IRC17:03
*** shengjiemin has joined #openstack-ceilometer17:04
*** yeungp has joined #openstack-ceilometer17:05
*** exploreshaifali has joined #openstack-ceilometer17:07
*** _nadya_ has quit IRC17:08
*** IvanBerezovskiy_ has quit IRC17:15
*** nellysmitt has quit IRC17:17
*** nellysmitt has joined #openstack-ceilometer17:18
*** rwsu-afk is now known as rwsu17:18
openstackgerritMerged openstack/ceilometer: Add ceph object storage meters  https://review.openstack.org/15865617:23
*** shengjiemin has quit IRC17:23
*** sbfox has joined #openstack-ceilometer17:24
*** jmatthews has joined #openstack-ceilometer17:25
*** shengjiemin has joined #openstack-ceilometer17:29
*** Ala has quit IRC17:30
*** _cjones_ has quit IRC17:33
*** _cjones_ has joined #openstack-ceilometer17:36
*** sbfox has quit IRC17:41
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer  https://review.openstack.org/15851917:50
openstackgerritEric Brown proposed openstack/ceilometer: Use secure default for bind interfaces  https://review.openstack.org/15852317:51
openstackgerritEric Brown proposed openstack/ceilometer: Use secure default for bind interfaces  https://review.openstack.org/15852317:52
*** ParsectiX has joined #openstack-ceilometer17:54
*** shengjiemin has quit IRC17:55
*** karimb has quit IRC17:56
*** eglynn is now known as eglynn-afk18:00
*** sbfox has joined #openstack-ceilometer18:02
swamireddy_afkildikov:  I have reviewed the patch  https://review.openstack.org/#/c/158250/. Its fine. And added a comment to include one href for rgw usage log enable. If possible, you can add that.18:05
swamireddy_afkildikov:  Please let me know if you need more info here18:08
*** prad has quit IRC18:09
*** promulo has joined #openstack-ceilometer18:11
*** devlaps has joined #openstack-ceilometer18:11
*** Longgeek has quit IRC18:15
openstackgerritgordon chung proposed openstack/ceilometer: add option to store raw notification  https://review.openstack.org/15379018:15
*** changbl has quit IRC18:17
*** harlowja has joined #openstack-ceilometer18:20
*** harlowja has quit IRC18:24
*** harlowja has joined #openstack-ceilometer18:24
*** changbl has joined #openstack-ceilometer18:27
*** ParsectiX has quit IRC18:37
*** ParsectiX has joined #openstack-ceilometer18:38
*** ParsectiX has quit IRC18:41
*** ParsectiX has joined #openstack-ceilometer18:41
*** cdent has quit IRC18:45
ildikovswamireddy_afk: thanks, I will check your comment18:46
*** zul has quit IRC18:52
*** zul has joined #openstack-ceilometer18:52
*** zul has quit IRC18:53
*** zul has joined #openstack-ceilometer18:53
*** changbl has quit IRC18:58
*** yeungp has quit IRC19:03
*** fabiog has joined #openstack-ceilometer19:07
*** Longgeek has joined #openstack-ceilometer19:15
*** _cjones_ has quit IRC19:16
openstackgerritMerged openstack/ceilometer: Adding links API and CLI query examples  https://review.openstack.org/15745919:18
*** _cjones_ has joined #openstack-ceilometer19:21
*** moravec has joined #openstack-ceilometer19:21
fabioggordc: ping19:21
gordcfabiog: hey, whats up?19:26
*** prad has joined #openstack-ceilometer19:31
*** devlaps has quit IRC19:33
fabioggordc: there is a patch for client that failed gate19:35
fabioggordc: https://review.openstack.org/#/c/139659/19:35
fabioggordc: and I cannot figure out why is failing19:35
fabioggordc: is kind of important to support V319:35
gordcfabiog: i see. i'll try out patch. (i think zqfan might be on new years holidays still)19:37
fabioggordc: yes, I guess. Thanks19:37
*** jmatthews has quit IRC19:40
*** afazekas has joined #openstack-ceilometer19:42
*** prad has quit IRC19:43
afazekasIs the new python-ceilometerclient really requires new python-keystoneclient or it just added because the Proposal bot suggested it ?19:44
afazekashttp://logs.openstack.org/62/158162/6/check/check-tempest-dsvm-full-icehouse/e759fb4/logs/screen-s-proxy.txt.gz19:44
afazekashttps://pypi.python.org/pypi/python-ceilometerclient/1.0.13 is released today and the icehouse jobs started to fail19:44
afazekasgordc, ^19:45
gordcafazekas: :(19:46
gordcfabiog: the keystone work you added, does it require new python-keystoneclient?19:46
afazekasOn icehouse we have python-keystoneclient==0.11.219:47
gordcafazekas: considering fabiog's work was using pre 1.x builds we don't have a real need for it... probably just an autopatch19:50
gordcafazekas: ah i see failure now... i'm wondering why it let us merge originally. i guess we locked the icehouse versions in last few weeks?19:53
afazekasprobably bacuse we do not have icehouse job on the clients19:53
afazekaslooks like the ceilo client version will be limited to 1.0.12 on the icehouse jobs (#openstack-qa)19:54
*** ddieterly has quit IRC19:56
*** moravec1 has joined #openstack-ceilometer19:57
*** ddieterly has joined #openstack-ceilometer19:58
gordcafazekas: ok. just looking at the bug related to new release... seems ok to limit it for icehouse.19:58
gordceglynn-afk: ^ for reference.19:58
*** moravec has quit IRC19:58
afazekasaccording the discussion the ceilo client after icehouse release should have use a version number >=1.119:59
afazekasespecially if it break things ..20:00
afazekasgordc: the issue will be solved on the gate soon : https://review.openstack.org/#/c/158842/20:00
gordcafazekas: sorry i'm not aware of the discussion for version number... do you have a link?20:01
gordcafazekas: cool cool. thanks for checking here first20:01
gordcfabiog: looks to be a merge issue with https://review.openstack.org/#/c/155164/20:02
*** _nadya_ has joined #openstack-ceilometer20:08
*** _nadya_ has quit IRC20:12
jrollgordc: hi! are you gordon chung?20:21
gordcjroll: depends on the follow up question is...20:24
jrollhehe20:24
jrollmind a PM?20:24
gordcjroll: sure.20:24
*** exploreshaifali has quit IRC20:24
*** nellysmitt has quit IRC20:26
*** eglynn-afk is now known as eglynn20:28
*** jmatthews has joined #openstack-ceilometer20:38
fabioggordc: a rebase would do then?20:39
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer  https://review.openstack.org/15851920:40
gordcfabiog: there's probably a code change involved20:40
*** prad has joined #openstack-ceilometer20:40
gordcfabiog: i can hit the rebase button if you want to give that a try first.20:41
*** moravec1 has quit IRC20:41
fabioggordc: yeah, give it a go. If that does not work I will ping ZhiQuang20:42
openstackgerritgordon chung proposed openstack/python-ceilometerclient: Enable specified project_id in CLI commands  https://review.openstack.org/13965920:42
*** jmatthews has quit IRC20:42
gordcfabiog: yeah the reason i didn't fix it was because i won't be able to +2 it and you might have difficulty finding +2's this week because of holidays20:43
jrolldo I need mongodb installed to run tox on ceilometer?20:47
gordcjroll: ... i believe you do. you can try running tox -epy-mysql, that may avoid the mongo req20:48
jrollgordc: thanks20:48
*** ddieterly has quit IRC20:50
*** ddieterly has joined #openstack-ceilometer20:50
fabioggordc: np20:59
fabioggordc: do you think it would help if I run a Hangs out session on the db to explain how it works ... I feel it is kind of stuck and not making progress in the reviews21:00
fabioggordc: cdent was saying it is difficult to understand21:01
gordcfabiog: sorry was in a random meeting21:07
gordcfabiog: yeah i think it might help... for me i'm just wondering the intended interaction pattern is.21:08
gordcright now it seems like it's basically 'load this file into db', not sure how it works after that... ie if you change it, it will somehow trigger a service restart?21:09
gordcfabiog: or i guess maybe a doc/slide that shows how it differs from using static files (aside from the store in db aspect).21:13
*** changbl has joined #openstack-ceilometer21:14
*** ParsectiX has quit IRC21:19
*** ParsectiX has joined #openstack-ceilometer21:19
*** Longgeek has quit IRC21:24
*** sbfox has quit IRC21:26
*** sbfox has joined #openstack-ceilometer21:28
*** _cjones_ has quit IRC21:39
*** ParsectiX has quit IRC21:41
*** akrivoka has quit IRC21:42
*** ParsectiX has joined #openstack-ceilometer21:42
*** jmatthews has joined #openstack-ceilometer21:42
*** _cjones_ has joined #openstack-ceilometer21:50
*** karimb has joined #openstack-ceilometer21:52
*** moravec has joined #openstack-ceilometer22:05
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer  https://review.openstack.org/15851922:06
*** eglynn has quit IRC22:07
openstackgerritRohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer  https://review.openstack.org/15851922:08
*** moravec has quit IRC22:11
*** changbl has quit IRC22:17
*** ViswaV has joined #openstack-ceilometer22:28
fabioggordc: I will put together a deck and publish it in Slideshare. I will probably do it before our thu meeting22:32
gordcfabiog: sounds good.22:34
*** jmatthews has quit IRC22:35
*** openstackgerrit has quit IRC22:37
*** openstackgerrit has joined #openstack-ceilometer22:37
openstackgerritgordon chung proposed openstack/ceilometer: add missing objectstore entry points  https://review.openstack.org/15890722:57
*** harlowja is now known as harlowja_away23:02
*** jmatthews has joined #openstack-ceilometer23:03
*** harlowja_away is now known as harlowja23:05
*** fabiog has quit IRC23:09
*** Longgeek has joined #openstack-ceilometer23:09
*** thomasem has quit IRC23:10
*** _cjones_ has quit IRC23:12
*** _cjones_ has joined #openstack-ceilometer23:14
openstackgerritgordon chung proposed openstack/ceilometer: add missing objectstore entry points  https://review.openstack.org/15890723:24
*** karimb has quit IRC23:25
*** rbak has quit IRC23:25
*** _cjones_ has quit IRC23:30
*** jmatthews has quit IRC23:32
openstackgerritgordon chung proposed openstack/ceilometer: add an option to disable non-metric meters  https://review.openstack.org/15674723:33
*** gordc has quit IRC23:33
*** Longgeek has quit IRC23:42
*** _cjones_ has joined #openstack-ceilometer23:43
*** prad has quit IRC23:57

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