Closed
Bug 678400
Opened 13 years ago
Closed 13 years ago
Need a way to get to different 5.0 beta and release reports
Categories
(Socorro :: General, task)
Socorro
General
Tracking
(Not tracked)
VERIFIED
FIXED
2.2
People
(Reporter: kairo, Assigned: brandon)
Details
Attachments
(1 file, 1 obsolete file)
516 bytes,
patch
|
rhelmer
:
review+
|
Details | Diff | Splinter Review |
In the new model, a final release like 5.0 should have both a beta and a release report for TCBS and graphs, but currently I get only one "5.0" entry in the version selection on stage. We need some way to get to the two different reports there.
Comment 1•13 years ago
|
||
Can we retroactively extend our sql changes backwards to include this?
Updated•13 years ago
|
Target Milestone: --- → 2.2
Reporter | ||
Comment 2•13 years ago
|
||
We can invent some label there for the "final release on beta" entry, like for example "5.0 (beta)" or something, as long as there's a way for us as users to know what it means.
Comment 3•13 years ago
|
||
OK. Josh wrote a new stored procedure to try and find these in the data. It's deployed on staging. There's a lot of junk data, so there will likely be a few crashes aggregated that don't fit into the right bucket. You will now see these showing up in the UI (as 6.0 (beta), 5.0 (beta), but the links are busted. Will take a look at those next.
Updated•13 years ago
|
Assignee: nobody → bsavage
Comment 4•13 years ago
|
||
Over to Brandon to fix the UI. If you're not using the version_string to retrieve these, then make sure you match on: - product - version AND - release channel.
Assignee | ||
Comment 5•13 years ago
|
||
This patch fixes the 404 by correcting the regular expression that determines the proper URL. The rest of the data loads properly but the graphs are returning a 404.
Attachment #552667 -
Flags: review?(chris.lonnen)
Assignee | ||
Comment 6•13 years ago
|
||
Chris, there is a patch to fix the URL issues in the bug. However, the middleware is returning an error on all requests for this, probably because "6.0 (beta)" is a string it can't understand. Here is the traceback: Traceback (most recent call last): File "/usr/lib/python2.6/site-packages/web.py-0.33-py2.6.egg/web/application.py", line 242, in process return self.handle() File "/usr/lib/python2.6/site-packages/web.py-0.33-py2.6.egg/web/application.py", line 233, in handle return self._delegate(fn, self.fvars, args) File "/usr/lib/python2.6/site-packages/web.py-0.33-py2.6.egg/web/application.py", line 396, in _delegate return handle_class(f) File "/usr/lib/python2.6/site-packages/web.py-0.33-py2.6.egg/web/application.py", line 387, in handle_class return tocall(*args) File "/home/bsavage/trunk/socorro/webapi/webapiService.py", line 60, in GET raise Exception(stringLogger.getMessages()) Exception: ERROR Caught Error: <type 'exceptions.KeyError'> ERROR ('Firefox', '6.0+(beta)') ERROR trace back follows: ERROR Traceback (most recent call last): ERROR File "/home/bsavage/trunk/socorro/webapi/webapiService.py", line 46, in GET result = self.get(*args) ERROR File "/home/bsavage/trunk/socorro/services/topCrashBySignatureTrends.py", line 86, in get return impl[table_type].twoPeriodTopCrasherComparison(cursor, parameters) ERROR File "/home/bsavage/trunk/socorro/services/tcbs/classic.py", line 198, in twoPeriodTopCrasherComparison context.version) ERROR File "/home/bsavage/trunk/socorro/lib/productVersionCache.py", line 47, in getId raise KeyError((product, version)) ERROR KeyError: ('Firefox', '6.0+(beta)') When you resolve this, please combine it with my patch so that there is only one patch for this bug.
Assignee: bsavage → chris.lonnen
Comment 7•13 years ago
|
||
The space in the name is getting encoded, and decoding is a bit of a headache. It seems much simpler to prohibit the space in the version name. Is that possible?
Updated•13 years ago
|
Assignee: chris.lonnen → josh
Severity: normal → blocker
Comment 8•13 years ago
|
||
Josh: so basically instead of calling these "6.0 (beta)" we should call them "6.0(beta)" (no space). Actions are therefore: 1. modify the stored proc 2. rerun the backfill Then you can kick this back over to lonnen for testing.
Comment 9•13 years ago
|
||
Actually, eliminating the space is a lot easier than that. breakpad=# update product_versions set version_string = '5.0(beta)' where version_string = '5.0 (beta)'; UPDATE 2 breakpad=# update product_versions set version_string = '6.0(beta)' where version_string = '6.0 (beta)'; UPDATE 2 Done. Fixing the SP and checking it in now.
Comment 10•13 years ago
|
||
Checked in. I'm done, back to brandon/lonnen.
Updated•13 years ago
|
Assignee: josh → chris.lonnen
Comment 11•13 years ago
|
||
Comment on attachment 552667 [details] [diff] [review] Patch for the UI It looks like this patch is already in 2.2 branch. Do we need a corresponding mware fix per comment 6, or should this be backed out per comment 7? Sounds like the latter...
Comment 12•13 years ago
|
||
Backed out please.
Comment 13•13 years ago
|
||
(In reply to Laura Thomson :laura from comment #12) > Backed out please. With or without this patch, it still doesn't work.
Comment 14•13 years ago
|
||
Chris can you take a look? If not please kick it over to rhelmer or myself.
Assignee | ||
Comment 15•13 years ago
|
||
This is the correct patch for the UI bug that we're seeing here. For some reason, the wrong patch was uploaded and this was not previously caught.
Attachment #552667 -
Attachment is obsolete: true
Attachment #552667 -
Flags: review?(chris.lonnen)
Attachment #552884 -
Flags: review?(rhelmer)
Comment 16•13 years ago
|
||
Comment on attachment 552884 [details] [diff] [review] Correct patch for the webui Yes this fixes the 404 on my dev instance
Attachment #552884 -
Flags: review?(rhelmer) → review+
Assignee | ||
Comment 17•13 years ago
|
||
Applying the correct patch, along with Josh's changes to remove the space, solve this bug.
Assignee: chris.lonnen → bsavage
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 18•13 years ago
|
||
QA verified: https://crash-stats.allizom.org/products/Firefox/versions/5.0%28beta%29 https://crash-stats.allizom.org/products/Firefox/versions/5.0.1 https://crash-stats.allizom.org/products/Firefox/versions/5.0b6 https://crash-stats.allizom.org/products/Firefox/versions/6.0 https://crash-stats.allizom.org/products/Firefox/versions/6.0%28beta%29
Updated•13 years ago
|
Status: RESOLVED → VERIFIED
Updated•12 years ago
|
Component: Socorro → General
Product: Webtools → Socorro
You need to log in
before you can comment on or make changes to this bug.
Description
•