Summary: | [results] Performance test not grey from degredation comment | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Darin Wright <darin.eclipse> | ||||
Component: | Releng | Assignee: | Frederic Fusier <frederic_fusier> | ||||
Status: | RESOLVED FIXED | QA Contact: | |||||
Severity: | normal | ||||||
Priority: | P3 | CC: | kim.moir, philippe_mulet | ||||
Version: | 3.4 | Keywords: | performance, test | ||||
Target Milestone: | 3.5 M3 | ||||||
Hardware: | PC | ||||||
OS: | Windows XP | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Darin Wright
2008-06-16 11:17:35 EDT
I'll try to verify this assumption while generating the results locally... Created attachment 105166 [details]
Proposed patch
My assumption was correct, as a summary was got from the baseline that prevented to look at comments from the builds. The method internalQueryScenarioSummaries of DB_Results wrongly assumed that baseline and builds have both summaries or they both have not... I need to know if we decide to put this in 3.4 or leave with it and wait 3.5 to fix it... Patch released in HEAD. I verified it works while generating M20080910-1030 results. At that time, it was not possible to verify this on M20080911-1700 (aka 3.4.1), results because no failure was logged in the database, hence all results get the OK icon :-( |