*** shaneduan is now known as shaneduan[afk] | 00:16 | |
*** evanjfraser has quit IRC | 00:46 | |
*** evanjfraser has joined #openstack-metering | 00:59 | |
*** shaneduan[afk] is now known as shaneduan | 01:16 | |
*** shaneduan is now known as shaneduan[afk] | 01:21 | |
*** nosnos has joined #openstack-metering | 01:22 | |
*** flwang has quit IRC | 01:42 | |
*** matsuhashi has joined #openstack-metering | 01:56 | |
*** d34dh0r53 has joined #openstack-metering | 01:59 | |
*** d34dh0r53 has quit IRC | 02:01 | |
*** flwang has joined #openstack-metering | 02:24 | |
*** anteaya has quit IRC | 02:54 | |
*** shaneduan[afk] is now known as shaneduan | 03:21 | |
*** shaneduan is now known as shaneduan[afk] | 03:30 | |
*** dhellmann has quit IRC | 04:01 | |
*** shaneduan[afk] is now known as shaneduan | 04:22 | |
*** shaneduan is now known as shaneduan[afk] | 04:31 | |
*** matsuhas_ has joined #openstack-metering | 05:00 | |
*** matsuhashi has quit IRC | 05:00 | |
*** shaneduan[afk] is now known as shaneduan | 05:06 | |
*** matsuhas_ has quit IRC | 05:26 | |
*** evanjfraser has quit IRC | 05:28 | |
*** matsuhashi has joined #openstack-metering | 05:29 | |
*** nosnos has quit IRC | 05:32 | |
*** nosnos has joined #openstack-metering | 05:32 | |
*** matsuhashi has quit IRC | 05:34 | |
*** boris-42 has joined #openstack-metering | 05:39 | |
*** matsuhashi has joined #openstack-metering | 05:39 | |
openstackgerrit | Jenkins proposed a change to openstack/ceilometer: Imported Translations from Transifex https://review.openstack.org/45175 | 06:02 |
---|---|---|
*** eglynn_ has joined #openstack-metering | 06:07 | |
openstackgerrit | Nick Barcet proposed a change to openstack/ceilometer: Update the high level architecture https://review.openstack.org/44774 | 06:27 |
openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Add testing of migrations from oslo https://review.openstack.org/43872 | 06:41 |
*** shaneduan is now known as shaneduan[afk] | 06:44 | |
*** mmcardle has joined #openstack-metering | 06:50 | |
*** matsuhashi has quit IRC | 06:57 | |
*** nosnos_ has joined #openstack-metering | 07:00 | |
*** nosnos has quit IRC | 07:01 | |
*** matsuhashi has joined #openstack-metering | 07:01 | |
*** boris-42 has quit IRC | 07:11 | |
*** eglynn_ has quit IRC | 07:13 | |
*** shaneduan[afk] is now known as shaneduan | 07:13 | |
*** Ruetobas has joined #openstack-metering | 07:20 | |
*** shaneduan is now known as shaneduan[afk] | 07:22 | |
*** Ruetobas has quit IRC | 07:25 | |
*** Ruetobas has joined #openstack-metering | 07:32 | |
*** shardy is now known as shardy_afk | 07:41 | |
*** eglynn_ has joined #openstack-metering | 07:56 | |
*** flwang has quit IRC | 08:09 | |
*** flwang has joined #openstack-metering | 08:10 | |
*** boris-42 has joined #openstack-metering | 08:31 | |
*** flwang has quit IRC | 08:39 | |
*** flwang has joined #openstack-metering | 08:39 | |
*** flwang has quit IRC | 08:53 | |
*** flwang1 has joined #openstack-metering | 08:53 | |
*** flwang1 has quit IRC | 08:57 | |
nijaba | jd__: is it me or is one of the test broken atm? http://logs.openstack.org/74/44774/3/check/gate-ceilometer-python26/96cc90f | 08:58 |
jd__ | nijaba: looks like it | 09:02 |
* nijaba somehow feels better :) | 09:02 | |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: run-tests: fix MongoDB start wait https://review.openstack.org/45642 | 09:05 |
jd__ | maybe that'll help | 09:06 |
*** zbitter is now known as zaneb | 09:11 | |
*** flwang has joined #openstack-metering | 09:11 | |
openstackgerrit | Julien Danjou proposed a change to openstack/ceilometer: run-tests: fix MongoDB start wait https://review.openstack.org/45642 | 09:13 |
jd__ | it doesn't | 09:14 |
jd__ | sileht: around? | 09:14 |
sileht | jd__, o/ | 09:16 |
jd__ | sileht: mongodb is broken again | 09:17 |
sileht | jd__, .... | 09:17 |
jd__ | hum | 09:17 |
jd__ | pymongo 2.6.2 released on 6th september | 09:17 |
jd__ | coincidence | 09:17 |
jd__ | I DON'T THINK SO | 09:17 |
jd__ | @*$#&@*(#&@*#@( | 09:17 |
*** flwang has quit IRC | 09:21 | |
sileht | jd__, I have spent a tone of time to find the reference leak, I have isolated the it to the api test, but I have not found where exactly | 09:21 |
jd__ | the diff between 2.6 and 2.6.2 doesn't look suspicious | 09:22 |
jd__ | well at least I can reproduce locally with pymongo 2.6.2 | 09:23 |
jd__ | I try with 2.6 to be sure | 09:23 |
sileht | jd__, Alexei have implemented the workaround with a pool by host, https://review.openstack.org/#/c/45202/ | 09:23 |
sileht | jd__, it seems the work when the mongoclient is created without the database/user/pass | 09:24 |
sileht | jd__, you can try it workaround... but this is just a other work around | 09:24 |
jd__ | :] | 09:25 |
jd__ | sileht: I triggered a recheck on it to be sure it works | 09:25 |
jd__ | I'm happy with workarounds for now | 09:26 |
sileht | I'm not very happy to manipulate the mongo url, but ... if it works | 09:26 |
*** matsuhashi has quit IRC | 09:26 | |
*** Alexei_987 has joined #openstack-metering | 09:26 | |
*** matsuhashi has joined #openstack-metering | 09:26 | |
jd__ | sileht: pymongo==2.6 makes things work back to normal | 09:32 |
sileht | damn it | 09:32 |
jd__ | trying with 2.6.1 | 09:32 |
sileht | jd__, changelog: Version 2.6.2 fixes a :exc:`TypeError` problem when max_pool_size=None | 09:33 |
jd__ | sileht: yeah, that's probably what triggers the problem | 09:35 |
jd__ | self.lock.acquire() | 09:36 |
jd__ | - if (len(self.sockets) < self.max_size | 09:36 |
jd__ | - and sock_info.pool_id == self.pool_id): | 09:36 |
jd__ | + too_many_sockets = (self.max_size is not None | 09:36 |
jd__ | + and len(self.sockets) >= self.max_size) | 09:36 |
jd__ | + | 09:36 |
jd__ | 09:36 | |
jd__ | int < None works in Py2 indeed | 09:36 |
jd__ | and it's False so the code wasn't adding sockets | 09:37 |
jd__ | now it does | 09:37 |
*** Alexei_987 has quit IRC | 09:38 | |
sileht | jd__, Do you change the requirements and propose a upstream fix ? | 09:50 |
jd__ | sileht: I don't know if upstream is broken in this regard? | 09:51 |
jd__ | or fixed | 09:51 |
jd__ | the new behaviour maybe more logical? | 09:51 |
jd__ | anyway I'll try to fix it once for all | 09:51 |
jd__ | if you've time to dig into it feel free, I'm multitasking | 09:52 |
sileht | jd__, I don't think I will find the time today :( | 09:52 |
jd__ | sileht: np | 09:52 |
openstackgerrit | Svetlana Shturm proposed a change to openstack/ceilometer: Implement testing of migrations https://review.openstack.org/44809 | 10:14 |
*** matsuhashi has quit IRC | 10:30 | |
*** flwang has joined #openstack-metering | 10:31 | |
*** nosnos_ has quit IRC | 10:39 | |
*** ruhe has joined #openstack-metering | 10:42 | |
jd__ | I conclude we cannot do better than https://review.openstack.org/#/c/45202 for now so if someone agrees, please approve | 10:55 |
*** ruhe has quit IRC | 10:59 | |
sileht | jd__, ok | 11:08 |
jd__ | sileht: I didn't check but I think the time wait connection are handled as active connection by mongodb or something like that | 11:10 |
jd__ | the protocol should have some sort of QUIT or something it doesn't have | 11:11 |
sileht | jd__, I think it have that, TIME_WAIT is just to keep opened the socket on the kernel part, to catch packets lost in the network that arrive too late (after the APP quit) | 11:13 |
jd__ | sileht: but that doesn't mean the server socket is considered closed by MongoDB AFAICT | 11:25 |
*** ruhe has joined #openstack-metering | 11:30 | |
flwang | jd__: ping | 11:33 |
jd__ | flwang: ? | 11:33 |
flwang | jd__: about this bug you opened https://bugs.launchpad.net/ceilometer/+bug/1222737 | 11:33 |
flwang | I think it works as designed from the neutron perspective | 11:34 |
jd__ | I don't know, though you'd better post your comment in the bug report | 11:34 |
flwang | jd__: since when the network.create.start sending, there is no network id | 11:34 |
flwang | jd__: sure, I will | 11:34 |
jd__ | you're probably right indeed | 11:35 |
flwang | jd__: so if it's right action for Neutron, what should we do? | 11:36 |
jd__ | flwang: fix Ceilometer's code? | 11:38 |
jd__ | if there's no resource id there's no point listening in start events I guess | 11:39 |
flwang | jd__: yep, though maybe it's the last sort | 11:39 |
flwang | jd__: I will review the Neutron code to confirm if it's existed for other notifications | 11:39 |
jd__ | thanks flwang | 11:41 |
flwang | jd__: sure, I will update the bug comments | 11:42 |
*** anteaya has joined #openstack-metering | 11:53 | |
*** ruhe has quit IRC | 11:59 | |
*** shengjie has joined #openstack-metering | 12:00 | |
*** dhellmann has joined #openstack-metering | 12:06 | |
*** shengjie has quit IRC | 12:08 | |
*** thomasm has joined #openstack-metering | 12:22 | |
thomasm | Hey all | 12:22 |
*** ruhe has joined #openstack-metering | 12:46 | |
*** swann has joined #openstack-metering | 12:58 | |
*** mmcardle has quit IRC | 13:03 | |
*** changbl has quit IRC | 13:12 | |
openstackgerrit | A change was merged to openstack/ceilometer: Improved MongoClient pooling to avoid out of connections error https://review.openstack.org/45202 | 13:13 |
jd__ | hey | 13:40 |
jd__ | if anyone has 10 minutes to dig into https://review.openstack.org/#/c/39237/ failure I'll be interested | 13:41 |
jd__ | I can't see what's wrong | 13:41 |
*** eglynn_ has quit IRC | 13:47 | |
thomasm | jd__, I'd be interested to see what the URL is that's being built | 13:50 |
*** mmcardle has joined #openstack-metering | 13:51 | |
jd__ | thomasm: according to log it seems to be http://localhost:8777/something | 13:51 |
jd__ | which seems correct to me | 13:51 |
jd__ | and it works on my devstack btw | 13:51 |
thomasm | weird | 13:52 |
thomasm | Hmmm | 13:52 |
*** fnaval_ has joined #openstack-metering | 13:52 | |
thomasm | Ah, yeah, it gets the token, then hits the alarms API. Okay. | 13:52 |
*** gordc has joined #openstack-metering | 13:53 | |
sileht | jd__, ceilometer services seems not enabled in the devstack | 13:53 |
jd__ | sileht: hint? | 13:54 |
sileht | jd__, look for 'local_services=' in the log | 13:55 |
jd__ | sileht: which log? | 13:55 |
jd__ | that's true for grenade | 13:55 |
jd__ | not for the others | 13:55 |
sileht | jd__, arf I have take a look in the grenade one ... | 13:56 |
jd__ | yeah sorry, I didn't point out that | 13:56 |
thomasm | jd__, Is there any connection limiting that's not being accounted for? | 14:00 |
jd__ | thomasm: there's no such thing as far as I know | 14:01 |
thomasm | Hmm | 14:02 |
thomasm | jd__, How did all of those failures in the first patch set still get marked as 'works for me' from Jenkins? | 14:11 |
thomasm | Meh, separate issue | 14:13 |
*** changbl has joined #openstack-metering | 14:18 | |
*** eglynn has joined #openstack-metering | 14:24 | |
thomasm | Running the tests on my devstack now | 14:24 |
*** litong has joined #openstack-metering | 14:30 | |
*** shang has joined #openstack-metering | 14:33 | |
*** SergeyLu_ has joined #openstack-metering | 14:42 | |
*** ruhe has quit IRC | 14:43 | |
*** SergeyLu_ has quit IRC | 14:46 | |
thomasm | Whoa these things take a while | 14:53 |
*** ruhe has joined #openstack-metering | 14:54 | |
*** Ruetobas has quit IRC | 14:57 | |
*** Ruetobas has joined #openstack-metering | 14:58 | |
litong | @thomasm, hi, the patchset is just sitting there. | 15:00 |
thomasm | litong, Yeah, I'm probably going to have to rebase now. | 15:01 |
litong | if you simply click on the rebase on the web page, it may work if no conflict. | 15:01 |
litong | @thomasm, not sure how many patchset got merged last few days, I did not see a lot though. | 15:02 |
*** krtaylor has quit IRC | 15:02 | |
thomasm | jd__, dhellmann, gordc: Might I get some reviews here? =] https://review.openstack.org/#/c/44277/ | 15:03 |
thomasm | litong, Yeah, only one touching the drivers - connection pooling in mongo | 15:03 |
thomasm | That I saw anyway | 15:04 |
litong | you mean this one? https://review.openstack.org/#/c/45202/ | 15:05 |
thomasm | yep | 15:05 |
litong | yeah, I looked at that patch set & reviewed it. should have no impact to your patch, but need rebase. | 15:06 |
thomasm | Yeah | 15:06 |
jd__ | thomasm: why the pot file is changed? | 15:07 |
thomasm | jd__, A couple of exceptions didn't get added there. I just realized I need to remove the exception that's no longer applicable, though. | 15:08 |
jd__ | thomasm: ok, I don't think you shuold care about it since it's automatically managed AFAICT | 15:09 |
thomasm | jd__, Oh, I didn't know that. I just remember Wang making a comment about being sure the POT file was updated. | 15:09 |
thomasm | jd__, Does it automate off of the nationalization method '_(?)'? | 15:10 |
jd__ | thomasm: no just the po file generation | 15:10 |
thomasm | jd__, Well, sure, but the problem there was that those messages weren't nationalized yet. When I added the '_(?)' to those log messages? how would it know? | 15:11 |
thomasm | Is the question I'm asking | 15:11 |
jd__ | thomasm: transifex does the magic AFAIK | 15:11 |
thomasm | hmmmm | 15:11 |
thomasm | okay | 15:11 |
thomasm | jd__, Ought I remove the POT file changes, then? | 15:11 |
jd__ | thomasm: I think so | 15:11 |
thomasm | jd__, Okay | 15:11 |
openstackgerrit | Cyril Roelandt proposed a change to openstack/python-ceilometerclient: Fix a typo in "sample-create" help message https://review.openstack.org/45702 | 15:33 |
*** shaneduan[afk] is now known as shaneduan | 15:34 | |
*** sdake has joined #openstack-metering | 15:38 | |
*** jergerber has joined #openstack-metering | 15:41 | |
*** boris-42 has quit IRC | 15:43 | |
*** Ruetobas has quit IRC | 16:01 | |
*** ruhe has quit IRC | 16:01 | |
*** Ruetobas has joined #openstack-metering | 16:04 | |
*** zaneb has quit IRC | 16:06 | |
*** shaneduan is now known as shaneduan[afk] | 16:09 | |
*** Ruetobas has quit IRC | 16:09 | |
*** Ruetobas has joined #openstack-metering | 16:13 | |
thomasm | Anyone else having version conflicts in their tests? Looks like something was changed to require 'requests <1.2.3'. | 16:17 |
*** zaneb has joined #openstack-metering | 16:22 | |
*** ruhe has joined #openstack-metering | 16:23 | |
*** ruhe has quit IRC | 16:24 | |
*** shaneduan[afk] is now known as shaneduan | 16:25 | |
*** ruhe has joined #openstack-metering | 16:27 | |
*** eglynn has quit IRC | 16:29 | |
*** eglynn has joined #openstack-metering | 16:29 | |
thomasm | Can anyone tell me where our requirements.txt are derived from? Is it just CM needs, or overall? My unit tests are failing, it looks like, because the venv is using python-cinderclient, which requires a lesser version of the requests library. | 16:37 |
*** ruhe has quit IRC | 16:39 | |
*** bmqq123_away is now known as yjiang5 | 16:41 | |
*** Bada has joined #openstack-metering | 16:47 | |
*** ruhe has joined #openstack-metering | 16:48 | |
*** mmcardle has quit IRC | 16:56 | |
*** boris-42 has joined #openstack-metering | 17:04 | |
*** openstackgerrit has quit IRC | 17:05 | |
*** openstackgerrit has joined #openstack-metering | 17:05 | |
*** fandikurnia02 has joined #openstack-metering | 17:09 | |
gordc | thomasm: requirements are based off: https://github.com/openstack/requirements/blob/master/global-requirements.txt | 17:17 |
thomasm | gordc, Thanks | 17:18 |
openstackgerrit | Thomas Maddox proposed a change to openstack/ceilometer: Fix to return latest resource metadata https://review.openstack.org/44277 | 17:24 |
thomasm | litong, rebased: https://review.openstack.org/#/c/44277/ | 17:25 |
litong | great. thanks. | 17:25 |
thomasm | ran into some testing version snags and what-not? Not sure what all happened there. | 17:25 |
thomasm | jd__, rebased and removed POT file changes: https://review.openstack.org/#/c/44277/ | 17:26 |
litong | @thomasm, so they are resolved I figure? | 17:26 |
thomasm | litong, yeah, the tests passed. | 17:26 |
litong | k. nice, | 17:27 |
thomasm | =] | 17:27 |
thomasm | gordc, if you have a minute or two: https://review.openstack.org/#/c/44277/ | 17:28 |
thomasm | I'd so very very much appreciate it | 17:28 |
gordc | thomasm: sure. will take a look later today. | 17:29 |
thomasm | Wunderbar! Thanks! | 17:29 |
*** nijaba has quit IRC | 17:35 | |
*** SergeyLukjanov has joined #openstack-metering | 17:39 | |
*** ruhe has quit IRC | 17:52 | |
*** jergerber has quit IRC | 17:56 | |
*** shaneduan is now known as shaneduan[afk] | 18:00 | |
*** shaneduan[afk] is now known as shaneduan | 18:07 | |
thomasm | gordc, Oh, yeah - I had left it that way since you had asked me to initially. Let me refactor that really fast, sorry for wasting your time. | 18:19 |
thomasm | gordc, I was concerned about making changes since I don't have a DB2 deployment. | 18:19 |
thomasm | gordc, But, I'll just use itertools and not change any of the DB query. | 18:19 |
gordc | thomasm: np. i kind of assumed it was left out because we didn't formally decide whether to change it or not. | 18:20 |
thomasm | gordc, Sure. I am running a test on my refactor now. Removes both additional functions. | 18:20 |
gordc | thomasm: based on Tong's comment, you can just remove line 427 and 467 | 18:21 |
gordc | thomasm: cool cool | 18:21 |
*** changbl has quit IRC | 18:27 | |
*** changbl has joined #openstack-metering | 18:30 | |
thomasm | Meh? really wish I had aggregation, lol. I am not entirely sure how well this will perform. | 18:44 |
thomasm | *crosses fingers* last test I hope | 18:48 |
*** herndon_ has joined #openstack-metering | 18:50 | |
openstackgerrit | Thomas Maddox proposed a change to openstack/ceilometer: Fix to return latest resource metadata https://review.openstack.org/44277 | 18:54 |
thomasm | gordc, ^^ | 18:54 |
thomasm | =] | 18:54 |
litong | @thomasm, ping. | 18:54 |
thomasm | litong, pong | 18:54 |
thomasm | litong, ^^ will that change work? | 18:55 |
litong | just saw your patch to address the comment, | 18:55 |
thomasm | I didn't touch the DB query, just what we do with the result. | 18:55 |
litong | let me take a look. | 18:55 |
thomasm | Well, explicitly stated 'desc' sort_dir, but that was defaulted in the build_sort_instruction func anyway. | 18:56 |
thomasm | I just wanted to be sure defaults don't hurt us. | 18:56 |
litong | @thomasm, let me try it with db2. you refactored quite a bit. | 18:58 |
thomasm | litong, Yeah? It's like a hybrid between the HBase implementation and Mongo implementation. HBase has the same problem where the library doesn't really facilitate aggregation or anything. | 18:59 |
thomasm | At least, not well. =] | 19:00 |
litong | @thomasm, let me try it against real db2, looks find to me. | 19:00 |
thomasm | litong, Awesome. Thanks! | 19:00 |
litong | @thomasm, thanks. | 19:00 |
litong | @thomasm, I tested it, and it worked fine to me. | 19:06 |
litong | for me. | 19:06 |
thomasm | litong, great! Thanks for doing that. | 19:06 |
litong | @thomasm, thanks a lot. | 19:06 |
thomasm | My pleasure! I got to learn more about itertools. =] | 19:07 |
*** krtaylor has joined #openstack-metering | 19:08 | |
*** Ruetobas has quit IRC | 19:11 | |
*** ruhe has joined #openstack-metering | 19:16 | |
*** jtran has joined #openstack-metering | 19:33 | |
jtran | jd__: do u know if there was a bug opened for this issue (notifications.info queue) http://openstack.10931.n7.nabble.com/Re-Openstack-Ceilometer-and-notifications-td16642.html | 19:33 |
*** Ranzpirat has quit IRC | 19:40 | |
*** Ranzpirat has joined #openstack-metering | 19:47 | |
*** Ranzpirat has quit IRC | 19:48 | |
*** Ranzpirat has joined #openstack-metering | 19:49 | |
*** ruhe has quit IRC | 19:58 | |
*** boris-42 has quit IRC | 20:11 | |
*** boris-42 has joined #openstack-metering | 20:13 | |
*** fnaval__ has joined #openstack-metering | 20:17 | |
*** fnaval___ has joined #openstack-metering | 20:17 | |
*** fnaval_ has quit IRC | 20:20 | |
*** boris-42 has quit IRC | 20:21 | |
*** fnaval__ has quit IRC | 20:21 | |
*** evanjfraser has joined #openstack-metering | 20:23 | |
*** russell_h has quit IRC | 20:31 | |
*** russell_h has joined #openstack-metering | 20:34 | |
*** thomasm has quit IRC | 20:41 | |
Daviey | et | 20:44 |
*** Bada has quit IRC | 20:54 | |
litong | privmsg thomasm, hi | 21:11 |
litong | privmsg @thomasm, ping | 21:12 |
litong | raw PRIVMSG thomasm :hi | 21:14 |
*** nati_uen_ has joined #openstack-metering | 21:36 | |
*** shang has quit IRC | 21:57 | |
*** sdake has quit IRC | 22:00 | |
*** sdake has joined #openstack-metering | 22:00 | |
*** SergeyLukjanov has quit IRC | 22:01 | |
*** zaneb has quit IRC | 22:17 | |
*** changbl has quit IRC | 22:26 | |
*** nati_uen_ has quit IRC | 22:30 | |
*** nati_ueno has joined #openstack-metering | 22:30 | |
*** herndon_ has quit IRC | 22:39 | |
*** krtaylor has quit IRC | 22:45 | |
*** gordc has quit IRC | 22:53 | |
*** zaneb has joined #openstack-metering | 23:10 | |
*** fnaval_ has joined #openstack-metering | 23:44 | |
*** fnaval___ has quit IRC | 23:47 | |
*** fnaval_ has quit IRC | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!