Merge lp:~matt-goodall/canonical-identity-provider/no-more-bson-madness-other-than-bson-itself into lp:canonical-identity-provider/release

Proposed by Matt Goodall
Status: Merged
Approved by: Matt Goodall
Approved revision: no longer in the source branch.
Merged at revision: 1367
Proposed branch: lp:~matt-goodall/canonical-identity-provider/no-more-bson-madness-other-than-bson-itself
Merge into: lp:canonical-identity-provider/release
Diff against target: 36 lines (+2/-4)
2 files modified
requirements.txt (+1/-3)
src/identityprovider/tests/test_wsgi.py (+1/-1)
To merge this branch: bzr merge lp:~matt-goodall/canonical-identity-provider/no-more-bson-madness-other-than-bson-itself
Reviewer Review Type Date Requested Status
Matias Bordese (community) Approve
Review via email: mp+278162@code.launchpad.net

Commit message

Upgrade oops-amqp to simplify bson dependency.

Description of the change

Fix bson madness:

- upgrade oops-amqp to version with no pymongo dependency
- remove pymongo dependency
- remove pip bson as it's already a deb dependency

To post a comment you must log in.
Revision history for this message
Matias Bordese (matiasb) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'requirements.txt'
2--- requirements.txt 2015-11-18 16:24:16 +0000
3+++ requirements.txt 2015-11-20 15:37:10 +0000
4@@ -1,4 +1,3 @@
5-bson==0.3.3
6 canonical-raven==0.0.3
7 convoy==0.4.1
8 dj-static==0.0.6.dev1
9@@ -17,13 +16,12 @@
10 oath==1.4.0
11 oauthlib==0.7.2
12 oops==0.0.13
13-oops-amqp==0.0.7
14+oops-amqp==0.0.8b1
15 oops-datedir-repo==0.0.22
16 oops-dictconfig==0.0.6
17 oops-timeline==0.0.2
18 oops-wsgi==0.0.11
19 paste==2.0.1
20-pymongo==3.0.1
21 raven==5.6.0
22 requests-oauthlib==0.4.2
23 static3==0.5.1.dev1
24
25=== modified file 'src/identityprovider/tests/test_wsgi.py'
26--- src/identityprovider/tests/test_wsgi.py 2014-12-01 11:10:45 +0000
27+++ src/identityprovider/tests/test_wsgi.py 2015-11-20 15:37:10 +0000
28@@ -52,7 +52,7 @@
29 oops_path = self.oops_dir + '/' + todays_oops_path + '/' + oops_list[0]
30 with open(oops_path, "r") as oops_file:
31 oops_file_contents = oops_file.read()
32- return bson.decode_all(oops_file_contents)[0]
33+ return bson.loads(oops_file_contents)
34
35 def request_wsgi_response(self, environ):
36 app = make_app()