Daisy

Merge lp://staging/~brian-murray/daisy/release-package-counter into lp://staging/daisy

  1. release-package-counter
  2. Merge into trunk
Proposed by Brian Murray
Status: Merged
Merged at revision: 208
Proposed branch: lp://staging/~brian-murray/daisy/release-package-counter
Merge into: lp://staging/daisy
Diff against target: 78 lines (+19/-2)
1 file modified
submit.wsgi (+19/-2)
To merge this branch: bzr merge lp://staging/~brian-murray/daisy/release-package-counter
Reviewer Review Type Date Requested Status
Evan (community) Approve
Review via email: mp+143366@code.staging.launchpad.net

Description of the change

As a part of the phased updates blueprint I've created a column family in counters for tracking release:package crash counts on a daily basis.

As we'll only use 2 weeks of data in the rate of crashes calculation I've set a time to live for the column for 4 weeks. However, if you think more than having more than 4 weeks of data would be handy I'm happy to change the value or remove it altogether.

To post a comment you must log in.
lp://staging/~brian-murray/daisy/release-package-counter updated
208. By Brian Murray

also add a counter for crashes that are missing StacktraceAddressSignatures

Revision history for this message
Brian Murray (brian-murray) wrote :

I also added in a counter for missing StacktraceAddressSignatures.

Revision history for this message
Evan (ev) wrote :

This looks good. At some point we should refactor this code so we're not returning mid-block in several locations and needing to call update_package_count from multiple locations.

review: Approve
lp://staging/~brian-murray/daisy/release-package-counter updated
209. By Brian Murray

submit.wsgi: switch from using the binary package to the source package for crash rate test

210. By Brian Murray

submit.wsgi: only increment the counter for problem types of crash

Revision history for this message
Brian Murray (brian-murray) wrote :

I also modified the branch so that the counter is only incremented for ProblemType: Crash.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: