Bug 8569

Summary: Make yocto autobuilder submitter id configurable
Product: [Infrastructure] AutoBuilder Reporter: Belen Barros Pena <belen.barros.pena>
Component: autobuilderAssignee: Beth Flanagan <elizabeth.flanagan>
Status: VERIFIED FIXED QA Contact: Mihail Stanciu <stanciux.mihail>
Severity: enhancement    
Priority: Medium+ CC: alexandru.c.georgescu, elizabeth.flanagan, infras.ab.watcher, Infras.watcher, mariano.lopez, ross.burton, sjolley.yp.pm
Version: 2.1   
Target Milestone: 2.1 M4   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Belen Barros Pena 2015-10-26 09:28:29 UTC 
The "Latest autobuilder errors" tab is supposed to show only errors submitted by the Yocto Project public autobuilder instance. However, errors submitted by other autobuilders are being shown in that tab. This is an example:

http://errors.yoctoproject.org/Errors/Details/20801/

You can tell by the recipe and machine (and also the link back URL http://localhost:8010/builders/nightly-mqclient/builds/7)

We must improve the way we identify the errors from the Yocto Project public autobuilder to make sure that tab shows only those.
Comment 1 Michael Wood 2015-10-26 16:26:02 UTC 
The autobuilder doesn't currently have the submitter id as part of its configuration which means that all submissions from anyone using the yocto-autobuilder project come as the same submitter id.

http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/tree/lib/python2.7/site-packages/autobuilder/buildsteps/SendErrorReport.py?#n44

The error-report-web on errors.yp looks to see if the submitter starts with yocto-autobuilder and if it does it puts them in the Autobuilder filtered tab.
Comment 2 Michael Wood 2015-10-28 12:08:59 UTC 
*** Bug 8071 has been marked as a duplicate of this bug. ***
Comment 3 Paul Eggleton 2015-11-12 16:13:20 UTC 
*** Bug 8683 has been marked as a duplicate of this bug. ***
Comment 5 Mihail Stanciu 2016-04-13 13:19:31 UTC 
This doesn't seem to be an error from the yocto project public autobuilder:

http://errors.yoctoproject.org/Errors/Details/60025/

It seems to have been submitted today, so...unless I'm missing something, this seems to still be a problem.
Comment 6 Beth Flanagan 2016-04-13 14:24:42 UTC 
Relooking at this. Patch later today.
Comment 7 Beth Flanagan 2016-04-14 16:29:41 UTC 
http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/commit/?id=2ed10ad8339165989adf38a1813152529724dbcd

Don't use os.environ.get expecting worker env as it's running on the controller.
Comment 8 Mihail Stanciu 2016-04-18 13:54:04 UTC 
Verified. IDs are OK now, just need to update the website to take them into consideration.