Merge lp:~al-maisan/launchpad/sync-470411 into lp:launchpad/db-devel

Proposed by Muharem Hrnjadovic
Status: Merged
Merged at revision: not available
Proposed branch: lp:~al-maisan/launchpad/sync-470411
Merge into: lp:launchpad/db-devel
Diff against target: 12 lines
1 file modified
scripts/ftpmaster-tools/_syncorigins.py (+1/-1)
To merge this branch: bzr merge lp:~al-maisan/launchpad/sync-470411
Reviewer Review Type Date Requested Status
Michael Nelson (community) release-critical Approve
Henning Eggers (community) code Approve
Review via email: mp+14301@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Muharem Hrnjadovic (al-maisan) wrote :

This change was requested by the distro team:

{{{
<slangasek> kirkland, james_w`, jdstrand, al-maisan, StevenK: until bug #470411 is addressed, sync-source.py -a will pull from the wrong Debian suite by default, so please take care when autosyncing
<bugbot> Launchpad bug 470411 in soyuz "please change the default source suite for Debian to 'testing' in _syncorigins.py" [Undecided,New] https://launchpad.net/bugs/470411
* al-maisan gets going on the bug above ..
<slangasek> al-maisan: ok, cheers :)
<al-maisan> so, here's the change for bug #470411:
<al-maisan> === modified file 'scripts/ftpmaster-tools/_syncorigins.py'
<al-maisan> --- scripts/ftpmaster-tools/_syncorigins.py 2009-07-23 02:33:14 +0000
<al-maisan> +++ scripts/ftpmaster-tools/_syncorigins.py 2009-11-02 12:27:28 +0000
<al-maisan> @@ -12,7 +12,7 @@
<al-maisan> "debian": {
<al-maisan> "name": "Debian",
<al-maisan> "url": "http://ftp.debian.org/debian/",
<al-maisan> - "default suite": "unstable",
<al-maisan> + "default suite": "testing",
<al-maisan> "default component": "main",
<al-maisan> "dsc": "must be signed and valid"
<al-maisan> },
<bugbot> Launchpad bug 470411 in soyuz "please change the default source suite for Debian to 'testing' in _syncorigins.py" [High,Triaged] https://launchpad.net/bugs/470411
<al-maisan> Is that all that's needed?
* njpatel is now known as njpatel|lunch
<al-maisan> cjwatson: ^^
<cjwatson> al-maisan: sounds right
<al-maisan> cjwatson: thanks.
<cjwatson> al-maisan: will it be easy to revert this for lucid+1?
<cjwatson> (this is why syncorigins.py should be controlled by platform, not in the launchpad source, but hey ...)
<al-maisan> cjwatson: it's a 1-line change again
<cjwatson> right, just a matter of timing
<al-maisan> yup
<cjwatson> we'll need it applied on production in roughly the four-day window following the lucid release
}}}

Revision history for this message
Henning Eggers (henninge) :
review: Approve
Revision history for this message
Henning Eggers (henninge) :
review: Approve (code)
Revision history for this message
Michael Nelson (michael.nelson) wrote :

Thanks Muharem - does our test-suite reference this? Guessing we should land via ec2 as normal.

review: Approve (release-critical)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'scripts/ftpmaster-tools/_syncorigins.py'
2--- scripts/ftpmaster-tools/_syncorigins.py 2009-07-23 02:33:14 +0000
3+++ scripts/ftpmaster-tools/_syncorigins.py 2009-11-02 13:45:24 +0000
4@@ -12,7 +12,7 @@
5 "debian": {
6 "name": "Debian",
7 "url": "http://ftp.debian.org/debian/",
8- "default suite": "unstable",
9+ "default suite": "testing",
10 "default component": "main",
11 "dsc": "must be signed and valid"
12 },

Subscribers

People subscribed via source and target branches

to status/vote changes: