| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network https://review.openstack.org/60473 | 01:49 |
|---|---|---|
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network-from-opendaylight https://review.openstack.org/63890 | 01:49 |
| *** flwang has quit IRC | 01:51 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/python-ceilometerclient: fix help message of deprecated command https://review.openstack.org/72198 | 01:58 |
| *** xianghui has joined #openstack-ceilometer | 02:12 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: Alembic migrations not tested https://review.openstack.org/71688 | 02:32 |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: rename meter table to sample https://review.openstack.org/71691 | 02:32 |
| *** Yathi has quit IRC | 02:33 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: sample table contains redundant/duplicate data https://review.openstack.org/65786 | 02:35 |
| *** flwang has joined #openstack-ceilometer | 02:40 | |
| openstackgerrit | Jeremy Stanley proposed a change to openstack/ceilometer: Remove tox locale overrides https://review.openstack.org/72202 | 02:57 |
| openstackgerrit | Fengqian.gao proposed a change to openstack/ceilometer: Add pagination support for sqlalchemy database https://review.openstack.org/35454 | 03:25 |
| openstackgerrit | Fengqian.gao proposed a change to openstack/ceilometer: Change pagination related methods of mongodb and db2 https://review.openstack.org/41869 | 03:25 |
| *** Yathi has joined #openstack-ceilometer | 03:45 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: fix column name and alignment https://review.openstack.org/69524 | 04:35 |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network https://review.openstack.org/60473 | 04:36 |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/ceilometer: Implements monitoring-network-from-opendaylight https://review.openstack.org/63890 | 04:36 |
| *** Yathi has quit IRC | 04:47 | |
| *** _nadya_ has joined #openstack-ceilometer | 04:47 | |
| *** _nadya_ has quit IRC | 04:54 | |
| *** Yathi has joined #openstack-ceilometer | 05:18 | |
| *** Yathi has quit IRC | 05:22 | |
| *** Yathi has joined #openstack-ceilometer | 05:35 | |
| *** Yathi has quit IRC | 05:40 | |
| *** AMike has joined #openstack-ceilometer | 05:47 | |
| *** AMike has joined #openstack-ceilometer | 05:47 | |
| *** boris-42_ has quit IRC | 05:58 | |
| openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/62808 | 06:03 |
| *** Yathi has joined #openstack-ceilometer | 06:35 | |
| *** Yathi has quit IRC | 06:40 | |
| *** saju_m has joined #openstack-ceilometer | 06:53 | |
| *** saju_m has quit IRC | 07:00 | |
| *** saju_m has joined #openstack-ceilometer | 07:16 | |
| *** Yathi has joined #openstack-ceilometer | 07:35 | |
| *** Yathi has quit IRC | 07:40 | |
| *** ildikov_ has joined #openstack-ceilometer | 07:43 | |
| *** boris-42_ has joined #openstack-ceilometer | 07:57 | |
| *** lsmola has joined #openstack-ceilometer | 08:05 | |
| *** mihgen has joined #openstack-ceilometer | 08:13 | |
| *** Yathi has joined #openstack-ceilometer | 08:35 | |
| *** Yathi has quit IRC | 08:40 | |
| openstackgerrit | Swann Croiset proposed a change to openstack/ceilometer: Handle Heat notifications for stack CRUD https://review.openstack.org/71939 | 08:42 |
| *** flwang has quit IRC | 08:52 | |
| *** yassine has joined #openstack-ceilometer | 09:12 | |
| *** mihgen has quit IRC | 09:29 | |
| *** eglynn has joined #openstack-ceilometer | 09:30 | |
| *** mihgen has joined #openstack-ceilometer | 09:44 | |
| *** flwang has joined #openstack-ceilometer | 09:50 | |
| *** andreykurilin has quit IRC | 10:02 | |
| *** andreykurilin has joined #openstack-ceilometer | 10:02 | |
| *** xianghui has quit IRC | 10:31 | |
| *** Yathi has joined #openstack-ceilometer | 10:35 | |
| *** Yathi has quit IRC | 10:39 | |
| openstackgerrit | Jia Dong proposed a change to openstack/ceilometer: Implement meter query by 'counter_volume' field https://review.openstack.org/67384 | 10:49 |
| *** yassine has quit IRC | 10:53 | |
| *** mihgen has quit IRC | 11:12 | |
| *** Yathi has joined #openstack-ceilometer | 11:35 | |
| *** Yathi has quit IRC | 11:40 | |
| *** promulo has quit IRC | 12:01 | |
| *** mihgen has joined #openstack-ceilometer | 12:28 | |
| *** Yathi has joined #openstack-ceilometer | 12:35 | |
| *** Yathi has quit IRC | 12:40 | |
| *** eglynn is now known as eglynn-lunch | 12:59 | |
| *** saju_m has quit IRC | 13:12 | |
| nprivalova | eglynn-lunch: hi! please ping me when you become just eglynn :) | 13:13 |
| *** ilyashakhat has quit IRC | 13:14 | |
| *** yassine has joined #openstack-ceilometer | 13:16 | |
| *** promulo has joined #openstack-ceilometer | 13:18 | |
| *** jdob has joined #openstack-ceilometer | 13:20 | |
| *** Alexei_987 has joined #openstack-ceilometer | 13:25 | |
| *** ilyashakhat has joined #openstack-ceilometer | 13:32 | |
| openstackgerrit | Alexei Kornienko proposed a change to openstack/ceilometer: Refactored run-tests script https://review.openstack.org/71255 | 13:33 |
| *** Yathi has joined #openstack-ceilometer | 13:35 | |
| *** tongli has joined #openstack-ceilometer | 13:38 | |
| openstackgerrit | Ilya Tyaptin proposed a change to openstack/ceilometer: Skip unit tests with mongo or db2 when environment variables aren't set https://review.openstack.org/69644 | 13:39 |
| *** Yathi has quit IRC | 13:40 | |
| eglynn-lunch | nick /eglynn | 13:42 |
| *** eglynn-lunch is now known as eglynn | 13:42 | |
| eglynn | nprivalova: ping as requested ... | 13:42 |
| nprivalova | eglynn: I've already discussed with Alexei_987. The question was about difficult impl of get_meters in sql_impl. In hbase and mongo we just go to Resource table, but in sql we join, merge and so on. | 13:44 |
| eglynn | nprivalova: k ... is it now all clear to you, or does it appear that the sql impl is doing the wrong thing? | 13:46 |
| eglynn | (or rather, the suboptimal thing) | 13:46 |
| nprivalova | eglynn: I'm not a big specialist in sql unfortunately so I'm ok with explanation 'We cannot do as in Mongo because db schema is not very good for SQL'. But looks too complicated :( | 13:51 |
| eglynn | nprivalova: ... yeah TBH there is probably much room for improvement in the sqlalchemy driver | 13:51 |
| nprivalova | eglynn: I have a problem https://review.openstack.org/#/c/68435/ . HBase driver improvements are in my plans now but without this HBase doesn't work :( | 13:53 |
| *** julienvey_ has joined #openstack-ceilometer | 13:53 | |
| *** julienvey_ has quit IRC | 13:53 | |
| eglynn | nprivalova: ... is expecting a new release of happybase at all realistic before icehouse? | 13:55 |
| eglynn | nprivalova: ... (if not, it doesn't seem like Thomas Goirand's -1 is realistic, given that the issue apparently can't be fixed on the ceilo side) | 13:55 |
| nprivalova | eglynn: have no idea about their plans actually. But the last comment looks not very optimistic https://github.com/wbolster/happybase/issues/54 It's a bit more complicated; see issue #56. | 13:57 |
| *** thomasem has joined #openstack-ceilometer | 13:59 | |
| eglynn | nprivalova: ... so here are the folks can could potentially over-rule Thomas Goirand's objection | 14:00 |
| eglynn | nprivalova: ... https://review.openstack.org/#/admin/groups/131,members | 14:00 |
| eglynn | jd__: you've got +1 rights on the global requirements repo by the looks of it ^^^ | 14:01 |
| jd__ | eglynn: ? | 14:11 |
| eglynn | jd__: ... wrt the discussion on nadya's attempt to put an upper bound on the happybase dependency version | 14:12 |
| eglynn | jd__: ... seems the ceilo hbase driver is completely borked without the latest happybase, and unclear when the upstream will fix | 14:13 |
| eglynn | jd__: ... for context, here's the global requirements patch | 14:15 |
| eglynn | https://review.openstack.org/#/c/68435/ | 14:15 |
| jd__ | eglynn: borked with or without? | 14:17 |
| eglynn | jd__: borked with latest, so the patch caps the upper version at latest-1 | 14:17 |
| openstackgerrit | Noorul Islam K M proposed a change to openstack/python-ceilometerclient: Remove tox locale overrides https://review.openstack.org/72358 | 14:27 |
| jd__ | eglynn: ok I wasn't sure I got this right | 14:28 |
| jd__ | I'll approve | 14:28 |
| eglynn | jd__: thanks! | 14:28 |
| nprivalova | jd__, eglynn, thanks! | 14:32 |
| *** Yathi has joined #openstack-ceilometer | 14:35 | |
| *** boris-42_ has quit IRC | 14:36 | |
| *** prad has joined #openstack-ceilometer | 14:38 | |
| *** Yathi has quit IRC | 14:39 | |
| *** gordc has joined #openstack-ceilometer | 14:43 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements in operator for complex query functionality https://review.openstack.org/66687 | 14:47 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements field validation for complex query functionality https://review.openstack.org/65302 | 14:47 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarms https://review.openstack.org/62158 | 14:47 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for alarm history https://review.openstack.org/62159 | 14:47 |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/ceilometer: Implements complex query functionality for samples https://review.openstack.org/62157 | 14:47 |
| ildikov_ | gordc:hi. are you around? | 14:59 |
| gordc | ildikov_: yep | 15:00 |
| gordc | whatsup? | 15:00 |
| ildikov_ | gordc: I've found this bug not so long ago: https://bugs.launchpad.net/ceilometer/+bug/1202744 | 15:00 |
| gordc | ildikov_: cool cool. did you want to take over the bug? | 15:01 |
| ildikov_ | gordc: first of all I wanted to discuss your opinion about my idea about how this bug could be handled | 15:01 |
| gordc | ildikov_: sure. let me read it over again to refresh my mind. | 15:01 |
| ildikov_ | gordc: ok, thanks | 15:02 |
| *** saju_m has joined #openstack-ceilometer | 15:02 | |
| gordc | ildikov_: ok. got it. let's hear your master plan. | 15:02 |
| *** saju_m has quit IRC | 15:03 | |
| ildikov_ | gordc: it seems that it's behavior is because the '/' after meter means that you invoke the request with an empty meter id, that is never checked later | 15:04 |
| *** saju_m has joined #openstack-ceilometer | 15:04 | |
| ildikov_ | gordc: in case of the meter endpoint it has currently two functionalities as it returns meters and samples also depending on the request | 15:04 |
| ildikov_ | gordc: so in this case we got an additional feature to retrieve all the samples, if no meter_id is specified in the url | 15:05 |
| ildikov_ | gordc: in my opinion, as it semmes to be useful functionality, it should not be changed | 15:05 |
| ildikov_ | gordc: I'm not sure if the documentation can be fixed somehos or not | 15:06 |
| ildikov_ | gordc: I'm not on that level in sphinx configuration and hacking :) | 15:07 |
| *** gordc has quit IRC | 15:07 | |
| ildikov_ | gordc: so I was wondering if maybe a warning could be added to the documentation about the behavior of this endpoint | 15:08 |
| ityaptin | llu: Hi! | 15:08 |
| *** jmckind has joined #openstack-ceilometer | 15:08 | |
| ildikov_ | gordc:the best would be, if the doc could be properly fixed, but as that part is generated, I'm not sure if it is possible at the moment | 15:09 |
| ildikov_ | gordc: what do you think? | 15:10 |
| *** gordc has joined #openstack-ceilometer | 15:13 | |
| gordc | ildikov_: sorry about that. my internet cut out. i'll read logs to see what you typed. | 15:14 |
| *** andreykurilin has quit IRC | 15:14 | |
| ildikov_ | gordc: ok, np | 15:14 |
| *** rwsu has joined #openstack-ceilometer | 15:17 | |
| gordc | ildikov_: i agree. i think the functionality is correct (maybe not intuitive but correct) | 15:19 |
| gordc | ildikov_: to be honest, i'm not sure how to edit the docs to note that this is intended functionality. | 15:19 |
| ildikov_ | gordc: I think there is a possibility to add a warning box, that the generated doc is not correct | 15:20 |
| ildikov_ | gordc: I do not know, if there is any chance to make sphinx generate it correctly, I think not or at least I could not find a way until now | 15:21 |
| gordc | ah ok... maybe we can find a fix in wsme then. dhellmann might know. | 15:22 |
| gordc | ildikov_: from my point of view.... all the api paths in docs should have the trailing slash dropped. | 15:23 |
| dhellmann | gordc: what's up? | 15:23 |
| ildikov_ | gordc: that sounds reasonable | 15:24 |
| gordc | dhellmann: :) we were talking about this bug: https://bugs.launchpad.net/ceilometer/+bug/1202744 | 15:24 |
| openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Enhance the documentation of the cpu measurement https://review.openstack.org/69637 | 15:24 |
| gordc | seems like the docs generated for api all include a trailing slash in the api paths but that is wrong... should that be fixed in wsme or can that be done in ceilometer? | 15:24 |
| dhellmann | gordc: that sounds like a bug with sphinxcontrib-pecanwsme | 15:25 |
| dhellmann | https://github.com/dreamhost/sphinxcontrib-pecanwsme | 15:25 |
| gordc | dhellmann: ah right. there's that project as well. | 15:25 |
| dhellmann | https://github.com/dreamhost/sphinxcontrib-pecanwsme/blob/master/sphinxcontrib/pecanwsme/rest.py#L119 | 15:25 |
| dhellmann | https://github.com/dreamhost/sphinxcontrib-pecanwsme/blob/master/sphinxcontrib/pecanwsme/rest.py#L119 | 15:26 |
| gordc | ildikov_: did you want to try having a look at that code? i think it'd be a cleaner fix rather than adding a note in. | 15:26 |
| dhellmann | oops, that 2nd link should be line 137 | 15:26 |
| dhellmann | one of those 2 spots, certainly in that method somewhere, we need to not add / | 15:26 |
| ildikov_ | gordc, dhellmann: sure, I'm looking | 15:27 |
| gordc | dhellmann: cool cool. thanks for the pointer | 15:27 |
| dhellmann | it probably needs to be smarter to figure out when to include it and when not | 15:27 |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/python-ceilometerclient: Sync with Oslo. https://review.openstack.org/72392 | 15:29 |
| ildikov_ | gordc, dhellmann: thanks for the help, I will have a deeper look at and I will propose a fix | 15:31 |
| gordc | ildikov_: thanks, i'll switch the bug over to you. | 15:31 |
| ildikov_ | gordc: thanks | 15:31 |
| ildikov_ | gordc: two more tiny things | 15:32 |
| gordc | ildikov_: sure | 15:32 |
| ildikov_ | gordc: I have another bug: https://bugs.launchpad.net/ceilometer/+bug/1272988 | 15:32 |
| ildikov_ | gordc: I wanted to add the tag that it is applicable for havana too | 15:33 |
| ildikov_ | gordc: I think I do not have permission to add it, but I plan to upload the fix for havana also after removing some later transformer specific things from my current uploaded doc fix | 15:35 |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/python-ceilometerclient: Sync with Oslo. https://review.openstack.org/72392 | 15:35 |
| *** Yathi has joined #openstack-ceilometer | 15:35 | |
| gordc | ildikov_: oh. um... let me play around with launchpad. i've never added it before. | 15:37 |
| gordc | ildikov_: you should take it with 'havana-backport-potential' | 15:37 |
| *** saju_m has quit IRC | 15:37 | |
| gordc | ildikov_: added havana target | 15:39 |
| *** saju_m has joined #openstack-ceilometer | 15:39 | |
| *** Yathi has quit IRC | 15:40 | |
| ildikov_ | gordc: thanks | 15:40 |
| *** boris-42_ has joined #openstack-ceilometer | 15:40 | |
| ildikov_ | gordc: the last thing is that I corrected the patch: https://review.openstack.org/#/c/62157/ | 15:41 |
| ildikov_ | gordc: I hope Jenkins will like it as the gate did not seem to be stable an hour ago, but anyway, I corrected the rebase issue, thanks again for reminding | 15:42 |
| gordc | ildikov_: yep. i took a look at it already. was waiting for jenkins but i have the feeling it's still not stable. | 15:43 |
| gordc | ildikov_: it looked fine to me though. | 15:43 |
| ildikov_ | gordc: thanks, good to hear :) | 15:44 |
| ildikov_ | gordc: and also thanks for the bug discussion | 15:45 |
| *** [1]davidp has joined #openstack-ceilometer | 15:46 | |
| gordc | ildikov_: no problem. thanks for looking at the bug (i completely forgot about it.) | 15:46 |
| ildikov_ | gordc: it was half a mistake as I usually filter to the non-assigned bugs, but last time launchpad did not show me the assignees so I just take a look at and I saw that it is an older one, so I thought to ask about it :) | 15:51 |
| *** vrovachev has joined #openstack-ceilometer | 15:52 | |
| openstackgerrit | Balazs Gibizer proposed a change to openstack/ceilometer: Enhance the documentation of the cpu measurement https://review.openstack.org/69637 | 15:59 |
| *** mihgen has quit IRC | 16:07 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/python-ceilometerclient: Python 3: fix format_nested_list_of_dict() https://review.openstack.org/72405 | 16:10 |
| openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: Test case for creating an alarm without auth headers https://review.openstack.org/72406 | 16:11 |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: DBDeadlock exception in sql backend https://review.openstack.org/72414 | 16:31 |
| *** jdob has quit IRC | 16:32 | |
| *** jdob_ has joined #openstack-ceilometer | 16:32 | |
| *** vrovachev is now known as vrovachev1 | 16:34 | |
| *** vrovachev1 is now known as vrovachev | 16:34 | |
| *** Yathi has joined #openstack-ceilometer | 16:35 | |
| *** Yathi has quit IRC | 16:40 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/python-ceilometerclient: Sync with Oslo https://review.openstack.org/72392 | 16:48 |
| *** saju_m has quit IRC | 16:59 | |
| *** saju_m has joined #openstack-ceilometer | 17:04 | |
| *** saju_m has quit IRC | 17:19 | |
| *** Alexei_987 has quit IRC | 17:24 | |
| *** mihgen has joined #openstack-ceilometer | 17:29 | |
| *** yassine has quit IRC | 17:29 | |
| *** saju_m has joined #openstack-ceilometer | 17:33 | |
| *** Yathi has joined #openstack-ceilometer | 17:35 | |
| *** xmltok has joined #openstack-ceilometer | 17:40 | |
| *** Yathi has quit IRC | 17:40 | |
| *** ildikov_ has quit IRC | 17:41 | |
| *** eglynn has quit IRC | 18:08 | |
| *** saju_m has quit IRC | 18:12 | |
| *** saju_m has joined #openstack-ceilometer | 18:29 | |
| *** _nadya_ has joined #openstack-ceilometer | 18:33 | |
| *** rwsu has quit IRC | 18:36 | |
| *** jdob_ has quit IRC | 18:37 | |
| *** jdob has joined #openstack-ceilometer | 18:37 | |
| *** eglynn has joined #openstack-ceilometer | 18:44 | |
| *** eglynn has quit IRC | 18:50 | |
| *** rwsu has joined #openstack-ceilometer | 18:51 | |
| *** saju_m has quit IRC | 18:51 | |
| *** saju_m has joined #openstack-ceilometer | 18:52 | |
| *** _nadya_ has quit IRC | 18:53 | |
| *** saju_m has quit IRC | 18:58 | |
| *** _nadya_ has joined #openstack-ceilometer | 19:03 | |
| *** ildikov_ has joined #openstack-ceilometer | 19:09 | |
| *** eglynn has joined #openstack-ceilometer | 19:13 | |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: DBDeadlock exception in sql backend https://review.openstack.org/72414 | 19:16 |
| openstackgerrit | gordon chung proposed a change to openstack/ceilometer: DBDeadlock exception in sql backend https://review.openstack.org/72414 | 19:17 |
| *** Alexei_987 has joined #openstack-ceilometer | 19:30 | |
| *** ruhe_ has joined #openstack-ceilometer | 19:32 | |
| *** saju_m has joined #openstack-ceilometer | 19:38 | |
| *** _nadya_ has quit IRC | 19:47 | |
| *** eglynn has quit IRC | 19:56 | |
| *** eglynn has joined #openstack-ceilometer | 19:58 | |
| *** saju_m has quit IRC | 20:01 | |
| *** openstack has joined #openstack-ceilometer | 21:33 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Replace BoundedInt with WSME's IntegerType https://review.openstack.org/71715 | 21:51 |
| *** eglynn has quit IRC | 22:09 | |
| *** eglynn has joined #openstack-ceilometer | 22:09 | |
| *** jdob has quit IRC | 22:18 | |
| *** jmckind has quit IRC | 22:27 | |
| *** thomasem has quit IRC | 22:27 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Added abc.ABCMeta metaclass for abstract classes https://review.openstack.org/68565 | 22:29 |
| openstackgerrit | A change was merged to openstack/ceilometer: Handle engine creation inside of Connection object https://review.openstack.org/67851 | 22:30 |
| *** tongli has quit IRC | 22:34 | |
| *** prad has quit IRC | 23:11 | |
| openstackgerrit | A change was merged to openstack/ceilometer: Fixes invalid key in Neutron notifications https://review.openstack.org/71875 | 23:27 |
| *** mihgen has quit IRC | 23:37 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!