ios_chrome_ui_egtests failing on iPAd |
||||||||||||||||
Issue description1ce3853d-d4b7-4bee-859d-0abde2ada4ad Builders failed on: - EarlGreyiOS: https://build.chromium.org/p/chromium.fyi/builders/EarlGreyiOS - ipad10-device-x64: http://master7.golo.chromium.org/i/internal.bling.main/builders/ipad10-device-x64 - ipad10-simulator-x64: http://master7.golo.chromium.org/i/internal.bling.main/builders/ipad10-simulator-x64 - ipad10-simulator: http://master7.golo.chromium.org/i/internal.bling.main/builders/ipad10-simulator - ipad9-device-x64: http://master7.golo.chromium.org/i/internal.bling.main/builders/ipad9-device-x64 - ipad9-simulator-x64: http://master7.golo.chromium.org/i/internal.bling.main/builders/ipad9-simulator-x64 - ipad9-simulator: http://master7.golo.chromium.org/i/internal.bling.main/builders/ipad9-simulator
,
Jul 18 2017
The test that timeout are SigninInteractionControllerTestCase. One example test failing is SigninInteractionControllerTestCase.testSignInOneUser that fails deterministically (fail even if this is the only test run).
Failure is when happens on the following line:
TapButtonWithLabelId(IDS_IOS_ACCOUNT_CONSISTENCY_CONFIRMATION_OK_BUTTON);
Error is:
Exception Name: TimeoutException
Exception Reason: Timed out while waiting to perform action.
Exception with Action: {
"Action Name" : "Tap",
"Element Matcher" : "(((respondsToSelector(isAccessibilityElement) && isAccessibilityElement) && accessibilityLabel('Ok, Got it')) && ((respondsToSelector(isAccessibilityElement) && isAccessibilityElement) && accessibilityTraits: UIAccessibilityTraitButton))"
}
Exception Details: Error Trace: [
{
"Description" : "Failed to execute block because idling resources below are busy.",
"Description Glossary" : {
"GREYAppStateTracker" : "Waiting for CAAnimations to finish. Continuous animations may never finish and must be stopped explicitly. Animations attached to hidden view may still be running in the background.
Waiting for UIView's draw/layout pass to complete. A draw/layout pass normally completes in the next runloop drain.
Assigning to jlebel@ as owner of signin, will disable the tests on iPad.
,
Jul 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1cb215be3baa5952f9538d032d79bda580c7b10a commit 1cb215be3baa5952f9538d032d79bda580c7b10a Author: Sylvain Defresne <sdefresne@chromium.org> Date: Tue Jul 18 17:09:29 2017 Disable SigninInteractionControllerTestCase EG tests. Those tests are consistently timing out on iPad, disabling them on iPad until the root cause is fixed to turn the bots green. Bug: 745798 Change-Id: I5f0dab981cae4fbef9f81d144ffa0492a16cfa12 Reviewed-on: https://chromium-review.googlesource.com/576171 Reviewed-by: Jérôme Lebel <jlebel@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#487506} [modify] https://crrev.com/1cb215be3baa5952f9538d032d79bda580c7b10a/ios/chrome/browser/ui/authentication/signin_interaction_controller_egtest.mm
,
Jul 19 2017
,
Jul 19 2017
It looks like the failures are caused by https://chromium-review.googlesource.com/c/569538/.
,
Jul 19 2017
,
Jul 21 2017
,
Jul 21 2017
The tests have been re-enabled.
,
Jul 21 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/68abc07bf96bcc39581d30741952c9d4b032a4a1 commit 68abc07bf96bcc39581d30741952c9d4b032a4a1 Author: Olivier Robin <olivierrobin@chromium.org> Date: Fri Jul 21 12:06:17 2017 Revert "Disable SigninInteractionControllerTestCase EG tests." This reverts commit 1cb215be3baa5952f9538d032d79bda580c7b10a. Reason for revert: Problem should be fixed. Reenable tests. Original change's description: > Disable SigninInteractionControllerTestCase EG tests. > > Those tests are consistently timing out on iPad, disabling them > on iPad until the root cause is fixed to turn the bots green. > > Bug: 745798 > Change-Id: I5f0dab981cae4fbef9f81d144ffa0492a16cfa12 > Reviewed-on: https://chromium-review.googlesource.com/576171 > Reviewed-by: Jérôme Lebel <jlebel@chromium.org> > Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> > Cr-Commit-Position: refs/heads/master@{#487506} TBR=sdefresne@chromium.org,jlebel@chromium.org Change-Id: I5ea0342aa4b6d0c2b1227e52c4f0c0f272695d63 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 745798 Reviewed-on: https://chromium-review.googlesource.com/577542 Reviewed-by: Olivier Robin <olivierrobin@chromium.org> Reviewed-by: Jérôme Lebel <jlebel@chromium.org> Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#488635} [modify] https://crrev.com/68abc07bf96bcc39581d30741952c9d4b032a4a1/ios/chrome/browser/ui/authentication/signin_interaction_controller_egtest.mm
,
Jul 21 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-61; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-61 label, otherwise remove Merge-TBD label. Thanks.
,
Jul 21 2017
,
Jul 21 2017
This bug requires manual review: Reverts referenced in bugdroid comments after merge request. Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid @(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 24 2017
This does not seem to fail on beta bots so not worth merging into M61. Let me know if I am missing something.
,
Jul 25 2017
martiw@ could you give more details why this should be in M61. cc'ing Rohit since Sylvain is out
,
Jul 25 2017
These tests were disabled on trunk because a different CL broke them. That CL was reverted before the branch was cut, and then the tests were reenabled on trunk after the branch was cut. On M61, the code is working (because M61 includes the revert), but the tests are still disabled. It seems like a good idea to re-enable these tests on the branch. M61 bots will not show any failures because these tests are currently disabled on the branch.
,
Jul 25 2017
This bug requires manual review: Reverts referenced in bugdroid comments after merge request. Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid @(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 26 2017
,
Jul 31 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 4 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 18 2017
|
||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||
Comment 1 by sdefresne@chromium.org
, Jul 18 2017