Issue metadata
Sign in to add a comment
|
desktopui_ScreenLocker failing on betty |
||||||||||||||||||||||
Issue descriptionFiled by sheriff-o-matic@appspot.gserviceaccount.com on behalf of rspangler@google.com /tmp/cbuildbotSb9dnC/smoke/test_harness/all/SimpleTestVerify/1_autotest_tests/results-36-desktopui_ScreenLocker [ FAILED ] /tmp/cbuildbotSb9dnC/smoke/test_harness/all/SimpleTestVerify/1_autotest_tests/results-36-desktopui_ScreenLocker FAIL: Screen not locked after using keyboard shortcut /tmp/cbuildbotSb9dnC/smoke/test_harness/all/SimpleTestVerify/1_autotest_tests/results-36-desktopui_ScreenLocker/desktopui_ScreenLocker [ FAILED ] /tmp/cbuildbotSb9dnC/smoke/test_harness/all/SimpleTestVerify/1_autotest_tests/results-36-desktopui_ScreenLocker/desktopui_ScreenLocker FAIL: Screen not locked after using keyboard shortcut Builders failed on: - betty-arc64-nyc-android-pfq: https://luci-milo.appspot.com/buildbot/chromeos/betty-arc64-nyc-android-pfq/346
,
Dec 6 2017
xiaoyinh@ is currently reworking the desktopui_ScreenLocker test to use views-based lock.
,
Dec 6 2017
,
Dec 6 2017
This is failing pre-CQ runs as well. If a proper fix is not incoming immediately, can we please get a revert to avoid blocking all other developers. For example: https://chromium-review.googlesource.com/c/806877/ https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/pre_cq/71542
,
Dec 6 2017
Reverted my CL which changed the autotest API. https://chromium-review.googlesource.com/c/chromium/src/+/812264
,
Dec 6 2017
Fingers crossed that this is fixed by the revert in comment 6; removing from sheriff queue.
,
Dec 6 2017
Just noting that a number of builds have continued to fail after the revert, but so far, all of them were started BEFORE the revert.
,
Dec 7 2017
This is still failing, including builds that started post revert. https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/pre_cq/71632
,
Dec 7 2017
It's time to remove this test from bvt-cq
,
Dec 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/4db1324ac3990d2ca0393b64346ef2da7f435c64 commit 4db1324ac3990d2ca0393b64346ef2da7f435c64 Author: Aviv Keshet <akeshet@chromium.org> Date: Thu Dec 07 02:57:57 2017 temporarily remove desktopui_ScreenLocker from smoke suite BUG= chromium:792565 TEST=None Change-Id: Ib86ae0cb2a2ead68a25e92c6702e9e96e2b6eb29 Reviewed-on: https://chromium-review.googlesource.com/813202 Tested-by: Aviv Keshet <akeshet@chromium.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org> [modify] https://crrev.com/4db1324ac3990d2ca0393b64346ef2da7f435c64/client/site_tests/desktopui_ScreenLocker/control
,
Dec 7 2017
,
Dec 7 2017
Test removed from smoke, fix at leisure.
,
Dec 7 2017
When this is fixed, it'd be nice to see some investigation of how it started failing on the CQ. Was it broken by a change to something not exercised by the Chrome OS CQ or Chrome PFQ?
,
Sep 21
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by rspangler@chromium.org
, Dec 6 2017Owner: jdufault@chromium.org
Status: Assigned (was: Available)