New issue
Advanced search Search tips

Issue 619128 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Remove chromium.mojo waterfall

Project Member Reported by jam@chromium.org, Jun 10 2016

Issue description

We don't need it anymore, so let's remove it.
 

Comment 1 by msw@chromium.org, Jun 13 2016

Cc: -msw@chromium.org
Owner: msw@chromium.org
Status: Assigned (was: Available)
We'll first need to move some tests to main bots to keep existing coverage.
testing/buildbot/chromium.mojo.json runs some tests not listed elsewhere:

  browser_tests --run-in-mash ... (not run on any Linux/Win main bots)

  media_mojo_shell_unittests (not run on any Win main bots)
  views_mus_interactive_ui_tests (not run on any Win main bots)
  views_mus_unittests (not run on any Win main bots)

  mus_ws_unittests (run on "Win8 Aura", move to "Win x64 GN"?)

Comment 2 by msw@chromium.org, Jun 13 2016

Cc: jamescook@chromium.org sadrul@chromium.org sky@chromium.org
Adding folks that might rely on this waterfall for mojo test bringup.
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 30 2016

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

commit 8e8c0de821df785afa924d8494a8796a6b763fdf
Author: msw <msw@chromium.org>
Date: Thu Jun 30 03:00:50 2016

Move chromium.mojo buildbot configs to chromium.fyi.

Remove unused chromium_mojo "Chromium Mojo Android".
Move chromium_mojo "Chromium Mojo Windows" to chromium_fyi "Mojo Windows".
Move chromium_mojo "Chromium Mojo Linux" to chromium_fyi "Mojo ChromiumOS".
(we now need fyi build/test coverage on chrome os, not desktop linux)

Remove now unused android_gn_release_bot config.

This CL will be accompanied by: http://codereview.chromium.org/2112743002/

BUG= 619128 
TEST=Moved builders perform expected build/test steps.
R=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2114453002
Cr-Commit-Position: refs/heads/master@{#403076}

[modify] https://crrev.com/8e8c0de821df785afa924d8494a8796a6b763fdf/testing/buildbot/chromium.fyi.json
[delete] https://crrev.com/ede2091f3f4a3a836e3e2efb13bd8c691a60f6d8/testing/buildbot/chromium.mojo.json
[modify] https://crrev.com/8e8c0de821df785afa924d8494a8796a6b763fdf/tools/mb/mb_config.pyl

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 30 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/71d600393f83c9b4ba693edbf61e79097db9a471

commit 71d600393f83c9b4ba693edbf61e79097db9a471
Author: msw <msw@chromium.org>
Date: Thu Jun 30 03:11:23 2016

Move Mojo bots to chromium_fyi; remove mojo master.

Remove unused chromium_mojo "Chromium Mojo Android".
Move chromium_mojo "Chromium Mojo Windows" to chromium_fyi "Mojo Windows".
Move chromium_mojo "Chromium Mojo Linux" to chromium_fyi "Mojo ChromiumOS".
(we now need fyi build/test coverage on chrome os, not desktop linux)

Move required slave configs; remove chromium_mojo master references.
Run "scripts/slave/unittests/recipe_simulation_test.py train"

This CL will be accompanied by: https://codereview.chromium.org/2114453002

BUG= 619128 
TEST=Chromium Mojo bots move to Chromium FYI master.
R=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2112743002

[modify] https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471/masters/master.chromium.fyi/master.cfg
[modify] https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471/masters/master.chromium.fyi/slaves.cfg
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/masters/master.chromium.mojo/Makefile
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/masters/master.chromium.mojo/buildbot
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/masters/master.chromium.mojo/buildbot.tac
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/masters/master.chromium.mojo/builders.pyl
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/masters/master.chromium.mojo/master.cfg
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/masters/master.chromium.mojo/master_site_config.py
[modify] https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471/scripts/slave/recipe_modules/chromium_tests/builders.py
[modify] https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471/scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/scripts/slave/recipe_modules/chromium_tests/chromium_mojo.py
[rename] https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Mojo_ChromiumOS.json
[rename] https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Mojo_Windows.json
[delete] https://crrev.com/b01754e381bcf57f2de3d5fdc1ee46ac94fa4d31/scripts/slave/recipes/chromium.expected/full_chromium_mojo_Chromium_Mojo_Android.json
[modify] https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471/tests/masters_test.py

Project Member

Comment 5 by bugdroid1@chromium.org, Jun 30 2016

Project Member

Comment 6 by bugdroid1@chromium.org, Jun 30 2016

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

commit b34fe559cd861d55a7fa6469273e79708b19abf3
Author: recipe-roller <recipe-roller@chromium.org>
Date: Thu Jun 30 03:28:57 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/71d600393f83c9b4ba693edbf61e79097db9a471 Move Mojo bots to chromium_fyi; remove mojo master. (msw@chromium.org)

R=msw@chromium.org,dpranke@chromium.org
BUG= 619128 

TBR=martiniss@chromium.org,phajdan.jr@chromium.org

Review-Url: https://codereview.chromium.org/2112603004

[modify] https://crrev.com/b34fe559cd861d55a7fa6469273e79708b19abf3/infra/config/recipes.cfg

Comment 7 by msw@chromium.org, Jun 30 2016

Status: Fixed (was: Assigned)
This is generally fixed; the bots are moved to FYI.

Sean is following up with our sysadmin to bring up the win bot
( it's currently offline; related to Issue 624839 )

I filed Issue 624875 to shut down the deprecated chromium.mojo master
( you can still access https://build.chromium.org/p/chromium.mojo/console )
Project Member

Comment 8 by bugdroid1@chromium.org, Jun 30 2016

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

commit 14746e2fcc5eafaa5cba62ca94f6441d3e3e7d71
Author: msw <msw@chromium.org>
Date: Thu Jun 30 19:51:43 2016

Remove msw per-file testing/buildbot/OWNERS entry.

The chromium.mojo.json file no longer exists.
It was folded into chromium.fyi.json in:
  https://codereview.chromium.org/2114453002/

BUG= 619128 
TEST=NONE
TBR=sky@chromium.org

Review-Url: https://codereview.chromium.org/2111463005
Cr-Commit-Position: refs/heads/master@{#403259}

[modify] https://crrev.com/14746e2fcc5eafaa5cba62ca94f6441d3e3e7d71/testing/buildbot/OWNERS

Project Member

Comment 9 by bugdroid1@chromium.org, Oct 18 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome-golo/chrome-golo/+/169798506b3603174f9ac36ff8aacc4445ea7c1a

commit 169798506b3603174f9ac36ff8aacc4445ea7c1a
Author: Peter Schmidt <pschmidt@google.com>
Date: Wed Oct 18 18:50:54 2017

Project Member

Comment 10 by bugdroid1@chromium.org, Oct 19 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/9c52920ab9a0e3da3b43561bc6cee6e8b83fb88d

commit 9c52920ab9a0e3da3b43561bc6cee6e8b83fb88d
Author: Peter Schmidt <pschmidt@google.com>
Date: Thu Oct 19 21:41:06 2017

Sign in to add a comment