Commits · b5effa1f24a7f909997800559edbc53e5ba08fbf · Linshizhi / Chromium_Depot_tools

14 Dec, 2016 4 commits

git cl + git numberer: fix 3+ lineage ordering. · b5effa1f

Andrii Shyshkalov authored Dec 14, 2016

In other words, end 2 end test was a really awesome idea.

R=machenbach@chromium.org,sergiyb@chromium.org
BUG=642493
TEST=git cl land of https://codereview.chromium.org/2575043003 succeeded

Change-Id: I568ce79baf109b2aa556e4343527b63f39c10d00
Reviewed-on: https://chromium-review.googlesource.com/419478
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Aaron Gable <agable@chromium.org>

b5effa1f

git cl refactor: s/should_git_number/should_add_git_number · 8f15f3ec

Andrii Shyshkalov authored Dec 14, 2016

R=sergiyb@google.com
BUG=chromium:642493

Change-Id: Id7b732d82c25ce88a3295ff54c79cf2a66e0d5b9
Reviewed-on: https://chromium-review.googlesource.com/419437Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

8f15f3ec

Revert "Generate patchset name from commit subject" · a5a1eea5

Andrii Shyshkalov authored Dec 14, 2016

This reverts commit 947f2ee8.

Reason for revert: unnecessarily sets the title for the first upload,
which isn't done for Rietveld.

Original change's description:
> Generate patchset name from commit subject
> 
> BUG=672332
> 
> Change-Id: If9c662f11b217a52cac7c9c47ad9f3d978148c2e
> Reviewed-on: https://chromium-review.googlesource.com/419330
> Commit-Queue: Aaron Gable <agable@chromium.org>
> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
> 

TBR=agable@chromium.org,tandrii@chromium.org,chromium-reviews@chromium.org
BUG=672332
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Change-Id: I9dcec1b02c8f16ceca8338acef4a1fc174aef35c
Reviewed-on: https://chromium-review.googlesource.com/419517
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

a5a1eea5

Roll recipe dependencies (trivial). · e79186f7

recipe-roller authored Dec 14, 2016

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)

recipe_engine:
  https://crrev.com/a556ab48b2ae66184ccc49bfd37775a245faa87c Add repr and str methods to config.Dict (dsansome@chromium.org)

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

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2571153002

e79186f7

13 Dec, 2016 3 commits

Generate patchset name from commit subject · 947f2ee8

Aaron Gable authored Dec 13, 2016

BUG=672332

Change-Id: If9c662f11b217a52cac7c9c47ad9f3d978148c2e
Reviewed-on: https://chromium-review.googlesource.com/419330
Commit-Queue: Aaron Gable <agable@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

947f2ee8

Clean up after failed gerrit cl description edits · e9373d6e

Aaron Gable authored Dec 13, 2016

If either the edit creation or edit publishing fail, we're left with
an unpublished edit hanging around. This can result in a follow-up
invocation of 'git cl desc' (e.g. if the user changes their mind, or
retries) will also fail. So, clean up after ourselves.

BUG=671091

Change-Id: Icfe53996acf2555b82be0695a3c86ebb8c50b936
Reviewed-on: https://chromium-review.googlesource.com/418449Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>

e9373d6e

Add cipd bootstrap scripts to depot_tools. · 2188fe94

Robert Iannucci authored Dec 02, 2016

This takes advantage of powershell on windows for a cleanish duplicate of the
posix version.

R=dnj@chromium.org, vadimsh@chromium.org
BUG=663843

Change-Id: Ib23a044ff912e3239b58848a26143eb6575826d5
Reviewed-on: https://chromium-review.googlesource.com/414228
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>

2188fe94

09 Dec, 2016 2 commits

git cl + git-numberer: add extra regression test case. · cbec8efc

Andrii Shyshkalov authored Dec 09, 2016

R=machenbach@chromium.org
BUG=chromium:642493,672043

Change-Id: Ie50637d6e01e40ddd15dcdfc1430c9554fa82976
Reviewed-on: https://chromium-review.googlesource.com/418715Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

cbec8efc

Gerrit: git cl set-commit -d: don't spam reviwers/owner. · 828701bd

Andrii Shyshkalov authored Dec 09, 2016

R=sergiyb@chromium.org,machenbach@chromium.org
BUG=

Change-Id: Ia89e91479c22adeebc85a4432cd9f7b8d95a8fde
Reviewed-on: https://chromium-review.googlesource.com/418655Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

828701bd

08 Dec, 2016 2 commits

git cl: insert "Review-Url" instead "Review URL" footer like CQ. · f01be229

Andrii Shyshkalov authored Dec 08, 2016

R=sergiyb@chromium.org

Change-Id: I2df867e381ceb1035add84563d3b444b0ee0ea94
Reviewed-on: https://chromium-review.googlesource.com/417267Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

f01be229

git cl: use gnumbd config instead of PENDING_REF_PREFIX of codereview.settings. · 768f1d88

Andrii Shyshkalov authored Dec 08, 2016

BUG=chromium:642493,672043
R=machenbach@chromium.org,iannucci@chromium.org

Change-Id: I0abc31b95b1766fd5fd24c1379b538d0c5291011
Reviewed-on: https://chromium-review.googlesource.com/417259Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Commit-Queue: Michael Achenbach <machenbach@chromium.org>

768f1d88

07 Dec, 2016 1 commit

git_cl: add GitNumbererState in preparation to stop gnumbd service. · cd6a9363

Andrii Shyshkalov authored Dec 07, 2016

The new class is unused and doesn't change any existing functionality.

BUG=chromium:642493
R=machenbach@chromium.org,iannucci@chromium.org

Change-Id: Id3fe71b07b694339f0a620b427816e52560069d8
Reviewed-on: https://chromium-review.googlesource.com/416430Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

cd6a9363

06 Dec, 2016 1 commit

git cl replacing gnumbd: set correct committer timestamp. · a6695810

Andrii Shyshkalov authored Dec 06, 2016

BUG=642493
R=sergiyb@chromium.org

Change-Id: I5d8588f2b6f30368188aecab2d2c462a11aa6883
Reviewed-on: https://chromium-review.googlesource.com/414369
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

a6695810

05 Dec, 2016 3 commits

recipes: add ndk_next gclient config. · 4f140d7e

John Budorick authored Dec 01, 2016

This adds an ndk_next gclient configuration to allow bots to use the
upcoming version of the Android NDK.

BUG=668202

Change-Id: I403e398094f4215cdb3783b68f465323264ca1ee
Reviewed-on: https://chromium-review.googlesource.com/414184
Commit-Queue: John Budorick <jbudorick@chromium.org>
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Mike Stipicevic <stip@chromium.org>

4f140d7e

Add authors check to canned presubmit checks · c850b961

Michael Achenbach authored Dec 05, 2016

BUG=v8:5603

Change-Id: Ib7bbdfae070ae4c55f99041befbade942d1d3f9c
Reviewed-on: https://chromium-review.googlesource.com/416859
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

c850b961

git cl: implement git number footer generation. · 15e50cc5

Andrii Shyshkalov authored Dec 02, 2016

BUG=642493

Change-Id: Ic8eb121b0ad7adcc7a3f3f1967ef2261f415e731
Reviewed-on: https://chromium-review.googlesource.com/414466Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

15e50cc5

02 Dec, 2016 2 commits

Disable problematic test until we can get git pinned. · e992b613

Mike Stipicevic authored Dec 02, 2016

BUG=670678

Change-Id: I70f39618eea7dd6428d18e5ba60b539cdd57889c
Reviewed-on: https://chromium-review.googlesource.com/416357
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

e992b613

Gerrit git cl desc: don't notify reviwers. · ea4fc839

Andrii Shyshkalov authored Dec 01, 2016

R=agable@chromium.org
BUG=644101
TEST=manual

Change-Id: I82b7209981a834c8f26d65d2701d665a4eb2ba51
Reviewed-on: https://chromium-review.googlesource.com/415829Reviewed-by: Aaron Gable <agable@chromium.org>

ea4fc839

01 Dec, 2016 2 commits

WebRTC: Make got_revision be a WebRTC revision for bots in chromium.webrtc.fyi · b6849ac4

Henrik Kjellander authored Dec 01, 2016

This is needed in order to be able to use the set_component_rev feature in
https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/chromium_tests/api.py?rcl=0&l=169
which is needed for having the testers to sync the same WebRTC revision
as the builders.

This is a follow-up for https://chromium-review.googlesource.com/c/414110

BUG=chromium:669818
TBR=tandrii@chromium.org

Change-Id: I6ca77e739513009e503a3b2a6fedacde8730e9a4
Reviewed-on: https://chromium-review.googlesource.com/415788Reviewed-by: Henrik Kjellander <kjellander-cc@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Henrik Kjellander <kjellander-cc@chromium.org>

b6849ac4

Make Tester bots in chromium.webrtc.fyi sync the right revisions. · 5f5b83bb

Henrik Kjellander authored Nov 30, 2016

BUG=chromium:669818
TBR=tandrii@chromium.org

Change-Id: Icbefd12bd891c8fdb7b8278083f8b1accb53500c
Reviewed-on: https://chromium-review.googlesource.com/414110
Commit-Queue: Henrik Kjellander <kjellander-cc@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>

5f5b83bb

29 Nov, 2016 3 commits

Fix bug in is_transient check of rietveld rpc errors. · 1bf69a12

Andrii Shyshkalov authored Nov 29, 2016

R=katthomas@chromium.org,sergiyb@chromium.org
BUG=chromium:601260

Change-Id: I79e1e807d1f8aed8d294742974d1f42a852c6b13
Reviewed-on: https://chromium-review.googlesource.com/415147
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>

1bf69a12

Only add master. prefix unless one is already specified · 935b93fb

Sergiy Byelozyorov authored Nov 28, 2016

BUG=668162

Change-Id: Ie38542eb4627aa8c07adf9d2665b80cdcee2758f
Reviewed-on: https://chromium-review.googlesource.com/414355
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

935b93fb

Use 'change' instead of 'issue' when talking about Gerrit CLs · a45ee11d

Aaron Gable authored Nov 22, 2016

Change-Id: I17884d9ef7efdaf75ed575e66aca1034db0672b5
Reviewed-on: https://chromium-review.googlesource.com/413289Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>

a45ee11d

28 Nov, 2016 2 commits

Retry up to three times on upload timeout. · a719d3dc

Mike Stipicevic authored Nov 28, 2016

BUG=668526

Change-Id: Id49f1c1394a72880410eb06ba7356307fff7e8c5
Reviewed-on: https://chromium-review.googlesource.com/414496
Commit-Queue: Mike Stipicevic <stip@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>

a719d3dc

Bump timeout to upload files to Rietveld · 1f14230d

Sergiy Byelozyorov authored Nov 28, 2016

BUG=668526

Change-Id: I8aa762e8836ddc4838baaca803de17f3ffcb1ad2
Reviewed-on: https://chromium-review.googlesource.com/414908Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

1f14230d

24 Nov, 2016 4 commits

Use codereview.settings style for GENERATE_GIT_NUMBER_FOOTERS. · b8c535f6

Andrii Shyshkalov authored Nov 24, 2016

R=sergiyb@chromium.org
BUG=642493

Change-Id: Ic4acf4076fdf27c1ed5e75f0ed1dda4b16d675fc
Reviewed-on: https://chromium-review.googlesource.com/414469Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

b8c535f6

git cl: rework setting to auto-generate git number footers. · 5fb47748

Andrii Shyshkalov authored Nov 24, 2016

BUG=642493

Change-Id: I1c83c54f7ad938cf4abeadb42f8d9414521d6dc5
Reviewed-on: https://chromium-review.googlesource.com/414465Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

5fb47748

git cl land + gnumbd: add asserts. · 813ec3c3

Andrii Shyshkalov authored Nov 24, 2016

BUG=642493

Change-Id: Ib1234922e4bcd4ee286fffb8f7c637e4d2442a42
Reviewed-on: https://chromium-review.googlesource.com/414504
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>

813ec3c3

git cl land: add test expectations for Rietveld. · 06a2502f

Andrii Shyshkalov authored Nov 24, 2016

This is preperatory work for Cr- footers generation in git cl instead
of gnumbd.

BUG=642493

Change-Id: I4cfdd882fe6caa7972e51ffa81d335104ddb56dd
Reviewed-on: https://chromium-review.googlesource.com/414464
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>

06a2502f

22 Nov, 2016 1 commit

Remove SVN support from PRESUBMIT · 0b65e732

agable authored Nov 22, 2016

R=maruel@chromium.org
BUG=475320

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

0b65e732

21 Nov, 2016 1 commit

Allow overriding via env var path to .gitcookies in gerrit_util.py · 0bfa9ad7

Ravi Mistry authored Nov 21, 2016

This is to make it easy to specify a different .gitcookies for bots.

BUG=skia:5979

Change-Id: Iaeb148d25f1c2c3d06ab5e1b155999566ef5f294
Reviewed-on: https://chromium-review.googlesource.com/413075Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Ravi Mistry <rmistry@google.com>

0bfa9ad7

19 Nov, 2016 3 commits

Roll recipe dependencies (trivial). · 850ce84c

recipe-roller authored Nov 19, 2016

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)

recipe_engine:
  https://crrev.com/a239f1551fb3ba35d72b9ad4d0cf80ee08c29b22 Vendor virtualenv, update to 15.1.0. (dnj@chromium.org)

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

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2512373002

850ce84c

tryserver module: fix in set_failure_hash · 174c2933

Nodir Turakulov authored Nov 19, 2016

set_failure_hash assumes that (StepFailure is raised) => (some steps
were ran), which is not a correct implication.
Check that there was a step before.

R=iannucci@chromium.org
BUG=

Change-Id: I36ad187584cb42696676e3339f6a77fc77dfd3b1
Reviewed-on: https://chromium-review.googlesource.com/412994
Commit-Queue: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

174c2933

Roll recipe dependencies (trivial). · 1d196c25

recipe-roller authored Nov 19, 2016

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)

recipe_engine:
  https://crrev.com/a1c2338808c1a2a02d520f4078d21f766b43df5b Add name, package_repo_resource and resource support to recipe scripts. (iannucci@chromium.org)

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

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2515913003

1d196c25

18 Nov, 2016 2 commits

Roll recipe dependencies (trivial). · 2527f74b

recipe-roller authored Nov 18, 2016

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)

recipe_engine:
  https://crrev.com/271a506617d4d308d2bde55d20744198343a5bf4 path module: documents std paths (nodir@chromium.org)

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

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2513963003

2527f74b

infra_paths: do nothing if path_config is not specified · 7da798be

Nodir Turakulov authored Nov 18, 2016

Currently if path_config is not specified, infra_paths
defaults to buildbot paths. We want to make this behavior explicit,
so we don't have to introduce a new "luci" path configuration which
tells infra_paths module to do nothing.

Buildbot now specifies path_config=buildbot since
https://chromium-review.googlesource.com/c/412029/ ,
so now we can remove this default.

This also affects all expectation files because this CL switches 
the default path configuration from Buildbot to LUCI.
In particular, it changes default git cache dir from [GIT_CACHE]
to [CACHE]/git.
Also code paths that work only in Buildbot environment has to 
have a test with path_config=buildbot.

R=iannucci@chromium.org, martiniss@chromium.org
BUG=662586

Change-Id: Ifeabe13e69810dd24a6deb4830699f8c9fafafae
Reviewed-on: https://chromium-review.googlesource.com/411989
Commit-Queue: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

7da798be

17 Nov, 2016 4 commits

deprecate git_cache path · e672f939

Nodir Turakulov authored Nov 17, 2016

We cannot easily move git cache location on all buildbot bots, but we
need to define a new value for LUCI.
Introduce infra_paths.default_git_cache_dir which must be used instead
of using path['git_cache'] directly.

R=iannucci@chromium.org, martiniss@chromium.org
BUG=662586

Change-Id: I5244d7a761da94858df798abbd423c85f55936af
Reviewed-on: https://chromium-review.googlesource.com/411993
Commit-Queue: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

e672f939

deprecate build path · ff416a73

Nodir Turakulov authored Nov 17, 2016

Update recipe modules that use path "build" to either check whether it
is defined or fail with an explanation if it is not defined.

This CL marks failure code paths as `# pragma: no cover`. This comment
will be removed in https://chromium-review.googlesource.com/c/411989/

R=iannucci@chromium.org, martiniss@chromium.org
BUG=662586

Change-Id: Ieb7637deefa6e366dfe4c30c7711d60daa06575a
Reviewed-on: https://chromium-review.googlesource.com/412225
Commit-Queue: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

ff416a73

Remove myself (dpranke@) from the WATCHLISTS. · c68d4d36

Dirk Pranke authored Nov 17, 2016

R=iannucci@chromium.org
BUG=

Change-Id: I118714ecb443f10aa69f9be3f2b7726d83b6884a
Reviewed-on: https://chromium-review.googlesource.com/412270Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@chromium.org>

c68d4d36

gclient module: remove revert function · 415cf53d

Nodir Turakulov authored Nov 17, 2016

Nothing uses it

R=iannucci@chromium.org
BUG=

Change-Id: I33932335e019d61f8e174e12c2b647ebfe3d6903
Reviewed-on: https://chromium-review.googlesource.com/412265Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Nodir Turakulov <nodir@chromium.org>

415cf53d