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 |
Related bugs: |
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.
+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 :)