| *** gordc has quit IRC | 00:02 | |
| *** herndon has quit IRC | 00:02 | |
| *** asalkeld has quit IRC | 00:37 | |
| *** asalkeld has joined #openstack-ceilometer | 00:50 | |
| *** prad has quit IRC | 00:58 | |
| *** boris-42 has quit IRC | 01:39 | |
| *** prad has joined #openstack-ceilometer | 01:52 | |
| *** prad has quit IRC | 01:52 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Change endpoint_type parameter https://review.openstack.org/59669 | 02:04 |
|---|---|---|
| *** sandywalsh has quit IRC | 02:55 | |
| openstackgerrit | Lianhao Lu proposed a change to openstack/ceilometer: Drop foreign key contraints of alarm in sqlalchemy https://review.openstack.org/65018 | 03:45 |
| *** gordc1 has quit IRC | 03:51 | |
| *** terriyu has quit IRC | 04:19 | |
| *** openstackgerrit has quit IRC | 05:01 | |
| *** openstackgerrit has joined #openstack-ceilometer | 05:01 | |
| *** zul has quit IRC | 05:30 | |
| *** zul has joined #openstack-ceilometer | 05:31 | |
| *** sandywalsh has joined #openstack-ceilometer | 06:01 | |
| *** boris-42 has joined #openstack-ceilometer | 06:22 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: AlarmChange definition added to doc/source/webapi/v2.rst https://review.openstack.org/65112 | 06:34 |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/62808 | 07:10 |
| *** urulama has joined #openstack-ceilometer | 07:29 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Add a wadl target to the documentation https://review.openstack.org/57574 | 07:41 |
| openstackgerrit | A change was merged to openstack/ceilometer: Remove redundant code in nova_client.Client https://review.openstack.org/64025 | 07:41 |
| *** nadya_ has joined #openstack-ceilometer | 07:57 | |
| *** s2r2 has left #openstack-ceilometer | 08:19 | |
| *** s2r2 has joined #openstack-ceilometer | 08:22 | |
| *** nadya_ has quit IRC | 08:22 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 08:37 | |
| *** flwang has quit IRC | 09:06 | |
| *** yassine has joined #openstack-ceilometer | 09:18 | |
| *** eglynn has joined #openstack-ceilometer | 09:19 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 09:47 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 10:07 | |
| *** yassine has quit IRC | 10:55 | |
| *** nadya_ has joined #openstack-ceilometer | 11:02 | |
| *** nadya_ has quit IRC | 11:44 | |
| *** SergeyLukjanov has quit IRC | 11:44 | |
| *** kwhitney has joined #openstack-ceilometer | 12:06 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 12:14 | |
| *** yassine has joined #openstack-ceilometer | 12:26 | |
| *** kwhitney has left #openstack-ceilometer | 12:49 | |
| *** kwhitney has joined #openstack-ceilometer | 12:51 | |
| *** urulama has quit IRC | 13:11 | |
| *** thomasem has joined #openstack-ceilometer | 13:18 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarms https://review.openstack.org/62158 | 13:25 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarm history https://review.openstack.org/62159 | 13:25 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for samples https://review.openstack.org/62157 | 13:25 |
| *** gordc has joined #openstack-ceilometer | 13:27 | |
| s2r2 | hi there. By any chance, could this fix be backported to stable/havana? https://bugs.launchpad.net/ceilometer/+bug/1093625 | 13:35 |
| s2r2 | related: how stable is the sqlalchemy connector (in havana)? | 13:35 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Convert alarm timestamp to PrecisionTimestamp https://review.openstack.org/63131 | 13:36 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Put a size for sql Float https://review.openstack.org/63132 | 13:36 |
| *** jdob has joined #openstack-ceilometer | 13:37 | |
| *** _ruhe is now known as ruhe | 13:40 | |
| openstackgerrit | A change was merged to stackforge/pycadf: Python 3: Use six.moves.configparser rather than ConfigParser https://review.openstack.org/65153 | 13:42 |
| openstackgerrit | A change was merged to stackforge/pycadf: Python 3: replace 'basestring' by 'six.string_types' https://review.openstack.org/65155 | 13:43 |
| boris-42 | jd__ ping | 13:46 |
| boris-42 | jd__ could we get approve to this BP https://blueprints.launchpad.net/ceilometer/+spec/profiler-rpc-collector | 13:46 |
| boris-42 | jd__ I changed Alexie approach | 13:46 |
| boris-42 | jd__ soon I will make some demo | 13:46 |
| jd__ | I think it would need more detail and more discussion to be approved | 13:48 |
| jd__ | I can click approved, but it's likely you'll work on something that is going to be rejected | 13:48 |
| boris-42 | jd__ to rejected?) | 13:48 |
| boris-42 | jd__ could we discuss then?) | 13:48 |
| boris-42 | jd__ than* | 13:48 |
| jd__ | write your plan a wiki page and ask for feedbacks on the mailing list | 13:48 |
| boris-42 | jd__ actually without demo it is unclear | 13:49 |
| boris-42 | jd__ for most of people | 13:49 |
| boris-42 | jd__ here is the lib | 13:49 |
| boris-42 | jd__ https://github.com/pboris/osprofiler | 13:49 |
| jd__ | well, write a wiki page and record a screencast if you want :) | 13:50 |
| boris-42 | jd__ omg | 13:50 |
| boris-42 | jd__ okay | 13:50 |
| boris-42 | jd__ but I am still woried about ceilometer | 13:50 |
| boris-42 | jd__ how to get via CLI something like select * from meter; ? | 13:51 |
| jd__ | GET /v2/samples | 13:51 |
| boris-42 | jd__ why samples requires meter? | 13:51 |
| boris-42 | jd__ and seems like samples doesn't return resource_metadata? | 13:52 |
| boris-42 | jd__ http://pastebin.com/ETd1ck1n | 13:52 |
| boris-42 | jd__ where is here resource_metadata?) | 13:52 |
| jd__ | samples does not require meter | 13:54 |
| jd__ | GET /v2/samples is a REST API call, not what you're sending in your pastebin | 13:54 |
| jd__ | and I'm not sure it's in the CLI yet | 13:54 |
| boris-42 | ceilometer sample-list | 13:54 |
| boris-42 | usage: ceilometer sample-list [-q <QUERY>] -m <NAME> [-l <NUMBER>] | 13:54 |
| boris-42 | ceilometer sample-list: error: argument -m/--meter is required | 13:54 |
| jd__ | that's what I just told, it's not implemented | 13:56 |
| jd__ | sample-list is a call to /v2/meters/<meterid> | 13:56 |
| jd__ | so it requires a meter id | 13:56 |
| eglynn | jd__: I thought samples where exposed via the MetersController in the API layer? | 13:56 |
| eglynn | jd__: (as opposed to their being a direct SamplesController decouple from meters) | 13:56 |
| eglynn | jd__: ah yeah, wot you just said ... | 13:57 |
| boris-42 | so can I help to get this implemented? | 13:57 |
| boris-42 | something like select * from meter +wehere_statement; | 13:57 |
| jd__ | eglynn: btw ttx and I are worried that all your bp for i2 are "Not Started" :) | 13:58 |
| boris-42 | jd__ I really need this stuff | 13:58 |
| boris-42 | jd__ because I am able to store profiling data, but not able to get it back.. | 13:58 |
| jd__ | boris-42: then work on it | 13:59 |
| eglynn | jd__: yeah, I've been up to my eyes in productization work before the holidays, but have refocused on upstream development since returning | 13:59 |
| eglynn | jd__: ... will update BP status in LP | 13:59 |
| *** dhellmann is now known as dhellmann_ | 13:59 | |
| jd__ | eglynn: thumbs up | 13:59 |
| openstackgerrit | A change was merged to stackforge/pycadf: Python 3: use six.with_metaclass https://review.openstack.org/65154 | 14:00 |
| *** urulama has joined #openstack-ceilometer | 14:03 | |
| ildikov | jd__, boris-42: related to queries, I've just uploaded the latest version of the patch set for this bp: https://blueprints.launchpad.net/ceilometer/+spec/complex-filter-expressions-in-api-queries | 14:06 |
| boris-42 | ildikov I will try to review them | 14:17 |
| boris-42 | ildikov https://review.openstack.org/#/c/62157/ this one is interesting for me? | 14:17 |
| boris-42 | =) | 14:17 |
| boris-42 | ildikov yep seems like it is the exactly what I need | 14:19 |
| boris-42 | ildikov btw does it returns "resource_metadata"? | 14:20 |
| ildikov | boris-42: yes, that one is for samples | 14:20 |
| boris-42 | ildikov so does it returns resource_metadata? | 14:20 |
| boris-42 | return* | 14:20 |
| ildikov | boris-42: the output format of the returned samples is the same as in case of the current query functionality | 14:21 |
| ildikov | boris-42: if you need metadata query as well, it will be also pushed for review this week, we are in the final refactor phase for that part | 14:22 |
| gibi | boris-42: current patchset return resource_metadata for samples, and as ildikov said if you need filtering for the metadata as well then it will be pushed soon | 14:24 |
| *** flwang has joined #openstack-ceilometer | 14:26 | |
| openstackgerrit | Cyril Roelandt proposed a change to stackforge/pycadf: Python 3: the request body should be bytes in test_api.py https://review.openstack.org/65279 | 14:30 |
| *** yassine has quit IRC | 14:31 | |
| *** yassine has joined #openstack-ceilometer | 14:32 | |
| *** sandywalsh has quit IRC | 14:32 | |
| openstackgerrit | Cyril Roelandt proposed a change to stackforge/pycadf: Python3: use six.moves.urllib.parse instead of urlparse https://review.openstack.org/65280 | 14:35 |
| *** sandywalsh has joined #openstack-ceilometer | 14:45 | |
| *** ruhe is now known as _ruhe | 14:49 | |
| *** s2r2 has quit IRC | 14:49 | |
| *** s2r2 has joined #openstack-ceilometer | 14:52 | |
| *** urulama has quit IRC | 14:53 | |
| *** dhellmann_ is now known as dhellmann | 15:09 | |
| boris-42 | gibi ildikov ok nice guys | 15:09 |
| boris-42 | gibi ildikov it's exactly what I need | 15:10 |
| *** urulama has joined #openstack-ceilometer | 15:12 | |
| *** urulama has quit IRC | 15:17 | |
| *** dperaza has joined #openstack-ceilometer | 15:26 | |
| dperaza | Hello all, happy new year, qq | 15:27 |
| dperaza | anyone knows why this line does not fail pep8 on master | 15:28 |
| dperaza | https://github.com/openstack/ceilometer/blob/master/ceilometer/event/converter.py#L378 | 15:28 |
| dperaza | I should be failing with E226 missing whitespace around arithmetic operator | 15:29 |
| dperaza | is E226 hidden somewhere? | 15:29 |
| *** urulama has joined #openstack-ceilometer | 15:30 | |
| *** sayali_ has joined #openstack-ceilometer | 15:34 | |
| *** sayali has joined #openstack-ceilometer | 15:34 | |
| *** sayali has quit IRC | 15:35 | |
| *** urulama has quit IRC | 15:35 | |
| *** urulama has joined #openstack-ceilometer | 15:35 | |
| *** _ruhe is now known as ruhe | 15:38 | |
| *** urulama has quit IRC | 15:41 | |
| *** gordc1 has joined #openstack-ceilometer | 15:52 | |
| *** gordc has quit IRC | 15:52 | |
| boris-42 | ildikov around? | 15:57 |
| *** tongli has joined #openstack-ceilometer | 15:59 | |
| ildikov | boris-42: yes | 16:01 |
| boris-42 | ildikov do you have patches to python clients? | 16:02 |
| boris-42 | lakspace to access query = QueryController() ? | 16:02 |
| boris-42 | ildikov & | 16:02 |
| boris-42 | lakspace sorry* | 16:02 |
| ildikov | boris-42: right now we don't | 16:02 |
| boris-42 | ildikov it will simplify a bit testing your patches | 16:03 |
| ildikov | boris-42: if this patch set will be accepted, we will do the cli support | 16:03 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 16:10 | |
| *** terriyu has joined #openstack-ceilometer | 16:10 | |
| ildikov | boris-42: I can't promise it for Icehouse-2, as it is so close, but it does not seem to be a difficult process, so hopefully it will not require as much time to implement after the patch set is merged | 16:11 |
| boris-42 | ildikov yep but it could be implemented in parallel =) | 16:12 |
| boris-42 | ildikov while you are waiting for reviews=) | 16:12 |
| boris-42 | ildikov seems like cli patch and this one will be absolutely independent | 16:12 |
| boris-42 | ildikov I mean comments in this one shouldn't require change in CLI patch | 16:12 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements field validation for complex query functionality https://review.openstack.org/65302 | 16:15 |
| gibi | boris-42: we added the CLI work to our todo list :) | 16:20 |
| boris-42 | gibi reviewing your patch | 16:20 |
| gibi | boris-42: thanks a lot! | 16:20 |
| boris-42 | gibi there are some nits. | 16:20 |
| ildikov | boris-42: if you can guarantee that this feature will be merged, then I'll immediately start working on the CLI ;) | 16:22 |
| boris-42 | ildikov it's impossible)) | 16:22 |
| boris-42 | ildikov lol | 16:22 |
| boris-42 | ildikov it's open source) | 16:22 |
| boris-42 | ildikov wild west) | 16:22 |
| boris-42 | ildikov if you would like guarantee from me then you are welcome in this project https://github.com/stackforge/rally =) | 16:26 |
| ildikov | boris-42: I know, that is why this implementation is on the top of the TODO list :) | 16:26 |
| dperaza | fyi E226 is ignored by default in pep8.py | 16:27 |
| *** flwang has left #openstack-ceilometer | 16:29 | |
| *** flwang has joined #openstack-ceilometer | 16:29 | |
| *** sayali_ has quit IRC | 16:30 | |
| ildikov | boris-42: thanks for the invitation, the project looks good, I'll consider, if that TODO list becomes a bit shorter :) | 16:32 |
| *** SergeyLukjanov has joined #openstack-ceilometer | 16:34 | |
| *** urulama has joined #openstack-ceilometer | 16:36 | |
| openstackgerrit | Ala Rezmerita proposed a change to openstack/ceilometer: Fixed spelling mistake https://review.openstack.org/65307 | 16:37 |
| *** urulama has quit IRC | 16:41 | |
| boris-42 | ildikov I started reviewing https://review.openstack.org/#/c/62157/ | 16:43 |
| boris-42 | ildikov so when you address this I will continue | 16:44 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Implement the /v2/samples/<sample-id> API https://review.openstack.org/61814 | 16:44 |
| *** terriyu has quit IRC | 16:48 | |
| *** ruhe is now known as ruhe_away | 16:49 | |
| *** ruhe_away is now known as ruhe | 16:54 | |
| openstackgerrit | David Peraza proposed a change to openstack/ceilometer: Check for pep8 E226 and E24 https://review.openstack.org/65317 | 16:54 |
| openstackgerrit | John Herndon proposed a change to openstack/ceilometer: Adds documentation for the Event API https://review.openstack.org/57353 | 16:55 |
| *** herndon has joined #openstack-ceilometer | 16:55 | |
| *** urulama has joined #openstack-ceilometer | 16:58 | |
| *** ruhe is now known as ruhe_away | 16:58 | |
| *** SergeyLukjanov has quit IRC | 16:59 | |
| ildikov | boris-42: thanks, I answered some of your comments and I'll do the corrections as well | 17:02 |
| herndon | any cores around? I'd love to be able to release the event client code in the ceilometer client this week: https://review.openstack.org/#/c/54582/ | 17:02 |
| *** ruhe_away is now known as ruhe | 17:03 | |
| *** urulama has quit IRC | 17:04 | |
| *** absubram_ has joined #openstack-ceilometer | 17:09 | |
| boris-42 | ildikov ok I will try to find a quantum of time to continue reviewing code | 17:09 |
| ildikov | boris-42: thanks :) | 17:10 |
| *** prad has joined #openstack-ceilometer | 17:10 | |
| eglynn | herndon: looking now | 17:12 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Set the SQL Float precision https://review.openstack.org/63132 | 17:12 |
| gibi | boris-42: can jsonschema validate the following two possible filters? | 17:13 |
| *** urulama has joined #openstack-ceilometer | 17:13 | |
| gibi | boris-42: json1 = ["not", ["=", "field", 13]] | 17:13 |
| gibi | boris-42: json2 = ["and", ["=", "field", 13], ["=", "field2", 42]] | 17:13 |
| gibi | boris-42: so basically the length of an array is depends on the first item of the array. | 17:14 |
| gibi | boris-42: and also the type of the second item in the array depends on the first item | 17:15 |
| dperaza | ceilometer rebased | 17:16 |
| boris-42 | gibi why do u use so strange format?) | 17:16 |
| boris-42 | gibi mongo format is much better imho | 17:18 |
| *** urulama has quit IRC | 17:18 | |
| boris-42 | gibi ? | 17:22 |
| gibi | boris-42: thinking... :) | 17:22 |
| boris-42 | gibi I mean format like in mongo is first of all well known and as second "+" is that we will be able to validate it by jsonschema | 17:22 |
| boris-42 | gibi and we will reduce 2 times amount of code=) | 17:23 |
| gibi | I'm trying to convert my examples to mongo format to see how it looks ... | 17:24 |
| herndon | eglynn: fyi, updating the patchset as tongli had a comment. | 17:26 |
| eglynn | herndon: k | 17:26 |
| openstackgerrit | John Herndon proposed a change to openstack/ceilometer: Adds documentation for the Event API https://review.openstack.org/57353 | 17:27 |
| *** yassine has quit IRC | 17:28 | |
| eglynn | herndon: tell you what, I'll publish my comments so far on the previous patch set | 17:29 |
| herndon | sounds good. | 17:29 |
| eglynn | herndon: ... then continue reviewing the latest'n'greatest | 17:29 |
| *** gordc1 has quit IRC | 17:30 | |
| *** gordc has joined #openstack-ceilometer | 17:31 | |
| eglynn | herndon: ... darn we were at cross purposes there | 17:31 |
| herndon | I got my wires corssed - the patch litong reviewed was different | 17:31 |
| herndon | toooo much going on | 17:31 |
| eglynn | herndon: ... I was reviewing https://review.openstack.org/54582 whereas you meant https://review.openstack.org/57353 right? | 17:32 |
| eglynn | herndon: ... no worries, take that as a partial review | 17:32 |
| eglynn | herndon: ... the sequel to follow | 17:32 |
| herndon | great. thanks :). I'll finish send up a new patch once both reviews are in | 17:32 |
| eglynn | cool | 17:33 |
| gibi | boris-42: can jsonschema validate that at certain key (e.g. $and) always followed with a value that has a type list but in case of other keys (e.g. field names) the type of the value is anytype? | 17:34 |
| boris-42 | gibi yep and you can use recursion as well | 17:35 |
| boris-42 | gibi in description of schema | 17:35 |
| boris-42 | gibi and you can specify the length of arrays | 17:35 |
| gibi | boris-42: I have to agree that mongo use a well known format and now I see that it is easier to verify. | 17:39 |
| boris-42 | gibi yep it could simplify your code | 17:40 |
| gibi | boris-42: thanks for the heads up. We will sit down with ildikov and discuss how we could go to this direction in the implementation. | 17:44 |
| boris-42 | gibi ildikov nice | 17:45 |
| gibi | especially in the sqlalchemy db engine as in mongo engine it will be trivial for sure | 17:45 |
| *** nsaje_ has joined #openstack-ceilometer | 17:45 | |
| boris-42 | gibi yep I really don't know how to make query on resource.metadata | 17:46 |
| gibi | boris-42: in our current implementation we planned to use the metadata.<metafield_name> syntax and that can be applied to this mongo like syntax as well | 17:48 |
| *** nsaje has quit IRC | 17:51 | |
| *** nsaje_ is now known as nsaje | 17:51 | |
| *** eglynn has quit IRC | 17:53 | |
| *** eglynn has joined #openstack-ceilometer | 17:54 | |
| eglynn | herndon: just finished that ceiloclient review ... gotta drop off now, but will be online again later | 17:54 |
| herndon | eglynn: ok, I will make the changes asap. thanks very much! | 17:54 |
| eglynn | herndon: np ... laters | 17:55 |
| *** eglynn has quit IRC | 17:59 | |
| *** urulama has joined #openstack-ceilometer | 18:01 | |
| *** ildikov has quit IRC | 18:05 | |
| *** urulama has quit IRC | 18:08 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 18:29 | |
| *** eglynn has joined #openstack-ceilometer | 18:41 | |
| *** tongli has quit IRC | 18:47 | |
| *** tongli has joined #openstack-ceilometer | 18:47 | |
| *** Ildi has joined #openstack-ceilometer | 18:49 | |
| *** ildikov has joined #openstack-ceilometer | 18:51 | |
| *** Ildi is now known as ildikov_ | 18:51 | |
| *** eglynn_ has joined #openstack-ceilometer | 18:53 | |
| thomasem | Hello! | 18:57 |
| *** eglynn has quit IRC | 19:01 | |
| *** urulama has joined #openstack-ceilometer | 19:04 | |
| *** urulama has quit IRC | 19:09 | |
| *** urulama has joined #openstack-ceilometer | 19:14 | |
| *** Alexei_987 has joined #openstack-ceilometer | 19:16 | |
| *** urulama has quit IRC | 19:16 | |
| *** jmckind has joined #openstack-ceilometer | 19:16 | |
| *** ildikov_ has quit IRC | 19:22 | |
| *** ildikov has quit IRC | 19:22 | |
| *** eglynn_ has quit IRC | 19:23 | |
| *** ildikov has joined #openstack-ceilometer | 19:23 | |
| *** ildikov_ has joined #openstack-ceilometer | 19:24 | |
| openstackgerrit | Pradeep Kilambi proposed a change to openstack/ceilometer: Fix to tackle instances without an image assigned https://review.openstack.org/65356 | 19:55 |
| *** urulama has joined #openstack-ceilometer | 20:05 | |
| *** urulama has quit IRC | 20:11 | |
| *** urulama has joined #openstack-ceilometer | 20:26 | |
| *** gordc has quit IRC | 20:45 | |
| *** SergeyLukjanov has quit IRC | 20:56 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 20:56 | |
| *** urulama_ has joined #openstack-ceilometer | 21:15 | |
| *** urulama_ has quit IRC | 21:17 | |
| *** ildikov_ has quit IRC | 21:34 | |
| openstackgerrit | John Herndon proposed a change to openstack/python-ceilometerclient: Support the Event API https://review.openstack.org/54582 | 21:35 |
| *** jdob has quit IRC | 21:36 | |
| *** urulama has quit IRC | 21:37 | |
| *** ruhe is now known as _ruhe | 22:08 | |
| *** sandywalsh has quit IRC | 22:09 | |
| *** sandywalsh has joined #openstack-ceilometer | 22:21 | |
| *** thomasem has quit IRC | 22:36 | |
| *** jmckind has quit IRC | 22:40 | |
| *** absubram_ has quit IRC | 23:02 | |
| *** asalkeld has quit IRC | 23:26 | |
| *** prad has quit IRC | 23:50 | |
| *** _ruhe is now known as ruhe | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!