Remove client.catapult builders? |
||||||
Issue descriptionAnybody know much about these builders and whether it's ok to remove them entirely? https://build.chromium.org/p/client.catapult/builders/ From what I can see, there are 4 builders on this waterfall: Catapult Cron Linux Catapult Linux Catapult Mac Catapult Windows Catapult Cron Linux and Catapult Linux have been offline since Feb 2017, so those seem ok to remove. Catapult Windows seemingly has never had a successful run, at least as far back as I can see. Catapult Mac looks like a clone of Catapult Mac Tryserver. Any objection to just removing all of these?
,
Aug 31
,
Aug 31
,
Aug 31
,
Aug 31
No objection from me
,
Aug 31
I think we should remove them. They were originally added to provide a waterfall that mirrors the CQ, but they're not used.
,
Aug 31
Ok so to take these down, looks like the master definitions are in build/master/master.client.catapult There's also a catapult_cron.py recipe and associated files that can be deleted. efoo@/hinoka@ sounds like you can ignore these and just that last android tryserver is needed to switch catapult over.
,
Aug 31
The following revision refers to this bug: https://chrome-internal.googlesource.com/infradata/master-manager/+/ec1c01d7daee863d01cd3684cb52edf31ea6d6ea commit ec1c01d7daee863d01cd3684cb52edf31ea6d6ea Author: Simon <simonhatch@google.com> Date: Fri Aug 31 21:33:33 2018
,
Aug 31
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/c87a9703c8f050ce28bb455e0e5973b5293295f3 commit c87a9703c8f050ce28bb455e0e5973b5293295f3 Author: Simon <simonhatch@google.com> Date: Fri Aug 31 21:57:50 2018 Remove master.client.catapult and cron recipe Bug: 879566 Change-Id: If7ac3708299bdacc04c3f001f8498f2820cfd1bc Reviewed-on: https://chromium-review.googlesource.com/1200285 Commit-Queue: Simon Hatch <simonhatch@chromium.org> Reviewed-by: Ryan Tseng <hinoka@chromium.org> [delete] https://crrev.com/3b2538294b409a87027e64fccc06e0256aed952d/masters/master.client.catapult/master_site_config.py [delete] https://crrev.com/3b2538294b409a87027e64fccc06e0256aed952d/masters/master.client.catapult/builders.pyl [delete] https://crrev.com/3b2538294b409a87027e64fccc06e0256aed952d/scripts/slave/recipes/catapult_cron.py [modify] https://crrev.com/c87a9703c8f050ce28bb455e0e5973b5293295f3/tests/masters_test.py [delete] https://crrev.com/3b2538294b409a87027e64fccc06e0256aed952d/masters/master.client.catapult/Makefile [delete] https://crrev.com/3b2538294b409a87027e64fccc06e0256aed952d/scripts/slave/recipes/catapult_cron.expected/OWNERS [delete] https://crrev.com/3b2538294b409a87027e64fccc06e0256aed952d/scripts/slave/recipes/catapult_cron.expected/basic.json [delete] https://crrev.com/3b2538294b409a87027e64fccc06e0256aed952d/masters/master.client.catapult/master.cfg
,
Sep 1
Marking as fixed. client.catapult Buildbot master is deleted. I'll take care of decomming the HW with labs. Thanks Simon! |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by simonhatch@chromium.org
, Aug 31