Merge lp://staging/~nataliabidart/magicicada-server/no-u1-devtools into lp://staging/magicicada-server

Proposed by Natalia Bidart
Status: Merged
Approved by: Natalia Bidart
Approved revision: 87
Merged at revision: 70
Proposed branch: lp://staging/~nataliabidart/magicicada-server/no-u1-devtools
Merge into: lp://staging/magicicada-server
Diff against target: 2948 lines (+433/-855)
28 files modified
build/config-manager.txt (+0/-1)
dependencies-devel.txt (+1/-1)
lib/ubuntuone/supervisor/heartbeat_listener.py (+0/-13)
lib/utilities/testlib.py (+4/-3)
magicicada/metrics/__init__.py (+2/-6)
magicicada/metrics/tests/test_base.py (+21/-39)
magicicada/monitoring/reactor.py (+20/-16)
magicicada/monitoring/tests/test_reactor.py (+3/-5)
magicicada/server/auth.py (+1/-1)
magicicada/server/content.py (+6/-7)
magicicada/server/integtests/test_server_scan.py (+0/-5)
magicicada/server/integtests/test_sync.py (+10/-31)
magicicada/server/logger.py (+0/-204)
magicicada/server/server.py (+36/-76)
magicicada/server/ssl_proxy.py (+2/-4)
magicicada/server/stats.py (+4/-8)
magicicada/server/testing/aq_helpers.py (+10/-5)
magicicada/server/testing/testcase.py (+1/-19)
magicicada/server/tests/test_auth.py (+25/-34)
magicicada/server/tests/test_basic.py (+14/-29)
magicicada/server/tests/test_content.py (+73/-111)
magicicada/server/tests/test_notify_delivery.py (+24/-14)
magicicada/server/tests/test_server.py (+106/-185)
magicicada/server/tests/test_ssl_proxy.py (+13/-32)
magicicada/settings/__init__.py (+50/-3)
magicicada/settings/api_server.py (+0/-2)
magicicada/settings/testing.py (+4/-0)
test (+3/-1)
To merge this branch: bzr merge lp://staging/~nataliabidart/magicicada-server/no-u1-devtools
Reviewer Review Type Date Requested Status
Facundo Batista Approve
Review via email: mp+296872@code.staging.launchpad.net

Commit message

- Complete cleanup of logging setup and logging testing.
- Removed dependency on ubuntuone-devtools.

To post a comment you must log in.
80. By Natalia Bidart

Less changes against trunk.

81. By Natalia Bidart

Merged trunk in.

82. By Natalia Bidart

Test fixed

83. By Natalia Bidart

No more ubuntu_sso symlink.

84. By Natalia Bidart

All tests passing for real :-)

85. By Natalia Bidart

Merged trunk into no-u1-devtools.

86. By Natalia Bidart

Fixed pending tests.

Revision history for this message
Facundo Batista (facundo) wrote :

+1

Couldn't really tested, have a broken env.

I got complains about magicicada/settings/testing.py, though; like "F405 os may be undefined, or defined from star imports: magicicada.settings"; maybe we could avoid that "import *"

Anyway, if you're sure that logs are being produced ok in test and normal server mode, go ahead :)

review: Approve
87. By Natalia Bidart

Allow for verbosity to be set for tests.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: