API missing "changes_file_url" for builds, and "debdiff" for source
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Julian Edwards | ||
Bug Description
A lot of things were added to the API due to the excellent work on bug 415583, but two things remain missing:
- per-build binary changes file. This is already linked to from the UI. I assume adding it to the "build" object, as already done for build_log_url and upload_log_url would make the most sense. For example, this URL looks like:
https:/
- a url link to the "debdiff" generated for a source. Preferably _not_ as part of the source_file_urls, as it shares the same extents as the source .diff.gz. Probably best as "debdiff_url" in source_
Once the per-build binary changes file is available, the security team will be using 100% API for all our publishing work. :):)
Related branches
- Graham Binns (community): Approve (code)
-
Diff: 273 lines (+131/-2)8 files modifiedlib/lp/soyuz/doc/binarypackagebuild.txt (+3/-0)
lib/lp/soyuz/interfaces/binarypackagebuild.py (+7/-0)
lib/lp/soyuz/interfaces/publishing.py (+11/-0)
lib/lp/soyuz/model/binarypackagebuild.py (+8/-0)
lib/lp/soyuz/model/publishing.py (+14/-2)
lib/lp/soyuz/stories/webservice/xx-builds.txt (+2/-0)
lib/lp/soyuz/stories/webservice/xx-source-package-publishing.txt (+59/-0)
lib/lp/testing/factory.py (+27/-0)
| Changed in soyuz: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| tags: | added: api easy |
| Changed in soyuz: | |
| assignee: | nobody → Muharem Hrnjadovic (al-maisan) |
| Changed in soyuz: | |
| assignee: | Muharem Hrnjadovic (al-maisan) → nobody |
| Changed in soyuz: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Julian Edwards (julian-edwards) |
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #1 |
| Changed in soyuz: | |
| milestone: | none → 10.10 |
| tags: | added: qa-needstesting |
| Changed in soyuz: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in soyuz: | |
| status: | Fix Committed → Fix Released |
| Kees Cook (kees) wrote | #2 |
| Julian Edwards (julian-edwards) wrote | #3 |
Fixed in stable r11648 <http://