New issue
Advanced search Search tips

Issue 612727 link

Starred by 0 users

Issue metadata

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



Sign in to add a comment

Gerrit git cl land should prompt about CQ

Project Member Reported by tandrii@chromium.org, May 18 2016

Issue description

if CQ label exists, git cl land should warn user whether bypassing CQ is intended.
 
Labels: Proj-Gerrit-Migration
Components: Infra>SDK
Labels: Pri-2
Status: Started (was: Untriaged)
Project Member

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

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/depot_tools.git/+/d60367ba30630ce4091baed2a6dae4bd55e15b5e

commit d60367ba30630ce4091baed2a6dae4bd55e15b5e
Author: tandrii <tandrii@chromium.org>
Date: Wed Jun 22 12:25:12 2016

Gerrit git cl land: add warning if Commit-Queue label exists.

BUG= 612727 
R=agable@chromium.org

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

[modify] https://crrev.com/d60367ba30630ce4091baed2a6dae4bd55e15b5e/git_cl.py

Project Member

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

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

commit ae486f237664c8d77f6fc11f8eda1e584c32257e
Author: recipe-roller <recipe-roller@chromium.org>
Date: Wed Jun 22 13:15:38 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/0f55d92ba93ab505710f15a8f6b764424d049328 Revert a host's persistent device list from recipes. (bpastene@chromium.org)
  https://crrev.com/ca3063aa341baf1d0faf6b061020ac8f785b14fa Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/cd5a58d28019ecbe9f984b9ea740bec02abeb0e7 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/bb1e9ceb37f542321b8b81e248873beee08994f5 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/634d5061a13ea05eb5cc45156fad00093507ef7a Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/e32e3eaebdd32dbfbc821e3416fc1a2afe78eb47 Add warning for non-https remotes in Gerrit git cl upload. (tandrii@chromium.org)
  https://crrev.com/9d2c7a3d9ab7c0694b1cbd6ffc519618b92330ed Rietveld git cl status: ignore CQ dry run and generated messages. (tandrii@chromium.org)
  https://crrev.com/27ed22e66e91b8cdb193afca7d5dd91306dc7c7e Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/d60367ba30630ce4091baed2a6dae4bd55e15b5e Gerrit git cl land: add warning if Commit-Queue label exists. (tandrii@chromium.org)
recipe_engine:
  https://crrev.com/2514f69431aba26d302fbc38b344a36864a94d9b Add 'remote' command to replace 'remote_run' (phajdan.jr@chromium.org)
  https://crrev.com/9540897c91022bfca86cac689bd52823f61b357f Add support for fetching repo updates from Gitiles (phajdan.jr@chromium.org)

R=bpastene@chromium.org,tandrii@chromium.org,shinyak@chromium.org,recipe-roller@chromium.org,martiniss@chromium.org,phajdan.jr@chromium.org,agable@chromium.org,sergiyb@chromium.org
BUG= 612727 ,614312,619707, chromium:459840 

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

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

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

Status: Fixed (was: Started)
Project Member

Comment 7 by bugdroid1@chromium.org, Jun 22 2016

Sign in to add a comment