New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 786430 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Tracker for OBBS Bot

Project Member Reported by ashleymarie@chromium.org, Nov 17 2017

Issue description

I'm just going to use this to track any changes I make to the OBBS bots

 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e471aaf919ae21fe38b0c033c625b2f5d366a5de

commit e471aaf919ae21fe38b0c033c625b2f5d366a5de
Author: Ashley Enstad <ashleymarie@chromium.org>
Date: Fri Nov 17 20:10:40 2017

Trying to green the obbs bot

BUG=chromium:786430

Change-Id: I6e7b2878295566a15f482f932d0ba033e38daab2
Reviewed-on: https://chromium-review.googlesource.com/777029
Commit-Queue: Ashley Enstad <ashleymarie@chromium.org>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#517528}
[modify] https://crrev.com/e471aaf919ae21fe38b0c033c625b2f5d366a5de/tools/perf/core/merge_script.py

Comment 2 by eyaich@chromium.org, Nov 21 2017

nit: we should really name this to ctu bot :)
Components: Speed>Benchmarks>Waterfall
Some more details on this...
I'm working on moving the code which uploads perf test results to the perf dashboard from the infra codebase to the src codebase. Once the code is moved to src side, it will be called from the infra side. In order to make sure this flow isn't broken as I slowly move code over, I'm setting up a trybot that will allow me to test the merge/upload script changes before submitting them. Since the code that tells infra to use this script is in testing/buildbot/chromium.perf.fyi.json, it's difficult to test the full flow locally. Thus, we've got a builder that provides the proper arguments and are adding a trybot matching that builder to test changes before submitting them (and breaking the builder).

FWIW, this bot is basically going to be used by me for development purposes. After we transition over to all of chromium perf using this script, I'll be deleting the bot. So for now I'm not concerned about renaming it :)

Comment 5 by eyaich@chromium.org, Nov 27 2017

sgtm :)  thanks for clarifying!
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bac382ff75578ccaadcdf3fdaf1e916d6332d4f9

commit bac382ff75578ccaadcdf3fdaf1e916d6332d4f9
Author: Ashley Enstad <ashleymarie@chromium.org>
Date: Mon Nov 27 20:12:41 2017

Adding an entry for the obbs trybot

BUG=chromium:786430

Change-Id: I047d574690c3203fdb4ec71b3a1cac1c71ec1d2f
Reviewed-on: https://chromium-review.googlesource.com/777392
Commit-Queue: Ashley Enstad <ashleymarie@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: David Tu <dtu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519379}
[modify] https://crrev.com/bac382ff75578ccaadcdf3fdaf1e916d6332d4f9/tools/mb/mb_config.pyl

Sign in to add a comment