New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 777483 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner: ----
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

changed CL URL format affects annotationsTAble.blame_url

Project Member Reported by nxia@chromium.org, Oct 23 2017

Issue description

Gerrit recently changed the format of the CL URLs, now some URLs have been too long to fit into the annotationsTAble.blame_url, so some URLS got truncated. I revised some URLs last week.

We can either request deputies to only put short CL URLs like: crosreview/* crosreview/i/* crrev.com/c/559225, or extend the length of blame_url.


mysql> select * from annotationsTable where failure_category = 'bad_cl'and build_id > 1935791;
+------+----------+---------------------+----------------+------------------+-----------------+----------------------------------------------------------------------------------+---------------------------------------------------------------------------------------------------+---------+
| id   | build_id | last_updated        | last_annotator | failure_category | failure_message | blame_url                                                                        | notes                                                                                             | deleted |
+------+----------+---------------------+----------------+------------------+-----------------+----------------------------------------------------------------------------------+---------------------------------------------------------------------------------------------------+---------+
| 6152 |  1937427 | 2017-10-13 02:17:40 | dgarrett       | bad_cl           |                 | https://crrev.com/c/559225                                                       |                                                                                                   |       0 |
| 6148 |  1937971 | 2017-10-12 23:16:50 | dgarrett       | bad_cl           |                 | https://crrev.com/c/559225                                                       |                                                                                                   |       0 |
| 6171 |  1941511 | 2017-10-13 22:14:26 | dgarrett       | bad_cl           |                 | https://crrev.com/c/713463                                                       |                                                                                                   |       0 |
| 6194 |  1943187 | 2017-10-16 18:15:25 | pprabhu        | bad_cl           |                 | https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/716 | Only for moblab failure                                                                           |       0 |
| 6213 |  1951089 | 2017-10-17 07:17:48 | akeshet        | bad_cl           |                 | https://chromium-review.googlesource.com/#/c/chromiumos/platform2/+/721018/      |                                                                                                   |       0 |
| 6214 |  1951471 | 2017-10-17 07:30:35 | akeshet        | bad_cl           |                 | https://chrome-internal-review.googlesource.com/#/c/chromeos/cheets-scripts/+/48 |                                                                                                   |       0 |
| 6215 |  1951471 | 2017-10-17 07:33:50 | akeshet        | bad_cl           |                 | https://chrome-internal-review.googlesource.com/#/c/chromeos/overlays/project-ch |                                                                                                   |       0 |
| 6227 |  1957734 | 2017-10-20 18:17:46 | nxia           | bad_cl           |                 | crosreview.com/724679                                                            | guado_moblab-paladin failed with Unhandled TimeoutError                                           |       0 |
| 6229 |  1958656 | 2017-10-19 20:46:08 | nxia           | bad_cl           |                 | crosreview.com/710324                                                            |                                                                                                   |       0 |
| 6247 |  1961595 | 2017-10-20 18:30:01 | nxia           | bad_cl           |                 | crosreview.com/728382                                                            | moblab_RunSuite: ERROR: Unhandled AttributeError: 'module' object has no attribute 'TimeoutError' |       0 |
| 6249 |  1963468 | 2017-10-20 18:38:57 | nxia           | bad_cl           |                 | crosreview.com/709155                                                            | guado_moblab-paladin: The UnitTest stage failed:                                                  |       0 |
| 6255 |  1964893 | 2017-10-20 21:27:18 | nxia           | bad_cl           |                 | crosreview.com/726188                                                            |                                                                                                   |       0 |
 
Mergedinto: 768113
Status: Duplicate (was: Untriaged)

Sign in to add a comment