Build node addon before running tests (#7884)

Fix a race condition in CI where the tests may be run before the node addon is built.
tensorflow · Jul 31, 2023 · 93e9af4 · 93e9af4
1 parent 5974699
commit 93e9af4
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/scripts/cloudbuild_tfjs_node_expected.yml b/scripts/cloudbuild_tfjs_node_expected.yml
@@ -150,6 +150,7 @@ steps:
       - test-ci
     waitFor:
       - yarn-tfjs-node
+      - build-addon-tfjs-node
       - lint-tfjs-node
       - yarn-common
       - yarn-link-package-build

diff --git a/tfjs-node-gpu/cloudbuild.yml b/tfjs-node-gpu/cloudbuild.yml
@@ -59,7 +59,7 @@ steps:
   entrypoint: 'yarn'
   id: 'test'
   args: ['test-ci']
-  waitFor: ['yarn', 'build-deps', 'lint']
+  waitFor: ['yarn', 'build-addon', 'build-deps', 'lint']
 
 # CPU / GPU package alignment.
 - name: 'gcr.io/learnjs-174218/release'

diff --git a/tfjs-node/cloudbuild.yml b/tfjs-node/cloudbuild.yml
@@ -51,7 +51,7 @@ steps:
   entrypoint: 'yarn'
   id: 'test'
   args: ['test-ci']
-  waitFor: ['yarn', 'build-deps', 'lint']
+  waitFor: ['yarn', 'build-addon', 'build-deps', 'lint']
 
 # CPU / GPU package alignment.
 - name: 'gcr.io/learnjs-174218/release'