change-override.py should forbid modifications of the RELEASE pocket of non-development series
Bug #530020 reported by
William Grant
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Colin Watson | ||
Bug Description
change-override.py works fine on sources in the RELEASE pocket of a non-development series. The new Pending record is never processed, leaving stranded publications like https:/
Related branches
lp:~cjwatson/launchpad/change-override-forbid-release-stable
- William Grant: Approve (code)
-
Diff: 665 lines (+193/-145)15 files modifiedlib/lp/archiveuploader/uploadpolicy.py (+4/-4)
lib/lp/buildmaster/model/packagebuild.py (+5/-6)
lib/lp/registry/doc/distroseries.txt (+26/-16)
lib/lp/registry/interfaces/distroseries.py (+0/-20)
lib/lp/registry/model/distroseries.py (+6/-55)
lib/lp/soyuz/adapters/copypolicy.py (+1/-2)
lib/lp/soyuz/doc/buildd-queuebuilder-lookup.txt (+3/-3)
lib/lp/soyuz/doc/publishing.txt (+2/-0)
lib/lp/soyuz/interfaces/archive.py (+22/-0)
lib/lp/soyuz/model/archive.py (+33/-1)
lib/lp/soyuz/model/binarypackagebuild.py (+3/-4)
lib/lp/soyuz/model/publishing.py (+30/-23)
lib/lp/soyuz/model/queue.py (+8/-10)
lib/lp/soyuz/scripts/tests/test_changeoverride.py (+4/-0)
lib/lp/soyuz/tests/test_publishing.py (+46/-1)
| Changed in soyuz: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| tags: | added: soyuz-publish |
| tags: | added: soyuz-ftpmaster-tools |
Revision history for this message
| Colin Watson (cjwatson) wrote | #1 |
| Changed in launchpad: | |
| assignee: | nobody → Colin Watson (cjwatson) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Colin Watson (cjwatson) wrote | #3 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
William points out that this should be fixed before fixing bug 853831, so I'm working on this.