New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 904577 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Nov 13
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug
Flaky-Test: TwoClientWalletSyncTest.UpdateCreditCardMetadata



Sign in to add a comment

*/TwoClientWalletSyncTest.UpdateCreditCardMetadata/* is flaky

Project Member Reported by Findit, Nov 12

Issue description


Flaky test: USS/TwoClientWalletSyncTest.UpdateCreditCardMetadata/1
Sample failed build due to flakiness: https://ci.chromium.org/buildbot/chromium.win/Win7%20Tests%20%28dbg%29%281%29/72883
Test output log: https://chromium-swarm.appspot.com/task?id=4122a85d552fc110
Culprit (100.0% confidence): r607282
Analysis: https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVyvQELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKGAWNocm9taXVtLndpbi9XaW43IFRlc3RzIChkYmcpKDEpLzcyODgzL3N5bmNfaW50ZWdyYXRpb25fdGVzdHMvVlZOVEwxUjNiME5zYVdWdWRGZGhiR3hsZEZONWJtTlVaWE4wTGxWd1pHRjBaVU55WldScGRFTmhjbVJOWlhSaFpHRjBZUzh4DAsSE01hc3RlckZsYWtlQW5hbHlzaXMYAQw

Please revert the culprit, or disable the test and find the appropriate owner.

If the culprit 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%20Analyzer%20-%20Wrong%20result%20for%20USS/TwoClientWalletSyncTest.UpdateCreditCardMetadata/1&comment=Link%20to%20Analysis%3A%20https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVyvQELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCKGAWNocm9taXVtLndpbi9XaW43IFRlc3RzIChkYmcpKDEpLzcyODgzL3N5bmNfaW50ZWdyYXRpb25fdGVzdHMvVlZOVEwxUjNiME5zYVdWdWRGZGhiR3hsZEZONWJtTlVaWE4wTGxWd1pHRjBaVU55WldScGRFTmhjbVJOWlhSaFpHRjBZUzh4DAsSE01hc3RlckZsYWtlQW5hbHlzaXMYAQw

Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 12

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

commit 7419fdde22eb141447d7c6192c9d65334bcd774b
Author: Findit <findit-for-me@appspot.gserviceaccount.com>
Date: Mon Nov 12 22:47:50 2018

Revert "[AF] Add integration tests for wallet metadata"

This reverts commit 4cde6da1b6c8534b12dbe8ced765651165f2bce2.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 607282 as the
culprit for flakes in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vNGNkZTZkYTFiNmM4NTM0YjEyZGJlOGNlZDc2NTY1MTE2NWYyYmNlMgw

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.win/Win7%20Tests%20%28dbg%29%281%29/72883

Sample Failed Step: sync_integration_tests

Sample Flaky Test: USS/TwoClientWalletSyncTest.UpdateCreditCardMetadata/1

Original change's description:
> [AF] Add integration tests for wallet metadata
> 
> This CL creates the first integration tests for the autofill wallet
> metadata model type.
> 
> Bug:  894001 
> Change-Id: Ie756264fe6b22d863a6965456ec0d7b627774c23
> Reviewed-on: https://chromium-review.googlesource.com/c/1306357
> Commit-Queue: Jan Krcal <jkrcal@chromium.org>
> Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#607282}

Change-Id: I6021aa4d092acfeb62f635bc04c76b7c11738101
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  894001 ,  904577 
Reviewed-on: https://chromium-review.googlesource.com/c/1332393
Cr-Commit-Position: refs/heads/master@{#607359}
[modify] https://crrev.com/7419fdde22eb141447d7c6192c9d65334bcd774b/chrome/browser/sync/test/integration/autofill_helper.cc
[modify] https://crrev.com/7419fdde22eb141447d7c6192c9d65334bcd774b/chrome/browser/sync/test/integration/autofill_helper.h
[modify] https://crrev.com/7419fdde22eb141447d7c6192c9d65334bcd774b/chrome/browser/sync/test/integration/single_client_wallet_sync_test.cc
[delete] https://crrev.com/5698367bdddad99c17041575f82eab54c200d6a0/chrome/browser/sync/test/integration/two_client_wallet_sync_test.cc
[delete] https://crrev.com/5698367bdddad99c17041575f82eab54c200d6a0/chrome/browser/sync/test/integration/wallet_helper.cc
[delete] https://crrev.com/5698367bdddad99c17041575f82eab54c200d6a0/chrome/browser/sync/test/integration/wallet_helper.h
[modify] https://crrev.com/7419fdde22eb141447d7c6192c9d65334bcd774b/chrome/test/BUILD.gn

Components: Services>Sync
Status: Fixed (was: Untriaged)
A fixed reland is ready: https://chromium-review.googlesource.com/c/chromium/src/+/1332934/4

Sign in to add a comment