Collect revision information for snap builds

Bug #1679157 reported by Colin Watson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Colin Watson
launchpad-buildd
Fix Released
High
Colin Watson

Bug Description

It would be very useful for build.snapcraft.io to be able to determine the code revision from which a given snap build ran. There's been talk of snapcraft generating a manifest of which VCS objects it used, but I'm not sure that's scheduled any time soon and I don't want to let the perfect be the enemy of the good. Furthermore, with a small amount of work in Launchpad we can collect revision information even about builds that are still running, which would be even more helpful to BSI and can't be done using the snapcraft manifest approach.

Related branches

Colin Watson (cjwatson)
Changed in launchpad-buildd:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Colin Watson (cjwatson)
Changed in launchpad:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Colin Watson (cjwatson)
Revision history for this message
Evan (ev) wrote :

How does getting VCS metadata relate to getting the snap revision for `snap install --edge --revision`?

Revision history for this message
Colin Watson (cjwatson) wrote :

Yeesh. Is that what you were asking for? I wish you'd been clearer if so, since it was ambiguous: we ultimately need both these things for build.snapcraft.io, and either one of them can reasonably be described as "revision".

Revision history for this message
Colin Watson (cjwatson) wrote :

Well, fortunately the other one is easier, and is now bug 1679197.

tags: added: api
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Revision history for this message
Colin Watson (cjwatson) wrote :

2017-04-26 13:10:21,319 INFO 2209-69-8 applied 2017-04-26 in 0.0 seconds

tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
removed: qa-ok
Changed in launchpad:
status: In Progress → Fix Committed
Colin Watson (cjwatson)
tags: added: qa-ok
removed: qa-needstesting
Colin Watson (cjwatson)
Changed in launchpad:
status: Fix Committed → Fix Released
Colin Watson (cjwatson)
Changed in launchpad-buildd:
status: In Progress → Fix Committed
Revision history for this message
Colin Watson (cjwatson) wrote :

Fixed in launchpad-buildd 143.

Changed in launchpad-buildd:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers