Merge lp:~leonardr/launchpadlib/death-to-edge into lp:launchpadlib

Proposed by Leonard Richardson
Status: Merged
Approved by: Leonard Richardson
Approved revision: 110
Merged at revision: 110
Proposed branch: lp:~leonardr/launchpadlib/death-to-edge
Merge into: lp:launchpadlib
Diff against target: 110 lines (+11/-13)
4 files modified
src/launchpadlib/NEWS.txt (+4/-2)
src/launchpadlib/launchpad.py (+2/-2)
src/launchpadlib/tests/test_launchpad.py (+1/-1)
src/launchpadlib/uris.py (+4/-8)
To merge this branch: bzr merge lp:~leonardr/launchpadlib/death-to-edge
Reviewer Review Type Date Requested Status
LAZR Developers Pending
Review via email: mp+46695@code.launchpad.net

Description of the change

This branch removes all references to edge from launchpadlib. Edge is broken for launchpadlib right now, and any application that's trying to connect to edge is already deprecated. It's time to remove edge from launchpadlib, if only to give our users comprehensible error messages.

To post a comment you must log in.
Revision history for this message
Leonard Richardson (leonardr) wrote :

Self-reviewed following discussion with benji and wgrant.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/launchpadlib/NEWS.txt'
2--- src/launchpadlib/NEWS.txt 2011-01-14 16:09:39 +0000
3+++ src/launchpadlib/NEWS.txt 2011-01-18 22:03:28 +0000
4@@ -2,8 +2,10 @@
5 NEWS for launchpadlib
6 =====================
7
8-Unreleased
9-==========
10+1.9.4 (2010-01-18)
11+==================
12+
13+- Removed references to the 'edge' service root, which is being phased out.
14
15 - Fixed a minor bug in the upload_release_tarball contrib script which
16 was causing tarballs to be uploaded with the wrong media type.
17
18=== modified file 'src/launchpadlib/launchpad.py'
19--- src/launchpadlib/launchpad.py 2011-01-07 18:01:06 +0000
20+++ src/launchpadlib/launchpad.py 2011-01-18 22:03:28 +0000
21@@ -45,9 +45,9 @@
22
23
24 # Import some constants for backwards compatibility. This way, old
25-# scripts that have 'from launchpad import EDGE_SERVICE_ROOT' will still
26+# scripts that have 'from launchpad import STAGING_SERVICE_ROOT' will still
27 # work.
28-from launchpadlib.uris import EDGE_SERVICE_ROOT, STAGING_SERVICE_ROOT
29+from launchpadlib.uris import STAGING_SERVICE_ROOT
30 OAUTH_REALM = 'https://api.launchpad.net'
31
32
33
34=== modified file 'src/launchpadlib/tests/test_launchpad.py'
35--- src/launchpadlib/tests/test_launchpad.py 2011-01-10 14:34:03 +0000
36+++ src/launchpadlib/tests/test_launchpad.py 2011-01-18 22:03:28 +0000
37@@ -68,7 +68,7 @@
38
39 def setUp(self):
40 self.aliases = sorted(
41- ['production', 'edge', 'qastaging', 'staging', 'dogfood', 'dev', 'test_dev'])
42+ ['production', 'qastaging', 'staging', 'dogfood', 'dev', 'test_dev'])
43
44 def test_short_names(self):
45 # Ensure the short service names are all supported.
46
47=== modified file 'src/launchpadlib/uris.py'
48--- src/launchpadlib/uris.py 2010-12-03 19:44:47 +0000
49+++ src/launchpadlib/uris.py 2011-01-18 22:03:28 +0000
50@@ -16,8 +16,8 @@
51
52 """Launchpad-specific URIs and convenience lookup functions.
53
54-The code in this module lets users say "edge" when they mean
55-"https://api.edge.launchpad.net/".
56+The code in this module lets users say "staging" when they mean
57+"https://api.staging.launchpad.net/".
58 """
59
60 __metaclass__ = type
61@@ -31,7 +31,6 @@
62 from lazr.uri import URI
63
64 LPNET_SERVICE_ROOT = 'https://api.launchpad.net/'
65-EDGE_SERVICE_ROOT = 'https://api.edge.launchpad.net/'
66 QASTAGING_SERVICE_ROOT = 'https://api.qastaging.launchpad.net/'
67 STAGING_SERVICE_ROOT = 'https://api.staging.launchpad.net/'
68 DEV_SERVICE_ROOT = 'https://api.launchpad.dev/'
69@@ -39,7 +38,6 @@
70 TEST_DEV_SERVICE_ROOT = 'http://api.launchpad.dev:8085/'
71
72 LPNET_WEB_ROOT = 'https://launchpad.net/'
73-EDGE_WEB_ROOT = 'https://edge.launchpad.net/'
74 QASTAGING_WEB_ROOT = 'https://qastaging.launchpad.net/'
75 STAGING_WEB_ROOT = 'https://staging.launchpad.net/'
76 DEV_WEB_ROOT = 'https://launchpad.dev/'
77@@ -49,7 +47,6 @@
78
79 service_roots = dict(
80 production=LPNET_SERVICE_ROOT,
81- edge=EDGE_SERVICE_ROOT,
82 qastaging=QASTAGING_SERVICE_ROOT,
83 staging=STAGING_SERVICE_ROOT,
84 dogfood=DOGFOOD_SERVICE_ROOT,
85@@ -60,7 +57,6 @@
86
87 web_roots = dict(
88 production=LPNET_WEB_ROOT,
89- edge=EDGE_WEB_ROOT,
90 qastaging=QASTAGING_WEB_ROOT,
91 staging=STAGING_WEB_ROOT,
92 dogfood=DOGFOOD_WEB_ROOT,
93@@ -87,7 +83,7 @@
94 def lookup_service_root(service_root):
95 """Dereference an alias to a service root.
96
97- A recognized server alias such as "edge" gets turned into the
98+ A recognized server alias such as "staging" gets turned into the
99 appropriate URI. A URI gets returned as is. Any other string raises a
100 ValueError.
101 """
102@@ -97,7 +93,7 @@
103 def lookup_web_root(web_root):
104 """Dereference an alias to a website root.
105
106- A recognized server alias such as "edge" gets turned into the
107+ A recognized server alias such as "staging" gets turned into the
108 appropriate URI. A URI gets returned as is. Any other string raises a
109 ValueError.
110 """

Subscribers

People subscribed via source and target branches