Bug 8569 - Make yocto autobuilder submitter id configurable
Summary: Make yocto autobuilder submitter id configurable
Status: VERIFIED FIXED
Alias: None
Product: AutoBuilder
Classification: Infrastructure
Component: autobuilder (show other bugs)
Version: 2.1
Hardware: x86 Multiple
: Medium+ enhancement
Target Milestone: 2.1 M4
Assignee: Beth Flanagan
QA Contact: Mihail Stanciu
URL:
Whiteboard:
: 8071 8683 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-10-26 09:28 UTC by Belen Barros Pena
Modified: 2016-04-18 13:54 UTC (History)
7 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.