New issue
Advanced search Search tips

Issue 830968 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Oct 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

CQ-DEPEND cannot handle 7 digit CL number on Gerrit

Project Member Reported by mqg@chromium.org, Apr 9 2018

Issue description

What steps will reproduce the problem?
(1)add CQ-DEPEND to CL commit message, depend on a CL whose CL number is 7 digit (very recent)
(2)
(3)

What is the expected result?
On Gerrit, we should be able to click on the CQ-DEPEND=CL:1234567 and go to that CL.

What happens instead?
The last digit is dropped. We are taken to CL:123456, which is wrong CL / does not exist. 

Not sure how this is going to affect cq, still observing. 
 
Status: Fixed (was: Untriaged)
I cannot reproduce. 

I just went to https://crrev.com/c/1287949 which has a 7 digit CL depend line "CL:1286286" clicking it takes me to the correct CL. 

This issue predates me being a Google employee or being on Chrome so I have no idea when this was fixed. 

Sign in to add a comment