New issue
Advanced search Search tips

Issue 899919 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug
Flaky-Test: __main__.SessionHandlingTest.testGetSessions



Sign in to add a comment

__main__.SessionHandlingTest.testGetSessions is flaky

Project Member Reported by Findit, Oct 29

Issue description

 Issue 899926  has been merged into this issue.
 Issue 899924  has been merged into this issue.
 Issue 899886  has been merged into this issue.
Unexpected Failures:
* __main__.SessionHandlingTest.testGetSessions
* __main__.SessionHandlingTest.testQuitASessionMoreThanOnce
* __main__.SupportIPv4AndIPv6.testSupportIPv4AndIPv6
* __main__.ZChromeStartRetryCountTest.testChromeStartRetryCount
Owner: johnchen@chromium.org
Status: Assigned (was: Untriaged)
Labels: -Sheriff-Chromium
Disable the test:
https://chromium-review.googlesource.com/c/chromium/src/+/1307594
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 30

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/49c25ee7051736320fc21db6dd686e5c1cd37ce6

commit 49c25ee7051736320fc21db6dd686e5c1cd37ce6
Author: Owen Min <zmin@chromium.org>
Date: Tue Oct 30 14:39:20 2018

Disable flaky chromedriver test

Disable the following test:
SessionHandlingTest.testGetSessions
SessionHandlingTest.testQuitASessionMoreThanOnce
SupportIPv4AndIPv6.testSupportIPv4AndIPv6
ZChromeStartRetryCountTest.testChromeStartRetryCount

Bug: 899919
Change-Id: I4fdebcce8d0784f4a03d740fe75cad489e7cc2f1
Tbr: johnchen@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/1307594
Reviewed-by: Owen Min <zmin@chromium.org>
Commit-Queue: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#603893}
[modify] https://crrev.com/49c25ee7051736320fc21db6dd686e5c1cd37ce6/chrome/test/chromedriver/test/test_expectations

Project Member

Comment 8 by bugdroid1@chromium.org, Oct 30

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b77581ad3c136743e71dbc9c9c9c0e90d12c2687

commit b77581ad3c136743e71dbc9c9c9c0e90d12c2687
Author: John Chen <johnchen@chromium.org>
Date: Tue Oct 30 19:12:44 2018

Revert "Disable flaky chromedriver test"

This reverts commit 49c25ee7051736320fc21db6dd686e5c1cd37ce6.

Reason for revert: The original CL doesn't have the desired effects.

Original change's description:
> Disable flaky chromedriver test
> 
> Disable the following test:
> SessionHandlingTest.testGetSessions
> SessionHandlingTest.testQuitASessionMoreThanOnce
> SupportIPv4AndIPv6.testSupportIPv4AndIPv6
> ZChromeStartRetryCountTest.testChromeStartRetryCount
> 
> Bug: 899919
> Change-Id: I4fdebcce8d0784f4a03d740fe75cad489e7cc2f1
> Tbr: johnchen@chromium.org
> Reviewed-on: https://chromium-review.googlesource.com/c/1307594
> Reviewed-by: Owen Min <zmin@chromium.org>
> Commit-Queue: Owen Min <zmin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#603893}

TBR=zmin@chromium.org,johnchen@chromium.org

Change-Id: Ia4d948632cb8141de5a8214569432e066ab7ab90
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 899919
Reviewed-on: https://chromium-review.googlesource.com/c/1308655
Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: John Chen <johnchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#603981}
[modify] https://crrev.com/b77581ad3c136743e71dbc9c9c9c0e90d12c2687/chrome/test/chromedriver/test/test_expectations

Project Member

Comment 9 by bugdroid1@chromium.org, Oct 30

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d312f641c62bfa1356319558880c235e553512a4

commit d312f641c62bfa1356319558880c235e553512a4
Author: John Chen <johnchen@chromium.org>
Date: Tue Oct 30 20:51:04 2018

[ChromeDriver] Disable flaky testGetSessions

SessionHandlingTest.testGetSessions appears to trigger some race
conditions and is flaky. Disabling it until a fix can be found.

TBR=zmin@chromium.org

Bug: 899919
Change-Id: I3c532559477a3597f20984783eb36acf6355ad7b
Reviewed-on: https://chromium-review.googlesource.com/c/1308657
Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: John Chen <johnchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604006}
[modify] https://crrev.com/d312f641c62bfa1356319558880c235e553512a4/chrome/test/chromedriver/test/run_py_tests.py

SupportIPv4AndIPv6.testSupportIPv4AndIPv6 seems to be timing out.

https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win%207%20Tests%20x64%20%281%29/46024

In particular, testSupportIPv4AndIPv6 seems to sometimes cause all tests run after it to time out.

Sign in to add a comment