Add link to the autopkgtest artifacts for silo runs

Bug #1532162 reported by Jean-Baptiste Lallement
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bileto
Invalid
Low
Unassigned
britney
Fix Released
Low
Robert Bruce Park

Bug Description

Currently, for autopkgtest running on silos, there is a link to excuses.html, which links to the test logs but it is not always enough to understand a failure.

Could you also add the link to the test artifacts?

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

What artifacts do you mean? Can you give an example?

Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

For example when you go to http://autopkgtest.ubuntu.com/packages/d/dbconfig-common/xenial/amd64/ you have the test log (which is available from excuses) and a link to the test artifacts (https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-xenial/xenial/amd64/d/dbconfig-common/20160114_005505@/artifacts.tar.gz for example)

The test artifacts usually include the output of stdout and stderr for each test, a summary of the results (easier to read than autopkgtest's output) and the list of packages installed to run the test. They sometimes contain additional files that the test writer thinks are worth adding.

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

I'm not sure how this would be possible. Bileto doesn't have access to that information, bileto just runs britney and then britney spits out an excuses.html at me.

Changed in bileto:
status: New → Opinion
Revision history for this message
Robert Bruce Park (robru) wrote :

Maybe pitti could weigh in on this.

Revision history for this message
Martin Pitt (pitti) wrote :

It's trivial to get the artifacts URL -- just replace the "/log.gz" at the end with "/artifacts.tar.gz". We don't want to do this for the main Ubuntu britney, as you can see the artifacts on autopkgtest.ubuntu.com and it would further clutter the already very large excuses.html. But indeed for silo runs this makes more sense.

I suggest to add an option to britney.conf for that, like ADT_LINK_ARTIFACTS, and then expose that in the resulting HTML.

This is a rather simple change and an excellent opportunity for getting to know the code, I'm tagging this for next week's virtual sprint.

tags: added: proposed-migration-vsprint
Changed in bileto:
status: Opinion → Triaged
affects: bileto → britney
summary: - Add link to the autopkgtest artifacts
+ Add link to the autopkgtest artifacts for silo runs
Revision history for this message
Martin Pitt (pitti) wrote :

Adding back bileto task. This conceptually affects bileto, and it will need an actual (small) code change to enable the new option after britney got taught about that.

Changed in bileto:
status: New → Triaged
Changed in britney:
importance: Undecided → Low
Changed in bileto:
importance: Undecided → Low
Revision history for this message
Martin Pitt (pitti) wrote :

> and it will need an actual (small) code change to enable the new option

that's actually not true -- we can use the ADT_PPAS option which, if non-empty, should trigger this behaviour.

Changed in britney:
assignee: nobody → Robert Bruce Park (robru)
status: Triaged → In Progress
Changed in bileto:
status: Triaged → In Progress
status: In Progress → Invalid
Revision history for this message
Martin Pitt (pitti) wrote :
Changed in britney:
status: In Progress → Fix Released
Revision history for this message
Robert Bruce Park (robru) wrote :

Should we push this to debian? I don't like deltas if they can be avoided.

Revision history for this message
Martin Pitt (pitti) wrote :

This isn't applicable at all to Debian yet, as their britney doesn't have any of the autopkgtest integration.

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.