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

Issue 888687 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 1
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Mac BoringSSL try jobs aren't working

Project Member Reported by davidben@chromium.org, Sep 24

Issue description

https://ci.chromium.org/p/boringssl/builders/luci.boringssl.try/mac/b8934473600841946416

Not sure where I'm supposed to file this after all the LUCI migrations, so apologies if I picked the wrong spot.

It looks like fetching some of the tools from storage is failing.
 
Components: -Infra>Labs Infra>Client>Chrome
Labels: -Pri-3 Pri-2
CCI folks -- isn't mac tooling now distributed as CIPD package?
Components: -Infra>Client>Chrome Infra>Platform
Labels: -Type-Bug Foundation-Troopers Type-Bug-Regression
Actually, nvm me. The issue here is that vpython failed to initialize for some reaosn:

[E2018-09-24T12:58:26.086042-07:00 60045 0 annotate.go:242] original error: exit status 1
[E2018-09-24T12:58:26.086085-07:00 60045 0 annotate.go:242] 
[E2018-09-24T12:58:26.086097-07:00 60045 0 annotate.go:242] goroutine 1:
[E2018-09-24T12:58:26.086107-07:00 60045 0 annotate.go:242] #0 go.chromium.org/luci/vpython/venv/venv.go:730 - venv.(*Env).installWheels()
[E2018-09-24T12:58:26.086116-07:00 60045 0 annotate.go:242]   reason: failed to install wheels
[E2018-09-24T12:58:26.086141-07:00 60045 0 annotate.go:242] 
[E2018-09-24T12:58:26.086152-07:00 60045 0 annotate.go:242] #1 go.chromium.org/luci/vpython/venv/venv.go:527 - venv.(*Env).createLocked.func1()
[E2018-09-24T12:58:26.086172-07:00 60045 0 annotate.go:242]   reason: failed to install wheels
[E2018-09-24T12:58:26.086183-07:00 60045 0 annotate.go:242] 
[E2018-09-24T12:58:26.086191-07:00 60045 0 annotate.go:242] #2 go.chromium.org/luci/common/system/filesystem/tempdir.go:55 - filesystem.(*TempDir).With()
[E2018-09-24T12:58:26.086201-07:00 60045 0 annotate.go:242] #3 go.chromium.org/luci/vpython/venv/venv.go:103 - venv.withTempDir()
[E2018-09-24T12:58:26.086211-07:00 60045 0 annotate.go:242] #4 go.chromium.org/luci/vpython/venv/venv.go:497 - venv.(*Env).createLocked()
[E2018-09-24T12:58:26.086225-07:00 60045 0 annotate.go:242] #5 go.chromium.org/luci/vpython/venv/venv.go:272 - venv.(*Env).ensure.func1()
[E2018-09-24T12:58:26.086236-07:00 60045 0 annotate.go:242]   reason: failed to create new VirtualEnv
Owner: iannucci@chromium.org
Status: Assigned (was: Untriaged)
Robbie as vpython owner: can you please take a look?
Cc: svaldez@chromium.org agl@chromium.org
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 26

The following revision refers to this bug:
  https://boringssl.googlesource.com/boringssl/+/c93711b1b51ab7bf6039af4572fe54af2cf43cf0

commit c93711b1b51ab7bf6039af4572fe54af2cf43cf0
Author: David Benjamin <davidben@google.com>
Date: Wed Sep 26 17:09:56 2018

Remove Mac try jobs from the CQ.

They're not working right now.

Bug:  chromium:888687 
Change-Id: I723a382d666e24c4f7f35c7b9369341b4bdebf1a
Reviewed-on: https://boringssl-review.googlesource.com/32124
Reviewed-by: Adam Langley <agl@google.com>

[modify] https://crrev.com/c93711b1b51ab7bf6039af4572fe54af2cf43cf0/infra/config/cq.cfg

It appears our Mac CI is also broken, so now we've lost all Mac coverage:
https://ci.chromium.org/p/boringssl/builders/luci.boringssl.ci/mac_small/b8934302890146586784
Filed  issue #890351  for comment #6 as the error appears to be different.
Status: Fixed (was: Assigned)
This issue seems to have gone away at some point.
Project Member

Comment 9 by bugdroid1@chromium.org, Oct 1

The following revision refers to this bug:
  https://boringssl.googlesource.com/boringssl/+/9e97c022e6459035625b842451fd8582fe022dcc

commit 9e97c022e6459035625b842451fd8582fe022dcc
Author: David Benjamin <davidben@google.com>
Date: Mon Oct 01 23:31:45 2018

Bring Mac and iOS builders back to the CQ.

The vpython issue appears to have gone away and hermetic Xcode sorted
out the other problem.

Bug:  chromium:888687 ,  chromium:890351 
Change-Id: I9da893b7f21f0bc7c03e1e70c0e3e86f9720cec1
Reviewed-on: https://boringssl-review.googlesource.com/32304
Reviewed-by: Adam Langley <agl@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>

[modify] https://crrev.com/9e97c022e6459035625b842451fd8582fe022dcc/infra/config/cq.cfg

Project Member

Comment 10 by bugdroid1@chromium.org, Oct 2

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

commit 6bd7dab6856e45a31b5b234b79fcb6ddfc56754f
Author: David Benjamin <davidben@chromium.org>
Date: Tue Oct 02 21:47:18 2018

Roll src/third_party/boringssl/src 13fd62744..ce00828c8

https://boringssl.googlesource.com/boringssl/+log/13fd627449cefbae7576d4b145cb24fac303fc7d..ce00828c89df3d4c40de7d715b1a032eb03c525c

The following commits have Chromium bugs associated:
  9e97c022e Bring Mac and iOS builders back to the CQ.
  7c3ce519e Actually disable RandTest.Fork on iOS.
  52483994c Mostly fix undefined casts around STACK_OF's comparator.
  fb4e2e0f0 Fix undefined casts in sk_*_pop_free and sk_*_deep_copy.
  cbc3e076f Take iOS builders out of the CQ rotation too.
  792c1dc43 Rewrite PEM_X509_INFO_read_bio.
  419144adc Fix undefined function pointer casts in {d2i,i2d}_Foo_{bio,fp}
  217bfd3c9 Fix undefined function pointer casts in IMPLEMENT_PEM_*.

I've also updated the roll_boringssl.py script to print the above message and
the below bug list, so we don't need to manually attach things to rolls.

Bug:  785442 ,  888687 , 890115,  890351 
Change-Id: If08496fcff132e71052a89c60aefb609d28ad3c7
Reviewed-on: https://chromium-review.googlesource.com/c/1257561
Commit-Queue: Steven Valdez <svaldez@chromium.org>
Reviewed-by: Steven Valdez <svaldez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595994}
[modify] https://crrev.com/6bd7dab6856e45a31b5b234b79fcb6ddfc56754f/DEPS
[modify] https://crrev.com/6bd7dab6856e45a31b5b234b79fcb6ddfc56754f/third_party/boringssl/roll_boringssl.py

Sign in to add a comment