Build status title changes break API and uglify forms
Bug #559143 reported by
William Grant
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Medium
|
Aaron Bentley | ||
Bug Description
Something changed the title of all BuildStatus members tonight. The breaks any API script that touches build statuses, and unbalances forms like https:/
Related branches
lp:~abentley/launchpad/revert-enums
- Edwin Grubbs (community): Approve
-
Diff: 513 lines (+77/-68)12 files modifiedlib/lp/buildmaster/interfaces/buildbase.py (+6/-6)
lib/lp/code/browser/sourcepackagerecipe.py (+14/-4)
lib/lp/soyuz/doc/build-failedtoupload-workflow.txt (+1/-1)
lib/lp/soyuz/doc/build-notification.txt (+2/-2)
lib/lp/soyuz/doc/build.txt (+1/-1)
lib/lp/soyuz/doc/buildd-mass-retry.txt (+4/-4)
lib/lp/soyuz/doc/buildd-slavescanner.txt (+6/-6)
lib/lp/soyuz/doc/gina.txt (+1/-1)
lib/lp/soyuz/stories/ppa/xx-copy-packages.txt (+1/-1)
lib/lp/soyuz/stories/soyuz/xx-build-record.txt (+10/-11)
lib/lp/soyuz/stories/soyuz/xx-builds-pages.txt (+30/-30)
lib/lp/soyuz/stories/soyuz/xx-private-builds.txt (+1/-1)
Revision history for this message
| Michael Nelson (michael.nelson) wrote | #1 |
| Changed in soyuz: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
Revision history for this message
| Julian Edwards (julian-edwards) wrote Re: [Bug 559143] [NEW] Build status title changes break API and uglify forms | #2 |
| Changed in soyuz: | |
| status: | Confirmed → Triaged |
| assignee: | nobody → Aaron Bentley (abentley) |
Revision history for this message
| Julian Edwards (julian-edwards) wrote | #3 |
Revision history for this message
| Ursula Junque (ursinha) wrote Bug fixed by a commit | #4 |
| Changed in soyuz: | |
| milestone: | none → 10.04 |
| status: | Triaged → Fix Committed |
| tags: | added: qa-needstesting |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in soyuz: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Hi William,
These were changed as the code team generalised them:
https:/
I agree that the titles should be one or two words - the description can be longer. It's unfortunate that we didn't pick up that changing the titles would break API scripts... but I guess that is what edge is for. If you get a chance, please read through Aaron's points on the above MP. Thanks!
BTW: I wonder if we could create a new buildmaster or buildfarm project for these bugs :)