| *** herndon has quit IRC | 00:03 | |
| *** yfujioka has joined #openstack-ceilometer | 00:16 | |
| openstackgerrit | Jenkins proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/60978 | 00:19 |
|---|---|---|
| *** herndon has joined #openstack-ceilometer | 00:41 | |
| *** ityaptin has quit IRC | 00:46 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: fix unit https://review.openstack.org/60987 | 00:48 |
| *** ityaptin has joined #openstack-ceilometer | 00:48 | |
| *** openstack has joined #openstack-ceilometer | 00:49 | |
| *** asalkeld has quit IRC | 00:57 | |
| *** asalkeld has joined #openstack-ceilometer | 00:57 | |
| *** djbkd has quit IRC | 01:17 | |
| *** asalkeld is now known as angus_is_eating | 02:14 | |
| *** angus_is_eating is now known as asalkeld | 03:14 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: add newly added constraints to expire clear_expired_metering_data https://review.openstack.org/60995 | 03:19 |
| *** herndon has quit IRC | 04:15 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 04:38 | |
| *** sayali has joined #openstack-ceilometer | 05:51 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/60154 | 06:05 |
| openstackgerrit | Shuangtai Tian proposed a change to openstack/ceilometer: Handle the metrics sent by nova notifier https://review.openstack.org/42838 | 06:06 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 06:10 | |
| *** _SergeyLukjanov has quit IRC | 06:10 | |
| *** eglynn-afk has joined #openstack-ceilometer | 06:53 | |
| *** eglynn-afk has quit IRC | 06:56 | |
| *** sayali has quit IRC | 07:22 | |
| *** urulama has joined #openstack-ceilometer | 07:27 | |
| *** eglynn-afk has joined #openstack-ceilometer | 07:32 | |
| *** eglynn-afk has quit IRC | 07:41 | |
| *** leegayeon has joined #openstack-ceilometer | 07:42 | |
| *** sayali has joined #openstack-ceilometer | 07:47 | |
| *** ildikov has joined #openstack-ceilometer | 07:50 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 07:53 | |
| *** eglynn-afk has joined #openstack-ceilometer | 07:59 | |
| *** yfujioka has quit IRC | 08:11 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Implement the /v2/sample API https://review.openstack.org/60540 | 08:22 |
| eglynn-afk | insanidade: that looks like a simple docco bug ... | 08:26 |
| *** eglynn-afk is now known as eglynn | 08:26 | |
| eglynn | insanidade: they seem to have conflated one the ceilometer service names (openstack-ceilometer-central) | 08:26 |
| eglynn | insanidade: ... with one of the cinder service name (openstack-cinder-volume) | 08:26 |
| eglynn | insanidade: ... and produced an ugly child that doesn't exist (openstack-cinder-agent-central) | 08:27 |
| eglynn | insanidade: ... so I think you just need a change the restart command line like so: s/openstack-cinder-agent-central/openstack-cinder-volume/ | 08:28 |
| eglynn | insanidade: I've raised a docco bug https://bugs.launchpad.net/openstack-manuals/+bug/1259456 | 08:37 |
| eglynn | insanidade: please tick the "This bug affects you" box in the launchpad link above | 08:38 |
| *** sayali has quit IRC | 08:49 | |
| *** sayali has joined #openstack-ceilometer | 09:10 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 09:34 | |
| *** _SergeyLukjanov has quit IRC | 09:34 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 09:45 | |
| *** eglynn is now known as eglynn-afk | 09:53 | |
| *** nprivalova has joined #openstack-ceilometer | 09:55 | |
| *** sayali has quit IRC | 10:11 | |
| *** sayali has joined #openstack-ceilometer | 10:33 | |
| *** kwhitney has quit IRC | 10:36 | |
| *** kwhitney has joined #openstack-ceilometer | 10:53 | |
| *** leegayeon has quit IRC | 11:14 | |
| *** ildikov has quit IRC | 11:22 | |
| *** ildikov has joined #openstack-ceilometer | 11:22 | |
| *** eglynn-afk has quit IRC | 12:00 | |
| *** sayali has quit IRC | 12:17 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 12:21 | |
| *** _SergeyLukjanov has quit IRC | 12:21 | |
| *** sayali has joined #openstack-ceilometer | 12:23 | |
| *** eglynn-afk has joined #openstack-ceilometer | 12:24 | |
| insanidade | eglynn-afk: thank. just did that :) | 12:25 |
| *** sayali has quit IRC | 12:32 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 12:38 | |
| *** sayali has joined #openstack-ceilometer | 12:46 | |
| *** ildikov has quit IRC | 13:00 | |
| *** nprivalova has quit IRC | 13:02 | |
| *** ildikov has joined #openstack-ceilometer | 13:05 | |
| *** sandywalsh_ has joined #openstack-ceilometer | 13:06 | |
| *** nprivalova has joined #openstack-ceilometer | 13:12 | |
| *** gordc has joined #openstack-ceilometer | 13:24 | |
| *** thomasem has joined #openstack-ceilometer | 13:27 | |
| *** sandywalsh__ has joined #openstack-ceilometer | 13:29 | |
| *** sandywalsh_ has quit IRC | 13:32 | |
| *** sayali has quit IRC | 13:35 | |
| *** openstackgerrit has quit IRC | 13:37 | |
| *** openstackgerrit has joined #openstack-ceilometer | 13:37 | |
| *** sayali has joined #openstack-ceilometer | 13:38 | |
| *** jdob has joined #openstack-ceilometer | 13:43 | |
| *** suo has quit IRC | 14:07 | |
| *** sandywalsh_ has joined #openstack-ceilometer | 14:08 | |
| *** sandywalsh__ has quit IRC | 14:08 | |
| *** suo has joined #openstack-ceilometer | 14:09 | |
| *** jdob has quit IRC | 14:10 | |
| *** litong has joined #openstack-ceilometer | 14:13 | |
| *** jd__ has quit IRC | 14:13 | |
| *** jd__ has joined #openstack-ceilometer | 14:13 | |
| litong | hi, guys, can any of you please take a look at this https://review.openstack.org/#/c/49802/ | 14:14 |
| litong | it has been approved but since it took so long for reviewer to approve it, it had to be rebased. | 14:14 |
| litong | now it seems going to happen again. any helping soul there please review it. it only takes few minutes. | 14:15 |
| thomasem | litong, I'll have a look to | 14:22 |
| litong | @thomasem, thanks. | 14:22 |
| litong | it was approved actually, just needed to be rebased. | 14:22 |
| litong | which I did. | 14:22 |
| litong | it was all for making the test coverage percentage a bit higher. | 14:23 |
| *** jdob has joined #openstack-ceilometer | 14:23 | |
| thomasem | litong, Gotcha. I remember those patches you put up. Thanks for that, we definitely need more coverage! :) | 14:25 |
| sileht | litong, done | 14:25 |
| litong | @sileht, thanks so much. @thomasem, thanks a lot. | 14:25 |
| nprivalova | litong, I'm looking into it | 14:29 |
| litong | @nprivalova, appreciate it. | 14:29 |
| *** sandywalsh__ has joined #openstack-ceilometer | 14:30 | |
| *** sandywalsh_ has quit IRC | 14:33 | |
| thomasem | litong, just noticed a double underscore in one of the test names. | 14:36 |
| litong | yes, that is because the method name the code was trying to test starts with underscore. | 14:36 |
| thomasem | Oh, I'm sorry | 14:36 |
| litong | private method. | 14:37 |
| thomasem | litong, :) Well, nonetheless I still +1'ed | 14:37 |
| thomasem | Just mentioned a follow up branch. Sorry about that, hehe. | 14:37 |
| litong | @oh, ok, thanks, well, I will comment so everybody know if someone really cares. | 14:38 |
| litong | actually that was one of the comments in the earlier patch sets. | 14:38 |
| thomasem | Ahhhh, I see. | 14:39 |
| litong | thanks guys. | 14:40 |
| thomasem | thank you also! | 14:40 |
| litong | really need one more core to get it merged. | 14:41 |
| nprivalova | sorry for my 1 comment :) | 14:41 |
| *** sandywalsh_ has joined #openstack-ceilometer | 14:43 | |
| *** sandywalsh__ has quit IRC | 14:44 | |
| *** sandywalsh__ has joined #openstack-ceilometer | 15:01 | |
| *** sandywalsh_ has quit IRC | 15:01 | |
| *** prad has joined #openstack-ceilometer | 15:01 | |
| *** sayali_ has joined #openstack-ceilometer | 15:05 | |
| *** sayali has quit IRC | 15:09 | |
| litong | @nprivalova, please take a look at the response to your comment, your assumption was not correct. | 15:12 |
| nprivalova | litong, ok, 1 sec | 15:12 |
| *** jdob_ has joined #openstack-ceilometer | 15:13 | |
| *** jdob has quit IRC | 15:13 | |
| nprivalova | litong, done :) | 15:17 |
| litong | @nprivalova, good deal. | 15:18 |
| *** jdob_ has quit IRC | 15:29 | |
| *** jdob has joined #openstack-ceilometer | 15:29 | |
| *** sandywalsh__ has quit IRC | 15:31 | |
| *** herndon has joined #openstack-ceilometer | 15:42 | |
| *** SergeyLukjanov has quit IRC | 15:55 | |
| *** urulama has quit IRC | 15:55 | |
| openstackgerrit | A change was merged to openstack/ceilometer: fix unit https://review.openstack.org/60987 | 16:14 |
| *** djbkd has joined #openstack-ceilometer | 16:17 | |
| *** apmelton1 has joined #openstack-ceilometer | 16:18 | |
| *** ildikov has quit IRC | 16:37 | |
| *** nprivalova has quit IRC | 16:40 | |
| *** prad has quit IRC | 16:40 | |
| *** sayali_ has quit IRC | 16:57 | |
| *** prad has joined #openstack-ceilometer | 16:59 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 17:10 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 17:11 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 17:11 | |
| *** terriyu has joined #openstack-ceilometer | 17:20 | |
| *** insanidade has quit IRC | 17:20 | |
| *** sandywalsh has quit IRC | 17:33 | |
| openstackgerrit | Monsyne Dragon proposed a change to openstack/ceilometer: Add configuration-driven conversion to Events https://review.openstack.org/42713 | 17:34 |
| *** sandywalsh_ has joined #openstack-ceilometer | 17:36 | |
| dragondm | gordc: eglynn-afk: I put up a new patchset with the conf group changes. Thanks for taking time for the reviews! | 17:40 |
| eglynn-afk | dragondm: thank you sir, I'll give the patch another pass this evening | 17:41 |
| eglynn-afk | dragondm: actually, I had one other question ... | 17:41 |
| *** eglynn-afk is now known as eglynn | 17:41 | |
| dragondm | Ok, fire away... | 17:42 |
| eglynn | (slight conflict of interest here, so clearly wearing my Red Hat hat for a second, if you know what I mean) | 17:42 |
| eglynn | just wondering about the variety of implementations of JSONPath and the packaging of same on the major distros | 17:42 |
| eglynn | AFAICS jsonpath-rw isn't widely packaged | 17:43 |
| eglynn | (though admitedly I didn't look very hard ...) | 17:43 |
| eglynn | to be clear, that would NOT be a valid reason to delay your patch | 17:43 |
| eglynn | ... just wondering if you'd any insight though on the state of the art there | 17:43 |
| dragondm | It's packaged on PyPI, and the project is updated reasonably well. Myself and doug looked at it when I added it to the openstack-requirements list. | 17:44 |
| dragondm | Don't know much on platform-specific packaging. It's pure python, tho, so it should be fairly simple, packaging-wise. | 17:45 |
| eglynn | sure, the pypi stuff will suffice for you patch to pass CI and land on master etc, but I was thinking more of distro packaging | 17:46 |
| eglynn | i.e. a python-jsonpath-rw RPM | 17:46 |
| eglynn | or the analoug on Deb & unbuntu | 17:47 |
| eglynn | ... not your concern though, or responsibility to ensure that happens | 17:47 |
| eglynn | ... just throwing the question out there in case you knew of any efforts to distro-package this library | 17:47 |
| dragondm | Not offhand. | 17:48 |
| dragondm | It is a pure python library, so I don't think it's be too difficult. (But I know enough of packaging detail-devils to not be to sure of that :-> ) | 17:50 |
| eglynn | yeah, it's probably trivial exercise once someone jumps on it | 17:51 |
| dragondm | Probably worthwhile anyway. It's a damn handy library to have if you're rooting through complex JSON. | 17:52 |
| eglynn | yep, I'll unleash the Red Hat packaging hounds on it :) | 17:52 |
| eglynn | they can usually knock out RPMs for the entire food-chain (Fedora, EPEL, RHEL) pretty darn quick | 17:52 |
| eglynn | ... as long as there aren't odd transitive deps | 17:52 |
| dragondm | Yah, tho it's only deps are ply, and six. (and I think decorator) should be simple. | 18:00 |
| dragondm | the only semi complex one there is ply | 18:00 |
| *** eglynn has quit IRC | 18:08 | |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Add new meters for swift https://review.openstack.org/56019 | 18:24 |
| *** Alexei_987 has quit IRC | 18:43 | |
| *** eglynn has joined #openstack-ceilometer | 18:45 | |
| *** gordc1 has joined #openstack-ceilometer | 18:49 | |
| *** gordc has quit IRC | 18:54 | |
| *** gordc1 has quit IRC | 18:55 | |
| *** viktors1 has joined #openstack-ceilometer | 18:55 | |
| *** viktors1 has left #openstack-ceilometer | 18:55 | |
| openstackgerrit | Vladislav Kuzmin proposed a change to openstack/ceilometer: Add new meters for swift https://review.openstack.org/56019 | 18:56 |
| *** Alexei_987 has joined #openstack-ceilometer | 19:37 | |
| *** eglynn has quit IRC | 19:50 | |
| *** lakspace has joined #openstack-ceilometer | 20:16 | |
| *** eglynn has joined #openstack-ceilometer | 20:17 | |
| lakspace | Just to add to our conversation over the phone about Cassandra as a possible data store: https://github.com/pycassa/pycassa | 20:19 |
| *** gordc has joined #openstack-ceilometer | 20:29 | |
| *** gordc has quit IRC | 21:02 | |
| *** gordc has joined #openstack-ceilometer | 21:03 | |
| *** terriyu has quit IRC | 21:20 | |
| *** herndon has quit IRC | 21:33 | |
| *** eglynn has quit IRC | 21:38 | |
| *** jdob has quit IRC | 21:52 | |
| *** litong has quit IRC | 22:23 | |
| *** eglynn has joined #openstack-ceilometer | 22:38 | |
| *** Alexei_987 has quit IRC | 22:38 | |
| *** SergeyLukjanov has quit IRC | 22:41 | |
| *** herndon has joined #openstack-ceilometer | 22:44 | |
| *** thomasem has quit IRC | 22:58 | |
| *** eglynn has quit IRC | 22:59 | |
| *** gordc has quit IRC | 23:11 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/61275 | 23:40 |
| openstackgerrit | Jenkins proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/60978 | 23:45 |
| *** prad has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!