New issue
Advanced search Search tips

Issue 798253 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 798578
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 2
Type: Bug-Regression



Sign in to add a comment

URLLoaderTest.BasicSSL and 2 other(s) in content_unittests failing on chromium.linux/Fuchsia x64

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Jan 2 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of johnylin@google.com

URLLoaderTest.BasicSSL and 2 other(s) in content_unittests failing on chromium.linux/Fuchsia x64

Builders failed on: 
- Fuchsia x64: 
  https://build.chromium.org/p/chromium.linux/builders/Fuchsia%20x64
- Started regression from https://ci.chromium.org/buildbot/chromium.linux/Fuchsia%20x64/2629

There is only one commit in this build but seems not related:
Roll src/third_party/skia/ fc043dcd5..124d5afbf
https://chromium.googlesource.com/chromium/src/+/feee97efc12d9de017d0cb00187469ad119b5711

 
Cc: maxmorin@chromium.org
Components: Infra>Client>Chrome Internals>Network>SSL
Labels: OS-Fuchsia Type-Bug-Regression
Y2K18 bug? I see a "ERROR: Time is before notBefore" in the logs. I'm guessing it's related to how time works on the VMs somehow. Doesn't look like a real issue.
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 2 2018

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

commit 0a57f7db67a683e0811a107c9bfb7c8aebddb2b4
Author: maxmorin <maxmorin@chromium.org>
Date: Tue Jan 02 11:38:34 2018

Disable some SSL tests on Fuchsia

Bug:  798253 
TBR: kinuko
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I98878c3452c00dccaecfd1c73223176f474d782c
Reviewed-on: https://chromium-review.googlesource.com/847434
Reviewed-by: Max Morin <maxmorin@chromium.org>
Commit-Queue: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526461}
[modify] https://crrev.com/0a57f7db67a683e0811a107c9bfb7c8aebddb2b4/content/network/url_loader_unittest.cc

Status: Untriaged (was: Available)
Components: -Internals>Network>SSL Internals>Network>Certificate
Components: Infra>Flakiness
Adding a Flakiness label, as it sounds like from comment #2 the issue is the bots, not the tests
Cc: w...@chromium.org
Components: Internals>PlatformIntegration
Owner: sergeyu@chromium.org
Status: Assigned (was: Untriaged)
Wez, do you think this can be caused by the same RTC driver issue as in  bug 798578 ?

Comment 7 by w...@chromium.org, Jan 3 2018

Re #6: Looking at the content_unittests failures on the FYI bot, they seem to have been occurring since https://ci.chromium.org/buildbot/chromium.fyi/Fuchsia/12631 - that was at ~4.15pm on December 31st 2017, i.e. already 2018 in GMT and beyond.

So yes, sounds like the same cause as  issue 798578 ; I'm not sure why it took longer to start affecting base_unittests, though...


Comment 8 by w...@chromium.org, Jan 3 2018

Mergedinto: 798578
Status: Duplicate (was: Assigned)
Project Member

Comment 9 by bugdroid1@chromium.org, Jan 9 2018

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

commit e0e5c91ded7619a987ae412256c00dc1a92a49fe
Author: Wez <wez@chromium.org>
Date: Tue Jan 09 00:08:47 2018

Revert "Disable some SSL tests on Fuchsia"

This reverts commit 0a57f7db67a683e0811a107c9bfb7c8aebddb2b4.

Reason for revert: These were disabled to work around an RTC driver bug in Fuchsia/x64, which has since been resolved.

Original change's description:
> Disable some SSL tests on Fuchsia
>
> Bug:  798253 
> TBR: kinuko
> Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
> Change-Id: I98878c3452c00dccaecfd1c73223176f474d782c
> Reviewed-on: https://chromium-review.googlesource.com/847434
> Reviewed-by: Max Morin <maxmorin@chromium.org>
> Commit-Queue: Max Morin <maxmorin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#526461}

TBR=kinuko@chromium.org,maxmorin@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  798253 ,  798578 
Change-Id: Ic06c08e6936ee13f1492001b0bb288a540a4db0d
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/854512
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527820}
[modify] https://crrev.com/e0e5c91ded7619a987ae412256c00dc1a92a49fe/content/network/url_loader_unittest.cc

Sign in to add a comment