| *** raymondr has joined #openstack-ceilometer | 00:00 | |
| *** fnaval has quit IRC | 00:08 | |
| *** yjiang5_1 has quit IRC | 00:19 | |
| *** yjiang5_1 has joined #openstack-ceilometer | 00:19 | |
| *** adriant has joined #openstack-ceilometer | 00:21 | |
| *** dhellmann has joined #openstack-ceilometer | 00:22 | |
| *** changbl has quit IRC | 00:24 | |
| *** matsuhashi has joined #openstack-ceilometer | 00:31 | |
| *** mihgen_ has joined #openstack-ceilometer | 00:47 | |
| *** swann_ has joined #openstack-ceilometer | 00:47 | |
| *** matsuhas_ has joined #openstack-ceilometer | 00:48 | |
| *** nati_uen_ has joined #openstack-ceilometer | 00:48 | |
| *** nijaba_ has joined #openstack-ceilometer | 00:48 | |
| *** apmelton1 has joined #openstack-ceilometer | 00:49 | |
| *** raymondr has quit IRC | 00:51 | |
| *** nati_ue__ has joined #openstack-ceilometer | 00:53 | |
| *** lakspace_ has joined #openstack-ceilometer | 00:54 | |
| *** Daviey_ has joined #openstack-ceilometer | 00:54 | |
| *** ruhe- has joined #openstack-ceilometer | 00:54 | |
| *** nati_uen_ has quit IRC | 00:54 | |
| *** SergeyLukjanov2 has joined #openstack-ceilometer | 00:55 | |
| *** matsuhashi has quit IRC | 00:55 | |
| *** llu has quit IRC | 00:55 | |
| *** nati_ueno has quit IRC | 00:55 | |
| *** nijaba has quit IRC | 00:55 | |
| *** jaypipes has quit IRC | 00:55 | |
| *** mihgen has quit IRC | 00:55 | |
| *** lakspace has quit IRC | 00:55 | |
| *** SergeyLukjanov has quit IRC | 00:55 | |
| *** ruhe has quit IRC | 00:55 | |
| *** swann has quit IRC | 00:55 | |
| *** apmelton has quit IRC | 00:55 | |
| *** ityaptin has quit IRC | 00:55 | |
| *** SergeyLukjanov2 is now known as SergeyLukjanov | 00:55 | |
| *** mihgen_ is now known as mihgen | 00:55 | |
| *** Daviey_ is now known as Daviey | 00:55 | |
| *** lakspace_ is now known as lakspace | 00:55 | |
| *** ruhe- is now known as ruhe | 00:55 | |
| *** llu has joined #openstack-ceilometer | 00:58 | |
| *** llu has quit IRC | 01:02 | |
| *** llu has joined #openstack-ceilometer | 01:02 | |
| *** ityaptin has joined #openstack-ceilometer | 01:02 | |
| *** jaypipes has joined #openstack-ceilometer | 01:03 | |
| *** yjiang5_1 has quit IRC | 01:06 | |
| *** yjiang5_1 has joined #openstack-ceilometer | 01:07 | |
| *** flwang has quit IRC | 01:12 | |
| *** openstackgerrit has quit IRC | 01:18 | |
| *** openstackgerrit has joined #openstack-ceilometer | 01:18 | |
| *** Ruetobas has quit IRC | 01:31 | |
| *** Ruetobas has joined #openstack-ceilometer | 01:32 | |
| *** nosnos has joined #openstack-ceilometer | 01:34 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Fix missing source field content on /v2/samples API https://review.openstack.org/74790 | 01:35 |
|---|---|---|
| openstackgerrit | A change was merged to openstack/ceilometer: Remove code that works around a (now-resolved) bug in pecan https://review.openstack.org/74824 | 01:36 |
| *** flwang has joined #openstack-ceilometer | 01:40 | |
| *** shakayumi has quit IRC | 01:53 | |
| *** flwang has quit IRC | 02:01 | |
| *** _cjones__ has quit IRC | 02:08 | |
| *** xianghui has joined #openstack-ceilometer | 02:09 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Updated from global requirements https://review.openstack.org/74873 | 02:16 |
| *** mihgen has quit IRC | 02:19 | |
| *** ryanpetrello has quit IRC | 02:21 | |
| *** flwang has joined #openstack-ceilometer | 02:22 | |
| *** matsuhas_ has quit IRC | 02:34 | |
| *** matsuhas_ has joined #openstack-ceilometer | 02:36 | |
| *** fnaval has joined #openstack-ceilometer | 02:45 | |
| *** thomasem has joined #openstack-ceilometer | 02:59 | |
| *** thomasem has quit IRC | 02:59 | |
| *** matsuhas_ has quit IRC | 03:03 | |
| *** matsuhas_ has joined #openstack-ceilometer | 03:06 | |
| *** raymondr has joined #openstack-ceilometer | 03:07 | |
| *** fnaval has quit IRC | 03:14 | |
| *** fnaval has joined #openstack-ceilometer | 03:15 | |
| *** _cjones_ has joined #openstack-ceilometer | 03:19 | |
| *** _cjones_ has quit IRC | 03:24 | |
| *** matsuhas_ has quit IRC | 03:29 | |
| *** flwang1 has joined #openstack-ceilometer | 03:35 | |
| *** flwang has quit IRC | 03:37 | |
| openstackgerrit | Jia Dong proposed a change to openstack/ceilometer: Implement meter query by 'counter_volume' field https://review.openstack.org/67384 | 03:37 |
| *** changbl has joined #openstack-ceilometer | 03:53 | |
| *** matsuhashi has joined #openstack-ceilometer | 04:25 | |
| *** nati_ue__ has quit IRC | 04:25 | |
| *** adriant has quit IRC | 04:38 | |
| *** matsuhashi has quit IRC | 05:23 | |
| *** raymondr has quit IRC | 05:24 | |
| openstackgerrit | A change was merged to openstack/python-ceilometerclient: Python 3 compatibility https://review.openstack.org/74874 | 05:24 |
| *** raymondr has joined #openstack-ceilometer | 05:25 | |
| *** raymondr has quit IRC | 05:29 | |
| *** matsuhashi has joined #openstack-ceilometer | 05:33 | |
| *** _nadya_ has joined #openstack-ceilometer | 05:55 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/62808 | 06:03 |
| *** _nadya_ has quit IRC | 06:05 | |
| *** ildikov_ has quit IRC | 06:16 | |
| *** _nadya_ has joined #openstack-ceilometer | 06:33 | |
| *** _nadya_ has quit IRC | 06:36 | |
| *** saju_m has joined #openstack-ceilometer | 06:36 | |
| *** lsmola has joined #openstack-ceilometer | 06:42 | |
| *** lsmola has quit IRC | 06:42 | |
| *** lsmola has joined #openstack-ceilometer | 06:43 | |
| openstackgerrit | Jia Dong proposed a change to openstack/python-ceilometerclient: py3kcompat: remove in python-ceilometerclient https://review.openstack.org/73511 | 07:23 |
| nprivalova | lsmola, hi! Please ping me when you are online | 08:29 |
| lsmola | nprivalova: hello | 08:30 |
| *** saju_m has quit IRC | 08:31 | |
| *** ildikov_ has joined #openstack-ceilometer | 08:52 | |
| *** mihgen has joined #openstack-ceilometer | 08:57 | |
| *** saju_m has joined #openstack-ceilometer | 09:01 | |
| *** saju_m has quit IRC | 09:02 | |
| openstackgerrit | Piyush Masrani proposed a change to openstack/ceilometer: VMware vSphere support https://review.openstack.org/74702 | 09:04 |
| *** yassine has joined #openstack-ceilometer | 09:04 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: storage: store recording timestamp https://review.openstack.org/70166 | 09:14 |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: api: export recorded_at in returned samples https://review.openstack.org/73992 | 09:14 |
| *** saju_m has joined #openstack-ceilometer | 09:21 | |
| *** eglynn has joined #openstack-ceilometer | 09:26 | |
| *** eglynn is now known as eglynn-headdown- | 09:27 | |
| *** eglynn-headdown- is now known as eglynnheaddowni3 | 09:27 | |
| *** flwang1 has quit IRC | 09:30 | |
| *** fnaval has quit IRC | 10:13 | |
| *** fnaval has joined #openstack-ceilometer | 10:14 | |
| *** fnaval has quit IRC | 10:36 | |
| *** yassine has quit IRC | 10:40 | |
| *** yassine has joined #openstack-ceilometer | 10:40 | |
| *** nosnos has quit IRC | 10:58 | |
| *** matsuhashi has quit IRC | 11:02 | |
| *** xianghui has quit IRC | 11:25 | |
| *** fnaval has joined #openstack-ceilometer | 11:31 | |
| *** fnaval has quit IRC | 11:34 | |
| *** fnaval has joined #openstack-ceilometer | 11:35 | |
| *** flwang has joined #openstack-ceilometer | 11:44 | |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: storage: store recording timestamp https://review.openstack.org/70166 | 11:46 |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: api: export recorded_at in returned samples https://review.openstack.org/73992 | 11:46 |
| *** mihgen has quit IRC | 11:55 | |
| *** promulo has quit IRC | 12:11 | |
| *** mihgen has joined #openstack-ceilometer | 12:29 | |
| openstackgerrit | Nejc Saje proposed a change to openstack/ceilometer: Adds time constraints to alarms [WIP] https://review.openstack.org/75391 | 12:30 |
| *** ityaptin has quit IRC | 12:52 | |
| *** ildikov_ has quit IRC | 12:55 | |
| *** ildikov_ has joined #openstack-ceilometer | 12:55 | |
| *** sayali has joined #openstack-ceilometer | 13:08 | |
| *** zul has quit IRC | 13:09 | |
| *** zul has joined #openstack-ceilometer | 13:09 | |
| *** AMike has quit IRC | 13:11 | |
| *** julim has joined #openstack-ceilometer | 13:13 | |
| *** julim has quit IRC | 13:14 | |
| *** sayali has quit IRC | 13:18 | |
| *** promulo has joined #openstack-ceilometer | 13:19 | |
| eglynnheaddowni3 | ... and the prize for the most audacious bid to score a free pass to summit goes to | 13:21 |
| eglynnheaddowni3 | ... drum-roll | 13:21 |
| eglynnheaddowni3 | ... https://review.openstack.org/71796 | 13:21 |
| eglynnheaddowni3 | arggh wrong link. https://review.openstack.org/75215 | 13:22 |
| nsaje_ | Well, looks like it worked :) | 13:23 |
| eglynnheaddowni3 | yeap ;) | 13:23 |
| *** jdob has joined #openstack-ceilometer | 13:24 | |
| nsaje_ | thanks for the super quick review eglynnheaddowni3 ! I replied with some follow-up questions to your inline comments | 13:29 |
| eglynnheaddowni3 | nsaje_: cool, looking ... | 13:29 |
| nsaje_ | also, who do I need to nag to get https://review.openstack.org/#/c/75055/ through? Probably jd__ ? :) | 13:30 |
| *** sayali has joined #openstack-ceilometer | 13:30 | |
| eglynnheaddowni3 | nsaje_: here's your list of potential nagees: https://review.openstack.org/#/admin/groups/131,members | 13:32 |
| openstackgerrit | Eugeniya Kudryashova proposed a change to openstack/python-ceilometerclient: Use HTTPClient from common Oslo code https://review.openstack.org/68939 | 13:34 |
| nsaje_ | eglynnheaddowni3: ah, thanks a lot! I couldn't find list of group members | 13:34 |
| eglynnheaddowni3 | nsaje_: np! | 13:34 |
| nsaje_ | eglynnheaddowni3: not sure I understand your conclusion about DST though :D do we keep US/Eastern or go with UTC-5? | 13:36 |
| *** thomasem has joined #openstack-ceilometer | 13:38 | |
| *** thomasem has quit IRC | 13:38 | |
| *** thomasem has joined #openstack-ceilometer | 13:39 | |
| eglynnheaddowni3 | nsaje_: sorry for the lack of clarity, I was agreeing with you that the convenience factor around keeping the DST transitions transparent is worth sticking with your approach | 13:39 |
| nsaje_ | eglynnheaddowni3: great, thanks | 13:39 |
| *** tongli has joined #openstack-ceilometer | 13:40 | |
| ildikov_ | tongli: hi | 13:49 |
| ildikov_ | tongli: do you have any news for me? :) | 13:49 |
| tongli | @ildikov_, doing it. | 13:49 |
| tongli | should have something by noon. | 13:50 |
| ildikov_ | tongli: thanks it sounds good | 13:50 |
| tongli | @ildikov_, could not get my devstack running, | 13:50 |
| tongli | did you ever see an error like this? | 13:51 |
| tongli | Traceback (most recent call last): | 13:51 |
| tongli | File "/usr/lib/python2.7/logging/__init__.py", line 851, in emit | 13:51 |
| tongli | msg = self.format(record) | 13:51 |
| tongli | File "/opt/stack/glance/glance/openstack/common/log.py", line 684, in format | 13:51 |
| tongli | return logging.StreamHandler.format(self, record) | 13:51 |
| tongli | File "/usr/lib/python2.7/logging/__init__.py", line 724, in format | 13:51 |
| tongli | return fmt.format(record) | 13:51 |
| tongli | File "/opt/stack/glance/glance/openstack/common/log.py", line 648, in format | 13:51 |
| tongli | return logging.Formatter.format(self, record) | 13:51 |
| tongli | File "/usr/lib/python2.7/logging/__init__.py", line 467, in format | 13:51 |
| tongli | s = self._fmt % record.__dict__ | 13:51 |
| tongli | KeyError: 'user_identity' | 13:51 |
| tongli | Logged from file middleware.py, line 103 | 13:51 |
| tongli | 13:51 | |
| tongli | @ildikov_, not from ceilometer. | 13:52 |
| ildikov_ | tongli: thinking, last time I did not have problems with devstack, but it was at the beginning of this week, when I last tried | 13:53 |
| tongli | @ildikov_, ok. not a problem. | 13:54 |
| dhellmann | jd__: do we need a minimum version for posix_ipc in https://review.openstack.org/#/c/68945/2 | 13:56 |
| ildikov_ | tongli: check this commit: https://git.openstack.org/cgit/openstack/glance/commit/?id=954073d0f26c5b2bbe86a2099158de57911a1a1c | 13:59 |
| ildikov_ | tongli: it's a change in glance common logging, related to this bug: https://bugs.launchpad.net/nova/+bug/904307 | 14:00 |
| ildikov_ | tongli: in log_opts there is a change from %(user)s %(tenant)s to %(user_identity)s | 14:01 |
| tongli | @ildikov_, yes, that is right, I saw that, the changes were made in the common code. it is also in Ceilometer now. | 14:02 |
| tongli | actually, most of OS projects already synched up with that. but it only blows up in Glance. | 14:02 |
| ildikov_ | yes, you're right, I've just checked the status | 14:04 |
| ildikov_ | tongli: yes, it's weird a bit | 14:05 |
| tongli | @ildikov_, not sure if there is an option to turn it off. | 14:05 |
| ildikov_ | tongli: I'm not sure either | 14:10 |
| *** saju_m has quit IRC | 14:10 | |
| tongli | @ildikov_, yeah, use_syslog_rfc_format | 14:11 |
| tongli | I will try that option and see if that makes any difference. | 14:11 |
| *** gordc has joined #openstack-ceilometer | 14:11 | |
| tongli | @ildikov_, test your patch is my highest priority. | 14:11 |
| gibi_ | tp[ | 14:12 |
| *** julim has joined #openstack-ceilometer | 14:12 | |
| gibi_ | sorry, wrong terminal | 14:12 |
| ildikov_ | tongli: I saw that and it's default value IIRC was False | 14:19 |
| tongli | @ildikov_, I will change it to True and see what happens. | 14:20 |
| *** zul has quit IRC | 14:20 | |
| *** zul has joined #openstack-ceilometer | 14:21 | |
| *** sayali has quit IRC | 14:21 | |
| ildikov_ | tongli: thanks, it would be good to know the state, as it is hard to keep those patch sets up to date after a while, that is the only reason I asked, if you have any updates | 14:21 |
| *** raymondr has joined #openstack-ceilometer | 14:22 | |
| tongli | @ildikov_, totally understood, | 14:22 |
| tongli | @ildikov_, will get you the results by noon. | 14:22 |
| ildikov_ | tongli: I will check that what can be the reason for this user_identity issue with glance, maybe some default values not set properly | 14:22 |
| *** raymondr_ has joined #openstack-ceilometer | 14:22 | |
| ildikov_ | tongli: thanks | 14:22 |
| tongli | @ildikov_, yeah, I searched for bugs on that, but did not find anything. | 14:23 |
| *** raymondr_ has quit IRC | 14:23 | |
| *** flwang has quit IRC | 14:25 | |
| *** raymondr has quit IRC | 14:26 | |
| *** flwang has joined #openstack-ceilometer | 14:28 | |
| tongli | @ildikov_, that option did not do anything. | 14:31 |
| ildikov_ | tongli: we've tried it now and we see the same trace, that you copied earlier | 14:32 |
| tongli | @ildikov_, yeah. | 14:32 |
| tongli | let me open up a bug agiinst devstack. | 14:33 |
| gibi_ | tongli: I can reproduce the same exception in my env after pulled in the glance change | 14:34 |
| ildikov_ | tongli: it's a good idea, I do not know, what can be the cause of this problem | 14:34 |
| tongli | @ildikov_, since I am not seeing same problem in Ceilometer or Nova, though they all use that code already. | 14:35 |
| tongli | I suspect it maybe something in devstack that was not setup right for glance. I do not know. | 14:35 |
| tongli | it is all guessing. I talked to a glance core, and he had no idea with that change. | 14:36 |
| tongli | and it is very hard to even know when a dependency suddenly change stuff. and to know the impact. | 14:36 |
| tongli | @ildikov_, check out this. | 14:38 |
| tongli | https://review.openstack.org/#/c/68530/1/lib/keystone | 14:38 |
| tongli | @ildikov_, seems that is the problem. | 14:39 |
| gordc | ildikov_: tongli: what's the error you see? | 14:40 |
| tongli | key_error user_identity from glance console. | 14:41 |
| tongli | yesterday, when I tried that, I also saw in other project, this morning, got latest code, only saw it from glance. | 14:41 |
| tongli | do not really know what the problem was, it seems that logging actually failed when the logger looks for user_identity key, but found none. | 14:42 |
| tongli | it falls on create image during the setup, so glance is not having any images that you can work with. | 14:43 |
| gordc | tongli: ah got it... i've no idea then... i know this bug was messing with me a while back but i can't remember what error the bug causes: https://bugs.launchpad.net/devstack/+bug/1276029 | 14:43 |
| *** ityaptin has joined #openstack-ceilometer | 14:44 | |
| tongli | @gordc, yeah, aware of that, that is why I disabled swift. | 14:44 |
| gordc | cool cool | 14:44 |
| tongli | @ildikov_, @gordc, that patch did not fix the problem. | 14:45 |
| tongli | I am opening up a bug against devstack. | 14:45 |
| tongli | well, actually against glance. | 14:46 |
| ildikov_ | tongli, gordc: there is a context.py in ceilometer, in which there is a user_identity field in to_dict: https://github.com/openstack/ceilometer/blob/master/ceilometer/openstack/common/context.py#L78, which is missing from the context.py file in glance: https://github.com/openstack/glance/blob/master/glance/context.py#L50 | 14:52 |
| tongli | @ildikov_, can you comment on this bug? | 14:53 |
| tongli | https://bugs.launchpad.net/glance/+bug/1283080 | 14:53 |
| ildikov_ | tongli: sure, will do that | 14:54 |
| tongli | @ildikov_, thanks. | 14:54 |
| ildikov_ | tongli: sorry, gibi_ will do that, he found the commits in oslo and ceilometer, which introduce this change | 14:56 |
| tongli | @ildikov_, that is fine, so he is aware of this problem as well? | 14:56 |
| ildikov_ | tongli: yes, he's sitting right next to me, we were investigating this together in the past few minutes :) | 14:57 |
| *** prad_ has joined #openstack-ceilometer | 14:58 | |
| tongli | @ildikov_, cool, cool. thanks. | 14:58 |
| gibi_ | tongli: I added comment to the bug. It seems some code is missing from Glance | 14:58 |
| openstackgerrit | A change was merged to openstack/ceilometer: storage: store recording timestamp https://review.openstack.org/70166 | 14:58 |
| gibi_ | tongli: /me is trying to push some workaround/fix for the bug soon | 15:02 |
| tongli | @gibi_, cool. thanks. let me know. | 15:03 |
| openstackgerrit | ZhiQiang Fan proposed a change to openstack/python-ceilometerclient: Adjust Exception inheritance relationship https://review.openstack.org/75428 | 15:05 |
| gibi_ | tongli: I pushed a patch for the bug https://review.openstack.org/#/c/75435/ | 15:26 |
| gibi_ | tongli: with this patch I dont get the exception any more | 15:27 |
| tongli | @gibi_, let me pull that and give it a try. will comment if I am not seeing any issues. | 15:27 |
| gibi_ | tongli: OK. Thanks | 15:28 |
| tongli | @gibi_, currently I can not create an image, if with this patch, I can, then it should be good. | 15:28 |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: rename meter table to sample https://review.openstack.org/71691 | 15:28 |
| tongli | @gibi_, probably link that patch to the bug that I opened earlier? | 15:29 |
| *** clouded_tux__ has quit IRC | 15:29 | |
| gibi_ | I added the patch to the bug as a comment | 15:30 |
| tongli | @gibi_, it works great. | 15:39 |
| gibi_ | tongli: Thanks for the test! :) | 15:40 |
| tongli | @gibi_, thanks for the fix. | 15:40 |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: sample table contains redundant/duplicate data https://review.openstack.org/65786 | 15:42 |
| *** lnxnut has joined #openstack-ceilometer | 15:43 | |
| *** raymondr has joined #openstack-ceilometer | 15:46 | |
| thomasem | gordc, ping | 15:54 |
| gordc | thomasem: whatsup? | 15:54 |
| thomasem | gordc, Hey hey, regarding that get_events being inefficient, I believe that's still a problem. | 15:54 |
| thomasem | gordc, Unfortunately I'm working on another project right now, so that's why I didn't get to it. :( | 15:55 |
| gordc | thomasem: cool cool, thought i'd ask... haven't tried myself | 15:55 |
| thomasem | gordc, I should have some notes around about making that faster, though. | 15:55 |
| thomasem | Essentially the problem was that it did a full join on trait_type before applying any useful filters. | 15:55 |
| thomasem | full join between trait and trait_type, I mean. | 15:56 |
| gordc | thomasem: ah, i see. | 15:56 |
| thomasem | Which, with 70M rows will be a monster. | 15:56 |
| openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Rename id to alarm_id of Alarm in SqlAlchemy https://review.openstack.org/75116 | 15:56 |
| gordc | thomasem: i'll keep that in mind if/when i get around to it. | 15:56 |
| thomasem | 70M events, I mean? which would be well over 1B traits. | 15:56 |
| gordc | thomasem: how long it take you to build 70M events? maybe i'll start with a few hundred thousand. | 15:57 |
| thomasem | gordc, Err, I could get it pretty quick with enough workers. | 15:57 |
| thomasem | gordc, let me get you my link | 15:57 |
| gordc | thomasem: k | 15:58 |
| thomasem | gordc, 70M was a few days, IIRC. | 15:58 |
| thomasem | gordc, https://etherpad.openstack.org/p/ceilometer-data-store-scale-testing | 15:59 |
| thomasem | gordc, That last MySQL test was pretty faster? I would do 10 writers (maybe more) and see if you can push that Events Per Second | 15:59 |
| thomasem | pretty fast* | 15:59 |
| thomasem | gordc, For some reason the test-19's etherpad isn't working, but use the config from 18. | 16:01 |
| *** Ruetobas has quit IRC | 16:01 | |
| gordc | thomasem: cool cool. i'll scan through that. thanks for the info. | 16:01 |
| thomasem | gordc, Sure thing, let me know if you have any questions! | 16:01 |
| *** thomasem has left #openstack-ceilometer | 16:01 | |
| *** thomasem has joined #openstack-ceilometer | 16:01 | |
| gordc | thomasem: will do | 16:02 |
| *** prad__ has joined #openstack-ceilometer | 16:04 | |
| tongli | @ildikov_, ping. | 16:04 |
| ildikov_ | tongli: I'm here | 16:05 |
| *** Ruetobas has joined #openstack-ceilometer | 16:05 | |
| tongli | @ildikov_, I think pymongo we use has been upgrade. now I can not even connect server_info fails. | 16:05 |
| *** prad_ has quit IRC | 16:06 | |
| tongli | @ildikov_, let me figure out some real quick. | 16:07 |
| ildikov_ | tongli: hm, I extracted the ConnectPool class and we modified the log message, but it should not be the problem as on the gate it worked fine | 16:09 |
| *** giroro_ has joined #openstack-ceilometer | 16:09 | |
| *** Ruetobas has quit IRC | 16:10 | |
| tongli | @ildikov_, db2 handles server_info call differently. It does not provide all the information pymongo looks for. so pymongo falls over. I do not htink this is because of your patch. | 16:10 |
| tongli | it is just that db2 works differently. | 16:10 |
| ildikov_ | tongli: it would be weird, if my patch caused the problem, as the log message should not affect the server_info call | 16:12 |
| tongli | @ildikov_, no, I do not think it is your patch. db2 missed something in server_info that pymongo looks for. | 16:13 |
| tongli | need to find why. | 16:13 |
| ildikov_ | tongli: ok, I assume I cannot help this time :( | 16:15 |
| tongli | @ildikov_, not a problem, I made some changes, it connects now. | 16:17 |
| ildikov_ | tongli: that was fast :) | 16:18 |
| tongli | @ildikov_, now meters all go into ceilometer db2 database. yeah!!! | 16:21 |
| tongli | @ildikov_, this patch is fine, https://review.openstack.org/#/c/74462/ | 16:22 |
| tongli | @ildikov_, which one you would like me to try next. | 16:22 |
| ildikov_ | tongli: the small joyful moments in a developer's life :) | 16:22 |
| tongli | @ildikov_, I know that you have a set of patches. | 16:22 |
| ildikov_ | tongli: this one contains the query: https://review.openstack.org/#/c/74778/ | 16:23 |
| tongli | does this one depend on the previous one? | 16:23 |
| ildikov_ | tongli: yes | 16:24 |
| tongli | ok | 16:24 |
| tongli | doing it now. | 16:24 |
| ildikov_ | tongli: ok, thanks | 16:26 |
| tongli | @ildikov_, can you give me an example to test it using the API? | 16:27 |
| ildikov_ | tongli: one sec | 16:28 |
| ildikov_ | tongli: the example here should work: http://docs.openstack.org/developer/ceilometer/webapi/v2.html#complex-query | 16:29 |
| ildikov_ | tongli: the timestamps should be fixed or removed | 16:30 |
| *** sayali has joined #openstack-ceilometer | 16:30 | |
| tongli | @ildikov_, ok, let me try that. | 16:30 |
| *** promulo has quit IRC | 16:33 | |
| *** Yathi has joined #openstack-ceilometer | 16:36 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Decouple source and sink configuration for pipelines [WIP] https://review.openstack.org/75459 | 16:41 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Pluggable resource discovery for agents [WIP] https://review.openstack.org/75006 | 16:41 |
| *** promulo has joined #openstack-ceilometer | 16:43 | |
| tongli | @ildikov_, hi, good news, the complex query worked fine. | 16:48 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Remove code duplication Part 2 https://review.openstack.org/74778 | 16:48 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Remove code duplication https://review.openstack.org/74462 | 16:48 |
| tongli | @ildikov_, however I need to change the driver for the conn.server_info. | 16:48 |
| tongli | @ildikov_, this does not affect your patch though. | 16:49 |
| ildikov_ | tongli: you made me really happy now :) | 16:49 |
| ildikov_ | tongli: I reabsed these two patches and I uploaded them and then I will not mark them as WIP any more | 16:50 |
| tongli | @ildikov_, cool. cool. | 16:50 |
| ildikov_ | tongli: in a third patch, I will upload the record alarm change and the alarm change query functions | 16:50 |
| ildikov_ | tongli: I will let you know, when it is ready for test | 16:50 |
| tongli | @ildikov_, that is to add alarm capabilities to db2 driver, right? | 16:51 |
| ildikov_ | tongli: yes, as a side effect, if it works than it will mean that db2 will have those features supported | 16:51 |
| tongli | @ildikov_, db2 does not support map_reduce, anywhere if the code uses map_reduce, then the function has to be | 16:52 |
| tongli | changed to use aggregate. | 16:52 |
| ildikov_ | tongli: no, our query does not use map-reduce | 16:52 |
| tongli | @ildikov_, yes, that is why it worked. | 16:52 |
| ildikov_ | tongli: I'm aware of that db2 does not support it, so the statistics functions will remain untouched in this first round for sure | 16:52 |
| tongli | I mean anywhere that you want to use or see mongo uses map reduce, then the code will diverge. | 16:52 |
| ildikov_ | tongli: yes, I know that, we will see if it worth to touch those functions or not | 16:53 |
| tongli | cool, cool, I do not think map-reduce performs better than aggreagte in cases. | 16:53 |
| tongli | any way, great work, let me try to find a permenant solution to the server_info problem. this will most likely need db2 team to make some changes. | 16:54 |
| tongli | hope that ceilometer code does not have to change. | 16:54 |
| ildikov_ | tongli: I hope so too | 16:54 |
| tongli | actually the code breaks is that I was trying to use server_info to figure out if the test env. is using db2 or mongo for tests. | 16:55 |
| tongli | on jenkins, there is no db2 instance, so I have to use mongo to test db2 driver.so db2 driver has to work against both db2 and mongo. | 16:55 |
| tongli | that makes the task more difficult. | 16:55 |
| ildikov_ | tongli: in the last step of this refactor, I will try to figure out the those functions that are looks really different now will worth to extract or not | 16:55 |
| tongli | k. | 16:56 |
| ildikov_ | tongli: yes, I know that it is a tricky process to make tests work with db2 :( | 16:56 |
| tongli | cool. have a great weekend. | 16:56 |
| ildikov_ | tongli: anyway, I think it is a good news that the refactor and also the complex query works with db2 also | 16:56 |
| ildikov_ | tongli: thanks much for testing | 16:57 |
| tongli | @ildikov_, not a problem, thanks for working on this driver. | 16:57 |
| *** promulo has quit IRC | 16:57 | |
| ildikov_ | tongli: can you mark the second patch as working too? | 16:58 |
| ildikov_ | tongli: and then hopefully it can be merged soon to avoid the continuous rebase issues | 17:00 |
| tongli | @ildikov_, just did that. | 17:00 |
| ildikov_ | tongli: thanks much | 17:00 |
| tongli | thanks | 17:00 |
| ildikov_ | tnogli: I will ping you next week, when the third patch is ready | 17:00 |
| ildikov_ | tongli: have a great weekend | 17:00 |
| tongli | @ildikov_, cool. thanks. u have have a great weekend. what is time now at where you are? | 17:01 |
| ildikov_ | tongli: it's 6pm (in Hungary) | 17:03 |
| tongli | @ildikov_, oh, quite late for weekend. have a wonderful Friday night. | 17:03 |
| *** _cjones_ has joined #openstack-ceilometer | 17:03 | |
| ildikov_ | tongli: thanks :) what is the time at your place? | 17:04 |
| tongli | 12:05pm | 17:04 |
| tongli | lunch time , I am on the east coast. | 17:05 |
| ildikov_ | tongli: then enjoy your lunch soon :) | 17:05 |
| ildikov_ | tongli: ok, I will keep that in mind | 17:05 |
| tongli | @ildikov_, later. | 17:05 |
| ildikov_ | tongli: 6 hours difference can be handled | 17:05 |
| ildikov_ | tongli: laters | 17:06 |
| *** sayali has quit IRC | 17:07 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Update docs for new decoupled pipeline config. https://review.openstack.org/75466 | 17:10 |
| ildikov_ | gordc: are you around? | 17:11 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Update docs for new decoupled pipeline config. https://review.openstack.org/75466 | 17:13 |
| gordc | ildikov_: whatsup? | 17:14 |
| ildikov_ | gordc: tongli tested the mongo/db2 refactor and it worked for both patches | 17:15 |
| ildikov_ | gordc: could you take a look at the patches? | 17:15 |
| ildikov_ | gordc: I rebased them and uploaded the rebased version | 17:15 |
| ildikov_ | gordc: the link: https://review.openstack.org/#/q/status:open+project:openstack/ceilometer+branch:master+topic:mongo-db2-refactor,n,z | 17:16 |
| gordc | ildikov_: sure. i'll take a look at it later today. | 17:16 |
| ildikov_ | gordc: thank you | 17:16 |
| *** mihgen has quit IRC | 17:18 | |
| *** mihgen has joined #openstack-ceilometer | 17:18 | |
| *** lsmola has quit IRC | 17:20 | |
| openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Rename id to alarm_id of Alarm in SqlAlchemy https://review.openstack.org/75116 | 17:22 |
| *** mihgen has quit IRC | 17:22 | |
| *** eglynnheaddowni3 has quit IRC | 17:31 | |
| *** tongli has quit IRC | 17:31 | |
| *** ildikov_ has quit IRC | 17:35 | |
| *** dtalton has joined #openstack-ceilometer | 17:39 | |
| *** dtalton has left #openstack-ceilometer | 17:40 | |
| *** vrovachev has left #openstack-ceilometer | 17:40 | |
| *** _nadya_ has joined #openstack-ceilometer | 17:52 | |
| *** prad_ has joined #openstack-ceilometer | 18:02 | |
| *** prad__ has quit IRC | 18:05 | |
| *** fnaval has quit IRC | 18:23 | |
| *** marcoemorais has joined #openstack-ceilometer | 18:33 | |
| *** Yathi has quit IRC | 18:34 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging https://review.openstack.org/57457 | 18:40 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/ceilometer: Replace oslo.rpc by oslo.messaging https://review.openstack.org/57457 | 18:45 |
| marcoemorais | dumb newbie question — why do ceilometer api calls require a password instead of accepting a token? http://paste.openstack.org/show/68096/ | 18:46 |
| *** fnaval has joined #openstack-ceilometer | 19:09 | |
| *** ildikov_ has joined #openstack-ceilometer | 19:24 | |
| *** jergerber has joined #openstack-ceilometer | 19:26 | |
| *** raymondr has quit IRC | 19:37 | |
| *** _nadya_ has quit IRC | 19:39 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Decouple source and sink configuration for pipelines [WIP] https://review.openstack.org/75459 | 19:46 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Update docs for new decoupled pipeline config. https://review.openstack.org/75466 | 19:46 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Pluggable resource discovery for agents [WIP] https://review.openstack.org/75006 | 19:46 |
| *** bada has quit IRC | 19:49 | |
| *** bada has joined #openstack-ceilometer | 19:50 | |
| *** raymondr has joined #openstack-ceilometer | 19:52 | |
| *** lnxnut has quit IRC | 19:56 | |
| *** lnxnut has joined #openstack-ceilometer | 19:57 | |
| *** raymondr has quit IRC | 19:57 | |
| *** lnxnut has quit IRC | 20:01 | |
| *** raymondr has joined #openstack-ceilometer | 20:01 | |
| *** _nadya_ has joined #openstack-ceilometer | 20:02 | |
| *** raymondr has quit IRC | 20:06 | |
| *** raymondr has joined #openstack-ceilometer | 20:08 | |
| *** raymondr_ has joined #openstack-ceilometer | 20:09 | |
| *** raymondr has quit IRC | 20:13 | |
| *** lnxnut has joined #openstack-ceilometer | 20:21 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Remove code duplication Part 2 https://review.openstack.org/74778 | 20:32 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Remove code duplication https://review.openstack.org/74462 | 20:32 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Decouple source and sink configuration for pipelines [WIP] https://review.openstack.org/75459 | 20:36 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Update docs for new decoupled pipeline config https://review.openstack.org/75466 | 20:36 |
| *** _nadya_ has quit IRC | 20:41 | |
| *** thomasem has quit IRC | 20:46 | |
| *** bada_ has joined #openstack-ceilometer | 20:48 | |
| *** bada has quit IRC | 20:48 | |
| *** marcoemorais has quit IRC | 20:56 | |
| *** promulo has joined #openstack-ceilometer | 20:57 | |
| *** marcoemorais has joined #openstack-ceilometer | 20:58 | |
| *** marcoemorais has quit IRC | 21:10 | |
| *** marcoemorais has joined #openstack-ceilometer | 21:10 | |
| *** thomasem has joined #openstack-ceilometer | 21:20 | |
| *** dtalton has joined #openstack-ceilometer | 21:21 | |
| *** dtalton2 has joined #openstack-ceilometer | 21:22 | |
| *** promulo has quit IRC | 21:26 | |
| *** dtalton has quit IRC | 21:26 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Decouple source and sink configuration for pipelines [WIP] https://review.openstack.org/75459 | 21:39 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Update docs for new decoupled pipeline config https://review.openstack.org/75466 | 21:39 |
| *** dtalton2 has quit IRC | 21:46 | |
| *** dtalton has joined #openstack-ceilometer | 21:52 | |
| *** dtalton2 has joined #openstack-ceilometer | 21:54 | |
| *** yassine has quit IRC | 21:54 | |
| *** dtalton3 has joined #openstack-ceilometer | 21:55 | |
| *** dtalton3 has left #openstack-ceilometer | 21:56 | |
| *** dtalton has quit IRC | 21:58 | |
| *** dtalton2 has quit IRC | 21:59 | |
| *** jdob has quit IRC | 22:00 | |
| gordc | marcoemorais: massive delay in response... but it should work with token... if it ain't, something is broke: https://bugs.launchpad.net/python-ceilometerclient/+bugs | 22:02 |
| gordc | marcoemorais: althought it may be this bug: https://review.openstack.org/#/c/58051 | 22:03 |
| *** promulo has joined #openstack-ceilometer | 22:31 | |
| *** julim has quit IRC | 22:36 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Decouple source and sink configuration for pipelines [WIP] https://review.openstack.org/75459 | 22:37 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack/ceilometer: Update docs for new decoupled pipeline config https://review.openstack.org/75466 | 22:37 |
| *** gordc has quit IRC | 22:53 | |
| *** prad_ has quit IRC | 23:01 | |
| *** thomasem has quit IRC | 23:09 | |
| *** raymondr_ has quit IRC | 23:12 | |
| *** jergerber has quit IRC | 23:16 | |
| *** mihgen has joined #openstack-ceilometer | 23:31 | |
| *** _nadya_ has joined #openstack-ceilometer | 23:42 | |
| *** sdake_ has joined #openstack-ceilometer | 23:45 | |
| *** sdake_ has joined #openstack-ceilometer | 23:45 | |
| *** _nadya_ has quit IRC | 23:46 | |
| *** mihgen has quit IRC | 23:50 | |
| *** fnaval has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!