madecoste / swarming

Automatically exported from code.google.com/p/swarming
Apache License 2.0
0 stars 1 forks source link

ereporter2 will not report errors on a tainted version even if it is the default one #99

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Repro:
1. Upload a tainted version on prod
2. Make it default
3. Have it throw exceptions

Expected:
Get reports.

Actual:
Tainted versions are filtered.

I see two potential solutions:
1. Stop filtering on tainted versions and report everything.
2. Keep filtering tainted versions but forcibly add default version.
3. Keep filtering tainted on canary and report everything on prod.

The problem with 1. is potentially being overwhelmed with someone doing manual 
testing. 3. is a potential work around. I dislike 2. because there's a gap 
since it's looking at the current default version when the cron job runs at 
every hour.

Assigning to Vadim since he was the one initially who preferred filtering out 
tainted versions and left the isolate server running with a tainted versions 
for days so we lost reports. :P

Original issue reported on code.google.com by maruel@chromium.org on 11 Apr 2014 at 5:42

GoogleCodeExporter commented 9 years ago
I hacked it up eventually.

Original comment by maruel@chromium.org on 5 Feb 2015 at 12:15