Issue metadata
Sign in to add a comment
|
__main__.InstallerTest.ChromeUserLevel is flaky |
||||||||||||||||||||||||
Issue description__main__.InstallerTest.ChromeUserLevel is flaky. Findit has detected 3 flake occurrences of this test within the past 24 hours. List of all flake occurrences can be found at: https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyTwsSBUZsYWtlIkRjaHJvbWl1bUBtaW5pX2luc3RhbGxlcl90ZXN0c0BfX21haW5fXy5JbnN0YWxsZXJUZXN0LkNocm9tZVVzZXJMZXZlbAw. Unless the culprit CL is found and reverted, please disable this test first within 30 minutes then find an appropriate owner. If the result above is wrong, please file a bug using this link: https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Detection%20-%20Wrong%20result%3A%20__main__.InstallerTest.ChromeUserLevel&comment=Link%20to%20flake%20details%3A%20https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyTwsSBUZsYWtlIkRjaHJvbWl1bUBtaW5pX2luc3RhbGxlcl90ZXN0c0BfX21haW5fXy5JbnN0YWxsZXJUZXN0LkNocm9tZVVzZXJMZXZlbAw Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
,
Dec 19
https://chromium-review.googlesource.com/c/chromium/src/+/1384004 may help. One crash I looked at is issue 916461. There may be others.
,
Dec 19
https://chromium-review.googlesource.com/c/chromium/src/+/1384025 removes the DCHECK described in issue 916461.
,
Dec 19
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d5cf001c4db2cc46df12665af782e67df0db395b commit d5cf001c4db2cc46df12665af782e67df0db395b Author: Greg Thompson <grt@chromium.org> Date: Wed Dec 19 20:05:39 2018 Include Chrome logs in the test output when crash reports are found. It's possible that the logs will include information, such as a log message or stack trace, from a failed [D]CHECK. This CL emits the Chrome logs in the test output and copies them into the output directory in case a crash report is found. BUG=907979, 916389 Change-Id: I5817d1f561259ee70e2b341be0d0fa7d92c9072d Reviewed-on: https://chromium-review.googlesource.com/c/1384004 Reviewed-by: Roger Tawa <rogerta@chromium.org> Commit-Queue: Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#617920} [modify] https://crrev.com/d5cf001c4db2cc46df12665af782e67df0db395b/chrome/test/mini_installer/test_chrome_with_chromedriver.py
,
Dec 20
__main__.InstallerTest.ChromeUserLevel is flaky. Findit has detected 4 new flake occurrences of this test. List of all flake occurrences can be found at: https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyTwsSBUZsYWtlIkRjaHJvbWl1bUBtaW5pX2luc3RhbGxlcl90ZXN0c0BfX21haW5fXy5JbnN0YWxsZXJUZXN0LkNocm9tZVVzZXJMZXZlbAw. Since these tests are still flaky, this issue has been moved back onto the Sheriff Bug Queue if it hasn't already. If the result above is wrong, please file a bug using this link: https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Detection%20-%20Wrong%20result%3A%20__main__.InstallerTest.ChromeUserLevel&comment=Link%20to%20flake%20details%3A%20https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyTwsSBUZsYWtlIkRjaHJvbWl1bUBtaW5pX2luc3RhbGxlcl90ZXN0c0BfX21haW5fXy5JbnN0YWxsZXJUZXN0LkNocm9tZVVzZXJMZXZlbAw Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
,
Dec 20
[Sheriff] Removing label Sheriff-Chromium as grt@ is investigating.
,
Dec 21
__main__.InstallerTest.ChromeUserLevel is flaky. Findit has detected 4 new flake occurrences of this test. List of all flake occurrences can be found at: https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyTwsSBUZsYWtlIkRjaHJvbWl1bUBtaW5pX2luc3RhbGxlcl90ZXN0c0BfX21haW5fXy5JbnN0YWxsZXJUZXN0LkNocm9tZVVzZXJMZXZlbAw. Since these tests are still flaky, this issue has been moved back onto the Sheriff Bug Queue if it hasn't already. If the result above is wrong, please file a bug using this link: https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Detection%20-%20Wrong%20result%3A%20__main__.InstallerTest.ChromeUserLevel&comment=Link%20to%20flake%20details%3A%20https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyTwsSBUZsYWtlIkRjaHJvbWl1bUBtaW5pX2luc3RhbGxlcl90ZXN0c0BfX21haW5fXy5JbnN0YWxsZXJUZXN0LkNocm9tZVVzZXJMZXZlbAw Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
,
Dec 21
,
Dec 21
Issue 917051 has been merged into this issue.
,
Dec 21
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a8f33aee59bc7840fedcd7e169acb487ed75da7b commit a8f33aee59bc7840fedcd7e169acb487ed75da7b Author: Greg Thompson <grt@chromium.org> Date: Fri Dec 21 15:53:26 2018 Remove DCHECK on CreateTrialsFromSwitchValue failures. This DCHECK is hit in utility processes during test_installer runs. The attempt to duplicate the parent proc's handle in DeserializeSharedMemoryHandleMetadata fails because the browser process is being torn down. BUG= 916389 ,916461 R=asvitkine@chromium.org Change-Id: I27c03717c0272bdf309f78e0a1724d7d0f205c57 Reviewed-on: https://chromium-review.googlesource.com/c/1384025 Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Reviewed-by: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#618539} [modify] https://crrev.com/a8f33aee59bc7840fedcd7e169acb487ed75da7b/base/metrics/field_trial.cc
,
Jan 7
Flakes have stopped for now. Issue 917051 tracks making the test harness retry so that flakes have less impact on cycle time. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by ljusten@chromium.org
, Dec 19Components: Internals>Installer
Labels: -Pri-1 -Sheriff-Chromium OS-Windows Pri-2
Owner: grt@chromium.org
Status: Assigned (was: Untriaged)