New issue
Advanced search Search tips

Issue 921903 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 15
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 0
Type: Bug



Sign in to add a comment

All pre-CQ runs failing: CIDB Connection could not be initialized

Project Member Reported by nya@chromium.org, Jan 15

Issue description

Headed to bed but potentially https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1405769 is related. If so, try chumping it.  Will look in the morning.
Cc: jclinton@chromium.org
Owner: nya@chromium.org
Status: Started (was: Assigned)
Thanks Jason, the submission time of that CL matches with the start of failures. I will try reverting the change.

Looks like it is my CL. Will investigate tomorrow. nya@, please chump the revert.
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 15

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/5587a0d79ff054811137802e2e8363d6b166bfb5

commit 5587a0d79ff054811137802e2e8363d6b166bfb5
Author: Shuhei Takahashi <nya@chromium.org>
Date: Tue Jan 15 06:03:10 2019

Revert "BuildStore: Redirect GetBuildStatus through BuildStore"

This reverts commit 15ee5f53c6a9575492b79e353a5a9f5ae817342c.

Reason for revert: Pre-CQ failing:  crbug.com/921903 

Original change's description:
> BuildStore: Redirect GetBuildStatus through BuildStore
> 
> Redirects CIDB GetBuildStatus(), GetBuildStatuses(),
> GetBuildStatusesWithBuildbucketIds() calls in
> cbuildbot/stages/*_stages.py through
> BuildStore.GetBuildStatuses(). Separate CLs will follow
> targeting other calls.
> 
> BUG=chromium:920430
> TEST=tryjob
> 
> Change-Id: I35a51a0bc0713610303bdefe1f6648f217ecd5ea
> Reviewed-on: https://chromium-review.googlesource.com/1405769
> Commit-Ready: Jason Clinton <jclinton@chromium.org>
> Tested-by: Jason Clinton <jclinton@chromium.org>
> Reviewed-by: Jason Clinton <jclinton@chromium.org>

Bug=chromium:920430
Bug= chromium:921903 

Change-Id: I83afc086b32622b2dcf992755fa1713b59b64e12
Reviewed-on: https://chromium-review.googlesource.com/c/1409629
Tested-by: Shuhei Takahashi <nya@chromium.org>
Trybot-Ready: Shuhei Takahashi <nya@chromium.org>
Reviewed-by: Dhanya Ganesh <dhanyaganesh@chromium.org>

[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/lib/buildstore.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/completion_stages.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/sync_stages.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/completion_stages_unittest.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/lib/fake_cidb.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/handle_changes_stages.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/sync_stages_unittest.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/build_stages.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/handle_changes_stages_unittest.py
[modify] https://crrev.com/5587a0d79ff054811137802e2e8363d6b166bfb5/cbuildbot/stages/report_stages.py

I see this line in the logs:
" If the buildbucket_id to insert is duplicated to the buildbucket_id of an old build and the old build was canceled because of a waterfall master restart, please ignore this error. Else, the error needs more investigation. More context:  crbug.com/679974  and  crbug.com/685889 
"

If the issue isn't fixed after the revert, the PreCQ Launcher needs to be restarted.
Status: Fixed (was: Started)
We saw at least two successes, looks good.

Sign in to add a comment