Issue 635157 link

Starred by 2 users

Issue metadata

Status:	Duplicate
Merged:	issue 602906
Owner:	----
Closed:	Aug 2016
Cc:	tandrii@chromium.org █ andyb...@chromium.org
Components:	Infra>CQ
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	1
Type:	Bug

Gerrit based CQ in infra.git doesn't handle DEPS rolls correctly

Project Member Reported by vadimsh@chromium.org, Aug 5 2016

Issue description

Two identical CLs. Rietveld one landed, Gerrit one didn't:

https://chromium-review.googlesource.com/c/366846
https://codereview.chromium.org/2219193002/

Looks like tryjobs for Gerrit CL didn't apply DEPS change before running the tests.

Comment 1 by no...@chromium.org, Aug 9 2016

Cc: andyb...@chromium.org tandrii@chromium.org
Labels: -Pri-2 Pri-1

this is a breaking change since switching from rietveld

Comment 2 by no...@chromium.org, Aug 9 2016

another example
gerrit: https://chromium-review.googlesource.com/c/367327/
rietveld: https://codereview.chromium.org/2225073005

Comment 3 by tandrii@chromium.org, Aug 9 2016

Mergedinto: 602906
Status: Duplicate (was: Unconfirmed)

Thanks for reporting! It's KI in bot_update. Andy is on it this week.

►

Issue 635157 link

Issue metadata

Gerrit based CQ in infra.git doesn't handle DEPS rolls correctly

Issue description

Comment 1 by no...@chromium.org, Aug 9 2016

Comment 1 Deleted

Comment 2 by no...@chromium.org, Aug 9 2016

Comment 2 Deleted

Comment 3 by tandrii@chromium.org, Aug 9 2016

Comment 3 Deleted

Sign in to add a comment