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
--
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
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
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:
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
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:
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 +
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 +
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:
9 matches
Mail list logo