| openstackgerrit | A change was merged to stackforge/pycadf: Python 3: update setup.cfg to advertise python 3 compatibility https://review.openstack.org/66468 | 00:03 |
|---|---|---|
| *** flwang has quit IRC | 00:03 | |
| *** herndon has quit IRC | 00:05 | |
| *** flwang has joined #openstack-ceilometer | 00:19 | |
| *** xuhanp has quit IRC | 00:31 | |
| openstackgerrit | Luis A. Garcia proposed a change to openstack/ceilometer: Sync gettextutils from Oslo https://review.openstack.org/66982 | 00:55 |
| openstackgerrit | Luis A. Garcia proposed a change to openstack/ceilometer: Re-enable lazy translation https://review.openstack.org/66983 | 00:55 |
| *** gordc has joined #openstack-ceilometer | 01:01 | |
| *** boris-42 has quit IRC | 01:05 | |
| *** gordc has quit IRC | 01:09 | |
| *** gordc has joined #openstack-ceilometer | 01:10 | |
| openstackgerrit | liusheng proposed a change to openstack/python-ceilometerclient: return sample info when creating sample with CLI https://review.openstack.org/66125 | 01:26 |
| openstackgerrit | liusheng proposed a change to openstack/python-ceilometerclient: return sample info when creating sample with CLI https://review.openstack.org/66125 | 01:26 |
| *** xuhanp has joined #openstack-ceilometer | 01:37 | |
| *** gordc has quit IRC | 01:45 | |
| *** gordc has joined #openstack-ceilometer | 02:05 | |
| *** gordc has quit IRC | 02:05 | |
| *** prad has joined #openstack-ceilometer | 02:14 | |
| *** herndon has joined #openstack-ceilometer | 02:30 | |
| *** flwang has quit IRC | 03:04 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Add resource loader support https://review.openstack.org/58747 | 03:33 |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Added hardware agent's inspector and snmp implementation https://review.openstack.org/43073 | 03:46 |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Added hardware pollsters for the central agent https://review.openstack.org/43074 | 03:47 |
| *** prad has quit IRC | 03:48 | |
| *** herndon has quit IRC | 04:19 | |
| *** boris-42 has joined #openstack-ceilometer | 04:30 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 04:59 | |
| *** asalkeld has quit IRC | 05:51 | |
| *** asalkeld has joined #openstack-ceilometer | 06:04 | |
| *** flwang has joined #openstack-ceilometer | 06:14 | |
| *** asalkeld has quit IRC | 06:15 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/62808 | 06:22 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 06:22 | |
| *** chmouel has quit IRC | 06:24 | |
| *** chmouel has joined #openstack-ceilometer | 06:27 | |
| *** nadya_ has joined #openstack-ceilometer | 06:51 | |
| *** asalkeld_ has joined #openstack-ceilometer | 06:56 | |
| *** Alexei_987 has left #openstack-ceilometer | 07:06 | |
| *** asalkeld_ has quit IRC | 07:20 | |
| *** nadya_ has quit IRC | 07:30 | |
| *** asalkeld_ has joined #openstack-ceilometer | 07:33 | |
| *** boris-42 has quit IRC | 07:43 | |
| *** ildikov_ has quit IRC | 07:45 | |
| *** ildikov_ has joined #openstack-ceilometer | 08:05 | |
| *** asalkeld_ has quit IRC | 08:36 | |
| *** yfujioka has joined #openstack-ceilometer | 08:52 | |
| *** Alexei_987 has joined #openstack-ceilometer | 09:00 | |
| *** _ruhe is now known as ruhe | 09:03 | |
| nprivalova | jd__, hi! last time you did some magic and make Yassine Lamgarchal available. could you please do it again :)? | 09:09 |
| jd__ | sure | 09:09 |
| jd__ | nprivalova: it seems he's not arrived at his office yet, I've just mailed him so it join when he's ready :) | 09:10 |
| nprivalova | jd__, ok, thank you! | 09:11 |
| *** yassine has joined #openstack-ceilometer | 09:18 | |
| *** ityaptin has joined #openstack-ceilometer | 09:44 | |
| *** RemyS has joined #openstack-ceilometer | 09:47 | |
| RemyS | morning | 09:47 |
| *** flwang has quit IRC | 09:58 | |
| *** ruhe is now known as ruhe_away | 09:59 | |
| *** boris-42 has joined #openstack-ceilometer | 10:03 | |
| *** ruhe_away is now known as _ruhe | 10:09 | |
| *** Alexei_987 has quit IRC | 10:11 | |
| *** Alexei_987 has joined #openstack-ceilometer | 10:12 | |
| *** sayali has joined #openstack-ceilometer | 10:15 | |
| *** sayali has quit IRC | 10:17 | |
| *** sayali has joined #openstack-ceilometer | 10:18 | |
| *** sayali_ has joined #openstack-ceilometer | 10:18 | |
| *** RemyS has quit IRC | 10:18 | |
| *** sayali has quit IRC | 10:19 | |
| Alexei_987 | sileht: ping | 10:22 |
| sileht | Alexei_987, pong | 10:23 |
| Alexei_987 | sileht: do you have time to talk about this sqla bug? | 10:24 |
| sileht | Alexei_987, yes | 10:24 |
| Alexei_987 | oslo.db bug* | 10:24 |
| Alexei_987 | sileht: so how do you plan to fix it? | 10:24 |
| Alexei_987 | sileht: we discussed this issue yesterday and we think that we should not apply fix in oslo | 10:25 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 10:25 | |
| Alexei_987 | sileht: and try to fix it locally | 10:25 |
| sileht | Alexei_987, I have started to split integration test to create on tox env per DB backend | 10:26 |
| Alexei_987 | sileht: :( I think that's a bad approach | 10:26 |
| Alexei_987 | sileht: it will cause tests to run much slower | 10:26 |
| Alexei_987 | sileht: cause of env creation | 10:26 |
| sileht | Alexei_987, yes but some people want to add other real database testing for cassanda, hbase, etc... | 10:27 |
| Alexei_987 | yes and we can do this cleanly with testscenarios | 10:27 |
| sileht | Alexei_987, and we cannot run all the databases at the same time in one VM | 10:27 |
| Alexei_987 | why not? | 10:27 |
| Alexei_987 | connection url doesn't have to point to the same VM | 10:27 |
| Alexei_987 | and storage can be located elsewhere | 10:28 |
| sileht | Alexei_987, not for gate | 10:28 |
| Alexei_987 | sileht: and how separate envs will help with that? | 10:29 |
| sileht | Alexei_987, if we start only one scenario in a VM we can that will reduce the resource usage of the VM | 10:30 |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Update oslo https://review.openstack.org/65661 | 10:30 |
| Alexei_987 | sileht: I'm afraid that resource usage will be even bigger cause of env setup :( | 10:31 |
| Alexei_987 | env setup takes much more time than actual tests run | 10:32 |
| sileht | Alexei_987, for local testing that can be tweaked to have a common virtualenv | 10:34 |
| sileht | Alexei_987, but I agree for gate it will take more time :( | 10:34 |
| Alexei_987 | sileht: so maybe we should try to find better approach? | 10:35 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 10:35 | |
| Alexei_987 | sileht: do we have other problems except this oslo.db bug preventing us from using multiple backends inside of one process? | 10:36 |
| sileht | Alexei_987, the problem to run all scenarios in one VM, is that we must change the oslo.db singleton API to a better API | 10:36 |
| Alexei_987 | sileht: this can be fixed I have a plan for a workaround | 10:36 |
| sileht | Alexei_987, how , I really interresting :) | 10:36 |
| Alexei_987 | sileht: we could manage engine creation on our own | 10:37 |
| Alexei_987 | sileht: basically we do it for nosql backends | 10:37 |
| Alexei_987 | sileht: if we'll avoid this oslo.db get_session stuff we won't have any problems | 10:37 |
| Alexei_987 | sileht: it should only require some changes in our impl_sqla | 10:38 |
| Alexei_987 | I can make a draft patch for this if you are interested | 10:38 |
| sileht | Alexei_987, that will duplicate code for for engine/session configuration | 10:39 |
| Alexei_987 | sileht: some of the code can be reused so I don't think that this will be a problem | 10:39 |
| Alexei_987 | sileht: I don't plan to write a new sqla wrapper | 10:39 |
| Alexei_987 | we could bypass oslo.db singleton by calling it's cleanup method | 10:40 |
| Alexei_987 | so the plan is: 1) use oslo.db to create engine 2) save engine locally 3) delete ref in oslo.db (this is needed to prevent engine disconnect) 4) call oslo.db.cleanup() | 10:41 |
| Alexei_987 | in such way we'll be able to have multiple connections at the same time | 10:41 |
| Alexei_987 | sileht: ^ what do you think? | 10:42 |
| sileht | I'm not sure it works many parts of the oslo.db code use the CONF.database.connection, what happend if this one change at run time | 10:43 |
| Alexei_987 | sileht: well I'll make a patch ready and we'll see | 10:45 |
| Alexei_987 | in production env connection string should not change | 10:45 |
| Alexei_987 | so it will only work for tests | 10:45 |
| sileht | sure | 10:46 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 10:51 | |
| *** _ruhe is now known as ruhe | 10:52 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Implement the /v2/samples/<sample-id> API https://review.openstack.org/61814 | 10:55 |
| *** xuhanp has quit IRC | 11:03 | |
| *** yfujioka has quit IRC | 11:14 | |
| *** nsaje has quit IRC | 11:17 | |
| *** nsaje_ has joined #openstack-ceilometer | 11:17 | |
| *** asalkeld has joined #openstack-ceilometer | 11:22 | |
| nprivalova | nsaje_, hi! How are you? Yesterday we had a conversation about tempest+ceilometer stuff. We still have a chance to commit https://review.openstack.org/#/c/39237/. Will you have time to rewrite it using https://review.openstack.org/#/c/55276/ ? | 11:30 |
| *** sayali_ has quit IRC | 12:03 | |
| *** kwhitney has left #openstack-ceilometer | 12:06 | |
| Alexei_987 | sileht: can I ask you a stupid question? | 12:23 |
| openstackgerrit | Dirk Mueller proposed a change to openstack/python-ceilometerclient: Remove dependencies on pep8, pyflakes and flake8 https://review.openstack.org/67136 | 12:24 |
| Alexei_987 | jd__: are you around? | 12:32 |
| openstackgerrit | Dirk Mueller proposed a change to openstack/ceilometer: Remove dependencies on pep8, pyflakes and flake8 https://review.openstack.org/67144 | 12:38 |
| *** ruhe is now known as ruhe_away | 12:53 | |
| *** ruhe_away is now known as ruhe | 12:54 | |
| *** sayali has joined #openstack-ceilometer | 12:55 | |
| *** sayali_ has joined #openstack-ceilometer | 12:55 | |
| *** prad has joined #openstack-ceilometer | 12:59 | |
| *** prad has quit IRC | 13:04 | |
| *** julienvey_ has joined #openstack-ceilometer | 13:06 | |
| *** flwang has joined #openstack-ceilometer | 13:15 | |
| jd__ | Alexei_987: more or less? | 13:25 |
| *** prad has joined #openstack-ceilometer | 13:29 | |
| *** xuhanp has joined #openstack-ceilometer | 13:30 | |
| *** jdob has joined #openstack-ceilometer | 13:30 | |
| *** prad has quit IRC | 13:34 | |
| *** ruhe is now known as ruhe_away | 13:41 | |
| *** ruhe_away is now known as ruhe | 13:44 | |
| *** jaypipes has joined #openstack-ceilometer | 13:53 | |
| Alexei_987 | jd__: wanted to ask do you know why we use subratransactions in sqla ceilometer storage method "create_or_update"? | 13:59 |
| jd__ | Alexei_987: no idea | 13:59 |
| Alexei_987 | jd__: Ok thanks have to figure out this somehow | 13:59 |
| *** ruhe is now known as ruhe_away | 13:59 | |
| *** eglynn has joined #openstack-ceilometer | 14:00 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Update oslo https://review.openstack.org/65661 | 14:01 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Exclude weak datapoints from alarm threshold evaluation https://review.openstack.org/67161 | 14:01 |
| eglynn | jd__: quick question on the new notification agent? | 14:25 |
| jd__ | eglynn: shoot | 14:25 |
| eglynn | jd__: so seems logical that each message on a notifications.info topic is only delivered to a single ceilometer subscriber | 14:26 |
| jd__ | right | 14:26 |
| eglynn | (otherwise we wouldn't be able to horizontally scale the ceilo notification agent) | 14:26 |
| eglynn | yep, so what if something outside ceilo also wanted visibility on these notifications? | 14:27 |
| jd__ | "I'm glad you asked this" | 14:27 |
| eglynn | :) | 14:27 |
| jd__ | in this case your notification sender must configure several notification topics | 14:27 |
| jd__ | to publisher the messages to different queues | 14:27 |
| eglynn | the sender in this case being the openstack services? | 14:28 |
| jd__ | yep | 14:28 |
| jd__ | nova, glance, whatever | 14:28 |
| jd__ | e.g. notification_topics=notifications,notification-for-somebody-else | 14:28 |
| jd__ | IIRC | 14:28 |
| jd__ | as Ceilometer by default consumes for notifications.* | 14:29 |
| jd__ | (well, .info only actually..) | 14:29 |
| eglynn | a-ha, I see, so those config options are generally lists? | 14:29 |
| jd__ | yes | 14:30 |
| jd__ | topicS :-) | 14:30 |
| eglynn | jd__: cool, thank you sir! | 14:31 |
| jd__ | you're welcome :) | 14:31 |
| eglynn | jd__: one other quickie while I've got you ... | 14:32 |
| eglynn | jd__: ... possible to resurrect https://blueprints.launchpad.net/ceilometer/+spec/exclude-low-sample-count back into icehouse-2 d'ya think? | 14:32 |
| jd__ | eglynn: is the a review yet? I'm still lagging a bit | 14:32 |
| eglynn | jd__: yep ...https://review.openstack.org/67161 | 14:32 |
| eglynn | (... just proposed a little while ago) | 14:33 |
| jd__ | you're back on i2 | 14:33 |
| eglynn | jd__: ... and again, thank you sir! | 14:34 |
| jd__ | np :) | 14:34 |
| *** SergeyLukjanov is now known as SergeyLukjanov_a | 14:35 | |
| *** prad has joined #openstack-ceilometer | 14:42 | |
| nprivalova | jd__, I cannot run tests on real mysql, right? | 14:46 |
| jd__ | nprivalova: we have a patch for that but it's not working yet | 14:47 |
| nprivalova | jd__, maybe you help me then... I run "tox -epy27 tests.storage.test_impl_sqlalchemy" where can I find sqlite file? (imagine that it will not be removed after test run) | 14:49 |
| jd__ | nprivalova: it runs in memory actually | 14:52 |
| jd__ | so you wan't find any file, sorry | 14:52 |
| nprivalova | fail... | 14:52 |
| jd__ | :-) | 14:54 |
| nprivalova | jd__, thank you :) Hope I will find another way to check my aggregation. I wanted to see tables for sure. Will try just to read from new tables | 14:55 |
| jd__ | nprivalova: yeah unfortunately it's kind of tricky to debug that way indeed | 14:56 |
| jd__ | nprivalova: what you can try to do is to specify a sqlite:// URL with a file, run just ONE test and add a sleep(100000) to stop the test and have time to copy the SQL file :) | 14:57 |
| jd__ | just a hackish way of getting what you want :) | 14:57 |
| nprivalova | jd__, I tried this scenario. but didn't find a file anyway | 14:58 |
| jd__ | nprivalova: you need to change the sqlite:// url in ceilometer/tests/db.py to point to a file | 14:58 |
| jd__ | as sqlite:// means memory | 14:58 |
| nprivalova | jd__, | 14:59 |
| nprivalova | jd__, ye, I did everything you described. test contains the line database_connection = 'mysql://localhost', maybe I should remove it too? | 14:59 |
| *** boris-42 has quit IRC | 15:00 | |
| *** julienvey_ has quit IRC | 15:00 | |
| *** SergeyLukjanov_a is now known as SergeyLukjanov | 15:02 | |
| *** sayali has quit IRC | 15:03 | |
| prad | could anyone tell me why jenkins is failing my patch on this error http://logs.openstack.org/75/66875/1/check/gate-ceilometer-python27/74c0ae2/console.html#_2014-01-15_20_59_10_853 | 15:04 |
| prad | this is suppose to be a cherry pick from master so shouldn't be any different and master merged successfully | 15:05 |
| nprivalova | prad, rebase on master | 15:05 |
| prad | ah ok, will give that a try thx | 15:06 |
| nprivalova | prad, it should be fixed by https://review.openstack.org/#/c/66402/ | 15:06 |
| prad | nprivalova: just to make sure ..so you're suggesting rebase my stable/havana with this in master? | 15:07 |
| prad | nprivalova: the failure is in stable/havana | 15:07 |
| nprivalova | prad, hmm, please give a link on review | 15:08 |
| prad | nprivalova: https://review.openstack.org/#/c/66875/ | 15:08 |
| prad | nprivalova: this is just a back port from master | 15:09 |
| prad | there is another stable/havana patch failing similarly as well https://review.openstack.org/#/c/66861/ | 15:10 |
| prad | so i'm hoping its not just me | 15:10 |
| openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Implements "not" operator for complex query https://review.openstack.org/66892 | 15:11 |
| nprivalova | prad, I see. Looks like keystone-client was backported too. Some change there causing the problem. Let me find the commit | 15:11 |
| prad | nprivalova: thx | 15:12 |
| *** kwhitney1 has quit IRC | 15:12 | |
| nprivalova | prad, check https://bugs.launchpad.net/nova/+bug/1268614 | 15:12 |
| nprivalova | prad, "Looks like it's due to updated python-keystoneclient. With python-keystoneclient 0.4.1 check_uptodate.sh works just fine. Keystoneclient 0.4.2 introduces include_service_catalog option in commit a97b293501fa504dd154fc921809a40bc2a34049." | 15:12 |
| *** kwhitney1 has joined #openstack-ceilometer | 15:12 | |
| prad | nprivalova: ah i see | 15:13 |
| nprivalova | prad, you need to be sure that keystone client was not backported to stable/havana. if it is you need to backport https://review.openstack.org/#/c/66402/ too | 15:13 |
| *** jmckind has joined #openstack-ceilometer | 15:15 | |
| ityaptin | jd__ hi! I again want ask to have a look blueprint https://blueprints.launchpad.net/ceilometer/+spec/run-all-pollsters-on-demand. | 15:17 |
| ityaptin | jd__, if you have any issues about it, tell me, please. | 15:17 |
| jd__ | ildikov_: k, looks good to me, can you set the implementation status and possibly a milestone target? | 15:18 |
| ildikov_ | jd__: does this "looks good to me " refer to my complex query implementation? ;) | 15:22 |
| * jd__ runs | 15:24 | |
| ildikov_ | jd__: any +2 is welcomed :) | 15:24 |
| jd__ | I know I really want to review code but they won't let me | 15:24 |
| nprivalova | who? we may beat them | 15:26 |
| ildikov_ | jd__: who should I force to let you? | 15:26 |
| prad | can i get some eyes on https://review.openstack.org/#/c/66746/ and https://review.openstack.org/#/c/66757/ please :) | 15:26 |
| gibi | I guess we can form a team :) | 15:26 |
| jd__ | ah, if I only knew who they are… ;-) | 15:27 |
| ildikov_ | nprivalova: +1 | 15:27 |
| *** gordc has joined #openstack-ceilometer | 15:34 | |
| *** herndon has joined #openstack-ceilometer | 15:36 | |
| prad | nprivalova: would "instance name" be more apt? the data i see doesn't seem like description of the instance | 15:37 |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix use the fact that empty sequences are false https://review.openstack.org/64768 | 15:40 |
| *** julienvey_ has joined #openstack-ceilometer | 15:41 | |
| nprivalova | prad, here should be a description what meter is about... let me think | 15:41 |
| prad | nprivalova: ok i'm open to suggestions.. if description seems best fit i'm ok with it, but curious how it fits the instance:<type> .. would we tell use its a instance description with type ? | 15:46 |
| prad | s/use/user | 15:46 |
| nprivalova | prad, as I understand instance:<type> is the same as instance but with explicit type. I may suggest to wait for more variants from team | 15:50 |
| prad | sure, np i'll wait for some more input | 15:51 |
| prad | thx! | 15:51 |
| *** boris-42 has joined #openstack-ceilometer | 15:59 | |
| *** sayali has joined #openstack-ceilometer | 16:03 | |
| *** ruhe_away is now known as ruhe | 16:04 | |
| *** sayali has quit IRC | 16:07 | |
| *** sayali has joined #openstack-ceilometer | 16:08 | |
| *** xuhanp has quit IRC | 16:09 | |
| openstackgerrit | A change was merged to openstack/ceilometer: tests: pass /dev/null as config for mongod https://review.openstack.org/65473 | 16:23 |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Remove unused imports https://review.openstack.org/66118 | 16:24 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 16:24 | |
| jd__ | ildikov_: around? | 16:29 |
| *** julienvey_ has quit IRC | 16:29 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 16:30 | |
| ildikov_ | jd__: yes | 16:31 |
| jd__ | ildikov_: quick question, why jsonschema? | 16:31 |
| *** julienvey_ has joined #openstack-ceilometer | 16:32 | |
| ildikov_ | jd__: it was mentioned by boris-42 | 16:32 |
| jd__ | ildikov_: ok | 16:32 |
| ildikov_ | jd__: it makes the validation of the query much easier | 16:32 |
| boris-42 | ildikov_ +1 | 16:32 |
| jd__ | ildikov_: definitely | 16:33 |
| ildikov_ | jd__: and also more generic as the tree traversal is not a hard coded mechanism anymore in case of the validation | 16:33 |
| boris-42 | ildikov_ btw sorry didn't have time to look one more time on path.. | 16:33 |
| jd__ | my main issue is that we're trying to regroup schema validation around one library, and not jsonschema in particular that is | 16:33 |
| ildikov_ | jd__: if you check the patch for field_name validation it shows well, that adding this functionality was a few lines | 16:34 |
| jd__ | so I'm not sure both dhellmann and I are going to be totally happy about that particular choice, this is something we should discuss I guess | 16:34 |
| jd__ | ildikov_: yeah you definitely want a validation library, it's just the lib choice that I might not be comfortable with | 16:34 |
| boris-42 | jd__ imho | 16:35 |
| boris-42 | jd__ we shouldn't make our own oslo.jsonschema | 16:35 |
| boris-42 | jd__ this is well know and great lib | 16:35 |
| jd__ | boris-42: we're not | 16:35 |
| ildikov_ | jd__: which would be the lib? | 16:35 |
| jd__ | boris-42: we also have other things than JSON to validate, so jsonschema does sounds like it's the best choice | 16:35 |
| jd__ | ildikov_: Colander looks like a good candidate for now | 16:36 |
| boris-42 | jd__ imho we should have only one format to valide | 16:36 |
| jd__ | ildikov_: I'd love to hear your opinion if you have time to take a look since you used jsonschema already | 16:36 |
| boris-42 | jd__ and it's JSON | 16:36 |
| jd__ | boris-42: looks like your opinion and reality do not agree then :) | 16:36 |
| boris-42 | jd__ idk why we should validate soothing else then json? | 16:37 |
| boris-42 | jd__ why we can't remove support of XML and use only json | 16:37 |
| boris-42 | jd__ ? | 16:37 |
| jd__ | boris-42: because everything is not JSON? | 16:37 |
| ildikov_ | jd__: I saw your Colander patch, I did not have time to have a deeper look into it, so I did not know that it will be suitable for us or no | 16:37 |
| jd__ | boris-42: there are other places than API where we need to validate data | 16:37 |
| boris-42 | jd__ why this data is not in JSON? | 16:37 |
| boris-42 | jd__ probably it should be refactored? | 16:38 |
| jd__ | ildikov_: yeah fair enough, it's just that if you have time to think about it and feed us with your opinion that would be valuable | 16:38 |
| boris-42 | jd__ to switch to rally | 16:38 |
| boris-42 | jd__ to json** | 16:38 |
| jd__ | boris-42: in a database? :) | 16:38 |
| boris-42 | jd__ use mongo+) | 16:38 |
| jd__ | /kick boris-42 | 16:38 |
| jd__ | :) | 16:39 |
| boris-42 | lol) | 16:39 |
| ildikov_ | jd__: I will have a deeper look, I saw that it seems to support our json format, but I cannot tell you now this for sure | 16:39 |
| jd__ | ildikov_: thanks | 16:39 |
| ildikov_ | jd__: I also wondering if Colander will be good also for field_name validation or it will check only the syntax of the incoming query | 16:41 |
| ildikov_ | jd__: and we also had some issues to figure out how to implement the recursion into the json schema, I have to investigate this part in COlander also | 16:42 |
| ildikov_ | jd__: what is the timing for Colander, is there any targeted release? | 16:42 |
| jd__ | ildikov_: if you have time to check if there's any big problem with all that would definitely helps | 16:42 |
| jd__ | ildikov_: there's no timing per se, we're just trying to stop the usage of millions of libs | 16:43 |
| jd__ | rationalizing a bit | 16:43 |
| ildikov_ | jd__: I would be happy if there wouldn't be a need for converting our incoming filter expressions to string just because wsme does not support complex structures in that way, as we would like to use it | 16:45 |
| ildikov_ | jd__: of course we cannot change right now, but for the future that would be better | 16:45 |
| jd__ | ildikov_: well that's part of my patch with Colander, I'd like to build a WSME/Colander layer too | 16:46 |
| ildikov_ | jd__: we've found a bug report regarding to recursion in Colander, we will check the details and the mentioned workaround | 16:46 |
| *** sayali has quit IRC | 16:46 | |
| jd__ | ildikov_: basically if we can leverage _one_ lib everywhere to validate data, it'd be a huge win | 16:46 |
| ildikov_ | jd__: does this mean that the other projects, who's already using json schema will also change to the common way? | 16:47 |
| openstackgerrit | John Herndon proposed a change to openstack/python-ceilometerclient: Update client to display data type of traits https://review.openstack.org/67224 | 16:50 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Exclude weak datapoints from alarm threshold evaluation https://review.openstack.org/67161 | 16:50 |
| herndon | eglynn: ceilometer client review is posted: https://review.openstack.org/67224 | 16:50 |
| eglynn | herndon: cool, looking ... | 16:50 |
| herndon | it is dependent on this review from ceilometer: https://review.openstack.org/#/c/66936/ | 16:51 |
| jd__ | ildikov_: AFAIK the ones using it are doing so in the API part and should/were supposed to switch to WSME at some point | 16:51 |
| ildikov_ | jd__: just to see it clearly, as at this point I may get lost a bit, the WSME/Colander layer you mentioned means something like the first layer of the API is WSME and the next layer downwards would be Colander? | 16:55 |
| jd__ | ildikov_: I meant Pecan/Colander, sorry | 16:56 |
| ildikov_ | jd__: ah, ok, np :) | 16:56 |
| ildikov_ | jd__: how should we proceed now with our bp implementation for i-2? | 16:57 |
| jd__ | ildikov_: wait | 16:58 |
| jd__ | ildikov_: but I gave you some hints to keep you busy though ;-) | 16:58 |
| ildikov_ | jd__: can leave the current implementation as is and trying Colander on a separate branch to see how it works for more complex structures and then we can discuss the outcome of our trial? | 16:58 |
| jd__ | ildikov_: definiteively | 16:59 |
| jd__ | or definitively | 16:59 |
| jd__ | "good idea!" | 16:59 |
| jd__ | :-) | 16:59 |
| ildikov_ | jd__: LOL :) | 16:59 |
| jd__ | ildikov_: I am definitely not asking you to rewrite everything from scratch with $OTHERLIB | 16:59 |
| jd__ | we just need a bit of (your) time to see if our ultimate solution to use one lib (i.e. Colander) to do the same job over and over is viable | 17:00 |
| jd__ | if you can help us with that and find out that you could, it would be awesome | 17:00 |
| *** ruhe is now known as ruhe_away | 17:00 | |
| ildikov_ | jd__: well, our query expression is an excellent candidate for this trial | 17:01 |
| jd__ | ildikov_: I think so too | 17:01 |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 17:01 | |
| ildikov_ | jd__: can our current bp implementation stay in the picture for i-2? | 17:02 |
| jd__ | ildikov_: sure, though honestly the code looks big and complicated I'm not sure it'll be entirely reviewed by then | 17:03 |
| ildikov_ | jd__: if we change only the validation part it will be invisible for outside, if the test are still green after changing to Colander :) | 17:03 |
| jd__ | but I'll leave it planned for i2 anyway | 17:03 |
| jd__ | ildikov_: I nod! | 17:03 |
| ildikov_ | jd__: the patch sets are vaible one-by-one | 17:03 |
| ildikov_ | jd__: I mean the first one is big because of the validation itself and it works for samples | 17:04 |
| jd__ | ildikov_: yeah, you might get a few merged but not everything -- I don't know, I'm just don't want you to have too big expectations :) | 17:04 |
| ildikov_ | jd_: the second one is reusing the whole common code for alarms, and third is also for alarm history | 17:04 |
| ildikov_ | jd__: the forth is a small one, only for field_name validation | 17:05 |
| jd__ | ack | 17:05 |
| ildikov_ | and the other can be retargeted for i-3 as the first 4 gives a quite well usabale functionality | 17:05 |
| ildikov_ | jd__: s/other/others/ | 17:06 |
| ildikov_ | jd__: if the first 4 could be merged, we would happly test Colander ;) | 17:08 |
| jd__ | ildikov_: well if you can test in the meantime that'd be even more awesome :) but I get your point | 17:08 |
| jd__ | I can't promise anything, I'm just a reviewer :) | 17:09 |
| openstackgerrit | John Herndon proposed a change to openstack/ceilometer: Return trait type from Event api https://review.openstack.org/66936 | 17:09 |
| *** ruhe_away is now known as _ruhe | 17:10 | |
| ildikov_ | jd__: but maybe you can ask the other reviewers to have a look at, with expecting some good points ;) | 17:10 |
| jd__ | ildikov_: or I could blackmail you… ;) | 17:11 |
| openstackgerrit | John Herndon proposed a change to openstack/ceilometer: Return trait type from Event api https://review.openstack.org/66936 | 17:11 |
| ildikov_ | jd__: should I be worried now? ;) | 17:13 |
| jd__ | ildikov_: hehehehe ;) | 17:13 |
| gibi | ildikov_: don't worry you are not alone agains jd__ :) | 17:14 |
| ildikov_ | jd__: anyway, we will test Colander, as it looked good after the first quick look at the home page, and then we will see what's behind the nice website | 17:15 |
| jd__ | ildikov_: thumbs up :) | 17:16 |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 17:19 | |
| *** julienvey_ has quit IRC | 17:20 | |
| ildikov_ | jd__: I will inform you, if we have any results with the testing | 17:27 |
| *** Alexei_987 has quit IRC | 17:29 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_ | 17:44 | |
| *** eglynn has quit IRC | 17:53 | |
| openstackgerrit | John Herndon proposed a change to openstack/ceilometer: Adds documentation for the Event API https://review.openstack.org/57353 | 18:00 |
| openstackgerrit | John Herndon proposed a change to openstack/ceilometer: Update dev docs to include notification-agent https://review.openstack.org/67252 | 18:04 |
| *** ildikov_ has quit IRC | 18:06 | |
| *** _ruhe is now known as ruhe | 18:25 | |
| *** gordc has quit IRC | 18:28 | |
| *** eglynn has joined #openstack-ceilometer | 18:28 | |
| *** yassine has quit IRC | 18:33 | |
| *** eglynn has quit IRC | 18:35 | |
| *** gordc has joined #openstack-ceilometer | 18:42 | |
| *** ruhe is now known as ruhe_away | 18:45 | |
| *** ruhe_away is now known as ruhe | 18:48 | |
| *** openstackgerrit has quit IRC | 18:52 | |
| *** openstackgerrit has joined #openstack-ceilometer | 18:52 | |
| *** tongli has joined #openstack-ceilometer | 18:55 | |
| *** SergeyLukjanov_ is now known as SergeyLukjanov | 19:03 | |
| *** ildikov_ has joined #openstack-ceilometer | 19:12 | |
| *** herndon has quit IRC | 19:15 | |
| *** Alexei_987 has joined #openstack-ceilometer | 19:28 | |
| *** xmltok has joined #openstack-ceilometer | 19:37 | |
| *** tongli has quit IRC | 21:41 | |
| *** asalkeld has quit IRC | 21:46 | |
| *** ildikov_ has quit IRC | 21:54 | |
| *** asalkeld has joined #openstack-ceilometer | 21:59 | |
| *** jdob has quit IRC | 22:05 | |
| *** ruhe is now known as _ruhe | 22:06 | |
| *** SergeyLukjanov is now known as SergeyLukjanov_a | 22:09 | |
| *** SergeyLukjanov_a is now known as SergeyLukjanov_ | 22:10 | |
| *** prad has quit IRC | 22:12 | |
| *** jaypipes has quit IRC | 22:21 | |
| *** gordc has quit IRC | 22:52 | |
| *** adriant_ has joined #openstack-ceilometer | 23:07 | |
| adriant_ | Anyone here with a knowledge of pipelines+transformers able to help with some implementation questions? | 23:08 |
| nijaba | an interesting blog on using transformers http://cjchand.wordpress.com/2014/01/16/transforming-cumulative-ceilometer-stats-to-gauges/ | 23:08 |
| adriant_ | Thanks, interesting stuff, but not really what I need sadly. | 23:10 |
| adriant_ | I've built my own transformer, that takes a gauge and make a cumulative. | 23:11 |
| nijaba | shoot your question, someon will eventually read and answer | 23:11 |
| nijaba | lots of europeans in this chan, so be patient | 23:11 |
| *** julienvey_ has joined #openstack-ceilometer | 23:12 | |
| adriant_ | But the issue I've run into it is that ceilometer spins up two transformer objects, since the gauge meter I am transforming is both a notifier and a pollster. So the issue is the resulting cumulative meter has inconsistent entries due to the two different transformer objects publishing to it. | 23:13 |
| *** julienvey_ has quit IRC | 23:14 | |
| adriant_ | nijaba: thanks, I'll be around all day so that's not a problem. | 23:14 |
| nijaba | adriant_: so publish the second one with a different name, or didn´t understand your question? | 23:15 |
| adriant_ | I'm currently building a new metric that has a pollster, and a notification feeding it. That works just fine, and is a gauge. | 23:16 |
| nijaba | ok, so you may have to wait for someone that knows about this better than I do :) | 23:17 |
| adriant_ | But I want to build a pipeline+transformer that builds a cumulative value from that metric, but it seems that the transformer is being run on the data coming in from the pollster, as well as the notifier, but separately. | 23:17 |
| adriant_ | Yeah, it's a weird issue, and depends heavily on how the hell the transformers work :P | 23:17 |
| * nijaba goes afk. have a good night | 23:19 | |
| *** ok_delta has joined #openstack-ceilometer | 23:40 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!