Merge lp:~nataliabidart/wsgi-oops/restore-bz2-serializer into lp:wsgi-oops

Proposed by Natalia Bidart
Status: Merged
Approved by: Eric Casteleijn
Approved revision: 44
Merged at revision: not available
Proposed branch: lp:~nataliabidart/wsgi-oops/restore-bz2-serializer
Merge into: lp:wsgi-oops
Diff against target: 21 lines
1 file modified
canonical/oops/wsgi.py (+2/-2)
To merge this branch: bzr merge lp:~nataliabidart/wsgi-oops/restore-bz2-serializer
Reviewer Review Type Date Requested Status
Eric Casteleijn (community) Approve
Rick McBride (community) Approve
Review via email: mp+14414@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Restoring Bz2 serializer for wsgi apps.

Revision history for this message
Rick McBride (rmcbride) wrote :

Looks good!

review: Approve
Revision history for this message
Eric Casteleijn (thisfred) wrote :

Looks good, is there any way I can run tests for wsgi-oops locally? (have not done any reviews for it before)

review: Approve
Revision history for this message
Rick McBride (rmcbride) wrote :

Eric Casteleijn wrote:
> Review: Approve
> Looks good, is there any way I can run tests for wsgi-oops locally? (have not done any reviews for it before)

'make test' works in that particular package.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'canonical/oops/wsgi.py'
--- canonical/oops/wsgi.py 2009-10-26 02:02:15 +0000
+++ canonical/oops/wsgi.py 2009-11-04 12:20:26 +0000
@@ -33,7 +33,7 @@
3333
34from storm.tracer import install_tracer, remove_tracer_type34from storm.tracer import install_tracer, remove_tracer_type
3535
36from canonical.oops.serializer import OOPSRFC822Serializer36from canonical.oops.serializer import OOPSBz2Serializer as OOPSSerializer
37from canonical.oops.oops import (37from canonical.oops.oops import (
38 OOPS, OOPSLog, OOPSMetaData, OOPSLoggingHandler )38 OOPS, OOPSLog, OOPSMetaData, OOPSLoggingHandler )
39from canonical.oops.stormtracer import OOPSStorm, OOPSStormTracer39from canonical.oops.stormtracer import OOPSStorm, OOPSStormTracer
@@ -60,7 +60,7 @@
60 else:60 else:
61 root_logger.addHandler(OOPSLoggingHandler())61 root_logger.addHandler(OOPSLoggingHandler())
62 fh = debug and sys.stderr or None62 fh = debug and sys.stderr or None
63 self.serial = OOPSRFC822Serializer("appserver", oops_dir, fh=fh)63 self.serial = OOPSSerializer("appserver", oops_dir, fh=fh)
6464
65 def __call__(self, environ, start_response):65 def __call__(self, environ, start_response):
66 """66 """

Subscribers

People subscribed via source and target branches