Commit Graph

22 Commits

Author SHA1 Message Date
Joe Gregorio
4cb778e367 Fix usage of depot_tools in containers.
depot_tools w/vpython now seems to require the depot_tools dir
to be on PATH, and also python and curl need to be installed.

  * Creates a new base image that has all the build tools we need for
building Skia: skia-build-tools.
  * Moves skia-release to use that new base image.
  * Moves both the skia-release and wasm builds to use fetch and gclient sync.
  * Changes gcr.io/skia-public/emsdk-base:prod to depend on skia-build-tools.


Bug: skia:993053
Change-Id: Ib6218d8c18dd59d43de84dcf063f204cfc8556d0
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/248556
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
2019-10-14 19:38:40 +00:00
Joe Gregorio
1494a7f1ec Revert "cloud-build: set HOME env variable."
This reverts commit b51a0c9a5b.

Reason for revert: Causes Cloud Build to fail: https://pantheon.corp.google.com/cloud-build/builds/933b93f5-1ceb-48d0-9f13-55e8b01b3e44?organizationId=433637338589&project=skia-public

Original change's description:
> cloud-build: set HOME env variable.
> 
> Bug: 993053
> Change-Id: I6069def4a063dbe7a4f8c6fe9cc5046d014e4f3c
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/247046
> Reviewed-by: Eric Boren <borenet@google.com>
> Commit-Queue: Joe Gregorio <jcgregorio@google.com>

TBR=borenet@google.com,jcgregorio@google.com

Change-Id: I311fe7240768ef0bf7015bdf0cd5a0e2aabe6c5c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 993053
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/247051
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-10-08 17:47:42 +00:00
Joe Gregorio
b51a0c9a5b cloud-build: set HOME env variable.
Bug: 993053
Change-Id: I6069def4a063dbe7a4f8c6fe9cc5046d014e4f3c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/247046
Reviewed-by: Eric Boren <borenet@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-10-08 17:00:21 +00:00
Joe Gregorio
710b5e616e Fix HOME directory in Docker containers built under cloudbuild.
The line:
    USER skia

also sets the $HOME environment variable to /home/skia unless
useradd is given another directory to use as the user's home
directory. So HOME was set to /home/skia, but that directory
doesn't exist in subsequent containers.

Bug: chromuim:993053
Change-Id: I3525d7cf11a1c1e5559cf9216c3bf6071a236949
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/244116
Reviewed-by: Ravi Mistry <rmistry@google.com>
Reviewed-by: Robbie Iannucci <iannucci@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-09-26 02:31:35 +00:00
Joe Gregorio
557309bf30 Add env vars to cloudbuild.yaml to enable vpython for depot_tools.
Bug: skia:9348
Change-Id: Ibcdb6448ab5573afb77506f2884093579c2e423f
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/239886
Auto-Submit: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
2019-09-06 16:59:15 +00:00
Joe Gregorio
87b57d0833 Revert "Add HOME env variable to cloudbuild so vpython can move forward."
This reverts commit feb3fc29b3.

Reason for revert: Might have broken the wasm cloudbuild.

Original change's description:
> Add HOME env variable to cloudbuild so vpython can move forward.
> 
> Bug: skia:9348
> Change-Id: Ic9d835f38b028c9a60f07b56f52527bcb440c454
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/236918
> Reviewed-by: Joe Gregorio <jcgregorio@google.com>
> Commit-Queue: Joe Gregorio <jcgregorio@google.com>

TBR=borenet@google.com,jcgregorio@google.com

Change-Id: I678e7027ad5f60a57b38c5d375ba16498b2cfa95
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:9348
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/237037
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-08-24 12:49:26 +00:00
Joe Gregorio
feb3fc29b3 Add HOME env variable to cloudbuild so vpython can move forward.
Bug: skia:9348
Change-Id: Ic9d835f38b028c9a60f07b56f52527bcb440c454
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/236918
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-08-23 21:40:50 +00:00
Joe Gregorio
024a912136 Make debugger-assets part of cloudbuild CI.
Change-Id: I5d5946304e3c20ffad5b0af69d7d43b6847ab724
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/222097
Reviewed-by: Nathaniel Nifong <nifong@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2019-06-19 20:44:40 +00:00
Joe Gregorio
df5b72535d Turn on CI/CD for api.skia.org. (skia)
Bug: skia:
Change-Id: I9f9b39f9903a4fa6476adbfb8c41d7aa7f06c0bc
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/199841
Reviewed-by: Kevin Lubick <kjlubick@google.com>
2019-03-12 18:30:38 +00:00
Kevin Lubick
b6b3188f81 [particles] Turn on CI/CD for particles.skia.org
Add entry to cloudbuild.yaml

Bug: skia:
Change-Id: I649e42f3e4e97e33a673a816ba7815abe1bc08a7
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/200462
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2019-03-12 16:29:42 +00:00
Kevin Lubick
22d6c27f31 Convert skottie's cloud build to use CanvasKit
Bug: skia:
Change-Id: Id550fe3a051e5000bc1254e3a300fb88c8e6a61e
Reviewed-on: https://skia-review.googlesource.com/c/170261
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2018-11-10 12:06:32 +00:00
Kevin Lubick
df3a6eee13 Add JSFiddle to cloudbuild
This is a little more complicated than other builds because we need
the webassembly binaries out fo skia-wasm-release when we do the webpack
step and we can't run docker (e.g. docker run skia-wasm-release cp foo.wasm)
inside of the skia-public/infra:prod image.

To get around this limitation, we add a step that extracts the binaries to
a /workspace/wasm-products folder (which persists between steps) and mount
that folder as a volume for when we make the jsfiddle output. The only tricky
thing is that we must manually mount the /workspace/__jsfiddle_staging folder
as a volume too, or it doesn't persist to the build step.

Something similar will be needed for skottie.skia.org after
https://skia-review.googlesource.com/c/buildbot/+/169700 lands.

Bug: skia:
Change-Id: Iec05fff7231a28ecfa209489e5c75a9621b334d2
Reviewed-on: https://skia-review.googlesource.com/c/169821
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2018-11-08 22:53:25 +00:00
Kevin Lubick
36b85be494 Revert "Autobuild the jsfiddle docker image"
This reverts commit 2969d27760.

Reason for revert: command "docker" not found inside image

Original change's description:
> Autobuild the jsfiddle docker image
> 
> With https://skia-review.googlesource.com/c/skia-public-config/+/169300
> it will auto-deploy too.
> 
> Bug: skia:
> Change-Id: Ic0a186c0e5c44328cc29085a74ac36e4a0c632ac
> Reviewed-on: https://skia-review.googlesource.com/c/169246
> Reviewed-by: Kevin Lubick <kjlubick@google.com>

TBR=jcgregorio@google.com,kjlubick@google.com

Change-Id: Ifad1dbf9cce4b70e42c2a4c70f8fa045becf9aea
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/c/169321
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
2018-11-07 19:55:21 +00:00
Kevin Lubick
2969d27760 Autobuild the jsfiddle docker image
With https://skia-review.googlesource.com/c/skia-public-config/+/169300
it will auto-deploy too.

Bug: skia:
Change-Id: Ic0a186c0e5c44328cc29085a74ac36e4a0c632ac
Reviewed-on: https://skia-review.googlesource.com/c/169246
Reviewed-by: Kevin Lubick <kjlubick@google.com>
2018-11-07 18:33:51 +00:00
Kevin Lubick
96175696de Add skia-wasm-release image.
This should let us pipe continuous builds into jsfiddle.skia.org and
skottie.skia.org

Bug: skia:
Change-Id: I587e1293237d42dfdacae0757f44634f29d6f251
Reviewed-on: https://skia-review.googlesource.com/c/168440
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
2018-11-06 15:45:51 +00:00
Joe Gregorio
e9326bebdf Use new build targets in continuous build of skottie and debugger.
The new targets were added here: https://skia-review.googlesource.com/c/buildbot/+/155820

Bug: skia:
Change-Id: Iac387fb033cf66dff353efb83f52a442b3441539
Reviewed-on: https://skia-review.googlesource.com/155841
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2018-09-20 15:26:18 +00:00
Joe Gregorio
b78becc40c Fix container builder command line.
The gcloud commands have moved.

Bug: skia:
Change-Id: I2af39ccfe0efe121766b7b0e2fe3cdb8cd8c4012
Reviewed-on: https://skia-review.googlesource.com/154040
Auto-Submit: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
2018-09-12 18:44:59 +00:00
Joe Gregorio
a83bb57bfe Use a larger machine for our cloud build.
Bug: skia:
Change-Id: I51ea222fb36f521f34f7a870a225423f8a46c887
Reviewed-on: https://skia-review.googlesource.com/144609
Auto-Submit: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
2018-07-31 20:40:33 +00:00
Joe Gregorio
6407c3d41d Add continuous deploy for skottie and debugger.
Bug: skia:
Change-Id: I90ad3445224a529f7b92951be9f43e37b4a063bd
Reviewed-on: https://skia-review.googlesource.com/142891
Auto-Submit: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
2018-07-23 18:05:24 +00:00
Joe Gregorio
3eff1e9647 cloudbuild.yaml also needs a top level timeout.
Bug: skia:
Change-Id: Idab02c36778feab40bf6750646929d037db7321a
Reviewed-on: https://skia-review.googlesource.com/139168
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2018-07-03 19:02:55 +00:00
Joe Gregorio
28e2423afd Add timeouts to cloudbuild.yaml
Bug: skia:
Change-Id: Ibe95223343bbb9980bca84c336ac3ef162a13c2a
Reviewed-on: https://skia-review.googlesource.com/139164
Auto-Submit: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2018-07-03 18:00:45 +00:00
Joe Gregorio
0725bdb787 Add cloudbuild.yaml file for continous builds of fiddler and skia-release.
Bug: skia:
Change-Id: I713fea23ce88af4dd2a1f28082274535119defee
Reviewed-on: https://skia-review.googlesource.com/139107
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Joe Gregorio <jcgregorio@google.com>
2018-07-03 14:43:43 +00:00