diff options
| author | Clark Boylan <clark.boylan@gmail.com> | 2022-10-07 13:29:25 -0700 |
|---|---|---|
| committer | Clark Boylan <clark.boylan@gmail.com> | 2022-10-07 13:29:25 -0700 |
| commit | b163a52c2590f2483518a9bd6debd5b03b2157b6 (patch) | |
| tree | fc15347e960fe48737264a2f2cd7d5277426345b | |
| parent | 2593a396d9d880a3a16580de8af3e6a3845b21ce (diff) | |
| download | zuul-b163a52c2590f2483518a9bd6debd5b03b2157b6.tar.gz | |
Only run yarn build when necessary in tools/pip.sh
The Zuul unittest jobs were timing out which prompted me to take a look
at what might be taking extra time. One thing I noticed is that we're
running the yarn build (which runs react-scripts) multiple times when we
really only need to run it once. The reason for that is our check to see
if zuul web has already built is looking for a file that the builds no
longer produce. Update that check to look for a current file and we'll
save a bit of time in our jobs and when running things locally.
Change-Id: Iae3604fbaf072d53895db850cfc989a832b12b27
| -rwxr-xr-x | tools/pip.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/pip.sh b/tools/pip.sh index 65d903a04..6539d5d76 100755 --- a/tools/pip.sh +++ b/tools/pip.sh @@ -39,7 +39,7 @@ then # Avoid writing yarn into package.json. npm install -g --no-package-lock --no-save yarn fi -if [[ ! -f zuul/web/static/status.html ]] +if [[ ! -f zuul/web/static/index.html ]] then mkdir -p zuul/web/static ln -sfn ../zuul/web/static web/build |