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

Issue 835262 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug-Regression



Sign in to add a comment

build CQ blocked on logdog_bootstrap_test

Project Member Reported by machenb...@chromium.org, Apr 20 2018

Issue description

Failing most of the time for unrelated CLs. E.g.:
https://logs.chromium.org/v/?s=infra%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8948755136069636256%2F%2B%2Fsteps%2Fpresubmit%2F0%2Fstdout

But there are very few passes, so maybe it's flaky.
 
People seem to be using manual retries until there's a pass, e.g. like here 3 times:
https://chromium-review.googlesource.com/c/chromium/tools/build/+/1020780
Skipping the test temporarily:
https://crrev.com/c/1021814
Components: Infra>Platform>LogDog
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build/+/4eb4663152f74ce86eba6d2d9953f28903625f81

commit 4eb4663152f74ce86eba6d2d9953f28903625f81
Author: Michael Achenbach <machenbach@chromium.org>
Date: Fri Apr 20 12:42:55 2018

Skip flaky logdog unittest

Bug:  835262 
Change-Id: I5d7bd56ed9fed94f2dd85a0f8444278462ca0c9b
Reviewed-on: https://chromium-review.googlesource.com/1021814
Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Michael Achenbach <machenbach@chromium.org>

[modify] https://crrev.com/4eb4663152f74ce86eba6d2d9953f28903625f81/scripts/slave/unittests/logdog_bootstrap_test.py

Labels: -Pri-0 Pri-1
Lower prio for fixing the test (which has been disabled). I couldn't repro this locally though. Maybe specific to the bot? Or to running on LUCI?
Labels: -Infra-Troopers Foundation-Troopers
Project Member

Comment 7 by bugdroid1@chromium.org, Apr 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build/+/6929050a8fc305177c7ed30a73d8f75e596e07a4

commit 6929050a8fc305177c7ed30a73d8f75e596e07a4
Author: Vadim Shtayura <vadimsh@chromium.org>
Date: Fri Apr 20 20:49:09 2018

Fix and enable logdog_bootstrap 'test_load' test.

Logdog bootstrap configs reference master that has been deleted. Not sure how
the deletion CL went through.

R=hinoka@chromium.org, nodir@chromium.org
BUG= 835262 

Change-Id: Ia059df08cd50152d9e53922433a09523273c9c1e
Reviewed-on: https://chromium-review.googlesource.com/1022522
Reviewed-by: Ryan Tseng <hinoka@chromium.org>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>

[modify] https://crrev.com/6929050a8fc305177c7ed30a73d8f75e596e07a4/scripts/slave/logdog-params.pyl
[modify] https://crrev.com/6929050a8fc305177c7ed30a73d8f75e596e07a4/scripts/slave/unittests/logdog_bootstrap_test.py

Owner: vadimsh@chromium.org
Status: Fixed (was: Untriaged)
This is failing for me too on https://chromium-review.googlesource.com/c/chromium/tools/build/+/1020041 ; from what I can tell it looks unrelated ot my CL.

(Can this bug be made public please?)
Cc: martiniss@chromium.org
Labels: -Restrict-View-Google
Stephen, this is probably due to deletion of chromium.gpu master in https://chromium.googlesource.com/chromium/tools/build/+/24d49765e4a7811543b27a47fc108a4abf29f412.

Please remove it also from logdog-params.pyl, as in #7
Owner: martiniss@chromium.org
Status: Assigned (was: Fixed)
This is breaking all cq jobs in the build/ repo: https://ci.chromium.org/p/infra/builders/luci.infra.try/Build%20Presubmit?limit=200

https://chromium-review.googlesource.com/c/chromium/tools/build/+/1081407 landed through the cq -- how can that be if it breaks the cq?
Cc: vadimsh@chromium.org
Cc: iannucci@chromium.org no...@chromium.org hinoka@chromium.org
 Issue 849150  has been merged into this issue.
Project Member

Comment 15 by bugdroid1@chromium.org, Jun 4 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build/+/e63b5c0428d5cdcdbf385cbace47bdec05ed2270

commit e63b5c0428d5cdcdbf385cbace47bdec05ed2270
Author: Ryan Tseng <hinoka@google.com>
Date: Mon Jun 04 20:39:25 2018

[logdog-params.pyl] Remove master.chromium.gpu

To fix the CQ

Bug:  835262 
Change-Id: I4a2073251e887f27f3d942b85387e3f0b026018e
Reviewed-on: https://chromium-review.googlesource.com/1085895
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Stephen Martinis <martiniss@chromium.org>

[modify] https://crrev.com/e63b5c0428d5cdcdbf385cbace47bdec05ed2270/scripts/slave/logdog-params.pyl

Status: Fixed (was: Assigned)
I'm assuming this is fixed.

Sign in to add a comment