checkbox writes to .cache/checkbox/submission before submission completes
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Checkbox |
Medium
|
Marc Tardif | ||
| checkbox (Ubuntu) |
High
|
Marc Tardif | ||
| Lucid |
High
|
Marc Tardif |
Bug Description
Binary package hint: checkbox
When running checkbox a new submission id is generated for use by the hardware database and the system id stays the same. However, if one exits checkbox before the information is submitted to Launchpad the submission id, found in .cache/
This becomes a problem when later someone reports a bug using apport and the HWD-Submission information in Launchpad (used by linkHWSubmission) is populated by this submission id. The bug will either be linked to a non-existent (or wrong) hwdb submission or be linked to nothing at all. Neither of these is the intended behavior.
ProblemType: Bug
Architecture: amd64
CheckboxSubmission: fee5e196cb921cb
CheckboxSystem: 2a6f54df59af338
Date: Mon Mar 1 15:03:33 2010
DistroRelease: Ubuntu 10.04
Package: checkbox 0.8.6
PackageArchitec
ProcEnviron:
PATH=(custom, user)
LANG=en_US.UTF-8
SHELL=/bin/zsh
ProcVersionSign
SourcePackage: checkbox
Uname: Linux 2.6.32-13-generic x86_64
Related branches
Brian Murray (brian-murray) wrote : | #1 |
Brian Murray (brian-murray) wrote : | #2 |
Brian Murray (brian-murray) wrote : | #3 |
Abel - what will happen in the case that an invalid or non-existent HWDB-Submission is used when reporting a bug in Launchpad?
Changed in checkbox (Ubuntu): | |
importance: | Undecided → High |
status: | New → Triaged |
Changed in checkbox (Ubuntu Lucid): | |
milestone: | none → ubuntu-10.04-beta-1 |
Abel Deuring (adeuring) wrote : Re: [Bug 530380] Re: checkbox writes to .cache/checkbox/submission before submission completes | #4 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 02.03.2010 00:17, Brian Murray wrote:
> Abel - what will happen in the case that an invalid or non-existent
> HWDB-Submission is used when reporting a bug in Launchpad?
>
unfortunately, nothing... The line
"CheckboxSubmis
simply not processed...
I filed bug 530556 about it.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iD8DBQFLjNbOekB
EIB3K8QqoJhSctU
=EV4U
-----END PGP SIGNATURE-----
Abel Deuring (adeuring) wrote : | #5 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 02.03.2010 10:14, Abel Deuring wrote:
> On 02.03.2010 00:17, Brian Murray wrote:
>> Abel - what will happen in the case that an invalid or non-existent
>> HWDB-Submission is used when reporting a bug in Launchpad?
>
>
> unfortunately, nothing... The line
> "CheckboxSubmis
> simply not processed...
>
> I filed bug 530556 about it.
...and this is complete nonsense -- I was too stupid to find the code
when I wrote the answer.
lp.bug.
create links between bugs and HWDB submissions:
for submission_key in extra_data.
if submission is not None:
(line 648).
So: A link between a bug and a HWDB submission is created, if the the
HWDB submission exists. If doesn't for whichever reason (mostly likely
because checkbox did create a submission ID but nothing was uploaded or
an error like a timeout occurred during the upload), the invalid /
non-extistent submission key is silently ignored.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iD4DBQFLjjryekB
bp0H3NEemAkQnnZ
=gXiC
-----END PGP SIGNATURE-----
Changed in checkbox: | |
assignee: | nobody → Marc Tardif (cr3) |
Changed in checkbox (Ubuntu Lucid): | |
assignee: | nobody → Marc Tardif (cr3) |
Changed in checkbox: | |
status: | New → Fix Committed |
importance: | Undecided → Medium |
Steve Langasek (vorlon) wrote : | #6 |
If the fix was committed 10 days ago, what's preventing the bug from being fixed in the archive? This has now missed beta-1.
Changed in checkbox (Ubuntu Lucid): | |
milestone: | ubuntu-10.04-beta-1 → ubuntu-10.04-beta-2 |
Marc Tardif (cr3) wrote : | #7 |
The reason is the review process for inclusion in the main archive was longer than expected. The merge request has been finally approved yesterday, see bug #548800 for more details.
Launchpad Janitor (janitor) wrote : | #8 |
This bug was fixed in the package checkbox - 0.9.1
---------------
checkbox (0.9.1) lucid; urgency=low
New upstream release (LP: #548800):
* Added cpu_scaling_test script.
* Fixed hard drive detection (LP: #549714)
* Fixed backend to handle empty messages (LP: #536645)
* Fixed parsing of package resource (LP: #539691)
* Fixed malformed xml report (LP: #485445)
* Fixed running root manual tests as normal user (LP: #383559)
* Fixed writing apport files only after submitting (LP: #530380)
* Fixed audio test instructions (LP: #529205)
* Fixed gathering chassis information (LP: #537435)
* Fixed detection of disks in kvm (LP: #552998)
* Fixed udev_resource script to be more resilient (LP: #552999)
* Fixed filter_packages script to use new resources.
-- Marc Tardif <email address hidden> Sun, 07 Mar 2010 15:05:44 -0400
Changed in checkbox (Ubuntu Lucid): | |
status: | Triaged → Fix Released |
Changed in checkbox: | |
status: | Fix Committed → Fix Released |
For example if you were to search the database for the CheckboxSubmission in this bug report you would not, or should not, find it.