[FFe] [UIFe] Checkbox GUI needs to be able to send test results to Launchpad

Bug #1302615 reported by Brendan Donegan
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
checkbox (Ubuntu)
Fix Released
High
Unassigned

Bug Description

We propose to add a new feature to the Checkbox UI, which will allow the user to send their test results to the Launchpad Hardware Database (https://launchpad.net/+hwdb/+submit). The UI will be changed to ask for the users Launchpad email address at the end of testing. After submitting they will see a response indicating whether submission was succesful or if some error occured.

N.B. This reinstates a feature that was in Checkbox in previous releases, but was removed during Trusty development due to re-implementation of the Checkbox core and UI.

This feature is considered beneficial as it will mean users will continue being able to send test reports to Launchpad, which may at a later date be used for currently undefined purposes (a past use has been to drive the Ubuntu Friendly database).

The visible UI changes (warranting a UIFe) are in checkbox-gui, but in order for this feature to work fully, the following syncs will need to be done from Debian:

- plainbox 0.5.3
- checkbox-ng 0.3
- plainbox-provider-checkbox 0.4
- checkbox-support 0.2

Testing that has been performed as of now:

- Package builds
- Package installs on Trusty
- Submit to Launchpad feature works, and no noticable regressions occur
  in regular usage of Checkbox

UIFe requests to ubuntu-doc and ubuntu-translators

https://lists.ubuntu.com/archives/ubuntu-doc/2014-April/018849.html
https://lists.ubuntu.com/archives/ubuntu-translators/2014-April/006442.html

affects: checkbox-gui → checkbox (Ubuntu)
Revision history for this message
Daniel Manrique (roadmr) wrote :

Here's a build log produced after integrating Brendan's branch into lp:ubuntu/checkbox.

Revision history for this message
Daniel Manrique (roadmr) wrote :

Here's the install log on a fully-upgraded Trusty system.

Installed version:

Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name Version Architecture Description
+++-=====================================================-===================================================-============-===============================================================================
ii checkbox-gui 0.17.6-0ubuntu5 amd64 QML based interface for system testing based on Plainbox.

dpkg --install of the package:

(Reading database ... 194471 files and directories currently installed.)
Preparing to unpack checkbox-gui_0.17.6-0ubuntu6_amd64.deb ...
Unpacking checkbox-gui (0.17.6-0ubuntu6) over (0.17.6-0ubuntu5) ...
Setting up checkbox-gui (0.17.6-0ubuntu6) ...
Processing triggers for gnome-menus (3.10.1-0ubuntu2) ...
Processing triggers for desktop-file-utils (0.22-1ubuntu1) ...
Processing triggers for bamfdaemon (0.5.1+14.04.20140310-0ubuntu1) ...
Rebuilding /usr/share/applications/bamf-2.index...
Processing triggers for mime-support (3.54ubuntu1) ...

Result after install is complete:
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name Version Architecture Description
+++-=====================================================-===================================================-============-===============================================================================
ii checkbox-gui 0.17.6-0ubuntu6 amd64 QML based interface for system testing based on Plainbox.

Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

This is fine from a docs POV.

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

Here's the debdiff, as requested by infinity

description: updated
Zygmunt Krynicki (zyga)
description: updated
Revision history for this message
Colin Watson (cjwatson) wrote :

I agree that we want this feature in trusty so that we can collect test reports, and I'd much rather we landed it now rather than trying to shove it in post-release, so let's do it. FFe approved.

Changed in checkbox (Ubuntu):
status: New → Confirmed
Daniel Manrique (roadmr)
Changed in checkbox (Ubuntu):
status: Confirmed → Fix Committed
importance: Undecided → High
milestone: none → ubuntu-14.04
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package checkbox - 0.17.6-0ubuntu6

---------------
checkbox (0.17.6-0ubuntu6) trusty; urgency=medium

  [ Brendan Donegan ]
  * Use select_jobs to ensure whitelist order is preserved (LP: #1213893)
  * Use partial_id to avoid showing overly-long, namespaced job names
    (LP: #1297500)
  * Enable customization of some checkbox-gui parameters (LP: #1303849)
  * Send test results to Launchpad (LP: #1302615)
 -- Brendan Donegan <email address hidden> Mon, 07 Apr 2014 20:49:45 +0100

Changed in checkbox (Ubuntu):
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

Remote bug watches

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