CL +1s from mthiesse@chromium.org not being recognized by presubmit |
|||||
Issue descriptionRecently presubmit has stopped recognizing my +1s on CLs, and reviewers have to TBR me after I +1 to submit their CLs. Examples: https://chromium-review.googlesource.com/c/chromium/src/+/1409817 https://chromium-review.googlesource.com/c/chromium/src/+/1401279
,
Jan 16
(6 days ago)
Is this happening consistently? Or flakily? Sample log from chromium-presubmit: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8924251487815163072/+/steps/presubmit/0/stdout ** Presubmit ERRORS ** Missing LGTM from an OWNER for these files: chrome/android/java/src/org/chromium/chrome/browser/vr/ArCoreJavaUtils.java chrome/android/java/src/org/chromium/chrome/browser/vr/SilenceLintErrors.java chrome/android/java/src/org/chromium/chrome/browser/vr/VrModuleProvider.java The bot ran at 2019-01-15 7:53 AM (PST), the +1 was added at 7:48am (5 min prior). The OWNERS file seem to be correct (especially if it worked earlier). Either the presubmit script changed somehow and doesn't understand the file inclusions in OWNERS files, or Gerrit is not reporting the +1 in time. I'll try to triage this someplace more specific a bit later.
,
Jan 16
(6 days ago)
Appears to be consistent.
,
Jan 17
(5 days ago)
Andrii - Given the theory of "Gerrit is not reporting the +1 in time" could this be a symptom of http://b/122924821 ?
,
Jan 17
(5 days ago)
I doubt it, though it's possible. In many tries, only my +1 didn't count - the other reviewer's +1s did count, and it hasn't been flaky.
,
Jan 17
(5 days ago)
This seems likely to either be a gerrit issue or an issue w/ presubmit support provided by depot_tools; updating components accordingly.
,
Jan 17
(5 days ago)
"Gerrit is not reporting the +1 in time" is primarily to do with +1 vote not making it to Gerrit index immediately upon backends accepting an update. presubmit tooling in depot_tools afair fetches data about gerrit CL w/o using Gerrit index, but asking the source of truth -- gerrit backend. OTH, gerrit backend actually has replicas, and if one is really stale, then yeah, this can be the case (see also http://b/122858799) [removing myself because nothing to do with CQ]
,
Jan 17
(5 days ago)
Seems like my +1s are working again, will re-open if they start failing. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by mthiesse@chromium.org
, Jan 16 (6 days ago)