*** _nadya_ has joined #openstack-ceilometer | 00:03 | |
*** _nadya_ has quit IRC | 00:07 | |
*** nati_ueno has quit IRC | 00:14 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 00:41 |
---|---|---|
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/pycadf: Updated from global requirements https://review.openstack.org/99091 | 00:47 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/99567 | 00:47 |
*** rwsu has quit IRC | 00:58 | |
*** fnaval has quit IRC | 01:09 | |
*** prad_ has quit IRC | 01:09 | |
*** fnaval has joined #openstack-ceilometer | 01:10 | |
*** flwang has quit IRC | 01:10 | |
*** matsuhashi has joined #openstack-ceilometer | 01:17 | |
openstackgerrit | lawrancejing proposed a change to openstack/ceilometer: Change the unit of hardware.network.bandwidth.bytes https://review.openstack.org/100595 | 01:22 |
*** _cjones_ has quit IRC | 01:31 | |
*** nosnos has joined #openstack-ceilometer | 01:31 | |
*** _cjones_ has joined #openstack-ceilometer | 01:31 | |
*** _cjones_ has quit IRC | 01:35 | |
*** yjiang5 is now known as yjiang5_away | 01:43 | |
openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Avoid reading real config files in unit test https://review.openstack.org/100054 | 01:52 |
*** _cjones_ has joined #openstack-ceilometer | 02:03 | |
*** zqfan_bot is now known as zqfan | 02:05 | |
*** _cjones_ has quit IRC | 02:13 | |
*** matsuhashi has quit IRC | 02:29 | |
*** matsuhashi has joined #openstack-ceilometer | 02:32 | |
*** Alice911 has joined #openstack-ceilometer | 02:47 | |
*** harlowja is now known as harlowja_away | 02:48 | |
openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Clean up oslo.middleware.{audit,notifier} https://review.openstack.org/99246 | 02:53 |
*** renlt has joined #openstack-ceilometer | 02:53 | |
openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Fixed various import issues exposed by unittest https://review.openstack.org/99280 | 03:00 |
*** matsuhashi has quit IRC | 03:00 | |
*** fnaval_ has joined #openstack-ceilometer | 03:06 | |
*** fnaval has quit IRC | 03:09 | |
openstackgerrit | liusheng proposed a change to openstack/ceilometer-specs: Specs for add-alarm-severity https://review.openstack.org/100752 | 03:18 |
*** changbl has joined #openstack-ceilometer | 03:19 | |
*** nosnos has quit IRC | 03:27 | |
*** yatin has joined #openstack-ceilometer | 03:35 | |
*** matsuhashi has joined #openstack-ceilometer | 03:53 | |
*** _cjones_ has joined #openstack-ceilometer | 04:04 | |
*** nosnos has joined #openstack-ceilometer | 04:07 | |
*** _cjones_ has quit IRC | 04:08 | |
*** matsuhashi has quit IRC | 04:12 | |
*** matsuhashi has joined #openstack-ceilometer | 04:12 | |
*** matsuhashi has quit IRC | 04:17 | |
openstackgerrit | ZhiQiang Fan proposed a change to openstack/ceilometer: Avoid reading real config files in unit test https://review.openstack.org/100054 | 04:18 |
*** matsuhashi has joined #openstack-ceilometer | 04:21 | |
*** r3pl4y has joined #openstack-ceilometer | 04:25 | |
*** Longgeek has joined #openstack-ceilometer | 04:26 | |
*** fnaval_ has quit IRC | 04:32 | |
*** matsuhashi has quit IRC | 04:36 | |
*** matsuhashi has joined #openstack-ceilometer | 04:36 | |
*** matsuhashi has quit IRC | 04:39 | |
*** matsuhashi has joined #openstack-ceilometer | 04:39 | |
*** r3pl4y has quit IRC | 04:41 | |
*** psharma has joined #openstack-ceilometer | 04:45 | |
*** piyushmasrani has joined #openstack-ceilometer | 04:50 | |
*** ildikov has quit IRC | 05:09 | |
*** matsuhashi has quit IRC | 05:11 | |
*** matsuhashi has joined #openstack-ceilometer | 05:11 | |
*** matsuhashi has quit IRC | 05:16 | |
*** matsuhashi has joined #openstack-ceilometer | 05:17 | |
*** inc0 has joined #openstack-ceilometer | 05:20 | |
*** Underyx|off is now known as underyx | 05:21 | |
*** matsuhashi has quit IRC | 05:24 | |
*** matsuhashi has joined #openstack-ceilometer | 05:24 | |
*** underyx is now known as Underyx|off | 05:32 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/88506 | 06:01 |
*** manoj has joined #openstack-ceilometer | 06:04 | |
manoj | hi all | 06:04 |
*** dragondm has quit IRC | 06:05 | |
*** _cjones_ has joined #openstack-ceilometer | 06:05 | |
manoj | Is there any option in openstack telemetry to display the lables(units and time) in x-axis and y-axis of statistics graph | 06:06 |
*** dragondm has joined #openstack-ceilometer | 06:08 | |
*** _cjones_ has quit IRC | 06:10 | |
openstackgerrit | Chinmaya Bharadwaj proposed a change to openstack/ceilometer: Fixes ceilometer-compute service start failure https://review.openstack.org/97831 | 06:18 |
*** psharma has quit IRC | 06:19 | |
*** openstackgerrit has quit IRC | 06:19 | |
*** psharma has joined #openstack-ceilometer | 06:23 | |
*** openstackgerrit has joined #openstack-ceilometer | 06:23 | |
openstackgerrit | Chinmaya Bharadwaj proposed a change to openstack/ceilometer: VMware:Update the ceilometer doc with VMware opts https://review.openstack.org/98132 | 06:26 |
*** _nadya_ has joined #openstack-ceilometer | 06:29 | |
*** ildikov has joined #openstack-ceilometer | 06:29 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Disable testrepository log capture https://review.openstack.org/100631 | 06:32 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Don't use oslo.messaging wait method https://review.openstack.org/100780 | 06:32 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Don't use oslo.messaging wait method https://review.openstack.org/100780 | 06:34 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Disable testrepository log capture https://review.openstack.org/100631 | 06:34 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Don't use oslo.messaging wait method https://review.openstack.org/100780 | 06:37 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Disable testrepository log capture https://review.openstack.org/100631 | 06:37 |
*** Ruetobas has quit IRC | 06:45 | |
manoj | Is there any option in openstack telemetry to display the lables(units and time) in x-axis and y-axis of statistics graph | 06:50 |
manoj | ildikov: hi | 06:51 |
*** Ruetobas has joined #openstack-ceilometer | 06:51 | |
ildikov | manoj: hi | 06:51 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Reenables the testr per test timeout https://review.openstack.org/100789 | 06:51 |
manoj | ildikov: Is there any option in openstack telemetry to display the lables(units and time) in x-axis and y-axis of statistics graph | 06:51 |
ildikov | manoj: do you mean in Horizon? | 06:52 |
manoj | ildikov: yes | 06:52 |
ildikov | manoj: good question :) | 06:52 |
ildikov | manoj: when I last saw the diagrams in Horizon they were not configurable IIRC | 06:53 |
manoj | ildikov:ok | 06:53 |
ildikov | manoj: it was late Havana or early Icehouse I guess, so maybe it has changed since then | 06:54 |
ildikov | manoj: you can ask the Horizon folks, if it is not clear now that it can be set or not | 06:54 |
manoj | ildikov: yeah sure, thanks | 06:55 |
*** Ruetobas has quit IRC | 06:55 | |
*** renlt has quit IRC | 06:58 | |
*** Ruetobas has joined #openstack-ceilometer | 07:01 | |
*** matsuhashi has quit IRC | 07:08 | |
*** matsuhas_ has joined #openstack-ceilometer | 07:08 | |
*** r3pl4y has joined #openstack-ceilometer | 07:10 | |
*** renlt has joined #openstack-ceilometer | 07:13 | |
*** Underyx|off is now known as underyx | 07:13 | |
*** r3pl4y has quit IRC | 07:19 | |
*** r3pl4y has joined #openstack-ceilometer | 07:20 | |
*** nacim has joined #openstack-ceilometer | 07:50 | |
*** nacim has quit IRC | 07:55 | |
*** nacim has joined #openstack-ceilometer | 08:00 | |
*** Ruetobas has quit IRC | 08:00 | |
*** safchain has joined #openstack-ceilometer | 08:02 | |
openstackgerrit | A change was merged to openstack/ceilometer: Clean up oslo.middleware.{audit,notifier} https://review.openstack.org/99246 | 08:04 |
*** renlt has quit IRC | 08:06 | |
*** renlt has joined #openstack-ceilometer | 08:06 | |
*** Ruetobas has joined #openstack-ceilometer | 08:08 | |
openstackgerrit | lawrancejing proposed a change to openstack/ceilometer: Change the unit of hardware.network.bandwidth.bytes https://review.openstack.org/100595 | 08:08 |
*** renlt has quit IRC | 08:10 | |
*** renlt has joined #openstack-ceilometer | 08:11 | |
*** renlt has quit IRC | 08:11 | |
*** renlt has joined #openstack-ceilometer | 08:11 | |
*** Ruetobas has quit IRC | 08:12 | |
*** Ruetobas has joined #openstack-ceilometer | 08:19 | |
*** r3pl4y has quit IRC | 08:21 | |
*** renlt has quit IRC | 08:25 | |
*** renlt has joined #openstack-ceilometer | 08:26 | |
*** renlt has quit IRC | 08:27 | |
*** renlt has joined #openstack-ceilometer | 08:28 | |
*** eglynn has joined #openstack-ceilometer | 08:28 | |
*** renlt has quit IRC | 08:32 | |
*** renlt has joined #openstack-ceilometer | 08:32 | |
*** IvanBerezovskiy has joined #openstack-ceilometer | 08:41 | |
*** r3pl4y has joined #openstack-ceilometer | 08:45 | |
*** Longgeek has quit IRC | 08:46 | |
*** Longgeek has joined #openstack-ceilometer | 08:46 | |
*** Longgeek has quit IRC | 08:48 | |
*** Longgeek has joined #openstack-ceilometer | 08:49 | |
*** Longgeek has quit IRC | 08:53 | |
*** Longgeek_ has joined #openstack-ceilometer | 08:53 | |
*** matsuhas_ has quit IRC | 09:01 | |
*** matsuhashi has joined #openstack-ceilometer | 09:09 | |
*** Alice911 has quit IRC | 09:09 | |
*** isviridov|away is now known as isviridov | 09:13 | |
openstackgerrit | A change was merged to openstack/ceilometer: Splits mongo storage code base https://review.openstack.org/99138 | 09:17 |
openstackgerrit | Stefano Zilli proposed a change to openstack/ceilometer: Parametrize table_prefix_separator in hbase https://review.openstack.org/100829 | 09:17 |
*** ilyashakhat has joined #openstack-ceilometer | 09:27 | |
*** yatin has quit IRC | 09:32 | |
*** matsuhashi has quit IRC | 09:38 | |
*** renlt has quit IRC | 09:48 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Reenables the testr per test timeout https://review.openstack.org/100789 | 09:49 |
eglynn | DinaBelova: FYI some chatter on the os-infra ML related to enabling mongodb-based coverage for ceilo on f20 | 09:49 |
eglynn | ... http://lists.openstack.org/pipermail/openstack-infra/2014-June/001368.html | 09:49 |
DinaBelova | eglynn, yep | 09:50 |
* DinaBelova reading it right now | 09:50 | |
eglynn | sileht: just to clarify, https://review.openstack.org/100789 is intended more to limit the damage (in terms of holding a gate node for an hour) | 09:51 |
DinaBelova | eglynn, btw - tomorrow I'mm be not in my city to bring documents myself to the French embassy | 09:51 |
DinaBelova | so I'm not sure I'll take part in the tomorrow meeting | 09:51 |
eglynn | DinaBelova: np! | 09:52 |
DinaBelova | eglynn, so I'd love to say you now that ityaptin and I have sevaral ideas about performance tests automation | 09:52 |
DinaBelova | eglynn - as currently infra folks are working on the multi-node Jenkins envs | 09:52 |
DinaBelova | there are some results already | 09:52 |
DinaBelova | so we'll have the opportunity to create Jenkins post-milestone-cut job, for instance | 09:53 |
eglynn | DinaBelova: great! :) | 09:53 |
*** ccrouch1 has joined #openstack-ceilometer | 09:53 | |
DinaBelova | that'll run these tests on multinode devstack with multinode storages | 09:53 |
DinaBelova | as said - there's no reason to test all-in-one storage installations :) | 09:54 |
*** ccrouch has quit IRC | 09:54 | |
eglynn | DinaBelova: ... should we take an item on the agenda for next week's meeting to let folks on the team know more about this? | 09:54 |
DinaBelova | eglynn, I guess it's really cool idea | 09:55 |
DinaBelova | eglynn to this time w'll have the research results of what's the current status for it | 09:55 |
DinaBelova | and what steps we need to do to reach this point :) | 09:55 |
eglynn | DinaBelova: excellent, I'll add that to the agenda for the 26th so | 09:55 |
DinaBelova | eglynn, thank you sir | 09:55 |
DinaBelova | :) | 09:55 |
eglynn | np! | 09:55 |
eglynn | ... thank *you*! | 09:56 |
sileht | eglynn, yes and to get which are the tests that block | 09:58 |
eglynn | sileht: a-ha, cool | 09:59 |
eglynn | sileht: ... I've already +2'd the patch in any case | 09:59 |
eglynn | sileht: ... let's get that puppy landed quickly if poss | 09:59 |
*** hoangdo has joined #openstack-ceilometer | 10:00 | |
*** hoangdo has left #openstack-ceilometer | 10:00 | |
*** underyx is now known as Underyx|off | 10:00 | |
sileht | eglynn, the issue has occurred only 1 times since yesterday, the job is in a better health | 10:01 |
eglynn | jd__, ildikov, llu, _nadya_: if any of you guys have a spare minute to do a quick review on a moderately urgent patch: https://review.openstack.org/100789 | 10:01 |
*** matsuhashi has joined #openstack-ceilometer | 10:01 | |
eglynn | sileht: great, that's a definite improvement | 10:01 |
*** manoj has quit IRC | 10:02 | |
openstackgerrit | A change was merged to openstack/ceilometer: Fix method mocked in a test https://review.openstack.org/99890 | 10:03 |
ildikov | sileht: eglynn: I've just wanted ask yesterday that what about adding back this timeout, but then I git interrupted with smth else | 10:03 |
eglynn | ildikov, _nadya_: thanks for the quick response! :) | 10:04 |
ildikov | sileht: eglynn: the gate in general looked to be in a better shape yesterday, so I think that had a good influence on our py26/py27 jobs too | 10:04 |
sileht | eglynn, last time I have got the issue I have found that even we have the eventlet.sleep() here: 'http://git.openstack.org/cgit/openstack/ceilometer/tree/ceilometer/tests/alarm/test_rpc.py#n176' the fanout queue can be not created, if this sleep() return but oslo.messaging eventlet thread have not been scheduled. | 10:04 |
sileht | This is my thought about the gate log here: http://logs.openstack.org/31/100631/4/check/gate-ceilometer-python27/69385b6/console.html | 10:05 |
eglynn | ildikov: yeah the py27 failure rate is at least off the 100% mark since tuesday 8am ... http://jogo.github.io/gate/ | 10:06 |
*** matsuhashi has quit IRC | 10:06 | |
ildikov | eglynn: yeap, looks much better | 10:07 |
eglynn | sileht: so that is eventlet.sleep() in the test is insufficient to ensure that oslo-messaging has done its thing? | 10:07 |
eglynn | sileht: ... is there anything else that could be used as a sentinel in the test? | 10:08 |
*** matsuhashi has joined #openstack-ceilometer | 10:08 | |
eglynn | sileht: ... i.e. to wait upon, to ensure that the o-m thread has been actually scheduled become continueing? | 10:09 |
sileht | sileht, I have no idea how to make a sentinel here | 10:09 |
*** _nadya_ has quit IRC | 10:09 | |
* eglynn would hate to suggest something dumb like multiple calls to sleep() ... | 10:10 | |
eglynn | sileht: ^^^ would such craziness have any potential? | 10:11 |
DinaBelova | eglynn, sileht, btw - when we are getting alarm history, the newest are the first ones in this dict? | 10:11 |
eglynn | DinaBelova: yep | 10:11 |
DinaBelova | hehe, so I just need to understand why our tests sometimes do the opposite thing and are failing ;d | 10:11 |
eglynn | DinaBelova: sorry, I meant nope :) | 10:11 |
DinaBelova | :D | 10:11 |
DinaBelova | heh | 10:12 |
DinaBelova | so the thing is opposite? | 10:12 |
eglynn | DinaBelova: didn't read you question properly, it's time-ordering IIRC | 10:12 |
DinaBelova | eglynn, okay, so it should be like creation -> rule change -> ... | 10:12 |
eglynn | DinaBelova: exactly | 10:13 |
sileht | eglynn, lets see how many times is happend now, before writing more eventlet crap. | 10:13 |
eglynn | sileht: fair nuffski, makes sense, thanks! | 10:13 |
DinaBelova | eglynn, hehe, so https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/api/v2/test_alarm_scenarios.py#L1769 and then https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/api/v2/test_alarm_scenarios.py#L1775 look quite suspicious :D at least for me :D | 10:14 |
eglynn | DinaBelova: hmmm | 10:15 |
DinaBelova | when it's checked that 'rule change' is first and 'creation' second | 10:15 |
eglynn | DinaBelova: https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/impl_sqlalchemy.py#L797 | 10:15 |
eglynn | DinaBelova: desc timestamp order in sql-a ... https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/impl_sqlalchemy.py#L846 | 10:15 |
*** nacim has quit IRC | 10:16 | |
eglynn | DinaBelova: also pymongo ... https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/pymongo_base.py#L227 | 10:16 |
eglynn | DinaBelova: ... so pretty much ignore my flip-flopping earlier | 10:16 |
* eglynn can't even remember his own code now ... | 10:16 | |
DinaBelova | eglynn, ok, so that means the ordet rule change -> creation is a good one here :) | 10:17 |
DinaBelova | ok, so that means we have some really strange stuff here | 10:17 |
eglynn | DinaBelova: yes, that is my read on the intent of that code | 10:17 |
DinaBelova | as I'm still fighting with mismatch errors | 10:17 |
DinaBelova | they have different roots | 10:17 |
*** matsuhashi has quit IRC | 10:17 | |
*** matsuhashi has joined #openstack-ceilometer | 10:18 | |
DinaBelova | and some of them are related to the alarms order | 10:18 |
DinaBelova | we are expecting desc one - as in the impl | 10:18 |
DinaBelova | but got the opposite one in the tests reality | 10:18 |
eglynn | DinaBelova: however I'm beging to doubt my own sanity now, as I wrote the RDO docco in the opposite order http://openstack.redhat.com/CeilometerQuickStart | 10:18 |
eglynn | DinaBelova: ... and I'm pretty sure I just copied'n'pasted actual output from '$ ceilometer alarm-history ...' in that docco | 10:19 |
DinaBelova | 0_0, funny... | 10:19 |
eglynn | DinaBelova: unless of course the CLI re-orders | 10:19 |
eglynn | DinaBelova: ... where are you seeing the unexpected order? | 10:19 |
eglynn | DinaBelova: ... against a particular storage driver? | 10:20 |
ildikov | eglynn: good to know that I have to sanity check the full doc, before writing anything based on it ;) | 10:20 |
DinaBelova | in the alarms scenarios tests | 10:20 |
DinaBelova | for the hbase and sql drivers | 10:20 |
eglynn | DinaBelova: consistently or intermittently? | 10:20 |
DinaBelova | no fails when tox -e py27 is used, but consistently when I'm using the tox -e venv "python -m unittest discover -t . ceilometer/tests" | 10:22 |
eglynn | DinaBelova: forgive my ignorance, but waht's the effective difference between those two approaches? | 10:23 |
DinaBelova | no actual one.... | 10:23 |
eglynn | DinaBelova: ... i.e. both use a virtualenv, or? | 10:23 |
ildikov | DinaBelova: venv was changed not so long ago | 10:24 |
DinaBelova | but the thing is that is was the only way to reproduce the https://bugs.launchpad.net/ceilometer/+bug/1321826 | 10:24 |
DinaBelova | when this bug contained lots of issues | 10:24 |
DinaBelova | not related to each other | 10:24 |
DinaBelova | and locally this was not reproducible in the tox | 10:24 |
DinaBelova | tox with py27 env | 10:24 |
DinaBelova | but the direct discover helped to catch this error | 10:25 |
DinaBelova | and some others too | 10:25 |
DinaBelova | as it was a pack of them | 10:25 |
DinaBelova | as it turned out | 10:25 |
eglynn | a-ha ok ... so that bug relates to out-of-order alarm notifications? (as opposed to out-of-order history?) | 10:25 |
DinaBelova | it looks like so now, yes | 10:25 |
DinaBelova | it was one not related to the alarms fail too - but I fixed it in other place | 10:26 |
DinaBelova | and it turned out they were not related | 10:26 |
DinaBelova | >_< | 10:26 |
eglynn | DinaBelova: TBH I don't know why running the unit tests in different ways would change the ordering | 10:27 |
eglynn | DinaBelova: ... but I guess the actual ordering we see in deployment (e.g. in a devstack) is the canonical one? | 10:27 |
eglynn | DinaBelova: ... if that's different to the reverse ordering intended by the code, then we've a definite issue | 10:27 |
DinaBelova | eglynn, I guess the only one order is the right one here.... I'm trying to understand what do we have in the tests, that make all this stuff false... | 10:28 |
DinaBelova | or probably in the drivers - or in the interaction between storage driver and api... | 10:28 |
DinaBelova | all happening looks pretty suspecious for me | 10:28 |
eglynn | DinaBelova: ... indeeed | 10:29 |
*** r3pl4y has quit IRC | 10:29 | |
eglynn | DinaBelova: ... do you happen to have a fresh devstack running that you could check against? | 10:29 |
DinaBelova | suspicious* | 10:29 |
DinaBelova | I'll create one now | 10:29 |
eglynn | DinaBelova: thanks! | 10:29 |
DinaBelova | np, I'm worried about this behaviour too... | 10:29 |
* eglynn doesn't have a fresh devstack on-hand, I guess that says something about what he's doing with his time ... | 10:30 | |
DinaBelova | eglynn, :D | 10:30 |
*** Underyx|off is now known as underyx | 10:30 | |
*** r3pl4y has joined #openstack-ceilometer | 10:30 | |
DinaBelova | eglynn, ildikov - btw, may you try to run << tox -e venv "python -m unittest discover -t . ceilometer/tests" >> somewhere when you have fresh ceilo cloned? | 10:31 |
DinaBelova | I'd love to know if you see the fails and which ones | 10:31 |
* eglynn will spin a new f20 VM to get coverage on that platform also | 10:32 | |
DinaBelova | eglynn, ok, cool | 10:32 |
DinaBelova | :) | 10:32 |
eglynn | (... just in case it's something platform-specific) | 10:32 |
DinaBelova | eglynn, sure | 10:33 |
* eglynn taps fingers impatiently at internal cloud slowness ... | 10:34 | |
eglynn | ... the joys of dog-fooding | 10:34 |
* DinaBelova having new devstack VM in progress :) | 10:37 | |
DinaBelova | eglynn, it looks like ceiloclient does not change the order got from the ceilo server.... | 10:39 |
DinaBelova | hehehe | 10:39 |
eglynn | DinaBelova: a-ha, ok, that's a good datapoint | 10:39 |
DinaBelova | eglynn, it just uses _list method bithout any changes | 10:40 |
DinaBelova | without* | 10:40 |
DinaBelova | https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/v2/alarms.py#L167 | 10:40 |
*** fnaval has joined #openstack-ceilometer | 10:41 | |
*** Longgeek_ has quit IRC | 10:58 | |
*** Longgeek has joined #openstack-ceilometer | 10:59 | |
*** ildikov has quit IRC | 11:01 | |
*** ildikov_ has joined #openstack-ceilometer | 11:01 | |
*** piyushmasrani has quit IRC | 11:07 | |
*** Qu310 has quit IRC | 11:08 | |
*** Qu310 has joined #openstack-ceilometer | 11:09 | |
*** Longgeek has quit IRC | 11:13 | |
*** Longgeek has joined #openstack-ceilometer | 11:14 | |
*** idegtiarov has quit IRC | 11:16 | |
*** Longgeek has quit IRC | 11:19 | |
*** Longgeek has joined #openstack-ceilometer | 11:19 | |
*** Ruetobas has quit IRC | 11:21 | |
*** eglynn is now known as eglynn-call | 11:22 | |
*** idegtiarov has joined #openstack-ceilometer | 11:25 | |
*** Ruetobas has joined #openstack-ceilometer | 11:27 | |
openstackgerrit | A change was merged to openstack/ceilometer: Reenables the testr per test timeout https://review.openstack.org/100789 | 11:30 |
*** Ruetobas has quit IRC | 11:31 | |
*** Ruetobas has joined #openstack-ceilometer | 11:31 | |
*** prad_ has joined #openstack-ceilometer | 11:36 | |
*** fnaval has quit IRC | 11:38 | |
*** fnaval_ has joined #openstack-ceilometer | 11:39 | |
*** matsuhashi has quit IRC | 11:48 | |
*** matsuhashi has joined #openstack-ceilometer | 11:51 | |
openstackgerrit | Stefano Zilli proposed a change to openstack/ceilometer: Parametrize table_prefix_separator in hbase https://review.openstack.org/100829 | 11:52 |
*** fnaval_ has quit IRC | 11:53 | |
*** fnaval has joined #openstack-ceilometer | 11:54 | |
*** matsuhashi has quit IRC | 11:56 | |
*** matsuhashi has joined #openstack-ceilometer | 11:56 | |
*** Longgeek has quit IRC | 11:57 | |
*** Longgeek has joined #openstack-ceilometer | 11:58 | |
*** matsuhashi has quit IRC | 12:01 | |
*** idegtiarov has quit IRC | 12:03 | |
*** nacim has joined #openstack-ceilometer | 12:03 | |
*** idegtiarov has joined #openstack-ceilometer | 12:03 | |
*** idegtiarov has quit IRC | 12:08 | |
*** fnaval has quit IRC | 12:10 | |
*** fnaval_ has joined #openstack-ceilometer | 12:10 | |
*** idegtiarov has joined #openstack-ceilometer | 12:13 | |
*** jdob has joined #openstack-ceilometer | 12:27 | |
*** promulo has quit IRC | 12:28 | |
*** _nadya_ has joined #openstack-ceilometer | 12:31 | |
*** fnaval_ has quit IRC | 12:34 | |
*** chmartinez has joined #openstack-ceilometer | 12:34 | |
*** fnaval has joined #openstack-ceilometer | 12:34 | |
DinaBelova | eglynn-call or eglynn-regus - fyi, canonical devstack installation has the creation and then rule_change order | 12:37 |
eglynn-call | DinaBelova: ok, that's a useful datapoint | 12:39 |
*** inc0 has quit IRC | 12:39 | |
eglynn-call | DinaBelova: though clearly not what the code expects to happen | 12:39 |
*** eglynn-call is now known as eglynn | 12:39 | |
DinaBelova | eglynn-call, I'll debug this... | 12:39 |
eglynn | DinaBelova: cool, thanks ... what storage driver BTW, sql-a? | 12:40 |
DinaBelova | yes | 12:40 |
eglynn | DinaBelova: BTW on f20, I see a large number of errors from the scenario tests complaining about the DB url being null | 12:41 |
DinaBelova | >_< | 12:41 |
DinaBelova | what the storage impl are you using? | 12:42 |
*** zhikunliu has quit IRC | 12:42 | |
eglynn | DinaBelova: well, it's the unit tests ... so runs against sqlite, mongodb, mock hbase etc. | 12:43 |
DinaBelova | ok, and the errors are for all the backends? | 12:43 |
DinaBelova | or just some of them? | 12:43 |
eglynn | actually good question | 12:43 |
* eglynn checks | 12:43 | |
eglynn | hmmm, seemingly only db2 & mongo | 12:44 |
eglynn | ... at least that's all I'm seeing in my (incomplete) terminal screen buffer | 12:44 |
eglynn | ... lemme dig some more into that | 12:45 |
*** gordc has joined #openstack-ceilometer | 12:45 | |
eglynn | DinaBelova: what platform are you on, precise or trusty? | 12:45 |
DinaBelova | eglynn, 12.04 Ubuntu | 12:48 |
*** Longgeek has quit IRC | 12:48 | |
DinaBelova | eglynn, btw, I found that Ceilometer itself returns history in the reversed order (rule change, creation) but it's formatted while the printing to the ordinary view you remember | 12:49 |
DinaBelova | so heh, unit tests should pass with the reversed order >_< | 12:50 |
eglynn | meh! ... it would help if eglynn remembered to run "$ sudo yum install -y mongodb-server" ;) | 12:50 |
DinaBelova | eglynn :D | 12:50 |
eglynn | DinaBelova: do you mean, the CLI formating reverses the ordering? | 12:50 |
DinaBelova | yes, it looks so | 12:50 |
* DinaBelova trying to find the code | 12:51 | |
DinaBelova | at least the REST response is reversed | 12:51 |
*** julim has joined #openstack-ceilometer | 12:51 | |
*** fnaval has quit IRC | 12:51 | |
eglynn | cool, and that's what the tests would be asserting | 12:51 |
eglynn | ... we should prolly fix the CLI | 12:52 |
* eglynn is relieved that his copy'n'paste skills remain unquestioned ... ;) | 12:52 | |
DinaBelova | heh, so here is one funny moment - 'why' did the ordinary order appeared in the tests? :D | 12:52 |
DinaBelova | eglynn, :D | 12:52 |
*** fnaval_ has joined #openstack-ceilometer | 12:52 | |
*** inc0 has joined #openstack-ceilometer | 12:53 | |
* DinaBelova still cannot find this magical formatting :D | 12:53 | |
*** ildikov_ has quit IRC | 12:53 | |
sileht | eglynn, FYI: https://bugs.launchpad.net/ceilometer/+bug/1331453 | 12:57 |
sileht | eglynn, to track the removal of the eventlet.sleep(), I hope when the next oslo.msg will be released | 12:57 |
*** Longgeek has joined #openstack-ceilometer | 12:57 | |
eglynn | sileht: excellent, thanks! | 12:58 |
eglynn | sileht: ... we'll be relying on your powerz as an oslo.msg core to expedite the release ;) | 12:58 |
eglynn | sileht: ... what the usual cadence of oslo.msg releases? | 12:58 |
*** jmckind has joined #openstack-ceilometer | 12:59 | |
eglynn | DinaBelova: is this it ... https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/openstack/common/cliutils.py#L146 | 12:59 |
sileht | eglynn, I don't know :p | 12:59 |
*** ildikov has joined #openstack-ceilometer | 12:59 | |
*** rbowen has joined #openstack-ceilometer | 13:00 | |
DinaBelova | eglynn, yes, it is, but the fact is that sortby=1 when it's in this method, but here https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/v2/shell.py#L823 I see it's passed as None :D | 13:01 |
DinaBelova | eglynn, oh my God! | 13:02 |
DinaBelova | https://github.com/openstack/python-ceilometerclient/blob/master/ceilometerclient/v2/shell.py#L654 | 13:02 |
DinaBelova | thanks God, I'm not crazy :) | 13:02 |
eglynn | DinaBelova: LOL :) | 13:02 |
sileht | eglynn, If you can take a look and perhaps push the magic button on https://review.openstack.org/#/c/98721/ | 13:03 |
*** erecio has joined #openstack-ceilometer | 13:04 | |
DinaBelova | eglynn, I'll fix the order in the client, I guess | 13:04 |
eglynn | DinaBelova: yeah, that would be great, thanks! | 13:05 |
eglynn | sileht: will do | 13:10 |
*** gordc has quit IRC | 13:10 | |
*** inc0 has quit IRC | 13:11 | |
*** psharma_ has joined #openstack-ceilometer | 13:14 | |
*** fnaval_ has quit IRC | 13:15 | |
*** fnaval_ has joined #openstack-ceilometer | 13:15 | |
*** underyx is now known as Underyx|off | 13:16 | |
*** psharma has quit IRC | 13:17 | |
*** psharma__ has joined #openstack-ceilometer | 13:19 | |
*** psharma_ has quit IRC | 13:22 | |
*** psharma__ has quit IRC | 13:22 | |
*** gordc has joined #openstack-ceilometer | 13:27 | |
*** Underyx|off is now known as underyx | 13:30 | |
openstackgerrit | A change was merged to openstack/ceilometer: Use hacking from test-requirements https://review.openstack.org/100710 | 13:37 |
*** fnaval_ has quit IRC | 13:38 | |
*** changbl has quit IRC | 13:38 | |
*** _nadya_ has quit IRC | 13:57 | |
*** Longgeek_ has joined #openstack-ceilometer | 13:57 | |
*** Longgeek has quit IRC | 13:59 | |
*** promulo has joined #openstack-ceilometer | 14:00 | |
*** dmsimard_away is now known as dmsimard | 14:02 | |
*** nosnos has quit IRC | 14:03 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 14:04 |
*** fnaval has joined #openstack-ceilometer | 14:06 | |
*** eglynn is now known as eglynn-call | 14:06 | |
openstackgerrit | Chinmaya Bharadwaj proposed a change to openstack/ceilometer: Fixes ceilometer-compute service start failure https://review.openstack.org/97831 | 14:07 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 14:11 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ceilometer: Updated from global requirements https://review.openstack.org/96819 | 14:19 |
*** rwsu has joined #openstack-ceilometer | 14:29 | |
openstackgerrit | Dina Belova proposed a change to openstack/python-ceilometerclient: Fix the alarm history order shown to user https://review.openstack.org/100921 | 14:31 |
DinaBelova | eglynn-call ^^ - sorry, needed to go afk, so providing the fix now | 14:31 |
eglynn-call | DinaBelova: thanks! | 14:32 |
DinaBelova | eglynn-call, np :) | 14:32 |
*** _nadya_ has joined #openstack-ceilometer | 14:37 | |
*** r3pl4y has quit IRC | 14:41 | |
*** Longgeek_ has quit IRC | 14:43 | |
*** Longgeek has joined #openstack-ceilometer | 14:44 | |
*** Longgeek has quit IRC | 14:45 | |
*** Longgeek has joined #openstack-ceilometer | 14:47 | |
*** _nadya_ has quit IRC | 14:49 | |
*** jergerber has joined #openstack-ceilometer | 15:04 | |
*** eglynn-call is now known as eglynn | 15:04 | |
*** r3pl4y has joined #openstack-ceilometer | 15:06 | |
*** fnaval has quit IRC | 15:06 | |
*** Longgeek has quit IRC | 15:20 | |
*** IvanBerezovskiy has left #openstack-ceilometer | 15:21 | |
*** shardy is now known as shardy_afk | 15:23 | |
*** _nadya_ has joined #openstack-ceilometer | 15:29 | |
openstackgerrit | gordon chung proposed a change to openstack/ceilometer: sync oslo code https://review.openstack.org/100297 | 15:33 |
openstackgerrit | gordon chung proposed a change to openstack/ceilometer: use urlparse from six https://review.openstack.org/100351 | 15:35 |
*** packet has joined #openstack-ceilometer | 15:38 | |
*** changbl has joined #openstack-ceilometer | 15:44 | |
*** jmckind has quit IRC | 15:46 | |
*** ildikov has quit IRC | 15:48 | |
*** dhandy has quit IRC | 15:50 | |
*** fnaval has joined #openstack-ceilometer | 16:00 | |
*** rbowen has quit IRC | 16:00 | |
*** Ruetobas has quit IRC | 16:01 | |
*** _cjones_ has joined #openstack-ceilometer | 16:03 | |
*** Ruetobas has joined #openstack-ceilometer | 16:03 | |
*** Ruetobas has quit IRC | 16:08 | |
*** Ruetobas has joined #openstack-ceilometer | 16:14 | |
*** yjiang5_away is now known as yjiang5 | 16:15 | |
*** amalagon has quit IRC | 16:22 | |
*** nacim has quit IRC | 16:36 | |
openstackgerrit | gordon chung proposed a change to openstack/ceilometer: test https://review.openstack.org/100587 | 16:36 |
*** _nadya_ has quit IRC | 16:51 | |
*** rbowen has joined #openstack-ceilometer | 16:53 | |
*** isviridov is now known as isviridov|away | 16:56 | |
*** eglynn has quit IRC | 16:56 | |
*** _nadya_ has joined #openstack-ceilometer | 16:57 | |
*** harlowja_away is now known as harlowja | 17:06 | |
*** _nadya_ has quit IRC | 17:09 | |
*** rbowen has quit IRC | 17:20 | |
*** rbowen has joined #openstack-ceilometer | 17:20 | |
*** jdob has quit IRC | 17:24 | |
*** jdob has joined #openstack-ceilometer | 17:24 | |
*** jdob has quit IRC | 17:39 | |
*** jdob has joined #openstack-ceilometer | 17:39 | |
*** changbl has quit IRC | 17:40 | |
*** nati_ueno has joined #openstack-ceilometer | 17:41 | |
*** ildikov has joined #openstack-ceilometer | 17:43 | |
*** safchain has quit IRC | 17:47 | |
*** fnaval has quit IRC | 17:49 | |
*** tong_ has joined #openstack-ceilometer | 17:52 | |
*** tongli has joined #openstack-ceilometer | 17:53 | |
openstackgerrit | gordon chung proposed a change to openstack/ceilometer: test https://review.openstack.org/100587 | 17:53 |
*** nati_ueno has quit IRC | 17:53 | |
*** _nadya_ has joined #openstack-ceilometer | 17:53 | |
*** tongli has left #openstack-ceilometer | 17:54 | |
*** nati_ueno has joined #openstack-ceilometer | 17:54 | |
*** packet has quit IRC | 18:05 | |
*** changbl has joined #openstack-ceilometer | 18:16 | |
*** packet has joined #openstack-ceilometer | 18:19 | |
*** jaypipes has joined #openstack-ceilometer | 18:37 | |
openstackgerrit | gordon chung proposed a change to openstack/ceilometer: test https://review.openstack.org/100587 | 18:40 |
openstackgerrit | A change was merged to openstack/python-ceilometerclient: Fix the alarm history order shown to user https://review.openstack.org/100921 | 18:56 |
*** _nadya_ has quit IRC | 18:58 | |
*** german__ has joined #openstack-ceilometer | 19:01 | |
german__ | Hi | 19:01 |
*** german__ has quit IRC | 19:02 | |
*** german__ has joined #openstack-ceilometer | 19:02 | |
german__ | I am with the Neutron LBaaS team and we would like to add more events to ceilometer. What wuld be the best way to do that? | 19:03 |
*** chmartinez has quit IRC | 19:03 | |
*** openstackgerrit has quit IRC | 19:06 | |
*** openstackgerrit_ has joined #openstack-ceilometer | 19:07 | |
*** openstackgerrit_ is now known as openstackgerrit | 19:08 | |
*** fnaval has joined #openstack-ceilometer | 19:08 | |
*** jaypipes has quit IRC | 19:09 | |
*** julim has quit IRC | 19:18 | |
gordc | german__: what metrics were you looking to add? we did just add a few LBaaS related metrics | 19:24 |
*** jaypipes has joined #openstack-ceilometer | 19:24 | |
gordc | http://docs.openstack.org/developer/ceilometer/measurements.html#loadbalancer-as-a-service-lbaas | 19:25 |
*** _nadya_ has joined #openstack-ceilometer | 19:29 | |
german__ | let me check the URL. We want to add MBit Dest/Protocoll and LB created,deleted, etc | 19:29 |
german__ | gordc, yeah we have new ones | 19:30 |
gordc | german__: cool cool. here's the bp for the original work: https://blueprints.launchpad.net/ceilometer/+spec/ceilometer-meter-lbaas | 19:32 |
gordc | german__: if you're just going to reuse the same polling features or if neutron emits notifications and you'll be adding notification listeners, i don't think we'd need a new bp. | 19:32 |
gordc | if you need to add something novel to add those metrics than i'd suggest a new bp. | 19:33 |
gordc | german__: i guess a good place to start would be to look at patches prad_ implemented to see if you can reuse some of his work. | 19:34 |
prad_ | german__, gordc, i'm looking to see if i can add resource discovery approach to what I initially added.. | 19:35 |
prad_ | so some of that stuff might change in the coming week or so | 19:35 |
prad_ | i think we already have the created/deleted info in the metadata, we might have to separate that into its own meter | 19:36 |
german__ | yep, I think that would be great | 19:36 |
prad_ | german__, anyway, let me know if you need help with any of this | 19:37 |
german__ | I do need help :-) | 19:37 |
gordc | german__: i should metion prad_ is the neutron/ceilo expert :) | 19:37 |
prad_ | hehe, definitely not an expert .. just learning as i go :) | 19:37 |
german__ | yeah, I am sitting here with Mark McClain and he suggested popping in the channel | 19:38 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/99567 | 19:40 |
*** Ruetobas has quit IRC | 19:40 | |
*** Ruetobas has joined #openstack-ceilometer | 19:48 | |
*** julim has joined #openstack-ceilometer | 19:51 | |
*** Ruetobas has quit IRC | 19:53 | |
*** Ruetobas has joined #openstack-ceilometer | 19:58 | |
openstackgerrit | gordon chung proposed a change to openstack/ceilometer-specs: 'big-data' sql part 2 https://review.openstack.org/101009 | 20:12 |
*** jaypipes has quit IRC | 20:15 | |
*** gordc has left #openstack-ceilometer | 20:21 | |
*** safchain has joined #openstack-ceilometer | 20:29 | |
*** julim has quit IRC | 20:34 | |
*** amalagon has joined #openstack-ceilometer | 20:34 | |
*** gyee has joined #openstack-ceilometer | 20:42 | |
openstackgerrit | gordon chung proposed a change to openstack/ceilometer: test https://review.openstack.org/100587 | 20:52 |
*** terriyu has joined #openstack-ceilometer | 20:55 | |
*** terriyu has quit IRC | 20:55 | |
*** promulo has quit IRC | 20:58 | |
*** erecio has quit IRC | 20:59 | |
*** _nadya_ has quit IRC | 21:01 | |
*** _nadya_ has joined #openstack-ceilometer | 21:02 | |
*** jdob has quit IRC | 21:04 | |
*** _nadya_ has quit IRC | 21:07 | |
*** tongli has joined #openstack-ceilometer | 21:12 | |
*** tongli has quit IRC | 21:13 | |
*** rbowen has quit IRC | 21:18 | |
*** tong_ has quit IRC | 21:20 | |
*** jergerber has quit IRC | 21:26 | |
*** safchain has quit IRC | 21:29 | |
*** packet has quit IRC | 21:38 | |
*** changbl has quit IRC | 21:38 | |
*** eglynn has joined #openstack-ceilometer | 21:46 | |
openstackgerrit | A change was merged to openstack/python-ceilometerclient: Provide explicit help string of resource-metadata https://review.openstack.org/97674 | 21:54 |
*** dmsimard is now known as dmsimard_away | 22:10 | |
*** eglynn has quit IRC | 22:47 | |
*** hi has joined #openstack-ceilometer | 22:52 | |
*** hi is now known as Guest77174 | 22:52 | |
Guest77174 | hi | 22:53 |
Guest77174 | i need help .. | 22:53 |
Guest77174 | i have install ceilometer on havana version | 22:53 |
Guest77174 | when i run cielomter meter-list i dont get cpu_util | 22:53 |
Guest77174 | \it seems i don t get meter that use pollster | 22:54 |
Guest77174 | any hint | 22:54 |
*** german__ has quit IRC | 22:56 | |
*** fnaval has quit IRC | 22:57 | |
*** prad_ has quit IRC | 23:20 | |
*** Guest77174 has quit IRC | 23:39 | |
*** yfujioka has joined #openstack-ceilometer | 23:49 | |
*** promulo has joined #openstack-ceilometer | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!