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

Issue 642813 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Dec 2016
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 537368



Sign in to add a comment

Add llvm test for chromiumos-sdk

Project Member Reported by yunlian@chromium.org, Aug 31 2016

Issue description

   We want chromiumos-sdk to test both gcc and llvm in the process of migration. Below is the plan for chromiumos-sdk buildbot change for LLVM migration on amd64 boards.

1) Add peppy to chromiumos-sdk test boards
2) Make sure clang/llvm updates are gated by the sdk
3) Migrate amd64-generic and falco to LLVM
3) Migrate all the amd64 boards to LLVM
4) Remove peppy from chromiumos-sdk 
 
Labels: Restrict-View-Google
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/crosutils/+/d1e571be89841539f1729ba4be1a2e0fcb93c3ef

commit d1e571be89841539f1729ba4be1a2e0fcb93c3ef
Author: Yunlian Jiang <yunlian@chromium.org>
Date: Fri Sep 02 21:00:01 2016

update_chroot: only update llvm when binpkgs are available

BUG= chromium:642813 
TEST=uprev llvm and run cros_sdk --delete and cros_sdk
     llvm was not updated.

Change-Id: Ic33a0ad96695516055ec0336ef6ed4f994934907
Reviewed-on: https://chromium-review.googlesource.com/380326
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/d1e571be89841539f1729ba4be1a2e0fcb93c3ef/update_chroot

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/67653ff6f3c1b3ff37d9eb346234033496758593

commit 67653ff6f3c1b3ff37d9eb346234033496758593
Author: Yunlian Jiang <yunlian@chromium.org>
Date: Wed Aug 31 18:04:03 2016

chromite: add peppy to the chromiumos-sdk bot

In the process of migrating from gcc to LLVM, we need chromiumos-sdk
to test both gcc and llvm for amd64 board. So we need to add another
amd64 board to chromiumos-sdk.

BUG= chromium:642813 
TEST=the peppy bot is added.

Change-Id: Idc30f32ff3de3cdae6282fc0788891ca98c4b267
Reviewed-on: https://chromium-review.googlesource.com/379136
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/67653ff6f3c1b3ff37d9eb346234033496758593/cbuildbot/config_dump.json
[modify] https://crrev.com/67653ff6f3c1b3ff37d9eb346234033496758593/cbuildbot/chromeos_config.py

Blocking: 537368
Labels: -Pri-3 Pri-2
can this be closed?
I prefer to keep this open until we finish the migration.
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/e81fce72a0e9da1fa627208341c45f2df3066afd

commit e81fce72a0e9da1fa627208341c45f2df3066afd
Author: Yunlian Jiang <yunlian@google.com>
Date: Sun Nov 06 16:17:52 2016

chromite: replace peppy with veyron_jaq board for chromiumos-sdk test

In the process of migrating from gcc to LLVM, we need chromiumos-sdk
to test both gcc and llvm for arm board. So we need to add another
arm board to chromiumos-sdk. Meanwhile, we have migrated amd64 boards,
we can remove peppy board from the chromiumo-sdk test.

BUG= chromium:642813 
CQ-DEPEND=CL:406491
TEST=the peppy board is replaced with veyron_jaq.

Change-Id: I4406ce7e4ead51f91680be1a7c15806347471e9a
Reviewed-on: https://chromium-review.googlesource.com/404472
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/e81fce72a0e9da1fa627208341c45f2df3066afd/cbuildbot/config_dump.json
[modify] https://crrev.com/e81fce72a0e9da1fa627208341c45f2df3066afd/cbuildbot/chromeos_config.py

Project Member

Comment 8 by bugdroid1@chromium.org, Nov 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/a1105cd858b2b58b48cd0b49565898685cca42de

commit a1105cd858b2b58b48cd0b49565898685cca42de
Author: Yunlian Jiang <yunlian@google.com>
Date: Wed Nov 02 20:40:06 2016

chromite: use commands.Setupboard for sdk test.

We want to add veyron_jaq to chromiumos-sdk buildbot. But we get
errors like
The following license changes are necessary to proceed:
 (see "package.license" in the portage(5) man page for more details)

This CL fixes that.

BUG= chromium:642813 
TEST=cbuildbot chromiumos-sdk passes.

Change-Id: Ib9273011b858b4c54856460a4a7d314c5172d41f
Reviewed-on: https://chromium-review.googlesource.com/406491
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/a1105cd858b2b58b48cd0b49565898685cca42de/cbuildbot/stages/sdk_stages.py
[modify] https://crrev.com/a1105cd858b2b58b48cd0b49565898685cca42de/cbuildbot/commands.py

Project Member

Comment 9 by bugdroid1@chromium.org, Dec 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/e29b9fc8207834950bb45f8bf82b2102554b98d5

commit e29b9fc8207834950bb45f8bf82b2102554b98d5
Author: Yunlian Jiang <yunlian@google.com>
Date: Wed Dec 07 18:06:44 2016

chromite: remove veyron_jaq from chromiumos-sdk test.

We have finished the process of migrating compiler for arm
boards from gcc to LLVM. So there is no need to keep two
arm boards in chromiumos-sdk test.

BUG= chromium:642813 
TEST=the veyron_jaq is removed.

Change-Id: I28902da812e9669899553b617108220de5b98233
Reviewed-on: https://chromium-review.googlesource.com/417231
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Luis Lozano <llozano@chromium.org>

[modify] https://crrev.com/e29b9fc8207834950bb45f8bf82b2102554b98d5/cbuildbot/config_dump.json
[modify] https://crrev.com/e29b9fc8207834950bb45f8bf82b2102554b98d5/cbuildbot/chromeos_config.py

Status: Fixed (was: Untriaged)
Labels: -Restrict-View-Google

Comment 12 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58
Status: Verified (was: Fixed)

Sign in to add a comment