This is a reland of commit 6f00b586dd
Original change's description:
> Reland "Migrate rust hook to deps and create sync_deps.py"
>
> This is a reland of commit 2af7751ec4, with the
> following changes:
> * update_rust - use VERSION instead of INSTALLED_VERSION
>
> gclient ensures that the rust package is not partially extracted, so
> INSTALLED_VERSION is no longer needed.
>
>
> Original change's description:
> > Migrate rust hook to deps and create sync_deps.py
> >
> > After someone runs upload_revision.py, triggers the packager bots,
> > and manually copies the packages to prod, they will now run
> > sync_deps.py to update the DEPS file. After doing that they can
> > trigger a CQ run to test the new packages.
> >
> > Bug: b/338612245
> > Change-Id: Ie95efbf2049e651192d52ca7531c20125b3dd537
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5536220
> > Reviewed-by: Hans Wennborg <hans@chromium.org>
> > Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1304616}
>
> Bug: b/338612245
> Change-Id: I2038e57a2e767eea10df6e2b72b6d2e2344d630e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5560825
> Reviewed-by: Hans Wennborg <hans@chromium.org>
> Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
> Reviewed-by: danakj <danakj@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1305165}
Bug: b/338612245
Change-Id: I597a7ee0e894109e34cd374bf20d270a01705354
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5569382
Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1308016}
This reverts commit 6f00b586dd.
Reason for revert: Causing Compile failures on latest Chrome Canary 127.0.6498.0+
Original change's description:
> Reland "Migrate rust hook to deps and create sync_deps.py"
>
> This is a reland of commit 2af7751ec4, with the
> following changes:
> * update_rust - use VERSION instead of INSTALLED_VERSION
>
> gclient ensures that the rust package is not partially extracted, so
> INSTALLED_VERSION is no longer needed.
>
>
> Original change's description:
> > Migrate rust hook to deps and create sync_deps.py
> >
> > After someone runs upload_revision.py, triggers the packager bots,
> > and manually copies the packages to prod, they will now run
> > sync_deps.py to update the DEPS file. After doing that they can
> > trigger a CQ run to test the new packages.
> >
> > Bug: b/338612245
> > Change-Id: Ie95efbf2049e651192d52ca7531c20125b3dd537
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5536220
> > Reviewed-by: Hans Wennborg <hans@chromium.org>
> > Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1304616}
>
> Bug: b/338612245
> Change-Id: I2038e57a2e767eea10df6e2b72b6d2e2344d630e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5560825
> Reviewed-by: Hans Wennborg <hans@chromium.org>
> Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
> Reviewed-by: danakj <danakj@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1305165}
Bug: b/338612245
Change-Id: Ib0e8c4aeeab1636b058970b0956728bfb307021c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5564509
Commit-Queue: Prudhvikumar Bommana <pbommana@google.com>
Owners-Override: Prudhvikumar Bommana <pbommana@google.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Adrian Taylor <adetaylor@google.com>
Cr-Commit-Position: refs/heads/main@{#1305732}
This is a reland of commit 2af7751ec4, with the
following changes:
* update_rust - use VERSION instead of INSTALLED_VERSION
gclient ensures that the rust package is not partially extracted, so
INSTALLED_VERSION is no longer needed.
Original change's description:
> Migrate rust hook to deps and create sync_deps.py
>
> After someone runs upload_revision.py, triggers the packager bots,
> and manually copies the packages to prod, they will now run
> sync_deps.py to update the DEPS file. After doing that they can
> trigger a CQ run to test the new packages.
>
> Bug: b/338612245
> Change-Id: Ie95efbf2049e651192d52ca7531c20125b3dd537
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5536220
> Reviewed-by: Hans Wennborg <hans@chromium.org>
> Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1304616}
Bug: b/338612245
Change-Id: I2038e57a2e767eea10df6e2b72b6d2e2344d630e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5560825
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1305165}
After someone runs upload_revision.py, triggers the packager bots,
and manually copies the packages to prod, they will now run
sync_deps.py to update the DEPS file. After doing that they can
trigger a CQ run to test the new packages.
Bug: b/338612245
Change-Id: Ie95efbf2049e651192d52ca7531c20125b3dd537
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5536220
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1304616}
This way, the cq makes sure that we run all the bots we want to run
(as long as we use the upload script), which is less error-prone.
This doesn't work for internal chrome/try bots yet, so these stay on
the old system for now.
Bug: 1047347
Change-Id: Iedecf56d4cfa5f6e923b716b927147ac3c28dedc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016844
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#736982}
deqp uses exceptions for some reason, and 32-bit SEH is fairly
different on the compiler side than 64-bit SEH. Since we seem
to be bound to keep supporting deqp for now, add it to the cq
so that we learn about failures before rolls land.
Bug: 1045650
Change-Id: I820b514f8c659fa322b69d12f0d388552eb513d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2024689
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735886}
It's no longer run by the CQ by default
(see "[chromium-dev] [PSA] win7-rel become optional in CQ").
TBR=thakis
Bug: none
Change-Id: I2993390c12e0445b58ebb6868c512ee65f129c6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758063
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#687596}
A few years ago, we added ios-device to the try jobs run on clang rolls,
in #493848.
At some point, that bot disappeared, and after the luci switch non-existent
bots on a try invocation are apparently ignored silently. So we never noticed
that this bot stopped doing anything, but we _did_ notice clang rolls breaking
iOS device builds more frequently.
Remove the defunct ios-device bot and add the new and (for now) funct
iphone-device and ipad-device bots, which will hopefully catch iOS device
issues caused by clang rolls before the rolls land.
Bug: 754829,967387
Change-Id: I5cf9ff2bbb9458c547e1a0427fcc31ff788a3759
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1652541
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#668483}
The old SVN repository is expected to get turned off eventually, and
this prepares our scripts for that. It also simplifies the build a little
and gets rid of our dependency on SVN.
This increases the size of the Linux package slightly due to
http://llvm.org/PR41956 which will be investigated.
Bug: 884608, 897796
Change-Id: I272cb2516bb99a6dc4643f32c42d6e852583174e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1615030
Commit-Queue: Hans Wennborg <hans@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662087}
Currently, every clang package contains a buildlog.txt that's ~7MB.
It compresses well, and after #656764 it will be less than half the size,
but there's still no need for all devs and bots to download this data.
Upload it to a (gzip-compressed) separate file 'clang-12345-buildlog.txt'
instead.
While here, remove unreferenced function GsutilArchiveExists().
Also pass `args.update` instead of `args` as first argument of
MaybeUpload(), so the function doesn't have two separate `args`
parameters.
Bug: none
Change-Id: I5a3a9aa4ad2af17bd3fad19d11a64a66720ea9e1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1596351
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#656979}
It seems the target to build is no longer telemetry_perf_tests but
performance_test_suite.
TBR=thakis
Bug: none
Change-Id: Ib15b98921faf1fed5c0451ffb06d9ba884f2874e
Reviewed-on: https://chromium-review.googlesource.com/c/1269868
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597858}
This adds llvm-strip for Linux and Mac and llvm-ar for Mac. These are
needed to build Fuchsia on Mac and Linux hosts. For now, llvm-strip is
in a separate package.
Bug: 877080
Change-Id: If10284d6f76b0cf2397ccbe9d64b98abf04589d0
Reviewed-on: https://chromium-review.googlesource.com/1193949
Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588084}
For some reason, the ```shell stuff is getting output in the html.
TBR=thakis
Bug: none
Change-Id: Idecf83e029e5bca4cf969c2145a458193aa5df56
Reviewed-on: https://chromium-review.googlesource.com/1158238
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579735}
Updates some commands to run try jobs that have moved to LUCI, and
adds linux_chromium_chromeos_msan_rel_ng.
Change-Id: I9048da9f6dcb897b5125183d3f66b5995e740de8
Reviewed-on: https://chromium-review.googlesource.com/1112347
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569749}
LUCI bots apparently need -B instead of -m.
While here, also remove \ after && (the latter already tells the shell that
the command is continuing) and fix indentation.
TBR=hans
Bug: 844392
Change-Id: I9af96be984264e240257a5fa55e8b84c65f0602d
Reviewed-on: https://chromium-review.googlesource.com/1067417
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560276}
Adding llvm-cfi-verify to the chromium clang package allows us to use the tool for CFI coverage metrics to be uploaded to the chromium perf bots.
Bug: 784953
Change-Id: Ia25b419eda5f9fdf6bf6efab731640755c2d3b08
Reviewed-on: https://chromium-review.googlesource.com/769239
Reviewed-by: Peter Collingbourne <pcc@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517936}
