| *** ankita_w_ has joined #openstack-ceilometer | 00:02 | |
| *** david-lyle has quit IRC | 00:03 | |
| *** ankita___ has joined #openstack-ceilometer | 00:03 | |
| -openstackstatus- NOTICE: Zuul was restarted due to an error; events (such as approvals or new patchsets) since 23:01 UTC have been lost and affected changes will need to be rechecked | 00:04 | |
| *** ankita_wagh has quit IRC | 00:05 | |
| *** ankita_w_ has quit IRC | 00:06 | |
| *** ankita_wagh has joined #openstack-ceilometer | 00:11 | |
| *** ankita___ has quit IRC | 00:14 | |
| *** jruano has quit IRC | 00:23 | |
| *** thorst has joined #openstack-ceilometer | 00:36 | |
| *** rbak has joined #openstack-ceilometer | 00:39 | |
| *** thumpba has quit IRC | 00:42 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Drop the downgrade function of migration scripts https://review.openstack.org/211379 | 00:44 |
|---|---|---|
| *** lexholden has quit IRC | 00:49 | |
| *** rbak has quit IRC | 00:50 | |
| *** david-lyle has joined #openstack-ceilometer | 00:54 | |
| *** thumpba has joined #openstack-ceilometer | 00:56 | |
| *** thumpba has quit IRC | 00:59 | |
| *** _cjones_ has quit IRC | 01:00 | |
| *** thumpba has joined #openstack-ceilometer | 01:08 | |
| *** ankita_wagh has quit IRC | 01:11 | |
| *** thumpba_ has joined #openstack-ceilometer | 01:13 | |
| *** thumpba has quit IRC | 01:14 | |
| *** thumpba_ has quit IRC | 01:15 | |
| *** thumpba has joined #openstack-ceilometer | 01:15 | |
| *** thorst has quit IRC | 01:16 | |
| *** liusheng has quit IRC | 01:19 | |
| *** liusheng has joined #openstack-ceilometer | 01:19 | |
| *** ankita_wagh has joined #openstack-ceilometer | 01:37 | |
| *** jruano has joined #openstack-ceilometer | 01:40 | |
| *** alejandrito has quit IRC | 01:55 | |
| *** thumpba has quit IRC | 02:04 | |
| *** thumpba has joined #openstack-ceilometer | 02:05 | |
| *** thumpba has quit IRC | 02:05 | |
| *** jwcroppe has joined #openstack-ceilometer | 02:23 | |
| *** david-lyle has quit IRC | 02:33 | |
| *** jruano has quit IRC | 02:34 | |
| *** jdandrea has quit IRC | 02:43 | |
| *** ankita_wagh has quit IRC | 02:56 | |
| *** jruano has joined #openstack-ceilometer | 03:03 | |
| *** jwcroppe has quit IRC | 03:15 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 03:22 | |
| *** david-lyle has joined #openstack-ceilometer | 03:30 | |
| *** htruta has quit IRC | 03:36 | |
| *** dikonoor has joined #openstack-ceilometer | 03:36 | |
| *** htruta has joined #openstack-ceilometer | 03:40 | |
| *** slogan621 has joined #openstack-ceilometer | 03:43 | |
| *** rjaiswal has quit IRC | 04:00 | |
| *** jruano has quit IRC | 04:03 | |
| *** slogan621 has quit IRC | 04:06 | |
| *** fabian1 has quit IRC | 04:15 | |
| *** fabian1 has joined #openstack-ceilometer | 04:16 | |
| *** ankita_wagh has joined #openstack-ceilometer | 04:36 | |
| *** ankita_wagh has quit IRC | 04:36 | |
| *** ankita_wagh has joined #openstack-ceilometer | 04:37 | |
| *** yprokule has joined #openstack-ceilometer | 04:37 | |
| *** links has joined #openstack-ceilometer | 04:42 | |
| *** liusheng has quit IRC | 05:20 | |
| *** liusheng has joined #openstack-ceilometer | 05:20 | |
| *** fawadkhaliq has quit IRC | 05:38 | |
| *** fawadkhaliq has joined #openstack-ceilometer | 05:59 | |
| *** ankita_w_ has joined #openstack-ceilometer | 06:08 | |
| *** nadya has joined #openstack-ceilometer | 06:10 | |
| *** ankita_wagh has quit IRC | 06:10 | |
| *** exploreshaifali has joined #openstack-ceilometer | 06:29 | |
| *** exploreshaifali has quit IRC | 06:32 | |
| *** exploreshaifali has joined #openstack-ceilometer | 06:43 | |
| *** ildikov has quit IRC | 07:00 | |
| *** _gryf has quit IRC | 07:04 | |
| *** _gryf has joined #openstack-ceilometer | 07:05 | |
| *** ankita_w_ has quit IRC | 07:11 | |
| *** fawadkhaliq has quit IRC | 07:12 | |
| *** fawadk has joined #openstack-ceilometer | 07:13 | |
| *** ig0r_ has joined #openstack-ceilometer | 07:15 | |
| *** ankita_wagh has joined #openstack-ceilometer | 07:17 | |
| *** afazekas has joined #openstack-ceilometer | 07:31 | |
| *** ankita_wagh has quit IRC | 07:35 | |
| *** ig0r__ has joined #openstack-ceilometer | 07:39 | |
| *** ig0r_ has quit IRC | 07:41 | |
| *** boris-42 has quit IRC | 07:50 | |
| *** _gryf has quit IRC | 07:53 | |
| *** _gryf has joined #openstack-ceilometer | 07:58 | |
| *** ildikov has joined #openstack-ceilometer | 08:08 | |
| openstackgerrit | Merged openstack/aodh: Stop registering oslo.messaging option https://review.openstack.org/211025 | 08:11 |
| openstackgerrit | Merged openstack/aodh: storage: remove unused option db2nosql_resource_id_maxlen https://review.openstack.org/211657 | 08:12 |
| *** ig0r__ has quit IRC | 08:13 | |
| *** ig0r_ has joined #openstack-ceilometer | 08:17 | |
| *** ig0r_ has quit IRC | 08:19 | |
| *** yassine__ has joined #openstack-ceilometer | 08:19 | |
| *** liusheng has quit IRC | 08:19 | |
| *** liusheng has joined #openstack-ceilometer | 08:20 | |
| *** eglynn has joined #openstack-ceilometer | 08:20 | |
| openstackgerrit | liusheng proposed openstack/aodh: Remove the unused cpu_count utils method https://review.openstack.org/211895 | 08:26 |
| *** nadya has quit IRC | 08:30 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Change and move the workers options to corresponding service section https://review.openstack.org/209333 | 08:44 |
| openstackgerrit | liusheng proposed openstack/aodh: Remove the unused cpu_count utils method https://review.openstack.org/211895 | 08:45 |
| *** lexholden has joined #openstack-ceilometer | 08:52 | |
| *** lexholden has quit IRC | 08:53 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Change and move the workers options to corresponding service section https://review.openstack.org/209333 | 08:53 |
| *** lexholden has joined #openstack-ceilometer | 08:54 | |
| *** nadya has joined #openstack-ceilometer | 08:55 | |
| *** nadya has quit IRC | 09:02 | |
| *** nadya has joined #openstack-ceilometer | 09:06 | |
| *** nadya has quit IRC | 09:07 | |
| *** nadya has joined #openstack-ceilometer | 09:08 | |
| openstackgerrit | liusheng proposed openstack/aodh: Updated from global requirements https://review.openstack.org/211921 | 09:17 |
| openstackgerrit | liusheng proposed openstack/aodh: Use the Serializer from oslo.messaging https://review.openstack.org/211922 | 09:19 |
| *** fabian1 has quit IRC | 09:20 | |
| *** fabian1 has joined #openstack-ceilometer | 09:21 | |
| *** exploreshaifali has quit IRC | 09:23 | |
| *** exploreshaifali has joined #openstack-ceilometer | 09:25 | |
| *** exploreshaifali has quit IRC | 09:32 | |
| *** exploreshaifali has joined #openstack-ceilometer | 09:33 | |
| *** exploreshaifali_ has joined #openstack-ceilometer | 09:38 | |
| *** exploreshaifali has quit IRC | 09:39 | |
| *** Kennan2 is now known as Kennan | 09:42 | |
| *** dikonoo has joined #openstack-ceilometer | 09:44 | |
| *** divya__ has joined #openstack-ceilometer | 09:44 | |
| *** dikonoo has quit IRC | 09:44 | |
| *** yprokule has quit IRC | 09:46 | |
| *** dikonoor has quit IRC | 09:48 | |
| *** jwcroppe has joined #openstack-ceilometer | 09:49 | |
| *** yprokule has joined #openstack-ceilometer | 09:50 | |
| *** ig0r_ has joined #openstack-ceilometer | 10:10 | |
| openstackgerrit | Merged openstack/ceilometer: Drop the downgrade function of migration scripts https://review.openstack.org/211379 | 10:14 |
| *** jkraj has joined #openstack-ceilometer | 10:19 | |
| *** exploreshaifali_ has quit IRC | 10:20 | |
| *** lexholden has quit IRC | 10:28 | |
| *** jwcroppe has quit IRC | 10:35 | |
| *** jwcroppe has joined #openstack-ceilometer | 10:36 | |
| *** jwcroppe has quit IRC | 10:40 | |
| *** fawadk has quit IRC | 10:41 | |
| *** ig0r_ has quit IRC | 10:46 | |
| openstackgerrit | Julien Danjou proposed openstack/aodh: tox: generate sample config file on default target https://review.openstack.org/210842 | 10:49 |
| *** exploreshaifali has joined #openstack-ceilometer | 10:50 | |
| *** alexpilotti has joined #openstack-ceilometer | 10:55 | |
| *** cdent has joined #openstack-ceilometer | 11:07 | |
| *** lexholden has joined #openstack-ceilometer | 11:24 | |
| *** thorst has joined #openstack-ceilometer | 11:37 | |
| *** julim has joined #openstack-ceilometer | 11:38 | |
| *** gordc has joined #openstack-ceilometer | 11:49 | |
| openstackgerrit | Merged openstack/aodh: Remove the unused cpu_count utils method https://review.openstack.org/211895 | 12:11 |
| *** idegtiarov has quit IRC | 12:28 | |
| *** liusheng has quit IRC | 12:31 | |
| openstackgerrit | liusheng proposed openstack/ceilometer: Change and move the workers options to corresponding service section https://review.openstack.org/209333 | 12:32 |
| openstackgerrit | liusheng proposed openstack/aodh: Updated from global requirements https://review.openstack.org/211921 | 12:35 |
| *** jruano has joined #openstack-ceilometer | 12:38 | |
| *** edmondsw has joined #openstack-ceilometer | 12:41 | |
| *** alexpilotti has quit IRC | 12:44 | |
| openstackgerrit | liusheng proposed openstack/aodh: Use the Serializer from oslo.messaging https://review.openstack.org/211922 | 12:45 |
| *** alexpilotti has joined #openstack-ceilometer | 12:46 | |
| cdent | gee, thanks very much gordc | 13:07 |
| gordc | promises be promises | 13:08 |
| *** jwcroppe has joined #openstack-ceilometer | 13:10 | |
| *** jasonamyers has joined #openstack-ceilometer | 13:11 | |
| *** IBerezovskiy has quit IRC | 13:15 | |
| *** ityaptin has quit IRC | 13:15 | |
| *** IBerezovskiy has joined #openstack-ceilometer | 13:23 | |
| *** jwcroppe has quit IRC | 13:23 | |
| *** ityaptin has joined #openstack-ceilometer | 13:23 | |
| *** jwcroppe has joined #openstack-ceilometer | 13:23 | |
| *** edmondsw has quit IRC | 13:24 | |
| *** idegtiarov has joined #openstack-ceilometer | 13:27 | |
| *** jwcroppe has quit IRC | 13:27 | |
| idegtiarov | gordc, Hi, please take a look at my comment to this bug https://bugs.launchpad.net/ceilometer/+bug/1484117 there are two way for fixing I need to choose the best | 13:35 |
| openstack | Launchpad bug 1484117 in Ceilometer "None image.size value crashes on get_sample" [Undecided,New] - Assigned to Igor Degtiarov (idegtiarov) | 13:35 |
| gordc | idegtiarov: is image.size a notification or a poll? | 13:40 |
| gordc | that's really weird it gives you a None size. | 13:41 |
| idegtiarov | gordc, both according to our docs | 13:41 |
| gordc | idegtiarov: i agree with your comment... it's probably safer to just ignore None | 13:44 |
| *** jasonamyers has quit IRC | 13:44 | |
| *** pradk has joined #openstack-ceilometer | 13:44 | |
| *** pradk has quit IRC | 13:44 | |
| gordc | or any non-numerical value. | 13:44 |
| idegtiarov | gordc, it is the same operation on devstack via glance api gives 0 values for such images neither creation with glanceclient | 13:45 |
| idegtiarov | gordc, got it will propose cr soon | 13:45 |
| gordc | idegtiarov: i think your work might overlap with prad's declarative meter work. | 13:46 |
| gordc | we should probably validate the meters we make to ensure volume is a number | 13:46 |
| idegtiarov | gordc, right, do you know is adding of such validation in prad's work-plan? Or I can start to work on it? | 13:48 |
| gordc | idegtiarov: i'm not sure he's working on it... but we'll need to ask him. | 13:51 |
| *** links has quit IRC | 13:51 | |
| gordc | or you can just put a patch up if you think it's easy enough. | 13:51 |
| *** annasort has quit IRC | 13:51 | |
| *** annasort has joined #openstack-ceilometer | 13:52 | |
| *** edmondsw has joined #openstack-ceilometer | 13:52 | |
| idegtiarov | gordc, I hope it is easy enough, will add a patch %) | 13:54 |
| *** nadya has quit IRC | 14:03 | |
| *** annasort has quit IRC | 14:04 | |
| *** nadya has joined #openstack-ceilometer | 14:05 | |
| *** rbak has joined #openstack-ceilometer | 14:10 | |
| gordc | idegtiarov: yeah i think it's pretty simple. have at it! | 14:12 |
| idegtiarov | gordc, thanks! | 14:12 |
| *** thumpba has joined #openstack-ceilometer | 14:15 | |
| *** jruano has quit IRC | 14:17 | |
| *** ildikov has quit IRC | 14:33 | |
| *** exploreshaifali has quit IRC | 14:42 | |
| *** afazekas has quit IRC | 14:44 | |
| *** jdandrea has joined #openstack-ceilometer | 14:54 | |
| openstackgerrit | Rohit Jaiswal proposed openstack/aodh: Fixes querying alarm history with severity field https://review.openstack.org/200315 | 14:59 |
| *** jdandrea has left #openstack-ceilometer | 15:01 | |
| *** alexpilotti has quit IRC | 15:03 | |
| *** serverascode has quit IRC | 15:06 | |
| *** serverascode has joined #openstack-ceilometer | 15:09 | |
| *** annasort has joined #openstack-ceilometer | 15:16 | |
| gordc | cdent: you understand devstack right? | 15:19 |
| cdent | that's a huge statement | 15:20 |
| gordc | lol | 15:20 |
| cdent | sometimes, when the stars are aligned, it does what I want | 15:20 |
| cdent | what's up? | 15:20 |
| gordc | any idea why requirements patch changes here: http://logs.openstack.org/14/211314/1/experimental/gate-tempest-dsvm-ceilometer-es/50029a6/logs/devstacklog.txt.gz#_2015-08-12_13_29_56_274 | 15:20 |
| *** yassine__ has quit IRC | 15:20 | |
| gordc | from here http://logs.openstack.org/14/211314/1/experimental/gate-tempest-dsvm-ceilometer-es/50029a6/logs/devstacklog.txt.gz#_2015-08-12_13_23_24_398 | 15:21 |
| gordc | for some reason it's searching in /opt/stack/new/requirements and then all of a sudden drops the new part when searching for elasticsearch | 15:21 |
| cdent | weird | 15:22 |
| cdent | one moment pelase | 15:22 |
| cdent | gordc: it is what ever is resetting $DEST a few lines above | 15:24 |
| *** yassine__ has joined #openstack-ceilometer | 15:25 | |
| gordc | cdent: yeah... i just noticed that too | 15:27 |
| gordc | i'm assuming the idea was to download package and share it between old/new? | 15:27 |
| cdent | Not clear, but if that's the case something elsewhere in the environment would be setting it | 15:28 |
| cdent | and it looks like it only becomes /opt/stack if it isn't already set | 15:28 |
| cdent | and I think it is supposed to be | 15:28 |
| * cdent does more grepping | 15:28 | |
| gordc | but it's failing on pip_install_gr which uses REQUIREMENTS_DIR | 15:29 |
| gordc | looks like when glance did their patch, they f'd up ours. | 15:31 |
| gordc | https://github.com/openstack-dev/devstack/commit/dd62293591fd1e822f59754cece645639a4d2d2c | 15:31 |
| *** thumpba has quit IRC | 15:31 | |
| cdent | i'm not sure that's the case gordc | 15:32 |
| cdent | the problem is that a value for DEST is not being set earlier, or is being lost | 15:33 |
| *** thumpba has joined #openstack-ceilometer | 15:33 | |
| cdent | earlier in the log file DEST has the right value | 15:33 |
| cdent | the infra file needs DEST to be set | 15:33 |
| cdent | http://logs.openstack.org/14/211314/1/experimental/gate-tempest-dsvm-ceilometer-es/50029a6/logs/devstacklog.txt.gz#_2015-08-12_13_20_12_824 | 15:35 |
| gordc | cdent: what am i suppose to be looking at? | 15:37 |
| gordc | cdent: DEST value being set? | 15:37 |
| cdent | that's just just showing that DEST has the right value | 15:37 |
| cdent | _there_ | 15:37 |
| gordc | ah, got it. | 15:37 |
| nadya | gordc: error during postres job: Could not find mongod command. So sad. Looks like I need to do smth with Ceilo code | 15:42 |
| cdent | gordc: got it as in understand me, or got it as in figured it out/ | 15:43 |
| gordc | cdent: oh. got as in understood you. | 15:45 |
| *** adurbin_ has joined #openstack-ceilometer | 15:45 | |
| gordc | i don't understand why the lines were added as part of glance patch. | 15:45 |
| gordc | cdent: it was fine for experimental jobs before | 15:46 |
| cdent | I don't understand either | 15:46 |
| gordc | "i don't understand why the lines were added as part of glance patch. | 15:48 |
| gordc | is that a good commit message? | 15:48 |
| cdent | best ever, really | 15:48 |
| *** ocaner has joined #openstack-ceilometer | 15:49 | |
| gordc | nadya: yeah, seems like it's not passing in anything and just assumes mongodb | 15:53 |
| cdent | nadya: take a look at the gate_hook and post_gate_hook changes in aodh | 15:54 |
| cdent | those were copied from ceilo and the ones in ceilo are _wrong_ | 15:54 |
| nadya | gordc: no, I guess that even if backend is pastres tests are run on mongo as well | 15:55 |
| nadya | cdent: postgres job for aodh doesn't work? | 15:56 |
| cdent | it does _now_ | 15:56 |
| cdent | but it needed to be fixed | 15:56 |
| cdent | (for it to run in the gate) | 15:56 |
| *** thumpba has quit IRC | 15:57 | |
| *** thumpba has joined #openstack-ceilometer | 15:57 | |
| nadya | cdent: I'll take a look at your changes...:) | 15:58 |
| *** ocaner has quit IRC | 15:58 | |
| *** thumpba has quit IRC | 15:58 | |
| cdent | I'd offer more help, but I'm currently juggling too many eggs :) | 15:58 |
| *** thumpba has joined #openstack-ceilometer | 15:58 | |
| nadya | cdent: I know what you are talking about... | 16:00 |
| *** nadya has quit IRC | 16:00 | |
| *** cmyster has left #openstack-ceilometer | 16:05 | |
| *** jkraj has quit IRC | 16:11 | |
| *** _cjones_ has joined #openstack-ceilometer | 16:26 | |
| *** pradk has joined #openstack-ceilometer | 16:42 | |
| openstackgerrit | Ryota MIBU proposed openstack/aodh: Storage: add 'exclude' constraint to get_alarms() https://review.openstack.org/210443 | 16:56 |
| openstackgerrit | Ryota MIBU proposed openstack/aodh: Exclude event type from targets of alarm evaluator https://review.openstack.org/200969 | 16:56 |
| *** yprokule has quit IRC | 16:56 | |
| *** nadya has joined #openstack-ceilometer | 17:01 | |
| *** nadya has quit IRC | 17:05 | |
| *** _gryf has quit IRC | 17:09 | |
| *** _cjones_ has quit IRC | 17:16 | |
| *** drjones has joined #openstack-ceilometer | 17:16 | |
| *** belmoreira has joined #openstack-ceilometer | 17:19 | |
| *** belmoreira has quit IRC | 17:21 | |
| *** ankita_wagh has joined #openstack-ceilometer | 17:21 | |
| *** _gryf has joined #openstack-ceilometer | 17:22 | |
| *** yassine__ has quit IRC | 17:24 | |
| *** annasort has quit IRC | 17:24 | |
| *** vishwanathj has joined #openstack-ceilometer | 17:27 | |
| *** jwcroppe has joined #openstack-ceilometer | 17:30 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/ceilometer: Convert instance, bandwidth and SwiftMiddleware meters https://review.openstack.org/209637 | 17:38 |
| *** divya__ has quit IRC | 17:45 | |
| gordc | pradk: what do you think about leaving this as code and deleting it next cycle: https://github.com/openstack/ceilometer/blob/master/ceilometer/network/notifications.py | 17:48 |
| pradk | gordc, i'm actually working on converting those as we speak | 17:48 |
| gordc | i can probably try to convert it... but the logic is absurd and i don't want to waste my time. | 17:48 |
| pradk | gordc, we could if there is a reason we dont want to | 17:48 |
| gordc | pradk: oh... is it easy? | 17:48 |
| pradk | gordc, yea the list aspect is a bit weird | 17:48 |
| pradk | so there are two scenarios | 17:49 |
| gordc | just asking because i don't think we should waste our time on it. | 17:49 |
| pradk | 1. payload just has network key and it has a single resource which is easy to convert | 17:49 |
| pradk | the werid part is bulk | 17:49 |
| gordc | it's definitely getting dropped in 1 cycle so... | 17:49 |
| gordc | err 1 month i mean | 17:49 |
| pradk | yea i'm fine with that.. i think llu is doing the ipmi hardware metrics right? | 17:50 |
| pradk | he had a blueprint for that iirc | 17:50 |
| gordc | ... i'm not sure. i think llu has been busy with somethign else... he's been mia | 17:51 |
| gordc | there was a declarative snmp patch... but i haven't seen/heard anything... it's probably going ot get pushed | 17:51 |
| gordc | pradk: is the network stuff only thing remaining? i think we should target real metrics before trying to convert the non-metric ones | 17:52 |
| gordc | (unless they are really straight forward) | 17:52 |
| pradk | lemme look whats remaining | 17:53 |
| gordc | i think there's rohit's patch for dns | 17:53 |
| gordc | that is the math one | 17:54 |
| pradk | yea dns, trove, profiler, ceilometer middleware | 17:54 |
| pradk | so we're planning to drop vol=1 meters once liberty is cut? | 17:55 |
| gordc | pradk: yes. we deprecated in Juno so we gave the 2 cycle heads up | 17:56 |
| gordc | and i've yet to be told a proper use case for them | 17:56 |
| pradk | that would take care of all network meters except bandwidth which i already converted | 17:56 |
| gordc | regarding statistics that is... i'm still not sure there is one | 17:56 |
| pradk | right | 17:57 |
| pradk | gordc, the telemetry.notifications looks very similar to swift multi meter one | 17:58 |
| gordc | pradk: crap. ok. yeah that involves all values being multi-valued | 17:59 |
| gordc | pradk: i can readjust the logic | 17:59 |
| *** ildikov has joined #openstack-ceilometer | 17:59 | |
| gordc | hmm. trove and dns will be similar | 18:00 |
| pradk | the profiler looks straight forward except for resource id | 18:00 |
| gordc | the meter names look very strange. i'm going to ask rohit about that. | 18:00 |
| gordc | pradk: yeah i noticed that. | 18:00 |
| gordc | pradk: i wonder if we can scrap that 'profiler' tag | 18:01 |
| gordc | pradk: want me to send a ML about that? | 18:01 |
| pradk | yea seems like the prefix has no value unless they are doing something with it later | 18:01 |
| gordc | or you can | 18:02 |
| pradk | profiler-* vs payload.base_id | 18:02 |
| pradk | sure | 18:02 |
| gordc | 'sure' -> prad will send or 'sure' -> gord will send? | 18:02 |
| pradk | hehe.. i can send, this is regarding scraping profiler tag from resource id correct? | 18:03 |
| gordc | yep :) | 18:03 |
| gordc | you should mention the part where we want ids to be proper | 18:04 |
| pradk | ok | 18:04 |
| gordc | http://lists.openstack.org/pipermail/openstack-dev/2015-June/068104.html | 18:04 |
| gordc | pradk: yeah i would target the dns/trove meters next... not sure if you see anything of higher importance. | 18:06 |
| pradk | gordc, i just need to revisit math one i pushed to see if i can figure out a cleaner way to handle the combination resource id instead of having it as a list and joining with _ | 18:07 |
| pradk | i've been looking at network ones, i'll scrape that for now.. those are useless anyway | 18:08 |
| gordc | pradk: yeah the list is odd... tbh, it might be possible to pair it with the trove/dns volume evluations. | 18:08 |
| gordc | instead of doing time - time, it's str + str. | 18:08 |
| pradk | yea similar logic i guess.. one option i was thinking is have attribute tag with reference and then do an expression with it | 18:09 |
| pradk | something like | 18:09 |
| pradk | attr: - &foo -&bar | 18:09 |
| pradk | expr: *foo + *bar | 18:09 |
| pradk | expr: *foo + '_' + *bar | 18:10 |
| pradk | fact that its a string manipulation within yaml is kinda weird | 18:10 |
| gordc | i have no idea what you just typed.haha | 18:11 |
| pradk | hehe | 18:12 |
| gordc | did the pyparsing stuff not pan out? i didn't actually try it out. | 18:12 |
| pradk | gordc, thats for the math one right? this is string concat? | 18:12 |
| pradk | i dint look at pyparsing closely yet | 18:13 |
| gordc | does it matter? i'm wondering if it's smarter enough to not care. | 18:13 |
| gordc | https://github.com/openstack/ceilometer/blob/master/ceilometer/ipmi/notifications/ironic.py#L123-L124 haha | 18:15 |
| gordc | i think we rushed that patch in | 18:16 |
| pradk | uh oh | 18:17 |
| *** jfluhmann has joined #openstack-ceilometer | 18:21 | |
| gordc | yeah the ipmi stuff is some craziness... there's no way that's being converted into declarative meter... there's some much cleaning that is done. | 18:26 |
| pradk | yea | 18:29 |
| pradk | email sent | 18:29 |
| openstackgerrit | Merged openstack/aodh: Fixes querying alarm history with severity field https://review.openstack.org/200315 | 18:30 |
| *** nadya has joined #openstack-ceilometer | 18:32 | |
| *** annasort has joined #openstack-ceilometer | 18:34 | |
| *** nadya has quit IRC | 18:37 | |
| gordc | pradk: haha! i was joking about the dropping part but oh well | 18:40 |
| gordc | time to make bold moves. | 18:40 |
| pradk | hehe i just said if we dont hear back we'll assume it, which is reasonable :) | 18:41 |
| pradk | gordc, this looks interesting http://pyparsing.wikispaces.com/file/view/eval_arith.py | 18:42 |
| gordc | pradk: agreed... (i've no idea how it works) | 18:43 |
| *** nadya has joined #openstack-ceilometer | 18:53 | |
| *** boris-42 has joined #openstack-ceilometer | 18:54 | |
| *** exploreshaifali has joined #openstack-ceilometer | 18:56 | |
| *** jfluhmann has quit IRC | 19:12 | |
| openstackgerrit | Nadya Shakhat proposed openstack/ceilometer: Make it possible to run postgesql functional job https://review.openstack.org/211651 | 19:12 |
| gordc | pradk: oh fyi, idegtiarov is adding some field valiation logic, ie. make sure volume == float/int | 19:30 |
| pradk | ok | 19:30 |
| *** adurbin_ has quit IRC | 19:40 | |
| *** exploreshaifali has quit IRC | 19:44 | |
| *** lexholden has quit IRC | 19:57 | |
| *** Ephur has joined #openstack-ceilometer | 20:03 | |
| openstackgerrit | gordon chung proposed openstack/ceilometer: remove alembic requirement https://review.openstack.org/212179 | 20:24 |
| *** jruano has joined #openstack-ceilometer | 20:28 | |
| *** thumpba has quit IRC | 20:40 | |
| *** jruano has quit IRC | 21:00 | |
| *** thorst has quit IRC | 21:16 | |
| cdent | gordc: you still around | 21:21 |
| cdent | should I revive this: https://review.openstack.org/#/c/203109/ | 21:22 |
| openstackgerrit | Chris Dent proposed openstack/ceilometer: Grenade plugin using devstack plugin for ceilometer https://review.openstack.org/196441 | 21:24 |
| gordc | cdent: yes... why are you still around | 21:25 |
| cdent | cuz I'm stoopid | 21:26 |
| gordc | cool cool... um... i don't know if it'll make it in before code cutoff tbh | 21:27 |
| cdent | on that grenade thing ^ I just changed the commit message to remove the depends-on | 21:27 |
| cdent | Yeah, that was my thinking too | 21:27 |
| cdent | I was just scanning my pending reviews and the dogpile one stuck out as something that would need extensive work to recover | 21:28 |
| gordc | cdent: was that real code or pseudo code? | 21:29 |
| cdent | a) I think the depends-on was blocking it merging b) I think the experimental job will stop working correctly without the depends-on | 21:29 |
| cdent | it worked, but was not suitable for consumption | 21:29 |
| cdent | the dog pile stuff was hardwired in | 21:29 |
| gordc | i have a slight concern it's just going to be masking real issues in gnocchi | 21:31 |
| cdent | at the moment the dispatcher sends a ton of useless resource updates to gnocchi | 21:32 |
| cdent | so the issue is not that I'm trying to protect gnocchi | 21:32 |
| cdent | but rather to stop the dispatcher being stupid | 21:32 |
| gordc | cdent: is the logic currently to just send a patch request to gnocchi, and let gnocchi figure out if needs to process it? | 21:35 |
| cdent | no, it's a good deal more complex than that | 21:36 |
| gordc | :( | 21:36 |
| cdent | complex enough that when I look at it when I made that lame bug that I got a headache and decided it was best to come back later | 21:36 |
| gordc | understood | 21:37 |
| cdent | I was also tired at the time | 21:37 |
| gordc | yeah, i like the idea of caching... i wanted it for sql backend originally. i'm just concerned we didn't actually think it through and it's more a reactionary(?) patch | 21:37 |
| gordc | what happens if we just dispatch generic resources? does it go crazy with updates still? | 21:38 |
| gordc | i'm not super confident with the whole idea yet but that could be because i haven't thought about it until you asked just now | 21:40 |
| gordc | idea == dogpile.cache | 21:40 |
| cdent | I'm not sure I understand what you mean by just...generic resources? | 21:40 |
| gordc | cdent: i can't find the model but https://github.com/openstack/gnocchi/blob/master/gnocchi/indexer/sqlalchemy_base.py#L215 | 21:43 |
| *** fawadkhaliq has joined #openstack-ceilometer | 21:43 | |
| gordc | is it the fact we have to update attr/history that makes it go nuts with updates? | 21:44 |
| cdent | well what it does in its most lame form is make an update for every meter | 21:44 |
| cdent | but if you have a block of them it will try to group the meters by resource | 21:45 |
| cdent | but since the collector doesn't get stuff in batches | 21:45 |
| cdent | it has no memory of what it sent recently | 21:45 |
| gordc | cdent: understood... (there is an (stupid?) accumulator transformer in pipeline that will batch things) | 21:46 |
| gordc | cdent: i was just asking because i'm not sure if it's the storage part that is slow or the indexer part | 21:47 |
| gordc | or both. | 21:47 |
| cdent | storage (which is just measures) is not slow | 21:48 |
| cdent | nor is the indexer | 21:48 |
| cdent | not trying to prevent slowness | 21:48 |
| cdent | trying to prevent duplicated data | 21:48 |
| cdent | on both side | 21:48 |
| cdent | s | 21:48 |
| *** fawadkhaliq has quit IRC | 21:50 | |
| gordc | the cache will block duplicated data? why would we have duplicated data? | 21:51 |
| *** ankita_wagh has quit IRC | 21:58 | |
| *** ankita_wagh has joined #openstack-ceilometer | 21:59 | |
| *** ankita_wagh has quit IRC | 22:00 | |
| *** ankita_wagh has joined #openstack-ceilometer | 22:00 | |
| cdent | duplicated resource information | 22:02 |
| cdent | we don't want to send a request about a resource changing if the resource hasn't actually changed | 22:02 |
| *** edmondsw has quit IRC | 22:06 | |
| gordc | got it... this is a similar issue we had with the sql backend... if we implement your cache proposal, we need another global storage layer (redis). but i guess there's less issue since gnocchi is the promoted db | 22:08 |
| cdent | redis or memcache or something is already required for tooz-based coordination... | 22:10 |
| gordc | yeah... just recapping history. it's wonderful journey. | 22:12 |
| gordc | cdent: it seems like this is needed to improve gnocchi so i'm not against it... it's matter of do we have time and is this solution the right way to implement the cache. | 22:12 |
| gordc | it's similar enough to the hash stuff in sql that i think it's right... so i guess the question is, do you/someone have time. | 22:13 |
| cdent | i'm gonna go with a solid maybe | 22:14 |
| gordc | i'll put your name down on the bp | 22:15 |
| cdent | oh another thing gordc: can you put in a showing at the monasca meeting this coming tuesday? I'm going to be away so won't be able to do it and I made some gestures of "let's work together" on the os-dev list that were well received | 22:18 |
| *** pradk has quit IRC | 22:18 | |
| * gordc just read mailing list | 22:21 | |
| gordc | i should be around | 22:21 |
| cdent | it is time for me to die | 22:24 |
| cdent | back tomorrow, with less death | 22:25 |
| *** cdent has quit IRC | 22:25 | |
| *** gordc has quit IRC | 22:31 | |
| *** ankita_wagh has quit IRC | 22:32 | |
| *** ankita_wagh has joined #openstack-ceilometer | 22:32 | |
| *** ankita_wagh has quit IRC | 22:34 | |
| *** ankita_wagh has joined #openstack-ceilometer | 22:34 | |
| *** rbak has quit IRC | 22:37 | |
| *** devlaps has joined #openstack-ceilometer | 22:44 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!