Make rebase and sync the default for Gerrit updates

Make rebase and sync the default for Gerrit updates.

BUG=602906
R=tandrii@chromium.org

Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/6911b067504584996204cf0b2535508ac4207106

[Bons, 2016-08-31 18:29:43]

andybons@chromium.org changed reviewers:
+ agable@chromium.org, tandrii@chromium.org

[Bons, 2016-08-31 18:29:52]

The CQ bit was checked by andybons@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-31 18:29:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tandrii(chromium), 2016-08-31 18:31:18]

LGTM but this will cause non-trivial rolls, so be ready.

https://codereview.chromium.org/2298883002/diff/1/recipe_modules/bot_update/e...
File recipe_modules/bot_update/example.py (right):

https://codereview.chromium.org/2298883002/diff/1/recipe_modules/bot_update/e...
recipe_modules/bot_update/example.py:37: gerrit_no_rebase_patch_ref =
bool(api.properties.get('gerrit_no_rebase_patch_ref'))
nit: 80 chars

[commit-bot: I haz the power, 2016-08-31 18:36:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-31 18:36:12]

Dry run: Try jobs failed on following builders:
  Depot Tools Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/30faaf176ab26010)

[Bons, 2016-08-31 18:47:55]

Added support for the old flag as well for minimal disruption.

https://codereview.chromium.org/2298883002/diff/1/recipe_modules/bot_update/e...
File recipe_modules/bot_update/example.py (right):

https://codereview.chromium.org/2298883002/diff/1/recipe_modules/bot_update/e...
recipe_modules/bot_update/example.py:37: gerrit_no_rebase_patch_ref =
bool(api.properties.get('gerrit_no_rebase_patch_ref'))
On 2016/08/31 at 18:31:18, tandrii(chromium) wrote:
> nit: 80 chars

Done

[Bons, 2016-08-31 18:48:45]

The CQ bit was checked by andybons@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-31 18:48:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tandrii(chromium), 2016-08-31 18:55:29]

lgtm

[tandrii(chromium), 2016-08-31 18:58:02]

Description was changed from

==========
Make rebase and sync the default for Gerrit updates

BUG=602906
==========

to

==========
Make rebase and sync the default for Gerrit updates.

BUG=602906

Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
==========

[Bons, 2016-08-31 19:00:39]

The CQ bit was unchecked by andybons@chromium.org

[Bons, 2016-08-31 19:00:46]

The CQ bit was checked by andybons@chromium.org

[commit-bot: I haz the power, 2016-08-31 19:00:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tandrii(chromium), 2016-08-31 19:06:12]

Description was changed from

==========
Make rebase and sync the default for Gerrit updates.

BUG=602906

Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
==========

to

==========
Make rebase and sync the default for Gerrit updates.

BUG=602906


Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
==========

[tandrii(chromium), 2016-08-31 19:06:16]

The CQ bit was unchecked by tandrii@chromium.org

[tandrii(chromium), 2016-08-31 19:06:19]

The CQ bit was checked by tandrii@chromium.org

[commit-bot: I haz the power, 2016-08-31 19:06:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tandrii(chromium), 2016-08-31 19:07:26]

Description was changed from

==========
Make rebase and sync the default for Gerrit updates.

BUG=602906


Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
==========

to

==========
Make rebase and sync the default for Gerrit updates.

BUG=602906
R=tandrii@chromium.org

Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
==========

[commit-bot: I haz the power, 2016-08-31 19:10:49]

Description was changed from

==========
Make rebase and sync the default for Gerrit updates.

BUG=602906
R=tandrii@chromium.org

Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
==========

to

==========
Make rebase and sync the default for Gerrit updates.

BUG=602906
R=tandrii@chromium.org

Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility.
Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/6911b067504584...
==========

[commit-bot: I haz the power, 2016-08-31 19:10:50]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/tools/depot_tools/+/6911b067504584...