"SingleClientSessionsSyncTest.CookieJarMismatch" is flaky |
|||||||||
Issue description"SingleClientSessionsSyncTest.CookieJarMismatch" is flaky. This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label. We have detected 5 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyOQsSBUZsYWtlIi5TaW5nbGVDbGllbnRTZXNzaW9uc1N5bmNUZXN0LkNvb2tpZUphck1pc21hdGNoDA. Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs This flaky test/step was previously tracked in issue 603794 .
,
Nov 9 2016
https://codereview.chromium.org/2489513004/
,
Nov 9 2016
,
Nov 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d89c55240ff7c736536efb3d419a904f7fdab0f2 commit d89c55240ff7c736536efb3d419a904f7fdab0f2 Author: scheib <scheib@chromium.org> Date: Wed Nov 09 03:15:49 2016 Disable SingleClientSessionsSyncTest.CookieJarMismatch Test is flaky on try job runs, as detected by chromium-try-flakes app and reported on the issue. BUG= 663547 TBR=stanisc@chromium.org,zea@chromium.org,skym@chromium.org Review-Url: https://codereview.chromium.org/2489513004 Cr-Commit-Position: refs/heads/master@{#430840} [modify] https://crrev.com/d89c55240ff7c736536efb3d419a904f7fdab0f2/chrome/browser/sync/test/integration/single_client_sessions_sync_test.cc
,
Nov 9 2016
I'm removing the Sheriff-Chromium label from this since the test has been disabled and it looks like we're not seeing any more failures at the moment.
,
Nov 9 2016
,
Nov 9 2016
,
Nov 9 2016
Judging by the comment https://cs.chromium.org/chromium/src/chrome/browser/sync/test/integration/single_client_sessions_sync_test.cc?l=166 it seems this test case relies on behavior of how the signing flow it uses works. I was suspicious of https://codereview.chromium.org/2478173003/ , but appears to have landed at 2016-11-08 20:41:35 UTC, and the first flake we saw is at 2016-11-08 20:08:09 UTC, ~30 minutes earlier. And this is the test's end time, not when it updated source files. I cannot seem to find anything preceding this by a few hours that looks suspicious. Also interesting to note, that this flaking seems to be happening exclusively on windows.
,
Nov 10 2016
Left this test case looping on my Linux machine over night and no failures. It seems like the sync backend initilaizes, then we sign in. This causes a (maybe real, not sure) list accounts to gaia, and we get back basically junk data that looks like: ["gaia.l.a.r",[]] This parses to no accounts in the cookie jar. ProfileSyncService::OnGaiaAccountsInCookieUpdated is called, accounts is empty, and so cookie_jar_mismatch stays true and we don't fail at single_client_sessions_sync_test.cc(170). Sprinkling sleeps all over CookieJarMismatch, at least on Linux, doesn't seem to have any affect. My windows machine is currently blocked on bit9 and all my trybot attempts have had Windows specific compile failures. Hope to have a repro on Windows soon.
,
Nov 10 2016
FYI sync integration tests don't talk to real servers. Also, we should always sign in before initializing the sync backend. That said, it does sound like there's some sort of race condition with our integration with ListAccounts. Grepping for "gaia.l.a.rs" shows that fake_gaia.cc usees it for it's ListAccounts response format.
,
Nov 10 2016
I too searched for that string and found https://cs.chromium.org/chromium/src/google_apis/gaia/fake_gaia.cc?q=%22gaia.l%22&sq=package:chromium&l=68&dr=C , but the bizarre thing was that I could modify that file to have a different list accounts response, but when I ran the integration test, I got the old value. Maybe this file is not re-compiled correctly, I'm really not sure yet. Still trying to understand all the [fake] url fetching logic, and where exactly the intercepting happens.
,
Nov 14 2016
Downgrading to priority 2. This appears to me a timing issue with how the tests sign in, which doesn't reflect real usage. I still intend to fix this test soon, but I intend to work on a few other things first this week, so updating this bug to reflect that.
,
Nov 22 2016
I've been unable to reproduce the original flake, even on Windows. It's unclear if this problem is still oustanding. However, I found a different reason that could cause this test to flake, and a fix is in CL. I'm going to re-enable this case and see what happens on the bots.
,
Nov 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e1f491522b22424a488d6255ec1f066b0af45f8c commit e1f491522b22424a488d6255ec1f066b0af45f8c Author: skym <skym@chromium.org> Date: Tue Nov 22 23:47:17 2016 [Sync] Updated CookieJarMismatch to handle multiple sync cycles during OpenTabAndGetLocalWindows. CookieJarMismatch occasionally failed because two sync cycles would happen for the upper or lower logic block, and then the histogram checks would fail. It is a little bit unclear if the two sync cycles were a result of the implementation of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes() afterwards. Regardless, we now check the histograms in a manner that is resilient to multiple sync cycles that duplicate the histogram emissions. BUG= 663547 Review-Url: https://codereview.chromium.org/2522663005 Cr-Commit-Position: refs/heads/master@{#434024} [modify] https://crrev.com/e1f491522b22424a488d6255ec1f066b0af45f8c/chrome/browser/sync/test/integration/sessions_helper.h [modify] https://crrev.com/e1f491522b22424a488d6255ec1f066b0af45f8c/chrome/browser/sync/test/integration/single_client_sessions_sync_test.cc
,
Nov 29 2016
Doesn't look like SingleClientSessionsSyncTest.CookieJarMismatch has flaked since I re-enabled it 6 days ago. Resolving. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by scheib@chromium.org
, Nov 9 2016Components: Services>Sync