2012-11-28 23:43:56 |
Brian Murray |
bug |
|
|
added bug |
2012-11-28 23:48:21 |
Brian Murray |
attachment added |
|
test_mark_2nd_report_upload.diff https://bugs.launchpad.net/ubuntu/+source/apport/+bug/1084296/+attachment/3446332/+files/test_mark_2nd_report_upload.diff |
|
2012-11-28 23:55:11 |
Brian Murray |
apport (Ubuntu): importance |
Undecided |
High |
|
2012-11-28 23:56:01 |
Brian Murray |
bug task added |
|
apport |
|
2012-11-28 23:56:19 |
Brian Murray |
bug |
|
|
added subscriber Evan Dandrea |
2012-11-28 23:58:28 |
Launchpad Janitor |
branch linked |
|
lp:~brian-murray/apport/bug-1084296 |
|
2012-11-29 16:21:22 |
Launchpad Janitor |
branch linked |
|
lp:apport |
|
2012-11-29 16:22:02 |
Martin Pitt |
apport: status |
New |
Fix Released |
|
2012-11-29 16:22:12 |
Martin Pitt |
apport (Ubuntu): status |
New |
Fix Committed |
|
2012-11-29 16:31:13 |
Brian Murray |
nominated for series |
|
Ubuntu Precise |
|
2012-11-29 16:31:13 |
Brian Murray |
bug task added |
|
apport (Ubuntu Precise) |
|
2012-11-29 16:31:13 |
Brian Murray |
nominated for series |
|
Ubuntu Quantal |
|
2012-11-29 16:31:13 |
Brian Murray |
bug task added |
|
apport (Ubuntu Quantal) |
|
2012-11-29 16:31:30 |
Brian Murray |
apport (Ubuntu Precise): importance |
Undecided |
High |
|
2012-11-29 16:31:33 |
Brian Murray |
apport (Ubuntu Quantal): importance |
Undecided |
High |
|
2012-11-29 16:31:39 |
Brian Murray |
apport (Ubuntu Precise): status |
New |
Triaged |
|
2012-11-29 16:31:43 |
Brian Murray |
apport (Ubuntu Quantal): status |
New |
Triaged |
|
2012-11-30 17:55:23 |
Launchpad Janitor |
apport (Ubuntu): status |
Fix Committed |
Fix Released |
|
2012-12-04 10:51:20 |
Launchpad Janitor |
branch linked |
|
lp:~ubuntu-core-dev/ubuntu/precise/apport/ubuntu |
|
2012-12-04 10:55:12 |
Launchpad Janitor |
branch linked |
|
lp:~ubuntu-core-dev/ubuntu/quantal/apport/ubuntu |
|
2012-12-04 10:56:30 |
Martin Pitt |
apport (Ubuntu Precise): status |
Triaged |
In Progress |
|
2012-12-04 10:56:44 |
Martin Pitt |
apport (Ubuntu Quantal): status |
Triaged |
In Progress |
|
2012-12-04 12:32:25 |
Brian Murray |
description |
I was testing errors in the cloud using juju and created a crash report for d-feet. The first time I did this the a .crash file was created and then a corresponding .upload and .uploaded file were created. The second time I did this a new .crash file was created but a new .upload file was not and subsequently the 2nd occurrence of this crash was not sent to the error tracker.
My /var/crash directory:
-rw-rw-r-- 1 bdmurray whoopsie 0 Nov 28 10:25 _usr_bin_d-feet.1000.upload
-rw------- 1 whoopsie whoopsie 0 Nov 28 13:14 _usr_bin_d-feet.1000.uploaded
-rw-r----- 1 bdmurray whoopsie 3401262 Nov 28 13:43 _usr_bin_d-feet.1000.crash
In apport/fileutils.py we can see that there are no checks done to see if there is already a .upload or .uploaded file:
def mark_report_upload(report):
report = '%s.upload' % report.rsplit('.', 1)[0]
with open(report, 'a'):
pass
So if a .upload for the crash already exists a new one is not created nor is the existing one modified. |
[Test Case]
1) (xeyes &); sleep 1; killall -SEGV xeyes
2) check /var/crash
3) (xeyes &); sleep 1; killall -SEGV xeyes
3) check /var/crash notice that .crash is newer than .upload and .uploaded
Original Description:
I was testing errors in the cloud using juju and created a crash report for d-feet. The first time I did this the a .crash file was created and then a corresponding .upload and .uploaded file were created. The second time I did this a new .crash file was created but a new .upload file was not and subsequently the 2nd occurrence of this crash was not sent to the error tracker.
My /var/crash directory:
-rw-rw-r-- 1 bdmurray whoopsie 0 Nov 28 10:25 _usr_bin_d-feet.1000.upload
-rw------- 1 whoopsie whoopsie 0 Nov 28 13:14 _usr_bin_d-feet.1000.uploaded
-rw-r----- 1 bdmurray whoopsie 3401262 Nov 28 13:43 _usr_bin_d-feet.1000.crash
In apport/fileutils.py we can see that there are no checks done to see if there is already a .upload or .uploaded file:
def mark_report_upload(report):
report = '%s.upload' % report.rsplit('.', 1)[0]
with open(report, 'a'):
pass
So if a .upload for the crash already exists a new one is not created nor is the existing one modified. |
|
2012-12-04 12:34:49 |
Brian Murray |
apport (Ubuntu Quantal): status |
In Progress |
Fix Committed |
|
2012-12-04 12:34:51 |
Brian Murray |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
2012-12-04 12:34:52 |
Brian Murray |
bug |
|
|
added subscriber SRU Verification |
2012-12-04 12:35:00 |
Brian Murray |
tags |
|
verification-needed |
|
2012-12-04 12:42:19 |
Brian Murray |
apport (Ubuntu Precise): status |
In Progress |
Fix Committed |
|
2012-12-05 15:55:35 |
Brian Murray |
tags |
verification-needed |
verification-done-quantal verification-needed |
|
2012-12-13 22:46:12 |
Launchpad Janitor |
apport (Ubuntu Quantal): status |
Fix Committed |
Fix Released |
|
2012-12-18 13:22:18 |
Jamie Strandboge |
apport (Ubuntu Precise): status |
Fix Committed |
In Progress |
|
2012-12-18 13:23:12 |
Jamie Strandboge |
tags |
verification-done-quantal verification-needed |
verification-done-quantal |
|
2013-01-02 17:27:59 |
Brian Murray |
apport (Ubuntu Precise): status |
In Progress |
Fix Committed |
|
2013-01-02 17:28:04 |
Brian Murray |
tags |
verification-done-quantal |
verification-done-quantal verification-needed |
|
2013-01-04 17:37:50 |
Brian Murray |
tags |
verification-done-quantal verification-needed |
verification-done verification-done-quantal |
|
2013-01-10 16:26:50 |
Colin Watson |
removed subscriber Ubuntu Stable Release Updates Team |
|
|
|
2013-01-10 16:27:10 |
Launchpad Janitor |
apport (Ubuntu Precise): status |
Fix Committed |
Fix Released |
|