| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/147024 | 00:00 |
|---|---|---|
| *** atan8 has joined #openstack-ceilometer | 00:00 | |
| *** nijaba_ has joined #openstack-ceilometer | 00:05 | |
| *** yeungp1 has joined #openstack-ceilometer | 00:07 | |
| *** fc___ has joined #openstack-ceilometer | 00:07 | |
| *** changbl has quit IRC | 00:08 | |
| *** fc__ has quit IRC | 00:09 | |
| *** nealph has quit IRC | 00:09 | |
| *** shardy has quit IRC | 00:09 | |
| *** yeungp has quit IRC | 00:09 | |
| *** nijaba has quit IRC | 00:09 | |
| *** fc___ is now known as fc__ | 00:09 | |
| *** nijaba_ has quit IRC | 00:09 | |
| *** nijaba_ has joined #openstack-ceilometer | 00:09 | |
| *** nealph has joined #openstack-ceilometer | 00:09 | |
| *** Longgeek has quit IRC | 00:10 | |
| *** shardy has joined #openstack-ceilometer | 00:11 | |
| *** LinstatSDR has joined #openstack-ceilometer | 00:16 | |
| *** david-lyle has joined #openstack-ceilometer | 00:28 | |
| *** rbak has quit IRC | 00:38 | |
| *** EmilienM is now known as EmilienM|afk | 00:39 | |
| *** shengjiemin has joined #openstack-ceilometer | 00:39 | |
| *** LinstatSDR has quit IRC | 00:40 | |
| *** zhiyan has joined #openstack-ceilometer | 00:48 | |
| *** EmilienM|afk has quit IRC | 00:52 | |
| *** dougwig has quit IRC | 00:54 | |
| *** EmilienM has joined #openstack-ceilometer | 00:56 | |
| *** david-lyle has quit IRC | 00:58 | |
| *** dougwig has joined #openstack-ceilometer | 00:58 | |
| *** david-lyle has joined #openstack-ceilometer | 00:58 | |
| *** liuh has joined #openstack-ceilometer | 01:03 | |
| *** sbfox has quit IRC | 01:04 | |
| *** david-lyle has quit IRC | 01:05 | |
| *** EmilienM has quit IRC | 01:07 | |
| *** EmilienM has joined #openstack-ceilometer | 01:08 | |
| *** david-lyle has joined #openstack-ceilometer | 01:09 | |
| *** gzhai has joined #openstack-ceilometer | 01:10 | |
| kurtrao | zqfan: Hi Zhiqiang. Jenkins reports py27 failure against my patch https://review.openstack.org/#/c/144192/, and you mentioned that it was because of the sudo issue. But it looks like the patches you uploaded don't have such problem. Could you please tell me how can I fix it? | 01:10 |
| llu | gzhai, please take a look at https://bugs.launchpad.net/ceilometer/+bug/1410411 | 01:10 |
| kurtrao | The log says: sudo: unable to resolve host bare-trusty-1421072110 | 01:10 |
| clarkb | kurtrao: the unable to resolve host isn't the issue. http://logs.openstack.org/92/144192/4/check/gate-ceilometer-python27/1e26f65/console.html#_2015-01-13_10_15_34_630 is | 01:11 |
| clarkb | its also a bit scary that the tests themselves don't fail as a result | 01:13 |
| *** shakamunyi has quit IRC | 01:13 | |
| gzhai | llu: I'll look at this | 01:14 |
| kurtrao | clarkb: Is error because something of the patch? Or it's Jenkins's problem? | 01:19 |
| clarkb | its not a jenkins problem. It may be something in the patch or something in ceilometer | 01:20 |
| kurtrao | "This test has failed because it attempted to execute commands with sudo. See above for the exact commands used." | 01:20 |
| clarkb | yup | 01:20 |
| clarkb | you cannot use sudo in the unittests | 01:20 |
| clarkb | I think adam_g had said that ceilometer's tests sometimes do so it may not be related to your paticular change | 01:21 |
| *** rwsu has quit IRC | 01:21 | |
| kurtrao | So, as long as my local unit tests are passed, I don't need to worry about this failure? | 01:22 |
| clarkb | no, because you may be running sudo locally and not knowing it | 01:22 |
| clarkb | you should check to be sure that your change isn't running sudo in the tests | 01:22 |
| *** promulo_ has quit IRC | 01:24 | |
| *** david-lyle has quit IRC | 01:24 | |
| kurtrao | I need to change the unit test shell script to run sudo in the tests, right? | 01:26 |
| clarkb | you cannot run sudo in thetests | 01:28 |
| clarkb | so need to make sure sudo does not run | 01:28 |
| *** david-lyle has joined #openstack-ceilometer | 01:29 | |
| kurtrao | clarkb: ok, I'll check my patch again | 01:29 |
| kurtrao | Thanks a lot for the help~ | 01:29 |
| *** lsmola has quit IRC | 01:31 | |
| *** lsmola has joined #openstack-ceilometer | 01:32 | |
| *** _cjones_ has quit IRC | 01:33 | |
| *** _cjones_ has joined #openstack-ceilometer | 01:33 | |
| *** ddieterly has joined #openstack-ceilometer | 01:35 | |
| *** david-lyle has quit IRC | 01:35 | |
| *** amalagon has quit IRC | 01:39 | |
| *** _cjones_ has quit IRC | 01:40 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Fix incorrect test case name in test_net.py https://review.openstack.org/147050 | 01:50 |
| *** david-lyle has joined #openstack-ceilometer | 01:52 | |
| *** EmilienM has quit IRC | 01:52 | |
| *** Longgeek has joined #openstack-ceilometer | 01:55 | |
| *** EmilienM has joined #openstack-ceilometer | 01:56 | |
| zqfan-office | clarkb: I see ceilometer jenkins failed because: This test has failed because it attempted to execute commands with sudo. See above for the exact commands used.: sudo /usr/local/jenkins/slave_scripts/jenkins-sudo-grep.sh post | 01:56 |
| clarkb | thats not the command | 01:56 |
| clarkb | zqfan-office: can you link to the log I can show you were the commands are written | 01:56 |
| clarkb | (we have a rule that allows the jenkins user to run sudo for that specific command) | 01:57 |
| zqfan-office | clarkb http://logs.openstack.org/92/144192/4/check/gate-ceilometer-python27/1e26f65/console.html#_2015-01-13_10_15_34_499 | 01:57 |
| clarkb | zqfan-office: http://logs.openstack.org/92/144192/4/check/gate-ceilometer-python27/1e26f65/console.html#_2015-01-13_10_15_34_630 those are the commands | 01:57 |
| *** Yanyanhu has joined #openstack-ceilometer | 01:58 | |
| zqfan-office | clarkb: so it is: /home/jenkins/workspace/gate-ceilometer-python27/.tox/py27/bin/ceilometer-rootwrap /etc/ceilometer/rootwrap.conf ipmitool raw 0x0a 0x2c 0x00 | 01:58 |
| clarkb | ya | 01:59 |
| zqfan-office | clarkb: ok, thanks | 01:59 |
| *** david-lyle has quit IRC | 01:59 | |
| zqfan-office | ping llu: can you see this CI failure? http://logs.openstack.org/92/144192/4/check/gate-ceilometer-python27/1e26f65/console.html#_2015-01-13_10_15_34_630 | 01:59 |
| kurtrao | I turned off my sudo privilege and re-run the unit tests, and all passed | 02:00 |
| zqfan-office | kurtrao: I already comment recheck after your commit | 02:01 |
| kurtrao | Sorry, but, what does that mean? Do I need to do anything to recheck? | 02:03 |
| *** zhiyan has quit IRC | 02:05 | |
| openstackgerrit | Srinivas Sakhamuri proposed openstack/python-ceilometerclient: Fix improper parameter setup for cacert and client certs https://review.openstack.org/146951 | 02:06 |
| *** zhiyan has joined #openstack-ceilometer | 02:06 | |
| llu | zqfan-office: gzhai is checking this | 02:07 |
| kurtrao | zqfan-office: thanks. I got what you mean. I've found this doc: https://wiki.openstack.org/wiki/GerritJenkinsGit | 02:07 |
| adam_g | clarkb, kurtrao i filed https://bugs.launchpad.net/ceilometer/+bug/1410411 about the sudo thing. its turning up in weird places | 02:09 |
| adam_g | zqfan, ^ | 02:09 |
| *** Longgeek has quit IRC | 02:11 | |
| *** vishy has quit IRC | 02:11 | |
| *** vishy has joined #openstack-ceilometer | 02:16 | |
| *** LinstatSDR has joined #openstack-ceilometer | 02:21 | |
| openstackgerrit | mizeng proposed openstack/ceilometer: Expose vm's metadata to metrics https://review.openstack.org/134877 | 02:21 |
| *** EmilienM has quit IRC | 02:22 | |
| *** EmilienM has joined #openstack-ceilometer | 02:23 | |
| *** julim has quit IRC | 02:24 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: [Do Not Merge]Support Network Aggregate Metrics for Instance https://review.openstack.org/147054 | 02:38 |
| *** david-lyle has joined #openstack-ceilometer | 02:40 | |
| *** shakamunyi has joined #openstack-ceilometer | 02:41 | |
| *** harlowja is now known as harlowja_away | 02:46 | |
| *** shakamunyi has quit IRC | 02:47 | |
| *** pabelanger has quit IRC | 02:48 | |
| *** shakamunyi has joined #openstack-ceilometer | 02:48 | |
| *** zhiyan has quit IRC | 02:55 | |
| *** zhiyan has joined #openstack-ceilometer | 02:57 | |
| llu | adam_g, zqfan, gzhai has root cause the bug https://bugs.launchpad.net/ceilometer/+bug/1410411, he'll take it | 02:58 |
| *** yeungp1 has quit IRC | 02:58 | |
| *** pabelanger has joined #openstack-ceilometer | 02:59 | |
| *** yeungp has joined #openstack-ceilometer | 03:01 | |
| *** shengjiemin has quit IRC | 03:01 | |
| *** yeungp has quit IRC | 03:01 | |
| *** GeneralSalty has quit IRC | 03:02 | |
| *** shakamunyi has quit IRC | 03:05 | |
| *** shengjiemin has joined #openstack-ceilometer | 03:09 | |
| *** amalagon has joined #openstack-ceilometer | 03:10 | |
| *** LinstatSDR has quit IRC | 03:11 | |
| *** liuh_ has joined #openstack-ceilometer | 03:17 | |
| *** pabelanger has quit IRC | 03:17 | |
| *** liuh has quit IRC | 03:17 | |
| *** junhongl has quit IRC | 03:18 | |
| *** ddieterly has quit IRC | 03:19 | |
| *** junhongl has joined #openstack-ceilometer | 03:19 | |
| *** pabelanger has joined #openstack-ceilometer | 03:20 | |
| zqfan-office | llu: glad to here that, thanks | 03:20 |
| zqfan-office | s/here/hear/ | 03:20 |
| openstackgerrit | Srinivas Sakhamuri proposed openstack/python-ceilometerclient: Fix improper parameter setup for cacert and client certs https://review.openstack.org/146951 | 03:24 |
| *** david-lyle has quit IRC | 03:25 | |
| *** deepthi has joined #openstack-ceilometer | 03:28 | |
| *** atan8 has quit IRC | 03:39 | |
| *** EmilienM has quit IRC | 03:40 | |
| *** EmilienM has joined #openstack-ceilometer | 03:44 | |
| *** ddieterly has joined #openstack-ceilometer | 03:49 | |
| *** Longgeek has joined #openstack-ceilometer | 03:51 | |
| *** rbak has joined #openstack-ceilometer | 03:52 | |
| *** ddieterly has quit IRC | 03:54 | |
| *** pabelanger has quit IRC | 03:54 | |
| *** rbak has quit IRC | 03:55 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Support Network Aggregate Metrics for Instance https://review.openstack.org/147054 | 03:59 |
| *** pabelanger has joined #openstack-ceilometer | 04:00 | |
| *** EmilienM has quit IRC | 04:01 | |
| *** nijaba_ has quit IRC | 04:01 | |
| *** Yanyanhu has quit IRC | 04:02 | |
| *** Yanyan has joined #openstack-ceilometer | 04:02 | |
| *** LinstatSDR has joined #openstack-ceilometer | 04:03 | |
| *** nijaba has joined #openstack-ceilometer | 04:07 | |
| *** nijaba has quit IRC | 04:07 | |
| *** nijaba has joined #openstack-ceilometer | 04:07 | |
| *** EmilienM has joined #openstack-ceilometer | 04:08 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Check unsupported query filters of listing events https://review.openstack.org/96653 | 04:14 |
| *** flwang has quit IRC | 04:34 | |
| *** EmilienM has quit IRC | 04:35 | |
| *** EmilienM has joined #openstack-ceilometer | 04:38 | |
| *** flwang has joined #openstack-ceilometer | 04:41 | |
| *** _cjones_ has joined #openstack-ceilometer | 04:41 | |
| *** yatin has joined #openstack-ceilometer | 04:41 | |
| *** Longgeek has quit IRC | 04:42 | |
| *** Longgeek has joined #openstack-ceilometer | 04:44 | |
| *** _cjones_ has quit IRC | 04:45 | |
| *** changbl has joined #openstack-ceilometer | 04:46 | |
| *** shengjiemin has quit IRC | 04:49 | |
| *** shengjiemin has joined #openstack-ceilometer | 04:51 | |
| *** vishy has quit IRC | 04:57 | |
| *** vishy has joined #openstack-ceilometer | 05:03 | |
| *** EmilienM has quit IRC | 05:12 | |
| *** EmilienM has joined #openstack-ceilometer | 05:16 | |
| *** ddieterly has joined #openstack-ceilometer | 05:19 | |
| *** zigo has quit IRC | 05:20 | |
| *** zigo has joined #openstack-ceilometer | 05:21 | |
| *** ddieterly has quit IRC | 05:23 | |
| *** pabelanger has quit IRC | 05:24 | |
| *** pabelanger has joined #openstack-ceilometer | 05:28 | |
| *** liuh__ has joined #openstack-ceilometer | 05:31 | |
| *** liuh_ has quit IRC | 05:33 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: support time to live on event database for MongoDB https://review.openstack.org/146367 | 05:35 |
| openstackgerrit | Zhai, Edwin proposed openstack/ceilometer: Avoid executing ipmitool in IPMI unit test https://review.openstack.org/147067 | 05:40 |
| *** pabelanger has quit IRC | 05:49 | |
| *** ddieterly has joined #openstack-ceilometer | 05:49 | |
| *** pabelanger has joined #openstack-ceilometer | 05:51 | |
| *** yatin has quit IRC | 05:53 | |
| *** ddieterly has quit IRC | 05:54 | |
| *** yatin has joined #openstack-ceilometer | 05:56 | |
| *** lan_ has quit IRC | 06:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/146767 | 06:09 |
| *** ildikov has quit IRC | 06:14 | |
| *** LinstatSDR has quit IRC | 06:16 | |
| *** Longgeek has quit IRC | 06:19 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Support Network Aggregate Metrics for Instance https://review.openstack.org/147054 | 06:22 |
| zqfan-office | check-grenade-dsvm seems broken | 06:32 |
| *** Longgeek has joined #openstack-ceilometer | 06:34 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/python-ceilometerclient: Enable specified project_id in CLI commands https://review.openstack.org/139659 | 06:38 |
| *** ddieterly has joined #openstack-ceilometer | 06:49 | |
| *** Longgeek has quit IRC | 06:53 | |
| *** ddieterly has quit IRC | 06:54 | |
| openstackgerrit | Srinivas Sakhamuri proposed openstack/python-ceilometerclient: Fix improper parameter setup for cacert and client certs https://review.openstack.org/146951 | 06:55 |
| *** Longgeek has joined #openstack-ceilometer | 06:59 | |
| *** shengjiemin has quit IRC | 07:07 | |
| *** pabelanger has quit IRC | 07:07 | |
| *** zhiyan has quit IRC | 07:08 | |
| *** zhiyan has joined #openstack-ceilometer | 07:11 | |
| *** ildikov has joined #openstack-ceilometer | 07:14 | |
| *** pabelanger has joined #openstack-ceilometer | 07:15 | |
| *** ddieterly has joined #openstack-ceilometer | 07:19 | |
| *** ddieterly has quit IRC | 07:24 | |
| *** deepthi has quit IRC | 07:26 | |
| *** pabelanger has quit IRC | 07:37 | |
| *** pabelanger has joined #openstack-ceilometer | 07:38 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/python-ceilometerclient: Add Sample API support https://review.openstack.org/132577 | 07:42 |
| *** Ala has joined #openstack-ceilometer | 07:48 | |
| *** shengjiemin has joined #openstack-ceilometer | 08:08 | |
| *** deepthi has joined #openstack-ceilometer | 08:08 | |
| *** shengjiemin has quit IRC | 08:20 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: Add Event and Trait API to document https://review.openstack.org/132514 | 08:20 |
| *** pabelanger has quit IRC | 08:24 | |
| *** pabelanger has joined #openstack-ceilometer | 08:26 | |
| *** pabelanger has quit IRC | 08:39 | |
| *** pabelanger has joined #openstack-ceilometer | 08:39 | |
| *** pabelanger has joined #openstack-ceilometer | 08:39 | |
| *** safchain has joined #openstack-ceilometer | 08:45 | |
| *** lan__ has joined #openstack-ceilometer | 08:50 | |
| *** liuh__ has quit IRC | 09:03 | |
| *** nellysmitt has joined #openstack-ceilometer | 09:25 | |
| *** Sanchit has joined #openstack-ceilometer | 09:39 | |
| Sanchit | Hi, I am having a setup of objectStorage which can handle a load of about 10,000 requests. | 09:41 |
| Sanchit | If I setup a ceilometer server. will it be able to handle the same amount of requests? | 09:41 |
| Sanchit | for Incoming/Outgoing bytes publishing as well as polling | 09:42 |
| *** eglynn-officeafk is now known as eglynn-office | 09:44 | |
| *** exploreshaifali has joined #openstack-ceilometer | 09:49 | |
| *** vishy has quit IRC | 09:59 | |
| openstackgerrit | Dina Belova proposed openstack/ceilometer: Use oslo_db instead of deprecated oslo.db https://review.openstack.org/146424 | 10:00 |
| *** pabelanger has quit IRC | 10:03 | |
| *** pabelanger has joined #openstack-ceilometer | 10:05 | |
| *** vishy has joined #openstack-ceilometer | 10:09 | |
| openstackgerrit | Lena Novokshonova proposed openstack/ceilometer: Use oslo_config instead of deprecated oslo.config https://review.openstack.org/146421 | 10:17 |
| *** vishy has quit IRC | 10:21 | |
| *** yasu_ has joined #openstack-ceilometer | 10:29 | |
| *** yasu_ has quit IRC | 10:31 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: Upgrade to hacking 0.10 https://review.openstack.org/146846 | 10:31 |
| *** vishy has joined #openstack-ceilometer | 10:34 | |
| *** pabelanger has quit IRC | 11:00 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/ceilometer: use default trait type in event list query https://review.openstack.org/147111 | 11:01 |
| *** Yanyan has quit IRC | 11:01 | |
| *** pabelanger has joined #openstack-ceilometer | 11:02 | |
| *** alexpilotti has joined #openstack-ceilometer | 11:03 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: indexer: store aggregation methods in archive policies https://review.openstack.org/147114 | 11:06 |
| *** deepthi has quit IRC | 11:11 | |
| *** _nadya_ has joined #openstack-ceilometer | 11:14 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: indexer: use ArchivePolicy in create_archive_policy() https://review.openstack.org/146912 | 11:18 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: indexer: store aggregation methods in archive policies https://review.openstack.org/147114 | 11:18 |
| openstackgerrit | Merged stackforge/gnocchi: Upgrade to hacking 0.10 https://review.openstack.org/146846 | 11:23 |
| *** cdent has joined #openstack-ceilometer | 11:30 | |
| eglynn-office | cdent: hola ... just thinking some more about that puppet-redis issue with multiple Package['redis'] declarations in install.pp & sentinel.pp | 11:47 |
| eglynn-office | ... so instead of my original workaround of checking first explicitly if the package is already defined | 11:47 |
| * cdent listens | 11:47 | |
| eglynn-office | ... seems to me now a more complete solution woud be to include both preinstall & install in sentinel.pp | 11:47 |
| eglynn-office | (as the preinstall presumably may be needed in some cases, where the repos are being explicitly managed?) | 11:47 |
| eglynn-office | so that way we could drop the explicit package declaration in https://github.com/arioch/puppet-redis/blob/master/manifests/sentinel.pp#L137 | 11:49 |
| cdent | I had considered that early on but didn’t know one way or another if it was right. It does seem cleaner than using defined | 11:49 |
| cdent | if both classes include those files does that avoid a duplicate declaration or cause one? | 11:51 |
| eglynn-office | hmmm, from my shallow puppet knowledge I think the included module is not re-included | 11:52 |
| eglynn-office | EmilienM: ^^^ you might be able to speak definitively on that puppet question? | 11:53 |
| openstackgerrit | Lena Novokshonova proposed openstack/ceilometer: Use oslo_config instead of deprecated oslo.config https://review.openstack.org/146421 | 11:53 |
| * cdent feels lame for having thrown around a bunch of code created from his shallow knowledge | 11:54 | |
| cdent | I suppose I’m less shallow know than I was... | 11:54 |
| eglynn-office | yeah, it's all good, best way to learn :) | 11:55 |
| eglynn-office | ... in any case, I'll give the include approach a whirl in a fresh staypuft deploy | 11:55 |
| eglynn-office | ... and then cook up a quick PR for arioch if it does the job | 11:56 |
| cdent | ✔ | 11:56 |
| *** claudiub has joined #openstack-ceilometer | 12:09 | |
| *** lan__ has quit IRC | 12:10 | |
| * cdent lunches | 12:11 | |
| *** cdent has quit IRC | 12:11 | |
| *** cdent has joined #openstack-ceilometer | 12:35 | |
| *** _nadya_ has quit IRC | 12:37 | |
| *** zqfan_ has joined #openstack-ceilometer | 12:39 | |
| *** _nadya_ has joined #openstack-ceilometer | 12:45 | |
| *** ildikov has quit IRC | 12:48 | |
| *** ryanpetrello has joined #openstack-ceilometer | 12:53 | |
| *** boris-42 has joined #openstack-ceilometer | 12:56 | |
| *** nealph_ has joined #openstack-ceilometer | 13:02 | |
| *** ildikov has joined #openstack-ceilometer | 13:06 | |
| *** _nadya_ has quit IRC | 13:08 | |
| *** claudiub has quit IRC | 13:16 | |
| *** fc__ has quit IRC | 13:17 | |
| *** fc__ has joined #openstack-ceilometer | 13:18 | |
| *** yassine has joined #openstack-ceilometer | 13:20 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: indexer: store aggregation methods in archive policies https://review.openstack.org/147114 | 13:23 |
| EmilienM | eglynn-office: hello! let me check | 13:25 |
| EmilienM | cdent: eglynn-office: I need a bit of time to read the puppet manifests but I may have an idea | 13:25 |
| EmilienM | cdent: in d72af9ab3c2911b6dc18c5cc12e700630ebdcfb2 you should have not add the package resource | 13:27 |
| EmilienM | the require would have been enough. | 13:27 |
| cdent | I assume you saw the "shallow knowledge" excuses above EmilienM? :( | 13:28 |
| cdent | If the require is enough, how does it know about the package if only sentinel is being used? | 13:29 |
| EmilienM | cdent: no problem, I'll make a pull-request. | 13:29 |
| EmilienM | cdent: init.pp should be run on all redis nodes | 13:29 |
| EmilienM | it's a good practice in puppet | 13:29 |
| EmilienM | ::redis class if you prefer | 13:29 |
| cdent | Then some other things will need to change too? | 13:30 |
| cdent | for example this is the entirety of the sentinel manifest in packstack, which makes no reference to init.pp or ::redis: https://review.openstack.org/#/c/146548/4/packstack/puppet/templates/redis_sentinel.pp,cm | 13:31 |
| EmilienM | cdent: I don't know. I'll have to read the module to know what needs to be changes | 13:31 |
| EmilienM | cdent: it's an error then. | 13:31 |
| cdent | thanks very much for taking a look | 13:31 |
| EmilienM | cdent: it's a huge puppet design error | 13:32 |
| EmilienM | ::redis is included everywhere redis run | 13:32 |
| EmilienM | it's a common class | 13:32 |
| cdent | Which "it" is a huge design error? | 13:32 |
| EmilienM | cdent: not using "::redis" | 13:33 |
| EmilienM | I even wonder how the reviewer could have miss it | 13:33 |
| EmilienM | cdent: I'm working on this today and I'll let you know. | 13:35 |
| cdent | thanks | 13:35 |
| EmilienM | I'll send a PR maybe today | 13:35 |
| EmilienM | cdent: my pleasure :) | 13:35 |
| EmilienM | cdent: I need this feature, you know :P | 13:35 |
| cdent | I'll be glad to learn from your fixes. | 13:36 |
| eglynn-office | EmilienM: thanks for the insight! | 13:37 |
| eglynn-office | EmilienM: ... BTW explicitly including preinstall and install in sentinel.pp (instead of the explicit Package declaration) also seems to do the job | 13:37 |
| EmilienM | eglynn-office: it's not like this it works in Puppet | 13:38 |
| eglynn-office | EmilienM: ... but I'd be happier with the more "correct" puppet idiom | 13:38 |
| EmilienM | eglynn-office: init.pp == common class, run everywhere, which already include preinstall & install. | 13:38 |
| EmilienM | so you did a workaround to not use init.pp which is bad in Puppet | 13:38 |
| *** _nadya_ has joined #openstack-ceilometer | 13:39 | |
| eglynn-office | EmilienM: cool ... so will you submit a pull-request to arioch/puppet-redis with the preferred puppet approach to addressing this? | 13:39 |
| EmilienM | eglynn-office: yes Sir. | 13:39 |
| eglynn-office | EmilienM: thank you sir! :) | 13:39 |
| eglynn-office | gotta scoot ... back online shortly | 13:40 |
| *** eglynn-office is now known as eglynn-officeafk | 13:40 | |
| jd__ | eglynn-officeafk: I'm waiting for the file backend in tooz to be merged before releasing 0.11, I hope that's not an issue? | 13:43 |
| cdent | jd__: should be okay, there's delay on getting sentinel support in the other bits and pieces (packstack etc) | 13:44 |
| cdent | How long do you think it will be? | 13:45 |
| jd__ | cdent: a couple of days | 13:45 |
| cdent | yeah, should be fine | 13:46 |
| *** _nadya_ has quit IRC | 13:47 | |
| *** _nadya_ has joined #openstack-ceilometer | 13:50 | |
| *** LinstatSDR has joined #openstack-ceilometer | 13:50 | |
| *** yatin has quit IRC | 13:50 | |
| *** eglynn has joined #openstack-ceilometer | 13:53 | |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: indexer: store aggregation methods in archive policies https://review.openstack.org/147114 | 13:54 |
| *** gordc has joined #openstack-ceilometer | 13:54 | |
| eglynn | jd__: thanks for the heads-up on tooz-0.11 ... wot cdent said :) | 13:54 |
| *** deepthi has joined #openstack-ceilometer | 13:59 | |
| *** julim has joined #openstack-ceilometer | 14:02 | |
| *** eglynn has quit IRC | 14:05 | |
| *** eglynn has joined #openstack-ceilometer | 14:21 | |
| *** zqfan_ has quit IRC | 14:30 | |
| *** thomasem has joined #openstack-ceilometer | 14:35 | |
| *** nealph_ has quit IRC | 14:44 | |
| openstackgerrit | Mehdi Abaakouk proposed stackforge/gnocchi: Add ceph as storage driver https://review.openstack.org/139575 | 14:46 |
| openstackgerrit | Mehdi Abaakouk proposed stackforge/gnocchi: ceph: use ceph instead of tooz for locking https://review.openstack.org/139667 | 14:46 |
| *** nealph_ has joined #openstack-ceilometer | 14:47 | |
| *** nealph_ has quit IRC | 14:49 | |
| *** claudiub has joined #openstack-ceilometer | 14:53 | |
| *** exploreshaifali has quit IRC | 15:06 | |
| *** rbak has joined #openstack-ceilometer | 15:06 | |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Add a direct to database publisher https://review.openstack.org/146142 | 15:08 |
| EmilienM | sileht: jd__: stupid question, can we use ceph for tooz backend? | 15:11 |
| jd__ | EmilienM: as soon as sileht patches are merged yes | 15:12 |
| EmilienM | jd__: in tooz, not in gnocchi, right? | 15:12 |
| *** shengjiemin has joined #openstack-ceilometer | 15:12 | |
| EmilienM | I don't see the patches in Tooz | 15:12 |
| jd__ | EmilienM: ah sorry I thought you were talking about Gnocchi because it's not the tooz channe :p | 15:13 |
| EmilienM | :) | 15:13 |
| jd__ | EmilienM: yeah sileht has a TODO about adding Ceph as a tooz backend | 15:13 |
| EmilienM | jd__: for Kilo you think? | 15:13 |
| jd__ | EmilienM: it might be if sileht hurry up | 15:14 |
| openstackgerrit | Merged openstack/ceilometer: untie pipeline manager from samples https://review.openstack.org/140131 | 15:14 |
| openstackgerrit | Merged openstack/ceilometer: make transformers optional in pipeline https://review.openstack.org/145345 | 15:14 |
| EmilienM | jd__: last question - all ceph usage will be with RBD driver, not RADOS API right? | 15:14 |
| *** yeungp has joined #openstack-ceilometer | 15:16 | |
| *** ddieterly has joined #openstack-ceilometer | 15:18 | |
| EmilienM | jd__: and also, will it be possible to run Gnocchi API with WSGI process in Apache? | 15:18 |
| * gordc wonders how my changes merge if our gate is broken... oh well | 15:18 | |
| EmilienM | sileht: ^ | 15:18 |
| *** _nadya_ has quit IRC | 15:19 | |
| sileht | EmilienM, yes this is possible, the devstack patch do that by default | 15:19 |
| EmilienM | ok thanks | 15:19 |
| EmilienM | sileht: and for Ceph? | 15:19 |
| sileht | EmilienM, gnocchi store metric with rados API, and But I have not follow the what is retrieved by ceilometer | 15:20 |
| *** yeungp has quit IRC | 15:23 | |
| DinaBelova | gordc, magic :) | 15:26 |
| gordc | good enough for me. magic is my friend. | 15:27 |
| DinaBelova | gordc, lol | 15:29 |
| *** yeungp has joined #openstack-ceilometer | 15:29 | |
| *** nealph_ has joined #openstack-ceilometer | 15:32 | |
| *** nealph has quit IRC | 15:38 | |
| IvanBerezovskiy | cdent, Hi. I see that your patch for multiple sentinels is merged. will you create new tooz release soon? | 15:39 |
| cdent | IvanBerezovskiy: there's some delays because the sentinel support that I made in puppet-redis is not quite right, EmilienM is fixing it. jd__ is going to release a new tooz in the next couple days after the file backend is merged | 15:40 |
| openstackgerrit | Julien Danjou proposed stackforge/gnocchi: Use oslo.serialization.jsonutils to serialize JSON https://review.openstack.org/147207 | 15:40 |
| *** nealph_ has quit IRC | 15:41 | |
| *** ddieterly has quit IRC | 15:41 | |
| cdent | IvanBerezovskiy: if its something you're hoping to experiment with, you can get the master tooz off github, and configure your ceilometer and redis setup by hand? | 15:41 |
| *** nealph has joined #openstack-ceilometer | 15:41 | |
| cdent | If that's no good, it'll be a few days yet | 15:42 |
| IvanBerezovskiy | cdent, ok, great. Can you please provide a link for puppet-redis that you are working on? | 15:42 |
| *** ddieterly has joined #openstack-ceilometer | 15:42 | |
| cdent | https://github.com/arioch/puppet-redis | 15:42 |
| IvanBerezovskiy | thanks | 15:42 |
| *** nealph_ has joined #openstack-ceilometer | 15:42 | |
| cdent | The stuff in sentinel.pp is not idiomatic... | 15:42 |
| IvanBerezovskiy | of course, I can setup tooz from repository, but I prefer to package it first :) | 15:43 |
| *** r-daneel has joined #openstack-ceilometer | 15:48 | |
| EmilienM | cdent: IvanBerezovskiy: I'll produce a pull-request today or early tomorrow. | 15:48 |
| openstackgerrit | Lena Novokshonova proposed openstack/ceilometer: Enable double-quote check for ceilometer project https://review.openstack.org/147211 | 15:48 |
| openstackgerrit | Srinivas Sakhamuri proposed openstack/python-ceilometerclient: Fix improper parameter setup for cacert and client certs https://review.openstack.org/146951 | 15:49 |
| IvanBerezovskiy | EmilienM, thanks. please add me to CR | 15:49 |
| EmilienM | IvanBerezovskiy: CR? | 15:50 |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Declarative HTTP testing for the Ceilometer API https://review.openstack.org/146187 | 15:50 |
| IvanBerezovskiy | EmilienM, code review | 15:50 |
| EmilienM | IvanBerezovskiy: it will be on github | 15:50 |
| EmilienM | IvanBerezovskiy: but ok, I'll let you know. | 15:50 |
| IvanBerezovskiy | thanks a lot | 15:50 |
| openstackgerrit | Lena Novokshonova proposed openstack/ceilometer: Enable double-quote check for ceilometer project https://review.openstack.org/147211 | 15:51 |
| *** _nadya_ has joined #openstack-ceilometer | 15:51 | |
| openstackgerrit | Lena Novokshonova proposed openstack/ceilometer: Check triple double quotes are used in docstrings https://review.openstack.org/147211 | 15:52 |
| *** nealph_ has quit IRC | 15:54 | |
| openstackgerrit | Lena Novokshonova proposed openstack/ceilometer: Check triple double quotes are used in docstrings https://review.openstack.org/147211 | 15:55 |
| *** nealph_ has joined #openstack-ceilometer | 15:56 | |
| nealph_ | info nealph | 15:57 |
| nealph_ | ha, wrong window. :) | 15:57 |
| pabelanger | greetings. Aside from horizon, and python-ceilometerclient what are people using for frontends to ceilometer? Was curious if there was any other projects (maybe web) available | 15:58 |
| *** nealph has quit IRC | 15:59 | |
| *** nealph has joined #openstack-ceilometer | 15:59 | |
| *** nealph has quit IRC | 15:59 | |
| *** Ala has quit IRC | 16:00 | |
| *** nealph_ has quit IRC | 16:00 | |
| *** nealph has joined #openstack-ceilometer | 16:00 | |
| *** nealph_ has joined #openstack-ceilometer | 16:00 | |
| *** nealph has quit IRC | 16:02 | |
| *** nealph_ is now known as nealph | 16:02 | |
| *** ddieterly has quit IRC | 16:02 | |
| *** nealph_ has joined #openstack-ceilometer | 16:02 | |
| *** fwdit_mp has joined #openstack-ceilometer | 16:03 | |
| *** ryanpetrello_ has joined #openstack-ceilometer | 16:04 | |
| *** ryanpetrello has quit IRC | 16:08 | |
| *** ryanpetrello_ is now known as ryanpetrello | 16:08 | |
| *** yeungp has quit IRC | 16:08 | |
| sileht | jd__, I have a question about 'The need_percent_of_overlap check should only happen if both from_timestamp and to_timestamp are set; otherwise an automatic discovery should happen.' | 16:09 |
| sileht | jd__, automatic discovery of what ? the need_percent_of_overlap ? | 16:09 |
| *** alexpilotti_ has joined #openstack-ceilometer | 16:10 | |
| *** alexpilotti has quit IRC | 16:12 | |
| *** alexpilotti_ is now known as alexpilotti | 16:12 | |
| *** deepthi has quit IRC | 16:22 | |
| jd__ | sileht: url? I don't recall | 16:25 |
| sileht | jd__, https://review.openstack.org/#/c/140638/5/gnocchi/carbonara.py | 16:26 |
| *** ildikov has quit IRC | 16:29 | |
| *** Longgeek has quit IRC | 16:29 | |
| *** shengjiemin has quit IRC | 16:32 | |
| *** IvanBerezovskiy has left #openstack-ceilometer | 16:49 | |
| jd__ | sileht: I mean automatic discovery of what overlaps | 16:52 |
| jd__ | guess the to and from based on something that gives 100% | 16:52 |
| sileht | jd__, I returns the last block of timestamp found is all timeseries ? | 16:53 |
| sileht | is/in | 16:55 |
| *** nealph_ has quit IRC | 16:55 | |
| *** _nadya_ has quit IRC | 16:59 | |
| sileht | I guess I got it | 17:00 |
| *** exploreshaifali has joined #openstack-ceilometer | 17:02 | |
| *** ddieterly has joined #openstack-ceilometer | 17:05 | |
| jd__ | sileht: yep | 17:05 |
| *** claudiub has quit IRC | 17:06 | |
| *** amalagon has quit IRC | 17:06 | |
| *** nealph_ has joined #openstack-ceilometer | 17:08 | |
| *** ildikov has joined #openstack-ceilometer | 17:08 | |
| *** Longgeek has joined #openstack-ceilometer | 17:10 | |
| *** Longgeek has quit IRC | 17:24 | |
| *** nealph_ has quit IRC | 17:25 | |
| *** nealph_ has joined #openstack-ceilometer | 17:25 | |
| *** rwsu has joined #openstack-ceilometer | 17:29 | |
| *** sbfox has joined #openstack-ceilometer | 17:30 | |
| *** yassine has quit IRC | 17:30 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:31 | |
| *** _cjones_ has quit IRC | 17:32 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:32 | |
| *** EmilienM is now known as EmilienM|afk | 17:32 | |
| *** safchain has quit IRC | 17:32 | |
| *** _cjones_ has joined #openstack-ceilometer | 17:36 | |
| *** amalagon has joined #openstack-ceilometer | 17:37 | |
| *** gordc has quit IRC | 17:38 | |
| *** _nadya_ has quit IRC | 17:44 | |
| *** claudiub has joined #openstack-ceilometer | 17:47 | |
| *** Longgeek has joined #openstack-ceilometer | 17:49 | |
| *** vissborg has quit IRC | 18:02 | |
| *** nealph_ has quit IRC | 18:02 | |
| *** vissborg has joined #openstack-ceilometer | 18:05 | |
| *** prad has joined #openstack-ceilometer | 18:10 | |
| *** harlowja_away is now known as harlowja | 18:10 | |
| *** srsakhamuri has joined #openstack-ceilometer | 18:12 | |
| *** ddieterly has quit IRC | 18:21 | |
| *** gordc has joined #openstack-ceilometer | 18:21 | |
| *** exploreshaifali has quit IRC | 18:24 | |
| srsakhamuri | @gordc, @zqfan, @eglynn can you please review https://review.openstack.org/#/c/146951/ and provide your feedback | 18:26 |
| *** EmilienM|afk is now known as EmilienM | 18:29 | |
| *** _cjones_ has quit IRC | 18:57 | |
| *** _cjones_ has joined #openstack-ceilometer | 19:01 | |
| *** nealph_ has joined #openstack-ceilometer | 19:03 | |
| *** cdent has quit IRC | 19:06 | |
| *** _cjones_ has quit IRC | 19:07 | |
| *** _cjones_ has joined #openstack-ceilometer | 19:08 | |
| *** nealph_ has quit IRC | 19:08 | |
| *** alexpilotti has quit IRC | 19:10 | |
| *** ddieterly has joined #openstack-ceilometer | 19:32 | |
| *** GeneralSalty has joined #openstack-ceilometer | 19:37 | |
| *** sbfox has quit IRC | 19:45 | |
| *** sbfox has joined #openstack-ceilometer | 19:47 | |
| *** ccrouch has joined #openstack-ceilometer | 19:51 | |
| gordc | srsakhamuri: i added a question | 19:56 |
| *** exploreshaifali has joined #openstack-ceilometer | 20:00 | |
| *** prad has quit IRC | 20:00 | |
| *** claudiub has quit IRC | 20:03 | |
| *** nealph_ has joined #openstack-ceilometer | 20:04 | |
| *** nealph_ has quit IRC | 20:08 | |
| *** david-lyle has joined #openstack-ceilometer | 20:10 | |
| *** _cjones_ has quit IRC | 20:11 | |
| *** claudiub has joined #openstack-ceilometer | 20:15 | |
| srsakhamuri | gordc: thanks for looking. I added my response. | 20:17 |
| *** MVenesio has joined #openstack-ceilometer | 20:23 | |
| MVenesio | Hi guys, i'm trying to collect memory metrics with ceilometer in every compute and i could't do that | 20:29 |
| *** ddieterly has quit IRC | 20:29 | |
| *** harlowja is now known as harlowja_away | 20:30 | |
| *** _cjones_ has joined #openstack-ceilometer | 20:30 | |
| MVenesio | i have the instance_usage_audit and all the ceilometer settings in nova.conf and libvirt 1.2 quemu 1.6 and the balloon driver enabled | 20:30 |
| MVenesio | do you know if the memory metrics are supported in juno ? | 20:31 |
| *** ddieterly has joined #openstack-ceilometer | 20:31 | |
| srsakhamuri | MVenesio: I see this note in http://docs.openstack.org/developer/ceilometer/measurements.html. "Note To enable the libvirt memory.usage supporting, you need libvirt version 1.1.1+, qemu version 1.5+, and you need to prepare suitable balloon driver in the image, particularly for Windows guests, most modern Linuxes have it built in. The memory.usage meters can’t be fetched without image balloon driver. | 20:33 |
| srsakhamuri | " | 20:33 |
| srsakhamuri | oops sorry looks like you have the correct versions. | 20:34 |
| MVenesio | srsakhamuri: yes i have newer versions of all, do you have the memory metrics running ? | 20:35 |
| *** harlowja_away is now known as harlowja | 20:38 | |
| srsakhamuri | Actually we have disabled at present, looking at the old data we seem to have received "memory" measurement from nova | 20:39 |
| EmilienM | eglynn: https://github.com/arioch/puppet-redis/pull/19 | 20:40 |
| eglynn | EmilienM: excellent, thank you sir! | 20:41 |
| eglynn | EmilienM: ... I'll verify the patch in an staypuft/astapor deployment first thing tmrw | 20:41 |
| MVenesio | srsakhamuri: mmmm could you send me the nova.conf and ceilometer conf that you where using in your compute nodes | 20:41 |
| MVenesio | srsakhamuri: maybe is a configuration issue | 20:42 |
| EmilienM | eglynn: not tested though | 20:42 |
| EmilienM | cdent will I thikn | 20:42 |
| eglynn | EmilienM: cool ... so the basic idea is to dispense with sentinel.pp and move all the sentinel logic into init.pp, amiright? | 20:43 |
| EmilienM | eglynn: yes | 20:45 |
| eglynn | cool, thanks ... /me states the obvious :) | 20:46 |
| srsakhamuri | MVenesio: I don't think I have old config with me, but check your pipeline.yaml. Here is more information about that: https://openstack.redhat.com/CeilometerQuickStart | 20:47 |
| MVenesio | srsakhamuri: i'll check that thanks | 20:48 |
| *** promulo_ has joined #openstack-ceilometer | 20:52 | |
| *** david-lyle has quit IRC | 21:01 | |
| *** srsakhamuri has quit IRC | 21:04 | |
| *** _cjones_ has quit IRC | 21:10 | |
| *** _cjones_ has joined #openstack-ceilometer | 21:13 | |
| *** julim has quit IRC | 21:27 | |
| *** exploreshaifali has quit IRC | 21:38 | |
| *** exploreshaifali has joined #openstack-ceilometer | 21:39 | |
| *** david-lyle has joined #openstack-ceilometer | 21:42 | |
| *** MVenesio has quit IRC | 21:47 | |
| *** david-lyle has quit IRC | 21:49 | |
| *** r-daneel has quit IRC | 21:53 | |
| *** nellysmitt has quit IRC | 21:59 | |
| *** nealph_ has joined #openstack-ceilometer | 22:03 | |
| *** nealph_ has quit IRC | 22:03 | |
| *** nealph_ has joined #openstack-ceilometer | 22:04 | |
| *** nealph_ has quit IRC | 22:05 | |
| *** nealph is now known as nealph_afk | 22:05 | |
| *** nealph_afk is now known as nealph | 22:06 | |
| *** nealph has quit IRC | 22:06 | |
| *** _cjones_ has quit IRC | 22:11 | |
| *** _cjones_ has joined #openstack-ceilometer | 22:13 | |
| *** EmilienM is now known as EmilienM|afk | 22:32 | |
| *** david-lyle has joined #openstack-ceilometer | 22:33 | |
| *** gordc has quit IRC | 22:39 | |
| *** ryanpetrello has quit IRC | 23:02 | |
| *** ryanpetrello has joined #openstack-ceilometer | 23:03 | |
| *** _cjones_ has quit IRC | 23:09 | |
| *** _cjones_ has joined #openstack-ceilometer | 23:11 | |
| *** david-lyle has quit IRC | 23:16 | |
| *** _cjones_ has quit IRC | 23:18 | |
| *** _cjones_ has joined #openstack-ceilometer | 23:25 | |
| *** _cjones_ has quit IRC | 23:33 | |
| *** exploreshaifali has quit IRC | 23:36 | |
| *** _cjones_ has joined #openstack-ceilometer | 23:40 | |
| *** ddieterly has quit IRC | 23:46 | |
| *** ryanpetrello has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!