*** rbak has quit IRC | 00:02 | |
*** thomasem has quit IRC | 00:05 | |
*** thomasem has joined #openstack-ceilometer | 00:10 | |
*** ccrouch has quit IRC | 00:15 | |
openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer https://review.openstack.org/158519 | 00:42 |
---|---|---|
*** zul has quit IRC | 00:51 | |
*** zul has joined #openstack-ceilometer | 00:52 | |
*** promulo has joined #openstack-ceilometer | 00:56 | |
*** promulo__ has quit IRC | 00:56 | |
*** drjones has quit IRC | 01:06 | |
*** adam_g is now known as adam_g_out | 01:09 | |
*** devlaps has quit IRC | 01:11 | |
openstackgerrit | Eric Brown proposed openstack/ceilometer: Use secure default for bind interfaces https://review.openstack.org/158523 | 01:11 |
*** zhenq1 has joined #openstack-ceilometer | 01:14 | |
*** ddieterly has joined #openstack-ceilometer | 01:15 | |
*** sbfox has quit IRC | 01:17 | |
*** thomasem has quit IRC | 01:18 | |
*** david-lyle is now known as david-lyle_afk | 01:19 | |
*** promulo__ has joined #openstack-ceilometer | 01:29 | |
*** promulo has quit IRC | 01:33 | |
*** ParsectiX has quit IRC | 01:35 | |
*** ParsectiX has joined #openstack-ceilometer | 01:36 | |
*** _cjones_ has joined #openstack-ceilometer | 01:43 | |
*** himangi has joined #openstack-ceilometer | 02:03 | |
*** ybathia has quit IRC | 03:05 | |
*** _cjones_ has quit IRC | 03:14 | |
*** ddieterly has quit IRC | 03:23 | |
openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Storage Layer Implementation for Pipeline Configuration https://review.openstack.org/142592 | 03:28 |
*** amalagon has quit IRC | 03:30 | |
*** deepthi has joined #openstack-ceilometer | 03:50 | |
*** fnaval has quit IRC | 03:52 | |
*** himangi has quit IRC | 03:55 | |
*** claudiub has quit IRC | 03:56 | |
*** zhenq1 has quit IRC | 03:57 | |
*** amalagon has joined #openstack-ceilometer | 03:59 | |
*** fnaval has joined #openstack-ceilometer | 04:02 | |
*** harlowja is now known as harlowja_away | 04:14 | |
*** harlowja_away is now known as harlowja | 04:15 | |
*** devlaps has joined #openstack-ceilometer | 04:17 | |
*** yatin has joined #openstack-ceilometer | 04:21 | |
*** ddieterly has joined #openstack-ceilometer | 04:23 | |
*** ddieterly has quit IRC | 04:29 | |
*** fnaval has quit IRC | 04:33 | |
*** ParsectiX has quit IRC | 04:55 | |
*** ParsectiX has joined #openstack-ceilometer | 04:55 | |
*** ParsectiX has quit IRC | 04:57 | |
*** ParsectiX has joined #openstack-ceilometer | 04:58 | |
*** devlaps has quit IRC | 05:17 | |
*** ddieterly has joined #openstack-ceilometer | 05:25 | |
*** ddieterly has quit IRC | 05:29 | |
*** ParsectiX has quit IRC | 05:47 | |
*** ParsectiX has joined #openstack-ceilometer | 05:47 | |
*** ildikov has quit IRC | 06:00 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/155197 | 06:07 |
*** harlowja is now known as harlowja_away | 06:11 | |
*** ParsectiX has quit IRC | 06:13 | |
*** _cjones_ has joined #openstack-ceilometer | 06:14 | |
*** _cjones_ has quit IRC | 06:20 | |
*** ddieterly has joined #openstack-ceilometer | 06:26 | |
*** ddieterly has quit IRC | 06:30 | |
*** ildikov has joined #openstack-ceilometer | 06:46 | |
*** ybathia_ has joined #openstack-ceilometer | 06:57 | |
openstackgerrit | Eric Brown proposed openstack/ceilometer: Use oslo_config choices support https://review.openstack.org/158584 | 07:11 |
*** ddieterly has joined #openstack-ceilometer | 07:26 | |
*** ddieterly has quit IRC | 07:31 | |
*** shengjiemin has joined #openstack-ceilometer | 07:48 | |
*** Ala has joined #openstack-ceilometer | 07:55 | |
*** ybathia_ has quit IRC | 08:04 | |
*** eglynn has joined #openstack-ceilometer | 08:16 | |
*** promulo__ has quit IRC | 08:18 | |
*** ddieterly has joined #openstack-ceilometer | 08:27 | |
swamireddy | eglynn: 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 patch | 08:31 |
*** ddieterly has quit IRC | 08:31 | |
eglynn | swamireddy: excellent! ... if you can resubmit your patch soon, we'll fast-track the approval | 08:31 |
ildikov | swamireddy: hi | 08:32 |
ildikov | swamireddy: 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.html | 08:33 |
ildikov | swamireddy: here is the patch if you found any issues with that docco: https://review.openstack.org/#/c/158250/ | 08:34 |
ildikov | swamireddy: thanks in advance | 08:34 |
ildikov | eglynn: hi | 08:35 |
eglynn | ildikov: hey | 08:35 |
ildikov | eglynn: is there a chance that you will be able to have a look at Gordon's docco patches? | 08:35 |
eglynn | ildikov: sure | 08:35 |
swamireddy | eglynn: doing some more testing locally...and re-submit the patch.. | 08:35 |
eglynn | swamireddy: thank you sir! | 08:36 |
ildikov | eglynn: cool, many thanks | 08:36 |
eglynn | ildikov: np! | 08:36 |
swamireddy | ildikov: checking it... | 08:38 |
ildikov | swamireddy: cool, thanks much | 08:39 |
swamireddy | ildikov: This patch looks fine...and I can remove the doc patch from my patch set... | 08:55 |
ildikov | swamireddy: great, thanks for your feedback | 08:56 |
*** IvanBerezovskiy_ has joined #openstack-ceilometer | 08:57 | |
ildikov | swamireddy: 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 me | 08:57 |
swamireddy | ildikov: Thank you. Please let me know, if I need do review of this page... | 09:03 |
ildikov | swamireddy: do you mean the OS Manuals one I linked above? | 09:03 |
*** vissborg has quit IRC | 09:04 | |
*** Kennan has quit IRC | 09:05 | |
swamireddy | ildikov: Yep... | 09:07 |
ildikov | swamireddy: it's not a must, but if you have time, it would be good | 09:08 |
ildikov | swamireddy: it helps the doc team to let them know that the content is ok or not | 09:08 |
swamireddy | ildikov: cool... | 09:08 |
ildikov | swamireddy: and also it helps me too, as I'm not a Ceph expert... :) | 09:09 |
swamireddy | ildikov: Sure, will do that. | 09:10 |
ildikov | swamireddy: thanks much | 09:10 |
swamireddy | eglynn: can you please confirm that - I can remove the doc changes in my patch... | 09:11 |
eglynn | swamireddy: I'll defer to ildikov on that matter :) | 09:12 |
eglynn | ildikov: your preference is to remove docco from swamireddy's patch, amiright? | 09:12 |
ildikov | eglynn: 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 side | 09:13 |
ildikov | eglynn: this is the background and the answer to your question is yes | 09:14 |
eglynn | ildikov: coolness, agreed | 09:14 |
swamireddy | eglynn: Yes, thats correct.. | 09:15 |
ildikov | eglynn: cool :) | 09:15 |
eglynn | sounds like we're all on the same page here :) | 09:16 |
ildikov | eglynn: it is the joy of a Tuesday morning as opposed to Monday mornings ;) | 09:16 |
eglynn | ... think of it as, Monday part II | 09:17 |
ildikov | eglynn: ... as my Grafield part says | 09:17 |
ildikov | ok, I give it up, you won :) | 09:18 |
swamireddy | eglynn: 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-ceilometer | 09:21 | |
eglynn | swamireddy: the opposite I think | 09:21 |
eglynn | swamireddy: i.e. if you remove the docco from your patch | 09:22 |
ildikov | swamireddy: hmm, I thought the agreement was about eglyn confirmed that you can remove the docco part | 09:22 |
eglynn | swamireddy: ... then ildikov doesn't have to rebase her general docco-removal patch | 09:22 |
eglynn | swamireddy: ... and meanwhile, your new meters get added to os-manuals | 09:22 |
*** karimb has quit IRC | 09:23 | |
*** karimb has joined #openstack-ceilometer | 09:27 | |
*** ddieterly has joined #openstack-ceilometer | 09:28 | |
*** ddieterly has quit IRC | 09:32 | |
swamireddy | eglynn: OK. Will remove the doc changes from the patch in my patch submition.. | 09:33 |
swamireddy | ildikov: cool...agreed | 09:33 |
*** openstackgerrit has quit IRC | 09:36 | |
*** openstackgerrit has joined #openstack-ceilometer | 09:36 | |
ildikov | swamireddy: cool, thanks :) | 09:40 |
*** himangi has joined #openstack-ceilometer | 09:40 | |
*** nellysmitt has joined #openstack-ceilometer | 09:44 | |
eglynn | swamireddy: thank you sir! | 09:44 |
*** _nadya_ has joined #openstack-ceilometer | 10:00 | |
*** harlowja_away has quit IRC | 10:04 | |
*** shengjiemin has quit IRC | 10:05 | |
*** himangi has quit IRC | 10:08 | |
swamireddy | eglynn: with dhellmannś patch - the ImportError handled in the agent/base.py and displaying the an error msg as below: | 10:10 |
swamireddy | eglynn: ERROR ceilometer.agent.base [-] Failed to import extension for rgw.usage: No module named awsauth | 10:10 |
swamireddy | eglynn: so Now, if I add a importError handle in the rgw_client, what type of error message expected there? | 10:11 |
eglynn | swamireddy: all dhellmann's patch does is to explicitly catch any ImportErrors that occur when a plugin is loaded | 10:13 |
eglynn | swamireddy: ... and then log these, as opposed to allowing then to bubble up | 10:14 |
eglynn | swamireddy: ... the aws-requests dependency still needs to be loadable | 10:14 |
eglynn | swamireddy: ... previously you had this in the test-requirements.txt | 10:14 |
swamireddy | eglynn: 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 IRC | 10:16 | |
*** sbfox has joined #openstack-ceilometer | 10:17 | |
swamireddy | eglynn: yes, the requests-aws is in test-requirements.txt only | 10:17 |
eglynn | swamireddy: "add the ImportError" ... do you mean explicitly catching the ImportError? | 10:17 |
eglynn | swamireddy: no need to do that in two places | 10:17 |
swamireddy | eglynn: yes, I agree with 100%. No need to repeat the catching the ImportError in 2 places. | 10:18 |
swamireddy | eglynn: 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 |
eglynn | swamireddy: 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 |
eglynn | swamireddy: ... but yes, otherwise it can be treated as an optional requirement by openstack distributions | 10:26 |
eglynn | swamireddy: ... in the same way as such distributions might decide not to ship happybase as they don't support the hbase storage driver | 10:26 |
*** ddieterly has joined #openstack-ceilometer | 10:29 | |
swamireddy | eglynn: ceilometer unit tests also use the requests-aws pkg, so I have added to test-requirements.txt | 10:31 |
eglynn | swamireddy: cool | 10:31 |
swamireddy | eglynn: 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 IRC | 10:33 | |
*** himangi has joined #openstack-ceilometer | 10:33 | |
*** cdent has joined #openstack-ceilometer | 10:34 | |
eglynn | swamireddy: yes, please fo | 10:35 |
*** exploreshaifali has joined #openstack-ceilometer | 10:35 | |
eglynn | swamireddy: yes, please *do | 10:35 |
swamireddy | eglynn: Thank you very much Sir... | 10:36 |
eglynn | swamireddy: well, thank *you* sir! | 10:38 |
swamireddy | eglynn: Shall I use the same commit id for the patch submittion? | 10:40 |
*** shengjiemin has joined #openstack-ceilometer | 10:41 | |
eglynn | swamireddy: nope, I'd say different as the original was reverted ... so that review is now closed on master | 10:43 |
eglynn | swamireddy: IIUC the ChangeId can only reused once per branch | 10:43 |
eglynn | e.g. master & stable/juno | 10:43 |
swamireddy | eglynn: OK. will use the new commit id. | 10:44 |
*** himangi has quit IRC | 10:58 | |
*** deepthi has quit IRC | 11:09 | |
*** shardy is now known as shardy_afk | 11:13 | |
*** ildikov has quit IRC | 11:15 | |
*** eglynn is now known as eglynn-afk | 11:16 | |
*** eglynn-afk has quit IRC | 11:26 | |
*** ddieterly has joined #openstack-ceilometer | 11:29 | |
*** ddieterly has quit IRC | 11:34 | |
*** cdent has quit IRC | 11:40 | |
*** exploreshaifali has quit IRC | 11:40 | |
*** exploreshaifali has joined #openstack-ceilometer | 11:43 | |
*** sbfox has quit IRC | 11:43 | |
*** cmyster has quit IRC | 11:44 | |
*** cdent has joined #openstack-ceilometer | 11:48 | |
*** himangi has joined #openstack-ceilometer | 11:54 | |
openstackgerrit | Swami Reddy proposed openstack/ceilometer: Add ceph object storage meters https://review.openstack.org/158656 | 11:56 |
*** ildikov has joined #openstack-ceilometer | 11:59 | |
*** exploreshaifali has quit IRC | 12:08 | |
*** eglynn-afk has joined #openstack-ceilometer | 12:19 | |
*** ddieterly has joined #openstack-ceilometer | 12:30 | |
*** eglynn-afk is now known as eglynn | 12:31 | |
*** ddieterly has quit IRC | 12:35 | |
*** akrivoka has joined #openstack-ceilometer | 12:40 | |
openstackgerrit | Chris Dent proposed stackforge/gnocchi: Add a py27-cover tox target https://review.openstack.org/158672 | 12:42 |
*** _nadya_ has joined #openstack-ceilometer | 12:43 | |
akrivoka | hi ceilometer! | 12:43 |
akrivoka | I am trying to find some info/docs on IPMI agent | 12:44 |
akrivoka | I have ceilometer installed on one node, and would like to get IPMI metrics from another node | 12:44 |
akrivoka | I read this spec http://specs.openstack.org/openstack/ceilometer-specs/specs/juno/ipmi.html | 12:44 |
akrivoka | but there's no actual configuration instruction there | 12:44 |
akrivoka | what do I need to do in order to see the IPMI metrics from my second node, in my first node's ceilometer? | 12:45 |
ildikov | akrivoka: here is some docs: http://docs.openstack.org/admin-guide-cloud/content/section_telemetry-IPMI-agent.html | 12:56 |
ildikov | akrivoka: you will need to deploy IPMI agent on node 2 | 12:57 |
ildikov | akrivoka: but only in case it is not an Ironic-managed node and IPMI-capable | 12:58 |
akrivoka | ildikov: what if it is Ironic-managed? | 12:59 |
ildikov | akrivoka: in that case you need to configure Ironic to send notifications to Ceilometer by setting ‘conductor.send_sensor_data’ option to True | 13:00 |
ildikov | akrivoka: and should not deploy IPMI agent as that will cause duplication | 13:01 |
ildikov | akrivoka: here is the Ironic install guide: http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 13:02 |
ildikov | akrivoka: you should find here if anything additional will be needed in order to get the IPMI sensor data | 13:02 |
*** yatin has quit IRC | 13:04 | |
*** himangi has quit IRC | 13:09 | |
*** Longgeek has joined #openstack-ceilometer | 13:12 | |
*** exploreshaifali has joined #openstack-ceilometer | 13:17 | |
eglynn | akrivoka: BTW cdent and gzhai1 are the domain experts in IPMI metrics | 13: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-ceilometer | 13:28 | |
openstackgerrit | Romain Soufflet proposed stackforge/gnocchi: Add HTML rendering for "/v1/resource/generic/<resource_id>" URL https://review.openstack.org/158686 | 13:29 |
*** ddieterly has joined #openstack-ceilometer | 13:31 | |
*** ddieterly has quit IRC | 13:35 | |
*** ddieterly has joined #openstack-ceilometer | 13:35 | |
swamireddy | eglynn: gordc: cdent: I have re-submitted the (same) patch - https://review.openstack.org/#/c/158656/ (Add ceph object storage meters). | 13:37 |
gordc | swamireddy: cool stuff. did you ever figure out how to hide the error? | 13:38 |
*** _nadya_ has quit IRC | 13:40 | |
swamireddy | gordc: Are the errors related to ImportErrors?? | 13:46 |
*** sbfox has joined #openstack-ceilometer | 13:47 | |
swamireddy | gordc: 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 IRC | 13:47 | |
swamireddy | gordc: ..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 |
gordc | swamireddy: 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 |
linuxhermit | good morning | 13:52 |
gordc | swamireddy: tbh, i'm ok with merging your patch, i just think it's best not to have it as an error. | 13:52 |
gordc | linuxhermit: mornin' | 13:52 |
linuxhermit | woot another tiny doc change made it in | 13:53 |
*** moravec has quit IRC | 13:55 | |
*** moravec has joined #openstack-ceilometer | 13:55 | |
gordc | linuxhermit: premature celebration angers the gate gods... be ready for an elastic-recheck. | 13:56 |
linuxhermit | doh | 13:56 |
linuxhermit | I'm more interested in the results of the gabbi tests reviews | 13:57 |
swamireddy | gordc: 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 |
linuxhermit | since there are more of those to be done | 13:57 |
gordc | swamireddy: did you leave out measurements on purpose? | 13:57 |
gordc | not a big deal since ildikov is moving it all to doc.openstack.org anyways. | 13:58 |
ildikov | gordc: no worries, I asked for this change | 13:58 |
swamireddy | gordc: Quick Q: Is this ¨Enabling self-disabled pollster¨ ...some check need to add on setup.cfg or so? | 13:58 |
gordc | ildikov: cool cool. makes sense to me | 13:58 |
swamireddy | gordc: Yes, after discussion with ildikov and eglynn, I have left that doc changes (this dicussion a few hours back today) | 13:59 |
gordc | swamireddy: tbh, not exactly sure with details. i just noticed it has some functionality that is similar to what you'd want. | 13:59 |
gordc | swamireddy: cool cool. i reviewed it. i'm ok with fixing other issue in separate patch. | 14:00 |
swamireddy | gordc: Thank you... | 14:01 |
swamireddy | gordc: ..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 needed | 14:02 |
gordc | agreed. | 14:02 |
*** ddieterly has quit IRC | 14:07 | |
swamireddy | gordc: I will check the ¨Enabling self-disabled pollster¨ patch, and see if I can use it | 14:07 |
gordc | swamireddy: awesome! i was just going to ask if you could look at it. :) | 14:07 |
swamireddy | gordc: cool...As you suggested, will do that in a separate patch... | 14:08 |
gordc | swamireddy: perfect. | 14:09 |
*** moravec has quit IRC | 14:10 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: reorganise architecture page https://review.openstack.org/155941 | 14:10 |
*** changbl has quit IRC | 14:13 | |
*** _nadya_ has joined #openstack-ceilometer | 14:14 | |
openstackgerrit | Vadim Rovachev proposed openstack/ceilometer-specs: Add specification for integration tests. https://review.openstack.org/150794 | 14:15 |
cdent | thanks for the typo catch linuxhermit | 14:19 |
openstackgerrit | Chris Dent proposed openstack/ceilometer: Initial gabbi testing for alarms https://review.openstack.org/153293 | 14:19 |
*** swamireddy is now known as swamireddy_afk | 14:20 | |
*** zul has quit IRC | 14:23 | |
linuxhermit | sorry :/ | 14:24 |
linuxhermit | the tests ran great! | 14:24 |
*** zul has joined #openstack-ceilometer | 14:24 | |
vrovachev | cdent: I answered for your comments: https://review.openstack.org/#/c/150794/ :-) | 14:28 |
*** nellysmitt has quit IRC | 14:28 | |
cdent | Yeah, saw that vrovachev, and I +1d | 14:28 |
*** nellysmitt has joined #openstack-ceilometer | 14:29 | |
*** nellysmitt has quit IRC | 14:29 | |
vrovachev | cdent: thank you so much :-) | 14:29 |
*** nellysmitt has joined #openstack-ceilometer | 14:29 | |
*** _nadya_ has quit IRC | 14:39 | |
*** rbak has joined #openstack-ceilometer | 14:44 | |
*** ddieterly has joined #openstack-ceilometer | 14:44 | |
gordc | eglynn: wasm | 14:49 |
gordc | wasn't aware you still have core in glance | 14:49 |
eglynn | gordc: yeah, I should have been grandfathered out ages ago :) | 14:50 |
linuxhermit | gordc you were right I angered the gate gods | 14:54 |
*** nellysmitt has quit IRC | 14:55 | |
gordc | linuxhermit: lol never anger the gate gods. | 14:55 |
gordc | they put you in recheck purgatory. | 14:55 |
*** thomasem has joined #openstack-ceilometer | 14:55 | |
linuxhermit | :/ | 14:56 |
linuxhermit | haha | 14:56 |
*** exploreshaifali has quit IRC | 14:57 | |
*** eglynn has quit IRC | 15:02 | |
*** eglynn has joined #openstack-ceilometer | 15:05 | |
*** alexpilotti has quit IRC | 15:07 | |
gordc | sileht: are we deprecating the use of *.common.notifier.*? ie notification_driver = cinder.openstack.common.notifier.rpc_notifier | 15:07 |
openstackgerrit | Chris Dent proposed stackforge/gnocchi: Add a py27-cover tox target https://review.openstack.org/158672 | 15:09 |
sileht | gordc, not yet, but this is an oslo.msg todo | 15:10 |
gordc | sileht: what's the recommended usage? notification_driver = rabbit? | 15:11 |
sileht | notification_driver = messagingv2 | 15:11 |
sileht | gordc, ^ | 15:11 |
gordc | sileht: ah right. cool cool. thanks! | 15:12 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/157641 | 15:13 |
sileht | gordc, this is a MultiStrOpt, so you can set many drivers now | 15:13 |
sileht | :q | 15:13 |
*** ccrouch has joined #openstack-ceilometer | 15:14 | |
gordc | sileht: so i can have notification_driver = log as well if i have option of messagingv2 already? | 15:15 |
sileht | gordc, yes | 15:15 |
gordc | cool cool | 15:16 |
openstackgerrit | Merged openstack/ceilometer: compress events notes https://review.openstack.org/156342 | 15:17 |
openstackgerrit | Merged openstack/ceilometer: Fix unittests for supporting py-pgsql env https://review.openstack.org/157836 | 15:17 |
*** nellysmitt has joined #openstack-ceilometer | 15:18 | |
*** prad has joined #openstack-ceilometer | 15:19 | |
*** devlaps has joined #openstack-ceilometer | 15:24 | |
*** devlaps has quit IRC | 15:24 | |
*** ddieterl_ has joined #openstack-ceilometer | 15:25 | |
*** ddieterly has quit IRC | 15:25 | |
*** Ephur has quit IRC | 15:30 | |
*** ybathia has joined #openstack-ceilometer | 15:32 | |
*** nellysmitt has quit IRC | 15:32 | |
*** nellysmitt has joined #openstack-ceilometer | 15:32 | |
*** _nadya_ has joined #openstack-ceilometer | 15:35 | |
*** david-lyle_afk is now known as david-lyle | 15:47 | |
*** zul has quit IRC | 15:50 | |
*** zul has joined #openstack-ceilometer | 15:50 | |
*** ddieterl_ has quit IRC | 16:01 | |
*** ddieterly has joined #openstack-ceilometer | 16:02 | |
*** shengjiemin has quit IRC | 16:03 | |
*** sbfox has joined #openstack-ceilometer | 16:04 | |
*** shengjiemin has joined #openstack-ceilometer | 16:05 | |
*** fnaval has joined #openstack-ceilometer | 16:08 | |
*** sbfox has quit IRC | 16:08 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/157641 | 16:13 |
*** changbl has joined #openstack-ceilometer | 16:16 | |
*** ybathia has quit IRC | 16:20 | |
*** amalagon has quit IRC | 16:21 | |
*** _cjones_ has joined #openstack-ceilometer | 16:40 | |
*** _cjones_ has quit IRC | 16:41 | |
*** _cjones_ has joined #openstack-ceilometer | 16:42 | |
*** _cjones_ has quit IRC | 16:46 | |
*** _cjones_ has joined #openstack-ceilometer | 16:46 | |
*** shengjie_ has joined #openstack-ceilometer | 16:58 | |
*** shengjiemin has quit IRC | 16:58 | |
*** shengjie_ is now known as shengjiemin | 16:58 | |
*** shengjiemin has quit IRC | 17:03 | |
*** shengjiemin has joined #openstack-ceilometer | 17:04 | |
*** yeungp has joined #openstack-ceilometer | 17:05 | |
*** exploreshaifali has joined #openstack-ceilometer | 17:07 | |
*** _nadya_ has quit IRC | 17:08 | |
*** IvanBerezovskiy_ has quit IRC | 17:15 | |
*** nellysmitt has quit IRC | 17:17 | |
*** nellysmitt has joined #openstack-ceilometer | 17:18 | |
*** rwsu-afk is now known as rwsu | 17:18 | |
openstackgerrit | Merged openstack/ceilometer: Add ceph object storage meters https://review.openstack.org/158656 | 17:23 |
*** shengjiemin has quit IRC | 17:23 | |
*** sbfox has joined #openstack-ceilometer | 17:24 | |
*** jmatthews has joined #openstack-ceilometer | 17:25 | |
*** shengjiemin has joined #openstack-ceilometer | 17:29 | |
*** Ala has quit IRC | 17:30 | |
*** _cjones_ has quit IRC | 17:33 | |
*** _cjones_ has joined #openstack-ceilometer | 17:36 | |
*** sbfox has quit IRC | 17:41 | |
openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer https://review.openstack.org/158519 | 17:50 |
openstackgerrit | Eric Brown proposed openstack/ceilometer: Use secure default for bind interfaces https://review.openstack.org/158523 | 17:51 |
openstackgerrit | Eric Brown proposed openstack/ceilometer: Use secure default for bind interfaces https://review.openstack.org/158523 | 17:52 |
*** ParsectiX has joined #openstack-ceilometer | 17:54 | |
*** shengjiemin has quit IRC | 17:55 | |
*** karimb has quit IRC | 17:56 | |
*** eglynn is now known as eglynn-afk | 18:00 | |
*** sbfox has joined #openstack-ceilometer | 18:02 | |
swamireddy_afk | ildikov: 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_afk | ildikov: Please let me know if you need more info here | 18:08 |
*** prad has quit IRC | 18:09 | |
*** promulo has joined #openstack-ceilometer | 18:11 | |
*** devlaps has joined #openstack-ceilometer | 18:11 | |
*** Longgeek has quit IRC | 18:15 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add option to store raw notification https://review.openstack.org/153790 | 18:15 |
*** changbl has quit IRC | 18:17 | |
*** harlowja has joined #openstack-ceilometer | 18:20 | |
*** harlowja has quit IRC | 18:24 | |
*** harlowja has joined #openstack-ceilometer | 18:24 | |
*** changbl has joined #openstack-ceilometer | 18:27 | |
*** ParsectiX has quit IRC | 18:37 | |
*** ParsectiX has joined #openstack-ceilometer | 18:38 | |
*** ParsectiX has quit IRC | 18:41 | |
*** ParsectiX has joined #openstack-ceilometer | 18:41 | |
*** cdent has quit IRC | 18:45 | |
ildikov | swamireddy_afk: thanks, I will check your comment | 18:46 |
*** zul has quit IRC | 18:52 | |
*** zul has joined #openstack-ceilometer | 18:52 | |
*** zul has quit IRC | 18:53 | |
*** zul has joined #openstack-ceilometer | 18:53 | |
*** changbl has quit IRC | 18:58 | |
*** yeungp has quit IRC | 19:03 | |
*** fabiog has joined #openstack-ceilometer | 19:07 | |
*** Longgeek has joined #openstack-ceilometer | 19:15 | |
*** _cjones_ has quit IRC | 19:16 | |
openstackgerrit | Merged openstack/ceilometer: Adding links API and CLI query examples https://review.openstack.org/157459 | 19:18 |
*** _cjones_ has joined #openstack-ceilometer | 19:21 | |
*** moravec has joined #openstack-ceilometer | 19:21 | |
fabiog | gordc: ping | 19:21 |
gordc | fabiog: hey, whats up? | 19:26 |
*** prad has joined #openstack-ceilometer | 19:31 | |
*** devlaps has quit IRC | 19:33 | |
fabiog | gordc: there is a patch for client that failed gate | 19:35 |
fabiog | gordc: https://review.openstack.org/#/c/139659/ | 19:35 |
fabiog | gordc: and I cannot figure out why is failing | 19:35 |
fabiog | gordc: is kind of important to support V3 | 19:35 |
gordc | fabiog: i see. i'll try out patch. (i think zqfan might be on new years holidays still) | 19:37 |
fabiog | gordc: yes, I guess. Thanks | 19:37 |
*** jmatthews has quit IRC | 19:40 | |
*** afazekas has joined #openstack-ceilometer | 19:42 | |
*** prad has quit IRC | 19:43 | |
afazekas | Is the new python-ceilometerclient really requires new python-keystoneclient or it just added because the Proposal bot suggested it ? | 19:44 |
afazekas | http://logs.openstack.org/62/158162/6/check/check-tempest-dsvm-full-icehouse/e759fb4/logs/screen-s-proxy.txt.gz | 19:44 |
afazekas | https://pypi.python.org/pypi/python-ceilometerclient/1.0.13 is released today and the icehouse jobs started to fail | 19:44 |
afazekas | gordc, ^ | 19:45 |
gordc | afazekas: :( | 19:46 |
gordc | fabiog: the keystone work you added, does it require new python-keystoneclient? | 19:46 |
afazekas | On icehouse we have python-keystoneclient==0.11.2 | 19:47 |
gordc | afazekas: considering fabiog's work was using pre 1.x builds we don't have a real need for it... probably just an autopatch | 19:50 |
gordc | afazekas: 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 |
afazekas | probably bacuse we do not have icehouse job on the clients | 19:53 |
afazekas | looks like the ceilo client version will be limited to 1.0.12 on the icehouse jobs (#openstack-qa) | 19:54 |
*** ddieterly has quit IRC | 19:56 | |
*** moravec1 has joined #openstack-ceilometer | 19:57 | |
*** ddieterly has joined #openstack-ceilometer | 19:58 | |
gordc | afazekas: ok. just looking at the bug related to new release... seems ok to limit it for icehouse. | 19:58 |
gordc | eglynn-afk: ^ for reference. | 19:58 |
*** moravec has quit IRC | 19:58 | |
afazekas | according the discussion the ceilo client after icehouse release should have use a version number >=1.1 | 19:59 |
afazekas | especially if it break things .. | 20:00 |
afazekas | gordc: the issue will be solved on the gate soon : https://review.openstack.org/#/c/158842/ | 20:00 |
gordc | afazekas: sorry i'm not aware of the discussion for version number... do you have a link? | 20:01 |
gordc | afazekas: cool cool. thanks for checking here first | 20:01 |
gordc | fabiog: looks to be a merge issue with https://review.openstack.org/#/c/155164/ | 20:02 |
*** _nadya_ has joined #openstack-ceilometer | 20:08 | |
*** _nadya_ has quit IRC | 20:12 | |
jroll | gordc: hi! are you gordon chung? | 20:21 |
gordc | jroll: depends on the follow up question is... | 20:24 |
jroll | hehe | 20:24 |
jroll | mind a PM? | 20:24 |
gordc | jroll: sure. | 20:24 |
*** exploreshaifali has quit IRC | 20:24 | |
*** nellysmitt has quit IRC | 20:26 | |
*** eglynn-afk is now known as eglynn | 20:28 | |
*** jmatthews has joined #openstack-ceilometer | 20:38 | |
fabiog | gordc: a rebase would do then? | 20:39 |
openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer https://review.openstack.org/158519 | 20:40 |
gordc | fabiog: there's probably a code change involved | 20:40 |
*** prad has joined #openstack-ceilometer | 20:40 | |
gordc | fabiog: i can hit the rebase button if you want to give that a try first. | 20:41 |
*** moravec1 has quit IRC | 20:41 | |
fabiog | gordc: yeah, give it a go. If that does not work I will ping ZhiQuang | 20:42 |
openstackgerrit | gordon chung proposed openstack/python-ceilometerclient: Enable specified project_id in CLI commands https://review.openstack.org/139659 | 20:42 |
*** jmatthews has quit IRC | 20:42 | |
gordc | fabiog: 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 holidays | 20:43 |
jroll | do I need mongodb installed to run tox on ceilometer? | 20:47 |
gordc | jroll: ... i believe you do. you can try running tox -epy-mysql, that may avoid the mongo req | 20:48 |
jroll | gordc: thanks | 20:48 |
*** ddieterly has quit IRC | 20:50 | |
*** ddieterly has joined #openstack-ceilometer | 20:50 | |
fabiog | gordc: np | 20:59 |
fabiog | gordc: 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 reviews | 21:00 |
fabiog | gordc: cdent was saying it is difficult to understand | 21:01 |
gordc | fabiog: sorry was in a random meeting | 21:07 |
gordc | fabiog: yeah i think it might help... for me i'm just wondering the intended interaction pattern is. | 21:08 |
gordc | right 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 |
gordc | fabiog: 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-ceilometer | 21:14 | |
*** ParsectiX has quit IRC | 21:19 | |
*** ParsectiX has joined #openstack-ceilometer | 21:19 | |
*** Longgeek has quit IRC | 21:24 | |
*** sbfox has quit IRC | 21:26 | |
*** sbfox has joined #openstack-ceilometer | 21:28 | |
*** _cjones_ has quit IRC | 21:39 | |
*** ParsectiX has quit IRC | 21:41 | |
*** akrivoka has quit IRC | 21:42 | |
*** ParsectiX has joined #openstack-ceilometer | 21:42 | |
*** jmatthews has joined #openstack-ceilometer | 21:42 | |
*** _cjones_ has joined #openstack-ceilometer | 21:50 | |
*** karimb has joined #openstack-ceilometer | 21:52 | |
*** moravec has joined #openstack-ceilometer | 22:05 | |
openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer https://review.openstack.org/158519 | 22:06 |
*** eglynn has quit IRC | 22:07 | |
openstackgerrit | Rohit Jaiswal proposed openstack/ceilometer: Added option to only expire samples to expirer https://review.openstack.org/158519 | 22:08 |
*** moravec has quit IRC | 22:11 | |
*** changbl has quit IRC | 22:17 | |
*** ViswaV has joined #openstack-ceilometer | 22:28 | |
fabiog | gordc: I will put together a deck and publish it in Slideshare. I will probably do it before our thu meeting | 22:32 |
gordc | fabiog: sounds good. | 22:34 |
*** jmatthews has quit IRC | 22:35 | |
*** openstackgerrit has quit IRC | 22:37 | |
*** openstackgerrit has joined #openstack-ceilometer | 22:37 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add missing objectstore entry points https://review.openstack.org/158907 | 22:57 |
*** harlowja is now known as harlowja_away | 23:02 | |
*** jmatthews has joined #openstack-ceilometer | 23:03 | |
*** harlowja_away is now known as harlowja | 23:05 | |
*** fabiog has quit IRC | 23:09 | |
*** Longgeek has joined #openstack-ceilometer | 23:09 | |
*** thomasem has quit IRC | 23:10 | |
*** _cjones_ has quit IRC | 23:12 | |
*** _cjones_ has joined #openstack-ceilometer | 23:14 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add missing objectstore entry points https://review.openstack.org/158907 | 23:24 |
*** karimb has quit IRC | 23:25 | |
*** rbak has quit IRC | 23:25 | |
*** _cjones_ has quit IRC | 23:30 | |
*** jmatthews has quit IRC | 23:32 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add an option to disable non-metric meters https://review.openstack.org/156747 | 23:33 |
*** gordc has quit IRC | 23:33 | |
*** Longgeek has quit IRC | 23:42 | |
*** _cjones_ has joined #openstack-ceilometer | 23:43 | |
*** prad has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!