| *** mwagner_lap has quit IRC | 03:27 | |
| *** mwagner_lap has joined #openstack-ceilometer | 04:09 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/60154 | 06:04 |
|---|---|---|
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: (WIP)Implements monitoring-network https://review.openstack.org/60473 | 06:22 |
| *** ildikov has quit IRC | 06:28 | |
| *** sayali has joined #openstack-ceilometer | 07:13 | |
| *** sayali has quit IRC | 07:49 | |
| *** ildikov has joined #openstack-ceilometer | 08:08 | |
| *** sayali has joined #openstack-ceilometer | 08:10 | |
| *** nprivalova has joined #openstack-ceilometer | 08:32 | |
| *** kwhitney has quit IRC | 09:08 | |
| *** drozzzd has joined #openstack-ceilometer | 10:07 | |
| *** drozzzd has quit IRC | 10:08 | |
| *** eglynn has joined #openstack-ceilometer | 10:30 | |
| eglynn | sileht: watching the rugby at the weekend perchance? ;) | 10:32 |
| eglynn | sileht: http://www.lequipe.fr/Rugby/Actualites/Toulouse-tombe-de-haut/423533 | 10:32 |
| *** nprivalova has quit IRC | 10:40 | |
| *** yfujioka has joined #openstack-ceilometer | 10:43 | |
| EmilienM | eglynn: please don't annoy our sileht on a Monday morning :P | 10:48 |
| eglynn | EmilienM: LOL :) ... k, I'll go easy ;) | 10:49 |
| *** nprivalova has joined #openstack-ceilometer | 10:53 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 10:53 | |
| *** sayali has quit IRC | 10:58 | |
| ildikov | guys, this could be used for marketing purposes: Come and enjoy The Community life, no more boring Monday mornings! ;) | 11:00 |
| *** sayali has joined #openstack-ceilometer | 11:05 | |
| *** kwhitney has joined #openstack-ceilometer | 11:07 | |
| *** Alexei_987 has joined #openstack-ceilometer | 11:08 | |
| *** sayali has quit IRC | 11:24 | |
| *** eglynn is now known as eglynn-afk | 11:32 | |
| *** urulama has joined #openstack-ceilometer | 11:40 | |
| openstackgerrit | Swann Croiset proposed a change to openstack/ceilometer: Ganglia Satellite Pollster https://review.openstack.org/58249 | 11:41 |
| *** kwhitney has quit IRC | 11:41 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: (WIP)Implements monitoring-network https://review.openstack.org/60473 | 11:55 |
| *** yfujioka has quit IRC | 12:13 | |
| *** nprivalova has quit IRC | 12:14 | |
| *** nprivalova has joined #openstack-ceilometer | 12:15 | |
| *** eglynn-afk is now known as eglynn | 12:25 | |
| eglynn | folks, it's been pointed out to me that we should be explicit in the Havana release notes about the way the sqlalchemy driver is not yet feature-complete | 12:28 |
| eglynn | e.g. in https://wiki.openstack.org/wiki/ReleaseNotes/Havana#Collector | 12:29 |
| eglynn | s/Improved SQLAlchemy driver/ | 12:29 |
| eglynn | Improved SQLAlchemy driver (but not yet feature-complete due to the lack of metadata query support as needed by Horizon and Ceilometer Alarming)/ | 12:29 |
| eglynn | ... any objections to the above? | 12:33 |
| nprivalova | no objections. its very useful note | 12:33 |
| nprivalova | btw, eglynn, "Added a UDP based publisher" - we are creating tests for it and looks like it doesn't work | 12:35 |
| eglynn | nprivalova: can you file a bug for the issue you're seeing, I'll take a look if jd__ doesn't get to it first | 12:36 |
| nprivalova | eglynn, ok, will do | 12:37 |
| eglynn | nprivalova: thanks! | 12:37 |
| jd__ | nprivalova: I noticed there might be issue reading the code last week indeed | 12:41 |
| jd__ | nprivalova: if you have a bug report, give it to me I'll take a look :) | 12:41 |
| nprivalova | jd__, ok, we are creating report. Will publish soon | 12:42 |
| eglynn | FYI havana relnotes updated as proposed above | 13:04 |
| *** urulama has quit IRC | 13:09 | |
| *** nprivalova has quit IRC | 13:10 | |
| *** mwagner_lap has quit IRC | 13:11 | |
| *** nprivalova has joined #openstack-ceilometer | 13:13 | |
| *** eglynn_ has joined #openstack-ceilometer | 13:22 | |
| Alexei_987 | jd__: ping me when you'll have some time available | 13:23 |
| *** eglynn has quit IRC | 13:23 | |
| jd__ | Alexei_987: pong | 13:23 |
| Alexei_987 | jd__: Wanted to talk about our current data structure | 13:27 |
| Alexei_987 | jd__: and I think that we may need to rework/simplify it to make existing code work faster and new features easier to add | 13:28 |
| *** thomasem has joined #openstack-ceilometer | 13:28 | |
| Alexei_987 | jd__: mostly I'm talking about mongodb driver but it's related to sqlalchemy as well | 13:28 |
| thomasem | Hey everyone | 13:29 |
| Alexei_987 | jd__: what do you think about it? | 13:29 |
| *** kwhitney has joined #openstack-ceilometer | 13:29 | |
| Alexei_987 | jd__: currently I'm trying to pagination implementation | 13:31 |
| jd__ | Alexei_987: well you just wrote "I want to make things faster and better", how can I say no? | 13:31 |
| jd__ | :-) | 13:31 |
| thomasem | ^^ | 13:31 |
| jd__ | do you have any more details? | 13:31 |
| Alexei_987 | jd__: there is a complicated conversion between API requests and database querying. Another problem I see is pagination API. I think I could start working on a small draft intended to simplify both parts | 13:34 |
| Alexei_987 | jd__: this will allow to build more clear intermediate level of data handling and allow future refactoring | 13:35 |
| Alexei_987 | jd__: and a bit offtopic question: I'm trying to understand full ceilometer request life-cycle and I wonder why do we move agent and collector to separate service? | 13:37 |
| jd__ | Alexei_987: go ahead, draft :) | 13:37 |
| jd__ | Alexei_987: because they are different services doing different things? | 13:38 |
| Alexei_987 | yeah.. but what's the point in having many separate services on system level? | 13:38 |
| Alexei_987 | we can separate them all in our code base and on system level have only 1 service called ceilometer | 13:39 |
| *** litong has joined #openstack-ceilometer | 13:39 | |
| Alexei_987 | it's quite hard for me to track every possible entry-point with such separation | 13:39 |
| jd__ | Alexei_987: you want only one daemon doing the sampling, central agent, compute agent, API? | 13:40 |
| jd__ | (etc) | 13:40 |
| Alexei_987 | you can call it like this :) | 13:41 |
| Alexei_987 | I want to have a common lifecycle and bootstrap for the application | 13:41 |
| Alexei_987 | jd__: so I would now exactly when specific objects (db connections, etc.) are created and destroyed | 13:44 |
| Alexei_987 | jd__: and how they can be reused between different parts of application | 13:44 |
| jd__ | not sure I follow you | 13:44 |
| Alexei_987 | jd__: I mean that ceilometer-api uses 1 type of boostrap made for WSGI app, agent-compute uses second one, and agent-notification handles rpc messages using another code path | 13:46 |
| *** jdob has joined #openstack-ceilometer | 13:46 | |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: test code should be excluded from test coverage summary https://review.openstack.org/60309 | 13:47 |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: add more test cases to improve the test code coverage #5 https://review.openstack.org/49802 | 13:48 |
| jd__ | Alexei_987: that's actually what we do, no? | 13:49 |
| openstackgerrit | Sushil Kumar proposed a change to openstack/python-ceilometerclient: Updates tox.ini to use new features https://review.openstack.org/60818 | 13:50 |
| Alexei_987 | jd__: yes but I mean is there any reason to separate application parts into different processes except for the logical separation? | 13:51 |
| nprivalova | Alexei_987, I thought that we can run several collectors but only one central agent. Maybe I'm wrong | 13:59 |
| nprivalova | jd__, JFYI https://bugs.launchpad.net/ceilometer/+bug/1259171 | 13:59 |
| *** zoresvit has joined #openstack-ceilometer | 14:03 | |
| Alexei_987 | jd__: ok forget about service stuff. let's get back to storage/queries. what would you think about such types of urls in ceilometer API: /v2/meters/volume?id<=10 ? | 14:09 |
| *** ildikov has quit IRC | 14:10 | |
| Alexei_987 | I mean change query string format to make it more readable and easier to handle | 14:10 |
| *** gordc has joined #openstack-ceilometer | 14:10 | |
| openstackgerrit | John Herndon proposed a change to openstack/ceilometer: Event Storage Layer https://review.openstack.org/57304 | 14:16 |
| *** herndon_ has joined #openstack-ceilometer | 14:16 | |
| openstackgerrit | litong01 proposed a change to openstack/ceilometer: test code should be excluded from test coverage summary https://review.openstack.org/60309 | 14:18 |
| *** zoresvit has quit IRC | 14:21 | |
| *** zoresvit has joined #openstack-ceilometer | 14:21 | |
| *** zoresvit has quit IRC | 14:21 | |
| *** zoresvit has joined #openstack-ceilometer | 14:21 | |
| *** zoresvit has quit IRC | 14:22 | |
| *** zoresvit has joined #openstack-ceilometer | 14:22 | |
| nprivalova | jd__, if you have not time for fixing udp we may start working on this because we're creating publisher's tests for tempest. jfyi | 14:27 |
| jd__ | nprivalova: go ahead | 14:31 |
| nprivalova | jd__, go ahead fixing :)? | 14:32 |
| jd__ | nprivalova: yes :) | 14:32 |
| nprivalova | jd__, ok | 14:33 |
| nprivalova | jd__, two more questions. first about https://review.openstack.org/#/c/59214/ . Looks like your change will not cover the case that was fixed. So as I understand Ilya should create his own unit test. I think he may just add a test without creating dependency on https://review.openstack.org/#/c/59489/, right? | 14:36 |
| *** sayali has joined #openstack-ceilometer | 14:37 | |
| *** sayali has quit IRC | 14:47 | |
| herndon_ | litong, regarding your comment on the event storage layer patch, rackspace is adding the event_storage code to the mogo driver. thomasem can you confirm that? | 14:53 |
| thomasem | Yeah. We're using it for load testing now to try and figure out a good trait/index document schema | 14:53 |
| litong | @herndon_, that is cool, just need to know that so that this won't be a gap moving forward. | 14:54 |
| thomasem | Since Mongo is one of the data stores we're looking at leveraging to handle our ~1M events/day requirement | 14:54 |
| litong | also I think if there are some doc updates, that will be really cool. | 14:54 |
| thomasem | litong, For the DBAPI or Event web API? | 14:54 |
| litong | api, since this is really an api update. | 14:55 |
| herndon_ | event web api will be taken care of by this branch: https://review.openstack.org/#/c/57353/ | 14:55 |
| thomasem | ^^ | 14:55 |
| herndon_ | still have some branches to get reviews and merged after event storage | 14:55 |
| litong | I always think that the api change is a big deal though this is an addition. | 14:55 |
| herndon_ | I broke it up into smaller chunks so it would be easier to review | 14:55 |
| thomasem | Yeah, herndon_ broke.. ^^ yeah | 14:55 |
| herndon_ | :p | 14:56 |
| litong | @herndon_, yeah, that will be fine. @thomasem, if you can just add that mongo will be also implementing that, I can change it. | 14:56 |
| litong | I actually pulled down your patch and run the tests, looks ok. | 14:56 |
| thomasem | litong, Added a review comment about it with two links for you to browse at your leisure. | 14:58 |
| litong | @thomasem, looking | 14:59 |
| *** jdob has quit IRC | 15:05 | |
| *** sayali has joined #openstack-ceilometer | 15:12 | |
| *** jdob has joined #openstack-ceilometer | 15:22 | |
| *** herndon_ has quit IRC | 15:25 | |
| openstackgerrit | Sushil Kumar proposed a change to openstack/python-ceilometerclient: Updates tox.ini to use new features https://review.openstack.org/60818 | 15:28 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ceilometerclient: Replace inheritance hierarchy with composition https://review.openstack.org/60857 | 15:44 |
| *** djbkd has joined #openstack-ceilometer | 15:46 | |
| *** SergeyLukjanov has quit IRC | 15:54 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Implement the /v2/sample API https://review.openstack.org/60540 | 16:01 |
| *** herndon has joined #openstack-ceilometer | 16:03 | |
| nprivalova | gentle reminder about https://review.openstack.org/#/c/52670/ :) | 16:08 |
| *** nprivalova has quit IRC | 16:20 | |
| *** sandywalsh has joined #openstack-ceilometer | 16:23 | |
| sandywalsh | hey y'all, can we get a +2 from a core on https://review.openstack.org/#/c/42713/ please? | 16:23 |
| *** SergeyLukjanov has joined #openstack-ceilometer | 16:27 | |
| herndon | litong: do you still think -1 for this patch? https://review.openstack.org/#/c/57304/ If not can you remove the -1? It also needs another core to take a look. | 16:27 |
| litong | @herndon, had to go to a meeting, yeah, will do that in a minute. | 16:29 |
| herndon | thanks | 16:29 |
| openstackgerrit | David Peraza proposed a change to openstack/ceilometer: Add import for publisher_rpc option https://review.openstack.org/60634 | 16:30 |
| *** zoresvit has quit IRC | 16:46 | |
| *** zoresvit has joined #openstack-ceilometer | 16:48 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 16:54 | |
| *** _SergeyLukjanov has quit IRC | 16:54 | |
| *** SergeyLukjanov has joined #openstack-ceilometer | 17:02 | |
| *** zoresvit has quit IRC | 17:07 | |
| *** sayali has quit IRC | 17:19 | |
| openstackgerrit | Alexei Kornienko proposed a change to openstack/ceilometer: Working on models refactoring https://review.openstack.org/60885 | 17:24 |
| Alexei_987 | jd__: ping | 17:25 |
| Alexei_987 | jd__: could you please review the patch ^ when you'll have time. I would like to get your opinion on this topic (-1,-2) ? :) | 17:27 |
| Alexei_987 | also all core developers are welcome | 17:27 |
| sandywalsh | sileht jd__ eglynn_ litong ^ anyone? review? | 18:12 |
| eglynn_ | sandywalsh: just leaving the office now, online again later ... what's the gerrit link? | 18:13 |
| sandywalsh | eglynn_, thanks .... https://review.openstack.org/#/c/42713/ | 18:14 |
| eglynn_ | k, on my list ... laters | 18:14 |
| sandywalsh | cheers | 18:15 |
| *** eglynn_ has quit IRC | 18:21 | |
| *** prad has joined #openstack-ceilometer | 18:44 | |
| *** eglynn_ has joined #openstack-ceilometer | 19:01 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Add import for publisher_rpc option https://review.openstack.org/60634 | 19:05 |
| *** gibi has joined #openstack-ceilometer | 19:10 | |
| *** Alexei_987 has quit IRC | 19:17 | |
| *** djbkd has quit IRC | 19:29 | |
| *** eglynn_ is now known as eglynn-afk | 19:42 | |
| *** gordc has quit IRC | 20:19 | |
| *** djbkd has joined #openstack-ceilometer | 20:55 | |
| *** ildikov has joined #openstack-ceilometer | 20:55 | |
| *** herndon has quit IRC | 21:02 | |
| *** herndon has joined #openstack-ceilometer | 21:03 | |
| *** openstackgerrit has quit IRC | 21:37 | |
| *** openstackgerrit has joined #openstack-ceilometer | 21:37 | |
| *** sandywalsh has quit IRC | 21:47 | |
| *** litong has quit IRC | 21:56 | |
| *** openstackgerrit has quit IRC | 21:57 | |
| *** openstackgerrit has joined #openstack-ceilometer | 21:57 | |
| *** sandywalsh has joined #openstack-ceilometer | 21:58 | |
| *** jdob has quit IRC | 22:00 | |
| *** SergeyLukjanov has quit IRC | 22:07 | |
| *** djbkd has quit IRC | 22:26 | |
| *** ildikov has quit IRC | 22:35 | |
| *** Alexei_987 has joined #openstack-ceilometer | 22:35 | |
| *** thomasem has quit IRC | 22:37 | |
| *** insanidade has joined #openstack-ceilometer | 22:38 | |
| insanidade | anyone awake ? just following http://docs.openstack.org/havana/install-guide/install/yum/content/cinder-install-cinder.html but looks like there's an issue (there's even a bug at launchpad for this) | 22:39 |
| insanidade | the last service I have to restard does not exist. | 22:40 |
| *** eglynn-afk has quit IRC | 23:10 | |
| *** jburkhart has joined #openstack-ceilometer | 23:22 | |
| *** djbkd has joined #openstack-ceilometer | 23:36 | |
| *** prad has quit IRC | 23:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!