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

Issue 803137 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocked on: View detail
issue 801289
issue 807273
issue 785603
issue 790422
issue 792928
issue 810414
issue 818319



Sign in to add a comment

Converting perf infra to LUCI

Project Member Reported by nedngu...@google.com, Jan 17 2018

Issue description

This is the tracking bug for the work of converting perf infra to LUCI. 

Guidance:  go/migrate2luci


 

Comment 1 by eyaich@chromium.org, Jan 23 2018

Blockedon: 801289
Blockedon: 807273
Cc: charliea@chromium.org
I will try to convert perf fyi builders to LUCI first as it won't be too consequential in case anything goes wrong.

My first target would be this builder: https://ci.chromium.org/buildbot/chromium.perf.fyi/Battor%20Agent%20Linux/


+charlie FYI
SGTM
Blockedon: 790422
Owner: nednguyen@chromium.org
Blockedon: 792928
Blockedon: 810414
Blockedon: 785603
Blockedon: 818319
Owner: jbudorick@chromium.org
John owns this effort now
Project Member

Comment 13 by bugdroid1@chromium.org, Aug 29

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

commit 6e2db3754c142493e00487eb34c7b849969e78e9
Author: nednguyen <nednguyen@google.com>
Date: Wed Aug 29 18:38:54 2018

Update all milo entry of LUCI perf builder to use luci.chrome.ci bucket

Bug: 803137
Change-Id: I0ecd7f1885cca5b15c3aa2e6c95cddf867753a75
Reviewed-on: https://chromium-review.googlesource.com/1195746
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#587212}
[modify] https://crrev.com/6e2db3754c142493e00487eb34c7b849969e78e9/infra/config/global/luci-milo.cfg

Project Member

Comment 14 by bugdroid1@chromium.org, Oct 9

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

commit d9f7516c88d2a7cc58253a11d60ab0c898e3302c
Author: Ned Nguyen <nednguyen@google.com>
Date: Tue Oct 09 00:37:26 2018

Update src side config after migrating all chromium.perf.fyi builders to LUCI

Bug: 803137
Change-Id: I7d92b8a873d4566b09e94ac93fe47c4ab296e5b4
Reviewed-on: https://chromium-review.googlesource.com/c/1270097
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#597755}
[modify] https://crrev.com/d9f7516c88d2a7cc58253a11d60ab0c898e3302c/infra/config/global/luci-milo.cfg
[modify] https://crrev.com/d9f7516c88d2a7cc58253a11d60ab0c898e3302c/tools/mb/mb_config.pyl

Owner: nedngu...@google.com
I ended up doing a bunch of work here. Taking this back from John
Project Member

Comment 16 by bugdroid1@chromium.org, Oct 9

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

commit 2bc28e922bec102da107766a3bd060400b19319c
Author: Ned Nguyen <nednguyen@google.com>
Date: Tue Oct 09 20:37:46 2018

Remove service account argument in process_perf_results.py

As these builders are converted to LUCI, they are using the luci service account
and no longer need the pre-deployed service account files.

Bug: 803137
Change-Id: I27c496d0168ac41f87aff6d81ffe80da22b2c025
Reviewed-on: https://chromium-review.googlesource.com/c/1271518
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#598069}
[modify] https://crrev.com/2bc28e922bec102da107766a3bd060400b19319c/testing/buildbot/chromium.perf.fyi.json

Project Member

Comment 17 by bugdroid1@chromium.org, Oct 13

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

commit aab9a5f504b4e91cbe81925aa930646ae04fa417
Author: Ned Nguyen <nednguyen@google.com>
Date: Sat Oct 13 00:09:23 2018

Remove credential file arg from perf merge script

As these builders are converted to LUCI, they are using the luci service account
and no longer need the pre-deployed service account files.

Bug: 803137
Change-Id: If9f8c9507a4bb40d83185b5c0a2a5e332838495a
Reviewed-on: https://chromium-review.googlesource.com/c/1279174
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599429}
[modify] https://crrev.com/aab9a5f504b4e91cbe81925aa930646ae04fa417/testing/buildbot/chromium.perf.json
[modify] https://crrev.com/aab9a5f504b4e91cbe81925aa930646ae04fa417/tools/perf/core/perf_data_generator.py
[modify] https://crrev.com/aab9a5f504b4e91cbe81925aa930646ae04fa417/tools/perf/core/perf_data_generator_unittest.py

Owner: jbudorick@chromium.org
There is only issue 807273 left blocking this & John is reviewing the work, so I am reassigning this to John

Sign in to add a comment