Converting perf infra to LUCI |
|||||||||||
Issue description
,
Jan 23 2018
Perf specific considerations: https://docs.google.com/document/d/1ui7z0W7f3Qf5JBmzGgy-kDnNZzXYPkx1nJJsioQ5saM/edit#heading=h.kg0vkrv1q5an
,
Jan 30 2018
,
Feb 2 2018
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
,
Feb 2 2018
SGTM
,
Feb 2 2018
monorail query "master:perf via=luci-migration" can be used to find individual builder bugs https://bugs.chromium.org/p/chromium/issues/list?can=2&q=master%3Aperf+via%3Dluci-migration&sort=&groupby=master&colspec=ID+Pri+M+Stars+ReleaseBlock+Component+Status+Owner+Summary+OS+Modified
,
Feb 7 2018
,
Feb 7 2018
,
Feb 8 2018
,
Feb 15 2018
,
Mar 2 2018
,
Aug 6
John owns this effort now
,
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
,
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
,
Oct 9
I ended up doing a bunch of work here. Taking this back from John
,
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
,
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
,
Nov 1
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 |
|||||||||||
Comment 1 by eyaich@chromium.org
, Jan 23 2018