New issue
Advanced search Search tips

Issue 687007 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Gerrit dependency prevents landing even if future parent patch has landed.

Project Member Reported by d...@chromium.org, Jan 31 2017

Issue description

B dependson A@PS1
A@PS2 lands
CQ will now refuse to land B.

See: https://chromium-review.googlesource.com/c/434945/
 
Owner: tandrii@chromium.org
Status: WontFix (was: Untriaged)
Yeah, I've been working on this since November, even proposed a workaround in  issue 673835 . However, the actual bug in Gerrit has been fixed last week, so this should start working next week.   

Glad you to see another pipeline user - check this pipeline of mine https://chrome-internal-review.googlesource.com/c/322305/ :D
btw, short term fix is clicking "rebase" to do rebase manually.

Comment 3 by d...@chromium.org, Jan 31 2017

Status: Available (was: WontFix)
Hah okay cool. Mind if we keep this open until it's confirmed that it's actually fixed in Gerrit?

Nice pipeline :x
Status: Started (was: Available)
It seems it still doesn't work, at last for me.
Project Member

Comment 5 by bugdroid1@chromium.org, Feb 8 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal/+/33766d8043ea15f680c5f0fec1e63081d04ccfc0

commit 33766d8043ea15f680c5f0fec1e63081d04ccfc0
Author: Andrii Shyshkalov <tandrii@chromium.org>
Date: Wed Feb 08 21:16:33 2017

Project Member

Comment 6 by bugdroid1@chromium.org, Feb 8 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal/+/33766d8043ea15f680c5f0fec1e63081d04ccfc0

commit 33766d8043ea15f680c5f0fec1e63081d04ccfc0
Author: Andrii Shyshkalov <tandrii@chromium.org>
Date: Wed Feb 08 21:16:33 2017

Status: ExternalDependency (was: Started)
SO, fix was deployed: https://cl-status.corp.google.com/#/summary/gerritcodereview.gerrit.frontend?status=145983420 

However, I still got error, a different on this time: http://b/35150070 
Project Member

Comment 8 by bugdroid1@chromium.org, Feb 8 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal/+/c8c2af9a21d1e1f8cdf1820e244995943c417fef

commit c8c2af9a21d1e1f8cdf1820e244995943c417fef
Author: Andrii Shyshkalov <tandrii@chromium.org>
Date: Wed Feb 08 21:36:41 2017

Status: Verified (was: ExternalDependency)
 http://b/35150070  was a Gerrit flake. THis actually works now.

Sign in to add a comment