Performance machines are now in the internal sandbox. Need to be removed from playercore buildbot.
Created attachment 523042 [details] [diff] [review] Remove perf builders and code
Attachment #523042 - Flags: review?(brbaker)
Comment on attachment 523042 [details] [diff] [review] Remove perf builders and code r- for a single missing item, assume r+ when addressed 1) need to update public_html/announce.html to remove the "performance" section since these builders are going to be removed.
Attachment #523042 - Flags: review?(brbaker) → review-
Additional work items that I think need to be addressed: - link/visibility to performance waterfall from the current waterfall view - does the "buildbot watcher performance" service need to be running on playercore? If not uninstall the service and remove performance watcher config file - should build/buildbot/*-performance/buildbot.tac be removed from source or updated?
Created attachment 523144 [details] [diff] [review] Updated patch - public_html/announce.html updated with a link to internal sandbox - buildbot watcher perf service has been removed from playercore - buildbot.tac files updated
Comment on attachment 523144 [details] [diff] [review] Updated patch r+ but please make sure to maintain and add the proper license header in the buildbot.tac files (patch shows header being removed and some of the new files do not have the header)
Attachment #523144 - Flags: review?(brbaker) → review+
changeset: 6216:faef081aaecf user: Chris Peyer <email@example.com> summary: Bug 646522: remove performance builders (r=brbaker) http://hg.mozilla.org/tamarin-redux/rev/faef081aaecf
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.