[Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-05-12 Thread Alexandre Fayolle - camptocamp
The proposal to merge lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138 --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-05-12 Thread Alexandre Fayolle - camptocamp
thanks for the review, Daniel. I'm setting this back to WIP until I get the time for take your comments into account. -- https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138 Your team Server Environment And Tools Core Editors is subscribed to branch

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-05-02 Thread Daniel Reis
On load stress, I was able to produce SERIALIZATION_FAILURE errors, but not sure if this is rather an oe-server performance limitaion instead of a problem specific of the module. (I opened several browser windows and loaded in all of them a page that takes several seconds). 2014-05-02

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-05-01 Thread Nicolas Bessi - Camptocamp
I can confirm I got the same bug as Daniel. -- https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-04-23 Thread Daniel Reis
I also noticed that only 20-25% of process log lines have the model field filled. Why is this? -- https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing

[Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-04-10 Thread Alexandre Fayolle - camptocamp
Alexandre Fayolle - camptocamp has proposed merging lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools. Requested reviews: Server Environment And Tools Core Editors (server-env-tools-core-editors) For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-04-10 Thread Nicolas Bessi - Camptocamp
Review: Needs Fixing Hello, Thanks for the module, it is really a valuable tool. Some comments below: Some PEP8 error in manifest 728+ name = '%s.%s' % (cls.__module__, cls.__name__) There is a logical risk that cls is not initialized if it is not an old/new style class. 729 +

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-04-10 Thread Alexandre Fayolle - camptocamp
Hello, Thanks for the module, it is really a valuable tool. Some comments below: Some PEP8 error in manifest fixed 728+ name = '%s.%s' % (cls.__module__, cls.__name__) There is a logical risk that cls is not initialized if it is not an old/new style class. fixed 729 +

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-04-10 Thread Nicolas Bessi - Camptocamp
Review: Approve Ahh tests, I'm feeling better now. Thanks for the fixes LGTM :-D -- https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: