provide link to diff in PPA description

Bug #1649629 reported by Brian Murray
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bileto
Fix Released
Medium
Robert Bruce Park

Bug Description

After the presentation in Seville, I tested using bileto to upload an SRU. While the process went smoothly from an SRU sponsor standpoint, looking at the SRU in the queue its hard to see what changed in the package. Looking at the trusty queue, we can see there is no diff for uvtool.

https://launchpad.net/ubuntu/trusty/+queue?queue_state=1

If I click on "Sync from 2286 - 2016-12-12" I arrive at a PPA page:

https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/2286

If the diff in the ticket is the canonical one, it'd be helpful if there was a link to the diff in the PPA description (which currently just contains a link to the ticket) thereby reducing the number of clicks an SRU member needs to perform.

What I want is a link to https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_39a8dbb93caf4ec889f8a1b7f69885db/bileto-2286/2016-12-12_19:45:38/trusty_uvtool_content.diff so I can easily see the changes that will end up in -proposed.

Revision history for this message
Robert Bruce Park (robru) wrote :

Yeah this makes sense. We actually used to dump a full ticket summary into the PPA description, I can't remember why we stopped doing that. I guess because it would get out of sync.

Changed in bileto:
assignee: nobody → Robert Bruce Park (robru)
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Robert Bruce Park (robru) wrote :

Alright I've just rolled this out. This code isn't retroactive but all future builds will put the diffs into the PPA description.

Changed in bileto:
status: Triaged → Fix Released
Revision history for this message
Robert Bruce Park (robru) wrote :

Hrm well one problem is that bileto is setting the description at the start of the jobs so this happens before diffs are known, but then the diffs would be set correctly at publish time. Grumble...

Changed in bileto:
status: Fix Released → In Progress
Revision history for this message
Robert Bruce Park (robru) wrote :

Ok mangled the build job to update the PPA description after diffs are known, this should be working now.

Changed in bileto:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.