| 00:00.15 | *** join/#storm wallflower (n=wallflow@ip-205-246-113-216.pool.grokthis.net) |
| 00:09.35 | *** join/#storm dotz (n=dotz@chello089078004026.chello.pl) |
| 00:34.39 | *** join/#storm ivan (n=ivan@unaffiliated/ivan/x-000001) |
| 03:19.34 | *** join/#storm radix (n=radix@67-207-132-219.slicehost.net) |
| 04:29.52 | *** join/#storm lymxz (n=lymxz@123.116.112.187) |
| 05:28.23 | *** join/#storm radix (n=radix@67-207-132-219.slicehost.net) |
| 06:02.07 | *** join/#storm jukart (n=jukart@85-124-221-45.static.xdsl-line.inode.at) |
| 06:04.14 | *** part/#storm lymxz (n=lymxz@123.116.112.187) |
| 06:14.29 | *** join/#storm bigdog (n=scmikes@72-197-8-8-arpa.cust.cinci.current.net) |
| 06:42.48 | *** join/#storm jukart (n=jukart@194.183.146.181) |
| 07:10.07 | *** join/#storm bozzo (n=bozzo@BSN-210-226-215.dial-up.dsl.siol.net) |
| 07:12.35 | *** join/#storm bozzo (n=bozzo@BSN-210-226-215.dial-up.dsl.siol.net) |
| 07:36.51 | *** join/#storm dobee (n=dobeee@194.183.146.186) |
| 09:14.32 | *** join/#storm goschtl (n=goschtl@p5495D8DB.dip.t-dialin.net) |
| 09:58.34 | *** join/#storm bozzo (n=bozzo@85.10.14.223) |
| 10:40.19 | *** join/#storm jml (n=jml@li2-200.members.linode.com) [NETSPLIT VICTIM] |
| 10:40.19 | *** join/#storm tarjei (n=tarjeih@mail2.bergfald.no) [NETSPLIT VICTIM] |
| 10:56.40 | *** join/#storm lymxz (n=lymxz@123.116.112.187) |
| 12:00.25 | *** join/#storm dobee (n=dobeee@194.183.146.186) |
| 12:39.55 | *** join/#storm niemeyer (n=niemeyer@201-14-56-127.ctame705.dsl.brasiltelecom.net.br) |
| 13:42.22 | *** join/#storm mcella (n=michele@ip-171-30.sn3.eutelia.it) |
| 14:44.20 | *** part/#storm lymxz (n=lymxz@123.116.112.187) |
| 14:47.31 | *** part/#storm jukart (n=jukart@194.183.146.181) |
| 14:48.02 | *** join/#storm jukart (n=jukart@194.183.146.181) |
| 14:48.10 | jukart | test |
| 15:13.52 | radix | hello jukart |
| 15:15.08 | jukart | hi radix |
| 15:15.43 | jukart | niemeyer: did you see my bug report |
| 15:15.51 | radix | I saw it. |
| 15:16.15 | radix | I didn't understand why you added list() around the iteration argument. |
| 15:16.50 | jukart | radix: because the iterates list ist changed inside repeat |
| 15:17.10 | jukart | it is changes inside remove |
| 15:17.15 | radix | huh? it can't be, it's not bound to a variable anywhere. |
| 15:17.31 | radix | ohhh, you're affecting the source of the generator. |
| 15:17.39 | jukart | right |
| 15:18.14 | jukart | do you think this change will made it into the next egg release ? |
| 15:18.35 | jukart | because right now I made a monkey patch for us |
| 15:18.43 | jukart | so that we can work |
| 15:19.00 | radix | hm. |
| 15:19.21 | radix | jukart: out of curiosity, how does the bug manifest? |
| 15:19.39 | radix | We use zstorm thoroughly in our layer-using test suite in Landscape |
| 15:20.40 | radix | jukart: also, I'd be much happier if you actually triggered the zope cleanup mechanism in your unit test instead of calling _reset directly |
| 15:21.02 | radix | that way we make sure that it's properly registered |
| 15:23.03 | jukart | _reset is called as cleanup because it is defined in zstorm.py, but it did not removed the StoreSyncronizer leaving a store with a _connection but the connection had no _raw_connection maybe because the store was closed in _reset |
| 15:23.39 | jukart | with this fix it will work properly even without my monkey patch |
| 15:25.12 | radix | ah, so you get an AttributeError? |
| 15:25.41 | radix | that's kind of strange. was it "NoneType has no attribute _raw_connection"? that would make a bit more sense to me |
| 15:26.27 | jukart | no, NoneType has no commit for _raw_connection |
| 15:26.34 | jukart | so _raw_connection was None |
| 15:27.38 | radix | ahhh. ok. |
| 15:27.58 | radix | jukart: I wonder why this is happening for you but not us |
| 15:28.12 | radix | maybe because our zstorm-using layers run *last*.. |
| 15:28.37 | radix | jukart: anyway yes, we can probably get this fix in before the next release |
| 15:40.08 | jukart | radix: yes if your layer runs last then you do not realize the bug |
| 15:40.57 | jukart | it would be really nice if the fix is in the next release, after more than one day working on this :) |
| 16:15.52 | *** join/#storm bigdog (n=scmikes@72-197-8-8-arpa.cust.cinci.current.net) |
| 16:57.28 | radix | niemeyer: Just +1ed your mysql-reconnect branch |
| 16:57.36 | radix | looks like it still needs another :\ |
| 16:57.42 | radix | hopefully jkakar can help with it today |
| 17:00.57 | radix | niemeyer: did you have other things you wanted to do on that strong-cache branch |
| 17:00.58 | radix | ? |
| 17:12.40 | *** join/#storm bigdog (n=scmikes@72-197-8-8-arpa.cust.cinci.current.net) |
| 17:15.08 | *** part/#storm bigdog (n=scmikes@72-197-8-8-arpa.cust.cinci.current.net) |
| 17:36.06 | jkakar | niemeyer: I'll take a peek at the mysql-connection branch in a few minutes. |
| 17:37.11 | jkakar | niemeyer: If you're around I'd like to make the release today. |
| 17:37.18 | jkakar | My debian-package branch needs a review, too. |
| 17:50.21 | niemeyer | radix: THanks |
| 17:50.31 | niemeyer | radix: Nope.. this one is good to go I believe |
| 17:50.40 | niemeyer | jkakar: Thanks |
| 17:51.26 | radix | niemeyer: ok, I'm reviewing it.. well, I was, until the drama over in that other channel started :) |
| 17:52.26 | jkakar | Heh |
| 18:12.38 | jukart | radix: are there any issues with multiple threads in zope and the datamanager |
| 18:12.49 | radix | jukart: not sure what you mean |
| 18:13.04 | radix | jukart: the transaction manager is of course meant to be thread safe (otherwise there would be no nead for the transaction manager) |
| 18:13.24 | jukart | right, i explain my current problem : |
| 18:14.02 | jukart | when i start zope the first transaction works fine |
| 18:14.30 | jukart | the second did not activate a datamanager, therfore I get no commit |
| 18:15.28 | jukart | after some more requests it activates a new datamanager and commits all other changes from the old requests |
| 18:16.01 | jukart | very strange |
| 18:16.18 | radix | I'm not terribly surprised by the "commits all other changes from the old requests" bit |
| 18:16.27 | radix | but I don't know why they're failing in the first place |
| 18:16.41 | radix | jukart: is this only happening with your patch, or with current trunk? |
| 18:17.06 | jukart | radix: I removed my path and it's the same |
| 18:18.56 | jukart | radix: I'm debugging a little more |
| 18:19.57 | *** join/#storm mcella (n=michele@ip-171-30.sn3.eutelia.it) |
| 18:30.51 | jkakar | Hmm... looks like there is already a python-storm Debian package in universe... I wonder if it's still worth us publishing our own packages in the Storm Team PPA. |
| 18:30.52 | jkakar | I think it is. |
| 18:43.54 | jkakar | niemeyer: Review delivered for mysql-reconnection. |
| 18:51.19 | jukart | radix: I found the problem, I used a lazy property to get the store which is really a bad thing :( |
| 18:59.17 | niemeyer | jkakar: Thanks |
| 18:59.31 | radix | jkakar: or, you can just get the guy who packaged it for universe to update the packages :) |
| 18:59.53 | radix | well, I guess if he doesn't do dapper et al packages we'd have to build them ourselves |
| 19:00.08 | jkakar | niemeyer: np |
| 19:01.03 | jkakar | radix: Yeah. |
| 19:35.39 | *** join/#storm dobee (n=dobeee@81-223-53-162.dornbirn.xdsl-line.inode.at) |
| 20:07.15 | *** join/#storm bigdog (n=scmikes@72-197-8-8-arpa.cust.cinci.current.net) |
| 20:24.10 | *** join/#storm ubot3 (n=ubot3@unaffiliated/nalioth/bot/ubot3) |
| 22:04.25 | radix | welp, there's a review |
| 22:04.29 | radix | for strong-caching |